Bug#833779: devscripts: [uscan] please strip whitespace in href when matching URLs

2016-08-08 Thread James McCoy
On Mon, Aug 08, 2016 at 11:40:39AM -0400, Daniel Kahn Gillmor wrote:
> the HTML there is decidedly weird, with extra whitespace in the href:
> 
> -
>  sks-1.1.6.tgz
> 
> -
> 
> So uscan isn't able to find it.
> 
> I reported the issue to bitbucket:
> 
> https://bitbucket.org/site/master/issues/13130/downloads-page-has-spurious-whitespace
> 
> and got this response on IRC (#bitbucket on freenode):
> 
> 11:29 <+dwfreed> dkg: browsers ignore the whitespace
> 11:30 <+dwfreed> URLs can't have spaces in them anyway, which is what the 
> field is
> 11:30 <+dwfreed> it must be encoded, either as + or %20
> 11:30 <+dwfreed> so teach the tracker to adhere to the spec

Referencing the HTML spec[0], the value of the href attribute is a
"valid URL potentially surrounded by spaces", so uscan does indeed need
to be fixed.

[0]: https://www.w3.org/TR/html5/links.html#links-created-by-a-and-area-elements

Cheers,
-- 
James
GPG Key: 4096R/91BF BF4D 6956 BD5D F7B7  2D23 DFE6 91AE 331B A3DB



Bug#833779: devscripts: [uscan] please strip whitespace in href when matching URLs

2016-08-08 Thread Daniel Kahn Gillmor
Package: devscripts
Version: 2.16.6
Severity: normal

sks is released here:

 https://bitbucket.org/skskeyserver/sks-keyserver/downloads

the HTML there is decidedly weird, with extra whitespace in the href:

-
 sks-1.1.6.tgz

-

So uscan isn't able to find it.

I reported the issue to bitbucket:

https://bitbucket.org/site/master/issues/13130/downloads-page-has-spurious-whitespace

and got this response on IRC (#bitbucket on freenode):

11:29 <+dwfreed> dkg: browsers ignore the whitespace
11:30 <+dwfreed> URLs can't have spaces in them anyway, which is what the field 
is
11:30 <+dwfreed> it must be encoded, either as + or %20
11:30 <+dwfreed> so teach the tracker to adhere to the spec


i don't really care which party fixes this issue -- or if both parties
do -- i just want uscan to work again for sks without my working
around it by adding an extra whitespace matcher to sks:debian/watch.

thanks for maintaining this infrastructure,

   --dkg

-- Package-specific info:

--- /etc/devscripts.conf ---

--- ~/.devscripts ---
Not present

-- System Information:
Debian Release: stretch/sid
  APT prefers testing-debug
  APT policy: (500, 'testing-debug'), (500, 'testing'), (200, 
'unstable-debug'), (200, 'unstable'), (1, 'experimental-debug'), (1, 
'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 4.7.0-rc7-amd64 (SMP w/4 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)

Versions of packages devscripts depends on:
ii  dpkg-dev 1.18.10
ii  libc62.23-4
ii  perl 5.22.2-3
pn  python3:any  

Versions of packages devscripts recommends:
ii  apt 1.3~pre2
ii  at  3.1.20-1
ii  curl7.47.0-1
ii  dctrl-tools 2.24-2
ii  debian-keyring  2016.07.02
ii  dput-ng [dput]  1.10
ii  dupload 2.7.0
ii  equivs  2.0.9+nmu1
ii  fakeroot1.21-1
ii  file1:5.28-4
ii  gnupg   2.1.14-2
ii  gnupg2  2.1.14-2
ii  libdistro-info-perl 0.14
ii  libencode-locale-perl   1.05-1
ii  liblwp-protocol-https-perl  6.06-2
pn  libsoap-lite-perl   
ii  liburi-perl 1.71-1
ii  libwww-perl 6.15-1
pn  licensecheck
ii  lintian 2.5.45
ii  man-db  2.7.5-1
ii  patch   2.7.5-1
ii  patchutils  0.3.4-1
ii  python3-debian  0.1.28
ii  python3-magic   1:5.28-4
ii  sensible-utils  0.0.9
ii  strace  4.12-3
ii  unzip   6.0-20
ii  wdiff   1.2.2-1+b1
ii  wget1.18-2
ii  xz-utils5.1.1alpha+20120614-2.1

Versions of packages devscripts suggests:
ii  build-essential  12.2
pn  cvs-buildpackage 
ii  devscripts-el35.12
ii  diffoscope   56
pn  dose-extra   
pn  gnuplot  
ii  gpgv 2.1.14-2
ii  gpgv22.1.14-2
pn  libauthen-sasl-perl  
pn  libfile-desktopentry-perl
ii  libnet-smtp-ssl-perl 1.03-1
pn  libterm-size-perl
ii  libtimedate-perl 2.3000-2
pn  libyaml-syck-perl
ii  mailutils [mailx]1:2.99.99-1
ii  mozilla-devscripts   0.47
pn  mutt 
ii  openssh-client [ssh-client]  1:7.2p2-8
ii  s-nail [mailx]   14.8.9-1
ii  svn-buildpackage 0.8.6
ii  w3m  0.5.3-29

-- no debconf information