On 25.08.2016 18:08, Bas Couwenberg wrote:
> Source: libphonenumber
> Version: 7.1.0-4
> Severity: serious
> Justification: makes the package in question unusable or mostly so
> Control: block 835170 by -1
> 
> Dear Maintainer,
> 
> The recent upload of protobuf 3.0.0 to unstable has started an
> uncoordinated transition (#835170). Unfortunately your package FTBFS due
> to an unrelated issue:

Hi Bas,

I think the issue is not unrelated to protobuf 3.0.0. Yesterday I filed
https://bugs.debian.org/835358 because protobuf used a wrong pom.xml
file which provides important information like groupids or artifactids.

So this bug got resolved today but now I see that the new pom.xml file
differs in one important aspect from earlier ones, e.g. from  version
2.6.1 [1]. The pom.xml file for version 3.0.0 [2] declares junit and
easymock as dependencies while the older one sets the scope to test
only. At the moment the dependencies are required at build time and I
think that shouldn't be necessary.

I'm attaching a patch that should address this issue for all
reverse-dependencies but it must be applied in src:protobuf. If there
are no objections, I'm going ahead and file another bug report tomorrow.

Regards,

Markus


[1]
https://anonscm.debian.org/cgit/pkg-protobuf/pkg-protobuf.git/tree/java/pom.xml?h=debian/2.6.1-2

[2] https://sources.debian.net/src/protobuf/3.0.0-4/java/core/pom.xml/
From: Markus Koschany <a...@debian.org>
Date: Thu, 25 Aug 2016 18:58:17 +0200
Subject: protobuf-java

---
 java/core/pom.xml | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/java/core/pom.xml b/java/core/pom.xml
index 39d6781..20ed02a 100644
--- a/java/core/pom.xml
+++ b/java/core/pom.xml
@@ -22,14 +22,17 @@
     <dependency>
       <groupId>junit</groupId>
       <artifactId>junit</artifactId>
+      <scope>test</scope>
     </dependency>
     <dependency>
       <groupId>org.easymock</groupId>
       <artifactId>easymock</artifactId>
+      <scope>test</scope>
     </dependency>
     <dependency>
       <groupId>org.easymock</groupId>
       <artifactId>easymockclassextension</artifactId>
+      <scope>test</scope>
     </dependency>
   </dependencies>
 

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to