Skip to content

openssl: run make install with -j$NIX_BUILD_CORES#212918

Closed
ghost wants to merge 1 commit intostagingfrom
unknown repository
Closed

openssl: run make install with -j$NIX_BUILD_CORES#212918
ghost wants to merge 1 commit intostagingfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Jan 27, 2023

Description of changes

The kernel building derivation does the same thing.

Maybe it's time for setup.sh to learn enableParallelInstall=true?

Things done
  • Built on platform(s)
    • x86_64-linux
    • powerpc64le-linux
    • aarch64-linux
  • Fits CONTRIBUTING.md.

openssl3 runs `pod2man` as part of `make install`, which means that
the installPhase takes longer than the buildPhase without this.
@ghost ghost marked this pull request as ready for review January 27, 2023 08:08
Comment on lines +157 to +158
# openssl3 runs `pod2man` as part of `make install`, which means that the
# installPhase takes longer than the buildPhase without this:
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Are you sure it's not simply reaching a previously unbuilt doc target or thereof in order to reach the install target?

In that case you should rather make the buildPhase explicitly build that target in addition to the regular targets via buildFlags.

@ofborg ofborg bot added 10.rebuild-darwin-stdenv This PR causes stdenv to rebuild on Darwin and must target a staging branch. 10.rebuild-darwin: 501+ This PR causes many rebuilds on Darwin and should normally target the staging branches. 10.rebuild-darwin: 5001+ This PR causes many rebuilds on Darwin and must target the staging branches. 10.rebuild-linux: 501+ This PR causes many rebuilds on Linux and should normally target the staging branches. 10.rebuild-linux: 5001+ This PR causes many rebuilds on Linux and must target the staging branches. labels Jan 27, 2023
@Artturin
Copy link
Copy Markdown
Member

Superseded by #217568

@Artturin Artturin closed this Feb 22, 2023
@ghost ghost deleted the pr/openssl/make-install-fast2 branch February 23, 2023 04:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

10.rebuild-darwin: 501+ This PR causes many rebuilds on Darwin and should normally target the staging branches. 10.rebuild-darwin: 5001+ This PR causes many rebuilds on Darwin and must target the staging branches. 10.rebuild-darwin-stdenv This PR causes stdenv to rebuild on Darwin and must target a staging branch. 10.rebuild-linux: 501+ This PR causes many rebuilds on Linux and should normally target the staging branches. 10.rebuild-linux: 5001+ This PR causes many rebuilds on Linux and must target the staging branches.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants