Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 436618 - virtualx.eclass: Add variable VIRTUALX_RES to virtualx.eclass to allow overriding resolution from Gimp ebuilds
Summary: virtualx.eclass: Add variable VIRTUALX_RES to virtualx.eclass to allow overri...
Status: RESOLVED WONTFIX
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Eclasses (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo X packagers
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 414853
  Show dependency tree
 
Reported: 2012-09-29 20:25 UTC by Sebastian Pipping
Modified: 2019-03-03 05:41 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 Sebastian Pipping gentoo-dev 2012-09-29 20:25:46 UTC
Please see bug #414853 comment 5 for why we need something like this.

https://bugs.gentoo.org/show_bug.cgi?id=414853#c5

Thank you!
Comment 1 Chí-Thanh Christopher Nguyễn gentoo-dev 2012-09-29 20:29:56 UTC
I think that making Xvfb style parameters the external interface of the eclass might not be a good idea.

We may have to replace Xvfb with something else in the future (bug 409925).
Comment 2 Sebastian Pipping gentoo-dev 2012-09-29 20:32:09 UTC
Does that mean that xf86-video-dummy does not allow controlling the resolution?

If it does, It would be no problem, right?
Comment 3 Chí-Thanh Christopher Nguyễn gentoo-dev 2012-09-29 20:38:51 UTC
It allows controlling the resolution, but not in the way that it is done with Xvfb. So in its suggested format, ${VIRTUALXRES} would need to be parsed and converted to xorg.conf syntax.
Comment 4 Sebastian Pipping gentoo-dev 2012-09-29 20:42:10 UTC
How about three variables then to save any parsing by preserving the structure:

  VIRTUALX_RES_WIDTH or _X
  VIRTUALX_RES_HEIGHT or _Y
  VIRTUALX_RES_DEPTH or _BITS

Would that work?
Comment 5 Chí-Thanh Christopher Nguyễn gentoo-dev 2012-09-29 22:27:56 UTC
I increased the Xvfb resolution to address the immediate problem in bug 414853. When I add xf86-video-dummy support to virtualx.eclass, I will include a way to specify resolution, possibly the one from comment 4.
Comment 6 Xake 2012-09-29 22:46:03 UTC
(In reply to comment #4)
> How about three variables then to save any parsing by preserving the
> structure:
> 
>   VIRTUALX_RES_WIDTH or _X
>   VIRTUALX_RES_HEIGHT or _Y
>   VIRTUALX_RES_DEPTH or _BITS
> 
> Would that work?

Personally my mind screams KISS when I see that.
As most (all?) RANDR1.2 drivers creates a set of modes with names in the format <width>x<height> I think we only need on variable (VIRTUALX_RES?) that takes one parameter of that format. This is the same as Option "PreferredMode" in xorg.conf usually wants as a parameter, and the same goes for xrandr.

With other word, what I propose is something in the style of VIRTUALX_RES="1280x1024" which could be parsed directly into a 

Option "PreferredMode" "1280x1024"

in xorg.conf or set with

xrandr --output <something> --mode "1280x1024"


At the same time I start to think that virtualx always should unset DISPLAY. I am somewhat tired of getting my session highjacked by a testcase when I do a "emerge -e world" in the background....
Comment 7 Chí-Thanh Christopher Nguyễn gentoo-dev 2012-09-29 22:53:55 UTC
xf86-video-dummy does not support xrandr-1.2 yet.
Comment 8 Matt Turner gentoo-dev 2019-03-03 05:41:21 UTC
We haven't found another need for this parameter in more than six years.