Port Bug 553073 (and bug 554997) CFBundleIdentifier in Info.plist.in is hardcoded as "org.mozilla.thunderbird"

RESOLVED FIXED in Thunderbird 3.3a1

Status

RESOLVED FIXED
9 years ago
8 years ago

People

(Reporter: standard8, Assigned: Nomis101)

Tracking

Trunk
Thunderbird 3.3a1
All
Mac OS X
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

9 years ago
+++ This bug was initially created as a clone of Bug #553073 +++

The CFBundleIdentifier in /mail/app/macbuild/Contents/Info.plist.in is hardcoded as "org.mozilla.thunderbird", which causes default mail issues when running shredder and thunderbird on the same machine.

See bug 553073 for the rest of the info. I'd also like a solution to bug 554997 before porting this, hence the dependencies.
(Reporter)

Comment 1

9 years ago
Note to self: do after 3.1b2, then if bug 554997 hasn't been fixed, we're not affecting users who are upgrading from 3.1b1 (Lanikai branding) to 3.1b2 (Thunderbird branding), we just need to make sure that bug is fixed by the next time we do such a switch.
Whiteboard: [do after 3.1beta2]
(Assignee)

Comment 2

9 years ago
Created attachment 439656 [details] [diff] [review]
patch (C&P from 553073)

I have a patch for this (simple copy&paste from bug 553073). I've tested this with official and unofficial branding. With official branding CFBundleIdentifier is set to org.mozilla.thunderbird and with unofficial branding it is org.mozilla.shredder (only tested with 3.2, but I assume it will work the same for 3.1).
(Reporter)

Updated

8 years ago
Depends on: 314794
No longer depends on: 554997
(Reporter)

Updated

8 years ago
Depends on: 554997
(Reporter)

Comment 3

8 years ago
Comment on attachment 439656 [details] [diff] [review]
patch (C&P from 553073)

>-	sed -e "s/%APP_VERSION%/$(MOZ_APP_VERSION)/" -e "s/%APP_NAME%/$(APP_NAME)/" $(srcdir)/macbuild/Contents/Info.plist.in > $(DIST)/$(APP_NAME).app/Contents/Info.plist
...
>+	sed -e "s/%APP_VERSION%/$(APP_VERSION)/" -e "s/%APP_NAME%/$(APP_NAME)/" -e "s/%LOWER_APP_NAME%/$(LOWER_APP_NAME)/" $(srcdir)/macbuild/Contents/Info.plist.in > $(DIST)/$(APP_NAME).app/Contents/Info.plist

Our makefile doesn't have APP_VERSION, so that bit should remain as MOZ_APP_VERSION.

Nomis101, want to update now that the dependent bugs seem fixed?
(Reporter)

Updated

8 years ago
Assignee: bugzilla → Nomis101
(Assignee)

Comment 4

8 years ago
Created attachment 480517 [details] [diff] [review]
patch v2

OK, fixed the overlooked thing.
Attachment #439656 - Attachment is obsolete: true
Attachment #480517 - Flags: review?(bugzilla)
(Reporter)

Comment 5

8 years ago
Comment on attachment 480517 [details] [diff] [review]
patch v2

Thanks for the update
Attachment #480517 - Flags: review?(bugzilla) → review+
(Assignee)

Updated

8 years ago
Keywords: checkin-needed
(Reporter)

Comment 6

8 years ago
Checked in: http://hg.mozilla.org/comm-central/rev/3df8b762deca
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Whiteboard: [do after 3.1beta2]
Target Milestone: --- → Thunderbird 3.3a1
You need to log in before you can comment on or make changes to this bug.