"thunderbird -version" outputs an extra space

RESOLVED FIXED in mozilla25

Status

()

Toolkit
Startup and Profile System
--
trivial
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: Stefan Persson, Assigned: sshagarwal)

Tracking

Trunk
mozilla25
x86
All
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

4 years ago
User Agent: Mozilla/5.0 (X11; Linux i686; rv:20.0) Gecko/20100101 Firefox/20.0 SeaMonkey/2.17 (Beta/Release)
Build ID: 20130403195639

Steps to reproduce:

wget http://ftp.mozilla.org/pub/mozilla.org/thunderbird/nightly/2013-07-03-03-13-28-comm-central/thunderbird-25.0a1.en-US.linux-i686.tar.bz2
tar xvjf thunderbird-25.0a1.en-US.linux-i686.tar.bz2
cd thunderbird
./thunderbird -version


Actual results:

Thunderbird outputs " Thunderbird 25.0a1" with a space at the beginning.


Expected results:

Thunderbird should output "Thunderbird 25.0a1" without a space at the beginning. Bug also present in Thunderbird 17.0.7. Not a big issue, but the extra space looks a bit odd.
confirmed.
doesn't happen for firefox or seamonkey.
Thanks for the bug report
Severity: normal → trivial
Status: UNCONFIRMED → NEW
Component: Untriaged → General
Ever confirmed: true
OS: Linux → All

Comment 2

4 years ago
For some reason TB nor Firefox print anything for me on Win XP.
Wayne, what do they print for you?

I found this spot in the code that could be the culprit:
http://mxr.mozilla.org/comm-central/source/mozilla/toolkit/xre/nsAppRunner.cpp#1399

I suspect our gAppData->vendor is empty because we have no definition for it in http://mxr.mozilla.org/comm-central/source/mail/app/application.ini .

If that is intentional and we do not want to add Vendor=Mozilla then we may need to fix the format string in the toolkit file.

Can anybody first test out the theory?
Output on windows has been broken for years and I forget the bug# but iirc both bsmedberg and I are cc on it.  Workaround is to pipe output to "more" 

Thunderbird.exe | more
(In reply to Wayne Mery (:wsmwk) from comment #3)
> Output on windows has been broken for years and I forget the bug# but iirc
> both bsmedberg and I are cc on it.  Workaround is to pipe output to "more" 
> 
> Thunderbird.exe | more

bug 355889, windows-only.  According to bsmedberg related to "Windows virtual consoles" but whether the issue is fixable in mozilla is unstated.  (bug 26761 might be instructive)
(Assignee)

Comment 5

4 years ago
Created attachment 776575 [details] [diff] [review]
Patch

Thanks for mentioning the code spot.
Possible fix.
Attachment #776575 - Flags: feedback?(acelists)

Comment 6

4 years ago
Comment on attachment 776575 [details] [diff] [review]
Patch

Yes, this is what I had in mind.

I have also tested that adding Vendor=<something> to application.ini makes -version output the vendor name too and then the space is useful (formats finely also without the patch).

So this is now up to standard8 to decide which way we want to go.
But I would propose to land this patch in toolkit too even if we add Vendor to Thunderbird.
Attachment #776575 - Flags: feedback?(mbanner)
Attachment #776575 - Flags: feedback?(acelists)
Attachment #776575 - Flags: feedback+
Comment on attachment 776575 [details] [diff] [review]
Patch

This is the right way to go. There's various significant issues that would make it difficult for us to move to a world where we specified the vendor which are covered in a different bug somewhere.
Attachment #776575 - Flags: feedback?(mbanner) → feedback+

Comment 8

4 years ago
OK, let's propose it to the toolkit.
Assignee: nobody → syshagarwal
Component: General → Startup and Profile System
Product: Thunderbird → Toolkit
Version: 25 → Trunk

Updated

4 years ago
Attachment #776575 - Flags: review?(benjamin)

Updated

4 years ago
Status: NEW → ASSIGNED
Attachment #776575 - Flags: review?(benjamin) → review+
(Assignee)

Updated

4 years ago
Keywords: checkin-needed
https://hg.mozilla.org/integration/mozilla-inbound/rev/bdf71cb870ac
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/bdf71cb870ac
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla25
You need to log in before you can comment on or make changes to this bug.