Comment 30 for bug 387189
Revision history for this message
| Jonathan Marsden (jmarsden) wrote Re: /dev/null corrupted (/dev/null.1) | #30 |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
0e1f616
(Get the code!)
OK. Do we have a test case for which my patch fails?
If the remaining issue is that my patch contains a closing '}', that is easily "solved", because the {} around the variable name are only there for style reasons, and are not required.
Here is a modified patch without the {} . Is there a test case for which this one fails?
If this issue is deemed significant enough to SRU for Hardy and Lucid, then it seems easier to me to get a small patch to one script accepted, than to get a newer version of logrotate accepted.
Is it better to backport the logrotate patch? To do both the logrotate patch *and* fix the apache2 logrotate script?