[Github-comments] Re: [geany/geany] Help command should check browser tool's return value (Issue #3468)

2024-04-21 Thread Enrico Tröger via Github-comments
Closed #3468 as completed.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/3468#event-12550837837
You are receiving this because you are subscribed to this thread.

Message ID: 

[Github-comments] Re: [geany/geany] Help command should check browser tool's return value (Issue #3468)

2024-04-05 Thread Nick Treleaven via Github-comments
Reopened #3468.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/3468#event-12372656110
You are receiving this because you are subscribed to this thread.

Message ID: 

[Github-comments] Re: [geany/geany] Help command should check browser tool's return value (Issue #3468)

2024-04-05 Thread Nick Treleaven via Github-comments
Closed #3468 as completed.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/3468#event-12372639390
You are receiving this because you are subscribed to this thread.

Message ID: 

[Github-comments] Re: [geany/geany] Help command should check browser tool's return value (Issue #3468)

2024-04-05 Thread Nick Treleaven via Github-comments
@eht16 Yep, works now (at least on latest Linux Mint anyway) - thanks! Sorry 
for the late reply.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/3468#issuecomment-2039586529
You are receiving this because you are subscribed to this thread.

Message ID: 

[Github-comments] Re: [geany/geany] Help command should check browser tool's return value (Issue #3468)

2023-10-15 Thread Enrico Tröger via Github-comments
I think we can close this now since #3178 is merged.
@ntrel when use start current GIT master Geany with a fresh config, will it use 
the correct browser?

FTR, the "sensible-browser" default is patched in by Debian (and Ubuntu 
inherits this change). As pointed out earlier, Geany had the default "firefox" 
but now it uses GTK to find the system's default browser.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/3468#issuecomment-1763341750
You are receiving this because you are subscribed to this thread.

Message ID: 

[Github-comments] Re: [geany/geany] Help command should check browser tool's return value (Issue #3468)

2023-04-26 Thread elextr via Github-comments
Ahhh, I figured it out, when I ran the wrong version of geany (see #3451) 
specifying this config it probably wrote the LM system defaults. Nothing a 
quick delete won't fix :-).  So yes its Ubuntu that sets those values for 
terminal and browser, but thats totally allowable for distros to do that, they 
may not provide xterm or firefox by default.

@ntrel I totally agree that results should be checked, but I'm not sure what 
Geany can do if it detects failure (except write some message to debug).

Possibly one of the solutions from #2444 or #3178 might be better than fixed 
values, security oriented distros, lightweight distros, libre oriented distros, 
all commonly change the browser (even if its a de-trademarked firefox, its name 
changes).  I wouldn't worry if the solution is Linux specific, windows and mac 
are `#ifdef`ed anyway.

And users (me for eg) often prefer different browsers.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/3468#issuecomment-1523201927
You are receiving this because you are subscribed to this thread.

Message ID: 

[Github-comments] Re: [geany/geany] Help command should check browser tool's return value (Issue #3468)

2023-04-26 Thread elextr via Github-comments
> For a Geany built from git on LM I get "sensible-browser" in the preference. 
> So its probably not Ubuntu, somehow Geany build selects that value, but I ran 
> out of time to find how/where.

Hmmm, no actual solution, but I ran the built from git Geany with a 
non-existant config dir, and it switched the settings back to usual defaults:

`x-terminal-emulator` -> `xterm`
`sensible-browser` -> `firefox`

and now `Help` runs firefox but does not work because I didn't build the local 
doco so its looking for `https://www.geany.org/manual/1.39/index.html` which of 
course does not exist.  @ntrel Is that your problem?

As to how the setting values above got into the local config 
`~/git-geany/config` from the default config `~/.config/geany` I don't know.  
Maybe a GP plugin does something wrong, will check later.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/3468#issuecomment-1523056271
You are receiving this because you are subscribed to this thread.

Message ID: 

[Github-comments] Re: [geany/geany] Help command should check browser tool's return value (Issue #3468)

2023-04-26 Thread elextr via Github-comments
Note #2444 #3178 as well.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/3468#issuecomment-1522956888
You are receiving this because you are subscribed to this thread.

Message ID: 

[Github-comments] Re: [geany/geany] Help command should check browser tool's return value (Issue #3468)

2023-04-25 Thread elextr via Github-comments
> Hmm just realized Geany defaults to Firefox, Ubuntu must have overridden it 
> (and Mint is using the Ubuntu package).

For a Geany built from git on LM I get "sensible-browser" in the preference.  
So its probably not Ubuntu, somehow Geany build selects that value, but I ran 
out of time to find how/where.

But help does work for me (opening in the preferred browser, chrome, selected 
in the LM desktop "Preferred applications" dialog).

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/3468#issuecomment-1522554359
You are receiving this because you are subscribed to this thread.

Message ID: