Closed
Bug 509147
Opened 15 years ago
Closed 15 years ago
Port |Bug 483856 - Use $(CURDIR) to make pymake happy| to comm-central
Categories
(MailNews Core :: Build Config, defect)
MailNews Core
Build Config
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 3.1b2
People
(Reporter: kairo, Assigned: sgautherie)
References
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
standard8
:
review+
|
Details | Diff | Splinter Review |
As mentioned in bug 509095, it's OK to wallpaper over the lack of core_abspath in comm-central config.mk as long as our mainline is based on 1.9.1, but in the long run we probably want to do things right and port bug 483856 to comm-central (after we branch).
Assignee | ||
Updated•15 years ago
|
Assignee | ||
Updated•15 years ago
|
Summary: Port bug 483856 to comm-central → Port |Bug 483856 - Use $(CURDIR) to make pymake happy| to comm-central
Assignee | ||
Updated•15 years ago
|
Assignee | ||
Comment 1•15 years ago
|
||
*Copy bug 483856 itself.
*PP_LOCALIZED_FILES part: was removed by TB bug 533043 and SM bug 521523.
*core_abspath: undo TB bug 509095, SM+TB changeset a09a7d1c82d7 (bug 463605) and part of SB changeset b9ea76f33831.
Attachment #426993 -
Flags: review?(bugzilla)
Comment 2•15 years ago
|
||
Comment on attachment 426993 [details] [diff] [review]
(Av1) Just copy it, Remove previous c-c workaround(s)
I did review all parts; but I'm not technically a mail/ or a calendar/ reviewer so leaving mark's request open.
Attachment #426993 -
Flags: review+
Comment 3•15 years ago
|
||
Comment on attachment 426993 [details] [diff] [review]
(Av1) Just copy it, Remove previous c-c workaround(s)
This either needs a refresh or this is fixed now.
Attachment #426993 -
Flags: review?(bugzilla)
Assignee | ||
Comment 4•15 years ago
|
||
What's left of patch Av1 after bug 545085 :-/
Attachment #426993 -
Attachment is obsolete: true
Attachment #431064 -
Flags: review?(bugzilla)
Comment 5•15 years ago
|
||
Comment on attachment 431064 [details] [diff] [review]
(Av2) Just copy it
[Checkin: See comment 6]
>diff --git a/mail/base/Makefile.in b/mail/base/Makefile.in
>--- a/mail/base/Makefile.in
>+++ b/mail/base/Makefile.in
>@@ -68,9 +68,8 @@ endif
...
> include $(topsrcdir)/config/rules.mk
>-
Please drop this change, its unnecessary here.
r=Standard8 without that.
Attachment #431064 -
Flags: review?(bugzilla) → review+
Assignee | ||
Comment 6•15 years ago
|
||
Comment on attachment 431064 [details] [diff] [review]
(Av2) Just copy it
[Checkin: See comment 6]
http://hg.mozilla.org/comm-central/rev/7353fbf8aafe
Av2, with comment 5 suggestion(s).
Attachment #431064 -
Attachment description: (Av2) Just copy it → (Av2) Just copy it
[Checkin: See comment 6]
Assignee | ||
Updated•15 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Target Milestone: Thunderbird 3.1b1 → Thunderbird 3.1b2
You need to log in
before you can comment on or make changes to this bug.
Description
•