Closed Bug 897909 Opened 11 years ago Closed 11 years ago

Cleanup some SIMPLE_PROGRAMS

Categories

(Firefox Build System :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla26

People

(Reporter: Ms2ger, Assigned: Ms2ger)

References

(Blocks 1 open bug)

Details

Attachments

(2 files, 1 obsolete file)

Attached patch Patch v1 (obsolete) — Splinter Review
It's a bit of a mess, I hope this improves things.
Attachment #780888 - Flags: review?(gps)
Blocks: nomakefiles
Attached patch Patch v1.1Splinter Review
Found another file.
Attachment #780888 - Attachment is obsolete: true
Attachment #780888 - Flags: review?(gps)
Attachment #780899 - Flags: review?(gps)
Comment on attachment 780899 [details] [diff] [review]
Patch v1.1

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

I'm happy if try is happy.

I really wish we had a decent grammar for representing binaries. This is quickly growing out of hand...
Attachment #780899 - Flags: review?(gps) → review+
Attached patch Some moreSplinter Review
I seem to have missed some.
Attachment #784905 - Flags: review?(gps)
Comment on attachment 784905 [details] [diff] [review]
Some more

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

::: tools/codesighs/moz.build
@@ +19,1 @@
>          'nm2tsv.c',

Consider these .c's gone.
Review ping.
Comment on attachment 784905 [details] [diff] [review]
Some more

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

Sorry for the review lag.
Attachment #784905 - Flags: review?(gps) → review+
https://hg.mozilla.org/mozilla-central/rev/fdeb80c577b3
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → mozilla26
See Also: → 872086
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: