Skip to content

Conversation

phbasler
Copy link
Contributor

@phbasler phbasler commented Jun 3, 2024

Specify library name and version: cmaes/0.10.0

Add new package cmaes according to issue #24201
Fixes #24201

I am not the author of cmaes, but according to the authors in CMA-ES/libcmaes#233 I can provide this recipe.


Maintainers changes

  • Dropped MSVC support. Upstream still has not accepted the patch. Originally, the project did not support MSVC and is delaying this PR. In case of accepting and merging, we can have a new PR in the future.
  • Fixed both the generated CMake file and pkgconf (.pc) file name to follow libcmaes.

ericLemanissier and others added 7 commits May 30, 2024 16:21
* Protobug mutator compatible with Conan 2.x

Signed-off-by: Uilian Ries <uilianries@gmail.com>

* Add version 1.3

Signed-off-by: Uilian Ries <uilianries@gmail.com>

* Enforce protobuf link

Signed-off-by: Uilian Ries <uilianries@gmail.com>

* update abseil usage

Signed-off-by: Uilian Ries <uilianries@gmail.com>

* consume all libs in tests

Signed-off-by: Uilian Ries <uilianries@gmail.com>

* define cxx language in cmakelists

Signed-off-by: Uilian Ries <uilianries@gmail.com>

* static-library on Windows

Signed-off-by: Uilian Ries <uilianries@gmail.com>

* Exclude examples from CMakeLists.txt

The upstream uses EXCLUDE_FROM_ALL CMake feature to avoid building
examples by default, however, CMake still parses the example cmake
file and requires more external dependencies like expat.

Signed-off-by: Uilian Ries <uilianries@gmail.com>

* Do not build shared on Windows

Signed-off-by: Uilian Ries <uilianries@gmail.com>

---------

Signed-off-by: Uilian Ries <uilianries@gmail.com>
Signed-off-by: Uilian Ries <uilianries@gmail.com>
@CLAassistant
Copy link

CLAassistant commented Jun 3, 2024

CLA assistant check
All committers have signed the CLA.

@conan-center-bot

This comment has been minimized.

2 similar comments
@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@uilianries
Copy link
Member

@phbasler Thank you for your PR! Please, take a look in you emails/accounts used in this PR: #24202 (comment)

The CLA bot listed 2 emails and it will not approve if both agrees to the CLA.

@phbasler
Copy link
Contributor Author

phbasler commented Jun 3, 2024

@uilianries
Thanks.
The first commit was signed with an old e-mail adress which is not valid any longer.
I will have a look at it on how to change the assigned e-mail adress. Otherwise I will just redo the PR with the correct e-mail

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot
Copy link
Contributor

Conan v1 pipeline ✔️

All green in build 29 (1434ba5995303bf9f13c109c6553277451cc2830):

  • cmaes/0.10.0:
    All packages built successfully! (All logs)

Conan v2 pipeline ✔️

Note: Conan v2 builds are now mandatory. Please read our discussion about it.

All green in build 29 (1434ba5995303bf9f13c109c6553277451cc2830):

  • cmaes/0.10.0:
    All packages built successfully! (All logs)

@phbasler
Copy link
Contributor Author

@AbrilRBS why did you remove the openmp option? Should the package not be used anymore?

Copy link
Member

@AbrilRBS AbrilRBS left a comment

Choose a reason for hiding this comment

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

Hi! Thanks for taking the time to create the PR, sorry that we didn't get back to you on a timely manner. I've left some comments on the PR, and made a cleanup now that Conan 1 support is no longer needed :)

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 4a38212..dccfea4 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
Copy link
Member

Choose a reason for hiding this comment

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

I see this comes from CMA-ES/libcmaes#234, do you have any insight into wether it will be merged eventually?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I hope so. I will trigger @beniz again. If there is a new version with the PR, then I can adjust the conanfile here to not include this patch for the new version

@AbrilRBS
Copy link
Member

AbrilRBS commented Nov 22, 2024

@AbrilRBS why did you remove the openmp option? Should the package not be used anymore?

Not that is should not be used per-se, it's more that its funcionality currently does not align with our expectations on CCI

@phbasler
Copy link
Contributor Author

phbasler commented Dec 8, 2024

@AbrilRBS Just a little bump. Do you need anything for the last two points?

@phbasler phbasler requested a review from AbrilRBS January 13, 2025 21:34
@phbasler
Copy link
Contributor Author

@AbrilRBS @danimtb Anything to help with the review process?

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.

@phbasler Thank you for your time and effort in pushing this PR 😄

Just like @AbrilRBS, I'm concerned about these patches listed in this PR: We try to keep as close as possible to the upstream in terms of code to reflect the exact same behavior from there. We can accept patches, but usually as a last resort, we prefer to push a newer version with those fixes instead, so we do not need to "maintain" patches in the repository. You can obtain more information in policy about patching section. Also, when patching, please keep it as minimal as possible and well referenced from where it came, including using patch_source in the conandata.yml if possible, it helps all the community in terms of security and reliability.

I also would keep this recipe as simple as possible, avoiding the option surrog. First, because the CI will not validate that option as False, so we need to build locally to check it. Second, because it can be configured via the Conan config tools.cmake.cmaketoolchain:extra_variables when needed from consumer side.

Co-authored-by: Uilian Ries <uilianries@gmail.com>
@AbrilRBS AbrilRBS added the blocked Affected by an external issue and waiting until it is solved label Jun 18, 2025
@AbrilRBS AbrilRBS removed the blocked Affected by an external issue and waiting until it is solved label Jun 24, 2025
@AbrilRBS AbrilRBS requested a review from uilianries June 24, 2025 16:22
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.

The linter accuses of using a bad URL for downloading, because it's a tag, not a proper release package. However, the upstream has no distributed release: https://github.com/CMA-ES/libcmaes/releases

@phbasler
Copy link
Contributor Author

phbasler commented Jul 1, 2025

@uilianries Thanks for adjusting this.
I will open a new PR once the PR on upstream is done.

@jcar87 jcar87 merged commit 6c4ac8a into conan-io:master Jul 9, 2025
8 checks passed
@phbasler phbasler deleted the packages/cmaes branch July 29, 2025 21:16
@phbasler phbasler mentioned this pull request Jul 29, 2025
3 tasks
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.

[request] cmaes/0.10.0
10 participants