threads / patch / 12896

patchSilence cpio's "N blocks" output when cloning locally

Subject: [PATCH] Silence cpio's "N blocks" output when cloning locally

## tl;dr

6 messages between Mar 28, 2008 and Mar 31, 2008. Diffs are folded; open one to read it.

replies: 5people: 5as markdown or json

Bryan Donlan· Mar 28, 2008, 06:19 UTC · lore

Pass --quiet to cpio in git-clone to hide the (confusing) "0 blocks" message. For compatibility with operating systems which might not support GNUisms, the presence of --quiet is probed for by grepping cpio's --help output.

Signed-off-by: Bryan Donlan <bdonlan@fushizen.net>
---
 git-clone.sh |    6 +++++-
 1 files changed, 5 insertions(+), 1 deletions(-)
Show changes to git-clone.sh +5 −1
diff --git a/git-clone.sh b/git-clone.sh
index e981122..2636159 100755
--- a/git-clone.sh
+++ b/git-clone.sh
@@ -310,6 +310,9 @@ yes)
 		mkdir -p "$GIT_DIR/objects/info"
 		echo "$repo/objects" >>"$GIT_DIR/objects/info/alternates"
 	else
+		cpio_quiet_flag=""
+		cpio --help 2>&1 | grep -- --quiet >/dev/null && \
+			cpio_quiet_flag=--quiet
 		l= &&
 		if test "$use_local_hardlink" = yes
 		then
@@ -330,7 +333,8 @@ yes)
 			fi
 		fi &&
 		cd "$repo" &&
-		find objects -depth -print | cpio -pumd$l "$GIT_DIR/" || exit 1
+		find objects -depth -print | cpio $cpio_quiet_flag -pumd$l "$GIT_DIR/" || \
+			exit 1
 	fi
 	git-ls-remote "$repo" >"$GIT_DIR/CLONE_HEAD" || exit 1
 	;;

-- 
1.5.4.3
Johannes Schindelin· Mar 28, 2008, 13:01 UTC · re: Bryan Donlan · lore

Re: [PATCH] Silence cpio's "N blocks" output when cloning locally

Hi,
On Fri, 28 Mar 2008, Bryan Donlan wrote:
> Pass --quiet to cpio in git-clone to hide the (confusing) "0 blocks" 
> message. For compatibility with operating systems which might not 
> support GNUisms, the presence of --quiet is probed for by grepping 
> cpio's --help output.

We have a builtin clone which is almost ready for 'next', and I think that this issue would be resolved there anyway.

Ciao, Dscho

Wincent Colaiuta· Mar 28, 2008, 16:12 UTC · re: Johannes Schindelin · lore

Re: [PATCH] Silence cpio's "N blocks" output when cloning locally

El 28/3/2008, a las 14:01, Johannes Schindelin escribió:
Show 10 quoted lines
> On Fri, 28 Mar 2008, Bryan Donlan wrote:
>
>> Pass --quiet to cpio in git-clone to hide the (confusing) "0 blocks"
>> message. For compatibility with operating systems which might not
>> support GNUisms, the presence of --quiet is probed for by grepping
>> cpio's --help output.
>
> We have a builtin clone which is almost ready for 'next', and I  
> think that
> this issue would be resolved there anyway.

In any case, I'd rather see the cpio check at configure-time rather than run-time.

Cheers, Wincent

Johannes Schindelin· Mar 28, 2008, 16:29 UTC · re: Wincent Colaiuta · lore

Re: [PATCH] Silence cpio's "N blocks" output when cloning locally

Hi,
On Fri, 28 Mar 2008, Wincent Colaiuta wrote:
Show 14 quoted lines
> El 28/3/2008, a las 14:01, Johannes Schindelin escribió:
> 
> >On Fri, 28 Mar 2008, Bryan Donlan wrote:
> >
> > >Pass --quiet to cpio in git-clone to hide the (confusing) "0 blocks" 
> > >message. For compatibility with operating systems which might not 
> > >support GNUisms, the presence of --quiet is probed for by grepping 
> > >cpio's --help output.
> >
> >We have a builtin clone which is almost ready for 'next', and I think 
> >that this issue would be resolved there anyway.
> 
> In any case, I'd rather see the cpio check at configure-time rather than 
> run-time.

That would have to be done via ./configure, missing out those people who do not run ./configure, but just make.

Ciao, Dscho

Bryan Donlan· Mar 28, 2008, 18:59 UTC · re: Johannes Schindelin · lore

Re: [PATCH] Silence cpio's "N blocks" output when cloning locally

On Fri, Mar 28, 2008 at 12:29 PM, Johannes Schindelin <Johannes.Schindelin@gmx.de> wrote:

Show 22 quoted lines
> Hi,
>
>
>  On Fri, 28 Mar 2008, Wincent Colaiuta wrote:
>
>  > El 28/3/2008, a las 14:01, Johannes Schindelin escribió:
>  >
>  > >On Fri, 28 Mar 2008, Bryan Donlan wrote:
>  > >
>  > > >Pass --quiet to cpio in git-clone to hide the (confusing) "0 blocks"
>  > > >message. For compatibility with operating systems which might not
>  > > >support GNUisms, the presence of --quiet is probed for by grepping
>  > > >cpio's --help output.
>  > >
>  > >We have a builtin clone which is almost ready for 'next', and I think
>  > >that this issue would be resolved there anyway.
>  >
>  > In any case, I'd rather see the cpio check at configure-time rather than
>  > run-time.
>
>  That would have to be done via ./configure, missing out those people who
>  do not run ./configure, but just make.

Some distributions don't run configure either, including debian (and by extension, likely ubuntu). That said, if git-clone's going builtin anyway, we might as well wait for that :)

Junio C Hamano· Mar 31, 2008, 00:19 UTC · re: Johannes Schindelin · lore

Re: [PATCH] Silence cpio's "N blocks" output when cloning locally

Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
Show 19 quoted lines
> On Fri, 28 Mar 2008, Wincent Colaiuta wrote:
>
>> El 28/3/2008, a las 14:01, Johannes Schindelin escribió:
>> 
>> >On Fri, 28 Mar 2008, Bryan Donlan wrote:
>> >
>> > >Pass --quiet to cpio in git-clone to hide the (confusing) "0 blocks" 
>> > >message. For compatibility with operating systems which might not 
>> > >support GNUisms, the presence of --quiet is probed for by grepping 
>> > >cpio's --help output.
>> >
>> >We have a builtin clone which is almost ready for 'next', and I think 
>> >that this issue would be resolved there anyway.
>> 
>> In any case, I'd rather see the cpio check at configure-time rather than 
>> run-time.
>
> That would have to be done via ./configure, missing out those people who 
> do not run ./configure, but just make.

Runtime is just fine for this case. Unlike other commands, clone is not something you run all the time. Anything more elaborate is just overengineered.

← back to recent threads