Bug 913120 - maven-local: mvn-rpmbuild does not pass MAVEN_OPTS to Maven
Summary: maven-local: mvn-rpmbuild does not pass MAVEN_OPTS to Maven
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: javapackages-tools
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Mikolaj Izdebski
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 863801 914595
TreeView+ depends on / blocked
 
Reported: 2013-02-20 12:37 UTC by Krzysztof Daniel
Modified: 2014-01-13 00:27 UTC (History)
7 users (show)

Fixed In Version: 0.12.2-1
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-02-20 12:52:29 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Krzysztof Daniel 2013-02-20 12:37:41 UTC
To build Eclipse on arm it is necessary to disable JIT for certain method. This is done by:
export MAVEN_OPTS="-Xmx700m -XX:CompileCommand=exclude,org/eclipse/tycho/core/osgitools/EquinoxResolver,newState"

Unfortunately, after the migration to xmvn, the build starting failing again as if the exclusion would not be applied. Is it possible that the MAVEN_OPTS is lost somewhere between calling 

mvn-rpmbuild clean install \
   -Dmaven.test.skip=true -Dnative=gtk.linux.%{eclipse_arch} -DskipTychoVersionCheck \
   -Dmaven.local.mode=true -Dtycho.local.keepTarget -Dbuilddoc=$((%{bootstrap} != 1))

and the underlying maven?

Comment 1 Mikolaj Izdebski 2013-02-20 12:39:43 UTC
mvn-rpmbuild is part of maven-local, not xmvn. Reassigning to javapackages-tool component.

Comment 2 Mikolaj Izdebski 2013-02-20 12:52:29 UTC
I believe that this bug is fixed in javapackages-tools-0.12.2-1,
which is available in Fedora Rawhide.  I am closing the bug now,
thank you for reporting it.  A summary of changes is included below.

* Wed Feb 20 2013 Mikolaj Izdebski <mizdebsk> - 0.12.2-1
- Update to upstream version 0.12.2
- Resolves: rhbz#913120 (MAVEN_OPTS are not passed to Maven)

Comment 3 Michael Simacek 2013-11-13 09:52:14 UTC
Testcase for mvn_build, which replaces mvn_rpmbuild, was added in commit 1f1487d.


Note You need to log in before you can comment on or make changes to this bug.