kill preprocessor.pl

RESOLVED FIXED in mozilla6

Status

()

Core
Build Config
RESOLVED FIXED
7 years ago
6 years ago

People

(Reporter: ted, Assigned: Mitch)

Tracking

Trunk
mozilla6
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

7 years ago
Anything that's still using preprocessor.pl should be using Preprocessor.py. We should just switch anything that's left and hg rm it.
(Assignee)

Comment 1

7 years ago
Created attachment 448963 [details] [diff] [review]
Patch (v1)

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)
(Assignee)

Updated

7 years ago
Blocks: 569839
(Reporter)

Updated

7 years ago
Attachment #448963 - Flags: review?(ted.mielczarek) → review+
(Reporter)

Comment 2

7 years ago
Comment on attachment 448963 [details] [diff] [review]
Patch (v1)

Looks good, thanks.
Duplicate of this bug: 608234
http://hg.mozilla.org/mozilla-central/rev/d7692aed054f
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Duplicate of this bug: 653409
(Reporter)

Comment 6

6 years ago
Why did this get backed out?
Seeing http://hg.mozilla.org/mozilla-central/rev/2535ab584956 happening before http://hg.mozilla.org/mozilla-central/rev/bde78741b33b , maybe there were other bustages?
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
Last Resolved: 7 years ago6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla6
You need to log in before you can comment on or make changes to this bug.