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

Do not install man-pages during openssl-static build to speed up things. #391

Merged
merged 1 commit into from Sep 18, 2018

Conversation

normanmaurer
Copy link
Member

Motivation:

We do not care about the openssl manpages so there is no need to install them during the build.

Modifications:

Use make install_sw which will skip manpage installation.

Result:

Faster build

Motivation:

We do not care about the openssl manpages so there is no need to install them during the build.

Modifications:

Use make install_sw which will skip manpage installation.

Result:

Faster build
Copy link
Member

@carl-mastrangelo carl-mastrangelo left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@ejona86 ejona86 left a comment

Choose a reason for hiding this comment

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

How are you going to know how to use OpenSSL now? 😄

@normanmaurer
Copy link
Member Author

@ejona86 its all in my head 🤣

@normanmaurer normanmaurer merged commit e75534b into master Sep 18, 2018
fzakaria pushed a commit to fzakaria/netty-tcnative that referenced this pull request Feb 4, 2019
…gs. (netty#391)

Motivation:

We do not care about the openssl manpages so there is no need to install them during the build.

Modifications:

Use make install_sw which will skip manpage installation.

Result:

Faster build
@normanmaurer normanmaurer deleted the no_manpages branch February 16, 2019 01:51
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.

None yet

3 participants