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
open-vm-tools: 10.1.10 -> 10.3.0 #46921
Conversation
Work in progress, will remove "WIP: " from title when ready. Would like feedback on which of the things to check down below are relevant for a PR like this. Still wrapping my head around Nix* so I'll have to revisit those things when/if needed. What I have done is |
Unfortunately I don't think an automated test can be written for this as nixos can't provision vmware images, but you can manually test the binary. |
Ok, so, as part of trying to get automatic mounting of shared folders to work with open-vm-tools, I feel that I've tried this version update well enough to establish that things like copy/paste and manually mounting shared folders works just as well as in the previous version. I've also ran most of the binaries in the package successfully. Unless someone else wants more testing to be done here, I'd say merge this so we get the fix that actually makes the service run at all :-) |
Awesome! Glad you got this resolved. |
Actually, should we cherry-pick this into 18.09 as well? If so, don't merge yet if you want it in the same PR.. |
Yeah I'd like to see this merged into 18.09 as well. I don't have write access so you'll have to ping someone who does. |
I realise the backport is a separate PR, so this is ready for merging. Any idea when this one will be merged so I can cherry-pick? :) |
Often times I jump on IRC to pester people for merging changes when I'm on a deadline. This fixes a bug and is a dead simple change so you can probably find someone to merge this for you easy enough. |
@GrahamcOfBorg build open-vm-tools |
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: open-vm-tools Partial log (click to expand)
|
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: open-vm-tools Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: open-vm-tools Partial log (click to expand)
|
Motivation for this change
Update open-vm-tools to latest stable version, fixes #46527.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)