Closed Bug 453689 Opened 12 years ago Closed 11 years ago
Firefox needs to register the proper name with session management for restart
2.47 KB, patch
|Details | Diff | Splinter Review|
936 bytes, patch
|Details | Diff | Splinter Review|
2.71 KB, patch
|Details | Diff | Splinter Review|
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:18.104.22.168) Gecko/20080715 Fedora/22.214.171.124-1.fc8 Firefox/126.96.36.199 Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:188.8.131.52) Gecko/20080715 Fedora/184.108.40.206-1.fc8 Firefox/220.127.116.11 Firefox currently registers the full path to the binary executable with Unix session management. This causes problems because this skips running the usual shell wrapper script. I think Firefox should register "firefox" with session management and let it be found in the search path. Reproducible: Always Steps to Reproduce: 1. 2. 3.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Doesn't really meet the "wanted" criteria (security, stability, regression from maintenance release) for 1.9.0.x. However, we'll look at a reviewed and baked patch.
Flags: wanted1.9.0.x? → wanted1.9.0.x-
Originates from bug #262258#c40
downstream bug - https://bugzilla.redhat.com/show_bug.cgi?id=437596
What about this solution? MOZILLA_APP_LAUNCHER env variable (or something) gives an opportunity to set different application launcher (e.g. in some system-wide start up script).
I just tripped over this in Fedora 10. Any chance of a fix or workaround anytime soon?
Flags: blocking-firefox3.1? → blocking-firefox3.1-
I guess this would fix my problems on debian lenny, too. And the same problem exists with icedove (thunderbird) - maybe a core problem?
Component: Shell Integration → Startup and Profile System
Product: Firefox → Toolkit
QA Contact: shell.integration → startup
Version: unspecified → Trunk
The patch attached needs to be reviewed for inclusion to the the tree. Please ask for review from one of the toolkit developers http://www.mozilla.org/projects/toolkit/review.html https://developer.mozilla.org/en/Getting_your_patch_in_the_tree
Attachment #352541 - Flags: review?(benjamin)
Comment on attachment 352541 [details] [diff] [review] Env variable can suppress the default mozilla binary Benjamin, can you please check this one?
Comment on attachment 352541 [details] [diff] [review] Env variable can suppress the default mozilla binary >diff -U10 -up src/toolkit/xre/nsNativeAppSupportUnix.cpp.old src/toolkit/xre/nsNativeAppSupportUnix.cpp >+ #define ARGC 1 >+ char* argv[ARGC]; The array is unnecessary (and the #define is ugly!). You should be able to do: char *argv1 = nsnull; And then just pass &argv1 whenever you need an argv.
Attachment #352541 - Flags: review?(benjamin) → review-
Thanks for looking at it! There's an updated one there, it fixes one more problem in the original patch (gnome_client_set_restart_command() was executed only when MOZILLA_APP_LAUNCHER was not set).
Comment on attachment 375966 [details] [diff] [review] v2 [Checkin: Comment 13] Thanks! Is sr+ requested here?
Set checkin-needed, regards to https://developer.mozilla.org/en/Getting_your_patch_in_the_tree sr+ is not requested here (no cross-module, API change or security).
http://hg.mozilla.org/mozilla-central/rev/68e92b917783 Thanks, sorry about the delay.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Would be nice to get this into 1.9.1 at some point? And one nit even if it is too late: "MOZILLA_APP_LAUNCHER" almost every other env variable which is used within mozilla is just prefixed with "MOZ_". That looks quite inconsistent ;-)
You're right, here is the change.
Attachment #387405 - Flags: review?(benjamin)
Comment on attachment 387405 [details] [diff] [review] change "MOZILLA_APP_LAUNCHER" -> "MOZ_APP_LAUNCHER" Benjamin, do you agree with this change?
Comment on attachment 387405 [details] [diff] [review] change "MOZILLA_APP_LAUNCHER" -> "MOZ_APP_LAUNCHER" Please, attach an hg patch.
Comment on attachment 375966 [details] [diff] [review] v2 [Checkin: Comment 13] Can you explain why this is wanted for 1.9.1? What benefits does it have? Please renominate the appropriate patch (this isn't it) for 1.9.1 along with a detailed message of what this does and why we want it and we'll reconsider.
Comment on attachment 387405 [details] [diff] [review] change "MOZILLA_APP_LAUNCHER" -> "MOZ_APP_LAUNCHER" http://hg.mozilla.org/mozilla-central/rev/16da05181c51
Comment on attachment 389628 [details] [diff] [review] complete 1.9.1 hg patch [Checkin: Comment 24] This one merges the two trunk patches into one, gives us ability to register one, system wide firefox launcher by MOZ_APP_LAUNCHER variable (like /usr/bin/firefox). It's critical for us because the launcher script sets up language packs, plug-ins directories and so on. Without the patch, firefox launched by gnome session (after user login or session restoration) is broken (it launches the binary from default install dir, like /usr/lib(64)/firefox-XXX/firefox).
Attachment #389628 - Flags: approval18.104.22.168?
Comment on attachment 389628 [details] [diff] [review] complete 1.9.1 hg patch [Checkin: Comment 24] Approved for 22.214.171.124. a=ss for release-drivers Please land on mozilla-1.9.1 and use the ".2-fixed" option of the "status1.9.1" flag.
Attachment #389628 - Flags: approval126.96.36.199? → approval188.8.131.52+
Target Milestone: --- → mozilla1.9.2a1
Attachment #389624 - Attachment description: hg patch → hg patch [Checkin: Comment 21]
Attachment #389628 - Attachment description: complete 1.9.1 hg patch → complete 1.9.1 hg patch [Checkin: Comment 24]
Attachment #387405 - Attachment is obsolete: true
Martin, could you help us verify this in the 3.5.2 (build1) candidates?
Something is wrong probably. My testing wasn't successful on 3.5.2. In my firefox startscript I did this before starting Firefox: export MOZ_APP_LAUNCHER="fasel" After starting Firefox and saving my session before quitting my WM it hasn't taken this command for the session information.
I also tried that patch with Thunderbird 3.0b and it's also not working there. None of my tests had a real command saved in the session information. I'm not sure if the callback is even working correctly currently in general?
When I tested the patch I run it as export MOZ_APP_LAUNCHER="/usr/bin/firefox" because /usr/bin/firefox is the firefox system launcher in Fedora. And it worked fine (it was Fedora 10) but i can check it on some recent system.
Ok, I did a few more testing. Some info about my environment: - Firefox is started through /usr/bin/firefox - it's a shell script which starts /usr/lib64/firefox/firefox (which is actually the xulrunner stub as Firefox is xulrunner based here) Regardless if I set something for MOZ_APP_LAUNCHER or not the saved restart command is always "firefox". Looking at http://mxr.mozilla.org/mozilla1.9.1/source/toolkit/xre/nsNativeAppSupportUnix.cpp#128 that should never happen at all as either the provided MOZ_APP_LAUNCHER definition is used or the full path to the called executable which would be /usr/lib64/firefox/firefox and not just firefox. So my guess is that gnome_client_set_restart_command() is never called at all. From just reading the patch that seems unlikely as long as the callback works in general.
It works as expected, I've tested it on F11/latest 1.9.1. The steps are: 1) export the right launcher (export MOZ_APP_LAUNCHER="/home/komat/tmp516/191src/dist/bin/firefox" in my case) 2) Set up gnome session management (to remember application after logout) 3) Logout 4) Login 5) Firefox is launched right after login by gnome session management, with application set by MOZ_APP_LAUNCHER. Note: If you're interested in proper firefox restart command (because of new extension installation and so on), there's a different bug for it.
You need to log in before you can comment on or make changes to this bug.