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
perlPackages.IOAsync: 0.77 -> 0.78 #110440
Conversation
Hey. It would be great to have the |
On a sidenote, it would be great to avoid |
@stigtsp yep, I only made this NetAsyncPing change as it now fails unlike before, indeed better to still keep some basic tests in and just disable/change network tests. Let me spin that off into a separate PR 💪 |
This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch). Result of 10 packages built:
|
This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch). Result of 10 packages built:
|
40d9779
to
79ff3de
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Thanks @SuperSandro2000 @stigtsp ! 🎉 |
Motivation for this change
https://metacpan.org/release/PEVANS/IO-Async-0.78
This includes a minor change for
perlPackages.NetAsyncPing
to disable network tests (i.e. cannot ping onlocalhost
, and also needs to pull inpkgs.iputils
for/bin/ping
if otherwise.) Let me know if this particular change needs its own PR.Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)