-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
MPC Conan 2.0 Compatibility #14770
MPC Conan 2.0 Compatibility #14770
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Hooks produced the following warnings for commit b3d588empc/1.1.0
mpc/1.2.0
|
This comment has been minimized.
This comment has been minimized.
Hooks produced the following warnings for commit 2040aabmpc/1.2.0
mpc/1.1.0
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Co-authored-by: Stella Smith <40411082+StellaSmith@users.noreply.github.com>
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Inject VirtualRunEnv in build scope for native build, otherwise configure step of autotools on Linux & Windows will break if dependencies are shared
Co-authored-by: SpaceIm <30052553+SpaceIm@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* MPC Conan 2.0 Compatibility * Fixed lint errors * Fixed license copying code * Tweaked how win_bash is set * Added new metadata for patch * Try adding VirtrualBuildEnv for v1.x * Bump required_conan_version * Define CL-related flag macro values. * Import is_msvc * Add automake as a tool_requires * Fixed typo * Recognize that recipe lacks Visual Studio support * Formatting tweak * Added blank line to trigger CI build * Use autotools.autoreconf() per review suggestion * Add "m" as system_libs * Add call to fix_apple_shared_install_name Co-authored-by: Stella Smith <40411082+StellaSmith@users.noreply.github.com> * Removed setting of shared/static options per code review * Incorporate @spacelm's review feedback Co-authored-by: SpaceIm <30052553+SpaceIm@users.noreply.github.com> * Per review feedback --------- Co-authored-by: Stella Smith <40411082+StellaSmith@users.noreply.github.com> Co-authored-by: SpaceIm <30052553+SpaceIm@users.noreply.github.com>
Specify library name and version: mpc/1.2.0
Migrated recipe to Conan 2.0. Resolves #14768. Note that these changes are untested on Windows with Conan 2.0 because some of the required dependencies don't build with 2.0 yet