[Merge] lp:~sil2100/ubuntu-archive-tools/plus-sign-review into lp:ubuntu-archive-tools

Łukasz Zemczak lukasz.zemczak at canonical.com
Wed Aug 2 14:40:43 UTC 2017


Łukasz Zemczak has proposed merging lp:~sil2100/ubuntu-archive-tools/plus-sign-review into lp:ubuntu-archive-tools.

Commit message:
Stop urlencoding source package name in sru-review when trying to access the debdiff as it makes it impossible to review packages with + in the name.

Requested reviews:
  Ubuntu Package Archive Administrators (ubuntu-archive)

For more details, see:
https://code.launchpad.net/~sil2100/ubuntu-archive-tools/plus-sign-review/+merge/328449

Do we really need to urlencode the source package name? In the current state it just makes the script not be able to access the debdiff, e.g.:
http://launchpadlibrarian.net/330880566/gtk+2.0_2.24.30-1ubuntu1.16.04.1_2.24.30-1ubuntu1.16.04.2.diff.gz

With quote() being used, the regexp cannot find the link in the queue webpage contents as the + sign is urlencoded. I wonder why this was added in the first place? Did something change in how links to the librarian are used?
-- 
Your team Ubuntu Package Archive Administrators is requested to review the proposed merge of lp:~sil2100/ubuntu-archive-tools/plus-sign-review into lp:ubuntu-archive-tools.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: review-diff.txt
Type: text/x-diff
Size: 1212 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/ubuntu-archive/attachments/20170802/a931e945/attachment-0001.diff>


More information about the ubuntu-archive mailing list