Summary: | gnome-extra/yelp fails to configure with net-libs/xulrunner-2.0 | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Cyprien Nicolas (fulax) <cyprien> |
Component: | [OLD] GNOME | Assignee: | Gentoo Linux Gnome Desktop Team <gnome> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | bugzilla, flameeyes, jlec, meconiotech, mozilla, nao.nakashima, nico.schloemer, peach, sergio.bevilacqua, wasundwarum, z23 |
Priority: | High | ||
Version: | unspecified | ||
Hardware: | AMD64 | ||
OS: | Linux | ||
URL: | https://bugzilla.gnome.org/show_bug.cgi?id=625910 | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: |
/var/tmp/portage/gnome-extra/yelp-2.30.1-r1/work/yelp-2.30.1/config.log
'emerge --info =gnome-extra/yelp-2.30.1-r1' Build log yelp-xulrunner-20.patch |
Description
Cyprien Nicolas (fulax)
2010-08-02 15:12:30 UTC
Created attachment 241087 [details]
/var/tmp/portage/gnome-extra/yelp-2.30.1-r1/work/yelp-2.30.1/config.log
Created attachment 241089 [details]
'emerge --info =gnome-extra/yelp-2.30.1-r1'
Created attachment 241091 [details]
Build log
yelp checks for xulrunner-1.9 not 2.0, this is still beta so I don't think anything will be done to check compatibility at this point on gnome herd side. You are free to patch and report your findings upstream though, I'm sure any patches would end up in the gnome overlay sooner or later since it would probably be integrated in 2.31 dev cycle. (In reply to comment #4) > yelp checks for xulrunner-1.9 not 2.0, this is still beta so I don't think > anything will be done to check compatibility at this point on gnome herd side. > You are free to patch and report your findings upstream though, I'm sure any > patches would end up in the gnome overlay sooner or later since it would > probably be integrated in 2.31 dev cycle. > There have been no changes just need to update configure to check for 2.0 as well. If gnome herd does not want to fix not gonna happen on our end. Gnome should make a proper check and not depend on a specific version being avliable. I guess upstream didn't want to risk guessing API compatibility upfront given xulrunner history :) This is nothing more then an ebuild failure. --with-gecko=libxul-embedding ... this should be l --with-gecko=libxul and compile would be fine. meh ? that's weird, I remember we had to change this months ago for the same reason (1.8 to 1.9 or 1.9 to 1.9.1, can't remember). For now leave it closed failure will result either way. As we approach rc's we will reopen and worry about it then. (In reply to comment #4) > yelp checks for xulrunner-1.9 not 2.0, this is still beta so I don't think > anything will be done to check compatibility at this point on gnome herd side. > You are free to patch and report your findings upstream though, I'm sure any > patches would end up in the gnome overlay sooner or later since it would > probably be integrated in 2.31 dev cycle. > Hi Gilles, I think you are right about xulrunner, but not about yelp. The Gnome devs drop support for gecko, in favor of webkit-gtk-1.3, so I'm afraid nothing will be done in the 2.31 for xulrunner. I took a deep look at the gecko.m4 made by the Gnome devs for xulrunner detection, and it has been designed for xulrunner-1.[7-9]. So this is more an UPSTREAM Resolution: Gnome Bugzilla reference: https://bugzilla.gnome.org/show_bug.cgi?id=625910 A not really effective solution is to wait for xulrunner being slotted in 2.0, but I don't really want to have two xulrunners installed just for a feature I never use. *** Bug 331205 has been marked as a duplicate of this bug. *** (In reply to comment #10) > (In reply to comment #4) > > yelp checks for xulrunner-1.9 not 2.0, this is still beta so I don't think > > anything will be done to check compatibility at this point on gnome herd side. > > You are free to patch and report your findings upstream though, I'm sure any > > patches would end up in the gnome overlay sooner or later since it would > > probably be integrated in 2.31 dev cycle. > > > > Hi Gilles, I think you are right about xulrunner, but not about yelp. > > The Gnome devs drop support for gecko, in favor of webkit-gtk-1.3, so I'm > afraid nothing will be done in the 2.31 for xulrunner. > > I took a deep look at the gecko.m4 made by the Gnome devs for xulrunner > detection, and it has been designed for xulrunner-1.[7-9]. > So this is more an UPSTREAM Resolution: Gnome Bugzilla reference: > https://bugzilla.gnome.org/show_bug.cgi?id=625910 > > A not really effective solution is to wait for xulrunner being slotted in 2.0, > but I don't really want to have two xulrunners installed just for a feature I > never use. > 2.0 will not create a new slot. There would be no point in it. Seems nobody bothered to re-open in rc-phase, now we have ff 4 in tree and it still fails. See with upstream (in URL) bug was marked Won't Fix - too old Created attachment 266925 [details, diff]
yelp-xulrunner-20.patch
OpenSUSE patch porting yelp to xul-2.0, please try it
Is xulrunner-2 really still slot 1.9 ? It seems to be. $ grep SLOT /usr/portage/net-libs/xulrunner/*.ebuild /var/lib/layman/*/net-libs/xulrunner/*.ebuild /usr/portage/net-libs/xulrunner/xulrunner-1.9.2.12.ebuild:SLOT="1.9" /usr/portage/net-libs/xulrunner/xulrunner-1.9.2.13.ebuild:SLOT="1.9" /usr/portage/net-libs/xulrunner/xulrunner-1.9.2.15.ebuild:SLOT="1.9" /usr/portage/net-libs/xulrunner/xulrunner-2.0.ebuild:SLOT="1.9" /var/lib/layman/mozilla/net-libs/xulrunner/xulrunner-2.0.ebuild:SLOT="1.9" (In reply to comment #16) > Is xulrunner-2 really still slot 1.9 ? Actually xulrunner should never had been slotted. This was done before I returned to gentoo. The reason we left it in slot-1.9 is due to the fact it will take more work then it is worth to patch it to suppport multiple instances being installed. Eventually I will get around to drop the slot all together. (In reply to comment #15) > Created attachment 266925 [details, diff] > yelp-xulrunner-20.patch > > OpenSUSE patch porting yelp to xul-2.0, please try it Tried it. Seems to work. However: >>> Installing (1 of 2) gnome-extra/yelp-2.30.2-r1 * Installing GNOME 2 GConf schemas WARNING: node <gettext_domain> not understood below <schema> WARNING: node <gettext_domain> not understood below <schema> WARNING: node <gettext_domain> not understood below <schema> WARNING: node <gettext_domain> not understood below <schema> WARNING: node <gettext_domain> not understood below <schema> WARNING: node <gettext_domain> not understood below <schema> WARNING: node <gettext_domain> not understood below <schema> WARNING: node <gettext_domain> not understood below <schema> * Reloading GConf schemas ... [ ok ] * Updating desktop mime database ... * Updating shared mime info database ... * Updating icons cache ... [ ok ] (In reply to comment #15) > Created attachment 266925 [details, diff] > yelp-xulrunner-20.patch > > OpenSUSE patch porting yelp to xul-2.0, please try it With the patch, yelp compiles and runs. Browsing help also seems to work. I'm going to commit it. This is now fixed in the tree, with 2.30.2-r2, thanks everyone! (In reply to comment #21) > This is now fixed in the tree, with 2.30.2-r2, thanks everyone! I still hit it: checking which gecko to use... libxul-embedding checking manual gecko home set... Package libxul-embedding-unstable was not found in the pkg-config search path. Perhaps you should add the directory containing `libxul-embedding-unstable.pc' to the PKG_CONFIG_PATH environment variable No package 'libxul-embedding-unstable' found checking for compiler -fshort-wchar option... yes checking whether to enable C++ RTTI... no checking whether we have a gtk 2 gecko build... configure: error: This program needs a gtk 2 gecko build * gnome-extra/yelp: 2.30.2-r5 * net-libs/xulrunner: Installed versions: 2.0.1-r1(1.9){tbz2}(23:38:26 06/19/11)(alsa crashreporter dbus ipc libnotify startup-notification webm -custom-optimization -debug -gconf -system-sqlite -wifi) for me too, emerge yelp-2.30.2-r5 fails with xulrunner-20, this patch provided in this bug fixes the build, but yelp starts with error "Could not initialize gecko!" People running xul-2 need -r6 (these problems tends to occur when mixing stable and testing, when this occurs again, please try moving to testing the affected package) *** Bug 373911 has been marked as a duplicate of this bug. *** I want to confirm this bug with yelp-2.30.2-r5 and xulrunner-2.0.1-r1 and that using yelp-2.30.2-r6 fixes the configure failure. Should be moved to stable quickly. *** Bug 378859 has been marked as a duplicate of this bug. *** (In reply to comment #26) > I want to confirm this bug with yelp-2.30.2-r5 and xulrunner-2.0.1-r1 and that > using yelp-2.30.2-r6 fixes the configure failure. > Should be moved to stable quickly. xulrunner-2 won't go to stable, so there's no point stabilizing it. The next stable firefox will be either 5.0 or 6.0, and both don't have any corresponding xulrunner versions. Just ran in to this bug and I see a huge point in stabilizing yelp-2.30.2-r6 as currently the ebuild is broken for many users. If there is a reason to not stabilize it then the ebuild needs to be updated to require a 1.9 version of xulrunner. *** Bug 389879 has been marked as a duplicate of this bug. *** |