Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 44140 - PATCH: /usr/bin/firefox handles local files
Summary: PATCH: /usr/bin/firefox handles local files
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Aron Griffis (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-03-09 08:24 UTC by John Russell
Modified: 2004-03-18 18:29 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
adds local file support to /usr/bin/firefox script (firefox_local_files.patch,824 bytes, patch)
2004-03-09 08:24 UTC, John Russell
Details | Diff
Replacement patch for local file support (firefox_local_file2.patch,1.12 KB, patch)
2004-03-09 12:24 UTC, John Russell
Details | Diff
simpler patch (firefox.patch,422 bytes, patch)
2004-03-17 19:07 UTC, Aron Griffis (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description John Russell 2004-03-09 08:24:09 UTC
This patch adds a check in the /usr/bin/firefox script to see if any of the command line urls are local files.  If they are, it prepends the pwd to the basename so that firebird will open the file.  

Reproducible: Always
Steps to Reproduce:
1. run /usr/bin/firefox <localfile>
2. 
3.

Actual Results:  
firefox tries to find the local file as a remote host and usually fails.

Expected Results:  
It should open the local file.
Comment 1 John Russell 2004-03-09 08:24:43 UTC
Created attachment 27099 [details, diff]
adds local file support to /usr/bin/firefox script
Comment 2 John Russell 2004-03-09 12:24:27 UTC
Created attachment 27106 [details, diff]
Replacement patch for local file support

Please disregard the first patch, it didn't open firefox if there were no
arguments specified.  This patch fixes that.
Comment 3 Aron Griffis (RETIRED) gentoo-dev 2004-03-17 19:07:38 UTC
Created attachment 27536 [details, diff]
simpler patch

I think this does the same thing bug is much simpler.  Please test.
Comment 4 John Russell 2004-03-18 06:23:15 UTC
You're right, this is much simpler.  I tested it on relative paths, absolute paths and ones with ../../ in them.  It all seems to work well.  Thanks for the new patch.
Comment 5 Aron Griffis (RETIRED) gentoo-dev 2004-03-18 18:29:28 UTC
Thanks for testing.  I committed this change.