[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#519279: red colored correctly watch version



Dmitry E. Oboukhov wrote:

> Package: qa.debian.org
> 
> Please, look at my developer page:
>     http://qa.debian.org/developer.php?login=unera@debian.org
> 
> Why is watch version of krank red colored?

I believe you meant magenta

> Debian contains the version 0.7+dfsg1-7, watch showes the version
> 0.7+dfsg1

If you run lintian with -I on the source package you will see it emits
debian-watch-file-should-dversionmangle-not-uversionmangle

N: debian-watch-file-should-dversionmangle-not-uversionmangle
N:
N:   The version of this package contains dfsg, ds, or debian, but a
N:   misleading upstream version mangling occurs in the debian/watch file.
N:   Since the dfsg string is not part of the upstream version and its
N:   addition is Debian-specific, the the debian/watch file should use the
N:   dversionmangle option to remove, instead of adding in uversionmangle,
N:   the dfsg before comparing version numbers.
N:
N:   Refer to http://wiki.debian.org/DEHS for details.
N:
N:   Severity: wishlist, Certainty: certain
N:


> 
> quote from http://dehs.alioth.debian.org/report.php?package=krank:
[...]
>           o Upstream Version: 0.7+dfsg1
[...]
>           o Debian Version: 0.7+dfsg1-7

Sure, the thing is: since there are still many packages[1] which don't
remove the dfsg bit from the Debian version, both DEHS and the DDPO strip
it for them. DEHS has an extra hack, which I've not copied to the DDPO
code, which attempts to detect whether the upstream version (whether
modified by the watch file or not) has the dfsg bit, and if it does it
doesn't strip it from the Debian version.

The best solution for this would be to:
1.- fix all watch files that trigger debian-watch-file-should-mangle-version
and debian-watch-file-should-mangle-version
2.- remove all the hacks
3.- everyone happy

[1]http://lintian.debian.org/tags/debian-watch-file-should-mangle-version.html

Cheers,
-- 
Raphael Geissert - Debian Maintainer
www.debian.org - get.debian.net





Reply to: