Merge lp:~laney/apturl/1726327 into lp:apturl

[Iain Lane]

[Iain Lane (laney)]

see the bug, I think it's because a trailing slash is being added

[Sebastien Bacher (seb128)]

that seems to address bug #1453576 as well

[Sebastien Bacher (seb128)]

change looks good to me

review: Approve

[Will Cooke (willcooke)]

Tested with xdg-open on Arful, and it has the desired effect.

=== modified file 'AptUrl/Parser.py'
--- AptUrl/Parser.py	2012-06-12 17:00:15 +0000
+++ AptUrl/Parser.py	2017-10-23 11:28:39 +0000
@@ -127,9 +127,9 @@
                 apt_url.repo_url = schema[len("apt+"):] + ":" + package.split("?",1)[0]
             else:
                 if "?" in package:
-                    apt_url.package = package.split("?")[0].lstrip("/")
+                    apt_url.package = package.split("?")[0].lstrip("/").rstrip("/")
                 else:
-                    apt_url.package = package.lstrip("/")
+                    apt_url.package = package.lstrip("/").rstrip("/")
 
             # now parse the ?... bits
             if "?" in package:

=== modified file 'debian/changelog'
--- debian/changelog	2015-12-16 10:23:15 +0000
+++ debian/changelog	2017-10-23 11:28:39 +0000
@@ -1,3 +1,10 @@
+apturl (0.5.2ubuntu11) UNRELEASED; urgency=medium
+
+  * AptUrl/parser.py: Strip trailing slashes that are added by xdg-open/gio
+    and which make package names invalid. (LP: #1726327)
+
+ -- Iain Lane <[e-mail]>  Mon, 23 Oct 2017 12:27:21 +0100
+
 apturl (0.5.2ubuntu10) xenial; urgency=medium
 
   * debian/apturl.install: