Debian Bug report logs - #637524
qa.debian.org: UDD: Vcs-* header with uppercase letters not considered

Package: qa.debian.org; Maintainer for qa.debian.org is debian-qa@lists.debian.org;

Reported by: Michael Prokop <mika@debian.org>

Date: Fri, 12 Aug 2011 10:39:01 UTC

Severity: normal

Done: Lucas Nussbaum <lucas@debian.org>

Bug is archived. No further changes may be made.

Toggle useless messages

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to debian-bugs-dist@lists.debian.org, debian-qa@lists.debian.org:
Bug#637524; Package qa.debian.org. (Fri, 12 Aug 2011 10:39:05 GMT) Full text and rfc822 format available.

Acknowledgement sent to Michael Prokop <mika@debian.org>:
New Bug report received and forwarded. Copy sent to debian-qa@lists.debian.org. (Fri, 12 Aug 2011 10:39:07 GMT) Full text and rfc822 format available.

Message #5 received at submit@bugs.debian.org (full text, mbox):

From: Michael Prokop <mika@debian.org>
To: Debian Bug Tracking System <submit@bugs.debian.org>
Subject: qa.debian.org: UDD: Vcs-* header with uppercase letters not considered
Date: Fri, 12 Aug 2011 12:36:49 +0200
Package: qa.debian.org
Severity: normal
User: qa.debian.org@packages.debian.org
Usertags: udd


AFAICS udd has a bug when the VCS is written with uppercase letters
in the source package. Vcs-CVS doesn't seem to be counted as Cvs
then:

udd=> SELECT version,vcs_type,vcs_url,vcs_browser FROM sources_uniq WHERE source='mksh';                                                                                                  version         | vcs_type | vcs_url |                   vcs_browser
-------------------------+----------+---------+--------------------------------------------------
 28.0-3                  |          |         |
 35.2-3                  |          |         |
 39.3.20100725-1~bpo50+1 |          |         | http://cvs.mirbsd.de/contrib/hosted/tg/deb/mksh/
 39.3.20100725-1         |          |         | http://cvs.mirbsd.de/contrib/hosted/tg/deb/mksh/
 40.2-1~bpo50+1          |          |         | http://cvs.mirbsd.de/contrib/hosted/tg/deb/mksh/
 40.2-2~bpo60+1          |          |         | http://cvs.mirbsd.de/contrib/hosted/tg/deb/mksh/
 40.2-2                  |          |         | http://cvs.mirbsd.de/contrib/hosted/tg/deb/mksh/
 40.2-2                  |          |         | http://cvs.mirbsd.de/contrib/hosted/tg/deb/mksh/
(8 rows)

% apt-cache showsrc mksh | grep Vcs | sort -u
Vcs-Browser: http://cvs.mirbsd.de/contrib/hosted/tg/deb/mksh/
Vcs-Cvs: :ext:_anoncvs@anoncvs.mirbsd.org:/cvs contrib/hosted/tg/deb/mksh

% apt-get source mksh >/dev/null
% grep Vcs mksh-40.2/debian/control
Vcs-CVS: :ext:_anoncvs@anoncvs.mirbsd.org:/cvs contrib/hosted/tg/deb/mksh
Vcs-Browser: http://cvs.mirbsd.de/contrib/hosted/tg/deb/mksh/

regards,
-mika-




Information forwarded to debian-bugs-dist@lists.debian.org, debian-qa@lists.debian.org:
Bug#637524; Package qa.debian.org. (Fri, 19 Aug 2011 11:00:17 GMT) Full text and rfc822 format available.

Acknowledgement sent to Michael Prokop <mika@debian.org>:
Extra info received and forwarded to list. Copy sent to debian-qa@lists.debian.org. (Fri, 19 Aug 2011 11:00:21 GMT) Full text and rfc822 format available.

Message #10 received at 637524@bugs.debian.org (full text, mbox):

From: Michael Prokop <mika@debian.org>
To: 637524@bugs.debian.org
Subject: Re: Bug#637524: qa.debian.org: UDD: Vcs-* header with uppercase letters not considered
Date: Fri, 19 Aug 2011 12:57:39 +0200
[Message part 1 (text/plain, inline)]
* Michael Prokop [Fre Aug 12, 2011 at 12:36:49 +0200]:

> AFAICS udd has a bug when the VCS is written with uppercase letters
> in the source package. Vcs-CVS doesn't seem to be counted as Cvs
> then:

