Re: [Flightgear-devel] diff for browser change for mac os x to use
Help Help from the gui on mac os x is still broken. The help application is STILL set to netscape even after the options.cxx change Doesn't it mean that something resets it between the options parsing and the GUI help stuff processing? I'd rather see a patch that finds the offending code modifying it, if that's the case. BTW: there's an option to debug/trace every property read/write for a given property, sorry I can't say more right now as I am logging into my mail remotely at this moment w/o any graphics around to test the things. V. ___ Flightgear-devel mailing list Flightgear-devel@flightgear.org http://mail.flightgear.org/mailman/listinfo/flightgear-devel 2f585eeea02e2c79d7b1d8c4963bae2d
Re: [Flightgear-devel] diff for browser change for mac os x to use safari
On things like Debian, this is also wrong because sensible-browser should be used instead. Is there some autoconf library function to discover the most likely browser on a system? Also the WIN32 section in src/GUI/gui_funcs.cxx with the 1024-long hardcoded buffers can be a crash trigger when the buffer overflows for somebody with a long enough path... V. ___ Flightgear-devel mailing list Flightgear-devel@flightgear.org http://mail.flightgear.org/mailman/listinfo/flightgear-devel 2f585eeea02e2c79d7b1d8c4963bae2d
Re: [Flightgear-devel] diff for browser change for mac os x to use safari
In the macflightgear binary I've been passing --browser-app=open to fgfs. This opens the url in the default browser. For me that's Firefox.Your patch could be changed to something like this:Index: FlightGear/src/Main/options .cxx===RCS file: /var/cvs/FlightGear-0.9/FlightGear/src/Main/options.cxx,vretrieving revision 1.73diff -u -r1.73 options.cxx --- FlightGear/src/Main/options.cxx 12 Oct 2005 08:55:58- 1.73+++ FlightGear/src/Main/options.cxx 12 Nov 2005 02:41:34 -@@ -172,7 +172,11 @@ fgSetString(/sim/control-mode, joystick); fgSetBool(/sim/auto-coordination, false);#if !defined(WIN32)+#ifdef __APPLE__+ fgSetString(/sim/startup/browser-app, open);+#else fgSetString(/sim/startup/browser-app, netscape); +#endif#else fgSetString(/sim/startup/browser-app, webrun.bat);#endifOn 11/12/05, Ima Sudonim [EMAIL PROTECTED] wrote:Arthur and James,Yes, I agree it's silly, but currently help was broken on my two mac systems (giving a log message that netscape couldn't be found),followed by a dialog that my browser was started.It's reallyimportant to me to have browser-based help work on 0.9.9... I forgotthat it was broken until now. (I know I had this working at one point, and thought I still did as I was building with a modifiedoptions.cxx. I was wrong). I promise to pay much more attention nexttime or at least be conscious when I try these thingsUnfortunately, my fix doesn't work. I think it should have been the following diff, but as neither diffis working, don't apply them to cvs please.I can't get this options.cxx change working on Mac OS X, I have touse the option --browser-app=open /Applications/Safari.app when starting flightgear.Any idea where else /sim/startup/browser-app is set or what (ifanything) is overwriting it?Never mind about the cvs update. It wouldn't break anything even ifapplied, because it didn't work anyway. 8-( It looks like the mac browser exec actually belongs in gui/gui_funcs.cxx. If I modify void helpCb (puObject *) (line 238) to read: string help_app = open /Applications/Safari.app ;//fgGetString (/sim/startup/browser-app);Safari starts up fine and opens help so this is the spot Arthur wouldneed to pop his code into, I guess.It would be nice if the returncode from the system call or ShellExecute could be checked before mkDialog is called.It's adding insult to injury when your browserhasn't started, but the dialog says it has 8-(I'm still a little confused as to where /sim/startup/browser-app isbeing set, as it doesn't seem to be in main/options.cxx as I expected. I will leave this to much wiser heads than my own.Sorry for thetrouble. Some days it doesn't pay to get out of bed in the morning...8-(I'll happily wait for arthur to fix it.Thanks to Arthur and James and all the rest... ImaIndex: FlightGear/src/Main/options.cxx===RCS file: /var/cvs/FlightGear-0.9/FlightGear/src/Main/options.cxx,vretrieving revision 1.73diff -u -r1.73 options.cxx--- FlightGear/src/Main/options.cxx 12 Oct 2005 08:55:58-1.73+++ FlightGear/src/Main/options.cxx 12 Nov 2005 02:41:34 -@@ -172,7 +172,11 @@fgSetString(/sim/control-mode, joystick); fgSetBool(/sim/auto-coordination, false);#if !defined(WIN32)+#ifdef __APPLE__+fgSetString(/sim/startup/browser-app, open /Applications/Safari.app);+#else fgSetString(/sim/startup/browser-app, netscape);+#endif#elsefgSetString(/sim/startup/browser-app, webrun.bat);#endif___ Flightgear-devel mailing listFlightgear-devel@flightgear.orghttp://mail.flightgear.org/mailman/listinfo/flightgear-devel 2f585eeea02e2c79d7b1d8c4963bae2d-- Arthur/- http://sourceforge.net/users/artooro/- http://artooro.blogspot.com ___ Flightgear-devel mailing list Flightgear-devel@flightgear.org http://mail.flightgear.org/mailman/listinfo/flightgear-devel 2f585eeea02e2c79d7b1d8c4963bae2d
Re: [Flightgear-devel] diff for browser change for mac os x to use safari
OK, is there a way to get sensible-browser at compile time? Is this a link know to the OS or something? is it callable or does it need to be read on Debian somewhere? It's a callable standard script on debian. It tries various intelligent decisions to guess what browser to run. It is pretty debian-dependent though (e.g., it relies on the fact that the browser packages register themselves via the alternatives mechanism available on debian). On Debian, however, it is the preferred thing to call in order to be user-friendly. My point is that fgfs doesn't have the means to know the target system defaults and thus the hardwired default is probably best left as a toplevel configure option. V. ___ Flightgear-devel mailing list Flightgear-devel@flightgear.org http://mail.flightgear.org/mailman/listinfo/flightgear-devel 2f585eeea02e2c79d7b1d8c4963bae2d
Re: [Flightgear-devel] diff for browser change for mac os x to use
Arthur Wiebe wrote: [...] SSBhZ3JlZSB3aXRoIEphbWVzLiBJJ3ZlIGJlZW4gdXNpbmcgdGhlIC0tb3Blbi13aXRoIG9wdGlv As Arthur obviously decided to ignore private mail on this topic I think I'd post my comment on the list: Could we please agree not to post encoded EMail on this list !?! Thanks, Martin. -- Unix _IS_ user friendly - it's just selective about who its friends are ! -- ___ Flightgear-devel mailing list Flightgear-devel@flightgear.org http://mail.flightgear.org/mailman/listinfo/flightgear-devel 2f585eeea02e2c79d7b1d8c4963bae2d
Re: [Flightgear-devel] diff for browser change for mac os x to use
Sorry Martin. I never received an email from you. It may have gone into spam.But sure. No encoded email from me anymore.On 11/12/05, Martin Spott [EMAIL PROTECTED] wrote:Arthur Wiebe wrote: [...] SSBhZ3JlZSB3aXRoIEphbWVzLiBJJ3ZlIGJlZW4gdXNpbmcgdGhlIC0tb3Blbi13aXRoIG9wdGlvAs Arthur obviously decided to ignore private mail on this topic Ithink I'd post my comment on the list:Could we please agree not to post encoded EMail on this list !?! Thanks,Martin.-- Unix _IS_ user friendly - it's just selective about who its friends are !--___ Flightgear-devel mailing listFlightgear-devel@flightgear.orghttp://mail.flightgear.org/mailman/listinfo/flightgear-devel 2f585eeea02e2c79d7b1d8c4963bae2d-- Arthur/- http://sourceforge.net/users/artooro/- http://artooro.blogspot.com ___ Flightgear-devel mailing list Flightgear-devel@flightgear.org http://mail.flightgear.org/mailman/listinfo/flightgear-devel 2f585eeea02e2c79d7b1d8c4963bae2d
Re: [Flightgear-devel] diff for browser change for mac os x to use safari
On 12 Nov 2005, at 00:58, Ima Sudonim wrote:With this change, FlightGear on Mac OS X launches the mac os x Safari browser instead of netscape (w/o this change, the browser won't launch without netscape installed, and netscape isn't one of the installed mac os x browsers). This approach seems silly - I've got Core Foundation code to launch a URL string via LaunchServices - using whatever browser the user has selected in the system. Hard-coding a browser is always going to annoy people.I'll post the patch once I dig out the code.HHJames___ Flightgear-devel mailing list Flightgear-devel@flightgear.org http://mail.flightgear.org/mailman/listinfo/flightgear-devel 2f585eeea02e2c79d7b1d8c4963bae2d
Re: [Flightgear-devel] diff for browser change for mac os x to use safari
I agree with James. I've been using the --open-with option for sometime in the mac builds which is set to open. This opens the url is whatever is the default browser.On 11/12/05, James Turner [EMAIL PROTECTED] wrote: On 12 Nov 2005, at 00:58, Ima Sudonim wrote: With this change, FlightGear on Mac OS X launches the mac os x Safari browser instead of netscape (w/o this change, the browser won't launch without netscape installed, and netscape isn't one of the installed mac os x browsers). This approach seems silly - I've got Core Foundation code to launch a URL string via LaunchServices - using whatever browser the user has selected in the system. Hard-coding a browser is always going to annoy people. I'll post the patch once I dig out the code.HHJames ___Flightgear-devel mailing listFlightgear-devel@flightgear.org http://mail.flightgear.org/mailman/listinfo/flightgear-devel 2f585eeea02e2c79d7b1d8c4963bae2d-- Arthur/- http://sourceforge.net/users/artooro/- http://artooro.blogspot.com ___ Flightgear-devel mailing list Flightgear-devel@flightgear.org http://mail.flightgear.org/mailman/listinfo/flightgear-devel 2f585eeea02e2c79d7b1d8c4963bae2d
