Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 463616 - dev-util/plan9port should optionally depend on x11-apps/xauth
Summary: dev-util/plan9port should optionally depend on x11-apps/xauth
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Development (show other bugs)
Hardware: All Linux
: Normal enhancement (vote)
Assignee: Andy Spencer
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-03-28 14:25 UTC by Alex Efros
Modified: 2013-06-21 14:39 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Updated ebuild with optional xauth (plan9port-20130320-r1.ebuild,2.27 KB, text/plain)
2013-06-20 08:10 UTC, Andy Spencer
Details
Patch to force build script to fail on build errors (plan9port-builderr.patch,936 bytes, patch)
2013-06-20 08:11 UTC, Andy Spencer
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alex Efros 2013-03-28 14:25:58 UTC
Is it possible to make dependency on xauth optional (controlled by USE-flag "X")?
Most of plan9port tools have nothing with X, and I dislike to install 18 X-related packages on server just as dependency of plan9port.
Comment 1 Anthony Basile gentoo-dev 2013-06-19 10:40:45 UTC
ping Andy, any interest in implementing this?

ping Alex, do you have an implementation?

Otherwise I'll close this bug.
Comment 2 Alex Efros 2013-06-19 12:59:44 UTC
No, I don't have a patch for this.
Comment 3 Andy Spencer 2013-06-19 17:35:29 UTC
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.
Comment 4 Anthony Basile gentoo-dev 2013-06-19 21:07:17 UTC
(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.
Comment 5 Andy Spencer 2013-06-20 08:10:59 UTC
Created attachment 351444 [details]
Updated ebuild with optional xauth
Comment 6 Andy Spencer 2013-06-20 08:11:35 UTC
Created attachment 351446 [details, diff]
Patch to force build script to fail on build errors
Comment 7 Andy Spencer 2013-06-20 08:15:43 UTC
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..)
Comment 8 Anthony Basile gentoo-dev 2013-06-21 14:39:28 UTC
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.