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

libre: use cmake for configurePhase #218518

Merged
merged 1 commit into from Mar 6, 2023
Merged

libre: use cmake for configurePhase #218518

merged 1 commit into from Mar 6, 2023

Conversation

ghost
Copy link

@ghost ghost commented Feb 26, 2023

Description of changes

Although libre is a cmake-based project, our expression for this library does not currently use cmake for the configurePhase. This commit causes it to do so.

This is necessary to get downstream libre and baresip to cross-compile.

Things done
  • Built on platform(s)
    • aarch64-linux (cross from x86_64-linux)

Although libre is a cmake-based project, our expression for this
library does not currently use cmake for the configurePhase.  This
commit causes it to do so.

This is necessary to get downstream libre and baresip to cross-compile.
@wegank
Copy link
Member

wegank commented Feb 27, 2023

What about removing makeFlags?

@7c6f434c 7c6f434c merged commit 5a65a41 into NixOS:master Mar 6, 2023
@ghost
Copy link
Author

ghost commented Mar 7, 2023

What about removing makeFlags?

cmake is a configurePhase alternative; the buildPhase still uses makeFlags.

@ghost ghost deleted the pr/libre/cross branch March 7, 2023 10:31
@wegank
Copy link
Member

wegank commented Mar 7, 2023

I wonder if the flags are automatically determined by cmake, but I guess it doesn't really matter.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants