738879 - Port |Bug 735594 - --enable-jsd=no fails to build| to SeaMonkey

Status: VERIFIED WONTFIX

(SeaMonkey :: Build Config, defect, P3)

Comment 1

[Serge Gautherie (:sgautherie)]

Attachment: (Av1) Fix '--enable-jsd=no' packaging

Comment 2

[Justin Wood (:Callek)]

Comment on attachment 608957 [details] [diff] [review]
(Av1) Fix '--enable-jsd=no' packaging

I know this fixes an obscure configure options packaging error for us, but I'm not granting review because if we have no js debugger support, I want to explicitly not package/build venkman.

It is more work, but I'd rather leave it as is (breaking) when packaging without jsd, then to have it package venkman but not jsd in any packaged build.

Comment 3

[Serge Gautherie (:sgautherie)]

Attachment: (Av2) Fix '--enable-jsd=no' packaging, Apply it to Venkman too

Av1, with comment 2 suggestion(s).

Comment 4

[Justin Wood (:Callek)]

Comment on attachment 609137 [details] [diff] [review]
(Av2) Fix '--enable-jsd=no' packaging, Apply it to Venkman too

We have stuff in http://mxr.mozilla.org/comm-central/source/suite/locales/Makefile.in to [not] do for venkman

As well as http://mxr.mozilla.org/comm-central/source/suite/app/Makefile.in and then there is the fact that even with this patch we still build venkman, and it needs to change in configure.in for that [xref: MOZ_EXTENSIONS_DEFAULT http://mxr.mozilla.org/comm-central/source/suite/confvars.sh#51 which itself shouldn't change, but we want to strip venkman out when this config var is passed]

Comment 5

[Philip Chee]

JSD is dead and so in Venkman. JSD2 is the new black.

Comment 6

[Serge Gautherie (:sgautherie)]

(In reply to Philip Chee from comment #5)
> JSD is dead and so in Venkman. JSD2 is the new black.

Ftr, see "Bug 1029513 - Stop building Venkman because Bug 800200 removes JSD1 which Venkman depends on."

V.WontFix.