gregor herrmann <gre...@debian.org> writes:

> On Sun, 12 Aug 2012 17:18:24 +0200, Sébastien Villemot wrote:
>> The patch also includes fixes for packaging issues.
>
> This is not a good idea for an NMU for an RC bug during the freeze

The package is currently not cleaning properly (violation of policy
§4.1) and not building twice in a row. While the former is clearly an RC
issue, it looks like the latter is indeed not considered as such.
Anyways I don't mind if these changes are not incorporated.

>> diff -Nru ocfs2-tools-1.6.4/debian/changelog 
>> ocfs2-tools-1.6.4/debian/changelog
>> --- ocfs2-tools-1.6.4/debian/changelog       2012-04-18 07:16:36.000000000 
>> +0000
>> +++ ocfs2-tools-1.6.4/debian/changelog       2012-08-12 15:11:33.000000000 
>> +0000
>> @@ -1,3 +1,16 @@
>> +ocfs2-tools (1.6.4-1.1) unstable; urgency=low
>> +
>> +  * Non-maintainer upload.
>> +  * Remove popen2-ignore-depreciation-warn.patch: DeprecationWarnings are 
>> now
>> +    ignored by default in Python 2.7, and this patch messes things up
>> +    (Closes: #682517, LP: #923754)
>
> Hm, and what happens if ocfs2console is run with python2.6?

I did not consider that possibility, given that the app is run with a
#!/usr/bin/python shebang, and that the /usr/bin/python symlink is not
managed by an alternative. Am I missing something?

Thanks for your review,

-- 
 .''`.    Sébastien Villemot
: :' :    Debian Maintainer
`. `'     http://www.dynare.org/sebastien
  `-      GPG Key: 4096R/381A7594

Attachment: pgpx41f3RKzBj.pgp
Description: PGP signature

Reply via email to