Skip to content

libre: use cmake for configurePhase#218518

Merged
7c6f434c merged 1 commit intomasterfrom
unknown repository
Mar 6, 2023
Merged

libre: use cmake for configurePhase#218518
7c6f434c merged 1 commit intomasterfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@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.
@ofborg ofborg bot requested review from 7c6f434c and elohmeier February 26, 2023 23:17
@ofborg ofborg bot added 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. labels Feb 26, 2023
@wegank
Copy link
Copy Markdown
Member

wegank commented Feb 27, 2023

What about removing makeFlags?

@7c6f434c 7c6f434c merged commit 5a65a41 into NixOS:master Mar 6, 2023
@ghost
Copy link
Copy Markdown
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
Copy Markdown
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

Labels

10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants