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

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

Closed
wants to merge 1 commit into from
Closed

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

wants to merge 1 commit into from

Conversation

ghost
Copy link

@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
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.

@Artturin
Copy link
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants