Summary: | dev-libs/xalan-c fails to build with parallel make | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Diego Elio Pettenò (RETIRED) <flameeyes> |
Component: | New packages | Assignee: | C++ Team [disbanded] <cpp+disabled> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | Martin.vGagern, torsten |
Priority: | High | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: |
Build log
Have objects depend on locale Combined parallel-build.patch |
Description
Diego Elio Pettenò (RETIRED)
2009-12-13 16:53:15 UTC
Created attachment 212900 [details]
Build log
Hit the same thing. Seems to be a parallel build issue. Looking at $S/src/xalanc/Utils/Makefile.in I see three different blocks of commands generating LocalMsgIndex.hpp and moving it to the include dir. I am quite surprised why they didn't factor that part into a separate rule. In any case, I consider the Makefile to be broken enough that upstream should deal with it, and it in my opinion be ok for Gentoo to simply do "emake -j1" to work around the problem. Reported upstream as https://issues.apache.org/jira/browse/XALANC-696 . And I found out that the Makefile to blame isn't the one in the Utils subdir, but the one in the xalanc dir instead. A patch should be possible, will try to write one. Created attachment 219093 [details, diff]
Have objects depend on locale
This patch, together with 1.11.0_pre797991-parallel-build.patch already present in the files dir, seems to solve the issue. They could be simply combined, but as this patch here makes some dependencies stated in that one superfluous (because transitivity takes care of them), one could try to combine them in such a way as to reduce the number of lines changed. Dunno if it's worth the effort.
Created attachment 220815 [details, diff]
Combined parallel-build.patch
Hi Martin,
I took the liberty to combine your patch with the parallel-build.patch that already existed in $FILESDIR.
This way one doesn't need to change the ebuild itself.
Best regards,
Torsten
Thanks, Martin and Torsten. Tested and committed in place for 1.11.0_pre797991. |