Skip to content
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: set localstatedir #112886

Merged
merged 1 commit into from
Mar 29, 2021
Merged

qemu: set localstatedir #112886

merged 1 commit into from
Mar 29, 2021

Conversation

alyssais
Copy link
Member

Otherwise qemu-qa, qemu-pr-helper, and virtiofsd, try to write to
$out/var at runtime.

Motivation for this change
Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.

@SuperSandro2000 SuperSandro2000 added the 2.status: merge conflict This PR has merge conflicts with the target branch label Mar 21, 2021
@SuperSandro2000
Copy link
Member

@ofborg eval

@illustris
Copy link
Contributor

Merge conflict caused by the removal of this line in c3b9c62

@RaitoBezarius
Copy link
Member

RaitoBezarius commented Mar 25, 2021

Still important to me. Let me know if I can take over and shepherd the PR so that it can get merged.

@SuperSandro2000
Copy link
Member

Still important to me. Let me know if I can take over and shepherd the PR so that it can get merged.

Feel free. It would also be great if we could use a patch instead of sed.

@RaitoBezarius RaitoBezarius mentioned this pull request Mar 26, 2021
10 tasks
Otherwise qemu-qa, qemu-pr-helper, and virtiofsd, try to write to
$out/var at runtime.

Fixes: NixOS#113909
Fixes: NixOS#112886
@ofborg ofborg bot removed the 2.status: merge conflict This PR has merge conflicts with the target branch label Mar 29, 2021
@alyssais
Copy link
Member Author

alyssais commented Mar 29, 2021 via email

@alyssais alyssais merged commit 8e71416 into NixOS:master Mar 29, 2021
@alyssais alyssais deleted the qemu-localstatedir branch March 29, 2021 17:19
@RaitoBezarius
Copy link
Member

Thank you a lot for your work @alyssais !

risicle pushed a commit to risicle/nixpkgs that referenced this pull request Apr 27, 2021
Otherwise qemu-qa, qemu-pr-helper, and virtiofsd, try to write to
$out/var at runtime.

Fixes: NixOS#113909
Fixes: NixOS#112886
(cherry picked from commit 8e71416)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants