Summary: | dev-util/plan9port should optionally depend on x11-apps/xauth | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Alex Efros <powerman-asdf> |
Component: | [OLD] Development | Assignee: | Andy Spencer <andy753421> |
Status: | RESOLVED FIXED | ||
Severity: | enhancement | CC: | blueness, proxy-maint |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: |
Updated ebuild with optional xauth
Patch to force build script to fail on build errors |
Description
Alex Efros
2013-03-28 14:25:58 UTC
ping Andy, any interest in implementing this? ping Alex, do you have an implementation? Otherwise I'll close this bug. No, I don't have a patch for this. It's on my todo list, but I've been slow getting around to it. There's a bit of work upstream that has to be done to make X11 optional as well. (In reply to Andy Spencer from comment #3) > It's on my todo list, but I've been slow getting around to it. There's a bit > of work upstream that has to be done to make X11 optional as well. Take your time. I just didn't want this bug to disappear under noise. It sounds legit. Created attachment 351444 [details]
Updated ebuild with optional xauth
Created attachment 351446 [details, diff]
Patch to force build script to fail on build errors
I don't know why I thought there was upstream work that needed to be done, probably because I didn't read the readme or something like that. I've attached an updated ebuild that make xauth optional, please review In addition, I added a patch that modifies the build script (INSTALL -b) to force it to error out when the build fails. Previously a build failure would return success anyway, so emerge would go ahead and install the broken build. As a side effect, the build no longer has the fancy incremental output thingy and just prints a bunch of CC commands like everything else. (note: I accidentally marked the ebuild upload as a patch, it's not actually a patch..) Thanks Andy. I've add this to the tree with a rev bump. I've also cleaned out the earlier stuff since it was superceded by later stuff with bug fixes --- there was a flurry of development earlier this year until things settled down. I did leave the version without the fixes in the bug on the tree just in case. Reopen this if its still an issue. |