Closed Bug 897909 Opened 12 years ago Closed 12 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+
Status: ASSIGNED → RESOLVED
Closed: 12 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: