[open-ils-commits] r19847 - branches/rel_2_1/Open-ILS/src/extras (dbs)

[svn at svn.open-ils.org]

Author: dbs
Date: 2011-03-21 21:25:54 -0400 (Mon, 21 Mar 2011)
New Revision: 19847

Modified:
   branches/rel_2_1/Open-ILS/src/extras/Makefile.install
Log:
Build libjs from source for now until the Perl threading problem can be reliably tested

The previous commit changed JavaScript::SpiderMonkey to be built without
thread safety, but on Fedora we had been installing the js-devel package
(which was threaded). As you can't build a non-threaded
JavaScript::SpiderMonkey on top of a threaded libjs, for now we'll make
Fedora build a non-threaded libjs from source just like the other distros.

Note that the problem with threaded JavaScript::SpiderMonkey surfaced
on Debian Lenny when targeting holds. Lenny has libthreads-perl 1.71, but
a number of fixes have been committed to subsequent versions
(http://cpansearch.perl.org/src/JDHEDDEN/threads-1.82/Changes) - so
threading support might be reliable on more current distros. Once we
have a reliable test, we might be able to go back to the packaged
version of libjs.


Modified: branches/rel_2_1/Open-ILS/src/extras/Makefile.install
===================================================================
--- branches/rel_2_1/Open-ILS/src/extras/Makefile.install	2011-03-22 01:24:43 UTC (rev 19846)
+++ branches/rel_2_1/Open-ILS/src/extras/Makefile.install	2011-03-22 01:25:54 UTC (rev 19847)
@@ -156,7 +156,6 @@
 FEDORA_13_RPMS = \
 	aspell \
 	aspell-en \
-	js-devel \
 	libdbi \
 	libdbi-dbd-pgsql \
 	libdbi-devel \
@@ -268,7 +267,7 @@
 rhel: install_redhat_pgsql centos_like
 centos_like: install_centos_rpms install_yaz install_cpan_marc install install_centos_perl create_ld_local install_cpan_safe install_cpan_force
 
-fedora13: install_fedora_13_rpms install_cpan install_cpan_fedora install_cpan_marc install_spidermonkey install_cpan_force
+fedora13: install_fedora_13_rpms install_cpan install_cpan_fedora install_cpan_marc install_js_sm install_cpan_force
 fedora14: fedora13
 
 debian-lenny: lenny generic_debian install_cpan_more install_cpan_safe