Skip to content
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

mfast: conan v2 support & fix CMakeDeps targets #15365

Merged
merged 2 commits into from
Jan 28, 2023

Conversation

SpaceIm
Copy link
Contributor

@SpaceIm SpaceIm commented Jan 19, 2023

Specify library name and version: lib/1.0


@conan-center-bot

This comment has been minimized.

@github-actions
Copy link
Contributor

Hooks produced the following warnings for commit bdf3105
mfast/1.2.2
post_package(): WARN: [MISSING SYSTEM LIBS (KB-H043)] Library './lib/libmfast_coder.so' links to system library 'm' but it is not in cpp_info.system_libs.
post_package(): WARN: [MISSING SYSTEM LIBS (KB-H043)] Library './lib/libmfast_xml_parser.so' links to system library 'm' but it is not in cpp_info.system_libs.
post_package(): WARN: [MISSING SYSTEM LIBS (KB-H043)] Library './lib/libmfast.so' links to system library 'm' but it is not in cpp_info.system_libs.
post_package(): WARN: [MISSING SYSTEM LIBS (KB-H043)] Library './lib/libmfast_json.so' links to system library 'm' but it is not in cpp_info.system_libs.
mfast/1.2.1
post_package(): WARN: [MISSING SYSTEM LIBS (KB-H043)] Library './lib/libmfast_coder.so' links to system library 'm' but it is not in cpp_info.system_libs.
post_package(): WARN: [MISSING SYSTEM LIBS (KB-H043)] Library './lib/libmfast_xml_parser.so' links to system library 'm' but it is not in cpp_info.system_libs.
post_package(): WARN: [MISSING SYSTEM LIBS (KB-H043)] Library './lib/libmfast.so' links to system library 'm' but it is not in cpp_info.system_libs.
post_package(): WARN: [MISSING SYSTEM LIBS (KB-H043)] Library './lib/libmfast_json.so' links to system library 'm' but it is not in cpp_info.system_libs.

StellaSmith
StellaSmith previously approved these changes Jan 19, 2023
recipes/mfast/all/conanfile.py Show resolved Hide resolved
jwillikers
jwillikers previously approved these changes Jan 25, 2023
Co-authored-by: Stella Smith <40411082+StellaSmith@users.noreply.github.com>
@prince-chrismc prince-chrismc dismissed stale reviews from jwillikers and StellaSmith via 48e7f9d January 28, 2023 05:28
@conan-center-bot
Copy link
Collaborator

Conan v1 pipeline ✔️

All green in build 2 (48e7f9d91c861e50c709d0589b4c5d697f938bca):

  • mfast/1.2.1@:
    All packages built successfully! (All logs)

  • mfast/1.2.2@:
    All packages built successfully! (All logs)


Conan v2 pipeline (informative, not required for merge) ❌

Note: Conan v2 builds are informative and they are not required for the PR to be merged.

The v2 pipeline failed. Please, review the errors and note this will be required for pull requests to be merged in the near future.

See details:

Failure in build 2 (48e7f9d91c861e50c709d0589b4c5d697f938bca):

  • mfast/1.2.2@:
    Error running command conan graph info --requires mfast/1.2.2@#c6f6387c9b99780f0ee05e25f99d0f39 -f json -pr:h /home/conan/w/prod-v2_cci_PR-15365/2/831f4208-083d-485c-81e0-b44951c3091e/profile_windows_192_release_dynamic_msvc_release_64.-shared-True.txt -pr:b /home/conan/w/prod-v2_cci_PR-15365/2/831f4208-083d-485c-81e0-b44951c3091e/profile_windows_192_release_dynamic_msvc_release_64.-shared-True.txt:

    -------- Input profiles --------
    Profile host:
    [settings]
    arch=x86_64
    build_type=Release
    compiler=msvc
    compiler.runtime=dynamic
    compiler.runtime_type=Release
    compiler.version=192
    os=Windows
    [options]
    */*:shared=True
    
    Profile build:
    [settings]
    arch=x86_64
    build_type=Release
    compiler=msvc
    compiler.runtime=dynamic
    compiler.runtime_type=Release
    compiler.version=192
    os=Windows
    [options]
    */*:shared=True
    
    
    -------- Computing dependency graph --------
    ERROR: tinyxml2/9.0.0: Error in configure() method, line 39
    	del self.options.fPIC
    	ConanException: option 'fPIC' doesn't exist
    Possible options are ['shared']
    
  • mfast/1.2.1@:
    Error running command conan graph info --requires mfast/1.2.1@#35d78d43196ff2a89b73eaec86c40674 -f json -pr:h /home/conan/w/prod-v2_cci_PR-15365/2/ee55cfef-8169-436a-bf2f-a10f6285f3e4/profile_windows_192_release_dynamic_msvc_release_64.-shared-True.txt -pr:b /home/conan/w/prod-v2_cci_PR-15365/2/ee55cfef-8169-436a-bf2f-a10f6285f3e4/profile_windows_192_release_dynamic_msvc_release_64.-shared-True.txt:

    -------- Input profiles --------
    Profile host:
    [settings]
    arch=x86_64
    build_type=Release
    compiler=msvc
    compiler.runtime=dynamic
    compiler.runtime_type=Release
    compiler.version=192
    os=Windows
    [options]
    */*:shared=True
    
    Profile build:
    [settings]
    arch=x86_64
    build_type=Release
    compiler=msvc
    compiler.runtime=dynamic
    compiler.runtime_type=Release
    compiler.version=192
    os=Windows
    [options]
    */*:shared=True
    
    
    -------- Computing dependency graph --------
    ERROR: tinyxml2/9.0.0: Error in configure() method, line 39
    	del self.options.fPIC
    	ConanException: option 'fPIC' doesn't exist
    Possible options are ['shared']
    

Note: To save resources, CI tries to finish as soon as an error is found. For this reason you might find that not all the references have been launched or not all the configurations for a given reference. Also, take into account that we cannot guarantee the order of execution as it depends on CI workload and workers availability.

@conan-center-bot conan-center-bot merged commit 231c8e6 into conan-io:master Jan 28, 2023
@SpaceIm SpaceIm deleted the mfast-conan-v2 branch January 28, 2023 14:44
StellaSmith added a commit to StellaSmith/conan-center-index that referenced this pull request Feb 2, 2023
* conan v2 support

* Update recipes/mfast/all/conanfile.py

Co-authored-by: Stella Smith <40411082+StellaSmith@users.noreply.github.com>

---------

Co-authored-by: Chris Mc <prince.chrismc@gmail.com>
Co-authored-by: Stella Smith <40411082+StellaSmith@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants