On Wed, Sep 22, 2004 at 04:05:12PM -0400, Andrew Pimlott wrote: > > > The patch by Gus (if you reverse the second half) looks like an > > > expedient solution, unless you think it will confuse people. > > > > That just changes one hard-coded default for another. I think -softdpi > > would be superior to both. > > Choosing a specific dpi with no knowledge of the display is (finally) > becoming an anachronism, so I don't see the harm in using a hard-coded > hack for this purpose. IOW, I think it should be viewed as a > last-ditch, can't possibly be right but better than nothing, fallback; > and for that, hard-coding is fine. If anyone needs to override it, > there is a much better mechanism: the DisplaySize parameter. The problem with that is that it is XFree86-specific. There are more X servers in the world than just XFree86's. > Besides, I think we're close to being able to do a better job with > debconf. I notice you already ask for screen size in some cases, and > I'm guessing EDID gives it to you when it works. It seems that all you > need to do is have dexconf set DisplaySize, and you can drop -dpi. What > is the obstacle to that? (Well, for the sake of upgraders, you might > want to change the hard-coded default in the X server, so they still > have a fall-back.) You're right. If you look at the package TODO you'll see that further overhaul of the X server debconfage is planned. It's not *quite* as simple as you describe, but it is true that by refactoring the logic of monitor configuration, we could kill more birds with that stone. I expect to be able to work on this in the post-sarge timeframe. If you're interested in helping (whether with code review, testing, or patches), please see: svn://necrotic.deadbeast.net/xfree86/branches/debconf-overhaul/ > Regardless, I'm sure the tide of progress will force you to remove -dpi > soon enough. :-) I can but hope. :) -- G. Branden Robinson | Eternal vigilance is the price of Debian GNU/Linux | liberty. branden@debian.org | -- Wendell Phillips http://people.debian.org/~branden/ |
Attachment:
signature.asc
Description: Digital signature