Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 436246

Summary: >=app-emulation/virtualbox[vboxwebsrv] with >=dev-libs/libxslt-1.1.27 :compilation error:file .../work/VirtualBox-4.2.0/src/VBox/Main/webservice/websrv-wsdl.xsl line 962 element attribute xsl:attribute: The attribute name 'xmlns' is not allowed.
Product: Gentoo Linux Reporter: Kobboi <gentoo>
Component: [OLD] UnspecifiedAssignee: Lars Wendler (Polynomial-C) (RETIRED) <polynomial-c>
Status: RESOLVED FIXED    
Severity: normal CC: bagrx, balint, bmadams, chris, edv, god, hwoarang, jouni.kosonen, kripton, madjic, mbucas, mieszkoslusarczyk, nikita.kipriyanov, patrick, rob, sniper, swapon, torsten
Priority: Normal Keywords: NeedPatch, UPSTREAM
Version: unspecified   
Hardware: All   
OS: Linux   
URL: https://www.virtualbox.org/ticket/10940
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 436716    
Attachments: build.log
virtualbox-4.2.0-libxslt-xmlns.patch
virtualbox-4.2.4-xmlns.patch

Description Kobboi 2012-09-25 21:20:17 UTC
Note that downgrading to libxslt-1.1.26-r4 is a workaround to this issue.

(...)
  xpidl   XPCOM - {C}/src/libs/xpcom18a4/ipc/ipcd/client/public/ipcIClientObserver.idl
  xpidl   XPCOM - {C}/src/libs/xpcom18a4/ipc/ipcd/extensions/lock/public/ipcILockService.idl
  xpidl   XPCOM - {C}/src/libs/xpcom18a4/ipc/ipcd/extensions/transmngr/public/ipcITransactionService.idl
  xpidl   XPCOM - {C}/src/libs/xpcom18a4/ipc/ipcd/extensions/dconnect/public/ipcIDConnectService.idl
  xpidl   XPCOM - {C}/src/libs/xpcom18a4/ipc/ipcd/extensions/transmngr/public/ipcITransactionObserver.idl
  GEN     {C}/out/linux.amd64/release/bin/sdk/bindings/webservice/vboxweb.wsdl
  GEN     {T}/webservice/typemap.dat
compilation error: file /var/tmp/portage/app-emulation/virtualbox-4.2.0-r1/work/VirtualBox-4.2.0/src/VBox/Main/webservice/websrv-wsdl.xsl line 962 element attribute
xsl:attribute: The attribute name 'xmlns' is not allowed.



Reproducible: Always
Comment 1 Kobboi 2012-09-25 21:20:50 UTC
Created attachment 324992 [details]
build.log
Comment 2 Torsten Kurbad 2012-09-26 05:37:30 UTC
Got that one, too.

Quick workaround: USE="-vboxwebsrv"
Comment 3 Balint SZENTE 2012-09-28 16:24:17 UTC
The solution is to mask >=dev-libs/libxslt-1.1.27

Upstream bug report:
<https://www.virtualbox.org/ticket/10940>

This is the same problem as bug 435994.
Comment 4 Jouni Kosonen 2012-09-30 07:15:05 UTC
Created attachment 325356 [details, diff]
virtualbox-4.2.0-libxslt-xmlns.patch

Then again, 'xmlns' really is not allowed as an attribute name, this is just the first version of libxslt that flags the error. 

See https://mail.gnome.org/archives/xslt/2005-September/msg00016.html

Maybe something like the attached patch would work instead?
Comment 5 Lars Wendler (Polynomial-C) (RETIRED) gentoo-dev 2012-10-01 09:23:56 UTC
+  01 Oct 2012; Lars Wendler <polynomial-c@gentoo.org>
+  -files/virtualbox-ose-3.2.8-mesa-check.patch,
+  -files/virtualbox-4.0.2-restore_old_machines_dir.patch,
+  -files/virtualbox-4.1.2-vboxpci-build.patch,
+  -files/virtualbox-4.1.4-asneeded.patch,
+  -files/virtualbox-4.1.6-yasm120-fix.patch,
+  -files/virtualbox-4.1.8-nopie.patch,
+  -files/virtualbox-4.1.10-libgsoapssl.patch,
+  -files/virtualbox-4.1.20-nopie.patch, virtualbox-4.1.22.ebuild,
+  virtualbox-4.2.0-r1.ebuild, -files/virtualbox-4.2.0-nopie.patch,
+  -files/virtualbox-4-asneeded.patch, -files/virtualbox-4-makeself-check.patch,
+  -files/virtualbox-4-mkisofs-check.patch:
+  Moved all patches into separate tar archives. Added a patch to fix
+  compilation with =dev-libs/libxslt-1.1.27 (bug #435994 and bug #436246).
+  Thanks to Jouni Kosonen for providing a patch.
+
Can you please sync and try again?
Comment 6 Jouni Kosonen 2012-10-04 16:43:12 UTC
It seems that the patch is somewhat incomplete, since there are now bugs like 437192 where the namespace is apparently not getting set correctly in the child elements.

That would mean that the description -element needs to be created using <xsl:element name="description" namespace="http://schemas.xmlsoap.org/wsdl/" ... 
instead of writing it out as it is now.
Comment 7 Gerald 2012-10-10 22:07:53 UTC
The Oct 01 change also broke reemerging virtualbox for me on all systems due to bug 437192. I was previously (Sep 18) able to emerge those just fine.
Comment 8 Lars Wendler (Polynomial-C) (RETIRED) gentoo-dev 2012-10-20 09:29:39 UTC
Okay, this bug still isn't fixed properly. As soon as I have my dev machine back online again I will upload a new patchset without the broken patch. 
Unfortunately this won't fix compiling virtualbox with =dev-libs/libxslt-1.1.27 so we have to wait for upstream to fix this...
Comment 9 Gerald 2012-10-28 16:48:57 UTC
Still fails to compile even with fridays new Virtualbox 4.2.4 release (I've renamed the ebuild to test it) - definitely not "resolved upstream".
Comment 10 Markos Chandras (RETIRED) gentoo-dev 2012-11-04 18:38:34 UTC
*** Bug 435994 has been marked as a duplicate of this bug. ***
Comment 11 Markos Chandras (RETIRED) gentoo-dev 2012-11-04 18:39:18 UTC
*** Bug 437192 has been marked as a duplicate of this bug. ***
Comment 12 Markos Chandras (RETIRED) gentoo-dev 2012-11-04 18:39:43 UTC
Reopening to avoid duplicates in the future. Please close this once this is really fixed
Comment 13 Jeroen Roovers (RETIRED) gentoo-dev 2012-11-08 15:47:45 UTC
*** Bug 442338 has been marked as a duplicate of this bug. ***
Comment 14 siebz0r 2012-11-14 18:13:00 UTC
Might it be an idea to add "<dev-libs/libxslt-1.1.27" to the RDEPEND of the virtualbox ebuild until this is fixed upstream?
Comment 15 Johannes 2012-11-14 20:50:17 UTC
in Bug #435994  Balint SZENTE found the solution:
with dev-libs/libxslt-1.1.26 

AND without 006_virtualbox-4.2.0-libxslt-xmlns.patch 

i can compile virtualbox ( headless ), too
Comment 16 László Szalma 2012-11-29 14:41:28 UTC
I suggest applying #14 in the tree, as it's better solution for the users than finding this bug, and hack it manually.
Comment 17 jannis 2012-12-01 15:32:37 UTC
(In reply to comment #14)
> Might it be an idea to add "<dev-libs/libxslt-1.1.27" to the RDEPEND of the
> virtualbox ebuild until this is fixed upstream?

Won't help since these (old) versions have been removed from the tree....
Comment 18 jannis 2012-12-01 15:54:13 UTC
(In reply to comment #17)
> (In reply to comment #14)
> > Might it be an idea to add "<dev-libs/libxslt-1.1.27" to the RDEPEND of the
> > virtualbox ebuild until this is fixed upstream?
> 
> Won't help since these (old) versions have been removed from the tree....

And since it doesn't work with dev-libs/libxslt-1.1.28 (latest unstable in tree), I don't have any chance to compile virtualbox successfully :(
Comment 19 Markos Chandras (RETIRED) gentoo-dev 2012-12-01 20:42:43 UTC
(In reply to comment #14)
> Might it be an idea to add "<dev-libs/libxslt-1.1.27" to the RDEPEND of the
> virtualbox ebuild until this is fixed upstream?

no because downgrades of the same keywording level are not acceptable. The bug really needs to be fixed upstream
Comment 20 Lars Wendler (Polynomial-C) (RETIRED) gentoo-dev 2012-12-13 08:57:24 UTC
*** Bug 447070 has been marked as a duplicate of this bug. ***
Comment 21 Joe Breuer 2012-12-18 16:15:27 UTC
Created attachment 332660 [details, diff]
virtualbox-4.2.4-xmlns.patch

I've attached a patch removing the use of the default namespace from the affected XSLTs, replacing it with an explicit wsdl namespace reference where appropriate.

Here, this allows virtualbox-2.4.2-r1 to build and run correctly, including the web service (by phpvirtualbox-4.2.4).
Comment 22 Lars Wendler (Polynomial-C) (RETIRED) gentoo-dev 2012-12-18 22:40:54 UTC
+  18 Dec 2012; Lars Wendler <polynomial-c@gentoo.org>
+  virtualbox-4.2.4-r1.ebuild:
+  Fixed compilation with vboxwebsrv USE flag being enabled. Thanks to Joachim
+  Breuer for providing a working patch in bug #436246.
+

Joachim can you please attach your patch in the upstream bug report?
Comment 23 Chris Mansfield 2012-12-18 22:55:35 UTC
(In reply to comment #21)
> Created attachment 332660 [details, diff] [details, diff]
> virtualbox-4.2.4-xmlns.patch
> 
> I've attached a patch removing the use of the default namespace from the
> affected XSLTs, replacing it with an explicit wsdl namespace reference where
> appropriate.
> 
> Here, this allows virtualbox-2.4.2-r1 to build and run correctly, including
> the web service (by phpvirtualbox-4.2.4).

I applied your patch and was also able to successfully build & install virtualbox-2.4.2-r1 with phpvirtualbox-4.2.4. Unfortunately I am having an issue getting it to run though, but I'm sure I probably just did something wrong. Anyhow at least the build seems to be fixed. I'll keep trying to figure out my other problem & post in the forum if I can't figure it out.