-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
fix(sys/hermit): remove obsolete network initialization #115309
Conversation
r? @cuviper (rustbot has picked a reviewer for you, use r? to override) |
This function does not exist as of hermit-sys 0.4.1 [1]. Once std does not call this function, we can remove it entirely. [1]: hermit-os/hermit-rs@e38f246#diff-a9b7fa816defab285f0d4fe69d4df4a0cbbcf1c89913000df3273aded949f257R10 Signed-off-by: Martin Kröning <[email protected]>
647faa2
to
5853c2b
Compare
I haven't looked at hermit before -- how does |
Sorry for the confusion. Hermit is a library operating system. That means users explicitly link their application against the whole OS to create a unikernel image, that can be run in VMs. Here, So users have to update |
OK, thanks for explaining! @bors r+ rollup |
☀️ Test successful - checks-actions |
Finished benchmarking commit (fef2f59): comparison URL. Overall result: no relevant changes - no action needed@rustbot label: -perf-regression Instruction countThis benchmark run did not return any relevant results for this metric. Max RSS (memory usage)ResultsThis is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.
CyclesThis benchmark run did not return any relevant results for this metric. Binary sizeThis benchmark run did not return any relevant results for this metric. Bootstrap: 629.903s -> 630.211s (0.05%) |
This function does not exist as of hermit-sys 0.4.1 (
e38f246
). Once std does not call this function, we can remove it entirely.CC: @stlankes