Bug#537695: Patch uses "sensible-browser"

[Sandro Tosi]

hello,

On Sun, Mar 28, 2010 at 11:00:24PM +0200, Daniel Hahler wrote:
> Here's the contents of debian/patches/003_use_sensible-browser.patch -
> it appears to be using sensible-browser.
>
>
> Author: Frank S. Thomas <frank at thomas-alfeld.de>
>
> Description: use sensible-browser only, because upstream's attempt to
> start a browser fails miserably.
> Disclaimer: This patch is Debian-specific. Don't use it for non Debian
> packages.
> The program "sensible-browser" is part of the essential package debianutils.
>
> --- boinc.orig/clientgui/hyperlink.cpp
> +++ boinc/clientgui/hyperlink.cpp
> @@ -171,7 +171,8 @@
>  }
>
>  void wxHyperLink::ExecuteLink (const wxString &strLink) {
> -    if (!wxLaunchDefaultBrowser(strLink)) {
> +    wxString cmd = wxT("sensible-browser ") + strLink;
> +    if (!::wxExecute(cmd, wxEXEC_ASYNC)) {
>          wxString strDialogTitle = wxEmptyString;
>          wxString strDialogMessage = wxEmptyString;

Shouldn't this bug be closed then? boinc-manager is using the right
way to spawn a browser, s-b. The only other thing the maint could do
is to document it in README.Debian or so, but I consider this patch
already working and perfectly fitting the debian env (I verified and
it works just fine)

Regards,
-- 
Sandro Tosi (aka morph, morpheus, matrixhasu)
My website: http://matrixhasu.altervista.org/
Me at Debian: http://wiki.debian.org/SandroTosi