Closed Bug 638429 Opened 13 years ago Closed 13 years ago

Disallow --disable-libxul

Categories

(Firefox Build System :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Mitch, Assigned: Mitch)

References

Details

(Whiteboard: fixed-in-bs)

Attachments

(1 file)

This concerns only allowing --enable-libxul, and not allowing --disable-libxul for building.
FWIW I think we should error on disable libxul rather than silently ignoring it.
Attached patch PatchSplinter Review
Attachment #516582 - Flags: review?(ted.mielczarek)
Comment on attachment 516582 [details] [diff] [review]
Patch

+MOZ_ARG_DISABLE_BOOL(libxul,
+[  --disable-libxul        Disable building libxul (deprecated)],

Maybe change this to (not supported) since it will actually raise an error?
Attachment #516582 - Flags: review?(ted.mielczarek) → review+
Blocks: 638755
http://hg.mozilla.org/mozilla-central/rev/d5ac43a73050
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Blocks: 644093
Blocks: 648911
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: