Package: libprotobuf-java
Version: 3.0.0-5
Severity: serious
Tags: patch

Hello,

thanks for fixing #835358 so quickly. Unfortunately there is another
issue with the pom.xml file now. java/core/pom.xml lists junit and
easymock artifacts as dependencies but libprotobuf-java does not
depend on these packages. This causes build failures like [1].

libprotobuf-java should either depend on the respective packages or
change the scope to test again as in earlier versions. I think the
latter solution is preferable because junit and easymock are only
needed for tests. If you take a look at version 2.6.1, you will see
that scope test was used before. [2] I don't know why they changed this
but it looks like an upstream bug to me.

I'm attaching a patch that resolves the issue and restores the old
behavior again.

Regards,

Markus

[1] https://bugs.debian.org/835430
[2] 
https://anonscm.debian.org/cgit/pkg-protobuf/pkg-protobuf.git/tree/java/pom.xml?h=debian/2.6.1-2
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>
 

Reply via email to