-
Notifications
You must be signed in to change notification settings - Fork 14
Deprecate in favor of Requests.jl? #21
Comments
Requests.jl is supposed to have multi-part data. Do you have a test case On Sat, Aug 29, 2015 at 12:53 PM, Sebastian Good notifications@github.com
|
Problem on my end; apologies! |
No worries. On Sat, Aug 29, 2015 at 12:58 PM, Sebastian Good notifications@github.com
|
Please see open thread at JuliaWeb/Roadmap#1 (comment) |
Can ask this question on Julia Users too - feedback from users of both packages will be useful. |
I prefer HTTPClient because it is more lightweight -- Requests feels rather slow to compile and use. Also, the last time I tried to work on requests (a month or two ago) it was still not passing tests in 0.4 due to a difficult bug with the multipart file encoding. I would prefer to continue having the option to use a lightweight curl-based HTTP library. |
I think we should just have a |
Ya that makes sense. On Thu, Sep 3, 2015 at 2:35 PM, Jacob Quinn notifications@github.com
|
An experience from a new Julia user: I was looking for a HTTP client and initially got quite confused by the various Julia packages around here. It took me time to figure out the differences between As far as I understood,
Advantages:
Disadvantages:
What do you think? |
If someone wanted to take the time to do that, I think it would be great. It's probably not something any of the current maintainers will get around to do doing though. |
Could you please consider adding some notices of "this package is DEPRECATED", as Requests.jl? I confirmed that this package does not work on Julia 0.7.0 or newer. Also, a PR to make this runnable on newer Julia is ignored. I believe this is effectively deprecated and putting some notices helps new comers to avoid using this. |
Is there something this does that Requests.jl doesn't? If so, I'd prefer to get that functionality into Requests.jl so there is only one canonical HTTP client in JuliaWeb. cc @amitmurthy - looks like you are the primary contributor.
The text was updated successfully, but these errors were encountered: