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

Bug#482040: [checks/patch-systems] check 00list.$ARCH for dpatch



Package: lintian
Version: 1.23.48
Severity: wishlist

On Tue, May 20, 2008 at 10:36:28AM +0200, Jörg Sommer wrote:
> Frank Lichtenheld schrieb am Tue 20. May, 03:44 (+0200):
> > On Sun, May 18, 2008 at 06:53:07PM +0200, Jörg Sommer wrote:
> > > > If you take a look at the code you will see that this is just a very big
> > > > cpp wrapper ;) The problem that the outcome of the test would depend on
> > > > the architecture you run lintian on remains.
> > > 
> > > What do you think about this patch? It simply removes all C and C++
> > > comments.
> > 
> > Yeah, I think that should suffice for now.
> 
> BTW: You don't check list.$ARCH
> 
> man dpatch:
>    However, there may be  situations  where  something  more  flexible  is
>    needed,  like  applying  the same patch on only two architectures, say,
>    m68k and powerpc. One solve this in at least two very  different  ways:
>    One  is  to  list the same patch in both debian/patches/00list.m68k and
>    debian/patches/00list.powerpc, […]
> 
> You might simply check all files 00list.*.

Let's file this as a bug for now.

Gruesse,
-- 
Frank Lichtenheld <frank@lichtenheld.de>
www: http://www.djpig.de/



Reply to: