Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 561758 - app-emulation/virtualbox-5.0.4 wouldn't build on 32-bit machine (patches needed)
Summary: app-emulation/virtualbox-5.0.4 wouldn't build on 32-bit machine (patches needed)
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal major
Assignee: Lars Wendler (Polynomial-C) (RETIRED)
URL: https://www.virtualbox.org/ticket/14564
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-09-28 16:08 UTC by ArticIce(Juice)
Modified: 2015-09-29 13:33 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---

Attachments
patches from VirtualBox SVN (r57658.patch,1.03 KB, patch)
2015-09-28 16:10 UTC, ArticIce(Juice) 		Details | Diff
patches from VirtualBox SVN (r57690.patch,967 bytes, patch)
2015-09-28 16:10 UTC, ArticIce(Juice) 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description ArticIce(Juice) 2015-09-28 16:08:55 UTC 
VirtualBox 5.0.4 wouldn't build on a 32-bit machine because of two coding errors described here https://www.virtualbox.org/ticket/14564

diffs are here:
https://www.virtualbox.org/changeset/57658/vbox
https://www.virtualbox.org/changeset/57690/vbox


Reproducible: Always

Steps to Reproduce:
1. emerge virtualbox-5.0.4 on a 32-bit
Actual Results:  
VirtualBox-5.0.4/src/VBox/ExtPacks/VNC/VBoxVNC.cpp:339:30: error: invalid conversion from 'rfbNewClientAction (__attribute__((regparm(0), cdecl)) *)(rfbClientPtr) {aka rfbNewClientAction (__attribute__((regparm(0), cdecl)) *)(_rfbClientRec*)}' to 'rfbNewClientHookPtr {aka rfbNewClientAction (*)(_rfbClientRec*)}' [-fpermissive]

VirtualBox-5.0.4/src/VBox/Additions/linux/lightdm-greeter/vbox-greeter.cpp:929:64: error: invalid conversion from 'void (*)(PRTLOGGER, RTLOGPHASE, PFNRTLOGPHASEMSG) {aka void (*)(RTLOGGER*, RTLOGPHASE, void (__attribute__((regparm(0), cdecl)) *)(RTLOGGER*, const char*, ...))}' to 'PFNRTLOGPHASE {aka void (__attribute__((regparm(0), cdecl)) *)(RTLOGGER*, RTLOGPHASE, void (__attribute__((regparm(0), cdecl)) *)(RTLOGGER*, const char*, ...))}' [-fpermissive]


Gets quick-fixed by adding two patches into /usr/portage/app-emulation/virtualbox/files/ and a bit of code into ebuild's src_prepare()

EPATCH_SOURCE="${FILESDIR}/" EPATCH_SUFFIX="patch" \
        EPATCH_FORCE="yes" epatch
Comment 1 ArticIce(Juice) 2015-09-28 16:10:20 UTC 
Created attachment 413214 [details, diff]
patches from VirtualBox SVN

https://www.virtualbox.org/changeset/57658/vbox
Comment 2 ArticIce(Juice) 2015-09-28 16:10:55 UTC 
Created attachment 413216 [details, diff]
patches from VirtualBox SVN

https://www.virtualbox.org/changeset/57690/vbox
Comment 3 ArticIce(Juice) 2015-09-28 17:27:58 UTC 
Please add these two patches into a separate virtualbox-5.0.4-patches-01.tar.xz file and add it to distfiles.

I confirmed the build working with them.

I don't know how to do it (don't have maintainer rights).
Comment 4 Lars Wendler (Polynomial-C) (RETIRED) gentoo-dev 2015-09-29 13:30:34 UTC 
commit 9f7b9fb31d387ec7783abba052b8e2d35fe75dad
Author: Lars Wendler <polynomial-c@gentoo.org>
Date:   Tue Sep 29 15:28:29 2015

    app-emulation/virtualbox: Added x86 buildfix patches (bug #561758).

    Package-Manager: portage-2.2.22
    Signed-off-by: Lars Wendler <polynomial-c@gentoo.org>