web: Fix auto-attach process for VBox projects #3080
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Problem: the new signup/download process (auto-attach) wasn't telling Mac and Linux users to install VirtualBox for projects that require it. (These platforms don't have a combined BOINC/VBox installer; the user has to install VBox separately). Solution: add links to the VirtualBox download page in appropriate places. Also: we currently have a <need_vbox/> config option for projects that require VBox (i.e. all their apps are VM) Add a <recommend_vbox/> option for projects with some but not all VM apps. Also: rename download.php to download_software.php. There was a name conflict with a file from the boinc-site repo. This caused problems on the BOINC web site, which uses both code bases. Projects with a link to download.php on their home page should change this to download_software.php.
|
I am happy to merge if there are no objections. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Problem: the new signup/download process (auto-attach) wasn't telling
Mac and Linux users to install VirtualBox for projects that require it.
(These platforms don't have a combined BOINC/VBox installer;
the user has to install VBox separately).
Solution: add links to the VirtualBox download page in appropriate places.
Also: we currently have a <need_vbox/> config option
for projects that require VBox (i.e. all their apps are VM)
Add a <recommend_vbox/> option for projects with some but not all VM apps.
Also: rename download.php to download_software.php.
There was a name conflict with a file from the boinc-site repo.
This caused problems on the BOINC web site, which uses both code bases.
Projects with a link to download.php on their home page
should change this to download_software.php.