Bug#172078: lintian: suggest test for PIC in .a

2013-09-15 Thread Kevin Ryde
Niels Thykier  writes:
>
> On 2012-12-20 12:57, Bastien ROUCARIES wrote:
>> control: tags -1 + moreinfo
>> 
>> The policy have changed and I could not found the text. Could you give
>> some verbatim copy ?

Oh, I see in the short time since I posted it's moved from 11.2 to 10.2.

I would probably re-word my .desc for clarity now and since the libtool
bug presumably no longer applies unless you're stuck at its 1.4.


10.2. Libraries
---

...

 As to the static libraries, the common case is not to have relocatable
 code, since there is no benefit, unless in specific cases; therefore
 the static version must not be compiled with the `-fPIC' flag.  Any
 exception to this rule should be discussed on the mailing list
 _debian-devel@lists.debian.org_, and the reasons for compiling with
 the `-fPIC' flag must be recorded in the file `README.Debian'.  [3]

...

[3]  Some of the reasons for linking static libraries with the `-fPIC' flag
 are if, for example, one needs a Perl API for a library that is under
 rapid development, and has an unstable API, so shared libraries are
 pointless at this phase of the library's development.  In that case,
 since Perl needs a library with relocatable code, it may make sense to
 create a static library with relocatable code.  Another reason cited
 is if you are distilling various libraries into a common shared
 library, like `mklibs' does in the Debian installer project.





-- 
The sigfile music jargon series:
Classical Music -- the composer is dead.
Classic Rock -- the composer is probably dead.


-- 
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/87ioy1ljt0@blah.blah



[lintian] branch master updated (ddc0824 -> beb1094)

2013-09-15 Thread Niels Thykier
This is an automated email from the git hooks/post-receive script.

nthykier pushed a change to branch master
in repository lintian.

  from  ddc0824   c/apache2.pm: Fix F-P when module name had uppercase 
letters
   new  153961e   L::Relation: pkg now implies pkg:any (but not vice versa)
   new  beb1094   d/scripts/interpreters: Add ":any" to the python 
dependency

The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "adds" were already present in the repository and have only
been added to this reference.


Summary of changes:
 data/scripts/interpreters |2 +-
 debian/changelog  |9 +
 lib/Lintian/Relation.pm   |7 ++-
 .../debian/debian/clean   |0
 .../debian/debian/control.in  |3 ++-
 .../debian/debian/install |0
 .../debian/debian/manpages|0
 .../debian/debian/rules   |0
 .../debian/python-script  |0
 .../debian/script.pod |0
 t/tests/scripts-missing-dep-fp/desc   |6 ++
 .../scripts-missing-dep-fp}/tags  |0
 12 files changed, 24 insertions(+), 3 deletions(-)
 copy t/tests/{scripts-exec => scripts-missing-dep-fp}/debian/debian/clean 
(100%)
 copy t/tests/{files-uses-perl4-libs-with-dep => 
scripts-missing-dep-fp}/debian/debian/control.in (90%)
 copy t/tests/{scripts-exec => scripts-missing-dep-fp}/debian/debian/install 
(100%)
 copy t/tests/{scripts-exec => scripts-missing-dep-fp}/debian/debian/manpages 
(100%)
 copy t/tests/{scripts-missing-dep => 
scripts-missing-dep-fp}/debian/debian/rules (100%)
 copy t/tests/{scripts-missing-dep => 
scripts-missing-dep-fp}/debian/python-script (100%)
 copy t/tests/{scripts-exec => scripts-missing-dep-fp}/debian/script.pod (100%)
 create mode 100644 t/tests/scripts-missing-dep-fp/desc
 copy t/{debs/deb-format-record-size => tests/scripts-missing-dep-fp}/tags 
(100%)

-- 
Alioth's /usr/local/bin/git-commit-notice on 
/srv/git.debian.org/git/lintian/lintian.git


-- 
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/20130915223538.22909.78...@vasks.debian.org



Processed: limit source to lintian, tagging 711988

2013-09-15 Thread Debian Bug Tracking System
Processing commands for cont...@bugs.debian.org:

> #lintian (2.5.18) UNRELEASED; urgency=low
> #
> #  * data/scripts/interpreters:
> #+ [NT] Use "python:any | python-minimal:any" as dependency
> #  for python scripts.  This fixes a false-positive for
> #  python-script-but-no-python-dep when dh_python2 generates
> #  a dependency of "python:any".  Thanks to Francois Gouget
> #  for the report.  (Closes: #711988)
> #
> limit source lintian
Limiting to bugs with field 'source' containing at least one of 'lintian'
Limit currently set to 'source':'lintian'

> tags 711988 + pending
Bug #711988 [lintian] lintian: Lintian does not recognize python:any dependency
Added tag(s) pending.
> thanks
Stopping processing here.

Please contact me if you need assistance.
-- 
711988: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=711988
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems


--
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.137928454625869.transcr...@bugs.debian.org



[lintian] branch master updated (7f0ac08 -> ddc0824)

2013-09-15 Thread Niels Thykier
This is an automated email from the git hooks/post-receive script.

nthykier pushed a change to branch master
in repository lintian.

  from  7f0ac08   t/scripts: Rename two "slow" tests so they are run earlier
   new  ddc0824   c/apache2.pm: Fix F-P when module name had uppercase 
letters

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "adds" were already present in the repository and have only
been added to this reference.


Summary of changes:
 checks/apache2.pm |4 +++-
 debian/changelog  |5 +
 2 files changed, 8 insertions(+), 1 deletion(-)

-- 
Alioth's /usr/local/bin/git-commit-notice on 
/srv/git.debian.org/git/lintian/lintian.git


-- 
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/20130915211734.2919.45...@vasks.debian.org



Processed: severity of 711988 is important

2013-09-15 Thread Debian Bug Tracking System
Processing commands for cont...@bugs.debian.org:

> severity 711988 important
Bug #711988 [lintian] lintian: Lintian does not recognize python:any dependency
Severity set to 'important' from 'normal'
> thanks
Stopping processing here.

Please contact me if you need assistance.
-- 
711988: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=711988
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems


--
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.13792804635803.transcr...@bugs.debian.org



Processed: limit source to lintian, tagging 723003

2013-09-15 Thread Debian Bug Tracking System
Processing commands for cont...@bugs.debian.org:

> #lintian (2.5.18) UNRELEASED; urgency=low
> #
> #  * checks/apache2.pm:
> #+ [NT] Fix false-positive warning about non-standard package
> #  name when the Apache2 module contained uppercase letters.
> #  Thanks to Jonas Smedegaard for the report.
> #  (Closes: #723003)
> #
> limit source lintian
Limiting to bugs with field 'source' containing at least one of 'lintian'
Limit currently set to 'source':'lintian'

> tags 723003 + pending
Bug #723003 [lintian] lintian: non-standard-apache2-module-package-name should 
ignore case
Added tag(s) pending.
> thanks
Stopping processing here.

Please contact me if you need assistance.
-- 
723003: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=723003
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems


--
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.13792798631768.transcr...@bugs.debian.org



[lintian] 01/01: c/apache2.pm: Fix F-P when module name had uppercase letters

2013-09-15 Thread Niels Thykier
This is an automated email from the git hooks/post-receive script.

nthykier pushed a commit to branch master
in repository lintian.

commit ddc0824f601900802c65bcf385185df09f7e9244
Author: Niels Thykier 
Date:   Sun Sep 15 23:16:36 2013 +0200

c/apache2.pm: Fix F-P when module name had uppercase letters

