Your message dated Tue, 23 Apr 2019 05:35:00 +0000
with message-id <[email protected]>
and subject line Re: Bug#927732: unblock: variety/0.7.1-2 (pre-approval)
has caused the Debian Bug report #927732,
regarding unblock: variety/0.7.1-2 (pre-approval)
to be marked as done.
This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.
(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact [email protected]
immediately.)
--
927732: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=927732
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Package: release.debian.org
Severity: normal
User: [email protected]
Usertags: unblock
Dear Release Team,
Please consider unblocking variety 0.7.1-2. I've backported a couple of
fixes from the newest upstream version, which fix a couple of subtle but
annoying bugs. The changelog is as follows:
variety (0.7.1-2) unstable; urgency=medium
* Backport bugfixes from Variety 0.7.2:
- fix-crash-on-help-version.patch: Don't forward --help or --version to
running Variety instances, as this causes it to crash.
- fix-spurious-error-when-analyzing-gifs.patch: Fix spurious
FileNotFoundError when analyzing GIFs inside a wallpaper folder
-- James Lu <[email protected]> Sun, 21 Apr 2019 19:10:58 -0700
The debdiff is attached.
Best,
James
diff -Nru variety-0.7.1/debian/changelog variety-0.7.1/debian/changelog
--- variety-0.7.1/debian/changelog 2018-10-28 20:45:18.000000000 -0700
+++ variety-0.7.1/debian/changelog 2019-04-21 19:10:58.000000000 -0700
@@ -1,3 +1,13 @@
+variety (0.7.1-2) unstable; urgency=medium
+
+ * Backport bugfixes from Variety 0.7.2:
+ - fix-crash-on-help-version.patch: Don't forward --help or --version to
+ running Variety instances, as this causes it to crash.
+ - fix-spurious-error-when-analyzing-gifs.patch: Fix spurious
+ FileNotFoundError when analyzing GIFs inside a wallpaper folder
+
+ -- James Lu <[email protected]> Sun, 21 Apr 2019 19:10:58 -0700
+
variety (0.7.1-1) unstable; urgency=medium
[ Jelmer Vernooij ]
diff -Nru variety-0.7.1/debian/patches/fix-crash-on-help-version.patch
variety-0.7.1/debian/patches/fix-crash-on-help-version.patch
--- variety-0.7.1/debian/patches/fix-crash-on-help-version.patch
1969-12-31 16:00:00.000000000 -0800
+++ variety-0.7.1/debian/patches/fix-crash-on-help-version.patch
2019-04-21 19:10:58.000000000 -0700
@@ -0,0 +1,33 @@
+Description: Don't forward --help or --version to running Variety instances,
as this causes it to crash.
+ Parse options before checking for running instance to show help.
+Author: Pedro Romano <[email protected]>
+Origin: upstream,
https://github.com/varietywalls/variety/commit/4977f86ffc662ae3909cf97f6baa892ea17b4fc5
+Applied-Upstream: 0.7.2
+---
+ variety/__init__.py | 6 ++++--
+ 1 file changed, 4 insertions(+), 2 deletions(-)
+
+--- a/variety/__init__.py
++++ b/variety/__init__.py
+@@ -148,6 +148,9 @@
+
+ arguments = sys.argv[1:]
+
++ # validate arguments
++ options, args = VarietyWindow.VarietyWindow.parse_options(arguments)
++
+ bus = dbus.SessionBus()
+ # ensure singleton
+ if bus.request_name(DBUS_KEY) !=
dbus.bus.REQUEST_NAME_REPLY_PRIMARY_OWNER:
+@@ -161,10 +164,9 @@
+ safe_print(result)
+ return
+
+- # validate arguments and set up logging
++ # set up logging
+ # set_up_logging must be called after the DBus checks, only by one
running instance,
+ # or the log file can be corrupted
+- options, args = VarietyWindow.VarietyWindow.parse_options(arguments)
+ set_up_logging(options.verbose)
+
+ if options.verbose >= 2:
diff -Nru
variety-0.7.1/debian/patches/fix-spurious-error-when-analyzing-gifs.patch
variety-0.7.1/debian/patches/fix-spurious-error-when-analyzing-gifs.patch
--- variety-0.7.1/debian/patches/fix-spurious-error-when-analyzing-gifs.patch
1969-12-31 16:00:00.000000000 -0800
+++ variety-0.7.1/debian/patches/fix-spurious-error-when-analyzing-gifs.patch
2019-04-21 19:10:58.000000000 -0700
@@ -0,0 +1,45 @@
+Description: Fix spurious FileNotFoundError when analyzing GIFs inside a
wallpaper folder.
+ The path being passed to the helper function was not correct.
+Author: James Lu <[email protected]>
+Origin: upstream,
https://github.com/varietywalls/variety/commit/25714cd13e80602a5e08bf02ab1baf2dd9e1a97e
+Bug: https://github.com/varietywalls/variety/issues/132
+Applied-Upstream: 0.7.2
+---
+ variety/Util.py | 9 ++++++---
+ 1 file changed, 6 insertions(+), 3 deletions(-)
+
+diff --git a/variety/Util.py b/variety/Util.py
+index 86c70de..e038ff0 100644
+--- a/variety/Util.py
++++ b/variety/Util.py
+@@ -363,6 +363,7 @@ class Util:
+ def list_files(files=(), folders=(), filter_func=(lambda f: True),
max_files=10000, randomize=True):
+ count = 0
+ for filepath in files:
++ logger.debug(lambda: 'checking file %s against filter_func %s' %
(filepath, filter_func))
+ if filter_func(filepath) and os.access(filepath, os.R_OK):
+ count += 1
+ yield filepath
+@@ -379,14 +380,16 @@ class Util:
+ random.shuffle(files)
+ random.shuffle(subFolders)
+ for filename in files:
+- if filter_func(filename):
++ logger.debug(lambda: 'checking file %s against
filter_func %s (root=%s)' % (filename, filter_func, root))
++ path = os.path.join(root, filename)
++ if filter_func(path):
+ count += 1
+ if count > max_files:
+ logger.info(lambda: "More than %d files
in the folders, stop listing" % max_files)
+ return
+- yield os.path.join(root, filename)
++ yield path
+ except Exception:
+- logger.exception(lambda: "Cold not walk folder " + folder)
++ logger.exception(lambda: "Could not walk folder " +
folder)
+
+ @staticmethod
+ def start_force_exit_thread(delay):
+--
+2.20.1
+
diff -Nru variety-0.7.1/debian/patches/series
variety-0.7.1/debian/patches/series
--- variety-0.7.1/debian/patches/series 2018-10-26 06:56:11.000000000 -0700
+++ variety-0.7.1/debian/patches/series 2019-04-21 19:10:58.000000000 -0700
@@ -1 +1,3 @@
+fix-spurious-error-when-analyzing-gifs.patch
+fix-crash-on-help-version.patch
remove-outdated-versions-quit.patch
signature.asc
Description: OpenPGP digital signature
--- End Message ---
--- Begin Message ---
James Lu:
> Control: tags -1 moreinfo
>
> Hi Niels,
>
> I've uploaded the package and it should be in unstable now.
>
> Best,
> James
>
> [...]
Unblocked, thanks.
~Niels
--- End Message ---
