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

[ci] Honor default profile according to the system #28

Merged
merged 3 commits into from
Jul 25, 2024

Conversation

uilianries
Copy link
Contributor

This PR is relate to the issue conan-io/conan-center-index#24313

Conan 2.x uses 2 profiles always, the build profile that reflects the system, and host profile which should match where the binary will be used.

The Mac machines provided in Github Actions are based on ARM, which means, the current profile configuration does not follow what is in the system.

The Conan profile detect command is capable to list what compiler is installed by default and the same can be used as build profile without risk.

When building to x86_64 on Mac, the same should be configured in the host profile, and can be reflected as -pr:h arch=x86_64.

This PR removes the current profiles and use what is available in the system by default. Plus, honor the host CPU and update the Boost version to 1.85.0

I running a fork CI with this current new change: https://github.com/uilianries/judy/actions/runs/10078669655

Signed-off-by: Uilian Ries <[email protected]>
Signed-off-by: Uilian Ries <[email protected]>
@denismakogon
Copy link
Owner

@uilianries thx for this PR. I approved CI runs, let's see how it'll go.

@denismakogon denismakogon merged commit 0e1e6a2 into denismakogon:main Jul 25, 2024
3 checks passed
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.

2 participants