Opened 14 years ago

Closed 14 years ago

#26041 closed defect (fixed)

spidermonkey 1.7.0 +fileobject does not actually enable the file object

Reported by: onne@… Owned by: akitada@…
Priority: Normal Milestone:
Component: ports Version: 1.9.1
Keywords: Cc: vjt@…
Port: spidermonkey

Description

somehow due to the config/Darwin.mk the JS_HAS_FILEOBJECT=1 fails to be passed on during compile as -DJS_HAS_FILEOBJECT

manually configuring, changing config/Darwin.mk and then installing it works out

a quick way to see if it has build correctly:

strings which js | grep js_NewFileObjectFromFILE
echo "no fileobject"

Change History (5)

comment:1 Changed 14 years ago by nerdling (Jeremy Lavergne)

Owner: changed from macports-tickets@… to akitada@…
Port: spidermonkey added

comment:2 Changed 14 years ago by vjt@…

Hello,

the following patch both fixes the reported problem, and enables readline support in the interactive JS shell.

The reason is that the DEFINES= parameter to make is shadowing the Makefile.ref generated $(DEFINES) variable. Anyway, if the -DJS_C_STRINGS_ARE_UTF8 is not specified on the command line, it gets added automagically by Makefile.ref, at least on my machine (Macports 1.9.1, Darwin 10.4.0/x86_64).

--- Portfile~	2010-11-04 12:26:07.000000000 +0100
+++ Portfile	2010-11-04 12:26:33.000000000 +0100
@@ -51,7 +51,7 @@
                     LIBDIR="/lib" \
                     SO_SUFFIX=dylib \
                     JS_THREADSAFE=1 \
-                    DEFINES=-DJS_C_STRINGS_ARE_UTF8
+                    JS_READLINE=1
 
 if { ![variant_isset universal] } {
     build.args-append CC='${configure.cc} -arch ${build_arch}'

comment:3 Changed 14 years ago by vjt@…

Cc: vjt@… added

Cc Me!

comment:4 Changed 14 years ago by kwatch@…

How about progress? I confirmed that the above patch works well (thanks vjt!).

I hope Macports to include the patch.

comment:5 Changed 14 years ago by pixilla (Bradley Giesbrecht)

Resolution: fixed
Status: newclosed

Fixed in r76712.

Note: See TracTickets for help on using tickets.