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

Bug#434744: Downgrade source-contains-CVS-dir to info



Thijs Kinkhorst <thijs@debian.org> writes:

> Package: lintian
> Version: 1.23.31
> Severity: wishlist

> I'm getting the following warnings for one of my packages:
>   W: phpgedview source: source-contains-CVS-dir places/IDN/CVS
>   W: phpgedview-places binary: package-contains-CVS-dir 
> usr/share/phpgedview/www/places/IDN/CVS/

> While I can and will handle the second one inside the packaging, making
> sure that it doesn't get installed, I'm not so sure about the first one.

> I've reported it upstream, but they haven't released a new version in
> ages. It seems to me I have the following options:
> 1) Repack upstream source;
> 2) Add an override.

> Both are not appropriate in my opinion: I shouldn't repack the source
> for removing one dir, and adding an override for this doesn't seem right
> since it's a real bug(let), but not in the packaging.

Yeah, this has been a problem for a long time, and it lingered mostly
because lintian was unable to distinguish between things like this that
were part of the upstream source and things added mistakenly by the Debian
maintainer.

I'm going to downgrade all of the similar tags to info level unless
there's something about them that really breaks the package and add new
tests for adding these files in the debian diff, which is the real problem
worth warning about.  (Unfortunately, that's going to mean duplicate tags,
one at info and one at warning, for these problems.  Hm.  That's annoying.
Maybe I'll unify debdiff and cruft to avoid that.)

-- 
Russ Allbery (rra@debian.org)               <http://www.eyrie.org/~eagle/>



Reply to: