[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: [SRM] Approval for xserver-xorg-video-intel/2:2.13.0-6

Adam D. Barratt <adam@adam-barratt.org.uk> (18/02/2011):
> That looks fine; thanks.


> > The second fix is (user-contributed) backports:
> >
> >   http://git.debian.org/?p=pkg-xorg/driver/xserver-xorg-video-intel.git;a=commitdiff;h=91af3c5ead6157ed065ccd3e3cc413c945182488
> >   http://git.debian.org/?p=pkg-xorg/driver/xserver-xorg-video-intel.git;a=commitdiff;h=bdca7df8d4238f0630aeda75ed02461d76a5ea05
> That would be far easier to read if the variables hadn't all been renamed
> in the process. :-/

Yeah. :(

> > I've reproduced both issues on my laptop, verified the fixes
> > worked fine, and I haven't noticed any regression.
> Were your tests against unstable, or squeeze + your proposed fixes?

1st patch:
 - tested in sid.
 - tested in squeeze by Enrico, Julien, and myself.

2nd patch:
 - not tested in sid, unaffected/fixes already upstream at that point.
 - tested in squeeze by the submitter and myself.
 - I initially usertagged it as a candidate for r1, wasn't sure, but
   Julien felt it would be nice to have, and I concurred.


Attachment: signature.asc
Description: Digital signature

Reply to: