1387505 - for installation of kuma on macOS, use "curl -O" rather than "wget -N"

Status: RESOLVED FIXED

(developer.mozilla.org Graveyard :: General, enhancement, P5)

Description

[tangmu]

:: Developer Documentation Request

      Request Type: Correction
     Gecko Version: unspecified
 Technical Contact: 

:: Details

For installation of kuma on macOS, it would be helpful to suggest "curl -O" as an alternative to "wget -N"

Comment 1

[John Whitlock [:jwhitlock]]

I would mentor a pull request that adds that note to the docs.

The reason I used wget (and that I install it using homebrew on OSX) is that if you run it twice, it doesn't download the file if nothing changed:

$ wget -N https://mdn-downloads.s3-us-west-2.amazonaws.com/mdn_sample_db.sql.gz
--2017-08-10 09:30:02--  https://mdn-downloads.s3-us-west-2.amazonaws.com/mdn_sample_db.sql.gz
Resolving mdn-downloads.s3-us-west-2.amazonaws.com... 54.231.185.69
Connecting to mdn-downloads.s3-us-west-2.amazonaws.com|54.231.185.69|:443... connected.
HTTP request sent, awaiting response... 304 Not Modified
File ‘mdn_sample_db.sql.gz’ not modified on server. Omitting download.

7.6 MB is not a big deal for me, but we have some contributors with slow internet connections, and every little bit helps.

I see the long-term solution as downloading and install the sample DB inside the docker container, as part of automating the rest of the setup tasks.  I find that every piece of host-installed software creates support issues, and would like to limit the host-instaled software to Docker.

Comment 2

[Deep]

I would be interested in taking this up, assuming that it still needs to be fixed. Does this need to be changed in the docs/installation.rst file?

Comment 3

[John Whitlock [:jwhitlock]]

Deep,

Yes, that is the file to change. I think it would be sufficient to show an alternate command using curl.

Comment 4

[Deep]

So can I take this up and give a PR corresponding to it?

Comment 5

[Deep]

Also, in the alternate command, do I need to specify it's only for macOS (as the issue says)?

Comment 6

[John Whitlock [:jwhitlock]]

Deep: yes, a pull request is the way to submit this change. I'd phrase it like:

Alternatively, you can use curl (installed by default on macOS):

curl -O https://mdn-downloads.s3-us-west-2.amazonaws.com/mdn_sample_db.sql.gz
docker exec -i kuma_web_1 bash -c "zcat | ./manage.py dbshell" < mdn_sample_db.sql.gz

Comment 7

[Deep]

I have submitted a pull request corresponding to this bug.

Comment 8

[Deep]

This is the link for the above mentioned pull request
https://github.com/mozilla/kuma/pull/4570

Comment 9

[[github robot]]

Commits pushed to master at https://github.com/mozilla/kuma

https://github.com/mozilla/kuma/commit/f4bae53989f1955ad390338016df3acb1bb7976f
Fix bug 1387505:Add curl as an alternative to wget

- Add curl -O as an alternative to wget -N in docs/installation.rst for
  macOS users

https://github.com/mozilla/kuma/commit/a79ef1896671d05921d7179a3d141aac036e61f8
Merge pull request #4570 from coderick14/patch-for-bug-1387505

Fix bug 1387505:Add curl as an alternative to wget