Opened 13 years ago

Closed 13 years ago

#32764 closed defect (wontfix)

virtualbox version modification throwing off downstream software

Reported by: bm55b@… Owned by: royliu@…
Priority: Normal Milestone:
Component: ports Version: 2.0.3
Keywords: Cc:
Port: virtualbox

Description (last modified by ryandesign (Ryan Carsten Schmidt))

$ VBoxManage --version
4.1.8_MacPortsr75467

I think this causes vagrant to fail

https://github.com/mitchellh/vagrant/issues/619

Why 4.1.8_MacPortsr75467? Why not 4.1.8_r75467 or 4.1.8_r75467_MacPorts which would be more compatible with existing string parsing?

Change History (7)

comment:1 Changed 13 years ago by bm55b@…

This is the version that comes out of the DMG installed VirtualBox for reference

$ VBoxManage --version
4.1.8r75467

So I'd amend my bug report and say... Why not 4.1.8r75467 or 4.1.8r75467_MacPorts which would be more compatible with existing string parsing?

comment:2 Changed 13 years ago by bm55b@…

Having format issues.. Copy/paste loses carriage returns!

Macports...

$ VBoxManage --version
4.1.8_MacPortsr75467

virtualbox.org

$ VBoxManage --version
4.1.8r75467

suggest

$ VBoxManage --version
4.1.8r75467_MacPorts

comment:3 Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)

Description: modified (diff)
Port: virtualbox added

comment:4 Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)

Yes please remember to use WikiFormatting.

comment:5 Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)

All we are doing is changing VBOX_BUILD_PUBLISHER from "_OSE" to "_MacPorts". The VirtualBox developers request that packagers like MacPorts do this. I am currently unable to find the URL where they say this, but I have seen it before. Perhaps vagrant needs to update its code to accommodate this recommended practice.

comment:6 Changed 13 years ago by royliu@…

Owner: changed from macports-tickets@… to royliu@…

Yes, we are complying to VirtualBox developers' wishes by appending "_MacPorts" to the version, but I think that bm55b is right by pointing out that 4.1.8r75467_MacPorts is more appropriate. Once I patch VirtualBox appropriately, I'll also generate a pull request for Vagrant on GitHub (I live in both worlds).

comment:7 Changed 13 years ago by royliu@…

Resolution: wontfix
Status: newclosed

On second thought, I don't think a fix is feasible. Performing a grep over VBOX_VERSION_STRING (which is 4.1.8_MacPorts in our case) yields:

...
src/VBox/Additions/solaris/DRM/vboxvideo_drm.c:    DEVICE_DESC_DRV " " VBOX_VERSION_STRING "r" VBOXSOLQUOTE(VBOX_SVN_REV),
src/VBox/Additions/solaris/Installer/vboxguest.pkginfo:VERSION="@VBOX_VERSION_STRING@,REV=r@VBOX_SVN_REV@.@VBOX_VERSION_REVSTAMP@"
src/VBox/Additions/solaris/Installer/vboxguest.pkginfo:SUNW_PRODVERS="@VBOX_VERSION_STRING@"
src/VBox/Additions/solaris/Makefile.kmk:        -e "s/@VBOX_VERSION_STRING@/$(VBOX_VERSION_STRING)/g" \
src/VBox/Additions/solaris/SharedFolders/vboxfs_vfs.c:  DEVICE_DESC " " VBOX_VERSION_STRING "r" VBOXSOLQUOTE(VBOX_SVN_REV),
src/VBox/Additions/solaris/Virtio/Makefile.kmk:virtionet_DEFS          = VBOX_WITH_HGCM VBOX_SVN_REV=$(VBOX_SVN_REV) VBOX_VERSION_STRING=\"$(VBOX_VERSION_STRING)\"
src/VBox/Additions/solaris/Virtio/VirtioNet-solaris.c:    DEVICE_DESC_DRV " " VBOX_VERSION_STRING "r"  RT_XSTR(VBOX_SVN_REV),
...

This tells me that the VirtualBox developers actually intend for the SVN revision to immediately come after VBOX_VERSION_STRING. Consequently, I'm going to close the issue and try to solve this issue in Vagrant itself.

Note: See TracTickets for help on using tickets.