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

cyclonedds: lowercase systemlibs + modernize more + fix CMake config & pkgconfig file names #15437

Merged

Conversation

SpaceIm
Copy link
Contributor

@SpaceIm SpaceIm commented Jan 23, 2023

for conan-io/hooks#480

CMake config & pkgconfig file names are based on PROJECT_NAME which is CycloneDDS: https://github.com/eclipse-cyclonedds/cyclonedds/blob/0.10.2/CMakeLists.txt#L13
CMake config file name: https://github.com/eclipse-cyclonedds/cyclonedds/blob/0.10.2/CMakeLists.txt#L299-L303
pkgconfig file name: https://github.com/eclipse-cyclonedds/cyclonedds/blob/0.10.2/CMakeLists.txt#L314-L317


- fix config file name
- fix pkgconfig file name
- bump iceoryx & openssl
- require cmake only if recent version not available on build machine
- more elegant removal of vc runtime files
@SpaceIm SpaceIm force-pushed the fix/cyclonedds-lowercase-systemlibs branch from 7ac5b78 to f594a04 Compare January 23, 2023 18:55
@conan-center-bot

This comment has been minimized.

@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 b78498a
cyclonedds/0.10.2
post_package(): WARN: [MISSING SYSTEM LIBS (KB-H043)] Library './lib/libddsc.so' links to system library 'dl' but it is not in cpp_info.system_libs.

@conan-center-bot
Copy link
Collaborator

Conan v1 pipeline ✔️

All green in build 8 (52d5caaa5c3b26a0962e5c75d0d33f02553a07a4):

  • cyclonedds/0.10.2@:
    All packages built successfully! (All logs)

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 conan-center-bot merged commit 14a155f into conan-io:master Jan 31, 2023
@SpaceIm SpaceIm deleted the fix/cyclonedds-lowercase-systemlibs branch January 31, 2023 10:45
StellaSmith pushed a commit to StellaSmith/conan-center-index that referenced this pull request Feb 2, 2023
…fix CMake config & pkgconfig file names

* lowercase system libs

* modernize more & few fixes

- fix config file name
- fix pkgconfig file name
- bump iceoryx & openssl
- require cmake only if recent version not available on build machine
- more elegant removal of vc runtime files

* lowercase sha256

* also remove pdb files

* add libdl to 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.

None yet

4 participants