sys-auth/polkit-0.109 fails to build with dev-lang/spidermonkey-1.8.7-r3 on hardened-amd64 because of undefined reference to JSVAL_VOID/JSVAL_NULL Reproducible: Always
Created attachment 333084 [details] 1356126031-install-sys-auth_polkit-0.109:0::gentoo.out
Created attachment 333086 [details] output of paludis info polkit
Dynamically load libmozjs185.so and cope with it not being available http://cgit.freedesktop.org/polkit/commit/?id=34cb6353b9affd6c04ea480df5fc39d0ca72319d So upstream is the reason for the breakage.
(In reply to comment #3) > So upstream is the reason for the breakage. So, what is next step to fix this issue? Should someone of us create issue on freedesktop's tracker (if it is exist), or should someone add spidermonkey:0/mozjs185 in pokit's deps? Or, may be, patch the sources to dlopen() 1.8.7? ;)
1. inherit flag-o-matic toolchain-funcs 2. Add these 2 lines to the start of src_configure() function: has_version '>=dev-lang/spidermonkey-1.8.7' && \ append-ldflags "$($(tc-getPKG_CONFIG) --libs mozjs187)" #448228 Now it will compile, believe it or not, but it's wrongly linking against -lmozjs187 since it's supposed to be dlopen()'d now FTR: -Wl,--as-needed has nothing to do with this, it seems So I believe the problem lies somewhere in js/jsapi.h of spidermonkey, maybe with 'extern' of JSVAL_VOID, JSVAL_NULL, and more if something else comes up after sorting those out
Is it possible to port USE=jit to 0/mozjs185 in the meanwhile? It is probably the reason that users on hardened prefer the masked 0/mozjs187.
Also fails to build on unhardened 32 bit x86, polkit-0.110, spidermonkey-1.8.7.
Can you finally do something? Never had a package not building for several versions and for such a long time, frustrating to see that nothing is done... ^^ And if you post "solutions" please post them correctly, copy past of those 2 lines won't work (I knwo what's wrong but please try yourself).
(In reply to comment #8) > Can you finally do something? Never had a package not building for several > versions and for such a long time, frustrating to see that nothing is > done... ^^ > > And if you post "solutions" please post them correctly, copy past of those 2 > lines won't work (I knwo what's wrong but please try yourself). Got a patch? This is not high priority as spidermonkey 1.8.7 is still in package.mask
(In reply to comment #8) > And if you post "solutions" please post them correctly, copy past of those 2 > lines won't work (I knwo what's wrong but please try yourself). And the workaround mentioned earlier in this bug is no longer valid. It used to work, but not for polkit-0.110 anymore. While waiting for new one...
(In reply to comment #9) > This is not high priority as spidermonkey 1.8.7 is still in > package.mask It is high priority on hardened, hence my suggestion to backport USE=jit in comment #6.
Created attachment 342280 [details, diff] let polkit-0.110 compile with spidermonkey-1.8.7
I'm not sure my patch is right way to go - because it will force polkitd to depend on libmozjs while I got impression polkit wanted to load libmozjs dynamically only if it needed - but it fix compile error on hardeded system with 1.8.7. Of course, you'll also include this patch in ebuild: --- /usr/portage/sys-auth/polkit/polkit-0.110.ebuild 2013-03-12 13:31:30.000000000 +0200 +++ /usr/local/portage/sys-auth/polkit/polkit-0.110.ebuild 2013-03-16 18:05:47.020410365 +0200 @@ -58,6 +58,7 @@ src_prepare() { epatch "${FILESDIR}"/${P}-W_define.patch + epatch "${FILESDIR}"/${P}-libjs.patch sed -i -e 's|unix-group:wheel|unix-user:0|' src/polkitbackend/*-default.rules || die #401513
(In reply to comment #12) > Created attachment 342280 [details, diff] [details, diff] > let polkit-0.110 compile with spidermonkey-1.8.7 The library is opened using dlopen() and should not be linked against. This patch is not correct, and is not going to Portage as is. I ended up with same patch as workaround on my machine :-) Thanks for adding it here for others to use as workaround too.
Created attachment 343429 [details, diff] polkit-0.110-spidermonkey-187.patch A patch that makes sys-auth/polkit-0.110 compile with spidermonkey-1.8.7... But not necessarily work with. I wrote it because of a request from another user, and I have not the slightest knowledge about polkit or spidermonkey. Basically my idea is to load the symbols of JSVAL_* global variables like all other dynamically loaded symbols in the patch. Within my test polkit did run correctly, and pmap shows libmozjs187.so is loaded. Tried compiling with clang and didn't notice a related warning. No error on syslog as far as I could see. As I never use polkit myself, nonetheless, I do not know whether it works, and for any consequences of using the patch I cannot be liable.
Applying the following commit from master git fix it : f102bda3ced9958ed16df3e1885d6c086c675a41 9943e730d0d7f26a239834a07acb3bc3039ba57c 6bf958af19a1bfb1f6aacae869c7ace372d5c48b 8ed4cb000ed420bf2f899770cab098dde96621f2 0966ffc16bbd2998d7dad3064506e7a998d54bbb Then add the following environnment variable : LDFLAGS=-lmozjs187 EXTRA_ECONF=--with-mozjs=auto Everything is fine else
Well, maybe it's different issue, but polkit-110 should be "<dev-lang/spidermonkey-1.8.8[-debug]" I have to fix it everytime when update portage (otherwise it picks up new spidermonkey from mozilla portage, which is incompat)
Fixed in polkit-0.111 by added spidermonkey-17.0 support, support for 1.8.7 won't be added
