Page MenuHomeSolus

Update openjdk-8 to 8.232
Needs ReviewPublic

Authored by serebit on Mon, Oct 28, 2:09 AM.

Details

Reviewers
DataDrake
Group Reviewers
Triage Team
Summary

Update openjdk-8 to 8u232-ga.

Test Plan

Built and ran a HelloFX sample program

Diff Detail

Repository
R2242 openjdk-8
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage

Event Timeline

serebit created this revision.Mon, Oct 28, 2:09 AM
serebit requested review of this revision.Mon, Oct 28, 2:09 AM

This was built with a slightly different package.yml due to my inability to upload sources to dev.getsol.us, but the resulting pspec should be the same if I didn't make any mistakes in converting it back.

DataDrake requested changes to this revision.Sun, Nov 10, 4:55 PM
DataDrake added a subscriber: DataDrake.

hash changed?

23abb7e548611b19fc24802165d27d217657d6648b4688573a0430168e86ea77
jdk8u232-ga.tar.bz2

This revision now requires changes to proceed.Sun, Nov 10, 4:55 PM

Oh, and I updated our sources for you.

Woke up just in time! I'm attempting to build with the updated sources after changing the hash, and I'm getting an error in solbuild that I'm not sure how to solve.

mount: /var/cache/solbuild/unstable-x86_64/openjdk-8/union/home/build/YPKG/sources/corba-8u232-ga.tar.bz2: mount point does not exist.
ERRO[12:08:23] Failed to bind mount source                   error="exit status 32" target="/var/cache/solbuild/unstable-x86_64/openjdk-8/union/home/build/YPKG/sources/corba-8u232-ga.tar.bz2"
ERRO[12:08:23] Cannot continue without sources              
ERRO[12:08:24] Failed to build packages
Jacalz added a subscriber: Jacalz.Sun, Nov 10, 5:18 PM

I have gotten that error sometimes too and the only way that I have found to fix it is to run yupdate on the link once again and that fixes it. Doesn’t really work well for multiple source sadly.

9958a19a9064a18e4cf249734ff1c03ad30444bff277d560027d02b8be5c3a68 corba-u232-ga.tar.bz2
ed88a5db0e9d1f81ff966e18940e1d7d9a3256aa324b396eb0e097893291d537 hotspot-u232-ga.tar.bz2
4b402bc08e642acb2b42833996e70b284a1a02c70d4cc96ac4ece1802daeb892 jaxp-u232-ga.tar.bz2
70276b4e04ec0d3f0d936f541ff27ee4e28602f28dfab7208b44a95ffa2f1a24 jaxws-u232-ga.tar.bz2
23abb7e548611b19fc24802165d27d217657d6648b4688573a0430168e86ea77 jdk8u232-ga.tar.bz2
4c51f8063382065ea4440c4ed37695408401a0aead314a1d185d4fc4b2c36b6f jdk-u232-ga.tar.bz2
bb8b55208e72ee59dd27ca033563f312ba5608685845efdf2882ce3eafcde1c3 langtools-u232-ga.tar.bz2
65401a6f11f67c198b54846fd8e28d452ffcb95939943c432f27e01d01fde29a nashorn-u232-ga.tar.bz2

Should do ya. The next version of ypkg will support multiple sources. Sorry for the current inconvenience.

9958a19a9064a18e4cf249734ff1c03ad30444bff277d560027d02b8be5c3a68 corba-u232-ga.tar.bz2
ed88a5db0e9d1f81ff966e18940e1d7d9a3256aa324b396eb0e097893291d537 hotspot-u232-ga.tar.bz2
4b402bc08e642acb2b42833996e70b284a1a02c70d4cc96ac4ece1802daeb892 jaxp-u232-ga.tar.bz2
70276b4e04ec0d3f0d936f541ff27ee4e28602f28dfab7208b44a95ffa2f1a24 jaxws-u232-ga.tar.bz2
23abb7e548611b19fc24802165d27d217657d6648b4688573a0430168e86ea77 jdk8u232-ga.tar.bz2
4c51f8063382065ea4440c4ed37695408401a0aead314a1d185d4fc4b2c36b6f jdk-u232-ga.tar.bz2
bb8b55208e72ee59dd27ca033563f312ba5608685845efdf2882ce3eafcde1c3 langtools-u232-ga.tar.bz2
65401a6f11f67c198b54846fd8e28d452ffcb95939943c432f27e01d01fde29a nashorn-u232-ga.tar.bz2
Should do ya. The next version of ypkg will support multiple sources. Sorry for the current inconvenience.

Swapped to those sources, and oddly enough, it's still giving me the error. I'll push the changes regardless.

serebit updated this revision to Diff 18255.Sun, Nov 10, 5:59 PM

Fixed hash for jdk8u232-ga.tar.bz2 and changed source URLs

lol guess we are both having a day. You used the langtools hash for nashorn and have the wrong hash for both in langtools.

lol guess we are both having a day. You used the langtools hash for nashorn and have the wrong hash for both in langtools.

...Yeah, that would do it. Building now, thanks.

serebit updated this revision to Diff 18256.Sun, Nov 10, 6:41 PM

Fix incorrect hashes, now builds successfully

serebit updated this revision to Diff 18258.Sun, Nov 10, 6:42 PM

Remove whitespace that bothers me