Summary: | sci-electronics/geda test phase causes make forkbomb | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Diego Elio Pettenò (RETIRED) <flameeyes> |
Component: | New packages | Assignee: | The Soldering-Iron Brotherhood <sci-electronics> |
Status: | RESOLVED FIXED | ||
Severity: | normal | ||
Priority: | High | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: | patch to geda-1.6.0-r1.ebuild |
Description
Diego Elio Pettenò (RETIRED)
2009-12-12 19:33:11 UTC
Hmm, Diego, I wonder as there is no sci-electronics/geda-gaf in the tree. Can you please recheck? I think you mean =sci-electonics/geda-1.6.0-r1. That package fails here too during test with your stated problem. Yeah sorry not -gaf in there :) After some fiddling with the problem it finds an easy solution. Add the following src_test() to the ebuild and all goes well. src_test() { cd "${S}" emake -j1 check || die "test failed" } Btw: The first tests on AMD64 are promising. The ebuild works well and seems to be made nicely. One question arises however on the definition of RDEPEND: RDEPEND="${RDEPEND} sci-electronics/electronics-menu" I am wondering if that should DEPEND or CDEPEND. (In reply to comment #4) > After some fiddling with the problem it finds an easy solution. > > Add the following src_test() to the ebuild and all goes well. > > src_test() { > cd "${S}" > emake -j1 check || die "test failed" > } > > Btw: The first tests on AMD64 are promising. The ebuild works well and seems to > be made nicely. I'll test this solution. > One question arises however on the definition of RDEPEND: > > RDEPEND="${RDEPEND} > sci-electronics/electronics-menu" > > I am wondering if that should DEPEND or CDEPEND. > Yes, it's wrong. The right var is ${CDEPEND} Created attachment 213498 [details, diff]
patch to geda-1.6.0-r1.ebuild
This patch (with Thomas's fixes :) fix this bug.
Unfortunatelly Denis is away, and I haven't commit privileges yet, so would be great if someone can commit this.
Thanks
After hitting this once again and almost losing the tinderbox I committed it ;) Let's hope it's all fixed by this (I'm a bit doubtful to be honest because I think it _was_ already running it that way, but still). |