Closed Bug 569365 Opened 10 years ago Closed 9 years ago

kill preprocessor.pl

Categories

(Firefox Build System :: General, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
mozilla6

People

(Reporter: ted, Assigned: Mitch)

References

Details

Attachments

(1 file)

Anything that's still using preprocessor.pl should be using Preprocessor.py. We should just switch anything that's left and hg rm it.
Attached patch Patch (v1)Splinter Review
This patch removes removes preprocessor.pl from config and js/src/config, and copies Preprocessor.py to the latter.
Assignee: nobody → mitchell.field
Status: NEW → ASSIGNED
Attachment #448963 - Flags: review?(ted.mielczarek)
Blocks: 569839
Attachment #448963 - Flags: review?(ted.mielczarek) → review+
Comment on attachment 448963 [details] [diff] [review]
Patch (v1)

Looks good, thanks.
http://hg.mozilla.org/mozilla-central/rev/d7692aed054f
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Duplicate of this bug: 653409
Why did this get backed out?
http://hg.mozilla.org/mozilla-central/rev/e7cf79fd6d32

Needed a copy of Expression.py. (And we don't need to update tools/relic/relic, according to gerv)
Status: REOPENED → RESOLVED
Closed: 9 years ago9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla6
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.