date:20171220

Bug#884870: lintian: vcs-field-has-unexpected-spaces and vcswatch don't agree

2017-12-20 Thread Mattia Rizzolo

On Wed, Dec 20, 2017 at 08:11:23PM +0100, Christoph Berg wrote:
> Fwiw, the -b syntax was not invented by vcswatch, it was in use in the
> archive before I wrote the service. I can't find a place where it is
> documented (I thought it was debcheckout(1), but it's not in there),
> but the idea behind it is that you can you paste the Vcs-Git header
> content to "git clone" and it will do the right thing.

It's in Debian Policy of course! :)
https://www.debian.org/doc/debian-policy/#version-control-system-vcs-fields
|In the case of Git, the value consists of a URL, optionally followed
|by the word -b and the name of a branch in the indicated repository,
|following the syntax of the git clone command. If no branch is
|specified, the packaging should be on the default branch.

> (I'm still pondering how a syntax for "package is located in this
> subdirectory" should look like, but as that's not supported by "git
> clone", I couldn't think of anything yet that would at least look like
> the -b syntax. There's a need for it, though.)

There was a very long discussion about this in d-d@ some months (years
already, maybe!?) while discussing a detail from DEP-14.

-- 
regards,
Mattia Rizzolo

GPG Key: 66AE 2B4A FCCF 3F52 DA18  4D18 4B04 3FCD B944 4540  .''`.
more about me:  https://mapreri.org : :'  :
Launchpad user: https://launchpad.net/~mapreri  `. `'`
Debian QA page: https://qa.debian.org/developer.php?login=mattia  `-

signature.asc
Description: PGP signature

Bug#884870: marked as done (lintian: vcs-field-has-unexpected-spaces and vcswatch don't agree)

2017-12-20 Thread Debian Bug Tracking System

Your message dated Wed, 20 Dec 2017 18:26:42 -0500
with message-id 

and subject line Re: Bug#884870: lintian: vcs-field-has-unexpected-spaces and 
vcswatch don't agree
has caused the Debian Bug report #884870,
regarding lintian: vcs-field-has-unexpected-spaces and vcswatch don't agree
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.)

-- 
884870: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=884870
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
--- Begin Message ---
Source: lintian
Version: 2.5.65

I changed webkit2gtk's Vcs-Git field from
Vcs-Git: https://anonscm.debian.org/git/pkg-webkit/webkit.git

to
Vcs-Git: https://anonscm.debian.org/git/pkg-webkit/webkit.git -b wk2/unstable

in order to satisfy
https://qa.debian.org/cgi-bin/vcswatch?package=webkit2gtk

which complained that it couldn't find the current debian/changelog in
HEAD. (Maybe it's time to change HEAD, but this bug is also a problem
for experimental and Stable Update branches.)

Now, I think (lintian's website is recovering from a typo) that this
now causes lintian to emit vcs-field-has-unexpected-spaces

I think the " -b BRANCHNAME" suffix should be considered valid syntax
for Vcs-Git.

Thanks,
Jeremy Bicha
--- End Message ---
--- Begin Message ---
On Wed, Dec 20, 2017 at 3:23 PM, Chris Lamb  wrote:
> Something else must be going on as this is indeed supported. Here is the
> code in question:

Never mind. Maybe I had a typo locally since I can't reproduce it now.

Thanks,
Jeremy Bicha--- End Message ---

Bug#884798: lintian could complain about pkg-config uses that fail to use $ac_tool_prefix

2017-12-20 Thread Chris Lamb

Hi Helmut,

> If yes, what would be a good check? Should it just check the toplevel
> configure.ac and configure.in or any file found on any directory level?

This is my primary concern — warning about broken configure.{am,in} in
code copies etc. that are not used would just be annoying.

It might be safe to simply check ./configure.{am,in} or perhaps a
hardcoded list of subdirs such as src/configure.{am,in}. What do you
think?

> I haven't performed a whole archive-analysis yet, so I cannot tell
> about the amount of fpos to be expected.

Would this be easy for you to do? Please go ahead if so and let us know.


Best wishes,

-- 
  ,''`.
 : :'  : Chris Lamb
 `. `'`  la...@debian.org / chris-lamb.co.uk
   `-

Processed: Re: Bug#505857: lintian: false positive debian-watch-file-should-mangle-version

2017-12-20 Thread Debian Bug Tracking System

Processing commands for cont...@bugs.debian.org:

> clone 505857 -1
Bug #505857 [lintian] [checks/watch-file] false positive 
debian-watch-file-should-mangle-version
Bug 505857 cloned as bug 884880
> reassign -1 devscripts 2.17.11
Bug #884880 [lintian] [checks/watch-file] false positive 
debian-watch-file-should-mangle-version
Bug reassigned from package 'lintian' to 'devscripts'.
No longer marked as found in versions lintian/2.0.0 and lintian/2.5.65.
No longer marked as fixed in versions lintian/2.5.54.
Bug #884880 [devscripts] [checks/watch-file] false positive 
debian-watch-file-should-mangle-version
Marked as found in versions devscripts/2.17.11.
> severity -1 wishlist
Bug #884880 [devscripts] [checks/watch-file] false positive 
debian-watch-file-should-mangle-version
Severity set to 'wishlist' from 'minor'
> retitle -1 [uscan] Please clarify that spaces in debian/watch "opts" are only 
> valid in version 4
Bug #884880 [devscripts] [checks/watch-file] false positive 
debian-watch-file-should-mangle-version
Changed Bug title to '[uscan] Please clarify that spaces in debian/watch "opts" 
are only valid in version 4' from '[checks/watch-file] false positive 
debian-watch-file-should-mangle-version'.
> thanks
Stopping processing here.

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

Bug#505857: lintian: false positive debian-watch-file-should-mangle-version

2017-12-20 Thread Chris Lamb

clone 505857 -1
reassign -1 devscripts 2.17.11
severity -1 wishlist
retitle -1 [uscan] Please clarify that spaces in debian/watch "opts" are only 
valid in version 4
thanks

Hi Christoph,

> The warning goes away when I remove the space after the comma.
> 
> The version with the space is the syntax documented in uscan(1):
> 
>   Multiple options option1, option2, option3, ... can be set as
>   opts="option1, option2, option3,  ...  " .

Only for version=4. :)


Regards,

-- 
  ,''`.
 : :'  : Chris Lamb
 `. `'`  la...@debian.org / chris-lamb.co.uk
   `-

Build failed in Jenkins: lintian-tests_stretch #1279

2017-12-20 Thread jenkins

See 


Changes:

[lamby] Add a regression test for a potential false positive in the

--
[...truncated 298.84 KB...]
tests::java-jars:dh_auto_configure
tests::java-jars:jh_linkjars
tests::java-jars:dh_auto_build
tests::java-jars:debian/rules override_jh_build
tests::java-jars: make[1]: Entering directory 
'/tmp/testrun/debian/test-out/tests/java-jars/java-jars-1.0'
tests::java-jars: jh_build --javacopts='-target 1.7' --javacopts='-source 1.7'
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javac -g -cp :debian/_jh_build.testa -d 
debian/_jh_build.testa -target 1.7 -source 1.7
tests::java-jars: warning: [options] bootstrap class path not set in 
conjunction with -source 1.7
tests::java-jars: 1 warning
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javadoc -locale en_US -classpath 
:debian/_jh_build.testa -d debian/_jh_build.javadoc/api -quiet -notimestamp 
-source 1.5
tests::java-jars: /usr/lib/jvm/default-java/bin/jar cfm 
/tmp/testrun/debian/test-out/tests/java-jars/java-jars-1.0/testa.jar 
../_jh_manifest.testa org
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javac -g -cp :debian/_jh_build.testb -d 
debian/_jh_build.testb -target 1.7 -source 1.7
tests::java-jars: warning: [options] bootstrap class path not set in 
conjunction with -source 1.7
tests::java-jars: 1 warning
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javadoc -locale en_US -classpath 
:debian/_jh_build.testb -d debian/_jh_build.javadoc/api -quiet -notimestamp 
-source 1.5
tests::java-jars: /usr/lib/jvm/default-java/bin/jar cfm 
/tmp/testrun/debian/test-out/tests/java-jars/java-jars-1.0/testb.jar 
../_jh_manifest.testb org
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javac -g -cp :debian/_jh_build.testc -d 
debian/_jh_build.testc -target 1.7 -source 1.7
tests::java-jars: warning: [options] bootstrap class path not set in 
conjunction with -source 1.7
tests::java-jars: 1 warning
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javadoc -locale en_US -classpath 
:debian/_jh_build.testc -d debian/_jh_build.javadoc/api -quiet -notimestamp 
-source 1.5
tests::java-jars: /usr/lib/jvm/default-java/bin/jar cfm 
/tmp/testrun/debian/test-out/tests/java-jars/java-jars-1.0/testc.jar 
../_jh_manifest.testc org
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javac -g -cp :debian/_jh_build.testd -d 
debian/_jh_build.testd -target 1.7 -source 1.7
tests::java-jars: warning: [options] bootstrap class path not set in 
conjunction with -source 1.7
tests::java-jars: 1 warning
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javadoc -locale en_US -classpath 
:debian/_jh_build.testd -d debian/_jh_build.javadoc/api -quiet -notimestamp 
-source 1.5
tests::java-jars: /usr/lib/jvm/default-java/bin/jar cfm 
/tmp/testrun/debian/test-out/tests/java-jars/java-jars-1.0/testd.jar 
../_jh_manifest.testd org
tests::java-jars: cp testa.jar testb.jar
tests::java-jars: unzip testa.jar
tests::java-jars: Archive:  testa.jar
tests::java-jars: replace META-INF/MANIFEST.MF? [y]es, [n]o, [A]ll, [N]one, 
[r]ename:  NULL
tests::java-jars: (EOF or read error, treating as "[N]one" ...)
tests::java-jars: debian/rules:6: recipe for target 'override_jh_build' failed
tests::java-jars: make[1]: *** [override_jh_build] Error 1
tests::java-jars: make[1]: Leaving directory 
'/tmp/testrun/debian/test-out/tests/java-jars/java-jars-1.0'
tests::java-jars: debian/rules:3: recipe for target 'binary' failed
tests::java-jars: make: *** [binary] Error 2
tests::java-jars: dpkg-buildpackage: error: fakeroot debian/rules binary gave 
error exit status 2
tests::java-jars:  END BUILD LOG
error tests::java-jars: internal error: cd 
/tmp/testrun/debian/test-out/tests/java-jars/java-jars-1.0 && dpkg-buildpackage 
-rfakeroot -us -uc -d -iNEVER_MATCH_ANYTHING -INEVER_MATCH_ANYTHING 
--source-option=--auto-commit 
>/tmp/testrun/debian/test-out/tests/java-jars/build.java-jars 2>&1 at 
t/runtests line 466.


tests::scripts-exec:  START BUILD LOG
tests::scripts-exec: dpkg-buildpackage: warning: using a gain-root-command 
while being root
tests::scripts-exec: dpkg-buildpackage: info: source package scripts-exec
tests::scripts-exec: dpkg-buildpackage: info: source version 1.0
tests::scripts-exec: dpkg-buildpackage: info: source distribution unstable
tests::scripts-exec: dpkg-buildpackage: info: source changed by Debian Lintian 
Maintainers

Bug#884870: lintian: vcs-field-has-unexpected-spaces and vcswatch don't agree

2017-12-20 Thread Chris Lamb

Hi Jeremy,

> Now, I think (lintian's website is recovering from a typo) that this
> now causes lintian to emit vcs-field-has-unexpected-spaces
> 
> I think the " -b BRANCHNAME" suffix should be considered valid syntax
> for Vcs-Git.

Something else must be going on as this is indeed supported. Here is the
code in question:

# git uri followed by optional " -b " + branchname:
git => sub { return shift =~ /^(.+?)(?:\s+-b\s+(\S*))?$/;},

… and I've even just committed a (passing) test for this:

  
https://anonscm.debian.org/git/lintian/lintian.git/commit/?id=8e1ba53fa85469ab4f2e67e7514b3a800c0a6004

:)


Best wishes,

-- 
  ,''`.
 : :'  : Chris Lamb
 `. `'`  la...@debian.org / chris-lamb.co.uk
   `-

[lintian] 01/01: Add a regression test for a potential false positive in the "vcs-field-has-unexpected-spaces" tag. (Ref: #884870)

2017-12-20 Thread Chris Lamb

This is an automated email from the git hooks/post-receive script.

lamby pushed a commit to branch master
in repository lintian.

commit 8e1ba53fa85469ab4f2e67e7514b3a800c0a6004
Author: Chris Lamb 
Date:   Wed Dec 20 20:22:32 2017 +

Add a regression test for a potential false positive in the 
"vcs-field-has-unexpected-spaces" tag. (Ref: #884870)
---
 debian/changelog|  3 +++
 .../debian/debian/control.in| 17 +
 t/tests/fields-malformed-vcs-fields-unrel/desc  |  5 +
 t/tests/fields-malformed-vcs-fields-unrel/tags  |  0
 4 files changed, 25 insertions(+)

diff --git a/debian/changelog b/debian/changelog
index 331a40c..c53ed25 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -37,6 +37,9 @@ lintian (2.5.66) UNRELEASED; urgency=medium
 + [CL] Correct invalid "else if" syntax with "elsif". Thanks to Uwe
   Kleine-König for the report.
 
+  * t/tests/fields-malformed-vcs-fields-unrel:
++ [CL] Add a regression test for a potential false positive in the
+  "vcs-field-has-unexpected-spaces" tag.  (Ref: #884870)
   * t/tests/files-multiarch-foreign-files:
 + [CL] Don't hardcode architecture triplet to fix FTBFS on non-amd64
   architectures.  (Closes: #884683)
diff --git a/t/tests/fields-malformed-vcs-fields-unrel/debian/debian/control.in 
b/t/tests/fields-malformed-vcs-fields-unrel/debian/debian/control.in
new file mode 100644
index 000..8a89a48
--- /dev/null
+++ b/t/tests/fields-malformed-vcs-fields-unrel/debian/debian/control.in
@@ -0,0 +1,17 @@
+Source: {$source}
+Priority: optional
+Section: {$section}
+Maintainer: {$author}
+Standards-Version: {$standards_version}
+Build-Depends: {$build_depends}
+Rules-Requires-Root: no
+Vcs-Git: https://anonscm.debian.org/test/test.git -b valid/syntax
+
+Package: {$source}
+Architecture: {$architecture}
+Depends: $\{misc:Depends\}
+Description: {$description}
+ This is a test package designed to exercise some feature or tag of
+ Lintian.  It is part of the Lintian test suite and may do very odd
+ things.  It should not be installed like a regular package.  It may
+ be an empty package.
diff --git a/t/tests/fields-malformed-vcs-fields-unrel/desc 
b/t/tests/fields-malformed-vcs-fields-unrel/desc
new file mode 100644
index 000..2f9aab3
--- /dev/null
+++ b/t/tests/fields-malformed-vcs-fields-unrel/desc
@@ -0,0 +1,5 @@
+Testname: fields-malformed-vcs-fields-unrel
+Description: Test malformed VCS-* fields
+Version: 1.0
+Test-Against:
+ vcs-field-has-unexpected-spaces
diff --git a/t/tests/fields-malformed-vcs-fields-unrel/tags 
b/t/tests/fields-malformed-vcs-fields-unrel/tags
new file mode 100644
index 000..e69de29

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

[lintian] branch master updated (edd6515 -> 8e1ba53)

2017-12-20 Thread Chris Lamb

This is an automated email from the git hooks/post-receive script.

lamby pushed a change to branch master
in repository lintian.

  from  edd6515   Check for override_dh_clean targets that are missing 
calls to dh_clean. Thanks to Andreas Beckmann for the idea. (Closes: #884817)
   new  8e1ba53   Add a regression test for a potential false positive in 
the "vcs-field-has-unexpected-spaces" tag. (Ref: #884870)

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:
 debian/changelog | 3 +++
 .../debian/debian/control.in | 3 +--
 t/tests/fields-malformed-vcs-fields-unrel/desc   | 5 +
 .../fields-malformed-vcs-fields-unrel}/tags  | 0
 4 files changed, 9 insertions(+), 2 deletions(-)
 copy t/tests/{description-homepage => 
fields-malformed-vcs-fields-unrel}/debian/debian/control.in (88%)
 create mode 100644 t/tests/fields-malformed-vcs-fields-unrel/desc
 copy t/{debs/deb-format-record-size => 
tests/fields-malformed-vcs-fields-unrel}/tags (100%)

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

Build failed in Jenkins: lintian-tests_stretch #1278

2017-12-20 Thread jenkins

See 


Changes:

[lamby] Check for override_dh_clean targets that are missing calls to dh_clean.

--
[...truncated 298.86 KB...]
tests::java-jars:dh_auto_configure
tests::java-jars:jh_linkjars
tests::java-jars:dh_auto_build
tests::java-jars:debian/rules override_jh_build
tests::java-jars: make[1]: Entering directory 
'/tmp/testrun/debian/test-out/tests/java-jars/java-jars-1.0'
tests::java-jars: jh_build --javacopts='-target 1.7' --javacopts='-source 1.7'
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javac -g -cp :debian/_jh_build.testa -d 
debian/_jh_build.testa -target 1.7 -source 1.7
tests::java-jars: warning: [options] bootstrap class path not set in 
conjunction with -source 1.7
tests::java-jars: 1 warning
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javadoc -locale en_US -classpath 
:debian/_jh_build.testa -d debian/_jh_build.javadoc/api -quiet -notimestamp 
-source 1.5
tests::java-jars: /usr/lib/jvm/default-java/bin/jar cfm 
/tmp/testrun/debian/test-out/tests/java-jars/java-jars-1.0/testa.jar 
../_jh_manifest.testa org
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javac -g -cp :debian/_jh_build.testb -d 
debian/_jh_build.testb -target 1.7 -source 1.7
tests::java-jars: warning: [options] bootstrap class path not set in 
conjunction with -source 1.7
tests::java-jars: 1 warning
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javadoc -locale en_US -classpath 
:debian/_jh_build.testb -d debian/_jh_build.javadoc/api -quiet -notimestamp 
-source 1.5
tests::java-jars: /usr/lib/jvm/default-java/bin/jar cfm 
/tmp/testrun/debian/test-out/tests/java-jars/java-jars-1.0/testb.jar 
../_jh_manifest.testb org
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javac -g -cp :debian/_jh_build.testc -d 
debian/_jh_build.testc -target 1.7 -source 1.7
tests::java-jars: warning: [options] bootstrap class path not set in 
conjunction with -source 1.7
tests::java-jars: 1 warning
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javadoc -locale en_US -classpath 
:debian/_jh_build.testc -d debian/_jh_build.javadoc/api -quiet -notimestamp 
-source 1.5
tests::java-jars: /usr/lib/jvm/default-java/bin/jar cfm 
/tmp/testrun/debian/test-out/tests/java-jars/java-jars-1.0/testc.jar 
../_jh_manifest.testc org
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javac -g -cp :debian/_jh_build.testd -d 
debian/_jh_build.testd -target 1.7 -source 1.7
tests::java-jars: warning: [options] bootstrap class path not set in 
conjunction with -source 1.7
tests::java-jars: 1 warning
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javadoc -locale en_US -classpath 
:debian/_jh_build.testd -d debian/_jh_build.javadoc/api -quiet -notimestamp 
-source 1.5
tests::java-jars: /usr/lib/jvm/default-java/bin/jar cfm 
/tmp/testrun/debian/test-out/tests/java-jars/java-jars-1.0/testd.jar 
../_jh_manifest.testd org
tests::java-jars: cp testa.jar testb.jar
tests::java-jars: unzip testa.jar
tests::java-jars: Archive:  testa.jar
tests::java-jars: replace META-INF/MANIFEST.MF? [y]es, [n]o, [A]ll, [N]one, 
[r]ename:  NULL
tests::java-jars: (EOF or read error, treating as "[N]one" ...)
tests::java-jars: debian/rules:6: recipe for target 'override_jh_build' failed
tests::java-jars: make[1]: *** [override_jh_build] Error 1
tests::java-jars: make[1]: Leaving directory 
'/tmp/testrun/debian/test-out/tests/java-jars/java-jars-1.0'
tests::java-jars: debian/rules:3: recipe for target 'binary' failed
tests::java-jars: make: *** [binary] Error 2
tests::java-jars: dpkg-buildpackage: error: fakeroot debian/rules binary gave 
error exit status 2
tests::java-jars:  END BUILD LOG
error tests::java-jars: internal error: cd 
/tmp/testrun/debian/test-out/tests/java-jars/java-jars-1.0 && dpkg-buildpackage 
-rfakeroot -us -uc -d -iNEVER_MATCH_ANYTHING -INEVER_MATCH_ANYTHING 
--source-option=--auto-commit 
>/tmp/testrun/debian/test-out/tests/java-jars/build.java-jars 2>&1 at 
t/runtests line 466.

...
tests::scripts-exec:  START BUILD LOG
tests::scripts-exec: dpkg-buildpackage: warning: using a gain-root-command 
while being root
tests::scripts-exec: dpkg-buildpackage: info: source package scripts-exec
tests::scripts-exec: dpkg-buildpackage: info: source version 1.0
tests::scripts-exec: dpkg-buildpackage: info: source distribution unstable
tests::scripts-exec: dpkg-buildpackage: info: source changed by Debian Lintian

Bug#884870: lintian: vcs-field-has-unexpected-spaces and vcswatch don't agree

2017-12-20 Thread Christoph Berg

Re: Jeremy Bicha 2017-12-20 

> to
> Vcs-Git: https://anonscm.debian.org/git/pkg-webkit/webkit.git -b wk2/unstable

> I think the " -b BRANCHNAME" suffix should be considered valid syntax
> for Vcs-Git.

Fwiw, the -b syntax was not invented by vcswatch, it was in use in the
archive before I wrote the service. I can't find a place where it is
documented (I thought it was debcheckout(1), but it's not in there),
but the idea behind it is that you can you paste the Vcs-Git header
content to "git clone" and it will do the right thing.

(I'm still pondering how a syntax for "package is located in this
subdirectory" should look like, but as that's not supported by "git
clone", I couldn't think of anything yet that would at least look like
the -b syntax. There's a need for it, though.)

Christoph

Bug#884817: lintian: check for override_dh_clean target missing call to dh_clean

2017-12-20 Thread Chris Lamb

tags 884817 + pending
thanks

Fixed in Git:

  
https://anonscm.debian.org/git/lintian/lintian.git/commit/?id=edd65158f2b8f02760cd2c5ef377b0e04188fb3f


Regards,

-- 
  ,''`.
 : :'  : Chris Lamb
 `. `'`  la...@debian.org / chris-lamb.co.uk
   `-

Bug#884870: lintian: vcs-field-has-unexpected-spaces and vcswatch don't agree

2017-12-20 Thread Jeremy Bicha

Source: lintian
Version: 2.5.65

I changed webkit2gtk's Vcs-Git field from
Vcs-Git: https://anonscm.debian.org/git/pkg-webkit/webkit.git

to
Vcs-Git: https://anonscm.debian.org/git/pkg-webkit/webkit.git -b wk2/unstable

in order to satisfy
https://qa.debian.org/cgi-bin/vcswatch?package=webkit2gtk

which complained that it couldn't find the current debian/changelog in
HEAD. (Maybe it's time to change HEAD, but this bug is also a problem
for experimental and Stable Update branches.)

Now, I think (lintian's website is recovering from a typo) that this
now causes lintian to emit vcs-field-has-unexpected-spaces

I think the " -b BRANCHNAME" suffix should be considered valid syntax
for Vcs-Git.

Thanks,
Jeremy Bicha

Processed: Re: lintian: check for override_dh_clean target missing call to dh_clean

2017-12-20 Thread Debian Bug Tracking System

Processing commands for cont...@bugs.debian.org:

> tags 884817 + pending
Bug #884817 [lintian] lintian: check for override_dh_clean target missing call 
to dh_clean
Added tag(s) pending.
> thanks
Stopping processing here.

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

Bug#884848: lintian: false positive for source-includes-file-in-files-excluded

2017-12-20 Thread Chris Lamb


Hi,

> lintian: false positive for source-includes-file-in-files-excluded

Indeed. The following patch *should* fix it IMHO:


  commit c807001cbbf101f74d227cde4482f535908e13a6
  Author: Chris Lamb 
  Date:   Wed Dec 20 18:17:11 2017 +
  
  collection/src-orig-index: Correct reference to generated filename.
  
   collection/src-orig-index | 6 +++---
   debian/changelog  | 3 +++
   2 files changed, 6 insertions(+), 3 deletions(-)
  
  commit 5c2a7611aaccc1ef835cca058ca1fd23abe7d1ed
  Author: Chris Lamb 
  Date:   Wed Dec 20 18:16:09 2017 +
  
  Use the list of files in the orig tarball rather than in the regular 
index to prevent false positives in the tag 
source-includes-file-in-files-excluded when a patch system adds files that have 
been removed. (Closes: #884848)
  
   checks/source-copyright.desc | 2 +-
   checks/source-copyright.pm   | 2 +-
   debian/changelog | 5 +
   3 files changed, 7 insertions(+), 2 deletions(-)


.. but it fails with:

warning: collect info file-info about package idlastro failed
warning: skipping check of source package idlastro
warning: collect info file-info about package gdl-astrolib failed
warning: skipping check of binary package gdl-astrolib
Can't 
readlink('/tmp/temp-lintian-lab-GwZNgZ4Df3/pool/i/idlastro/idlastro_2017.08.02+dfsg-1_source-all_changes/changes'):
 No such file or directory at 
/home/lamby/git/debian/lintian/lintian/checks/changes-file.pm line 192
internal error: cannot run changes-file check on package 
changes:idlastro/2017.08.02+dfsg-1/source_all
warning: skipping check of changes:idlastro/2017.08.02+dfsg-1/source_all

Presumably some kind of conflict between the two collection scripts.. Hm.


Regards,

-- 
  ,''`.
 : :'  : Chris Lamb
 `. `'`  la...@debian.org / chris-lamb.co.uk
   `-
From c807001cbbf101f74d227cde4482f535908e13a6 Mon Sep 17 00:00:00 2001
From: Chris Lamb 
Date: Wed, 20 Dec 2017 18:17:11 +
Subject: [PATCH 1/2] collection/src-orig-index: Correct reference to generated
 filename.

---
 collection/src-orig-index | 6 +++---
 debian/changelog  | 3 +++
 2 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/collection/src-orig-index b/collection/src-orig-index
index 7799e1a2c..f4efcdd77 100755
--- a/collection/src-orig-index
+++ b/collection/src-orig-index
@@ -39,14 +39,14 @@ sub collect {
 my ($pkg, $type, $dir) = @_;
 my $info = Lintian::Collect->new($pkg, $type, $dir);
 
-if (-f "$dir/orig-index.gz") {
-unlink("$dir/orig-index.gz");
+if (-f "$dir/src-orig-index.gz") {
+unlink("$dir/src-orig-index.gz");
 }
 
 # Nothing to do for native packages where the two indices are
 # identical.
 if ($info->native) {
-link("$dir/index.gz", "$dir/orig-index.gz");
+link("$dir/index.gz", "$dir/src-orig-index.gz");
 return;
 }
 
diff --git a/debian/changelog b/debian/changelog
index b73da400d..7c91ad2b2 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -19,6 +19,9 @@ lintian (2.5.66) UNRELEASED; urgency=medium
   (ie. "2017-11-30 < 2017-11-30"). Thanks to Andrea Bolognani
   e...@kiyuko.org> for the report.  (Closes: #884785)
 
+  * collection/src-orig-index:
++ [CL] Correct reference to generated filename.
+
   * data/debhelper/compat-level:
 + [MR] Bump the experimental debhelper compat level to 12.
   (Closes: #884678)
-- 
2.15.1

From 5c2a7611aaccc1ef835cca058ca1fd23abe7d1ed Mon Sep 17 00:00:00 2001
From: Chris Lamb 
Date: Wed, 20 Dec 2017 18:16:09 +
Subject: [PATCH 2/2] Use the list of files in the orig tarball rather than in
 the regular index to prevent false positives in the tag
 source-includes-file-in-files-excluded when a patch system adds files that
 have been removed. (Closes: #884848)

---
 checks/source-copyright.desc | 2 +-
 checks/source-copyright.pm   | 2 +-
 debian/changelog | 5 +
 3 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/checks/source-copyright.desc b/checks/source-copyright.desc
index 33666..8d47c5afb 100644
--- a/checks/source-copyright.desc
+++ b/checks/source-copyright.desc
@@ -2,7 +2,7 @@ Check-Script: source-copyright
 Author: Jakub Wilk 
 Abbrev: scpy
 Type: source
-Needs-Info: unpacked
+Needs-Info: unpacked, src-orig-index
 Info: This script checks if a source package conforms to policy
  with regard to copyright files.
  .
diff --git a/checks/source-copyright.pm b/checks/source-copyright.pm
index a5f66db47..352a391c0 100644
--- a/checks/source-copyright.pm
+++ b/checks/source-copyright.pm
@@ -615,7 +615,7 @@ sub get_all_files {
 
 sub check_files_excluded {
 my ($info, $excluded) = @_;
-my @files = get_all_files($info);
+my @files = grep { $_->is_file } $info->sorted_orig_index;
 my @wildcards = split /[\n\t ]+/, $excluded;
 for my $wildcard (@wildcards) {
 $wildcard =~

[lintian] 01/01: Check for override_dh_clean targets that are missing calls to dh_clean. Thanks to Andreas Beckmann for the idea. (Closes: #884817)

2017-12-20 Thread Chris Lamb

This is an automated email from the git hooks/post-receive script.

lamby pushed a commit to branch master
in repository lintian.

commit edd65158f2b8f02760cd2c5ef377b0e04188fb3f
Author: Chris Lamb 
Date:   Wed Dec 20 17:19:48 2017 +

Check for override_dh_clean targets that are missing calls to dh_clean. 
Thanks to Andreas Beckmann for the idea. (Closes: #884817)
---
 checks/rules.desc | 11 +++
 checks/rules.pm   | 11 +--
 debian/changelog  |  3 +++
 .../debian/debian/rules   |  8 
 .../rules-override-dh-clean-does-not-call-dh-clean-unrel/desc |  5 +
 .../rules-override-dh-clean-does-not-call-dh-clean-unrel/tags |  0
 .../debian/debian/rules   |  7 +++
 t/tests/rules-override-dh-clean-does-not-call-dh-clean/desc   |  5 +
 t/tests/rules-override-dh-clean-does-not-call-dh-clean/tags   |  1 +
 9 files changed, 49 insertions(+), 2 deletions(-)

diff --git a/checks/rules.desc b/checks/rules.desc
index 483d0d4..86b57fe 100644
--- a/checks/rules.desc
+++ b/checks/rules.desc
@@ -282,3 +282,14 @@ Info: The debian/rules file for this package 
appears to set
  noopt.
  .
  This has been obsoleted in favour of dpkg-buildflags.
+
+Tag: override_dh_clean-does-not-call-dh_clean
+Severity: normal
+Certainty: certain
+Info: The debian/rules file for this package uses
+ the override_dh_clean but this target does not reference
+ dh_clean. This can result in packages not cleaning up properly via
+ debian/rules clean.
+ .
+ Please add a call to dh_clean.
+Ref: #884419, #884815
diff --git a/checks/rules.pm b/checks/rules.pm
index cf34465..5c0754b 100644
--- a/checks/rules.pm
+++ b/checks/rules.pm
@@ -19,7 +19,7 @@ use warnings;
 use autodie;
 use Carp qw(croak);
 
-use List::MoreUtils qw(any);
+use List::MoreUtils qw(any none);
 
 use Lintian::Data;
 use Lintian::Tags qw(tag);
@@ -165,7 +165,7 @@ sub run {
 my @arch_rules = (qr/^clean$/, qr/^binary-arch$/, qr/^build-arch$/);
 my @indep_rules = (qr/^build$/, qr/^build-indep$/, qr/^binary-indep$/);
 my (@current_targets, %rules_per_target,  %debhelper_group);
-my %seen;
+my (%seen, %overridden);
 my $maybe_skipping;
 my %variables;
 my $uses_makefile_pl = 0;
@@ -308,6 +308,7 @@ sub run {
 qr/^$_$/;
 } split(' ', $target_dependencies);
 for my $target (@current_targets) {
+$overridden{$1}++ if $target =~ m/override_(.+)/;
 if ($target =~ m/%/o) {
 my $pattern = quotemeta $target;
 $pattern =~ s/\\%/.*/g;
@@ -425,6 +426,12 @@ sub run {
 tag 'binary-arch-rules-but-pkg-is-arch-indep' if $nonempty;
 }
 
+tag 'override_dh_clean-does-not-call-dh_clean'
+  if (
+$overridden{'dh_clean'}
+and none { m/^\t\s*-?dh_clean\b/ }
+@{$rules_per_target{'override_dh_clean'}});
+
 # Make sure that all the required build dependencies are there.  Don't
 # issue missing-build-dependency errors for debhelper, since there's
 # another test that does that and it would just be a duplicate.
diff --git a/debian/changelog b/debian/changelog
index b73da40..331a40c 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -13,6 +13,9 @@ lintian (2.5.66) UNRELEASED; urgency=medium
 + [CL] Warn about Python 2.x packages using ${python3:Depends} and
   Python 3.x packages using ${python:Depends}. Thanks to Mattia
   Rizzolo for the idea.  (Closes: #884676)
+  * checks/rules.{desc,pm}:
++ [CL] Check for override_dh_clean targets that are missing calls to
+  dh_clean. Thanks to Andreas Beckmann for the idea.  (Closes: #884817)
   * checks/standards-version.pm:
 + [CL] Avoid misleading tag descriptions when emitting valid
   timewarp-standards-version warnings if the date parts are identical
diff --git 
a/t/tests/rules-override-dh-clean-does-not-call-dh-clean-unrel/debian/debian/rules
 
b/t/tests/rules-override-dh-clean-does-not-call-dh-clean-unrel/debian/debian/rules
new file mode 100755
index 000..3236e08
--- /dev/null
+++ 
b/t/tests/rules-override-dh-clean-does-not-call-dh-clean-unrel/debian/debian/rules
@@ -0,0 +1,8 @@
+#!/usr/bin/make -f
+
+%:
+   dh $@
+
+override_dh_clean:
+   true
+   dh_clean # This rule is a false positive
diff --git a/t/tests/rules-override-dh-clean-does-not-call-dh-clean-unrel/desc 
b/t/tests/rules-override-dh-clean-does-not-call-dh-clean-unrel/desc
new file mode 100644
index 000..b89e55e
--- /dev/null
+++ b/t/tests/rules-override-dh-clean-does-not-call-dh-clean-unrel/desc
@@ -0,0 +1,5 @@
+Testname: rules-override-dh-clean-does-not-call-dh-clean-unrel
+Version: 1.0
+Description: Test for missing calls to dh_clean in override_dh_clean (false 
positive)
+Test-Against:
+

[lintian] branch master updated (0988b92 -> edd6515)

2017-12-20 Thread Chris Lamb

This is an automated email from the git hooks/post-receive script.

lamby pushed a change to branch master
in repository lintian.

  from  0988b92   debian/changelog: Use diacritic for Uwe Kleine-König. 
Thanks again!
   new  edd6515   Check for override_dh_clean targets that are missing 
calls to dh_clean. Thanks to Andreas Beckmann for the idea. (Closes: #884817)

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/rules.desc | 11 +++
 checks/rules.pm   | 11 +--
 debian/changelog  |  3 +++
 .../debian/debian/rules   |  8 
 .../rules-override-dh-clean-does-not-call-dh-clean-unrel/desc |  5 +
 .../tags  |  0
 .../debian/debian/rules   |  7 +++
 t/tests/rules-override-dh-clean-does-not-call-dh-clean/desc   |  5 +
 t/tests/rules-override-dh-clean-does-not-call-dh-clean/tags   |  1 +
 9 files changed, 49 insertions(+), 2 deletions(-)
 create mode 100755 
t/tests/rules-override-dh-clean-does-not-call-dh-clean-unrel/debian/debian/rules
 create mode 100644 
t/tests/rules-override-dh-clean-does-not-call-dh-clean-unrel/desc
 copy t/{debs/deb-format-record-size => 
tests/rules-override-dh-clean-does-not-call-dh-clean-unrel}/tags (100%)
 create mode 100755 
t/tests/rules-override-dh-clean-does-not-call-dh-clean/debian/debian/rules
 create mode 100644 t/tests/rules-override-dh-clean-does-not-call-dh-clean/desc
 create mode 100644 t/tests/rules-override-dh-clean-does-not-call-dh-clean/tags

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

Bug#884848: lintian: false positive for source-includes-file-in-files-excluded

2017-12-20 Thread Ole Streicher

Package: lintian
Version: 2.5.62
Severity: important

Dear lintian maintainers,

on the package idlastro (2017.08.02+dfsg-1 and just-uploaded
2017.12.18+dfsg-1), lintian shows the error

E: idlastro source: source-includes-file-in-files-excluded pro/ymd2dn.pro

However, the orig.tar.gz (to which File-Excluded refers to) does not
contain the mentioned file. A file with the same name (replacing the
removed, non-DFSG file with free content) is however re-introduced with
a patch.

Best regards

Ole

Build failed in Jenkins: lintian-tests_stretch #1277

2017-12-20 Thread jenkins

See 

--
[...truncated 298.80 KB...]
tests::java-jars:dh_auto_configure
tests::java-jars:jh_linkjars
tests::java-jars:dh_auto_build
tests::java-jars:debian/rules override_jh_build
tests::java-jars: make[1]: Entering directory 
'/tmp/testrun/debian/test-out/tests/java-jars/java-jars-1.0'
tests::java-jars: jh_build --javacopts='-target 1.7' --javacopts='-source 1.7'
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javac -g -cp :debian/_jh_build.testa -d 
debian/_jh_build.testa -target 1.7 -source 1.7
tests::java-jars: warning: [options] bootstrap class path not set in 
conjunction with -source 1.7
tests::java-jars: 1 warning
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javadoc -locale en_US -classpath 
:debian/_jh_build.testa -d debian/_jh_build.javadoc/api -quiet -notimestamp 
-source 1.5
tests::java-jars: /usr/lib/jvm/default-java/bin/jar cfm 
/tmp/testrun/debian/test-out/tests/java-jars/java-jars-1.0/testa.jar 
../_jh_manifest.testa org
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javac -g -cp :debian/_jh_build.testb -d 
debian/_jh_build.testb -target 1.7 -source 1.7
tests::java-jars: warning: [options] bootstrap class path not set in 
conjunction with -source 1.7
tests::java-jars: 1 warning
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javadoc -locale en_US -classpath 
:debian/_jh_build.testb -d debian/_jh_build.javadoc/api -quiet -notimestamp 
-source 1.5
tests::java-jars: /usr/lib/jvm/default-java/bin/jar cfm 
/tmp/testrun/debian/test-out/tests/java-jars/java-jars-1.0/testb.jar 
../_jh_manifest.testb org
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javac -g -cp :debian/_jh_build.testc -d 
debian/_jh_build.testc -target 1.7 -source 1.7
tests::java-jars: warning: [options] bootstrap class path not set in 
conjunction with -source 1.7
tests::java-jars: 1 warning
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javadoc -locale en_US -classpath 
:debian/_jh_build.testc -d debian/_jh_build.javadoc/api -quiet -notimestamp 
-source 1.5
tests::java-jars: /usr/lib/jvm/default-java/bin/jar cfm 
/tmp/testrun/debian/test-out/tests/java-jars/java-jars-1.0/testc.jar 
../_jh_manifest.testc org
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javac -g -cp :debian/_jh_build.testd -d 
debian/_jh_build.testd -target 1.7 -source 1.7
tests::java-jars: warning: [options] bootstrap class path not set in 
conjunction with -source 1.7
tests::java-jars: 1 warning
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javadoc -locale en_US -classpath 
:debian/_jh_build.testd -d debian/_jh_build.javadoc/api -quiet -notimestamp 
-source 1.5
tests::java-jars: /usr/lib/jvm/default-java/bin/jar cfm 
/tmp/testrun/debian/test-out/tests/java-jars/java-jars-1.0/testd.jar 
../_jh_manifest.testd org
tests::java-jars: cp testa.jar testb.jar
tests::java-jars: unzip testa.jar
tests::java-jars: Archive:  testa.jar
tests::java-jars: replace META-INF/MANIFEST.MF? [y]es, [n]o, [A]ll, [N]one, 
[r]ename:  NULL
tests::java-jars: (EOF or read error, treating as "[N]one" ...)
tests::java-jars: debian/rules:6: recipe for target 'override_jh_build' failed
tests::java-jars: make[1]: *** [override_jh_build] Error 1
tests::java-jars: make[1]: Leaving directory 
'/tmp/testrun/debian/test-out/tests/java-jars/java-jars-1.0'
tests::java-jars: debian/rules:3: recipe for target 'binary' failed
tests::java-jars: make: *** [binary] Error 2
tests::java-jars: dpkg-buildpackage: error: fakeroot debian/rules binary gave 
error exit status 2
tests::java-jars:  END BUILD LOG
error tests::java-jars: internal error: cd 
/tmp/testrun/debian/test-out/tests/java-jars/java-jars-1.0 && dpkg-buildpackage 
-rfakeroot -us -uc -d -iNEVER_MATCH_ANYTHING -INEVER_MATCH_ANYTHING 
--source-option=--auto-commit 
>/tmp/testrun/debian/test-out/tests/java-jars/build.java-jars 2>&1 at 
t/runtests line 466.


tests::scripts-exec:  START BUILD LOG
tests::scripts-exec: dpkg-buildpackage: warning: using a gain-root-command 
while being root
tests::scripts-exec: dpkg-buildpackage: info: source package scripts-exec
tests::scripts-exec: dpkg-buildpackage: info: source version 1.0
tests::scripts-exec: dpkg-buildpackage: info: source distribution unstable
tests::scripts-exec: dpkg-buildpackage: info: source changed by Debian Lintian 
Maintainers 
tests::scripts-exec:  dpkg-source -iNEVER_MATCH_ANYTHING

Build failed in Jenkins: lintian-tests_stretch #1276

2017-12-20 Thread jenkins

See 

--
[...truncated 298.75 KB...]
tests::java-jars:dh_autoreconf
tests::java-jars:dh_auto_configure
tests::java-jars:jh_linkjars
tests::java-jars:dh_auto_build
tests::java-jars:debian/rules override_jh_build
tests::java-jars: make[1]: Entering directory 
'/tmp/testrun/debian/test-out/tests/java-jars/java-jars-1.0'
tests::java-jars: jh_build --javacopts='-target 1.7' --javacopts='-source 1.7'
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javac -g -cp :debian/_jh_build.testa -d 
debian/_jh_build.testa -target 1.7 -source 1.7
tests::java-jars: warning: [options] bootstrap class path not set in 
conjunction with -source 1.7
tests::java-jars: 1 warning
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javadoc -locale en_US -classpath 
:debian/_jh_build.testa -d debian/_jh_build.javadoc/api -quiet -notimestamp 
-source 1.5
tests::java-jars: /usr/lib/jvm/default-java/bin/jar cfm 
/tmp/testrun/debian/test-out/tests/java-jars/java-jars-1.0/testa.jar 
../_jh_manifest.testa org
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javac -g -cp :debian/_jh_build.testb -d 
debian/_jh_build.testb -target 1.7 -source 1.7
tests::java-jars: warning: [options] bootstrap class path not set in 
conjunction with -source 1.7
tests::java-jars: 1 warning
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javadoc -locale en_US -classpath 
:debian/_jh_build.testb -d debian/_jh_build.javadoc/api -quiet -notimestamp 
-source 1.5
tests::java-jars: /usr/lib/jvm/default-java/bin/jar cfm 
/tmp/testrun/debian/test-out/tests/java-jars/java-jars-1.0/testb.jar 
../_jh_manifest.testb org
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javac -g -cp :debian/_jh_build.testc -d 
debian/_jh_build.testc -target 1.7 -source 1.7
tests::java-jars: warning: [options] bootstrap class path not set in 
conjunction with -source 1.7
tests::java-jars: 1 warning
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javadoc -locale en_US -classpath 
:debian/_jh_build.testc -d debian/_jh_build.javadoc/api -quiet -notimestamp 
-source 1.5
tests::java-jars: /usr/lib/jvm/default-java/bin/jar cfm 
/tmp/testrun/debian/test-out/tests/java-jars/java-jars-1.0/testc.jar 
../_jh_manifest.testc org
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javac -g -cp :debian/_jh_build.testd -d 
debian/_jh_build.testd -target 1.7 -source 1.7
tests::java-jars: warning: [options] bootstrap class path not set in 
conjunction with -source 1.7
tests::java-jars: 1 warning
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javadoc -locale en_US -classpath 
:debian/_jh_build.testd -d debian/_jh_build.javadoc/api -quiet -notimestamp 
-source 1.5
tests::java-jars: /usr/lib/jvm/default-java/bin/jar cfm 
/tmp/testrun/debian/test-out/tests/java-jars/java-jars-1.0/testd.jar 
../_jh_manifest.testd org
tests::java-jars: cp testa.jar testb.jar
tests::java-jars: unzip testa.jar
tests::java-jars: Archive:  testa.jar
tests::java-jars: replace META-INF/MANIFEST.MF? [y]es, [n]o, [A]ll, [N]one, 
[r]ename:  NULL
tests::java-jars: (EOF or read error, treating as "[N]one" ...)
tests::java-jars: debian/rules:6: recipe for target 'override_jh_build' failed
tests::java-jars: make[1]: *** [override_jh_build] Error 1
tests::java-jars: make[1]: Leaving directory 
'/tmp/testrun/debian/test-out/tests/java-jars/java-jars-1.0'
tests::java-jars: debian/rules:3: recipe for target 'binary' failed
tests::java-jars: make: *** [binary] Error 2
tests::java-jars: dpkg-buildpackage: error: fakeroot debian/rules binary gave 
error exit status 2
tests::java-jars:  END BUILD LOG
error tests::java-jars: internal error: cd 
/tmp/testrun/debian/test-out/tests/java-jars/java-jars-1.0 && dpkg-buildpackage 
-rfakeroot -us -uc -d -iNEVER_MATCH_ANYTHING -INEVER_MATCH_ANYTHING 
--source-option=--auto-commit 
>/tmp/testrun/debian/test-out/tests/java-jars/build.java-jars 2>&1 at 
t/runtests line 466.

.
tests::scripts-exec:  START BUILD LOG
tests::scripts-exec: dpkg-buildpackage: warning: using a gain-root-command 
while being root
tests::scripts-exec: dpkg-buildpackage: info: source package scripts-exec
tests::scripts-exec: dpkg-buildpackage: info: source version 1.0
tests::scripts-exec: dpkg-buildpackage: info: source distribution unstable
tests::scripts-exec: dpkg-buildpackage: info: source changed by Debian Lintian 
Maintainers 
tests::scripts-exec:  dpkg-source

Build failed in Jenkins: lintian-tests_stretch #1275

2017-12-20 Thread jenkins

See 


Changes:

[lamby] Correct invalid "else if" syntax with "elsif". Thanks to Uwe

[lamby] debian/changelog: Use diacritic for Uwe Kleine-König. Thanks again!

--
[...truncated 298.76 KB...]
tests::java-jars:dh_autoreconf
tests::java-jars:dh_auto_configure
tests::java-jars:jh_linkjars
tests::java-jars:dh_auto_build
tests::java-jars:debian/rules override_jh_build
tests::java-jars: make[1]: Entering directory 
'/tmp/testrun/debian/test-out/tests/java-jars/java-jars-1.0'
tests::java-jars: jh_build --javacopts='-target 1.7' --javacopts='-source 1.7'
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javac -g -cp :debian/_jh_build.testa -d 
debian/_jh_build.testa -target 1.7 -source 1.7
tests::java-jars: warning: [options] bootstrap class path not set in 
conjunction with -source 1.7
tests::java-jars: 1 warning
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javadoc -locale en_US -classpath 
:debian/_jh_build.testa -d debian/_jh_build.javadoc/api -quiet -notimestamp 
-source 1.5
tests::java-jars: /usr/lib/jvm/default-java/bin/jar cfm 
/tmp/testrun/debian/test-out/tests/java-jars/java-jars-1.0/testa.jar 
../_jh_manifest.testa org
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javac -g -cp :debian/_jh_build.testb -d 
debian/_jh_build.testb -target 1.7 -source 1.7
tests::java-jars: warning: [options] bootstrap class path not set in 
conjunction with -source 1.7
tests::java-jars: 1 warning
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javadoc -locale en_US -classpath 
:debian/_jh_build.testb -d debian/_jh_build.javadoc/api -quiet -notimestamp 
-source 1.5
tests::java-jars: /usr/lib/jvm/default-java/bin/jar cfm 
/tmp/testrun/debian/test-out/tests/java-jars/java-jars-1.0/testb.jar 
../_jh_manifest.testb org
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javac -g -cp :debian/_jh_build.testc -d 
debian/_jh_build.testc -target 1.7 -source 1.7
tests::java-jars: warning: [options] bootstrap class path not set in 
conjunction with -source 1.7
tests::java-jars: 1 warning
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javadoc -locale en_US -classpath 
:debian/_jh_build.testc -d debian/_jh_build.javadoc/api -quiet -notimestamp 
-source 1.5
tests::java-jars: /usr/lib/jvm/default-java/bin/jar cfm 
/tmp/testrun/debian/test-out/tests/java-jars/java-jars-1.0/testc.jar 
../_jh_manifest.testc org
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javac -g -cp :debian/_jh_build.testd -d 
debian/_jh_build.testd -target 1.7 -source 1.7
tests::java-jars: warning: [options] bootstrap class path not set in 
conjunction with -source 1.7
tests::java-jars: 1 warning
tests::java-jars: find src -name *.java -and -type f -print0 | xargs -s 512000 
-0 /usr/lib/jvm/default-java/bin/javadoc -locale en_US -classpath 
:debian/_jh_build.testd -d debian/_jh_build.javadoc/api -quiet -notimestamp 
-source 1.5
tests::java-jars: /usr/lib/jvm/default-java/bin/jar cfm 
/tmp/testrun/debian/test-out/tests/java-jars/java-jars-1.0/testd.jar 
../_jh_manifest.testd org
tests::java-jars: cp testa.jar testb.jar
tests::java-jars: unzip testa.jar
tests::java-jars: Archive:  testa.jar
tests::java-jars: replace META-INF/MANIFEST.MF? [y]es, [n]o, [A]ll, [N]one, 
[r]ename:  NULL
tests::java-jars: (EOF or read error, treating as "[N]one" ...)
tests::java-jars: debian/rules:6: recipe for target 'override_jh_build' failed
tests::java-jars: make[1]: *** [override_jh_build] Error 1
tests::java-jars: make[1]: Leaving directory 
'/tmp/testrun/debian/test-out/tests/java-jars/java-jars-1.0'
tests::java-jars: debian/rules:3: recipe for target 'binary' failed
tests::java-jars: make: *** [binary] Error 2
tests::java-jars: dpkg-buildpackage: error: fakeroot debian/rules binary gave 
error exit status 2
tests::java-jars:  END BUILD LOG
error tests::java-jars: internal error: cd 
/tmp/testrun/debian/test-out/tests/java-jars/java-jars-1.0 && dpkg-buildpackage 
-rfakeroot -us -uc -d -iNEVER_MATCH_ANYTHING -INEVER_MATCH_ANYTHING 
--source-option=--auto-commit 
>/tmp/testrun/debian/test-out/tests/java-jars/build.java-jars 2>&1 at 
t/runtests line 466.

..
tests::scripts-exec:  START BUILD LOG
tests::scripts-exec: dpkg-buildpackage: warning: using a gain-root-command 
while being root
tests::scripts-exec: dpkg-buildpackage: info: source package scripts-exec
tests::scripts-exec: dpkg-buildpackage: info: source version 1.0
tests::scripts-exec: dpkg-buildpackage: info: source

Processed: found 505857 in 2.5.65

2017-12-20 Thread Debian Bug Tracking System

Processing commands for cont...@bugs.debian.org:

> found 505857 2.5.65
Bug #505857 [lintian] [checks/watch-file] false positive 
debian-watch-file-should-mangle-version
Marked as found in versions lintian/2.5.65.
> thanks
Stopping processing here.

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

Bug#505857: lintian: false positive debian-watch-file-should-mangle-version

2017-12-20 Thread Christoph Berg

Version: 2.5.65

Re: Roger Shimizu 2017-10-16

[lintian] branch master updated (e23a9ac -> 0988b92)

2017-12-20 Thread Chris Lamb

This is an automated email from the git hooks/post-receive script.

lamby pushed a change to branch master
in repository lintian.

  from  e23a9ac   Correct invalid "else if" syntax with "elsif". Thanks to 
Uwe Kleine-Koenig for the report.
   new  0988b92   debian/changelog: Use diacritic for Uwe Kleine-König. 
Thanks again!

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:
 debian/changelog | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

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

[lintian] 01/01: debian/changelog: Use diacritic for Uwe Kleine-König. Thanks again!

2017-12-20 Thread Chris Lamb

This is an automated email from the git hooks/post-receive script.

lamby pushed a commit to branch master
in repository lintian.

commit 0988b927a1c67c4715edca9669eaec511f6df6d9
Author: Chris Lamb 
Date:   Wed Dec 20 09:02:56 2017 +

debian/changelog: Use diacritic for Uwe Kleine-König. Thanks again!
---
 debian/changelog | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/debian/changelog b/debian/changelog
index 8a459f9..b73da40 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -32,7 +32,7 @@ lintian (2.5.66) UNRELEASED; urgency=medium
 
   * reporting/templates/maintainer.tmpl:
 + [CL] Correct invalid "else if" syntax with "elsif". Thanks to Uwe
-  Kleine-Koenig for the report.
+  Kleine-König for the report.
 
   * t/tests/files-multiarch-foreign-files:
 + [CL] Don't hardcode architecture triplet to fix FTBFS on non-amd64

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

[lintian] 01/01: Correct invalid "else if" syntax with "elsif". Thanks to Uwe Kleine-Koenig for the report.

2017-12-20 Thread Chris Lamb

This is an automated email from the git hooks/post-receive script.

lamby pushed a commit to branch master
in repository lintian.

commit e23a9ac5685070bd6ce5cb1bc4e792ba48ba5f59
Author: Chris Lamb 
Date:   Wed Dec 20 08:58:13 2017 +

Correct invalid "else if" syntax with "elsif". Thanks to Uwe Kleine-Koenig 
for the report.
---
 debian/changelog| 4 
 reporting/templates/maintainer.tmpl | 2 +-
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/debian/changelog b/debian/changelog
index 7ba9163..8a459f9 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -30,6 +30,10 @@ lintian (2.5.66) UNRELEASED; urgency=medium
 + [CL] Also emit embedded-javascript-library for Twitter Bootstrap and
   "mustache".
 
+  * reporting/templates/maintainer.tmpl:
++ [CL] Correct invalid "else if" syntax with "elsif". Thanks to Uwe
+  Kleine-Koenig for the report.
+
   * t/tests/files-multiarch-foreign-files:
 + [CL] Don't hardcode architecture triplet to fix FTBFS on non-amd64
   architectures.  (Closes: #884683)
diff --git a/reporting/templates/maintainer.tmpl 
b/reporting/templates/maintainer.tmpl
index 7be6b7a..c63cc12 100644
--- a/reporting/templates/maintainer.tmpl
+++ b/reporting/templates/maintainer.tmpl
@@ -124,7 +124,7 @@
 $OUT .= qq(https://bugs.debian.org/src:$source;>Bugs\n);
 if ($errors) {
 $OUT .= qq(Full report\n);
-} else if (@error_tags) {
+} elsif (@error_tags) {
 $OUT .= qq(Short report\n);
 }
 $OUT .= qq[  (${status}, last processed by 
Lintian/$last_processed_by)];

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

[lintian] branch master updated (5be4539 -> e23a9ac)

2017-12-20 Thread Chris Lamb

This is an automated email from the git hooks/post-receive script.

lamby pushed a change to branch master
in repository lintian.

  from  5be4539   Avoid misleading tag descriptions when emitting valid 
timewarp-standards-version warnings if the date parts are identical (ie. 
"2017-11-30 < 2017-11-30"). Thanks to Andrea Bolognani e...@kiyuko.org> for the 
report. (Closes: #884785)
   new  e23a9ac   Correct invalid "else if" syntax with "elsif". Thanks to 
Uwe Kleine-Koenig for the report.

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:
 debian/changelog| 4 
 reporting/templates/maintainer.tmpl | 2 +-
 2 files changed, 5 insertions(+), 1 deletion(-)

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

Bug#884870: lintian: vcs-field-has-unexpected-spaces and vcswatch don't agree

Bug#884870: marked as done (lintian: vcs-field-has-unexpected-spaces and vcswatch don't agree)

Bug#884798: lintian could complain about pkg-config uses that fail to use $ac_tool_prefix

Processed: Re: Bug#505857: lintian: false positive debian-watch-file-should-mangle-version

Bug#505857: lintian: false positive debian-watch-file-should-mangle-version

Build failed in Jenkins: lintian-tests_stretch #1279

Bug#884870: lintian: vcs-field-has-unexpected-spaces and vcswatch don't agree

[lintian] 01/01: Add a regression test for a potential false positive in the "vcs-field-has-unexpected-spaces" tag. (Ref: #884870)

[lintian] branch master updated (edd6515 -> 8e1ba53)

Build failed in Jenkins: lintian-tests_stretch #1278

Bug#884870: lintian: vcs-field-has-unexpected-spaces and vcswatch don't agree

Bug#884817: lintian: check for override_dh_clean target missing call to dh_clean

Bug#884870: lintian: vcs-field-has-unexpected-spaces and vcswatch don't agree

Processed: Re: lintian: check for override_dh_clean target missing call to dh_clean

Bug#884848: lintian: false positive for source-includes-file-in-files-excluded

[lintian] 01/01: Check for override_dh_clean targets that are missing calls to dh_clean. Thanks to Andreas Beckmann for the idea. (Closes: #884817)

[lintian] branch master updated (0988b92 -> edd6515)

Bug#884848: lintian: false positive for source-includes-file-in-files-excluded

Build failed in Jenkins: lintian-tests_stretch #1277

Build failed in Jenkins: lintian-tests_stretch #1276

Build failed in Jenkins: lintian-tests_stretch #1275

Processed: found 505857 in 2.5.65

Bug#505857: lintian: false positive debian-watch-file-should-mangle-version

[lintian] branch master updated (e23a9ac -> 0988b92)

[lintian] 01/01: debian/changelog: Use diacritic for Uwe Kleine-König. Thanks again!

[lintian] 01/01: Correct invalid "else if" syntax with "elsif". Thanks to Uwe Kleine-Koenig for the report.

[lintian] branch master updated (5be4539 -> e23a9ac)

27 matches

Site Navigation

Mail list logo

Footer information