596951 - Relative path support for webapp and profile

Status: RESOLVED FIXED

(Mozilla Labs :: Prism, defect)

Description

[Matthew Gertner]

In some cases (e.g. placing an app on removable media) it is necessary to specify relative paths for the webapp configuration files and profile. Currently Prism supports only absolute paths.

Comment 1

[Matthew Gertner]

Attachment: Add relative path support to stub

Comment 2

[Matthew Gertner]

Attachment: Fixed relative path detection on OS X

I found a problem with the previous patch. On Mac, when the extension manager asks for restart, the directory is changed to the one where the executable is located in the app bundle. By coincidence, this is at the same level in the bundle as the directory containing application.ini, which is where the relative paths are calculated from. As a result, the stat() call in SetEnvironmentVariable (which I was using as a kluge, assuming the path might be relative if the file can't be found) succeeds and the path isn't resolved. Execution fails later since there are places where an absolute path is assumed.

Comment 3

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 476812 [details] [diff] [review]
Fixed relative path detection on OS X

nice explanation

Comment 4

[Matthew Gertner]

Sending        client/stub/nsPrismStub.cpp
Transmitting file data .
Committed revision 75207.