sony / nmos-cpp

An NMOS (Networked Media Open Specifications) Registry and Node in C++ (IS-04, IS-05)
Apache License 2.0
136 stars 79 forks source link

Windows Build #322

Closed emanueledimauro closed 8 months ago

emanueledimauro commented 1 year ago

I used nmos-cpp in linux environment and it works perfectly. However, I tried compiling on windows and had problems with conan (old_conan_cmake_install).

Has anyone compiled for windows yet ? Can I find a repository with the binaries ?

I would be very curious to try !

garethsb commented 1 year ago

It's built on Windows in every GitHub Actions run, and Windows binaries are available in Conan Center Index. What is your problem with conan? Note the recipe is limited to not work with Conan 2.0 yet, in case that's the issue!

emanueledimauro commented 1 year ago

I have this message :

ERROR: Error while parsing [options] in conanfile.txt
[cmake] CMake Error at build/conan.cmake:540 (message):
[cmake]   Conan install failed='1'

But I'm searching just a compiled .exe binary files for discovery IS-04, for me is enough but I can't find them on the web.

Would you have a link where I can download them for OS Windows? Thank you very much, Emanuele

garethsb commented 1 year ago

Would you have a link where I can download them for OS Windows?

Last time I checked, these instructions worked to get Conan to install a prebuilt binary: https://github.com/sony/nmos-cpp/issues/283#issuecomment-1302091773

garethsb commented 1 year ago

I have this message :

ERROR: Error while parsing [options] in conanfile.txt

I wonder if this means you have Conan 2.0, as I believe the syntax of options changed slightly. We haven't quite completed the work to support Conan 2.0 so that's why we recommend pip install conan=~1.47 for now.

emanueledimauro commented 1 year ago

Thanks, but 1.47 is not enough, I have this message :

ERROR: nasm/2.15.05: Cannot load recipe.
Error loading conanfile at 'C:\Users...export\conanfile.py': 
Current Conan version (1.53.0) does not satisfy the defined one (>=1.55.0).

I'll try with 1.55 !

garethsb commented 1 year ago

Sorry, yes - the nmos-cpp recipe itself only needs 1.47 or higher, but recipes for some of its dependencies may have been updated to require a higher version. Using conan=~1.47 means it should install the highest version >= 1.47 < 2.0, but you need to add --upgrade if you already have one in this range to accommodate the newer dependency requirements. The GitHub Action always uses the most recent 1.x version. Conan 1.59 is what I'm using locally on Windows and Linux.

emanueledimauro commented 1 year ago

It's ok ! Thank you very much for helping me !

lo-simon commented 8 months ago

Resolved