[maven-license-plugin] Fix FTBFS - Update to current guidelines and maven-local build macros

Guido Grazioli guidograzioli at fedoraproject.org
Fri Feb 27 04:24:53 UTC 2015


commit 679eba053adb111f31268ae02a7de4c296dc8b5d
Author: Guido Grazioli <guido.grazioli at gmail.com>
Date:   Fri Feb 27 15:24:30 2015 +1100

    Fix FTBFS - Update to current guidelines and maven-local build macros

 001-mavenlicenseplugin-fixbuild.patch | 21 ++++++------
 maven-license-plugin.spec             | 61 +++++++++--------------------------
 2 files changed, 28 insertions(+), 54 deletions(-)
---
diff --git a/001-mavenlicenseplugin-fixbuild.patch b/001-mavenlicenseplugin-fixbuild.patch
index fd37b01..0df130c 100644
--- a/001-mavenlicenseplugin-fixbuild.patch
+++ b/001-mavenlicenseplugin-fixbuild.patch
@@ -1,12 +1,3 @@
-From a1f7f0c16eb2333e527448c2e48a512ef8da50ed Mon Sep 17 00:00:00 2001
-From: Stanislav Ochotnicky <sochotnicky at redhat.com>
-Date: Tue, 24 Apr 2012 09:04:34 +0200
-Subject: [PATCH] Fix build by removing some unneeded dependencies
-
----
- pom.xml |   72 ++++++--------------------------------------------------------
- 1 files changed, 7 insertions(+), 65 deletions(-)
-
 diff --git a/pom.xml b/pom.xml
 index a806a49..e806c77 100644
 --- a/pom.xml
@@ -114,3 +105,15 @@ index a806a49..e806c77 100644
 -- 
 1.7.7.6
 
+--- a/src/main/java/com/google/code/mojo/license/util/FileUtils.java	2010-08-06 13:00:51.000000000 +1000
++++ b/src/main/java/com/google/code/mojo/license/util/FileUtils.java	2015-02-27 15:04:46.157580507 +1100
+@@ -52,7 +52,8 @@
+     }
+ 
+     public static String read(URL location, Map<String, String> properties) throws IOException {
+-        Reader reader = new InterpolationFilterReader(new BufferedReader(new InputStreamReader(location.openStream())), properties);
++        Map<Object,Object> propertiesMap = java.util.Collections.<Object,Object>unmodifiableMap(properties);
++        Reader reader = new InterpolationFilterReader(new BufferedReader(new InputStreamReader(location.openStream())), propertiesMap);
+         try {
+             return IOUtil.toString(reader);
+         } finally {
diff --git a/maven-license-plugin.spec b/maven-license-plugin.spec
index 586e32e..3feedad 100644
--- a/maven-license-plugin.spec
+++ b/maven-license-plugin.spec
@@ -1,6 +1,6 @@
 Name:           maven-license-plugin
 Version:        1.8.0
-Release:        17%{?dist}
+Release:        18%{?dist}
 Summary:        Maven plugin to update header licenses of source files
 
 Group:          Development/Libraries
@@ -19,35 +19,15 @@ BuildRequires:  java-devel
 BuildRequires:  jpackage-utils
 BuildRequires:  apache-resource-bundles
 BuildRequires:  maven-local
-BuildRequires:  maven-assembly-plugin
-BuildRequires:  maven-deploy-plugin
-BuildRequires:  maven-jar-plugin
-BuildRequires:  maven-javadoc-plugin
-BuildRequires:  maven-source-plugin
-BuildRequires:  maven-changelog-plugin
-BuildRequires:  maven-changes-plugin
-BuildRequires:  maven-clean-plugin
-BuildRequires:  maven-compiler-plugin
-BuildRequires:  maven-dependency-plugin
-BuildRequires:  maven-help-plugin
-BuildRequires:  maven-idea-plugin
-BuildRequires:  maven-install-plugin
 BuildRequires:  maven-plugin-plugin
-BuildRequires:  maven-pmd-plugin
-BuildRequires:  maven-resources-plugin
-BuildRequires:  maven-repository-plugin
-BuildRequires:  maven-remote-resources-plugin
-BuildRequires:  maven-site-plugin
 BuildRequires:  maven-shared
-BuildRequires:  maven-surefire-plugin
-BuildRequires:  maven-plugin-testing-harness
-BuildRequires:  maven-release-plugin
 BuildRequires:  plexus-utils
 BuildRequires:  plexus-classworlds
 BuildRequires:  xml-commons-apis
 BuildRequires:  xmltool
 
 Requires:       jpackage-utils
+Requires:       java-headless >= 1:1.6.0
 Requires:       maven
 Requires:       maven-shared
 Requires:       xmltool
@@ -82,37 +62,28 @@ sed -i 's/\r//' NOTICE.txt
 # Remove wagon-webdav extension which is not available
 %pom_xpath_remove pom:build/pom:extensions
 
+# Set sources/resources encoding
+%pom_xpath_inject "pom:properties" "<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>"
+
 %build
-mvn-rpmbuild -Dmaven.test.skip=true install javadoc:aggregate
+%mvn_build -f
 
 %install
+%mvn_install
 mkdir -p $RPM_BUILD_ROOT%{_javadir}
 
-# jar
-install -Dp -m 644 target/%{name}-%{version}.jar \
-$RPM_BUILD_ROOT%{_javadir}/%{name}.jar
-
-# javadoc
-mkdir -p $RPM_BUILD_ROOT%{_javadocdir}/%{name}
-cp -rp target/site/apidocs/  $RPM_BUILD_ROOT%{_javadocdir}/%{name}
-
-# pom
-install -d -m 755 $RPM_BUILD_ROOT%{_mavenpomdir}
-install -pm 644 pom.xml  $RPM_BUILD_ROOT%{_mavenpomdir}/JPP-%{name}.pom
-
-%add_maven_depmap JPP-%{name}.pom %{name}.jar
+%files -f .mfiles
+%dir %{_javadir}/%{name}
+%license LICENSE.txt
+%doc NOTICE.txt
 
-%files
-%doc NOTICE.txt LICENSE.txt
-%{_mavenpomdir}/*
-%{_mavendepmapfragdir}/*
-%{_javadir}/*
-
-%files javadoc
-%doc LICENSE.txt
-%{_javadocdir}/%{name}
+%files javadoc  -f .mfiles-javadoc
 
 %changelog
+* Fri Feb 27 2015 Guido Grazioli <guido.grazioli at gmail.com> - 1.8.0-18
+- Update to current guidelines and maven-local build macros
+- Fix FTBFS
+
 * Sat Jun 07 2014 Fedora Release Engineering <rel-eng at lists.fedoraproject.org> - 1.8.0-17
 - Rebuilt for https://fedoraproject.org/wiki/Fedora_21_Mass_Rebuild
 


More information about the scm-commits mailing list