Signed-off-by: Niels Thykier 
---
 checks/apache2.pm |4 +++-
 debian/changelog  |5 +
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/checks/apache2.pm b/checks/apache2.pm
index 7dd0a80..c746b36 100644
--- a/checks/apache2.pm
+++ b/checks/apache2.pm
@@ -133,7 +133,9 @@ sub check_module_package {
 
 # We want packages to be follow our naming scheme. Modules should be named
 # libapache2-mod- if it ships a mod_foo.so
-my $expected_name = 'libapache2-' . $module;
+# NB: Some modules have uppercase letters in them (e.g. Ruwsgi), but
+# obviously the package should be in all lowercase.
+my $expected_name = 'libapache2-' . lc($module);
 
 # Package depends on apache2-api-MMDD
 my $seen_api_dependency = 0;
diff --git a/debian/changelog b/debian/changelog
index 57cf039..a2a298f 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -2,6 +2,11 @@ lintian (2.5.18) UNRELEASED; urgency=low
 
   XXX: generate tag summary with private/generate-tag-summary
 
+  * checks/apache2.pm:
++ [NT] Fix false-positive warning about non-standard package
+  name when the Apache2 module contained uppercase letters.
+  Thanks to Jonas Smedegaard for the report.
+  (Closes: #723003)
   * checks/changelog.{desc,pm}:
 + [NT] Check that the weekday of the latest changelog entry
   is consistent with the date.  Thanks to Raphael Champeimont

-- 
Alioth's /usr/local/bin/git-commit-notice on 
/srv/git.debian.org/git/lintian/lintian.git


-- 
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/e1vljhg-m7...@vasks.debian.org



Bug#711988: python-script-but-no-python-dep while package depends on python:any

2013-09-15 Thread Emilien Klein
Hi Lintian maintainers,

I've encountered the same issue, it looks like Lintian doesn't recognize
python:any as being a satisfactory dependency on Python:

After having updated my Sid system (Lintian v2.5.17), while rebuilding the
gnuhealth package I get the following error:

E: gnuhealth-client: python-script-but-no-python-dep
usr/share/gnuhealth-client/gnuhealth-client.py

My control file lists the following:
Depends: ${python:Depends}, ${misc:Depends},
 tryton-client (>= 2.8~), tryton-client (<< 2.9~)

And, if I open the generated .deb and look at the /DEBIAN/control file:
Depends: python:any (>= 2.6.6-7~), python:any (<< 3.1), tryton-client (>=
2.8~), tryton-client (<< 2.9~)

Please advise.
   +Emilien


Bug#718169: marked as done (Leading underscore in desktop file entry not recognized)

2013-09-15 Thread Debian Bug Tracking System
Your message dated Sun, 15 Sep 2013 12:00:07 +0200
with message-id 

and subject line Re: Bug#718169:
has caused the Debian Bug report #718169,
regarding Leading underscore in desktop file entry not recognized
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact ow...@bugs.debian.org
immediately.)


-- 
718169: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=718169
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
--- Begin Message ---
Package: lintian

Running Lintian 2.5.15 in sid, I got this warning:

I: gnuhealth-client: desktop-entry-lacks-keywords-entry
usr/share/applications/gnuhealth-client.desktop

I follow the links from the info on the Lintian report page [0], and find
the following example from the Gnome website [1]:

_Keywords=chat;messaging;im;irc;voip;

According to that page, "Maintainers/developers must set a preceding
underscore in "_Keywords" to inform intltool that this field should be
extracted, and its translations merged back into the .desktop.in file."

When running Lintian again, instead of the warning disappearing, a new one
appears ;)


W: gnuhealth-client: desktop-entry-contains-unknown-key
usr/share/applications/gnuhealth-client.desktop:11 _Keywords
I: gnuhealth-client: desktop-entry-lacks-keywords-entry
usr/share/applications/gnuhealth-client.desktop


When removing the underscore, all warnings disappear.
I assume Lintian does not recognize the leading underscore in desktop file
entries (or maybe just for the newer Keywords)?

Cheers,
   +Emilien

[0] http://lintian.debian.org/tags/desktop-entry-lacks-keywords-entry.html
[1] https://wiki.gnome.org/GnomeGoals/DesktopFileKeywords
--- End Message ---
--- Begin Message ---
I agree this is non-standard. I don't have currently time to push to
standard body, so closing this bug.
--- End Message ---


Bug#723003: lintian: non-standard-apache2-module-package-name should ignore case

2013-09-15 Thread Jonas Smedegaard
Package: lintian
Version: 2.5.17
Severity: normal

Binary package libapache2-mod-ruwsgi contains Apache2 module Ruwsgi.

Seems to me that non-standard-apache2-module-package-name should compare
only after lowercasing (and possibly that Apache Policy should be
improved to reflect tnis, but arguably that's an implicit contraint,
since it is talking about package names).

 - Jonas


-- 
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/20130915093732.27253.86202.report...@auryn.jones.dk