[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: Requesting pre-approval to upload python-xattr to fix #609447 and #592860



Hello,

On pirmadienis 10 Sausis 2011 01:26:44 Federico Ceratto wrote:
> Hello,
> 
> I'm hereby requesting approval from the release team to upload a new
> version of python-xattr, meant to fix  #609447 and #592860
> 
> A dependency to python-setuptools has been added to fix #609447,
> while #592860 required a small patch to the code to prevent a crash
> when listing extended attributes:

FWIW, I don't see debian/control changes in the diff below.

> 
> --------------------
> 
> diff --git a/debian/changelog b/debian/changelog
> index 3a4eb78..51a0ece 100644
> --- a/debian/changelog
> +++ b/debian/changelog
> @@ -1,3 +1,10 @@
> +python-xattr (0.4-5+squeeze1) testing-proposed-updates; urgency=low
> +
> +  * Bugfix release: missing dependency from setuptools (Closes: #609447)
> +  * Bugfix: xattr crashes when listing file attributes (Closes: #592860)
> +
> + -- Federico Ceratto <federico.ceratto@gmail.com>  Sun, 09 Jan 2011
> 17:11:11 +0100
> +
>  python-xattr (0.4-5) unstable; urgency=low
> 
>    * New maintainer. (Closes: #579614)
> diff --git a/debian/patches/series b/debian/patches/series
> new file mode 100644
> index 0000000..cd49bce
> --- /dev/null
> +++ b/debian/patches/series
> @@ -0,0 +1 @@
> +tool_long_format.diff
> diff --git a/debian/patches/tool_long_format.diff
> b/debian/patches/tool_long_format.diff
> new file mode 100644
> index 0000000..33195e6
> --- /dev/null
> +++ b/debian/patches/tool_long_format.diff
> @@ -0,0 +1,11 @@
> +Initialize the long_format variable in tool.py
> +--- a/Lib/xattr/tool.py
> ++++ b/Lib/xattr/tool.py
> +@@ -38,6 +38,7 @@
> +     except getopt.GetoptError, e:
> +         usage(e)
> +
> ++    long_format = False
> +     for opt, arg in optargs:
> +         if opt in ("-h", "--help"):
> +             usage()
> 
> 
> 
> --
> Federico Ceratto


-- 
Modestas Vainius <modax@debian.org>

Attachment: signature.asc
Description: This is a digitally signed message part.


Reply to: