Skip to content

Commit

Permalink
Update script to match oracle website changes
Browse files Browse the repository at this point in the history 
flexiondotorg#55
  • Loading branch information
@kaharlichenko
kaharlichenko committed Aug 31, 2012
1 parent d2a5841 commit 75bbd89
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion oab-java.sh
View file
Original file line number Diff line number Diff line change
Expand Up @@ -440,7 +440,7 @@ pid=$!;progress $pid

# See if the Java version is on the download frontpage, otherwise look for it in
# the previous releases page.
DOWNLOAD_INDEX=`grep -P -o "/technetwork/java/javase/downloads/jdk${JAVA_VER}-downloads-\d+\.html" /tmp/oab-index.html | uniq`
DOWNLOAD_INDEX=`grep -P -o "/technetwork/java/javase/downloads/jdk${JAVA_VER}u${JAVA_VER}-downloads-\d+\.html" /tmp/oab-index.html | uniq`
ncecho " [x] Getting current release download page "
wget http://www.oracle.com/${DOWNLOAD_INDEX} -O /tmp/oab-download.html >> "$log" 2>&1 &
pid=$!;progress $pid
Expand Down

4 comments on commit 75bbd89

@abatistas1709
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With this commit the error still occurs.

It seems that the new line should be:

DOWNLOAD_INDEX=grep -P -o "/technetwork/java/javase/downloads/jdk${JAVA_VER}u${JAVA_UPD}-downloads-\d+\.html" /tmp/oab-index.html | uniq

@kaharlichenko
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You are totally right, it was I nasty copy & paste error :(. Thanks for feedback.

@mikkorantalainen
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This fixes the issues for Java 7u7. Has anybody verified that this does not break Java 6?

@kaharlichenko
Copy link
Owner Author

@kaharlichenko kaharlichenko commented on 75bbd89 Sep 3, 2012 via email

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.