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   
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: ---

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}
<mock-chroot> sh-4.4# cat /etc/fedora-release 
Fedora release 27 (Rawhide)
<mock-chroot> sh-4.4# rpm -qa | grep javapackages-tools

<mock-chroot> sh-4.4# rpmbuild  --eval %{_jvmjardir}
<mock-chroot> sh-4.4# cat /etc/fedora-release 
Fedora release 26 (Twenty Six)
<mock-chroot> sh-4.4#  rpm -qa | grep javapackages-tools

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


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.