Package: maven-debian-helper
Severity: wishlist
Tags: patch

Standard place where generated javadoc API documentation is installed on Debian 
system
is /usr/share/doc/<package-name>/api/. Maven dh build installs it also a JAR in 
maven-repo.
Having this duplicate file isn't necessary, so please consider not installing
the javadoc JAR by default.

Required change is implemented in attached patch.

Adding 'export MH_INSTALL_DOC=1' into d/rules overrides the setting.

Regards,

Jakub

>From 81c9f308c6a21979f8e3d7feb2a3af232ce825b7 Mon Sep 17 00:00:00 2001
From: Jakub Adam <jakub.a...@ktknet.cz>
Date: Mon, 10 Jun 2013 13:35:10 +0200
Subject: [PATCH] maven.pm: Don't install documentation jar into maven-repo by
 default

Add 'export MH_INSTALL_DOC=1' into d/rules to override the setting.
---
 share/perl/maven.pm |    8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/share/perl/maven.pm b/share/perl/maven.pm
index 00520df..ca94eff 100644
--- a/share/perl/maven.pm
+++ b/share/perl/maven.pm
@@ -97,9 +97,11 @@ sub install {
 	$this->doit_in_builddir("mh_resolve_dependencies", "--non-interactive",
 		"--offline", "-p$this->{package}", @resolvedep_args);
 	if ($this->{doc_package}) {
-		$this->doit_in_builddir(@{$this->{maven_cmd}},
-			"-Ddebian.package=$this->{doc_package}",
-			"org.debian.maven:debian-maven-plugin:$maven_debian_version:install-doc");
+		if ($ENV{MH_INSTALL_DOC} == 1) {
+			$this->doit_in_builddir(@{$this->{maven_cmd}},
+				"-Ddebian.package=$this->{doc_package}",
+				"org.debian.maven:debian-maven-plugin:$maven_debian_version:install-doc");
+		}
 		doit("cp","debian/$this->{package}.substvars",
 			"debian/$this->{doc_package}.substvars");
 		# clean up generated docs
-- 
1.7.10.4

Reply via email to