Opened 16 years ago
Closed 8 years ago
#15452 closed enhancement (wontfix)
openssl: enable parallel build
Reported by: | ryandesign (Ryan Carsten Schmidt) | Owned by: | larryv (Lawrence Velázquez) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | |
Keywords: | haspatch | Cc: | neverpanic (Clemens Lang) |
Port: | openssl |
Description
I think you should add "use_parallel_build yes
" to the openssl portfile. It seems to work for me, and it should speed up the build for those with more than one CPU core.
Attachments (2)
Change History (11)
comment:1 Changed 16 years ago by ryandesign (Ryan Carsten Schmidt)
Changed 16 years ago by ryandesign (Ryan Carsten Schmidt)
Attachment: | openssl-parallel.diff added |
---|
comment:2 Changed 16 years ago by mww@…
Resolution: | → fixed |
---|---|
Status: | new → closed |
I've marked the port as 'use_parallel_build no' as the added complexity by the patches is not worth the improvements in build time.
comment:3 Changed 16 years ago by (none)
Milestone: | Port Enhancements |
---|
Milestone Port Enhancements deleted
comment:4 Changed 10 years ago by jeremyhu (Jeremy Huddleston Sequoia)
Resolution: | fixed |
---|---|
Status: | closed → reopened |
Reopening. 6 years worth of inefficient building is annoying given that the patch is quite trivial. If the issue is fixed in current sources, please mark it as supporting parallel builds. If the issue still persists with current sources, please send the trivial patch that Ryan provided to upstream developers.
comment:5 Changed 10 years ago by jeremyhu (Jeremy Huddleston Sequoia)
A quick look in current sources' Makefile.org shows that the issue persists. Please just fix it and pass the fix upstream.
comment:6 Changed 10 years ago by jeremyhu (Jeremy Huddleston Sequoia)
Note that according to the mailing list post that Ryan pointed you to, the patch fixes part of the problem, but install_docs and install_sw are missing dependencies on the "all" target. You also need to do:
-install_sw: +install_sw: all ... -install_docs: +install_docs: all
Changed 10 years ago by jeremyhu (Jeremy Huddleston Sequoia)
Attachment: | patch-Makefile.org-parallel.diff added |
---|
patch-Makefile.org-parallel.diff
comment:7 Changed 10 years ago by jeremyhu (Jeremy Huddleston Sequoia)
I suspect this updated patch-Makefile.org-parallel.diff will work to address the remaining issue. I built fine at -j8 with that patch.
comment:8 Changed 8 years ago by mf2k (Frank Schima)
Cc: | neverpanic added |
---|---|
Keywords: | haspatch added |
Owner: | changed from mww@… to larryv |
Port: | openssl added |
Status: | reopened → assigned |
Version: | 1.6.0 |
Assigning to the current maintainers.
comment:9 Changed 8 years ago by neverpanic (Clemens Lang)
Resolution: | → wontfix |
---|---|
Status: | assigned → closed |
OpenSSL still seems to have some issues building in parallel, see #46719, where we decided to close this as wontfix until upstream gets it right.
Actually it looks like 0.9.8h does not build correctly when the parallel build is enabled:
There is a patch shown here which fixes it for 2 jobs at once:
http://www.mail-archive.com/openssl-dev@openssl.org/msg24065.html
But apparently not for 3 jobs at once:
http://www.mail-archive.com/openssl-dev@openssl.org/msg24106.html