Closed Bug 838915 Opened 10 years ago Closed 10 years ago

Add a JS_STANDALONE configure/make variable


(Firefox Build System :: General, defect)

Not set


(firefox20 wontfix, firefox21 fixed, firefox-esr17 fixed)

Tracking Status
firefox20 --- wontfix
firefox21 --- fixed
firefox-esr17 --- fixed


(Reporter: ted, Assigned: glandium)




(1 file, 1 obsolete file)

We should have a configure/make variable that indicates whether Spidermonkey is being built standalone. I think we could just pass this down as =0 from the top-level configure, and default it to =1 if it's unset.
Attached patch JS_STANDALONE (obsolete) — Splinter Review
So, something alone the lines of this patch then? top-level configure or would need to gain an 'export MOZILLA_BUILD'
Comment on attachment 711221 [details] [diff] [review]

Review of attachment 711221 [details] [diff] [review]:

::: js/src/
@@ -37,4 @@
>  SRCREL_ABI_VERSION  := $(word 1,$(subst ., ,$(SRCREL_VERSION)))
>  MODULE		    = js
> -ifneq (,$(MOZILLA_OFFICIAL))

That comes from bug 812265, but the dependency is the other way around: the patch from bug 812265 will have to be rewritten after this one.

::: js/src/
@@ +3924,5 @@
> +    JS_STANDALONE=0 )
> +if test -n "$MOZILLA_BUILD"; then
> +fi

Since it's going to be used in, it's AC_SUBST ; and we don't need a full-fledged configure option for that.
Assignee: ted → mh+mozilla
Attachment #711221 - Attachment is obsolete: true
Comment on attachment 711221 [details] [diff] [review]

Pretty close, yeah. I wouldn't bother adding a configure argument though, we can just export JS_STANDALONE=0 from the top-level configure.
Comment on attachment 711225 [details] [diff] [review]
Add a JS_STANDALONE configure/make variable

Review of attachment 711225 [details] [diff] [review]:

Attachment #711225 - Flags: review?(ted) → review+
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla21
Comment on attachment 711225 [details] [diff] [review]
Add a JS_STANDALONE configure/make variable

[Approval Request Comment]
If this is not a sec:{high,crit} bug, please state case for ESR consideration:

Necessary for the mozjs17 standalone release of the JS engine. Defines JS_STANDALONE for a standalone release. Does not affect non-standalone builds.

User impact if declined:

The patch is necessary for the upcoming JS standalone release based on esr17. If declined, the patch will have to be carried separately from the main repository for the lifetime of mozjs17 support.

Declining will also prevent Bug 812265 (proper library versioning) from landing on esr17.

Fix Landed on Version: 21
Risk to taking this patch (and alternatives if risky): None.
String or UUID changes made by this patch: None.

See for more info.
Attachment #711225 - Flags: approval-mozilla-esr17?
Comment on attachment 711225 [details] [diff] [review]
Add a JS_STANDALONE configure/make variable

Basically a NPOTB change in support of external packages. Approving.
Attachment #711225 - Flags: approval-mozilla-esr17? → approval-mozilla-esr17+
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.