Upstream bug is here: https://bugreports.qt-project.org/browse/QTMOBILITY-1990?focusedCommentId=170231&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel qt-mobility-1.2 was released with Qt 4.7 and was not updated for 4.8 . The git tree is fixed but an official release is unlikely to happen before Qt 5.
Created attachment 305461 [details, diff] qt-mobility-1.2.0-respect-QT_NO_XVIDEO.patch
Created attachment 305463 [details, diff] qt-mobility-1.2.0-ipc-fix-metacall-proto.patch
Created attachment 305465 [details] qt-mobility-1.2.0.ebuild
Comment on attachment 305465 [details] qt-mobility-1.2.0.ebuild --- qt-mobility-1.2.0.ebuild 2011-11-11 00:16:04.000000000 +0100 +++ - 2012-03-15 16:13:28.560121050 +0100 @@ -4,7 +4,7 @@ EAPI=4 -inherit multilib qt4-r2 +inherit eutils multilib qt4-r2 MY_P="${PN}-opensource-src-${PV}" @@ -128,6 +128,12 @@ if ! use qml; then sed -i -e '/SUBDIRS += declarative/d' plugins/plugins.pro || die fi + + if has_version ">=x11-libs/qt-core-4.8.0"; then + epatch "${FILESDIR}"/${P}-ipc-fix-metacall-proto.patch + fi + + epatch "${FILESDIR}"/${P}-respect-QT_NO_XVIDEO.patch } src_configure() { Patches shouldn't be applied conditionally...
I suppose both patches are taken from upstream git repo..?
(In reply to comment #4) > Comment on attachment 305465 [details] > qt-mobility-1.2.0.ebuild > > --- qt-mobility-1.2.0.ebuild 2011-11-11 00:16:04.000000000 +0100 > +++ - 2012-03-15 16:13:28.560121050 +0100 > @@ -4,7 +4,7 @@ > > EAPI=4 > > -inherit multilib qt4-r2 > +inherit eutils multilib qt4-r2 > > MY_P="${PN}-opensource-src-${PV}" > > @@ -128,6 +128,12 @@ > if ! use qml; then > sed -i -e '/SUBDIRS += declarative/d' plugins/plugins.pro || > die > fi > + > + if has_version ">=x11-libs/qt-core-4.8.0"; then > + epatch "${FILESDIR}"/${P}-ipc-fix-metacall-proto.patch > + fi > + > + epatch "${FILESDIR}"/${P}-respect-QT_NO_XVIDEO.patch > } > > src_configure() { > > > Patches shouldn't be applied conditionally... Then you can always apply a patch that checks QT_VERSION and use the old prototype for < 4.7.
(In reply to comment #5) > I suppose both patches are taken from upstream git repo..? Kind of. I took a look at the current git tree and extracted what was needed to make it work.
Hello, Do you intend to add the patch in the tree?
(In reply to comment #8) > Hello, > > Do you intend to add the patch in the tree? I'm working on it.
Created attachment 306913 [details, diff] qmetaobjectbuilder-qt48.patch Try this patch please. (extracted from upstream commits in the range b90cee3d3502c20f9dda5d90c305afd5a046552c^..f9c87a7e68fac52ed92c430b2360827438f2e33b)
The xvideo-related failure is caused by a bug in qt-gui, so please be sure to update to >=x11-libs/qt-gui-4.8.0-r4 before trying the above patch.
If you import a patch as big as that, I honestly think it would be better to create a snapshot ebuild that only works with 4.8 and keep the current one for 4.7.
I disagree. The patch is conceptually very simple, it does a few copies and renames, that's why it is big in size. Anyway, the xvideo fix really requires >=qt-gui-4.8.0-r4[xv], thus I'm considering doing a snapshot ebuild with raised deps anyway, for that reason.
As long as I got somethink that builds with 4.8 I'm happy. Thanks !
I think that a revision will be the best solution
what's the status on this bug? Today I tried to reemerge qt-mobility and it still fails.
(In reply to comment #16) > what's the status on this bug? > Today I tried to reemerge qt-mobility and it still fails. I'm working on it, I've found some additional issues that needs fixing.
Fixed in qt-mobility-1.2.2_p20120403. Dependencies in 1.2.0 have been restricted to 4.7.x. Thank you all!
