Closed Bug 927915 Opened 11 years ago Closed 11 years ago

Automatically build NSPR for SpiderMonkey when --with-system-nspr or --with-nspr-prefix are not defined

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 975011

People

(Reporter: terrence, Assigned: terrence)

References

Details

Attachments

(1 file)

A bare configure no longer works now that --enable-threadsafe is the default. This is suboptimal.
Yes please, we should have done this a long time ago. If you can make this work I'll buy you and the reviewers a beer (or a beverage of your choice) next time I'm in MV. Maybe more importantly, this may unbreak some fuzz tools like JSBugmon.
Attached patch autobuild_nspr-wip0.diff (deleted) — Splinter Review
That's my thinking as well. Unfortunately, it's going to take me at least a week at my current rate of progress. I'm fine with m4 and make, I'm just not at all familiar with mozilla's particular build pidgin. Attached is a semi-functional prototype: it runs NSPR's configure and make out of SpiderMonkey's configure and make. I think the configure is okay enough as is, but the build is being triggered out of defaults::, which is probably not okay. Sadly, nspr's build step has to run before we can include pratom.h, so we can't do this like we do for libffi and libicu. Also, it does not yet set the right libs, so that has to be done manually. And make is still finding and depending on the installed nspr. And I think I broke --enable-ctypes somehow. And make seems to be rebuilding everything all the time with this. But otherwise it kinda works. Hopefully someone more knowledgeable can take it from here because I'm down to really inefficient trial-and-error at this point and I'm sure to break other things if I keep fumbling about.
OK, let's try to find an owner. gps, would you be willing to take this?
Flags: needinfo?(gps)
I could probably review this if it's ready. glandium could deliver a higher quality review, however, as my knowledge on linking and system package subtleties pales in comparison to his.
Flags: needinfo?(gps)
I didn't know about this bug, but I ended up implementing it because I was having trouble getting Windows shell builds to work with manual nsprpub configure invocations, and glandium suggested doing this. Note that the default for the JS shell on posix platforms is now the posix emulation stuff, so this no longer matters for most people.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: