[Pm-utils] [PATCH] parse video quirks in uswsusp sleep module

[Victor Lowther]

On Mon, 2008-03-17 at 04:39 +0100, Michael Biebl wrote:
> 2008/3/17, Victor Lowther <victor.lowther at gmail.com>:
> > On Mon, 2008-03-17 at 04:00 +0100, Michael Biebl wrote:
> >  > I think, the correct way is to fix the hal fdi file for the given
> >  > laptop. That's as easy as editing a file in /etc/pm and well
> >  > documented [1].
> >  > I'm sure, Richard will agree.
> >
> >
> > To you and Richard, mabye.  To me, not so much.  To an end user, you may
> >  as well ask them to jump to the moon.
> 
> It's obvious that your solution is more intutive for you, as you wrote the code.
> 
> For an end user, editing a file in /etc is non-intuitive in either case.

Indeed. 

That is why I coded a solution that allows for a bit of sloppy editing
and that requires mostly one-line edits on the part of an end user.

> The difference is, that for hal exists a standardised format and lots
> of documentation.
> That's why I prefer this solution.

The problem is that the standardized format is XML.  XML is great for
machines to read and process, not so much for humans.

> If creating the fdi file for your machine is too complicated (which I
> don't deny), then we could wrap this into a small helper tool (just
> like hal-disable-polling).

Hek, a pointer to an examples that goes into the levels of detail I ahve
talked about would be nice.  None of the ones I have seen so far do.

> Let's hear, what others think about this issue.

Agreed.

> Cheers,
> Michael
> 
> 
-- 
Victor Lowther
Ubuntu Certified Professional