[BUILD][HOTFIX] Download Maven from regular mirror network rather than archive.apache.org #12262
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
JoshRosen
changed the title
Member
|
LGTM |
| install_mvn() { | ||
| local MVN_VERSION="3.3.9" | ||
| local APACHE_MIRROR=${APACHE_MIRROR:-https://archive.apache.org/dist} | ||
| local APACHE_MIRROR=${APACHE_MIRROR:-'https://www.apache.org/dyn/closer.lua?action=download&filename='} |
Member
There was a problem hiding this comment.
+1 i didn't know you could get a direct download out of the closer.lua redirector. Looks like it works.
Contributor
Author
There was a problem hiding this comment.
Yeah, I had to search around a bit and found a couple examples of other scripts / projects doing this. It's a bummer that this isn't authoritatively documented anywhere.
Contributor
Author
|
We should probably apply a similar fix in the other active backport branches so that their tests don't fail. The patch will be slightly different, though, because those branches are missing #12250. |
Contributor
Author
|
Merging to master now. |
|
Test build #55373 has finished for PR 12262 at commit
|
asfgit
pushed a commit
that referenced
this pull request
Apr 11, 2016
…n archive.apache.org [archive.apache.org](https://archive.apache.org/) is undergoing maintenance, breaking our `build/mvn` script: > We are in the process of relocating this service. To save on the immense bandwidth that this service outputs, we have put it in maintenance mode, disabling all downloads for the next few days. We expect the maintenance to be complete no later than the morning of Monday the 11th of April, 2016. This patch fixes this issue by updating the script to use the regular mirror network to download Maven. (This is a backport of #12262 to 1.6) Author: Josh Rosen <[email protected]> Closes #12307 from JoshRosen/fix-1.6-mvn-download.
zzcclp
pushed a commit
to zzcclp/spark
that referenced
this pull request
Apr 12, 2016
…n archive.apache.org [archive.apache.org](https://archive.apache.org/) is undergoing maintenance, breaking our `build/mvn` script: > We are in the process of relocating this service. To save on the immense bandwidth that this service outputs, we have put it in maintenance mode, disabling all downloads for the next few days. We expect the maintenance to be complete no later than the morning of Monday the 11th of April, 2016. This patch fixes this issue by updating the script to use the regular mirror network to download Maven. (This is a backport of apache#12262 to 1.6) Author: Josh Rosen <[email protected]> Closes apache#12307 from JoshRosen/fix-1.6-mvn-download. (cherry picked from commit f4110cd)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
archive.apache.org is undergoing maintenance, breaking our
build/mvnscript:This patch fixes this issue by updating the script to use the regular mirror network to download Maven.