The current code does not actually require this since the documentation URL should always pass through unmangled, but future users of start_browser() (as in the patch to follow) might require this. This patch has been sponsored by Novartis. Signed-off-by: Petr Baudis <pasky@xxxxxxx> --- git-gui/git-gui.sh | 6 +++++- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/git-gui/git-gui.sh b/git-gui/git-gui.sh index fc67eb8..4d2d600 100755 --- a/git-gui/git-gui.sh +++ b/git-gui/git-gui.sh @@ -2360,7 +2360,11 @@ if {[file isfile $doc_path]} { } proc start_browser {url} { - git "web--browse" $url + # We use --stdin here since passing URLs (especially with query + # strings) does insane things in MSysGit. + set fd [git_write "web--browse" "--stdin"] + puts $fd $url + close $fd } .mbar.help add command -label [mc "Online Documentation"] \ -- tg: (6e32399..) t/git-gui/web-browse-stdin (depends on: t/git-gui/web-browse t/web--browse/stdin) -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html