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

Restore purged curl to ubuntu boxes #280

Closed
wants to merge 1 commit into from
Closed

Restore purged curl to ubuntu boxes #280

wants to merge 1 commit into from

Conversation

eshamow
Copy link
Contributor

@eshamow eshamow commented Oct 11, 2014

Despite sharing a post-install script with Debian, purge on Ubuntu removes curl. This is a clumsy workaround, but it does ensure that curl is available on the box, which provides parity with other platforms.

Also removes a trailing space in an unrelated file.

@beddari
Copy link

beddari commented Oct 13, 2014

Hoping this will be merged as having curl on the boxes pushes back the point-in-time ⨝ version when SNI is supported by quite a stretch.

@juliandunn
Copy link
Contributor

I'm not so sure I care that much about "parity against other platforms", particularly insofar as curl is concerned. I'd gladly strip it out of all the boxes actually, assuming the entire userland wouldn't break...

@eshamow
Copy link
Contributor Author

eshamow commented Oct 15, 2014

It's more the fact that since it's available everywhere else, any downstream tool that uses these boxes has to specifically check for Ubuntu and then install curl.

It's such a basic tool to be able to grab anything - even a deb to install packages.

That said if you want it gone, happy to fix parity by closing this PR and opening a new one to pull from all. It just seems like a sensible and minimal tool to have across the board, and a small one.

@juliandunn
Copy link
Contributor

Yeah, I think that's the direction I'd rather go @eshamow.

juliandunn pushed a commit to juliandunn/bento that referenced this pull request Oct 30, 2014
@juliandunn
Copy link
Contributor

As discussed I'm going to close this as WONTFIX.

@juliandunn juliandunn closed this Nov 13, 2014
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