838915 - Add a JS_STANDALONE configure/make variable

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[(not currently active) Ted Mielczarek]

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.

Comment 1

[darkxst]

Attachment: JS_STANDALONE

So, something alone the lines of this patch then? top-level configure or client.mk would need to gain an 'export MOZILLA_BUILD'

Comment 2

[Mike Hommey [:glandium]]

Comment on attachment 711221 [details] [diff] [review]
JS_STANDALONE

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

::: js/src/Makefile.in
@@ -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/configure.in
@@ +3924,5 @@
> +    JS_STANDALONE=0 )
> +if test -n "$MOZILLA_BUILD"; then
> +    JS_STANDALONE=0
> +fi
> +AC_DEFINE_UNQUOTED(JS_STANDALONE, $JS_STANDALONE)

Since it's going to be used in Makefile.in, it's AC_SUBST ; and we don't need a full-fledged configure option for that.

Comment 3

[Mike Hommey [:glandium]]

Attachment: Add a JS_STANDALONE configure/make variable

Comment 4

[(not currently active) Ted Mielczarek]

Comment on attachment 711221 [details] [diff] [review]
JS_STANDALONE

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 5

[(not currently active) Ted Mielczarek]

Comment on attachment 711225 [details] [diff] [review]
Add a JS_STANDALONE configure/make variable

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

Sold.

Comment 6

[Mike Hommey [:glandium]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/d17e8470d7d9

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/d17e8470d7d9

Comment 8

[Sean Stangl [:sstangl]]

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 https://wiki.mozilla.org/Release_Management/ESR_Landing_Process for more info.

Comment 9

[Alex Keybl [:akeybl]]

Comment on attachment 711225 [details] [diff] [review]
Add a JS_STANDALONE configure/make variable

Basically a NPOTB change in support of external packages. Approving.

Comment 10

[Sean Stangl [:sstangl]]

https://hg.mozilla.org/releases/mozilla-esr17/rev/58682dcf381a