Skip to content

Conversation

drichardson
Copy link
Contributor

Changes

Use -L option in curl command to follow redirect returned by www.kernel.org. Otherwise, the file you are testing with is not the ~5MB git.tgz, but a 168 byte HTML page informing you about the redirect.

Context

Everything under https://www.kernel.org/pub/ seems to 301 (moved permanently) redirect to https://mirrors.edge.kernel.org/pub/. So the other option is to change the URL instead of adding the -L curl option. However, the www.kernel.org homepage still links to them as https://www.kernel.org/pub/, so it isn't clear to me what the most stable URL is, which is why I went with the -L.

@ben
Copy link
Member

ben commented Jul 2, 2021

Hmmm. Would it make more sense to update that URL to something more current? Do the newer releases have stable URLs?

@drichardson
Copy link
Contributor Author

I don't know. Since the content of the tarball is irrelevant in that example anyway, maybe you should just link to something else under your control (e.g. something on github) so you don't have to deal with the possibility of it changing.

@ben
Copy link
Member

ben commented Jul 3, 2021

Yeah, I guess you're right. Thanks!

@ben ben merged commit 75873a9 into progit:main Jul 3, 2021
@drichardson drichardson deleted the fix-curl branch July 3, 2021 18:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants