On Tue, 14 Aug 2012 20:05:50 +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.

I see your point, and I appreciate the fixes in general, but I guess
the release team would be a bit overwhelmed if we included fixes for
all not-yet-reported policy violations :)

BTW: When I try to build twice I get (after having added
o2monitor/o2hbmonitor to debian/clean already):

 dpkg-source -b ocfs2-tools-1.6.4
dpkg-source: info: using source format `3.0 (quilt)'
dpkg-source: info: building ocfs2-tools using existing 
./ocfs2-tools_1.6.4.orig.tar.gz
dpkg-source: warning: ignoring deletion of file o2monitor/o2hbmonitor
dpkg-source: warning: executable mode 0755 of 'vendor/common/o2cb.init' will 
not be represented in diff
dpkg-source: info: local changes detected, the modified files are:
 ocfs2-tools-1.6.4/mkfs.ocfs2/mkfs.ocfs2.8
 ocfs2-tools-1.6.4/o2image/o2image.8
 ocfs2-tools-1.6.4/vendor/common/o2cb.init

which doesn't exactly match the list of files you added to d/clean :)
 
> >> +  * 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?

No, it was me who was missing something (python details still confuse
me), thanks for the enlightenment!

And since the package has Depends: ...,  python (<< 2.8), python (>= 2.7), ...
we also have a safety net here.
 

Cheers,
gregor
 
-- 
 .''`.  Homepage: http://info.comodo.priv.at/ - OpenPGP key 0xBB3A68018649AA06
 : :' : Debian GNU/Linux user, admin, and developer  -  http://www.debian.org/
 `. `'  Member of VIBE!AT & SPI, fellow of the Free Software Foundation Europe
   `-   NP: Rolling Stones: Noexpec

Attachment: signature.asc
Description: Digital signature

Reply via email to