Created attachment 183659 [details] disable annoying version alert message phpvirtualbox have no updated version, current version reports that it old and need to update on every page reload. Patch disable annoying message.
Upstream is almost dead for this project but there are some contributors that have already added VirtualBox 5.1 support submitted as github pull request besides some other fixes. https://github.com/phpvirtualbox/phpvirtualbox/pulls I have not decided yet how to proceed with the phpvirtualbox port for FreeBSD since no better fork exists yet and other Linux distros seems to cherry pick from the pull requests currently. Maybe that is the best way short term and we should also do it the same way until either upstream sorts it out and does a release or some fork gains enough traction. In either case I think just patching out the warning is not the right thing to do for the port.
Created attachment 184313 [details] with pull reqs latest commit +: https://github.com/phpvirtualbox/phpvirtualbox/pull/22 Patch to support Virtualbox 5.1 #22 https://github.com/phpvirtualbox/phpvirtualbox/pull/24 Fix argument count mismatches revealed by PHP 7.1 #24 SF FR:#24 Allow usage of SparkGateway or Guacamole instead of Java vi… https://github.com/phpvirtualbox/phpvirtualbox/commit/2eefdbdfdb8c209799e26ab30f59321fda6d333c https://github.com/phpvirtualbox/phpvirtualbox/pull/55 Make decryption of images/VMs work by not providing a password ID https://github.com/phpvirtualbox/phpvirtualbox/pull/57 Tried to make clearOnSuspend work. #57 Bugs: wrong mem free/usage on VirtualBox host.
update to support 5.1
ping
The patch seems good to me except the SparkGateway stuff. I looked around a bit and the only notes and discussions that I could find about it is some random guy which attached it to the sourceforge ticket. Then someone picked that up and committed it to a testing branch on github. That really does not give me the feeling that it's ready yet. Could you please exclude that sparkgateway stuff from your patch? The other stuff seems very good to me.
Created attachment 184759 [details] without sparkweb
Created attachment 184760 [details] without sparkweb2
A commit references this bug: Author: decke Date: Thu Jul 27 15:21:32 UTC 2017 New revision: 446746 URL: https://svnweb.freebsd.org/changeset/ports/446746 Log: - Update to an unofficial 5.1-1 which supports VirtualBox 5.1 Sadly upstream is very silent and has not released a new version which supports VirtualBox 5.1, supports PHP 7.1 or fixes bugs. In the meantime the community has created patches which are collected in the github projects bugtracker and pull requests. This unofficial version cherry picks the useful stuff of it. Highlights of unofficial phpvirtualbox 5.1-1: * based on git commit 65ebced (latest commit from May 19, 2016) * Patch to support Virtualbox 5.1 https://github.com/phpvirtualbox/phpvirtualbox/pull/22 * Fix argument count mismatches revealed by PHP 7.1 https://github.com/phpvirtualbox/phpvirtualbox/pull/24 * Make decryption of images/VMs work by not providing a password ID https://github.com/phpvirtualbox/phpvirtualbox/pull/55 * Tried to make clearOnSuspend work https://github.com/phpvirtualbox/phpvirtualbox/pull/57 PR: ports/220171 Submitted By: rozhuk.im With hat: vbox@ (kind of) Changes: head/www/phpvirtualbox/Makefile head/www/phpvirtualbox/distinfo head/www/phpvirtualbox/files/ head/www/phpvirtualbox/files/patch-endpoints_lib_config.php head/www/phpvirtualbox/files/patch-endpoints_lib_vboxServiceWrappers.php head/www/phpvirtualbox/files/patch-endpoints_lib_vboxconnector.php head/www/phpvirtualbox/files/patch-endpoints_lib_vboxweb-5.1.wsdl head/www/phpvirtualbox/files/patch-endpoints_lib_vboxwebService-5.1.wsdl head/www/phpvirtualbox/files/patch-panes_mediumEncryptionPasswords.html head/www/phpvirtualbox/pkg-plist
I have done some small changes and committed it. Thanks a lot for the hard part of the work! my changes: * shortened the git revision * moved USES above USE_PHP (to pet portlint) * moved USE_GITHUB down (to pet portlint) * fixed the patchfile names (vboxweb-5.1.wsdl vs vboxwebService-5.1.wsdl) * regenerated the patches using "make makepatch"