dist/bin/phoenix is trying to launch $MOZILLA_FIVE_HOME/phoenix-bin, which is probably not there.
Created attachment 101781 [details] [diff] [review] patch to search PHOENIX_HOME, not MOZILLA_FIVE_HOME
*** Bug 173416 has been marked as a duplicate of this bug. ***
note that the current startup script can find the appropriate directory just fine without any *_HOME environment variable set. In the past, it was necessary to use the variable if the startup script was symlinked to. There might be a good reason to keep such functionality with Phoenix, but I can't think of one. The run-mozilla.sh script sets the MOZILLA_FIVE_HOME variable to the appropriate value, which is still needed by XPCOM.
andrew, agreed. the problem is that some systems have mozilla_five_home set, so starting phoenix is impossible.
Comment on attachment 101781 [details] [diff] [review] patch to search PHOENIX_HOME, not MOZILLA_FIVE_HOME bryner, can you review this patch please? Thanks.
Attachment #101781 - Flags: review?(bryner)
Comment on attachment 101781 [details] [diff] [review] patch to search PHOENIX_HOME, not MOZILLA_FIVE_HOME This is going to break things. run-mozilla.sh sets MOZILLA_FIVE_HOME in the environment, and XPCOM explicitly looks for the value of that environment variable.
Attachment #101781 - Flags: review?(bryner) → review-
bryner, this doesn't change that at all. this patch just ignores the user-set value if there is one. some package managers set the value of moz_five_home to the (correct) location of mozilla. this patch tells phoenix to ignore that, which I think it should. i.e., /usr/local/mozilla/phoenix-bin will NEVER exist if you installed mozilla via portage or whatever. also, XPCOM seems perfectly happy with this patch.
> run-mozilla.sh sets MOZILLA_FIVE_HOME in the environment and with the patch, it still does. the run-mozilla.sh script overrides anything that was set in the environment. and again, I don't know of any useful functionality that can be gained by looking at any *_HOME from the environment. My vote would be to remove it from the mozilla startup script completely.
Comment on attachment 101781 [details] [diff] [review] patch to search PHOENIX_HOME, not MOZILLA_FIVE_HOME Sorry, I should have read that closer. r=me.
Attachment #101781 - Flags: review- → review+
especially with phoenix potentially changing its name, I agree with Andrew that these lines should probably just be removed.
Summary: phoenix should not try to use $MOZILLA_FIVE_HOME → Mozilla Firebird should not try to use $MOZILLA_FIVE_HOME
I went ahead and just removed the environment variable check.
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.