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
qemu: libiscsi support #95274
qemu: libiscsi support #95274
Conversation
Think this'll need a rebase. |
Done! |
Builds on macos 10.14. @ofborg build qemu |
Please target master this is not a mass rebuild. |
Done! |
This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch) Result of 11 packages marked as broken and skipped:
4 packages failed to build and are new build failures:
6 packages built:
</details>
diff --git a/pkgs/development/python-modules/cot/default.nix b/pkgs/development/python-modules/cot/default.nix
index 89e26545d79..0c2bb961290 100644
--- a/pkgs/development/python-modules/cot/default.nix
+++ b/pkgs/development/python-modules/cot/default.nix
@@ -35,9 +35,10 @@ buildPythonPackage rec {
"TestQCOW2"
"TestRAW"
"TestVMDKConversion"
+ ] ++ lib.optionals stdenv.isDarwin [
+ "test_serial_fixup_invalid_host"
];
-
# argparse is part of the standardlib
prePatch = ''
substituteInPlace setup.py --replace "'argparse'," "" |
It builds just fine on my machine.
|
Do you have sandbox enabled? |
Yes. I have multi-user nix with explicitly enabled sandbox. |
This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch) Result of 2 packages marked as broken and skipped:
1 package blacklisted:
14 packages failed to build and are new build failures:
34 packages built:
The following issues got detected with the above build packages. virtualboxHeadless: Please consider this feature to be alpha. A substituteInPlace with an unused --replace got detected:
Please check the offending substituteInPlace for typos or changes in source. |
aqemu and virtualbox packages are already broken on master and unrelated to this change. |
Motivation for this change
Support for booting directly from iSCSI.
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)