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

vboxwrapper: virtualbox home directory fix MacOS #6019

Closed
wants to merge 2 commits into from

Conversation

NenTech
Copy link

@NenTech NenTech commented Jan 23, 2025

Directory was changed but not assigned to the environmental variable. Which caused a failure on MacOS.

Only change which need to be made for this pull request to work is create directory .VirtualBox by default or give boinc_project access to BOINC_data to be able to write the folder.

Related to #6016 and #6018

NenTech and others added 2 commits January 23, 2025 23:36
bug fix which prevented starting vboxmanage on MacOS
vboxwrapper: virtualbox home directory fix MacOS
@NenTech
Copy link
Author

NenTech commented Jan 23, 2025

@davidpanderson
@computezrmle
I hope I did this the right way..

@davidpanderson
Copy link
Contributor

Yes, the setenv() was definitely needed.
I'm going to fold this into another PR that puts .VirtualBox
in the BOINC data directory, not the user's home directory.
I don't think BOINC should touch the home directory
(and it may not have access to it anyway, like on the Mac)

@davidpanderson
Copy link
Contributor

This change is incorporated in #6018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants