Your message dated Thu, 28 Mar 2013 14:48:27 +0000
with message-id <e1ule7v-0003hd...@franck.debian.org>
and subject line Bug#688043: fixed in maven-debian-helper 1.6
has caused the Debian Bug report #688043,
regarding maven-debian-helper: Make intelligent decisions about whether to 
install to /usr/share/java
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.)


-- 
688043: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=688043
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
--- Begin Message ---
Package: maven-debian-helper
Version: 1.5.1
Severity: normal
Tags: patch
User: ubuntu-de...@lists.ubuntu.com
Usertags: origin-ubuntu quantal ubuntu-patch

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256

Dear Maintainer,

In Ubuntu, the attached patch was applied to achieve the following:

  * Make intelligent decisions about whether to install jar files
    to /usr/share/java (LP: #1052533):
    - SysInstallMojo: Automatically detect if artifact is being installed
      into a Java library package (excluding maven plugins) and set flags
      to ensure that jar files are installed to /usr/share/java IF config
      options relevant to /usr/share/java installation have not been
      explicitly specified.

Hopefully this makes m-d-h a little better in terms of trying todo the right
thing when no explicit options are provided.

Thanks for considering the patch.


- -- System Information:
Debian Release: wheezy/sid
  APT prefers quantal-updates
  APT policy: (500, 'quantal-updates'), (500, 'quantal-security'), (500, 
'quantal')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 3.5.0-14-generic (SMP w/8 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)

iQIcBAEBCAAGBQJQWJqLAAoJEL/srsug59jDx00P/1lWLhHLUIJ+DSwYVLKZmmui
DX59XskH+cdrn1H41tyN0NS7pIy52gxpaQGdMYEe5vV3I94P2523Oqg4/O21q3kA
DcHAJeWh/JW7iF3aSe0/fdsBukhPUcQlvOejYQ4MHrGmiyxhVLg09h7hB75eaVYk
VdU8hpKDOvJC9U+yMZMwCLESaqqOXYDTM8mJVl25GrOJmOroDHpNXxBmnHHwZpfs
oVTYGViECtYevIPLz3k+r6edeDGZGzaIbby/9KO8m+EMZYsIkOqZhVppTvuBzK6V
8XTpGHOZazScPHCDLnfn2snncqwMXOUrkLyEN9WQHeio/4hs0w3sLpFYEj2nF+oG
wFt54vUgd+4ZgiocOY5bdR3T7tNvEix7zXAzfo9NQPZQY+RQFbAu8dvJtig+IJ4o
ynynLy22JcYAhk2O/nofqcuAGSwximdCAMAnWPzoNz0BZvNcqUsfYS35M6vhOQNO
1t+S4ze7qOSzAFBRSxaoepHHWUCLnvQIjJLrFGK7gRoC2vqoqxtXRym6w7oEFKka
2Zzlmj3IzhvjibDzu146PZChOEU5Or8mOSZWKvWmbMQs/9lRuBv0U3xTdNgSeART
jH5y6Yo8XQ9ue5v2aqIToP0IJfvLo3oiwLdkx6G7D70W8RrJZGEZSlDF0aCfjnup
jjI2d33JTg4TWM3/RWhK
=gD8u
-----END PGP SIGNATURE-----
=== modified file 'debian-maven-plugin/src/main/java/org/debian/maven/plugin/SysInstallMojo.java'
--- debian-maven-plugin/src/main/java/org/debian/maven/plugin/SysInstallMojo.java	2012-05-04 00:51:14 +0000
+++ debian-maven-plugin/src/main/java/org/debian/maven/plugin/SysInstallMojo.java	2012-09-18 15:41:11 +0000
@@ -22,6 +22,8 @@
 import java.util.ArrayList;
 import java.util.List;
 import java.util.Properties;
+import java.util.regex.Pattern;
+import java.util.regex.Matcher;
 
 import org.apache.maven.plugin.AbstractMojo;
 import org.apache.maven.plugin.MojoExecutionException;
@@ -177,6 +179,17 @@
 
     private String classifier;
 
+    /**
+     * Regex for detecting that package is a libXXX-java package
+     */
+    private static final String javaLibRegex = new String("lib.*-java");
+
+    /**
+     * Regex for detecting that package is a maven plugin package
+     */
+    private static final String pluginRegex =
+        new String("lib.*-maven-plugin-java|libmaven-.*-plugin-java");
+
     // ----------------------------------------------------------------------
     // Public methods
     // ----------------------------------------------------------------------
@@ -658,7 +671,19 @@
                 classifier = pomOption.getClassifier();
             }
 
-            installToUsj = pomOption.isJavaLib();
+            // If package is a java libary (lib.*-java)and is not a maven plugin
+            // (lib.*-plugin-java) potentially install to /usr/share/java
+            boolean packageIsJavaLib =
+                  Pattern.matches(javaLibRegex, destPackage) &&
+                ! Pattern.matches(pluginRegex, destPackage);
+
+            // Its also possible to configure USJ install using 
+            // DEB_MAVEN_INSTALL_TO_USJ which we should honour
+            // over auto-detection if its set to 'false'.
+            // This is stored in 'installToUsj' on class instance
+            // creation
+            installToUsj = pomOption.isJavaLib() ||
+                           (packageIsJavaLib && installToUsj);
         }
 
         List params = new ArrayList();


