Closed Bug 1048042 Opened 5 years ago Closed 5 years ago

Port |Bug 780159 - Remove obsolete REGCHROME| and |Bug 935387 - Remove non recursed install targets| to comm-central

Categories

(MailNews Core :: Build Config, defect)

defect
Not set

Tracking

(thunderbird34 fixed)

RESOLVED FIXED
Thunderbird 34.0
Tracking Status
thunderbird34 --- fixed

People

(Reporter: iann_bugzilla, Assigned: iann_bugzilla)

References

Details

Attachments

(2 files)

Bug 935387 - Remove non recursed install targets
We've not recursed for make install for a while. There are a few places where the install targets are used, but most others are just pointless Makefile boilerplate. Remove them.

Dealt with mailnews, mail, suite, chat and im but left ldap alone. Also did not touch build.mk files.

Bug 780159 - Remove obsolete REGCHROME
It's used nowhere.

Bug 780159 almost made it to comm-central in Bug 780357 but it got lost on the way.
Attachment #8466809 - Flags: review?(standard8)
Attachment #8466809 - Flags: review?(standard8) → review+
Comment on attachment 8466809 [details] [diff] [review]
Port removal of non-recursed install targets and obsolete REGCHROME [Checked in: Comment 1]

https://hg.mozilla.org/comm-central/rev/c6e6b13f5319
Attachment #8466809 - Attachment description: Port removal of non-recursed install targets and obsolete REGCHROME → Port removal of non-recursed install targets and obsolete REGCHROME [Checked in: Comment 1]
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 34.0
Still REGCHROME escaped. Patch coming up.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
I think I got bitrotted and when I recreated the patch I lost the REGCHROME removal, so here it is for real.
Attachment #8468570 - Flags: review?(standard8)
Status: REOPENED → ASSIGNED
Attachment #8468570 - Flags: review?(standard8) → review+
Comment on attachment 8468570 [details] [diff] [review]
Actually remove obsolete REGCHROME code [Checked in: Comment 4]

http://hg.mozilla.org/comm-central/rev/c92946ba1b28
Attachment #8468570 - Attachment description: Actually remove obsolete REGCHROME code → Actually remove obsolete REGCHROME code [Checked in: Comment 4]
Status: ASSIGNED → RESOLVED
Closed: 5 years ago5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.