Bug#665296: Putting the sed snippet into a filename

2012-09-09 Thread Marcin Owsiany
On Sat, Sep 08, 2012 at 03:56:46PM -0400, Joey Hess wrote: Marcin Owsiany wrote: - dh_installxmlcatalogs passing an overly long string to autoscript(). I think whatever fix is implemented (unless someone knows an answer to my question above), it will mean a change to

Bug#665296: Putting the sed snippet into a filename

2012-09-09 Thread Joey Hess
Marcin Owsiany wrote: +# 4: either text: shell-quoted sed to run on the snippet. Ie, 's/#PACKAGE#/$PACKAGE/' +#or a sub to run on each line of the snippet. Ie sub { s/#PACKAGE#/$PACKAGE/ } I had not thought about making it operate on each line rather than the whole file, but I think I

Bug#665296: Putting the sed snippet into a filename

2012-09-09 Thread Marcin Owsiany
On Sun, Sep 09, 2012 at 01:34:36PM -0400, Joey Hess wrote: Marcin Owsiany wrote: +# 4: either text: shell-quoted sed to run on the snippet. Ie, 's/#PACKAGE#/$PACKAGE/' +#or a sub to run on each line of the snippet. Ie sub { s/#PACKAGE#/$PACKAGE/ } I had not thought about making

Bug#665296: Putting the sed snippet into a filename

2012-09-09 Thread Joey Hess
Marcin Owsiany wrote: Thanks for the fast review, please see the revised patch attached. Applied. I will upload it once the current debhelper gets into testing. -- see shy jo signature.asc Description: Digital signature

Bug#665296: Putting the sed snippet into a filename

2012-09-08 Thread Marcin Owsiany
I'm looking into this as part of BSP in Dublin. The problem with Niels's fix is that it requires code changes to the program invoking autoscript, dh_installxmlcatalogs in this case. I tried Tristan's idea - patch below - and although it works, MY PATCH INTRODUCES A SUBTLE INCOMPATIBILITY.

Bug#665296: Putting the sed snippet into a filename

2012-09-08 Thread Joey Hess
Marcin Owsiany wrote: - dh_installxmlcatalogs passing an overly long string to autoscript(). I think whatever fix is implemented (unless someone knows an answer to my question above), it will mean a change to dh_installxmlcatalogs. So perhaps this bug should be cloned against xml-core