这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@omeid
Copy link
Contributor

@omeid omeid commented Oct 16, 2015

dokku already requires curl for installation so it is better to use curl to download the bootstrap script as well.

dokku already requires curl for installation so it is better to use curl to download the bootstrap script as well.
@josegonzalez
Copy link
Member

curl isn't always available - in fact the bootstrap script installs it if it isn't there - so I don't think this change will work.

@omeid
Copy link
Contributor Author

omeid commented Oct 16, 2015

That is also true for wget, so why require two different tools for the same task?

@josegonzalez
Copy link
Member

There are quite a few other places where we use wget - same for curl.

If you want to switch all invocations to one or the other, I don't mind. wget appears to be installed with ubuntu by default - at least on my test machines - so that's probably the safer bet.

@callahad
Copy link
Contributor

callahad commented Nov 5, 2015

I suggest closing as wontfix. Curl is very nice, but wget is much more commonly available.

In both Ubuntu and Debian, wget packages are set as Priority: Important, while curl is Priority: Optional. Important packages are those which "should be found on any Unix-like system" and are thus much more likely to be present in default installations of the operating system. For instance, Digital Ocean's images have wget but not curl by default.

@callahad
Copy link
Contributor

callahad commented Nov 5, 2015

More info at https://www.debian.org/doc/manuals/debian-faq/ch-pkg_basics.en.html#s-priority

If you do a default Debian installation all the packages of priority Standard or higher will be installed in your system. If you select pre-defined tasks you will get lower priority packages too.

So all default installations of Debian will have wget but not curl. Ubuntu inherits this policy from Debian.

@josegonzalez
Copy link
Member

@callahad mind making a pr to switch us away from curl to wget then? That would be nice, since then we don't need to install curl...

callahad added a commit to callahad/dokku that referenced this pull request Nov 6, 2015
callahad added a commit to callahad/dokku that referenced this pull request Nov 6, 2015
josegonzalez added a commit that referenced this pull request Nov 10, 2015
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.

3 participants