[javapackages-tools] Fix path to ABRT agent DSO
Mikolaj Izdebski
mizdebsk at fedoraproject.org
Thu Oct 30 15:08:05 UTC 2014
commit 532a3d6f1c0074273bd13d4819561d90271b3743
Author: Mikolaj Izdebski <mizdebsk at redhat.com>
Date: Thu Oct 30 15:57:38 2014 +0100
Fix path to ABRT agent DSO
...t-Append-dependencies-to-metadata-if-we-h.patch | 2 +-
...pper-script-to-inject-extra-JVM-arguments.patch | 2 +-
...cture-independent-location-of-abrt-java-c.patch | 69 ++++++++++++++++++++
javapackages-tools.spec | 3 +
4 files changed, 74 insertions(+), 2 deletions(-)
---
diff --git a/0001-mvn_artifact-Append-dependencies-to-metadata-if-we-h.patch b/0001-mvn_artifact-Append-dependencies-to-metadata-if-we-h.patch
index a980643..791ff21 100644
--- a/0001-mvn_artifact-Append-dependencies-to-metadata-if-we-h.patch
+++ b/0001-mvn_artifact-Append-dependencies-to-metadata-if-we-h.patch
@@ -1,7 +1,7 @@
From eafaa146aace0885d17ff472db0a63aef75f2ce5 Mon Sep 17 00:00:00 2001
From: Michal Srb <msrb at redhat.com>
Date: Thu, 18 Sep 2014 10:48:07 +0200
-Subject: [PATCH 1/2] [mvn_artifact] Append dependencies to metadata, if we
+Subject: [PATCH 1/3] [mvn_artifact] Append dependencies to metadata, if we
have them
---
diff --git a/0002-Use-wrapper-script-to-inject-extra-JVM-arguments.patch b/0002-Use-wrapper-script-to-inject-extra-JVM-arguments.patch
index aefdebc..64669f8 100644
--- a/0002-Use-wrapper-script-to-inject-extra-JVM-arguments.patch
+++ b/0002-Use-wrapper-script-to-inject-extra-JVM-arguments.patch
@@ -1,7 +1,7 @@
From dee96bf521cdbae4c80ee328968e1ff5200f2aab Mon Sep 17 00:00:00 2001
From: Mikolaj Izdebski <mizdebsk at redhat.com>
Date: Thu, 30 Oct 2014 15:20:20 +0100
-Subject: [PATCH 2/2] Use wrapper script to inject extra JVM arguments
+Subject: [PATCH 2/3] Use wrapper script to inject extra JVM arguments
---
install | 1 +
diff --git a/0003-Use-architecture-independent-location-of-abrt-java-c.patch b/0003-Use-architecture-independent-location-of-abrt-java-c.patch
new file mode 100644
index 0000000..9c57283
--- /dev/null
+++ b/0003-Use-architecture-independent-location-of-abrt-java-c.patch
@@ -0,0 +1,69 @@
+From 97ee162d443fd954208c277c43f7195339b01e8d Mon Sep 17 00:00:00 2001
+From: Jakub Filak <jfilak at redhat.com>
+Date: Thu, 30 Oct 2014 08:36:56 +0100
+Subject: [PATCH 3/3] Use architecture-independent location of
+ abrt-java-connector
+
+The connector's library has been moved to architecture-independent
+directory /usr/lib/abrt-java-connector/, because Java does not
+officially support multilib and having the connector installed the arch
+specific directory was causing us problems if Java package doesn't match
+host architecture (e.g. Java i386 on host x86_64).
+
+Signed-off-by: Jakub Filak <jfilak at redhat.com>
+---
+ build | 1 +
+ configure | 2 ++
+ java-utils/java-functions | 4 ++--
+ 3 files changed, 5 insertions(+), 2 deletions(-)
+
+diff --git a/build b/build
+index 3f5631a..336ac2a 100755
+--- a/build
++++ b/build
+@@ -65,6 +65,7 @@ expand()
+ -e "s|%{rundir}|${rundir}|" \
+ -e "s|%{sysconfdir}|${sysconfdir}|" \
+ -e "s|%{pyinterpreter}|${pyinterpreter}|" \
++ -e "s|%{abrtlibdir}|${abrtlibdir}|" \
+ "${1}" >"${target}"
+ }
+
+diff --git a/configure b/configure
+index cc8b0a1..ab9388d 100755
+--- a/configure
++++ b/configure
+@@ -69,6 +69,7 @@ jvmprivdir
+ jvmsysconfdir
+ mavenpomdir
+ pyinterpreter
++abrtlibdir
+ "
+
+ vars="$vars mavendepmapdir mavendepmapfragdir" # @Deprecated
+@@ -90,6 +91,7 @@ test -z "${rpmconfigdir}" && rpmconfigdir="${prefix}/lib/rpm"
+
+ test -z "${m2home}" && m2home="${datadir}/xmvn"
+ test -z "${pyinterpreter}" && set_pyinterpreter
++test -z "${abrtlibdir}" && abrtlibdir="${prefix}/lib/abrt-java-connector"
+
+ eval $(sed -n 's/^%_\('"$vars_re"'\)\ *\(.*\)$/\1="\2"/;T;s/%{_\(.*}\)/${\1/;p' macros.d/macros.jpackage)
+
+diff --git a/java-utils/java-functions b/java-utils/java-functions
+index c57640f..8397d2c 100644
+--- a/java-utils/java-functions
++++ b/java-utils/java-functions
+@@ -110,8 +110,8 @@ _load_java_conf()
+ JAVACMD_OPTS="${java_opts_save}"
+ fi
+
+- if [ "_${JAVA_ABRT}" != "_off" ] && [ -f %{libdir}/libabrt-java-connector.so ] && [ -f %{rundir}/abrt/abrtd.pid ]; then
+- JAVACMD_OPTS="${JAVACMD_OPTS} -agentpath:%{libdir}/libabrt-java-connector.so=abrt=on"
++ if [ "_${JAVA_ABRT}" != "_off" ] && [ -f %{abrtlibdir}/libabrt-java-connector.so ] && [ -f %{rundir}/abrt/abrtd.pid ]; then
++ JAVACMD_OPTS="${JAVACMD_OPTS} -agentpath:%{abrtlibdir}/libabrt-java-connector.so=abrt=on"
+ fi
+ }
+
+--
+1.9.3
+
diff --git a/javapackages-tools.spec b/javapackages-tools.spec
index 00c6ef0..bbb6c10 100644
--- a/javapackages-tools.spec
+++ b/javapackages-tools.spec
@@ -18,6 +18,7 @@ Patch0: 0001-mvn_artifact-Append-dependencies-to-metadata-if-we-h.patch
Patch1: 0001-metadata-Read-OSGi-Requires-from-manifest-only-if-os.patch
Patch2: 0001-pom_editor-Fix-missing-space-between-xmlns-declarati.patch
Patch3: 0002-Use-wrapper-script-to-inject-extra-JVM-arguments.patch
+Patch4: 0003-Use-architecture-independent-location-of-abrt-java-c.patch
BuildArch: noarch
@@ -173,6 +174,7 @@ This package provides non-essential macros and scripts to support Java packaging
%patch1 -p1
%patch2 -p1
%patch3 -p1
+%patch4 -p1
%if 0%{?with_python3}
find . -name '*.py' | xargs sed -i '1s|^#!.*python|#!%{__python3}|'
@@ -240,6 +242,7 @@ popd
%changelog
* Thu Oct 30 2014 Mikolaj Izdebski <mizdebsk at redhat.com> - 4.2.0-5
- Use wrapper script to inject ABRT agent JVM argument
+- Fix path to ABRT agent DSO
- Resolves: rhbz#1153652
* Tue Oct 21 2014 Michael Simacek <msimacek at redhat.com> - 4.2.0-4
More information about the scm-commits
mailing list