Closed
Bug 947725
Opened 7 years ago
Closed 7 years ago
enable-xterm-updates no longer works in comm-central parts of the build
Categories
(MailNews Core :: Build Config, defect)
MailNews Core
Build Config
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 28.0
People
(Reporter: neil, Assigned: neil)
Details
Attachments
(1 file)
744 bytes,
patch
|
jcranmer
:
review+
|
Details | Diff | Splinter Review |
Bug 944558 was a step towards making mozilla-central build with GNU make for Windows. However they slipped in the removal of print-depth-path.sh which is still referenced in comm-central's rules.mk if you use enable-xterm-updates.
Assignee | ||
Comment 1•7 years ago
|
||
Just port the one line in question. (Normally I would investigate porting all the GNU make for Windows compatibility patches, but I'd like this particular bit to make it into 28.)
Comment 2•7 years ago
|
||
Comment on attachment 8344366 [details] [diff] [review] Proposed patch My current policy on porting build patches has been to only port things that are needed to avoid bustages or to fix ccrework, as I expect we can kill the c-c build system entirely in about 1-2 months.
Attachment #8344366 -
Flags: review?(Pidgeot18) → review+
Assignee | ||
Comment 3•7 years ago
|
||
Pushed comm-central changeset 07fc1446c1ab. Then, realised that the patch was malformed all along (sorry about that) and pushed comm-central changeset 4abeeaf9eb70.
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Updated•7 years ago
|
Target Milestone: --- → Thunderbird 28.0
You need to log in
before you can comment on or make changes to this bug.
Description
•