> udd=> SELECT version,vcs_type,vcs_url,vcs_browser FROM sources_uniq WHERE source='mksh';                                                                                                  version         | vcs_type | vcs_url |                   vcs_browser
> -------------------------+----------+---------+--------------------------------------------------
>  28.0-3                  |          |         |
>  35.2-3                  |          |         |
>  39.3.20100725-1~bpo50+1 |          |         | http://cvs.mirbsd.de/contrib/hosted/tg/deb/mksh/
>  39.3.20100725-1         |          |         | http://cvs.mirbsd.de/contrib/hosted/tg/deb/mksh/
>  40.2-1~bpo50+1          |          |         | http://cvs.mirbsd.de/contrib/hosted/tg/deb/mksh/
>  40.2-2~bpo60+1          |          |         | http://cvs.mirbsd.de/contrib/hosted/tg/deb/mksh/
>  40.2-2                  |          |         | http://cvs.mirbsd.de/contrib/hosted/tg/deb/mksh/
>  40.2-2                  |          |         | http://cvs.mirbsd.de/contrib/hosted/tg/deb/mksh/
> (8 rows)

> % apt-cache showsrc mksh | grep Vcs | sort -u
> Vcs-Browser: http://cvs.mirbsd.de/contrib/hosted/tg/deb/mksh/
> Vcs-Cvs: :ext:_anoncvs@anoncvs.mirbsd.org:/cvs contrib/hosted/tg/deb/mksh

> % apt-get source mksh >/dev/null
> % grep Vcs mksh-40.2/debian/control
> Vcs-CVS: :ext:_anoncvs@anoncvs.mirbsd.org:/cvs contrib/hosted/tg/deb/mksh
> Vcs-Browser: http://cvs.mirbsd.de/contrib/hosted/tg/deb/mksh/

Ok, it's not just about uppercase-only 'CVS', also lowercase-only
'cvs' seems to be affected:

udd=> SELECT version,vcs_type,vcs_url,vcs_browser FROM sources_uniq WHERE source='trustedqsl';
 version  | vcs_type | vcs_url |                   vcs_browser
----------+----------+---------+--------------------------------------------------
 1.11-4   |          |         |
 1.11-7   |          |         |
 1.11-8.1 |          |         |
 1.13-3   |          |         | http://trustedqsl.cvs.sourceforge.net/trustedqsl
 1.13-3   |          |         | http://trustedqsl.cvs.sourceforge.net/trustedqsl

% apt-cache showsrc trustedqsl | grep Vcs | sort -u
Vcs-Browser: http://trustedqsl.cvs.sourceforge.net/trustedqsl
Vcs-Cvs: :pserver:anonymous@trustedqsl.cvs.sourceforge.net:/cvsroot/trustedqsl co -P tqsl

% apt-get source trustedqsl >/dev/null
% grep Vcs trustedqsl-1.13/debian/control
Vcs-cvs: :pserver:anonymous@trustedqsl.cvs.sourceforge.net:/cvsroot/trustedqsl co -P tqsl
Vcs-Browser: http://trustedqsl.cvs.sourceforge.net/trustedqsl

regards,
-mika-
[signature.asc (application/pgp-signature, inline)]

Information forwarded to debian-bugs-dist@lists.debian.org, debian-qa@lists.debian.org:
Bug#637524; Package qa.debian.org. (Fri, 19 Aug 2011 19:33:03 GMT) Full text and rfc822 format available.

Acknowledgement sent to Michael Prokop <mika@debian.org>:
Extra info received and forwarded to list. Copy sent to debian-qa@lists.debian.org. (Fri, 19 Aug 2011 19:33:03 GMT) Full text and rfc822 format available.

Message #15 received at 637524@bugs.debian.org (full text, mbox):

From: Michael Prokop <mika@debian.org>
To: 637524@bugs.debian.org
Subject: Re: Bug#637524: qa.debian.org: UDD: Vcs-* header with uppercase letters not considered
Date: Fri, 19 Aug 2011 21:30:10 +0200
[Message part 1 (text/plain, inline)]
* Michael Prokop [Fri Aug 19, 2011 at 12:57:39PM +0200]:
> * Michael Prokop [Fre Aug 12, 2011 at 12:36:49 +0200]:

> > AFAICS udd has a bug when the VCS is written with uppercase letters
> > in the source package. Vcs-CVS doesn't seem to be counted as Cvs
> > then:
[...]

> Ok, it's not just about uppercase-only 'CVS', also lowercase-only
> 'cvs' seems to be affected:

Just in case it helps in debugging this issue, the following is a
list of packages that have a Vcs-[Cc][Vv][Ss] header but aren't
listed under vcs-type Cvs in UDD:

cvs
cvsd
deb-gview
dumbster
emacs-goodies-el
gss
gutenbrowser
hylafax
jsmath
jupp
kwalletcli
libbsd-arc4random-perl
libdisasm
libidn
libsgmls-perl
libxmltok
makefs
manpages
mksh
netpbm-free
oinkmaster
reprepro
rs
shishi
tqsllib
trustedqsl
xastir
xfm
xtrace

HTH && regards,
-mika-
[signature.asc (application/pgp-signature, inline)]

Information forwarded to debian-bugs-dist@lists.debian.org, debian-qa@lists.debian.org:
Bug#637524; Package qa.debian.org. (Sat, 03 Sep 2011 20:33:07 GMT) Full text and rfc822 format available.

Acknowledgement sent to Lucas Nussbaum <lucas@lucas-nussbaum.net>:
Extra info received and forwarded to list. Copy sent to debian-qa@lists.debian.org. (Sat, 03 Sep 2011 20:33:07 GMT) Full text and rfc822 format available.

Message #20 received at 637524@bugs.debian.org (full text, mbox):

From: Lucas Nussbaum <lucas@lucas-nussbaum.net>
To: Michael Prokop <mika@debian.org>, 637524@bugs.debian.org
Subject: Re: Bug#637524: qa.debian.org: UDD: Vcs-* header with uppercase letters not considered
Date: Sat, 3 Sep 2011 22:24:02 +0200
On 19/08/11 at 21:30 +0200, Michael Prokop wrote:
> * Michael Prokop [Fri Aug 19, 2011 at 12:57:39PM +0200]:
> > * Michael Prokop [Fre Aug 12, 2011 at 12:36:49 +0200]:
> 
> > > AFAICS udd has a bug when the VCS is written with uppercase letters
> > > in the source package. Vcs-CVS doesn't seem to be counted as Cvs
> > > then:
> [...]
> 
> > Ok, it's not just about uppercase-only 'CVS', also lowercase-only
> > 'cvs' seems to be affected:
> 
> Just in case it helps in debugging this issue, the following is a
> list of packages that have a Vcs-[Cc][Vv][Ss] header but aren't
> listed under vcs-type Cvs in UDD:

Hi,

I took a look at that, but it's not trivial to fix.
A proper fix would normalize the case of all field names, and then use
the normalized version everywhere.

Patch welcomed...

Lucas




Information forwarded to debian-bugs-dist@lists.debian.org, debian-qa@lists.debian.org:
Bug#637524; Package qa.debian.org. (Sun, 04 Sep 2011 07:21:02 GMT) Full text and rfc822 format available.

Acknowledgement sent to Raphael Hertzog <hertzog@debian.org>:
Extra info received and forwarded to list. Copy sent to debian-qa@lists.debian.org. (Sun, 04 Sep 2011 07:21:03 GMT) Full text and rfc822 format available.

Message #25 received at 637524@bugs.debian.org (full text, mbox):

From: Raphael Hertzog <hertzog@debian.org>
To: Lucas Nussbaum <lucas@lucas-nussbaum.net>, 637524@bugs.debian.org
Cc: Michael Prokop <mika@debian.org>
Subject: Re: Bug#637524: qa.debian.org: UDD: Vcs-* header with uppercase letters not considered
Date: Sun, 4 Sep 2011 09:16:24 +0200
On Sat, 03 Sep 2011, Lucas Nussbaum wrote:
> > Just in case it helps in debugging this issue, the following is a
> > list of packages that have a Vcs-[Cc][Vv][Ss] header but aren't
> > listed under vcs-type Cvs in UDD:
> 
> Hi,
> 
> I took a look at that, but it's not trivial to fix.
> A proper fix would normalize the case of all field names, and then use
> the normalized version everywhere.

Note that dpkg-source normalizes the field when it generates the .dsc
so I guess that you're parsing the unpacked source package ? Maybe you
could switch to parse the .dsc instead.

Cheers,
-- 
Raphaël Hertzog ◈ Debian Developer

Follow my Debian News ▶ http://RaphaelHertzog.com (English)
                      ▶ http://RaphaelHertzog.fr (Français)




Information forwarded to debian-bugs-dist@lists.debian.org, debian-qa@lists.debian.org:
Bug#637524; Package qa.debian.org. (Sun, 04 Sep 2011 09:36:03 GMT) Full text and rfc822 format available.

Acknowledgement sent to Lucas Nussbaum <lucas@lucas-nussbaum.net>:
Extra info received and forwarded to list. Copy sent to debian-qa@lists.debian.org. (Sun, 04 Sep 2011 09:36:04 GMT) Full text and rfc822 format available.

Message #30 received at 637524@bugs.debian.org (full text, mbox):

