/usr/lib/firefox-granparadiso/firefox doesn’t run

Bug #123687 reported by Anders Kaseorg
2
Affects Status Importance Assigned to Milestone
Mozilla Firefox
Fix Released
High
firefox-3.0 (Ubuntu)
Fix Released
High
Mozilla Bugs

Bug Description

Binary package hint: firefox-granparadiso

$ /usr/lib/firefox-granparadiso/firefox

run-mozilla.sh: Cannot execute /usr/lib/firefox-granparadiso/firefox-bin.

This directory has a firefox-granparadiso-bin but not a firefox-bin.

Tags: mt-eval
Revision history for this message
In , Hilario J. Montoliu (hjmf) (hmontoliu) wrote :

Created attachment 268232
patch to /usr/bin/firefox shell script

Revision history for this message
In , Hilario J. Montoliu (hjmf) (hmontoliu) wrote :

Created attachment 268233
The same patch but against ./browser/app/mozilla.in

This is for source MOZILLA_1_8_BRANCH -- last build I've made:

Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.1.5pre) Gecko/20070607 BonEcho/2.0.0.5pre

Revision history for this message
In , Hilario J. Montoliu (hjmf) (hmontoliu) wrote :

Forgot to mention that this issue is dealt too at ubuntu's bug tracker:

https://bugs.launchpad.net/firefox/+bug/117575

Revision history for this message
In , Alexander Sack (asac) wrote :

confirmed.

Revision history for this message
In , Benjamin Smedberg (Mozilla) [:bs] (benjamin-smedbergs) wrote :

Comment on attachment 268233
The same patch but against ./browser/app/mozilla.in

Chris, got time for this one? I'm a little lost in here

Revision history for this message
Anders Kaseorg (andersk) wrote :

Binary package hint: firefox-granparadiso

$ /usr/lib/firefox-granparadiso/firefox

run-mozilla.sh: Cannot execute /usr/lib/firefox-granparadiso/firefox-bin.

This directory has a firefox-granparadiso-bin but not a firefox-bin.

Revision history for this message
Hilario J. Montoliu (hjmf) (hmontoliu) wrote :

Thank you Anders Kaseorg for your report,

this is a known bug in the firefox launch script as shown in bug 117575

Changed in firefox-granparadiso:
assignee: nobody → mozilla-bugs
importance: Undecided → High
status: New → Confirmed
Revision history for this message
Hilario J. Montoliu (hjmf) (hmontoliu) wrote :

Not sure yet if it is the same problem.

I'm marking this report as mt-eval for further analisys

Changed in firefox:
status: Unknown → Confirmed
Revision history for this message
Alexander Sack (asac) wrote :

why is this a problem?

Revision history for this message
Anders Kaseorg (andersk) wrote :

Because if you allow firefox-granparadiso to set itself as the default browser (which it offers to do by default when it first gets run), the command that it sets is
  /usr/lib/firefox-granparadiso/firefox "%s"
and because this wrapper script is broken, http links from all other applications stop working!

Revision history for this message
In , Reed Loden (reed) wrote :

Note that Ubuntu has taken this patch downstream already (https://bugs.launchpad.net/ubuntu/+source/firefox/+bug/117575).

Revision history for this message
In , Reed Loden (reed) wrote :

Thanks for the patch!

Checking in browser/app/mozilla.in;
/cvsroot/mozilla/browser/app/mozilla.in,v <-- mozilla.in
new revision: 1.16; previous revision: 1.15
done

Changed in firefox:
status: Confirmed → Fix Released
Revision history for this message
In , Dveditz (dveditz) wrote :

Comment on attachment 268233
The same patch but against ./browser/app/mozilla.in

approved for 1.8.1.10, a=dveditz for release-drivers

Revision history for this message
In , Reed Loden (reed) wrote :

MOZILLA_1_8_BRANCH:

Checking in browser/app/mozilla.in;
/cvsroot/mozilla/browser/app/mozilla.in,v <-- mozilla.in
new revision: 1.12.8.2; previous revision: 1.12.8.1
done

Revision history for this message
In , Abillings (abillings) wrote :

Verified for branch in Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.1.10) Gecko/2007111504 Firefox/2.0.0.10 (the RC1 for 2.0.0.10). I can reproduce this with 2.0.0.9 but not the 2.0.0.10 build.

Revision history for this message
In , Reed Loden (reed) wrote :

Created attachment 295488
patch mail, sunbird, and xulrunner, too!

asac noticed that this same change wasn't made to the three other mozilla.in files in the tree.

Revision history for this message
In , Reed Loden (reed) wrote :

This is a blocker for Linux distros.

Changed in firefox:
status: Fix Released → Confirmed
Revision history for this message
In , Beltzner (beltzner) wrote :

Comment on attachment 295488
patch mail, sunbird, and xulrunner, too!

a=beltzner for 1.9

Revision history for this message
In , Reed Loden (reed) wrote :

Checking in mail/app/mozilla.in;
/cvsroot/mozilla/mail/app/mozilla.in,v <-- mozilla.in
new revision: 1.8; previous revision: 1.7
done
Checking in calendar/sunbird/app/mozilla.in;
/cvsroot/mozilla/calendar/sunbird/app/mozilla.in,v <-- mozilla.in
new revision: 1.11; previous revision: 1.10
done
Checking in xulrunner/app/mozilla.in;
/cvsroot/mozilla/xulrunner/app/mozilla.in,v <-- mozilla.in
new revision: 1.4; previous revision: 1.3
done

Revision history for this message
In , Kairo-kairo (kairo-kairo) wrote :
Revision history for this message
In , Reed Loden (reed) wrote :

(In reply to comment #15)
> Shouldn't have http://mxr.mozilla.org/mozilla/source/suite/app/seamonkey.in
> been patched as well?

It wasn't named mozilla.in, so I didn't find it. I blame SeaMonkey. :)

Revision history for this message
In , Reed Loden (reed) wrote :

Created attachment 295904
SeaMonkey patch

Just going to carry over r=cls since it's really just the same file as the others but in a different place.

KaiRo, can I get branch approval for this, too?

Revision history for this message
In , Reed Loden (reed) wrote :

Checking in suite/app/seamonkey.in;
/cvsroot/mozilla/suite/app/seamonkey.in,v <-- seamonkey.in
new revision: 1.3; previous revision: 1.2
done

Changed in firefox:
status: Confirmed → Fix Released
Revision history for this message
In , Kairo-kairo (kairo-kairo) wrote :

Comment on attachment 295904
SeaMonkey patch

a-sm1.1.8=me for that SeaMonkey patch - but note that it's at mozilla/xpfe/bootstrap/mozilla.in on branch.

Revision history for this message
In , Dveditz (dveditz) wrote :

Not blocking, but we'll look at approvals.

Revision history for this message
In , Reed Loden (reed) wrote :

Comment on attachment 295488
patch mail, sunbird, and xulrunner, too!

Bug 411710 filed to track landing this on the 1.8 branch, as per request from drivers.

Revision history for this message
In , Reed Loden (reed) wrote :

Checking in xpfe/bootstrap/mozilla.in;
/cvsroot/mozilla/xpfe/bootstrap/Attic/mozilla.in,v <-- mozilla.in
new revision: 1.8.8.8; previous revision: 1.8.8.7
done

Revision history for this message
Wouter Stomp (wouterstomp-deactivatedaccount) wrote :

Closing this, as firefox-granparadiso is no longer in use.

Changed in firefox-granparadiso:
status: Confirmed → Fix Released
Changed in firefox:
importance: Unknown → High
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.