Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix wget argument #2085

Merged
merged 2 commits into from
Jul 6, 2018
Merged

Fix wget argument #2085

merged 2 commits into from
Jul 6, 2018

Conversation

k2la
Copy link
Contributor

@k2la k2la commented Jul 5, 2018

Motivation

In https://pulsar.incubator.apache.org/docs/latest/getting-started/LocalCluster/ , wget argument is wrong.

$ wget http://www.apache.org/dyn/closer.cgi/incubator/pulsar/pulsar-2.0.1-incubating/apache-pulsar-2.0.1-incubating-bin.tar.gz

HTML file is downloaded if this command is executed.

$ wget http://www.apache.org/dyn/closer.cgi/incubator/pulsar/pulsar-2.0.1-incubating/apache-pulsar-2.0.1-incubating-bin.tar.gz
--2018-07-05 16:08:33--  http://www.apache.org/dyn/closer.cgi/incubator/pulsar/pulsar-2.0.1-incubating/apache-pulsar-2.0.1-incubating-bin.tar.gz
Resolving www.apache.org... 95.216.24.32, 40.79.78.1, 2a01:4f9:2a:185f::2
Connecting to www.apache.org|95.216.24.32|:80... connected.
HTTP request sent, awaiting response... 200 OK
Length: unspecified [text/html]
Saving to: “apache-pulsar-2.0.1-incubating-bin.tar.gz”

    [  <=>                                                                                                                                                                                                                 ] 18,878      66.9K/s   in 0.3s

2018-07-05 16:08:33 (66.9 KB/s) - “apache-pulsar-2.0.1-incubating-bin.tar.gz” saved [18878]

$ file apache-pulsar-2.0.1-incubating-bin.tar.gz
apache-pulsar-2.0.1-incubating-bin.tar.gz: HTML document text

Modifications

I changed wget argument:

$ wget 'https://www.apache.org/dyn/mirrors/mirrors.cgi?action=download&filename=incubator/pulsar/pulsar-2.0.1-incubating/apache-pulsar-2.0.1-incubating-bin.tar.gz' -O apache-pulsar-2.0.1-incubating-bin.tar.gz

Result

It enabled us to download apache-pulsar-2.0.1-incubating-bin.tar.gz successfully.

@k2la
Copy link
Contributor Author

k2la commented Jul 5, 2018

retest this please

2 similar comments
@k2la
Copy link
Contributor Author

k2la commented Jul 5, 2018

retest this please

@k2la
Copy link
Contributor Author

k2la commented Jul 5, 2018

retest this please

@nkurihar
Copy link
Contributor

nkurihar commented Jul 6, 2018

retest this please

@nkurihar nkurihar added the doc Your PR contains doc changes, no matter whether the changes are in markdown or code files. label Jul 6, 2018
@nkurihar nkurihar merged commit 0420e59 into apache:master Jul 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Your PR contains doc changes, no matter whether the changes are in markdown or code files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants