Closed Bug 612023 Opened 14 years ago Closed 14 years ago

Can't open URIs from the command line

Categories

(Thunderbird :: OS Integration, defect)

x86
Linux
defect
Not set
major

Tracking

(blocking-thunderbird5.0 alpha2+)

RESOLVED FIXED
Thunderbird 3.3a2
Tracking Status
blocking-thunderbird5.0 --- alpha2+

People

(Reporter: jcranmer, Assigned: protz)

References

(Blocks 1 open bug)

Details

(Keywords: regression)

Attachments

(1 file)

When I try doing mozilla/dist/bin/thunderbird -no-remote news:rec.games.pinball, I get an error to the effect that said URI is not a file. After doing some research, it looks like bug 287345 incorrectly assumed that anything hitting that statement was a file and not a URI. I believe this impacts shell integration for Linux, but I can't say about Windows or Mac OS X.
blocking-thunderbird5.0: --- → ?
blocking-thunderbird5.0: ? → alpha2+
Attached patch Patch (standard8's idea) (deleted) — Splinter Review
Bug 287345 introduced the issue probably, so what we do here is just revert the previous behavior in case the code from bug 287345 failed to take effect.
Assignee: nobody → jonathan.protzenko
Status: NEW → ASSIGNED
Attachment #496775 - Flags: review?(bugzilla)
Blocks: 549734
Comment on attachment 496775 [details] [diff] [review] Patch (standard8's idea) Although we could try being a URI first, and then try and be a file, but I don't think it really matters too much.
Attachment #496775 - Flags: review?(bugzilla) → review+
Keywords: checkin-needed
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 3.3a2
Should this fix be in http://mxr.mozilla.org/comm-central/source/mailnews/base/src/nsMailNewsCommandLineHandler.js instead? I think SeaMonkey would be happy to get it too.
(actually, never mind -- that file wouldn't handle your command-line at any rate.) We really should have a proper spec for our command-line stuff. :/
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: