Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 534312 - gnome2.eclass: unset DISPLAY
Summary: gnome2.eclass: unset DISPLAY
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] GNOME (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Linux Gnome Desktop Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: gnome2.eclass 571378
  Show dependency tree
 
Reported: 2015-01-02 11:43 UTC by Pacho Ramos
Modified: 2016-04-03 12:57 UTC (History)
1 user (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Pacho Ramos gentoo-dev 2015-01-02 11:43:35 UTC
After reading bug 527682 I also remembered that some test failures in the past were related with this random DISPLAY value, also not unsetting it makes some tests to ever draw windows while running making them really annoying. 

I am 100% for unsetting this, but I am unsure if maybe this would lead to many more gnome packages needing virtualx.eclass to be run and, then, I am unsure if maybe we should have a default src_test phase running the tests always inside virtualx :|

Thanks for your thoughts
Comment 1 Gilles Dartiguelongue (RETIRED) gentoo-dev 2015-12-06 22:35:26 UTC
The virtualx eclass was recently changed, maybe we should add that there ?
Comment 2 Pacho Ramos gentoo-dev 2015-12-11 19:23:54 UTC
I am unsure because lots of this packages only show this problems when a DISPLAY is found but, when it's not set (like, for example, when building from console) they won't show any problem and, then, they won't need to use virtualx.eclass
Comment 3 Gilles Dartiguelongue (RETIRED) gentoo-dev 2015-12-13 12:01:28 UTC
Hum, I see what you mean. In any case, if the package does indeed need X, we will need to fix its src_test. We will need to check this to some extent (like emerge -e @world) before pushing this to the tree.
Comment 4 Pacho Ramos gentoo-dev 2016-01-20 11:35:03 UTC
(In reply to Gilles Dartiguelongue from comment #3)
> Hum, I see what you mean. In any case, if the package does indeed need X, we
> will need to fix its src_test. We will need to check this to some extent
> (like emerge -e @world) before pushing this to the tree.

We can maybe introduce this only for EAPI6 to be safer ;)
Comment 5 Pacho Ramos gentoo-dev 2016-04-03 12:57:48 UTC
[master dc527ce] gnome2.eclass/gnome2-utils.eclass: implement eapi6 support and take advantage of that for finally fixing many bugs/enhancements that were waiting for this (#571378)
 2 files changed, 113 insertions(+), 47 deletions(-)