Skip to content
This repository has been archived by the owner on Apr 9, 2019. It is now read-only.

Fix 7u51 build #125

Closed
wants to merge 2 commits into from
Closed

Fix 7u51 build #125

wants to merge 2 commits into from

Conversation

ladios
Copy link
Contributor

@ladios ladios commented Jan 16, 2014

This should fix #123

Since upstream v7.45-2, it requires binary files for both architectures
again. This patch updates the condition on detecting whether we need
both files or just one for the version we are building against.
Upstream v7.45-2 introduced a preparing script to download and inflate
archives. If the version we are building against contains that script,
then run it to inflate archives in order to build packages.
@ladios
Copy link
Contributor Author

ladios commented Jan 16, 2014

Some commands in the upstream's preparing script might fail silently (in my case curl exited with error when resuming the already download completed JCE package), but doesn't affect the build.

@ykulbak
Copy link

ykulbak commented Jan 16, 2014

Thanks, this fixed 7u51 for me

@nifgraup
Copy link
Contributor

Works.

@tfnico
Copy link

tfnico commented Jan 17, 2014

Thanks! Works for me on Ubuntu 12.04 👍

@shinji257
Copy link

Yup. Confirmed it fixes it for me as well. Great job!

verveguy pushed a commit to verveguy/minecloud-ami that referenced this pull request Jan 21, 2014
toffer pushed a commit to toffer/minecloud-ami that referenced this pull request Jan 21, 2014
Use patched OAB from ladios/oab-java6 to resolve upstream changes
to Oracle's java download site.

See flexiondotorg/oab-java6#125

Fixes #16.
@wlk
Copy link

wlk commented Jan 21, 2014

working for me as well

@tuxinaut
Copy link

tuxinaut commented Feb 3, 2014

Thanks! Also works for me on Ubuntu 12.04 👍

@rmorgans
Copy link

Cheers - works for me on 13.10 saucy for version 7 (the -7 flag) -> however the version 6 I have the
Unmet build dependencies: lib32asound2 ia32-libs issue (which is unrelated to this I think)

@ladios
Copy link
Contributor Author

ladios commented Feb 16, 2014

@rmorgans That should be #122 which can be fixed by #124
I merged that in my fork's master branch ladios/oab-java6@218eddf

@timmc
Copy link

timmc commented Feb 26, 2014

Confirmed on Ubuntu 12.10 Quantal Quetzal. 👍 (For -7, at least.)

@dpursehouse
Copy link

👍 Works for me on Ubuntu 12.04 LTS with -7

@dol
Copy link

dol commented Mar 10, 2014

Thx, working with '-7' on Ubuntu 12.10.

@ladios ladios mentioned this pull request Mar 31, 2014
@ladios
Copy link
Contributor Author

ladios commented Mar 31, 2014

Replaced by #130

@ladios ladios closed this Mar 31, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

oab-java.sh (0.3.0) fails in current state