Skip to content
This repository has been archived by the owner. It is now read-only.

Fix 7u51 build #125

Closed
wants to merge 2 commits into from

Conversation

Projects
None yet
@ladios
Copy link
Contributor

ladios commented Jan 16, 2014

This should fix #123

ladios added some commits Jan 16, 2014

Update condition on downloading binary file(s)
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.
Run upstream's preparing script if it exists
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

This comment has been minimized.

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

This comment has been minimized.

Copy link

ykulbak commented Jan 16, 2014

Thanks, this fixed 7u51 for me

@nifgraup

This comment has been minimized.

Copy link
Contributor

nifgraup commented Jan 17, 2014

Works.

@tfnico

This comment has been minimized.

Copy link

tfnico commented Jan 17, 2014

Thanks! Works for me on Ubuntu 12.04 👍

@shinji257

This comment has been minimized.

Copy link

shinji257 commented Jan 18, 2014

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 added a commit to toffer/minecloud-ami that referenced this pull request Jan 21, 2014

Handle upstream changes to Oracle's download site.
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

This comment has been minimized.

Copy link

wlk commented Jan 21, 2014

working for me as well

@tuxinaut

This comment has been minimized.

Copy link

tuxinaut commented Feb 3, 2014

Thanks! Also works for me on Ubuntu 12.04 👍

@rmorgans

This comment has been minimized.

Copy link

rmorgans commented Feb 16, 2014

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

This comment has been minimized.

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

This comment has been minimized.

Copy link

timmc commented Feb 26, 2014

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

@dpursehouse

This comment has been minimized.

Copy link

dpursehouse commented Mar 6, 2014

👍 Works for me on Ubuntu 12.04 LTS with -7

@dol

This comment has been minimized.

Copy link

dol commented Mar 10, 2014

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

@ladios ladios referenced this pull request Mar 31, 2014

Open

Java8 #130

@ladios

This comment has been minimized.

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.
You can’t perform that action at this time.