Processed: lintian: wrong build-depends-without-arch-dep quilt again

2006-06-21 Thread Debian Bug Tracking System
Processing commands for [EMAIL PROTECTED]:

 reopen 365041
Bug#365041: lintian: wrong build-depends-without-arch-dep quilt
'reopen' is deprecated when a bug has been closed with a version;
use 'found' or 'submitter' as appropriate instead.
Bug reopened, originator not changed.

 thanks
Stopping processing here.

Please contact me if you need assistance.

Debian bug tracking system administrator
(administrator, Debian Bugs database)


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Bug#365041: lintian: wrong build-depends-without-arch-dep quilt

2006-06-21 Thread Marcus Better
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

I also get build-depends-without-arch-dep quilt although my
debian/rules has the line

include /usr/share/quilt/quilt.make

Marcus
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.3 (GNU/Linux)

iD8DBQFEmTvOXjXn6TzcAQkRAi+BAKCiIyG8jdJGf+14PuEgEPlK+gDPxACg11vA
8LvYF2pnD2KlfghmP5jTxzk=
=Zd4G
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Bug#365041: lintian: wrong build-depends-without-arch-dep quilt again

2006-06-21 Thread Russ Allbery
clone 365041 -1
retitle -1 direct invocation of quilt in debian/rules clean not recognized
close 365041

Sam Hocevar [EMAIL PROTECTED] writes:

 reopen 365041
 thanks

Hi! I have the following rule in my clean target:

   QUILT_PATCHES=debian/patches quilt pop -a -R || test $$? = 2

Yet lintian still complains about build-depends-without-arch-dep
 quilt.

Yes, because currently lintian doesn't look for direct invocations of
quilt, only for use of the CDBS or quilt makefile fragments.

This doesn't have anything to do with the CDBS problem in the bug that you
reopened.  Please file new bug reports for new problems so that we can
better track in debian/changelog what's been fixed in each release.
Thanks!

-- 
Russ Allbery ([EMAIL PROTECTED])   http://www.eyrie.org/~eagle/


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Bug#365041: lintian: wrong build-depends-without-arch-dep quilt

2006-06-21 Thread Marcus Better
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Russ Allbery wrote:
 I don't.  What version of lintian are you using?

lintian is 1.23.21.

 And do you have a dependency on the right version of quilt?

Here's debian/control:

Source: ixp400-accesslib
Section: misc
Priority: optional
Maintainer: Marcus Better [EMAIL PROTECTED]
Build-Depends: debhelper (= 5.0.0), quilt (= 0.40)
Build-Depends-Indep: dh-kpatches, bzip2
Standards-Version: 3.7.2

Package: ixp400-accesslib-source
Architecture: all
Depends: debhelper (= 5.0.0), module-assistant, bzip2
Description: Intel IXP400 Software Access Library
  yadda yadda yadda



And debian/rules:

#!/usr/bin/make -f

include /usr/share/quilt/quilt.make

PROJECT=ixp400-accesslib
etc...


Regards,

Marcus
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.3 (GNU/Linux)

iD8DBQFEmaGvXjXn6TzcAQkRAi45AJ9cWFogWq7r5Z768Hwa/xhDmc0NfwCg9+EJ
uSfrUHHJemvtWHMeLz3Klig=
=Eleb
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Processed: Lintian bugs fixed in revision r680

2006-06-21 Thread Debian Bug Tracking System
Processing commands for [EMAIL PROTECTED]:

 package lintian
Ignoring bugs not assigned to: lintian

 # Fixed in r680 by rra
 tag 374878 + pending
Bug#374878: direct invocation of quilt in debian/rules clean not recognized
There were no tags set.
Tags added: pending

 thanks
Stopping processing here.

Please contact me if you need assistance.

Debian bug tracking system administrator
(administrator, Debian Bugs database)


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Bug#365041: lintian: wrong build-depends-without-arch-dep quilt

2006-06-21 Thread Russ Allbery
Marcus Better [EMAIL PROTECTED] writes:
 Russ Allbery wrote:

 I don't.  What version of lintian are you using?

 lintian is 1.23.21.

 And do you have a dependency on the right version of quilt?

 Here's debian/control:

 Source: ixp400-accesslib
 Section: misc
 Priority: optional
 Maintainer: Marcus Better [EMAIL PROTECTED]
 Build-Depends: debhelper (= 5.0.0), quilt (= 0.40)
 Build-Depends-Indep: dh-kpatches, bzip2
 Standards-Version: 3.7.2

Oh, bleh, I see what's wrong.  Fixing this in Subversion.  Thank you!

-- 
Russ Allbery ([EMAIL PROTECTED])   http://www.eyrie.org/~eagle/


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



lintian: r681 - in trunk: checks debian testset/relations/debian

2006-06-21 Thread rra
Author: rra
Date: 2006-06-21 22:53:10 +0200 (Wed, 21 Jun 2006)
New Revision: 681

Modified:
   trunk/checks/fields
   trunk/debian/changelog
   trunk/testset/relations/debian/control
   trunk/testset/relations/debian/rules
Log:
  + [RA] Allow (but don't require due to perl-base) Build-Depends on
perl for arch-independent packages if perl is invoked in
debian/rules clean.

Modified: trunk/checks/fields
===
--- trunk/checks/fields 2006-06-21 20:31:12 UTC (rev 680)
+++ trunk/checks/fields 2006-06-21 20:53:10 UTC (rev 681)
@@ -80,12 +80,16 @@
[ debhelper = '^\t\s*dh_.+' ],
[ dpatch = '^\t\s*dpatch\s' ],
[ patch = '^\t\s*(?:perl debian/)?yada\s+unpatch' ],
-   [ perl = '^\t\s*perl\s' ],
[ $python_depend = '^\t\s*python\s' ],
[ quilt = '^\t\s*(\S+=\S+\s+)*quilt\s' ],
[ yada = '^\t\s*yada\s' ],
 );
 
+# Similar, but the resulting dependency is only allowed, not required.
+my @rule_clean_allowed = (
+   [ 'perl | perl-base (= 5.6.0-16)' = 
'(^\t|\|\|)\s*(perl|\$\(PERL\))\s' ],
+);
+
 sub run {
 
 my $pkg = shift;
@@ -534,6 +538,11 @@
$needed_clean{$rule-[0]} = $rule-[2] 
|| $needed_clean{$rule-[0]} || '';
}
}
+   for my $rule (@rule_clean_allowed) {
+   if ($_ =~ /$rule-[1]/) {
+   $allowed_clean{$rule-[0]} = 1;
+   }
+   }
}
}
close RULES;
@@ -636,12 +645,17 @@
# dependencies in Build-Depends, and then allow any dependency in
# Build-Depends that's implied by the dependencies we require or allow
# there.
+   #
+   # We also have to map | to , when building the list of allowed
+   # packages so that the implications will work properly.
+   #
+   # This is confusing.  There should be a better way to do this.
if (-e fields/build-depends  $arch_dep_packages == 0) {
open(FH, fields/build-depends) or fail(cannot read 
fields/build-depends: $!);
my $build_depends = FH;
close FH;
my @packages = split /\s*,\s*/, $build_depends;
-   my @allowed = map { s/[\(\[][^\)\]]+[\)\]]//g; $_ } keys 
(%needed_clean), keys (%allowed_clean);
+   my @allowed = map { s/[\(\[][^\)\]]+[\)\]]//g; s/\|/,/g; $_ } 
keys (%needed_clean), keys (%allowed_clean);
my $dep = Dep::parse (join (',', @allowed));
foreach my $pkg (@packages) {
my $name = $pkg;

Modified: trunk/debian/changelog
===
--- trunk/debian/changelog  2006-06-21 20:31:12 UTC (rev 680)
+++ trunk/debian/changelog  2006-06-21 20:53:10 UTC (rev 681)
@@ -16,11 +16,14 @@
 + [RA] Fix handling of versioned Build-Depends to avoid spurious
   build-depends-without-arch-dep warnings.  Reported by Marcus
   Better.
++ [RA] Allow (but don't require due to perl-base) Build-Depends on
+  perl for arch-independent packages if perl is invoked in
+  debian/rules clean.
   * checks/po-debconf:
 + [RA] Don't consider templates.in to be a stray translated template
   file.  Thanks, Thomas Huriaux.  (Closes: #367534)
 
- -- Russ Allbery [EMAIL PROTECTED]  Wed, 21 Jun 2006 13:29:37 -0700
+ -- Russ Allbery [EMAIL PROTECTED]  Wed, 21 Jun 2006 13:53:06 -0700
 
 lintian (1.23.21) unstable; urgency=low
 

Modified: trunk/testset/relations/debian/control
===
--- trunk/testset/relations/debian/control  2006-06-21 20:31:12 UTC (rev 
680)
+++ trunk/testset/relations/debian/control  2006-06-21 20:53:10 UTC (rev 
681)
@@ -1,7 +1,7 @@
 Source: relations
 Section: misc
 Priority: optional
-Build-Depends: mail-transport-agent, libc6-dev, findutils, foo ( 2) [!amd64 
!i386], bar, arch-test1 [i386], arch-test2 [!i386], quilt (= 0.40)
+Build-Depends: mail-transport-agent, libc6-dev, findutils, foo ( 2) [!amd64 
!i386], bar, arch-test1 [i386], arch-test2 [!i386], quilt (= 0.40), perl
 Build-Depends-Indep: make, bash, debmake, build-essential
 Build-Conflicts: foo [amd64 i386], bar [alpha test], xlibs-dev, arch-test1 
[powerpc], arch-test2 [!sparc]
 Build-Conflicts-Indep: debmake [!powerpc]

Modified: trunk/testset/relations/debian/rules
===
--- trunk/testset/relations/debian/rules2006-06-21 20:31:12 UTC (rev 
680)
+++ trunk/testset/relations/debian/rules2006-06-21 20:53:10 UTC (rev 
681)
@@ -35,6 +35,11 @@
 
 clean:
rm -rf debian/tmp debian/files
+
+# Test allowing quilt Build-Depends for manual quilt invocations.
TESTING=foo ANOTHER=bar quilt || true
 
+# Test requiring perl 

lintian: r682 - trunk/debian

2006-06-21 Thread rra
Author: rra
Date: 2006-06-21 22:57:16 +0200 (Wed, 21 Jun 2006)
New Revision: 682

Modified:
   trunk/debian/changelog
Log:
More accurate changelog message for the last change.


Modified: trunk/debian/changelog
===
--- trunk/debian/changelog  2006-06-21 20:53:10 UTC (rev 681)
+++ trunk/debian/changelog  2006-06-21 20:57:16 UTC (rev 682)
@@ -16,9 +16,10 @@
 + [RA] Fix handling of versioned Build-Depends to avoid spurious
   build-depends-without-arch-dep warnings.  Reported by Marcus
   Better.
-+ [RA] Allow (but don't require due to perl-base) Build-Depends on
-  perl for arch-independent packages if perl is invoked in
-  debian/rules clean.
++ [RA] Don't require Build-Depends on perl even if run explicitly in
+  debian/rules clean, since perl-base may be sufficient.  Catch more
+  ways perl could be run, including the common idiom for packages that
+  use Build.PL.
   * checks/po-debconf:
 + [RA] Don't consider templates.in to be a stray translated template
   file.  Thanks, Thomas Huriaux.  (Closes: #367534)


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Bug#374899: lintian: aspell-* packages should be Architecture: all

2006-06-21 Thread Piotr Engelking

Package: lintian
Version: 1.23.21
Severity: wishlist

As far as I am aware, aspell-* packages contain only architecture
independent data and should therefore be tagged Architecture: all. This
seems not to be the case with aspell-{da,et,fi,it,no,pt,pt-br,pt-common,ru}.
I believe that lintian could complain about this problem.

-- System Information:
Debian Release: testing/unstable
 APT prefers testing
 APT policy: (500, 'testing')
Architecture: i386 (x86_64)
Shell:  /bin/sh linked to /bin/dash
Kernel: Linux 2.6.17-rc3
Locale: LANG=C, LC_CTYPE=pl_PL.UTF-8 (charmap=UTF-8)

Versions of packages lintian depends on:
ii  binutils 2.16.1cvs20060413-1 The GNU assembler, linker and bina
ii  diffstat 1.41-1  produces graph of changes introduc
ii  dpkg-dev 1.13.21 package building tools for Debian
ii  file 4.17-1  Determines file type using magic
ii  gettext  0.14.5-4GNU Internationalization utilities
ii  intltool-debian  0.34.2+20060512 Help i18n of RFC822 compliant conf
ii  libparse-debianchang 1.0-1   parse Debian changelogs and output
ii  man-db   2.4.3-3 The on-line manual pager
ii  perl [libdigest-md5- 5.8.8-4 Larry Wall's Practical Extraction

lintian recommends no packages.

-- no debconf information


--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]