Bug 1473896

Summary: javapackages-tools 5 no longer define _jvmjardir
Product: [Fedora] Fedora Reporter: jiri vanek <jvanek>
Component: javapackages-toolsAssignee: Mikolaj Izdebski <mizdebsk>
Status: CLOSED NOTABUG QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: ctubbsii, java-sig-commits, mat.booth, mizdebsk, msimacek, msrb
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-07-24 06:58:30 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description jiri vanek 2017-07-22 06:53:06 UTC
_jvmjardir is no longer defined for rpmbuild in rawhide, causing jdk build to fail: https://koji.fedoraproject.org/koji/taskinfo?taskID=20645833

<mock-chroot> sh-4.4#  rpmbuild  --eval %{_jvmjardir}
%{_jvmjardir}
<mock-chroot> sh-4.4# cat /etc/fedora-release 
Fedora release 27 (Rawhide)
<mock-chroot> sh-4.4# rpm -qa | grep javapackages-tools
javapackages-tools-5.0.0-4.fc27.noarch


<mock-chroot> sh-4.4# rpmbuild  --eval %{_jvmjardir}
/usr/lib/jvm-exports
<mock-chroot> sh-4.4# cat /etc/fedora-release 
Fedora release 26 (Twenty Six)
<mock-chroot> sh-4.4#  rpm -qa | grep javapackages-tools
javapackages-tools-4.7.0-15.fc26.noarch


If you removed this dir/macro intentionally, I will be most happy to remove all usages of this macro.

TY!

Comment 1 Mikolaj Izdebski 2017-07-24 04:48:01 UTC
It was removed intentionally in https://github.com/fedora-java/javapackages/commit/e0e4cad

Sorry, I thought I checked that nothing used it before removal. If you think that this macro is still needed, I can re-add it. But it may be better to define it in OpenJDK spec file (or remove its usage, if possible).

Comment 2 jiri vanek 2017-07-24 06:58:30 UTC
I was hoping for this :) I hated those export dirs even in time they had some sense.

Removed
http://pkgs.fedoraproject.org/cgit/rpms/java-1.8.0-openjdk.git/commit/?id=4f0aee953b378391809ed0412962b9b9cd36a995
https://koji.fedoraproject.org/koji/buildinfo?buildID=921000

TY!