--- End Message ---
--- Begin Message ---
Source: maven-debian-helper
Source-Version: 1.6

We believe that the bug you reported is fixed in the latest version of
maven-debian-helper, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 688...@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Thomas Koch <tho...@koch.ro> (supplier of updated maven-debian-helper package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmas...@debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256

Format: 1.8
Date: Thu, 28 Mar 2013 11:54:00 +0200
Source: maven-debian-helper
Binary: maven-debian-helper
Architecture: source all
Version: 1.6
Distribution: experimental
Urgency: low
Maintainer: Debian Java Maintainers 
<pkg-java-maintainers@lists.alioth.debian.org>
Changed-By: Thomas Koch <tho...@koch.ro>
Description: 
 maven-debian-helper - Helper tools for building Debian packages with Maven
Closes: 688043 703382
Changes: 
 maven-debian-helper (1.6) experimental; urgency=low
 .
   [Ludovic Claude]
   * d/control:
     - Update Standards-Version to 3.9.2, no changes
   * d/rules: move 'unpatch all POM files' to
     cleanbuilddir target to restore all files to their pristine
     state after building.
   * share/perl/maven.pm: Apply patch to port share/cdbs/1/class/maven.mk to dh.
     Patch made by Steve Langasek <vor...@debian.org>
 .
   [Thomas Koch]
   * Massive code refactorings and cleanup.
 .
   [Hilko Bengen]
   * mh_make: Do not use sudo (Closes: #703382)
 .
   [James Page]
   * Make intelligent decisions about whether to install to /usr/share/java.
     (Closes: #688043) (LP: #1052533)
Checksums-Sha1: 
 883ef74d374fc96c8321e84add7865651849bff7 2050 maven-debian-helper_1.6.dsc
 5634c15a6b422acc4117fe413bb9c94051e66937 164847 maven-debian-helper_1.6.tar.gz
 8f53574521ae41fc98d32d4d343ed939edbe75ac 111434 maven-debian-helper_1.6_all.deb
Checksums-Sha256: 
 b86dfde5b5e3cf1446315ea871bd284adacfd17018e8d408f03178c768313e36 2050 
maven-debian-helper_1.6.dsc
 981c6d85d5955378cb1bda1d78dba406ee7fd0fb5319efb749925b5fab3c164f 164847 
maven-debian-helper_1.6.tar.gz
 484448cf8ff1d8a5f547ec383d4360d68ce866b0d5eb261758d19657da40e5f9 111434 
maven-debian-helper_1.6_all.deb
Files: 
 e5f96e28cefb209287f5e45b2f06019c 2050 java optional maven-debian-helper_1.6.dsc
 7e5963a666472410f5a7ed30d1cb4b9f 164847 java optional 
maven-debian-helper_1.6.tar.gz
 a30f91ce175150386007aaed98c2d031 111434 java optional 
maven-debian-helper_1.6_all.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.12 (GNU/Linux)

iQIcBAEBCAAGBQJRVD7mAAoJEAf8SJEEK6ZaHFgP+QF4lbz9jT5CKH+7C50I7MBc
Vo7vdUu7rByyLk3qL3t/gk3WaekNbjGP9n8jPxjFc5W7XMzfmACqXrGKIMtDJ512
OUfiHjHeiOg2ZYdZE1+3GAyM5H/zhKnXEV3EkftvAhNbPlcQw9iqxfFm3gDDZibJ
VhczFqI/cSgTP+oyCy8zsj89ufPfn/PExQjR7XuuVqAad+Z7cuCokB0pzYdMck3b
hpDqASxKqGpWue99PHUWZTo26iMnUbiSwCHOFKnZf9W8t8/oKiQtL3mSjQ72uuw5
snZcfXK4FdhCXTcW+aiwk1+znU7KYs6PYVHH0HZaPmnuEkawCUBRztP0HxhV/SEx
ajnAMiVdtU5C9ousYAEoyRYJYNqkuSWjSZSCfVKh+uOQWKgETZ8yidKyE1MLlgiV
1+9F+cPptyREWr+A6vVtXTWQVQRF6AeORcrCTrLrUkn2rnylp5PJOSRq51P1ZjvU
41CL+vWcdkmIjQNJvTdWGZjobkV1GX6GbOfloAz9IvfaL+NXfREVbJnHW/whYM6K
KaSYHeQu6u9dyX4YJcT7yMTVaWjj07zASxjITgN0sk0BKZGeFUNwMvvtMF5wzSnq
blEM9gfaSw+P2fhg8tZxxMXgDrzyi30dsJDzdgfAM9rCqcXN/KMftho6CDJY/4tQ
qtKvMf/rFym/v9EiXcro
=WxBP
-----END PGP SIGNATURE-----

--- End Message ---
__
This is the maintainer address of Debian's Java team
<http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-java-maintainers>. 
Please use
debian-j...@lists.debian.org for discussions and questions.

Reply via email to