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

Update is_extpack_installed #5820

Merged

Conversation

computezrmle
Copy link
Contributor

For many years and across a couple of major VirtualBox versions vboxwrapper was looking for "Oracle VM VirtualBox Extension Pack" to test whether the Extension Pack is present.
With VirtualBox version 7.1 Oracle changed the string to "Oracle VirtualBox Extension Pack".

Reported by an experienced volunteer at LHC@home.

To make vboxwrapper work with old as well as with recent VirtualBox versions the test string needs to be modified.

@AenBleidd AenBleidd merged commit da545cd into BOINC:master Sep 29, 2024
146 checks passed
@Toby-Broom
Copy link

Would it not be better to just look for VBoxVRDP as this didn't change? and allows back compatabilty from 7.0.x to 7.1.x

@computezrmle
Copy link
Contributor Author

@Toby-Broom
Thanks for the comment.
Tests have already been done with 7.0.x and 7.1.x.
They didn't show any issue.

Can you live with it if the code remains as is?

@computezrmle computezrmle deleted the computezrmle-is_extpack_installed branch October 1, 2024 05:34
@Toby-Broom
Copy link

sure no problem, thanks for making the update.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Merged
Development

Successfully merging this pull request may close these issues.

3 participants