threads / patch / 7137

patchgit-gui: '/dev/null' on Windows is named 'nul'

Subject: [PATCH] git-gui: '/dev/null' on Windows is named 'nul'

## tl;dr

2 messages between Mar 7, 2007 and Mar 7, 2007. Diffs are folded; open one to read it.

replies: 1people: 2as markdown or json

Johannes Sixt· Mar 7, 2007, 16:26 UTC · lore
From: Johannes Sixt <johannes.sixt@telecom.at>

Making (or amending) an initial commit on Windows fails because redirecting something to '/dev/null' must fail - this device does not exist. Redirect to 'nul' instead.

Signed-off-by: Johannes Sixt <johannes.sixt@telecom.at>
---
 I've noticed this while using the MinGW port, but I do not know whether
 this is needed or even works on Cygwin.
 (Now lets see whether my MUA wraps long lines...)
 -- Hannes
 git-gui/git-gui.sh |    6 +++++-
 1 files changed, 5 insertions(+), 1 deletions(-)
Show changes to git-gui/git-gui.sh +5 −1
diff --git a/git-gui/git-gui.sh b/git-gui/git-gui.sh
index f84ba33..971d5fe 100755
--- a/git-gui/git-gui.sh
+++ b/git-gui/git-gui.sh
@@ -1288,7 +1288,11 @@ proc commit_committree {fd_wt curHEAD msg} {
 		}
 	} else {
 		# git commit-tree writes to stderr during initial commit.
-		lappend cmd 2>/dev/null
+		if {[is_Windows]} {
+			lappend cmd 2>nul
+		} else {
+			lappend cmd 2>/dev/null
+		}
 	}
 	lappend cmd <$msg_p
 	if {[catch {set cmt_id [eval exec $cmd]} err]} {
-- 
1.5.0.2.279.g4808
Shawn O. Pearce· Mar 7, 2007, 16:57 UTC · re: Johannes Sixt · lore

Re: [PATCH] git-gui: '/dev/null' on Windows is named 'nul'

Johannes Sixt <J.Sixt@eudaptics.com> wrote:
Show 10 quoted lines
> From: Johannes Sixt <johannes.sixt@telecom.at>
> 
> Making (or amending) an initial commit on Windows fails because
> redirecting something to '/dev/null' must fail - this device does not exist.
> Redirect to 'nul' instead.
> 
> Signed-off-by: Johannes Sixt <johannes.sixt@telecom.at>
> ---
>  I've noticed this while using the MinGW port, but I do not know whether
>  this is needed or even works on Cygwin.
I had previously fixed this in gitgui-0.6.3, which is now also
merged into `maint` in git.git.  It was fixed by just removing that
redirection block, in git 1.5.0 and later commit-tree doesn't print
anything on stderr when making an initial commit.
 
-- 
Shawn.

← back to recent threads