guestfs-tools updates, flakify #15

Open
astro wants to merge 13 commits from astro/wfvm:master into master
astro commented 4 months ago
Collaborator

Pick esp. the guestfs-tools commit if you want to update your dependencies. The package is only now in nixos-unstable, eventually arriving in nixos-22.11.

I am using WFVM as a Flake now.

Pick esp. the guestfs-tools commit if you want to update your dependencies. The package is only now in nixos-unstable, eventually arriving in nixos-22.11. I am using WFVM as a Flake now.
astro added 9 commits 4 months ago
astro changed title from Updates to guestfs-tools updates, flakify 4 months ago
sb10q requested review from mwojcik 4 months ago
Owner

@astro Thanks a lot!

@astro Thanks a lot!
Owner

Looks good to me. Simple enough.

I'd just ask for updating README instructions with flake with at least a mention - maybe add demo-image derivation there too, although that's not entirely necessary.

Looks good to me. Simple enough. I'd just ask for updating README instructions with flake with at least a mention - maybe add demo-image derivation there too, although that's not entirely necessary.
Owner

Actually I would be very curious how you actually use it with flakes, as I can't get it to work. I'd be very grateful if you left me some more instructions maybe, either here or Mattermost.
I got the original Win10_21H1 ISO file, however calling:

nix repl
:lf .
:b lib.makeWindowsImage { imageSelection = "Windows 10 Pro"; productKey = "..."; }```
causes it to hang at:

[1/3/5 built, 0.0 MiB DL] building RESTRICTDIST-windows.img: qxl_send_events: spice-server bug: guest stopped, ignoring


which is something I have experienced already...
Actually I would be very curious how you actually use it with flakes, as I can't get it to work. I'd be very grateful if you left me some more instructions maybe, either here or Mattermost. I got the original Win10_21H1 ISO file, however calling: ``` nix repl :lf . :b lib.makeWindowsImage { imageSelection = "Windows 10 Pro"; productKey = "..."; }``` causes it to hang at: ``` [1/3/5 built, 0.0 MiB DL] building RESTRICTDIST-windows.img: qxl_send_events: spice-server bug: guest stopped, ignoring ``` which is something I have experienced already...
astro added 4 commits 4 months ago
Poster
Collaborator

I can reproduce the hang with demo-image...

I can reproduce the hang with demo-image...
Owner

Side note: maybe the MSVC stuff should be done as impure derivations, which have recently landed in Nix. In true Windows ME fashion, Microsoft worked very hard to make downloads and installations of the VS trashware as non-reproducible as possible, and the hashes there aren't really useful as they don't last longer than a few weeks.

Eventually we will move to MSYS2 which doesn't have this kind of problem, and also does not contain spyware.

Side note: maybe the MSVC stuff should be done as impure derivations, which have recently landed in Nix. In true Windows ME fashion, Microsoft worked very hard to make downloads and installations of the VS trashware as non-reproducible as possible, and the hashes there aren't really useful as they don't last longer than a few weeks. Eventually we will move to MSYS2 which doesn't have this kind of problem, and also does not contain spyware.
Poster
Collaborator

The demo-image only hangs for me in pure mode, not with impureMode = true. Upon the 20th retry the installation went through. This heisenbug makes me desperate, I need new ideas.

The demo-image only hangs for me in pure mode, not with `impureMode = true`. Upon the 20th retry the installation went through. This heisenbug makes me desperate, I need new ideas.
Poster
Collaborator

Also, the msvc layer requires an image to run on. In Windows 10 21H2 however the installation aborts early (hangs the build) if productKey is missing.

If running without a product key is still your use-case we should find an up to date version that runs for 90d like that.

Also, the msvc layer requires an image to run on. In Windows 10 21H2 however the installation aborts early (hangs the build) if `productKey` is missing. If running without a product key is still your use-case we should find an up to date version that runs for 90d like that.

Reviewers

mwojcik was requested for review 4 months ago
This pull request can be merged automatically.
You are not authorized to merge this pull request.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: M-Labs/wfvm#15
Loading…
There is no content yet.