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

curl: move to hyper backend #357409

Closed
milibopp opened this issue Nov 19, 2024 · 7 comments
Closed

curl: move to hyper backend #357409

milibopp opened this issue Nov 19, 2024 · 7 comments
Labels
9.needs: community feedback significant Novel ideas, large API changes, notable refactorings, issues with RFC potential, etc.

Comments

@milibopp
Copy link
Contributor

milibopp commented Nov 19, 2024

Project description

Hyper and curl are looking for Linux distributions to ship curl with hyper as an HTTP backend. The work is almost done, but not quite and they want someone (a Linux distro, e.g.) to commit to shipping it.

We already include rustls as an optional dependency for curl. Supporting hyper-ffi seems fairly similar.


Add a 👍 reaction to issues you find important.

@milibopp milibopp added the 0.kind: packaging request Request for a new package to be added label Nov 19, 2024
@milibopp
Copy link
Contributor Author

cc @seanmonstar @lovek323

@getchoo getchoo mentioned this issue Nov 20, 2024
13 tasks
@milibopp milibopp changed the title Add support for hyper to curl curl: move to hyper backend Nov 20, 2024
@milibopp
Copy link
Contributor Author

Updated title according to this suggestion.

@JohnRTitor JohnRTitor added 9.needs: community feedback significant Novel ideas, large API changes, notable refactorings, issues with RFC potential, etc. and removed 0.kind: packaging request Request for a new package to be added labels Nov 20, 2024
@nrabulinski
Copy link
Member

I asked about it on matrix yesterday and @K900 said that we can’t as curl is part of nixpkgs bootstrap, sadly

@niklaskorz
Copy link
Contributor

I asked about it on matrix yesterday and @K900 said that we can’t as curl is part of nixpkgs bootstrap, sadly

In what sense? That bootstrap can't have Rust dependencies?

@K900
Copy link
Contributor

K900 commented Nov 21, 2024

It can, in theory, but we'd have to pull Rust into the bootstrap, which is a long and rather painful process, or we'd need to have two differently behaving versions of curl in the bootstrap and the final system, which is also not great.

@stephank
Copy link
Contributor

Could gcc-rs make it easier to introduce Rust in the bootstrap? Though no idea if it's far enough along for building Hyper.

There don't currently seem to be any other incentives to add Rust to the bootstrap, looks like? Can't really find anything with a quick search, but perhaps more incentives make the effort more worthwhile.

(Random thoughts)

@niklaskorz
Copy link
Contributor

niklaskorz commented Dec 22, 2024

Curl has officially removed hyper support: https://daniel.haxx.se/blog/2024/12/21/dropping-hyper/

The hyper backend code has been removed in git as of December 21. There will be no traces left of it in the curl 8.12.0 release coming in February 2025.

@vcunat vcunat closed this as not planned Won't fix, can't repro, duplicate, stale Dec 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
9.needs: community feedback significant Novel ideas, large API changes, notable refactorings, issues with RFC potential, etc.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants