Description
Maxim Kammerer
2012-07-04 23:29:06 UTC
Created attachment 317184 [details]
app-emulation/virtualbox-guest-additions-4.1.12 build log
Created attachment 317186 [details]
x11-drivers/xf86-video-virtualbox-4.1.12 build log
Found a fix (for both packages): add "--with-linux=${KV_OUT_DIR}" to src_configure(). For virtualbox-guest-additions, there is still something problematic in the build log: * Preparing vboxguest module grep: /usr/src/linux/include/linux/version.h: No such file or directory but both packages produce exactly the same binaries (executables and modules) with regular and split kernel tree setup for me. Created attachment 317206 [details, diff]
fixes virtualbox-guest-additions-4.1.12.ebuild
Created attachment 317208 [details, diff]
fixes xf86-video-virtualbox-4.1.12.ebuild
Created attachment 319172 [details, diff]
fixes virtualbox-guest-additions-4.1.18.ebuild for kernel 3.4.2
Config.kmk has wrong path to generated includes.
Created attachment 319174 [details, diff]
fixes xf86-video-virtualbox-4.1.18.ebuild for kernel 3.4.2
Config.kmk has wrong path to generated includes.
Created attachment 320744 [details, diff]
fixes xf86-video-virtualbox-4.1.18.ebuild for kernel 3.4.2
Updated wrt. tabs in ebuild.
Hi, any response? I don't see any reason not to use --with-linux=${KV_OUT_DIR}, at least -- it's a configure switch provided by upstream for this exact purpose, after all. Created attachment 322499 [details]
build.log for xf86-video-virtualbox-4.1.20
+1 for this bug.
I confirmed the issue is reproduceable when emerging virtualbox-guest-additions-4.1.20 and xf86-video-virtualbox-4.1.20.
build.log for xf86-video-virtualbox-4.1.20 is attached.
My settings are:
1) Linux kernel source is on /usr/src/linux-3.5.0-gentoo
2) /usr/src/linux symlink points to /usr/src/linux-3.5.0-gentoo
3) KBUILD_OUTPUT=/home/tomo/work/kernel-build/current is set as a system-wide environment variable.
Maxim's fix works fine for {xf86-video-virtualbox, virtualbox-guest-additions}-4.1.20. Thanks.
(In reply to comment #3) > For virtualbox-guest-additions, there is still something problematic in the > build log: > > * Preparing vboxguest module > grep: /usr/src/linux/include/linux/version.h: No such file or directory > You can also see the same error when emerging xf86-video-virtualbox[+dri]. This error can be avoided by this patch (same fix is valid for virtualbox-guest-additions). --- xf86-video-virtualbox-4.1.20.ebuild.orig +++ xf86-video-virtualbox-4.1.20.ebuild @@ -48,7 +48,7 @@ QA_TEXTRELS_x86="usr/lib/VBoxOGL.so" pkg_setup() { linux-mod_pkg_setup - BUILD_PARAMS="KERN_DIR=${KV_DIR} KERNOUT=${KV_OUT_DIR}" + BUILD_PARAMS="KERN_DIR=${KV_OUT_DIR} KERNOUT=${KV_OUT_DIR}" python_set_active_version 2 python_pkg_setup > but both packages produce exactly the same binaries (executables and > modules) with regular and split kernel tree setup for me. Indeed. This check does nothing when building with >=linux-2.6.0. But IMO, KERN_DIR=${KV_OUT_DIR} is still reasonable, because VBox's Makefile assumes KERN_DIR as follows. KERN_DIR := /lib/modules/$(shell uname -r)/build (src/VBox/Installer/linux/Makefile.include.header, line 93) And I couldn't figure it out why there is "KERNOUT=${KV_OUT_DIR}"... Created attachment 330168 [details, diff]
patch for xf86-video-virtualbox-4.2.4.ebuild
Created attachment 330172 [details, diff]
patch for virtualbox-guest-additions-4.2.4.ebuild
Patches for 4.2.4 are attached above. Is it possible to reassign this bug to an active developer? No Gentoo developer commented in 4.5 months since this bug was opened. The fixes are quite simple, and there is clearly interest in being able to use VirtualBox packages with split kernels. (In reply to comment #11) > This error can be avoided by this patch (same fix is valid for > virtualbox-guest-additions). Thanks, I included your fix in patches for 4.2.4 above. I verified that version.h warning does not occur anymore. Created attachment 356544 [details, diff]
xf86-video-virtualbox-4.2.14.patch
Created attachment 356546 [details, diff]
virtualbox-guest-additions-4.2.14.patch
Is it possible to at least apply KERN_DIR=${KV_OUT_DIR} and --with-linux=${KV_OUT_DIR} to ebuilds in the tree? *** Bug 426514 has been marked as a duplicate of this bug. *** This bug has been fixed in VirtualBox 4.3. app-emulation/virtualbox-guest-additions-4.3.10 and x11-drivers/xf86-video-virtualbox-4.3.10 build fine after adding "--with-linux=${KV_OUT_DIR} \" to both ebuilds: --disable-alsa \ + --with-linux=${KV_OUT_DIR} \ --build-headless || die "configure failed" So if anyone still cares about VirtualBox (this bug has been open for two years with zero feedback from maintainers), please add the --with-linux switch to the 4.3 ebuilds, thanks! Time to close this (RESOLVED WON'T FIX). The oldest version in the tree is 4.2.24. should be all set now in the tree; thanks for the report! Commit message: Pass in the right kernel build dir that the user set up http://sources.gentoo.org/x11-drivers/xf86-video-virtualbox/xf86-video-virtualbox-4.3.26.ebuild?r1=1.2&r2=1.3 should be all set now in the tree; thanks for the report! Commit message: Pass in the right kernel build dir that the user set up http://sources.gentoo.org/app-emulation/virtualbox-guest-additions/virtualbox-guest-additions-4.3.26.ebuild?r1=1.2&r2=1.3 |