QA before uploading zope packages

[Brian Sutherland]

On Mon, Nov 07, 2011 at 01:22:14PM +0200, Gediminas Paulauskas wrote:
> 2011/10/17 Arnaud Fontaine <arnau at debian.org>:
> > Hi,
> >
> > Gediminas Paulauskas <menesis at pov.lt> writes:
> >
> >>> *  Format  in  debian/copyright   must  be  versioned,  for  example:
> >>>  http://svn.debian.org/wsvn/dep/web/deps/dep5.mdwn?op=file&rev=174
> >>
> >> Why do  you think it must  be versioned? Maybe it  made sense earlier,
> >> but   the    format   is   not   expected   to    change   any   more.
> >> http://dep.debian.net/deps/dep5/ is the  official URL of the proposal,
> >> and is a HTML page instead of text.
> >
> > Re-read it, for example:
> >
> >  Format
> >
> >  Required  single line:  URI  of the  format  specification, such  as:
> >  http://www.debian.org/doc/copyright-format/1.0
> >
> > And:
> >
> >  A  possible  debian/copyright  file  for  the  program  "X  Solitaire"
> >  distributed in the Debian source package xsol:
> >
> >  Format: <VERSIONED_FORMAT_URL>
> >  ...
> >
> > Moreover,  at least with  the latest  version of  lintian, it  reports a
> > warning if you don't use a versioned URL.
> 
> This warning is seen only with --pedantic flag. From lintian manpage:
> 
>               Pedantic tags are Lintian at its most pickiest and include
>               checks for particular Debian packaging styles and checks that
>               many people disagree with.  Expect false positives and Lintian
>               tags that you don't consider useful if you use this option.
> 
> I am very disappointed that I was forced to change the correct URL to
> an obsolete one just because there is a pedantic tag in the latest
> version of lintian. And yes, I disagree with this tag.
> 
> > I'm not saying  your packages are not lintian-clean,  I'm just saying to
> > check it before  I upload them (for example for  the DEP5 comment above,
> > with the latest version of lintian, it displays a warning).
> 
> The latest version of lintian does NOT display this warning.
> 
> If you have pedantic in your lintianrc, please remove it. It does not
> help to get things right, quite the opposite.

+LOTS for not using pedantic

Every change carries a risk of breaking something. We should not just
change things for the sake of change. It's nice to have packages that
conform to the absolute latest QA standards, but it's better to have
packages that work.

Blindly following what QA tools say is almost always bad and sometimes
worse: http://lwn.net/Articles/282230/

-- 
Brian Sutherland