On Fri, Jan 20, 2006 at 10:26:36PM +0200, Guillem Jover wrote: > I'm planning to apply the latest two patches on this bug report before > the mini-freeze starts if nobody opposes. You can check the whole > discussion (it's long though), or skip it and just go to the explanation > at the end. There's the patches and a small test suite. =) > > <http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=291939;msg=176> > <http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=291939;msg=185>
I've read the patches and found no obvious problems. Only one little error: at one point in your man page patch is a typo "Denian". Also note that you should change the line "local $not = $1;" in debian_arch_expand to "local $not = $1 || '';" since you will get warnings about undefined strings used in concatenation should someone attempt to run the code with warnings enabled (though there are probably way more problems like this is the controllib code). Gruesse, -- Frank Lichtenheld <[EMAIL PROTECTED]> www: http://www.djpig.de/ -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
