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

2011-01-10 Thread Modestas Vainius
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.cera...@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 000..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 000..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 mo...@debian.org


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


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

2011-01-09 Thread Federico Ceratto
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:



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.cera...@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 000..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 000..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


-- 
To UNSUBSCRIBE, email to debian-release-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/aanlktimr4bxjgpn16kc5yptgi63ag6pspnieojhgm...@mail.gmail.com