Closed Bug 335268 Opened 19 years ago Closed 19 years ago

Change SeaMonkey app id and vendor to sane, new settings

Categories

(SeaMonkey :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: kairo, Unassigned)

References

Details

Attachments

(1 file)

Currently, we're using a GUID as app ID for SeaMonkey and vendor is none or "Mozilla".
We should change the app ID to seamonkey@applications.mozilla.org and on suiterunner we should use "mozilla.org" as vendor tag.

The app ID change should happen across the board, including branches and UMO.
Blocks: 335269
Pain and death and suffering.
http://lxr.mozilla.org/seamonkey/source/extensions/irc/xul/content/static.js#410
(line 428)

Grumble.
Fixing that routine would be easy, actually, just needs one line added...

So, if UMO can understand that two app IDs are the same app (just different versions), it may be a good idea to change it for suiterunner only - else we should just stay with the sucky GUID, just like Firefox and Thunderbird have to.

The vendor tag for suiterunner should change for sure though.
Just remember to fix it on all the branches you change the ID.
Through various discussions we've come to the conclusion that mozilla.org is the vendor id that we should use. This will give us the following from --version:

mozilla.org SeaMonkey 1.5a, Copyright (c) 1998-2006 mozilla.org

and

.mozilla.org/seamonkey/

as the profile directory.

The patch updates the startup information for this and removes the then-obsolete comments.
Attachment #220307 - Flags: superreview?(neil)
Attachment #220307 - Flags: review?(kairo)
Comment on attachment 220307 [details] [diff] [review]
Change vendor id to a sane setting (checked in)

Hmm, maybe we should really use MOZ_APP_DISPLAYNAME there though...
Attachment #220307 - Flags: review?(kairo) → review+
(In reply to comment #6)
> (From update of attachment 220307 [details] [diff] [review] [edit])
> Hmm, maybe we should really use MOZ_APP_DISPLAYNAME there though...
> 

Whilst I can see your point, I tend to agree with benjamin's comment on bug 330053: 

--- quote ---
I don't think you want to do this: this value determines the location of the
profile, which should be determined separately froma "branding" setting. (e.g.
Firefox w/o official branding still uses ~/.mozilla/firefox)
--- end quote ---
Attachment #220307 - Flags: superreview?(neil) → superreview+
Attachment #220307 - Attachment description: Change vendor id to a sane setting. → Change vendor id to a sane setting (checked in)
Hmm, if someone wants to do a rebranded distribution of the suite (like Beonex used to do), I guess he'll need to look into that file anyways, so it's OK to hardcode it here...
Using different branding for offcial/unofficial like Firefox isn't my primary concern here, but as Benjamin says, it's the correct thing to do in that case as well.

And I think changing the app ID isn't worth the effort after what we've heard, so let's just call this one FIXED.
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: