Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 630506 - dev-lang/spidermonkey-52.2.0_pre20170719: wrong path in pkgconfig
Summary: dev-lang/spidermonkey-52.2.0_pre20170719: wrong path in pkgconfig
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Keywording (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Mozilla Gentoo Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-09-09 17:54 UTC by Jura
Modified: 2017-10-03 20:51 UTC (History)
0 users

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jura 2017-09-09 17:54:21 UTC
$ cat /usr/lib/pkgconfig/mozjs-52.pc
prefix=/usr
exec_prefix=${prefix}
libdir=/usr/lib64
includedir=${prefix}/include

Name: SpiderMonkey 52.2.0
Description: The Mozilla library for JavaScript
Version: 52.2.0
Requires.private: nspr >= 4.9.2
Libs: -L${libdir} -lmozjs-52
Cflags: -include ${includedir}/mozjs-52/js/RequiredDefines.h -I${includedir}/mozjs-52


$ equery f spidermonkey |grep "include"
/usr/include/mozjs
/usr/include/mozjs/fdlibm.h
/usr/include/mozjs/jemalloc_types.h
/usr/include/mozjs/js
/usr/include/mozjs/js-config.h
/usr/include/mozjs/js.msg
/usr/include/mozjs/js/CallArgs.h
/usr/include/mozjs/js/CallNonGenericMethod.h
/usr/include/mozjs/js/CharacterEncoding.h
/usr/include/mozjs/js/Class.h
/usr/include/mozjs/js/Conversions.h
/usr/include/mozjs/js/Date.h
.........
Comment 1 Ian Stakenvicius (RETIRED) gentoo-dev 2017-10-03 19:56:42 UTC
Yep this is a bug.  I've worked around it by moving the directory in src_install, but a proper fix is going to require a patch to the build system.

Leaving open until a proper fix is applied.
Comment 2 Larry the Git Cow gentoo-dev 2017-10-03 20:51:31 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=eff1f89fa6cddd59b6b8b7f668ba9c629ad27ec1

commit eff1f89fa6cddd59b6b8b7f668ba9c629ad27ec1
Author:     Ian Stakenvicius <axs@gentoo.org>
AuthorDate: 2017-10-03 20:50:53 +0000
Commit:     Ian Stakenvicius <axs@gentoo.org>
CommitDate: 2017-10-03 20:51:22 +0000

    dev-lang/spidermonkey: fix includedir install path on 52.4.0
    
    Restored upstream config/baseconfig.mk to undo something that was
    applied agaist firefox-52.4.0esr.
    
    Closes: http://bugs.gentoo.org/630506
    
    Package-Manager: Portage-2.3.8, Repoman-2.3.1

 .../files/spidermonkey-52-baseconfig.patch         | 22 ++++++++++++++++++++++
 .../spidermonkey-52.4.0_pre20171003-r1.ebuild      |  4 +---
 2 files changed, 23 insertions(+), 3 deletions(-)

Additionally, it has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=add57df3530ae2bc1cfc7ae3933444b6f9ed3bbe

commit add57df3530ae2bc1cfc7ae3933444b6f9ed3bbe
Author:     Ian Stakenvicius <axs@gentoo.org>
AuthorDate: 2017-10-03 19:56:59 +0000
Commit:     Ian Stakenvicius <axs@gentoo.org>
CommitDate: 2017-10-03 20:51:20 +0000

    dev-lang/spidermonkey: work around bad include directory
    
    Rename the header install directory to mozjs-52 to work around
    the bad target includedir, until a proper patch to the build system
    can be identified.
    
    Bug: http://bugs.gentoo.org/630506
    
    Package-Manager: Portage-2.3.8, Repoman-2.3.1

 ....0_pre20171003.ebuild => spidermonkey-52.4.0_pre20171003-r1.ebuild} | 3 +++
 1 file changed, 3 insertions(+)}