From: Lucas Nussbaum <lucas@lucas-nussbaum.net>
To: Raphael Hertzog <hertzog@debian.org>, 637524@bugs.debian.org
Cc: Michael Prokop <mika@debian.org>
Subject: Re: Bug#637524: qa.debian.org: UDD: Vcs-* header with uppercase letters not considered
Date: Sun, 4 Sep 2011 09:37:27 +0200
On 04/09/11 at 09:16 +0200, Raphael Hertzog wrote:
> On Sat, 03 Sep 2011, Lucas Nussbaum wrote:
> > > Just in case it helps in debugging this issue, the following is a
> > > list of packages that have a Vcs-[Cc][Vv][Ss] header but aren't
> > > listed under vcs-type Cvs in UDD:
> > 
> > Hi,
> > 
> > I took a look at that, but it's not trivial to fix.
> > A proper fix would normalize the case of all field names, and then use
> > the normalized version everywhere.
> 
> Note that dpkg-source normalizes the field when it generates the .dsc
> so I guess that you're parsing the unpacked source package ? Maybe you
> could switch to parse the .dsc instead.

I'm parsing the Packages and Sources files.

Ah, apparently, it's a problem in python-debian, which returns
'Vcs-Cvs:' instead of 'Vcs-Cvs' as field name if the value starts with
':'.

samosa uses python-debian 0.1.18.

Lucas




Information forwarded to debian-bugs-dist@lists.debian.org, debian-qa@lists.debian.org:
Bug#637524; Package qa.debian.org. (Sun, 04 Sep 2011 09:36:06 GMT) Full text and rfc822 format available.

Acknowledgement sent to Lucas Nussbaum <lucas@lucas-nussbaum.net>:
Extra info received and forwarded to list. Copy sent to debian-qa@lists.debian.org. (Sun, 04 Sep 2011 09:36:08 GMT) Full text and rfc822 format available.

Message #35 received at 637524@bugs.debian.org (full text, mbox):

From: Lucas Nussbaum <lucas@lucas-nussbaum.net>
To: Raphael Hertzog <hertzog@debian.org>, 637524@bugs.debian.org
Cc: Michael Prokop <mika@debian.org>
Subject: Re: Bug#637524: qa.debian.org: UDD: Vcs-* header with uppercase letters not considered
Date: Sun, 4 Sep 2011 11:33:02 +0200
On 04/09/11 at 09:37 +0200, Lucas Nussbaum wrote:
> On 04/09/11 at 09:16 +0200, Raphael Hertzog wrote:
> > On Sat, 03 Sep 2011, Lucas Nussbaum wrote:
> > > > Just in case it helps in debugging this issue, the following is a
> > > > list of packages that have a Vcs-[Cc][Vv][Ss] header but aren't
> > > > listed under vcs-type Cvs in UDD:
> > > 
> > > Hi,
> > > 
> > > I took a look at that, but it's not trivial to fix.
> > > A proper fix would normalize the case of all field names, and then use
> > > the normalized version everywhere.
> > 
> > Note that dpkg-source normalizes the field when it generates the .dsc
> > so I guess that you're parsing the unpacked source package ? Maybe you
> > could switch to parse the .dsc instead.
> 
> I'm parsing the Packages and Sources files.
> 
> Ah, apparently, it's a problem in python-debian, which returns
> 'Vcs-Cvs:' instead of 'Vcs-Cvs' as field name if the value starts with
> ':'.
> 
> samosa uses python-debian 0.1.18.

Ah, that's #597249.

So someone needs to prepare a squeeze backport for python-debian 0.1.21.

Lucas




Reply sent to Lucas Nussbaum <lucas@debian.org>:
You have taken responsibility. (Sun, 31 Mar 2013 13:33:10 GMT) Full text and rfc822 format available.

Notification sent to Michael Prokop <mika@debian.org>:
Bug acknowledged by developer. (Sun, 31 Mar 2013 13:33:10 GMT) Full text and rfc822 format available.

Message #40 received at 637524-done@bugs.debian.org (full text, mbox):

From: Lucas Nussbaum <lucas@debian.org>
To: Michael Prokop <mika@debian.org>, 637524-done@bugs.debian.org
Subject: Re: Bug#637524: qa.debian.org: UDD: Vcs-* header with uppercase letters not considered
Date: Sun, 31 Mar 2013 15:29:41 +0200
Hi,

I've deployed a new importer for archive data, and fixes that issue.

Lucas



Bug archived. Request was from Debbugs Internal Request <owner@bugs.debian.org> to internal_control@bugs.debian.org. (Mon, 29 Apr 2013 07:26:44 GMT) Full text and rfc822 format available.

Send a report that this bug log contains spam.


Debian bug tracking system administrator <owner@bugs.debian.org>. Last modified: Sun Apr 20 19:40:40 2014; Machine Name: beach.debian.org

Debian Bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.