Open psumbera opened 6 months ago
Thanks for the report! Would you be able to submit a PR addressing these issues in quinn-udp? #1863 might serve as an example of another recent PR adding platform support. It would be great if there would be a way to test that this doesn't regress in CI, if you know of a way doing that.
I will do that. But before that following need to be resolved. https://github.com/rust-lang/libc/pull/3717
It's okay to vendor those definitions in quinn-udp as a stopgap until libc updates.
For the sake of completeness, corresponding discussion on Mozilla's Bugzilla: https://bugzilla.mozilla.org/show_bug.cgi?id=1901295#c9
@psumbera what do you think of keeping this open to track adding a check in CI?
In case running the unit tests on Solaris on CI is too much work, cross-compilation only would be a good step already, e.g. would have caught https://github.com/quinn-rs/quinn/pull/2029.
I'm all to have Solaris on CI. I can imagine it can be quite complicated though. So let's keep this bug to track this effort.
@psumbera what do you think of keeping this open to track adding a check in CI?
In case running the unit tests on Solaris on CI is too much work, cross-compilation only would be a good step already, e.g. would have caught #2029.
@mxinden, can you please point me how to play with FreeBSD CI and/or with cross-compiling. I have Linux to play..
EDITED: Anyway. I was able to enable github actions and start Solaris there. So I'm on it :-) Now I need to provide recent Rust version as Rustup doesn't work for Solaris yet.