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

amqp-cpp: conan v2 support #15445

Merged
merged 2 commits into from
Feb 2, 2023
Merged

Conversation

SpaceIm
Copy link
Contributor

@SpaceIm SpaceIm commented Jan 24, 2023

Specify library name and version: lib/1.0


@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@github-actions
Copy link
Contributor

Hooks produced the following warnings for commit 19dae3a
amqp-cpp/4.3.16
post_package(): WARN: [MISSING SYSTEM LIBS (KB-H043)] Library './lib/libamqpcpp.so' links to system library 'm' but it is not in cpp_info.system_libs.
post_package(): WARN: [MISSING SYSTEM LIBS (KB-H043)] Library '.\bin\amqpcpp.dll' links to system library 'ws2_32' but it is not in cpp_info.system_libs.
amqp-cpp/4.3.18
post_package(): WARN: [MISSING SYSTEM LIBS (KB-H043)] Library './lib/libamqpcpp.so' links to system library 'm' but it is not in cpp_info.system_libs.
post_package(): WARN: [MISSING SYSTEM LIBS (KB-H043)] Library '.\bin\amqpcpp.dll' links to system library 'ws2_32' but it is not in cpp_info.system_libs.
amqp-cpp/4.3.10
post_package(): WARN: [MISSING SYSTEM LIBS (KB-H043)] Library './lib/libamqpcpp.so' links to system library 'm' but it is not in cpp_info.system_libs.
post_package(): WARN: [MISSING SYSTEM LIBS (KB-H043)] Library '.\bin\amqpcpp.dll' links to system library 'ws2_32' but it is not in cpp_info.system_libs.
amqp-cpp/4.3.11
post_package(): WARN: [MISSING SYSTEM LIBS (KB-H043)] Library './lib/libamqpcpp.so' links to system library 'm' but it is not in cpp_info.system_libs.
post_package(): WARN: [MISSING SYSTEM LIBS (KB-H043)] Library '.\bin\amqpcpp.dll' links to system library 'ws2_32' but it is not in cpp_info.system_libs.
amqp-cpp/4.1.7
post_package(): WARN: [MISSING SYSTEM LIBS (KB-H043)] Library './lib/libamqpcpp.so' links to system library 'm' but it is not in cpp_info.system_libs.
post_package(): WARN: [MISSING SYSTEM LIBS (KB-H043)] Library '.\bin\amqpcpp.dll' links to system library 'ws2_32' but it is not in cpp_info.system_libs.
amqp-cpp/4.1.6
post_package(): WARN: [MISSING SYSTEM LIBS (KB-H043)] Library './lib/libamqpcpp.so' links to system library 'm' but it is not in cpp_info.system_libs.
post_package(): WARN: [MISSING SYSTEM LIBS (KB-H043)] Library '.\bin\amqpcpp.dll' links to system library 'ws2_32' but it is not in cpp_info.system_libs.
amqp-cpp/4.2.1
post_package(): WARN: [MISSING SYSTEM LIBS (KB-H043)] Library './lib/libamqpcpp.so' links to system library 'm' but it is not in cpp_info.system_libs.
post_package(): WARN: [MISSING SYSTEM LIBS (KB-H043)] Library '.\bin\amqpcpp.dll' links to system library 'ws2_32' but it is not in cpp_info.system_libs.
amqp-cpp/4.1.5
post_package(): WARN: [MISSING SYSTEM LIBS (KB-H043)] Library './lib/libamqpcpp.so' links to system library 'm' but it is not in cpp_info.system_libs.
post_package(): WARN: [MISSING SYSTEM LIBS (KB-H043)] Library '.\bin\amqpcpp.dll' links to system library 'ws2_32' but it is not in cpp_info.system_libs.

@conan-center-bot

This comment has been minimized.

Copy link
Member

@uilianries uilianries left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@conan-center-bot

This comment has been minimized.

@SpaceIm
Copy link
Contributor Author

SpaceIm commented Feb 1, 2023

This PR has been triggered again because there are existing changes in master branch that are related to these same references resulting in different recipe-revisions. The build is being triggered on the PR context in case some additional fix is needed after merging those changes.

@danimtb Is there not a bug or a lack or robustness in the process testing whether a recipe has been updated in master branch between last build of a PR and when bot wants to merge this PR? It's not the first time that I see these false positive leading to unecessary rebuild, usually when c3i is struggling.

@SpaceIm SpaceIm closed this Feb 1, 2023
@SpaceIm SpaceIm reopened this Feb 1, 2023
@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot
Copy link
Collaborator

Conan v1 pipeline ✔️

All green in build 7 (6e10d4046f3ddff2f3e487dce95909811abe7586):

  • amqp-cpp/4.1.5@:
    All packages built successfully! (All logs)

  • amqp-cpp/4.2.1@:
    All packages built successfully! (All logs)

  • amqp-cpp/4.3.18@:
    All packages built successfully! (All logs)

  • amqp-cpp/4.3.16@:
    All packages built successfully! (All logs)

  • amqp-cpp/4.3.11@:
    All packages built successfully! (All logs)

  • amqp-cpp/4.3.10@:
    All packages built successfully! (All logs)

  • amqp-cpp/4.1.7@:
    All packages built successfully! (All logs)

  • amqp-cpp/4.1.6@:
    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 6 (6e10d4046f3ddff2f3e487dce95909811abe7586):

  • amqp-cpp/4.3.18@:
    Didn't run or was cancelled before finishing

  • amqp-cpp/4.3.11@:
    CI failed to create some packages (All logs)

    Logs for packageID 22327195a9601ac105bba082d4ad84cced4e3d51:
    [settings]
    arch=x86_64
    build_type=Release
    compiler=apple-clang
    compiler.libcxx=libc++
    compiler.version=13
    os=Macos
    [options]
    */*:shared=True
    
    [...]
    no_cms=False
    no_comp=False
    no_ct=False
    no_deprecated=False
    no_des=False
    no_dgram=False
    no_dh=False
    no_dsa=False
    no_dso=False
    no_ec=False
    no_ecdh=False
    no_ecdsa=False
    no_engine=False
    no_filenames=False
    no_gost=False
    no_hmac=False
    no_idea=False
    no_md4=False
    no_md5=False
    no_mdc2=False
    no_ocsp=False
    no_pinshared=False
    no_rc2=False
    no_rfc3779=False
    no_rmd160=False
    no_rsa=False
    no_seed=False
    no_sha=False
    no_sm2=False
    no_sm3=False
    no_sm4=False
    no_sock=False
    no_srp=False
    no_srtp=False
    no_sse2=False
    no_ssl=False
    no_ssl3=False
    no_stdio=False
    no_tests=False
    no_threads=False
    no_tls1=False
    no_ts=False
    no_whirlpool=False
    shared=True
    
    ERROR: Missing prebuilt package for 'openssl/1.1.1s'
    Use 'conan list packages openssl/1.1.1s --format=html -r=remote > table.html' and open the table.html file to see available packages
    Or try to build locally from sources with '--build=openssl/1.1.1s'
    
    More Info at 'https://docs.conan.io/en/latest/faq/troubleshooting.html#error-missing-prebuilt-package'
    
  • amqp-cpp/4.1.5@:
    Didn't run or was cancelled before finishing

  • amqp-cpp/4.3.16@:
    Didn't run or was cancelled before finishing

  • amqp-cpp/4.1.6@:
    Didn't run or was cancelled before finishing

  • amqp-cpp/4.3.10@:
    Didn't run or was cancelled before finishing

  • amqp-cpp/4.2.1@:
    Didn't run or was cancelled before finishing

  • amqp-cpp/4.1.7@:
    Didn't run or was cancelled before finishing


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 f093c9b into conan-io:master Feb 2, 2023
@SpaceIm SpaceIm deleted the amqp-cpp-conan-v2 branch February 2, 2023 06:45
StellaSmith pushed a commit to StellaSmith/conan-center-index that referenced this pull request Feb 2, 2023
* conan v2 support

* add missing system libs
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.

4 participants