Bug#836571: jessie-pu: package rabbitvcs/0.16-1
Package: release.debian.org
Tags: jessie
Followup-For: Bug #836571
User: release.debian.org@packages.debian.org
Usertags: pu
> *No*. The bug you're fixing may be critical, the request to fix it in
> stable is at most normal.
Appologies - my misunderstanding.
> Given the fact that the package has no reverse-dependencies and before
> your NMU in unstable had not been updated for two years, I wonder
> whether removal might have been a better option.
That's a fair point - but presumably a matter for the PAPT rather than me?
> I assume your rationale for suggesting a release via stable-updates,
> rather than simply waiting for the next point release (which will be in
> just under two weeks time) is the potential for data loss.
This may be me misunderstanding the roles of stable-updates and point releases.
The next point release sounds fine, particularly given that #817231 had already
been open for some time.
I've rebuilt the diff with version 0.16-1+deb8u1 as I started from the Jessie
source package. Hopefully I have got it right this time?
Thanks for your help.
Christopher Hoskin
diff -Nru rabbitvcs-0.16/debian/changelog rabbitvcs-0.16/debian/changelog
--- rabbitvcs-0.16/debian/changelog 2014-10-05 10:21:39.000000000 +0100
+++ rabbitvcs-0.16/debian/changelog 2016-09-04 18:52:26.000000000 +0100
@@ -1,3 +1,10 @@
+rabbitvcs (0.16-1+deb8u1) jessie; urgency=medium
+
+ * Non-maintainer upload.
+ * Fix "All files in home folder removed" Applied upstream fix as patch (Closes: #817231)
+
+ -- Christopher Hoskin <christopher.hoskin@gmail.com> Sun, 04 Sep 2016 18:52:18 +0100
+
rabbitvcs (0.16-1) unstable; urgency=medium
* Team upload.
diff -Nru rabbitvcs-0.16/debian/patches/fix-817231.patch rabbitvcs-0.16/debian/patches/fix-817231.patch
--- rabbitvcs-0.16/debian/patches/fix-817231.patch 1970-01-01 01:00:00.000000000 +0100
+++ rabbitvcs-0.16/debian/patches/fix-817231.patch 2016-09-04 07:05:41.000000000 +0100
@@ -0,0 +1,38 @@
+Subject: Repository Browser export empties target folder
+ A critical bug, where when the user attempts to export files from within the
+ RabbitVCS Repository Browser to a local target folder, the contents of the
+ target folder are deleted without warning, and the export fails.
+Origin: upstream, https://github.com/rabbitvcs/rabbitvcs/pull/89/commits/b3dc98601d57d2e589999e491a66b562a399ce1a
+Bug-Debian: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=817231
+Bug: https://github.com/rabbitvcs/rabbitvcs/issues/70
+Bug: https://github.com/rabbitvcs/rabbitvcs/issues/127
+Bug: https://github.com/rabbitvcs/rabbitvcs/issues/161
+From: abaheti <abaheti@tibco.com>
+Acked-by: Christopher Hoskin <christopher.hoskin@gmail.com>
+Last-Update: 2016-08-28
+Applied-Upstream: commit:b3dc986, https://github.com/rabbitvcs/rabbitvcs/pull/89/commits/b3dc98601d57d2e589999e491a66b562a399ce1a
+
+---
+ rabbitvcs/vcs/svn/__init__.py | 6 +-----
+ 1 file changed, 1 insertion(+), 5 deletions(-)
+
+diff --git a/rabbitvcs/vcs/svn/__init__.py b/rabbitvcs/vcs/svn/__init__.py
+index 96d821d..dee5e7f 100644
+--- a/rabbitvcs/vcs/svn/__init__.py
++++ b/rabbitvcs/vcs/svn/__init__.py
+@@ -1237,11 +1237,7 @@ class SVN:
+ @param recurse: Whether or not to run a recursive checkout.
+
+ """
+- if os.path.exists(dest_path):
+- if os.path.isdir(dest_path):
+- shutil.rmtree(dest_path)
+- else:
+- os.remove(dest_path)
++ revision=Revision("head")
+
+
+
+--
+2.9.3
+
diff -Nru rabbitvcs-0.16/debian/patches/series rabbitvcs-0.16/debian/patches/series
--- rabbitvcs-0.16/debian/patches/series 2014-10-05 10:12:07.000000000 +0100
+++ rabbitvcs-0.16/debian/patches/series 2016-09-04 07:05:41.000000000 +0100
@@ -1 +1,2 @@
99_setup.py.patch
+fix-817231.patch
Reply to: