Bug#534580: lintian: false positive in trap bashism check

2009-06-25 Thread Julien Cristau
Package: lintian
Version: 2.2.12
Severity: normal

Hi,

lintian complains about bashisms in some X maintainer scripts, because
they include the following code:

trap message;\
  message \Received signal.  Aborting $THIS_PACKAGE package $THIS_SCRIPT 
script.\;\
  message;\
  exit 1 HUP INT QUIT TERM

This seems to be due to the trap check:
$LEADIN . qr'trap\s+[\']?.*[\']?\s+.*[1-9]', # trap with signal numbers

Cheers,
Julien



-- 
To UNSUBSCRIBE, email to debian-lint-maint-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#534580: lintian: false positive in trap bashism check

2009-06-25 Thread Russ Allbery
Russ Allbery r...@debian.org writes:

 Ah, yes.  This is where regexes try a bit too hard to make a match.

 trap\s+(?:([\\'])(?:[^\'\\]|\\.)*\1\s+)?[\\']*[1-9]

 will work a bit better, I think, although I haven't run that through a
 test suite.

 Interestingly, though, I can't duplicate this.  That regex does not
 appear to match that string so far as I can tell from some experiments.
 Maybe I messed something up

I did.  The current regex does match that string, and my modified
version doesn't.  The only thing I'm not sure of is if it matches the
other things it's supposed to.

-- 
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



Re: Bug#516706: lintian: [new check] report on missing commas in build-deps split across several lines

2009-06-25 Thread Adam D. Barratt
On Mon, 2009-02-23 at 09:07 +, Jo Shields wrote:
 It is nice when dealing with very long Build-Depends lines to split them
 across several lines, like this:
 
 Build-Depends: foo,
  bar,
  baz
 
 However, lintian does not notice if one of the commas on the end of a
 line is missing, and the source package might enter the archive in an
 unbuildable state (e.g. if the package is tested in a dirty chroot, then
 the packager may not notice the problem)

fwiw, there's a (very) recent commit to dpkg's git repository meaning
that such broken stanzas will be rejected by dpkg.  This may cause one
or two issues with the test-cases for this tag. :-)

Adam


-- 
To UNSUBSCRIBE, email to debian-lint-maint-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Re: Bug#516706: lintian: [new check] report on missing commas in build-deps split across several lines

2009-06-25 Thread Russ Allbery
Adam D. Barratt a...@adam-barratt.org.uk writes:

 fwiw, there's a (very) recent commit to dpkg's git repository meaning
 that such broken stanzas will be rejected by dpkg.  This may cause one
 or two issues with the test-cases for this tag. :-)

Heh, I was surprised to see that they were regular tests already.  Yeah,
they should be moved into the raw Debian package test area to bypass
dpkg.

-- 
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



Processed: Lintian and debhelper need changes too

2009-06-25 Thread Debian Bug Tracking System
Processing commands for cont...@bugs.debian.org:

 clone 534638 -1 -2
Bug#534638: debian-policy: Section about Info documents needs to be updated
Bug 534638 cloned as bugs 534639-534640.

 reassign -1 debhelper 7.2.16
Bug#534639: debian-policy: Section about Info documents needs to be updated
Bug reassigned from package `debian-policy' to `debhelper'.

 retitle -1 dh_installinfo should not add call to install-info in postinsts
Bug#534639: debian-policy: Section about Info documents needs to be updated
Changed Bug title to `dh_installinfo should not add call to install-info in 
postinsts' from `debian-policy: Section about Info documents needs to be 
updated'.

 reassign -2 lintian 2.2.12
Bug#534640: debian-policy: Section about Info documents needs to be updated
Bug reassigned from package `debian-policy' to `lintian'.

 retitle -2 lintian should check that info files contains section and 
 directory entry
Bug#534640: debian-policy: Section about Info documents needs to be updated
Changed Bug title to `lintian should check that info files contains section and 
directory entry' from `debian-policy: Section about Info documents needs to be 
updated'.

 severity -2 wishlist
Bug#534640: lintian should check that info files contains section and directory 
entry
Severity set to `wishlist' from `normal'

 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



Bug#534640: lintian should check that info files contains section and directory entry

2009-06-25 Thread Russ Allbery
Tag: info-document-missing-dir-section
Severity: normal
Certainty: certain
Info: This info document has no INFO-DIR-SECTION line and no
 tt--section/tt option is passed to ttinstall-info/tt in the
 package ttpostinst/tt maintainer script.  ttinstall-info/tt will
 be unable to determine the section into which this info page should be
 put.  The best solution is to add a line like:
 .
   @dircategory Development
 .
 to the texinfo source so that the generated info file will contain a
 section.  See /usr/share/info/dir for sections to choose from.
 .
 Another alternative that doesn't require modifying the source is to
 explicitly pass a tt--section/tt option to ttinstall-info/tt in
 the ttpostinst/tt maintainer script, although in this case you will
 need to write the ttpostinst/tt yourself and not use tools like
 debhelper to do it for you.
Ref: policy 12.2

It's been there for quite a while.  It needs to be modified so that
passing --section to install-info isn't enough, but is there something
more that it needs besides that, or some other problem with the existing
Lintian check?

I assume we should drop the requirement that the maintainer scripts call
install-info and indeed warn if they do, noting that the calls can be
removed.

-- 
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



Bug#534640: lintian should check that info files contains section and directory entry

2009-06-25 Thread Raphael Hertzog
On Fri, 26 Jun 2009, Raphaël Hertzog wrote:
 Since the upload of install-info to sid, packages installing info
 documentation should no more call install-info in their postinst.

Lintian could check this, it's probably best to add this check only once
debhelper has been fixed (see #534639).

 You should however explain that info-files must have the required
 entries (INFO-DIR-SECTION, START-INFO-DIR-ENTRY / END-INFO-DIR-ENTRY)
 so that they can be registered in the info directory file automatically.
 Bugs have been filed about this, example:
 http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=528864

This is the most important check to add in lintian. The long description
should give some hints on how this can be fixed (like in the bugreport).

Cheers,
-- 
Raphaël Hertzog

Contribuez à Debian et gagnez un cahier de l'admin Debian Lenny :
http://www.ouaza.com/wp/2009/03/02/contribuer-a-debian-gagner-un-livre/



--
To UNSUBSCRIBE, email to debian-lint-maint-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#534640: lintian should check that info files contains section and directory entry

2009-06-25 Thread Raphael Hertzog
On Thu, 25 Jun 2009, Russ Allbery wrote:
 It's been there for quite a while.  It needs to be modified so that
 passing --section to install-info isn't enough, but is there something
 more that it needs besides that, or some other problem with the existing
 Lintian check?

No, I didn't knew about this check.

 I assume we should drop the requirement that the maintainer scripts call
 install-info and indeed warn if they do, noting that the calls can be
 removed.

Yes.

Cheers,
-- 
Raphaël Hertzog

Contribuez à Debian et gagnez un cahier de l'admin Debian Lenny :
http://www.ouaza.com/wp/2009/03/02/contribuer-a-debian-gagner-un-livre/



--
To UNSUBSCRIBE, email to debian-lint-maint-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



MisterBabel.com vous offre un iPod - Free iPod

2009-06-25 Thread MisterBabel.com
Pour être sûr de recevoir tous nos emails, ajoutez-nous à votre carnet 
d'adresses.
Si ce mail ne s'affiche pas correctement, suivez ce lien ( 
http://www.omkg.net/D1/Q22j5jQ/QhyE37rUq/Qnios5fqoxs.aspx ) 


MISTERBABEL.COM

Offre estivale   
Summer special offer
Oferta de verano
Zomeraanbod   

  



   Traduction professionnelle
 Toutes langues - Tous domaines
 7/24 - Demandez un devis ( 
mailto:cont...@misterbabel.com?subject=demande%20de%20devis )  



   Professionnal translation
 Any language - Any subject
 7/24 - Ask for a quote ( 
mailto:cont...@misterbabel.com?subject=quote%20request )  



   Traducción profesional
 Todos los idiomas - Todos los campos
 7/24 - Pida un presupuesto ( 
mailto:cont...@misterbabel.com?subject=presupuesto )  



   Professionele vertalingen
 Alle talen - Alle onderwerpen
 7/24 - Vraag een offerte ( mailto:cont...@misterbabel.com?subject=offerte )
 


 Si vous ne souhaitez plus recevoir de messages de la part de 
MisterBabel, cliquez ici ( 
http://www.omkg.net/D1/Q22j5jQ/QhyE47bUs/Qnios5fqoxs.aspx ) 
 click here ( http://www.omkg.net/D1/Q22j5jQ/QhyEF7dUt/Qnios5fqoxs.aspx )  to 
unsuscribe
 Si no desea seguir recibiendo información de parte de MisterBabel, pulse aquí 
( http://www.omkg.net/D1/Q22j5jQ/QhyED7fUu/Qnios5fqoxs.aspx ) 
 Klik hier ( http://www.omkg.net/D1/Q22j5jQ/QhyEE7gUv/Qnios5fqoxs.aspx )  als u 
geen berichten meer wilt ontvangen van MisterBabel.