Skip Menu |
 

This queue is for tickets about the JSPL CPAN distribution.

Report information
The Basics
Id: 81542
Status: patched
Priority: 0/
Queue: JSPL

People
Owner: Nobody in particular
Requestors: dschrag [...] oneupweb.com
Cc:
AdminCc:

Bug Information
Severity: Critical
Broken in:
  • 1.06
  • 1.07
Fixed in: (no value)



Subject: Brute-force header search fails due to simple typo
MIME-Version: 1.0
Content-Type: text/plain; charset="UTF-8"
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 1513
Download (untitled) / with headers
text/plain 1.4k
I'm building on debian (squeeze) and tried to find the simplest way to install this. The README recommends the xrunner-dev package to satisfy requirements, but it is really heavyweight. I installed libmozjs-dev instead. When installing, I got: Can't exec "pkg-config": No such file or directory at Makefile.PL line 109. Can't run pkg-config Will try a brute-force search, review the results... Sad, no SM headers found SpiderMonkey headers and libraries not found. The headers are there, in reasonable paths, but not found because the search is looking for 'jsapi.js' instead of 'jsapi.h' !! The following patch fixes: --- Makefile.PL.org 2012-01-16 13:59:08.000000000 -0500 +++ Makefile.PL 2012-11-29 11:37:18.000000000 -0500 @@ -165,7 +165,7 @@ unless($incdirs) { my $hash=0; for my $p (@ipaths) { - if(-f("$p/jsapi.js") && -f("$p/jsdbgapi.h")) { + if(-f("$p/jsapi.h") && -f("$p/jsdbgapi.h")) { push @incs, "-I$p"; $hash++; } Wow! Now it just works. This is a great module - I hope more people can try it out now. Until this is released, those who don't feel like applying the patch, but know where their files are located, can just 'touch /usr/include/mozjs/jsapi.js' (modify the path accordingly) and it should also work. The libmozjs installation I have is version 1.9.1, and it seems to be working (just started testing) so you might consider updating the README for the versions supported as well.
MIME-Version: 1.0
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
Content-Type: text/plain; charset="UTF-8"
Message-ID: <rt-3.8.HEAD-11681-1354303681-1008.81542-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 77
Good catch! Will be included in next release. Thank you so much. Salvador.


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.