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.

Sign up for GitHub

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

Jump to bottom

Get rid of curl #40

Open
frankshearar opened this issue Oct 21, 2013 · 4 comments
Open

Get rid of curl #40

frankshearar opened this issue Oct 21, 2013 · 4 comments

Comments

@frankshearar
Copy link
Member

Just use Net::HTTP or something?
@krono
Copy link
Member

krono commented Oct 21, 2013

but why?

@frankshearar
Copy link
Member Author

One less external dependency! If it's in Ruby, Ruby will pull the dependent libraries as necessary.

@krono
Copy link
Member

krono commented Oct 21, 2013

I think tools like curl tend to be much more stable than using Ruby libs for the same thing…

@frankshearar
Copy link
Member Author

Maybe so, but right now it's one more thing that you have to install yourself. With this PR, and #41, and possibly a few more, all you would need to do to is have a Ruby, and run "bundle install" to get a complete dev environment.

Shelling out to curl & tar was great to get things started, but using Ruby makes multiplatform support much easier.

frankshearar added a commit that referenced this issue Nov 7, 2013
@frankshearar
We need cURL. But see #40.
989871b
jgonis pushed a commit to jgonis/squeak-ci that referenced this issue Jan 9, 2015
@frankshearar @jgonis
We need cURL. But see squeak-smalltalk#40.
17979b4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@krono @frankshearar