Closed
Bug 781367
Opened 12 years ago
Closed 12 years ago
Clean up some old Mac code in comm-central
Categories
(Thunderbird :: Build Config, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 18.0
People
(Reporter: Nomis101, Assigned: Nomis101)
Details
Attachments
(2 files, 2 obsolete files)
18.53 KB,
patch
|
standard8
:
review+
|
Details | Diff | Splinter Review |
17.17 KB,
patch
|
Details | Diff | Splinter Review |
Comm-central code contains some code pieces which are not supported anymore (mostly powerpc-apple-darwin code). This patch removes all of this unsupported code. I also opened TBSpotlight.xcodeproj with Xcode, so everything gets updated automatically. Because it complained about "Thunderbird.mdimporter" vs "thunderbird.mdimporter" I changed every occurrence of the mdimporter to "Thunderbird".
Attachment #650342 -
Flags: review?(mbanner)
Comment 1•12 years ago
|
||
Comment on attachment 650342 [details] [diff] [review] Cleanup patch Using Thunderbird.mdimporter will regress an earlier change where we were broken with our symbol building due to the way case-sensitivity works badly on Mac, so we can't change that back again, it needs to be all lower-case.
Attachment #650342 -
Flags: review?(mbanner) → review-
Ah, interestingly, I didn't know that. So, I changed all to lower case than.
Attachment #650342 -
Attachment is obsolete: true
Attachment #650841 -
Flags: review?(mbanner)
Had to remove this stupid LastUpgradeCheck...
Attachment #650841 -
Attachment is obsolete: true
Attachment #650841 -
Flags: review?(mbanner)
Attachment #650842 -
Flags: review?(mbanner)
Comment 4•12 years ago
|
||
Can this be removed only after TB 17 so that TenFourbird PPC can build a 17ESR successfully?
Hm, I don't think this will hinder TenFourbird PPC in any way. This patch will only remove ony single leftover line of PPC code. Which can reverted easily, if needed. There are much bigger obstacles for PPC builds in Core code. But I'm up for whatever Mark thinks.
Comment 6•12 years ago
|
||
Try run for f506875c8e43 is complete. Detailed breakdown of the results available here: https://tbpl.mozilla.org/?tree=Try&rev=f506875c8e43 Results (out of 10 total builds): success: 8 warnings: 1 failure: 1 Builds (or logs if builds failed) available at: http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/bugzilla@standard8.plus.com-f506875c8e43
Comment 7•12 years ago
|
||
Comment on attachment 650842 [details] [diff] [review] Cleanup patch v2 Review of attachment 650842 [details] [diff] [review]: ----------------------------------------------------------------- Sorry for the delay in getting to this. r=me with the following dropped. ::: mail/installer/removed-files.in @@ +336,5 @@ > res/html/gopher-text.gif > res/html/gopher-unknown.gif > @DLL_PREFIX@sqlite3@DLL_SUFFIX@ > #ifdef XP_MACOSX > + Thunderbird.app/Contents/Library/Spotlight/thunderbird.mdimporter/Contents/Info.plist iirc, these were from a time we were placing the mdimporter in totally the wrong place, and so the case should remain the same (i.e. drop the changes in this file).
Attachment #650842 -
Flags: review?(mbanner) → review+
Updated•12 years ago
|
Assignee: nobody → Nomis101
Flags: in-testsuite-
Updated•12 years ago
|
Status: NEW → ASSIGNED
Keywords: checkin-needed
Comment 9•12 years ago
|
||
https://hg.mozilla.org/comm-central/rev/53dc82062ba8
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 18.0
You need to log in
before you can comment on or make changes to this bug.
Description
•