-
Notifications
You must be signed in to change notification settings - Fork 80
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Dependency bumps #290
Comments
@lo-simon I created PRs for:
but they're all failing the Conan 2.0 migration linter checks, so need some help with that... |
All mentioned version ups were included in #291 apart from Conan recipes for json-schema-validator, websocketpp, and cpprestsdk (after a couple of follow-up conan-center-index PRs and #297, #299 to work around resulting issues with these and the recently changed avahi recipe) are now migrated to v2. I've got a PR in progress for the conan-center-index nmos-cpp recipe, currently based off tip of master, so cci.20221116. |
In order to spot incompatibilities early, we have generally tried to keep the OSS master branch close to latest versions of build dependencies.
Right now, there are quite a few things we could try bumping...
In nmos-cpp GitHub Actions workflow
In Development/conanfile.txt
In Development/cmake/NmosCppConan.cmake
CONAN_VERSION_MIN
1.50.0CONAN_VERSION_CUR
1.53.0In Development/cmake/NmosCppDependencies.cmake
BOOST_VERSION_CUR
1.80.0In Development/third_party/nlohmann
In Documents/Dependencies.md
I suggest we leave Catch as the really old v1.10.0 for now since that's just a test dependency.
I suggest we remove Sandbox/conanfile.py.
In nmos-cpp recipe on Conan Center
required_conan_version
>=1.47 or >=1.50 (maybe??)But before updating that recipe we need...
In json-schema-validator on Conan Center:
In websocketpp recipe:
In cpprestsdk recipe:
The text was updated successfully, but these errors were encountered: