Closed Bug 559498 Opened 10 years ago Closed 10 years ago

configure.in: s/--enable-long-long-warning/--disable-long-long-warning/

Categories

(Firefox Build System :: General, defect, trivial)

Product:
Firefox Build System
Component:
General
Type:
defect
Priority:
Not set
Severity:
trivial

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
mozilla1.9.3a5

People

(Reporter: sgautherie, Assigned: sgautherie)

Details

Attachments

(2 files)

(Av1) Just do it, Re-indent++ this block [Checkin: Comment 4]
4.64 KB, patch
ted
: review+
Details | Diff | Splinter Review
(Bv1-CC) Copy it to comm-central [Checkin: Comment 5]
2.38 KB, patch
Callek
: review+
Details | Diff | Splinter Review 
Make it explicit that the default is to warn.
Flags: in-testsuite-
Comment on attachment 439173 [details] [diff] [review]
(Bv1-CC) Copy it to comm-central
[Checkin: Comment 5]

I'm not convinced here; but if m-c takes it we can too. this is identical to the m-c patch so r+. Hold off landing until m-c lands please.
Attachment #439173 - Flags: review?(bugspam.Callek) → review+
Attachment #439172 - Flags: review?(ted.mielczarek) → review+
Comment on attachment 439172 [details] [diff] [review]
(Av1) Just do it, Re-indent++ this block
[Checkin: Comment 4]


http://hg.mozilla.org/mozilla-central/rev/d7f2c8ae64da
Attachment #439172 - Attachment description: (Av1) Just do it, Re-indent++ this block → (Av1) Just do it, Re-indent++ this block [Checkin: Comment 4]
Comment on attachment 439173 [details] [diff] [review]
(Bv1-CC) Copy it to comm-central
[Checkin: Comment 5]


http://hg.mozilla.org/comm-central/rev/907994de5aeb
Attachment #439173 - Attachment description: (Bv1-CC) Copy it to comm-central → (Bv1-CC) Copy it to comm-central [Checkin: Comment 5]
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3a5
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.