Re: [gentoo-dev] virtualx eclass possible issue

2011-03-13 Thread Tomáš Chvátal
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Dne 12.3.2011 11:37, Paweł Hajdan, Jr. napsal(a): One of my ebuilds is using virtualx eclass, and I noticed the following code inside the eclass: retval=$? # Now kill Xvfb kill $(cat /tmp/.X${XDISPLAY}-lock) else

Re: [gentoo-dev] virtualx eclass possible issue

2011-03-12 Thread Michał Górny
On Sat, 12 Mar 2011 11:37:28 +0100 Paweł Hajdan, Jr. phajdan...@gentoo.org wrote: Shouldn't that last line look more like this (notice $retval instead of $?): [[ $retval -ne 0 ]] die ${FUNCNAME}: the ${VIRTALX_COMMAND} failed. What do you think? I'd say even '${VIRTUALX_COMMAND}'.

Re: [gentoo-dev] virtualx eclass

2008-10-22 Thread Petteri Räty
Doug Goldstein wrote: Doug Goldstein wrote: Doug Goldstein wrote: While the rule of thumb has been if an eclass needs something it should provide it's own depends. However the virtualx eclass needs to be different simply because in some cases it's only uses for tests (this is it's most

Re: [gentoo-dev] virtualx eclass

2008-10-20 Thread Doug Goldstein
Doug Goldstein wrote: Doug Goldstein wrote: While the rule of thumb has been if an eclass needs something it should provide it's own depends. However the virtualx eclass needs to be different simply because in some cases it's only uses for tests (this is it's most common usage in the

Re: [gentoo-dev] virtualx eclass

2008-10-16 Thread Doug Goldstein
Doug Goldstein wrote: While the rule of thumb has been if an eclass needs something it should provide it's own depends. However the virtualx eclass needs to be different simply because in some cases it's only uses for tests (this is it's most common usage in the whole) tree. When it's used for

Re: [gentoo-dev] virtualx eclass

2008-10-16 Thread Doug Goldstein
Doug Goldstein wrote: While the rule of thumb has been if an eclass needs something it should provide it's own depends. However the virtualx eclass needs to be different simply because in some cases it's only uses for tests (this is it's most common usage in the whole) tree. When it's used for

Re: [gentoo-dev] virtualx eclass

2008-10-16 Thread Donnie Berkholz
On 11:35 Thu 16 Oct , Doug Goldstein wrote: Doug Goldstein wrote: While the rule of thumb has been if an eclass needs something it should provide it's own depends. However the virtualx eclass needs to be different simply because in some cases it's only uses for tests (this is it's

Re: [gentoo-dev] virtualx eclass

2008-10-16 Thread Bo Ørsted Andresen
On Thursday 16 October 2008 23:54:32 Donnie Berkholz wrote: I'm not sure whether this would work, but one idea would be to handle dependencies depending on what's in IUSE of the ebuild inheriting. That would require ebuilds to set IUSE before inheriting the eclass. -- Bo Andresen