git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH] remove "nohup" from git-help--browse

From
Dmitry Potapov <dpotapov@gmail.com>
Date
Feb 8, 2008, 20:36 UTC
Message-ID
<1202502982-6822-1-git-send-email-dpotapov@gmail.com>
In-Reply-To
<7vhcgkm7yy.fsf@gitster.siamese.dyndns.org>

There is no good reason to run GUI browsers using "nohup". It does not solve any real problem but creates annoying "nohup.out" files in every directory where git help -w is run.

This patch removes "nohup" from git-help--browse.sh
Signed-off-by: Dmitry Potapov <dpotapov@gmail.com>
---
It seems that everyone is agree that there is no good reason to
keep "nohup" in git-help--browse. So here is the patch.
 git-help--browse.sh |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/git-help--browse.sh b/git-help--browse.sh
index 10b0a36..adc4d37 100755
--- a/git-help--browse.sh
+++ b/git-help--browse.sh
@@ -122,7 +122,7 @@ case "$browser" in
 	vers=$(expr "$($browser_path -version)" : '.* \([0-9][0-9]*\)\..*')
 	NEWTAB='-new-tab'
 	test "$vers" -lt 2 && NEWTAB=''
-	nohup "$browser_path" $NEWTAB $pages &
+	"$browser_path" $NEWTAB $pages &
 	;;
     konqueror)
 	case "$(basename "$browser_path")" in
@@ -136,7 +136,7 @@ case "$browser" in
 		eval "$browser_path" newTab $pages
 		;;
 	    *)
-	        nohup "$browser_path" $pages &
+	        "$browser_path" $pages &
 		;;
 	esac
 	;;
@@ -144,6 +144,6 @@ case "$browser" in
 	eval "$browser_path" $pages
 	;;
     dillo)
-	nohup "$browser_path" $pages &
+	"$browser_path" $pages &
 	;;
 esac
-- 
1.5.4
Previous: Dmitry PotapovNext: Christian Couder
Message 7 of 11 in “git help -w should not create nohup.out”
  1. git help -w should not create nohup.outDmitry Potapov, Feb 8, 2008
  2. Johannes SchindelinFeb 8, 2008
  3. Junio C HamanoFeb 8, 2008
  4. Christian CouderFeb 8, 2008
  5. Dmitry PotapovFeb 8, 2008
  6. Dmitry PotapovFeb 8, 2008
  7. remove "nohup" from git-help--browseDmitry Potapov, Feb 8, 2008
  8. Christian CouderFeb 9, 2008
  9. Dmitry PotapovFeb 9, 2008
  10. Junio C HamanoFeb 10, 2008
  11. Johannes SchindelinFeb 10, 2008

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.