Bug 1473896 - javapackages-tools 5 no longer define _jvmjardir
javapackages-tools 5 no longer define _jvmjardir
Status: CLOSED NOTABUG
Product: Fedora
Classification: Fedora
Component: javapackages-tools (Show other bugs)
rawhide
Unspecified Unspecified
unspecified Severity unspecified
: ---
: ---
Assigned To: Mikolaj Izdebski
Fedora Extras Quality Assurance
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2017-07-22 02:53 EDT by jiri vanek
Modified: 2017-07-24 02:58 EDT (History)
6 users (show)

See Also:
Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2017-07-24 02:58:30 EDT
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)

  None (edit)
Description jiri vanek 2017-07-22 02:53:06 EDT
_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 00:48:01 EDT
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 02:58:30 EDT
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!

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