Wladimir Palant Assignee
	Description • 17 years ago

I start a XULRunner application:

xulrunner/xulrunner.exe xul/application.ini

Somewhere in the application a restart is triggered:

    Components.classes["@mozilla.org/toolkit/app-startup;1"].getService(nsIAppStartup)
              .quit(nsIAppStartup.eAttemptQuit | nsIAppStartup.eRestart);

On restart I see the message "Error: couldn't parse application.ini". Reason is that the current directory changed, relative application.ini path is resolved to xulrunner/xul/application.ini.

I see two possible solutions:

1. Don't store relative paths, save the path after resolution of relative paths instead.

2. Save current directory and restore is when application is restarted.

The first should be easier to implement while the second is a more general solution for this kind of issues.

	Wladimir Palant Assignee
	Comment 1 • 17 years ago

This patch implements the first alternative. Please tell me if you think that we need a more general approach here.

	Mark Finkle (:mfinkle) (use needinfo?)
	Comment 2 • 17 years ago

For consistency, should the absolute path code be added here too?
http://mxr.mozilla.org/seamonkey/source/browser/app/nsBrowserApp.cpp#139

	Wladimir Palant Assignee
	Comment 3 • 17 years ago

You are right, we need to change browser as well.

	Mark Finkle (:mfinkle) (use needinfo?)
	Comment 4 • 17 years ago

FYI: the potential fix for bug 386826 will collide with this patch

	Wladimir Palant Assignee
	Comment 5 • 17 years ago

Yes, I am aware of that.

	Wladimir Palant Assignee
	Comment 6 • 17 years ago

Comment on attachment 280339 [details] [diff] [review]
Proposed patch v2

According to Roland Boon, his patch in bug 386826 fixes this issue as well.