Re: incorrectly reporting dbg-package-missing-depends

2010-03-03 Thread C.J. Adams-Collier
On Tue, 2010-03-02 at 17:49 -0800, Russ Allbery wrote:
 C.J. Adams-Collier c...@colliertech.org writes:
 
  lintian is telling me that my library has missing values in the Depends
  field.  This seems to be because there is an assumption that these deps
  will be architecture-specific.  This is not the case in IronRuby and
  IronPython:
 
  http://git.debian.org/?p=pkg-cli-libs/packages/dlr-languages.git;a=blob;f=debian/control;h=ba06ab70129aac92be71d98ec044c27f4f0328ae;hb=5dfefe868a6ccb59935359ae45eadb691f774e12
 
 Could you provide a bit more background on what the debug package is
 providing in this case?  I'm not familiar with IronRuby or IronPython and
 would like to understand the problem a little better.  The ironruby-dbg
 package doesn't appear to be in the archives at present.
 
 Thanks!

Sure!

The package hasn't made it in quite yet:

http://ftp-master.debian.org/new/dlr-languages_20090805+git.e6b28d27
+dfsg-1.html

the -dbg package contains .mdb files, which are out-of-band debugging
data:

$ dpkg -L ironruby-dbg 
/.
/usr
/usr/lib
/usr/lib/ironruby
/usr/lib/ironruby/IronRuby.dll.mdb
/usr/lib/ironruby/IronRuby.Libraries.Yaml.dll.mdb
/usr/lib/ironruby/ir.exe.mdb
/usr/lib/ironruby/IronRuby.Libraries.dll.mdb
/usr/share
/usr/share/doc
/usr/share/doc/ironruby-dbg
/usr/share/doc/ironruby-dbg/copyright
/usr/share/doc/ironruby-dbg/changelog.Debian.gz

The package builds from source on sid in case you have some spare tuits.

Cheers,

C.J.



signature.asc
Description: This is a digitally signed message part


Re: incorrectly reporting dbg-package-missing-depends

2010-03-03 Thread Russ Allbery
C.J. Adams-Collier c...@colliertech.org writes:

 the -dbg package contains .mdb files, which are out-of-band debugging
 data:

 $ dpkg -L ironruby-dbg 
 /.
 /usr
 /usr/lib
 /usr/lib/ironruby
 /usr/lib/ironruby/IronRuby.dll.mdb
 /usr/lib/ironruby/IronRuby.Libraries.Yaml.dll.mdb
 /usr/lib/ironruby/ir.exe.mdb
 /usr/lib/ironruby/IronRuby.Libraries.dll.mdb
 /usr/share
 /usr/share/doc
 /usr/share/doc/ironruby-dbg
 /usr/share/doc/ironruby-dbg/copyright
 /usr/share/doc/ironruby-dbg/changelog.Debian.gz

 The package builds from source on sid in case you have some spare tuits.

Ah, yes, I see that this is a common problem for CLI code; mono-dbg has
the same problem.  I'm a bit surprised that the debugging symbols are
architecture-independent, but apparently they are.

The ideal solution would be to special-case packages containing only *.mdb
files, but the architecture of where this Lintian check is done makes that
a bit tricky to do.

Could you file a bug against Lintian so that we can track this?

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


-- 
To UNSUBSCRIBE, email to debian-lint-maint-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/878wa9w3gj@windlord.stanford.edu



Re: incorrectly reporting dbg-package-missing-depends

2010-03-03 Thread Russ Allbery
C.J. Adams-Collier c...@colliertech.org writes:

 the -dbg package contains .mdb files, which are out-of-band debugging
 data:

 $ dpkg -L ironruby-dbg 
 /.
 /usr
 /usr/lib
 /usr/lib/ironruby
 /usr/lib/ironruby/IronRuby.dll.mdb
 /usr/lib/ironruby/IronRuby.Libraries.Yaml.dll.mdb
 /usr/lib/ironruby/ir.exe.mdb
 /usr/lib/ironruby/IronRuby.Libraries.dll.mdb
 /usr/share
 /usr/share/doc
 /usr/share/doc/ironruby-dbg
 /usr/share/doc/ironruby-dbg/copyright
 /usr/share/doc/ironruby-dbg/changelog.Debian.gz

 The package builds from source on sid in case you have some spare tuits.

Ah, yes, I see that this is a common problem for CLI code; mono-dbg has
the same problem.  I'm a bit surprised that the debugging symbols are
architecture-independent, but apparently they are.

The ideal solution would be to special-case packages containing only *.mdb
files, but the architecture of where this Lintian check is done makes that
a bit tricky to do.

Could you file a bug against Lintian so that we can track this?

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


-- 
To UNSUBSCRIBE, email to debian-lint-maint-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/877hptw3fn@windlord.stanford.edu



Bug#528001: Bug#566820: lintian: Warn about missing debian/source/format, advise switch to new 3.0 source formats

2010-03-03 Thread Raphael Hertzog
tag 566820 + patch
thanks

Hi,

On Tue, 02 Mar 2010, Russ Allbery wrote:
  Do you want a patch for this? If yes, should it be a new check file or
  do you want it integrated in one of the existing check file?
 
 At first glance, asking people to declare the source format explicitly
 seems like a good idea.  I'm more reluctant to have Lintian recommend
 switching to 3.0.  Based on the debian-devel discussion, I'm not sure the
 idea has aged enough to make people comfortable with it.  But if they
 don't have any declared source format at present, I'm certainly fine with
 Lintian suggesting they consider it.
 
 I think having a new check script that checks files in debian/source would
 be a good idea.  I'm not sure what to call it.  source-control, maybe?
 That could be confused with something that tests VCS usage, though.

Ok, I went for debian-source-dir. In the process, I found other oddities
that I fixed so you really have 5 patches to apply.

I kept the pedantic tag using-old-source-format but you can comment it
if you prefer.

  It might also be a good idea to have unknown-source-format when
  debian/source/format contains something else than 1.0, 2.0, 3.0
  (quilt), 3.0 (native), 3.0 (git), 3.0 (bzr). This one should
  result in an error.
 
 Yes, please.

Also done.

 If you feel energetic, there's also #528001, and it would be nice to warn
 about unrecognized files in debian/source to catch people who have
 misspelled the control file name.

I added the check for unrecognized files in debian/source but the rest
I won't do (at least not now).

The list of debian/source/ files recognized is now longer... here's the
description I used:

N: 
N:The source package contains a file in debian/source/ that lintian
N:doesn't know about. Currently the following files are recognized:
N:
N: * format
N: * include-binaries
N: * lintian-overrides
N: * options
N: * patch-header
N:
N:This tag is emitted in case you mistyped the name of one of the above
N:files. If that's not the case and if the file can be legitimately be
N:expected in source packages, please file a bug against lintian asking
N:for the file to be recognized.
N:
N:Severity: important, Certainty: possible

Cheers,
-- 
Raphaël Hertzog

Like what I do? Sponsor me: http://ouaza.com/wp/2010/01/05/5-years-of-freexian/
My Debian goals: http://ouaza.com/wp/2010/01/09/debian-related-goals-for-2010/
From a2f4384f5d4cbcf536b6a896c9c9a6189f3eb5d9 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Rapha=C3=ABl=20Hertzog?= hert...@debian.org
Date: Wed, 3 Mar 2010 20:04:34 +0100
Subject: [PATCH 1/5] Simplify collection/debfiles to copy the debian directory entirely
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Despite the comment in the source code, the collector has always collected
the full debian directory and several scripts now depend on this behaviour:
* patch-systems use files below debian/patches/
* po-debconf use files below debian/po/
* debian-source-dir will use files below debian/source/

The line “next if -d $file” should have been “next if -d
unpacked/debian/$file” for the script to have its intended behaviour.
---
 collection/debfiles |   13 ++---
 1 files changed, 2 insertions(+), 11 deletions(-)

diff --git a/collection/debfiles b/collection/debfiles
index a7664d9..6493741 100755
--- a/collection/debfiles
+++ b/collection/debfiles
@@ -35,14 +35,5 @@ if (-e debfiles) {
 	or fail(cannot rm old debfiles directory);
 }
 
-mkdir('debfiles', 0777) or fail(cannot mkdir debfiles: $!);
-
-# Don't copy the whole directory, just all files in it.
-opendir(DEBIAN, 'unpacked/debian')
-	or fail(cannot open unpacked/debian/ directory: $!);
-while (my $file=readdir(DEBIAN)) {
-	next if -d $file;
-	copy_dir(unpacked/debian/$file, 'debfiles/')
-		or fail(cannot copy unpacked/debian/$file: $!);
-}
-closedir(DEBIAN);
+# Copy the whole debian directory
+copy_dir(unpacked/debian, debfiles);
-- 
1.7.0

From 32662c68108c143e4c6644afb7a5dac9bfe9f147 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Rapha=C3=ABl=20Hertzog?= hert...@debian.org
Date: Wed, 3 Mar 2010 20:26:23 +0100
Subject: [PATCH 2/5] New check script debian-source-dir

This script is meant to contain all checks made on debian/source/* files.
This initial implementation only covers debian/source/format.
---
 checks/debian-source-dir  |   61 +
 checks/debian-source-dir.desc |   43 +
 2 files changed, 104 insertions(+), 0 deletions(-)
 create mode 100644 checks/debian-source-dir
 create mode 100644 checks/debian-source-dir.desc

diff --git a/checks/debian-source-dir b/checks/debian-source-dir
new file mode 100644
index 000..8c4281d
--- /dev/null
+++ b/checks/debian-source-dir
@@ -0,0 +1,61 @@
+# debian/source directory content -- lintian check script -*- perl -*-
+
+# Copyright (C) 2010 by Raphaël Hertzog
+#
+# This program is free software; you 

Processed: Re: Bug#566820: lintian: Warn about missing debian/source/format, advise switch to new 3.0 source formats

2010-03-03 Thread Debian Bug Tracking System
Processing commands for cont...@bugs.debian.org:

 tag 566820 + patch
Bug #566820 [lintian] lintian: Warn about missing debian/source/format, advise 
switch to new 3.0 source formats
Added tag(s) patch.
 thanks
Stopping processing here.

Please contact me if you need assistance.

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


-- 
To UNSUBSCRIBE, email to debian-lint-maint-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/handler.s.c.126764872318586.transcr...@bugs.debian.org



Processed: tagging 571397, block 571397 with 572430

2010-03-03 Thread Debian Bug Tracking System
Processing commands for cont...@bugs.debian.org:

 tags 571397 + sid
Bug #571397 [src:writer2latex] please change Build-Depends to ant1.7
Added tag(s) sid.
 block 571397 with 572430
Bug #571397 [src:writer2latex] please change Build-Depends to ant1.7
Was not blocked by any bugs.
Added blocking bug(s) of 571397: 572430
 thanks
Stopping processing here.

Please contact me if you need assistance.

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


-- 
To UNSUBSCRIBE, email to debian-lint-maint-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/handler.s.c.126768884722441.transcr...@bugs.debian.org