Skip to content

Conversation

@camelpunch
Copy link
Contributor

@camelpunch camelpunch commented Dec 4, 2024

Binding the entire /dev directory inside the Bubblewrap environment appears to fix MIDI, which broke after the Bubblewrap change in PR #360198.

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

Add a 👍 reaction to pull requests you find important.

Binding the entire /dev directory inside the Bubblewrap environment
appears to fix MIDI, which broke after the Bubblewrap change
in PR NixOS#360198.
@github-actions github-actions bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. labels Dec 4, 2024
@polygon
Copy link
Contributor

polygon commented Dec 4, 2024

Thanks for fixing this. For some reason, I put an outdated version of that file in my PR, very sorry for that. Actually, it should just work if you replace --bind / / with --dev-bind / / in the original code. Though I don't have a strong opinion if a separate bind for /dev is better.

@camelpunch
Copy link
Contributor Author

I have no clue either - I just tried things until it worked! Happy to change it if you think it makes more sense.

@ofborg ofborg bot added 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. and removed 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. labels Dec 5, 2024
@wegank wegank added the 12.approvals: 2 This PR was reviewed and approved by two persons. label Dec 8, 2024
@bfortz
Copy link
Contributor

bfortz commented Dec 11, 2024

Pinging @drupol : could you merge this please, it fixes an important issue.

@drupol drupol merged commit 4c43880 into NixOS:master Dec 11, 2024
39 of 41 checks passed
@drupol
Copy link
Contributor

drupol commented Dec 11, 2024

sure, done!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 12.approvals: 2 This PR was reviewed and approved by two persons.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants