Bug#159797: suggested fix

2011-09-08 Thread Paul Martin
On Thu, Sep 08, 2011 at 03:36:18PM +0200, Vitezslav Cizek wrote: > reading config file /etc/logrotate.conf > error: /etc/logrotate.conf:13 bad compressoptions path -7 -v OK. I think I see the problem. readPath() is being used rather than isolateLine(). -- Paul Martin signature.asc Description

Bug#159797: suggested fix

2011-09-08 Thread Vitezslav Cizek
> I think this is fixed in 3.8.0, as much of the parsing of > configuration lines was rewritten. > Are you able to test 3.8.0 (currently in experimental)? I've installed 3.8.0-1 from experimental, but still get: reading config file /etc/logrotate.conf error: /etc/logrotate.conf:13 bad compressop

Bug#159797: suggested fix

2011-09-08 Thread Vitezslav Cizek
* Paul Martin (p...@debian.org) wrote: > On Thu, Sep 08, 2011 at 10:26:03AM +0200, Vitezslav Cizek wrote: > > > you can use attached patch, which allows whitespace in compressoptions. > > What does this allow you to do, which you can't do with the standard > logrotate? > You can specify compres

Bug#159797: suggested fix

2011-09-08 Thread Paul Martin
On Thu, Sep 08, 2011 at 02:33:04PM +0200, Vitezslav Cizek wrote: > * Paul Martin (p...@debian.org) wrote: > > On Thu, Sep 08, 2011 at 10:26:03AM +0200, Vitezslav Cizek wrote: > > > > > you can use attached patch, which allows whitespace in compressoptions. > > > > What does this allow you to do,

Bug#159797: suggested fix

2011-09-08 Thread Paul Martin
On Thu, Sep 08, 2011 at 10:26:03AM +0200, Vitezslav Cizek wrote: > you can use attached patch, which allows whitespace in compressoptions. What does this allow you to do, which you can't do with the standard logrotate? -- Paul Martin signature.asc Description: Digital signature

Bug#159797: suggested fix

2011-09-08 Thread Vitezslav Cizek
Hi, you can use attached patch, which allows whitespace in compressoptions. It's build against 3.7.9, it shouldn't be hard to apply it to later versions. -- Vita Cizek Index: logrotate-3.7.9/config.c === --- logrotate-3.7.9.orig/con