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

VERIFIED WONTFIX

Status

SeaMonkey
Build Config
P3
normal
VERIFIED WONTFIX
6 years ago
4 years ago

People

(Reporter: sgautherie, Assigned: sgautherie)

Tracking

Dependency tree / graph
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 obsolete attachments)

Comment hidden (empty)
(Assignee)

Comment 1

6 years ago
Created attachment 608957 [details] [diff] [review]
(Av1) Fix '--enable-jsd=no' packaging
Attachment #608957 - Flags: review?(bugspam.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.
Attachment #608957 - Flags: review?(bugspam.Callek) → review-
(Assignee)

Comment 3

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

Av1, with comment 2 suggestion(s).
Attachment #608957 - Attachment is obsolete: true
Attachment #609137 - Flags: review?(bugspam.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]
Attachment #609137 - Flags: review?(bugspam.Callek) → review-

Updated

5 years ago
Target Milestone: seamonkey2.11 → ---

Comment 5

4 years ago
JSD is dead and so in Venkman. JSD2 is the new black.
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → WONTFIX
(Assignee)

Updated

4 years ago
Attachment #609137 - Attachment is obsolete: true
(Assignee)

Comment 6

4 years ago
(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.
Status: RESOLVED → VERIFIED
Depends on: 1029513
Resolution: WONTFIX → FIXED
(Assignee)

Updated

4 years ago
Resolution: FIXED → WONTFIX
You need to log in before you can comment on or make changes to this bug.