-
Notifications
You must be signed in to change notification settings - Fork 446
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
IPv6 #3
Comments
IPv6 will be definitely supported, and I expect this to happen soon. |
cc @edef1c |
I am definitely interested in working on this post #50 |
Does anyone want to contribute funds to this? |
I know @edef1c had some code they wanted to contribute to smoltcp. That would actually go a long way towards IPv6 support. |
Yeah, I need to find some spare time and probably go over some more details with you, but I've got a packet parser and stuff going, I don't think there's all that much work involved in integrating this — I'm mostly worried about testing it sufficiently exhaustively |
@edef1c Do you have a fork and branch you're working on? Two days ago I started working on an implementation of ipv6 for the |
@dlrobertson looks like you're on par with me, I'm not sure I'll have time before next week or so, have fun I guess? |
@edef1c I'll be largely unavailable for two weeks as well. While I am pumped to help with this issue, I'd hate to push you out of contributing to this. Perhaps a @whitequark thoughts on a separate ipv6 "feature branch" until the work is somewhat complete vs feature gating? |
No go on "feature branch". Just submit it in chunks however small you want, down to and including single-enum PRs (we use rebase-and-merge so this doesn't add any junk to history). |
Sounds good. Shouldn't be too hard to feature gate it |
What do you mean by "feature gate"? We're in the |
At least in the initial PR is only adding the packed parsers etc. So I just put everything behind |
tcpd: fix bug in partial reads that was breaking https in curl
I set #196 as the PR that "Fixes" this issue due to the basic server example working after the PRs changes. I'm okay with changing that if the consensus is otherwise. |
Basic on-link IPv6 support is added in #196. Any subsequent bugs or feature requrests for IPv6 should be filed as separate issues. |
…ion-benchmark added sixlowpan fragmentation benchmark
Is IPv6 something this library will support, may support, or will never support?
The text was updated successfully, but these errors were encountered: