All of lore.kernel.org
 help / color / mirror / Atom feed
* CR codes from git commands
@ 2009-01-20 16:26 Brent Goodrick
  2009-01-20 17:08 ` Johannes Schindelin
  2009-01-22  4:47 ` Daniel Barkalow
  0 siblings, 2 replies; 24+ messages in thread
From: Brent Goodrick @ 2009-01-20 16:26 UTC (permalink / raw)
  To: git


Hi,

I am considering converting from CVS over to using git. I'm currently
using git version 1.5.6.5 on Debian Linux "testing". One of the first
things I ran into was having to set PAGER to "cat" to avoid the
problems when running git from anything other than a terminal.  The
second thing is that "git pull" (and possibly other commands) are
emitting ^M (octal 013) codes on output, possibly caused by the same
assumption as causes the problem that is fixed by setting PAGER to
"cat".  This is not a big deal on small repos, but on larger ones I
actually do want to see status line output (or be given some option to
see them), so that I can then run "tail -1lf" on the log file that is
written during a long "git pull" operation.

Is there some configuration option or some environment variable I can
set that tells git to stop treating every invocation as if it is
coming from a terminal?

You can reproduce this on Linux with the following script (look for
the CR codes on the final git pull at the end of the script):

-- cut below this line ---
#!/bin/sh

# I could have simply used "set -x" here but then I wouldn't see the
# redirection syntax like ">file1", so instead use a PrintRun
# function:
PrintRun ()
{
    echo "COMMAND: $*"
    eval "$*; exitcode=\$?"
    if [ $exitcode != 0 ]
    then
        echo "ERROR: Command failed: $*"
        exit 1
    fi
}

# Failed attempt at hacking around git insisting on using ^M codes on stderr:
# cat >/tmp/git_pager <<EOF
# sed 's%^% >> %g'
# # This doesn't work either since the output I want to filter is on stderr
# # tr '\013' '\012'
# EOF
# chmod a+x /tmp/git_pager
# GIT_PAGER=/tmp/git_pager; export GIT_PAGER

# Clear out the scratch areas:
PrintRun rm -rf /tmp/git_area1
PrintRun rm -rf /tmp/git_area2
# Populate the initial area:
PrintRun mkdir -p /tmp/git_area1
PrintRun cd /tmp/git_area1
PrintRun git init
PrintRun "echo a new file 1 >file1"
PrintRun "echo a new file 2 >file2"
PrintRun git add file1
PrintRun git add file2
PrintRun git status
PrintRun "git commit -m \"first commit in git_area1\""
PrintRun find .
# Clone from the first area into a second area and add a file there:
PrintRun rm -rf /tmp/git_area2
PrintRun cd /tmp
PrintRun git clone /tmp/git_area1 git_area2
PrintRun cd /tmp/git_area2
PrintRun find .
PrintRun "echo a new file 3 >file3"
PrintRun git add file3
PrintRun git status
PrintRun "git commit -m \"second commit but in git_area2\""
PrintRun "git status; true" # true means don't fail inside PrintRun
PrintRun "git status; true" # true means don't fail inside PrintRun
# Now attempt to somehow refresh (what is the "git" word for "cvs update"?) into the first area:
PrintRun cd /tmp/git_area1
PrintRun "git status; true" # true means don't fail inside PrintRun
PrintRun "git diff; true" # true means don't fail inside PrintRun
# PrintRun "git pull /tmp/git_area2 master 2>&1"
# PrintRun "git pull /tmp/git_area2 master 2>&1 | tr '\013' '\012'"
PrintRun git pull /tmp/git_area2 master
-- cut above this line ---


Attempts at hacking around the problem: Redirecting stderr output from
git and then manually translating CR codes into LF codes yeilds the
following output (but I can't do this in practice and, no, I can't use
aliases in Bourne scripts (Bash/KSH yes, Bourne no)):

git> COMMAND: git pull /tmp/git_area2 master 2>&1
git> remote: Counting objects: 4, done.        
git> remote: Compressing objects:  50% (1/2)           
git>  --> remote: Compressing objects: 100% (2/2)           
git>  --> remote: Compressing objects: 100% (2/2        )Unpacking objects:  33% (1/3)   
git>  --> Unpacking objects:  66% (2/3)   
git>  --> Unpacking objects: 100% (3/3)   
git>  --> Unpacking objects: 100% (3/3), done.
git> remote: , done.        
git> remote: Total 3 (delta 0), reused 0 (delta 0)        
git> From /tmp/git_area2
git>  * branch            master     -> FETCH_HEAD
git> Updating b2f942d..4f9ba90
git> Fast forward
git>  file3 |    1 +
git>  1 files changed, 1 insertions(+), 0 deletions(-)
git>  create mode 100644 file3

Trying to automatically filter this with redirection and use of tr
fails to show the progress output completely which is a non-option
either:

git> COMMAND: git pull /tmp/git_area2 master 2>&1 | tr '\013' '\012'
git> From /tmp/git_area2
git>  * branch            master     -> FETCH_HEAD
git> Updating 49b1897..bb5f57c
git> Fast forward
git>  file3 |    1 +
git>  1 files changed, 1 insertions(+), 0 deletions(-)
git>  create mode 100644 file3

Thanks,
bgoodr

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: CR codes from git commands
  2009-01-20 16:26 CR codes from git commands Brent Goodrick
@ 2009-01-20 17:08 ` Johannes Schindelin
       [not found]   ` <18806.44057.477379.215492@hungover.brentg.com>
  2009-01-22  4:47 ` Daniel Barkalow
  1 sibling, 1 reply; 24+ messages in thread
From: Johannes Schindelin @ 2009-01-20 17:08 UTC (permalink / raw)
  To: Brent Goodrick; +Cc: git

Hi,

On Tue, 20 Jan 2009, Brent Goodrick wrote:

> I am considering converting from CVS over to using git. I'm currently
> using git version 1.5.6.5 on Debian Linux "testing".

First of all, 1.5.6.5 is from last August, so chances are that the 
behavior you complain about was fixed in the meantime.  We're at 1.6.1 at 
the moment.

> One of the first things I ran into was having to set PAGER to "cat" to 
> avoid the problems when running git from anything other than a terminal.  
> The second thing is that "git pull" (and possibly other commands) are 
> emitting ^M (octal 013) codes on output, possibly caused by the same 
> assumption as causes the problem that is fixed by setting PAGER to 
> "cat".

The only place I can think about where a CR is output is when showing the 
progress of downloading. 

Usually, our code checks if stdout is a tty, and does not show progress.

As a work-around, piping into cat should work, though.

Ciao,
Dscho

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: CR codes from git commands
       [not found]     ` <alpine.DEB.1.00.0901210930370.7929@racer>
@ 2009-01-21 14:42       ` Brent Goodrick
  2009-01-21 15:38         ` Johannes Schindelin
  0 siblings, 1 reply; 24+ messages in thread
From: Brent Goodrick @ 2009-01-21 14:42 UTC (permalink / raw)
  To: Johannes Schindelin; +Cc: git


Johannes Schindelin writes:
 > Hi,
 > 
 > is there a special reason you un-Cc:ed the list?

No, my mistake.  CCing the mailing list now. I was foiled into
thinking that the reply operation in my email client meant reply-all,
but instead it was set to reply-to-sender-only. Now fixed.

 > 
 > On Tue, 20 Jan 2009, Brent Goodrick wrote:
 > 
 > > Johannes Schindelin writes:
 > > 
 > >  > On Tue, 20 Jan 2009, Brent Goodrick wrote:
 > >  > 
 > >  > > I am considering converting from CVS over to using git. I'm 
 > >  > > currently using git version 1.5.6.5 on Debian Linux "testing".
 > >  > 
 > >  > First of all, 1.5.6.5 is from last August, so chances are that the 
 > >  > behavior you complain about was fixed in the meantime.  We're at 
 > >  > 1.6.1 at the moment.
 > > 
 > > Yes, I thought that was a good point, so I rebuilt from the source 
 > > tarball git version 1.6.1 and retried my script and got the same 
 > > behavior.
 > > 
 > >  > The only place I can think about where a CR is output is when showing 
 > >  > the progress of downloading.
 > >  > 
 > >  > Usually, our code checks if stdout is a tty, and does not show 
 > >  > progress.
 > >  >
 > >  > As a work-around, piping into cat should work, though.
 > > 
 > > Actually only redirecting stderr and then piping to cat seems to work, 
 > > e.g.,:
 > > 
 > >   get pull 2>&1 | cat
 > > 
 > > 
 > > I don't mind seeing the progress lines, I just don't want git to emit 
 > > any CR codes at all.
 > > 
 > > How about a config option to just turn off any tty-detecting logic 
 > > entirely, so that I don't have to wrap git with a lot of silly scripts 
 > > that set environment variables and redirect stdout and stderr and piped 
 > > into "cat"?
 > 
 > Nope, the config option is not needed.  This is just a Plain Old Bug which 
 > needs fixing, that's all.
 > 
 > Let's see what I can do today.

Thanks.  The fix should be to arrange it so that I can set something
so that a bare call such as (but just "git pull"):

  git pull

will emit no CR codes at all, ever, regardless of if there is a tty.
Even if it is an env var, but a config setting would be ok too.

Thanks,
Brent

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: CR codes from git commands
  2009-01-21 14:42       ` Brent Goodrick
@ 2009-01-21 15:38         ` Johannes Schindelin
  2009-01-22  4:00           ` Brent Goodrick
  0 siblings, 1 reply; 24+ messages in thread
From: Johannes Schindelin @ 2009-01-21 15:38 UTC (permalink / raw)
  To: Brent Goodrick; +Cc: git

Hi,

On Wed, 21 Jan 2009, Brent Goodrick wrote:

> Johannes Schindelin writes:
> 
>  > On Tue, 20 Jan 2009, Brent Goodrick wrote:
>  > 
>  > > Johannes Schindelin writes:
>  > > 
>  > >  > On Tue, 20 Jan 2009, Brent Goodrick wrote:
>  > >  > 
>  > >  > > I am considering converting from CVS over to using git. I'm 
>  > >  > > currently using git version 1.5.6.5 on Debian Linux "testing".
>  > >  > 
>  > >  > First of all, 1.5.6.5 is from last August, so chances are that 
>  > >  > the behavior you complain about was fixed in the meantime.  
>  > >  > We're at 1.6.1 at the moment.
>  > > 
>  > > Yes, I thought that was a good point, so I rebuilt from the source 
>  > > tarball git version 1.6.1 and retried my script and got the same 
>  > > behavior.
>  > > 
>  > >  > The only place I can think about where a CR is output is when 
>  > >  > showing the progress of downloading.
>  > >  > 
>  > >  > Usually, our code checks if stdout is a tty, and does not show 
>  > >  > progress.
>  > >  >
>  > >  > As a work-around, piping into cat should work, though.
>  > > 
>  > > Actually only redirecting stderr and then piping to cat seems to work, 
>  > > e.g.,:
>  > > 
>  > >   get pull 2>&1 | cat

In my test I performed one minute ago, "git pull | cat" did not show any 
CR.  Maybe it is the "git" instead of "get" :-)

>  > > I don't mind seeing the progress lines, I just don't want git to 
>  > > emit any CR codes at all.
>  > > 
>  > > How about a config option to just turn off any tty-detecting logic 
>  > > entirely, so that I don't have to wrap git with a lot of silly 
>  > > scripts that set environment variables and redirect stdout and 
>  > > stderr and piped into "cat"?
>  > 
>  > Nope, the config option is not needed.  This is just a Plain Old Bug 
>  > which needs fixing, that's all.
>  > 
>  > Let's see what I can do today.
> 
> Thanks.  The fix should be to arrange it so that I can set something so 
> that a bare call such as (but just "git pull"):
> 
>   git pull
> 
> will emit no CR codes at all, ever, regardless of if there is a tty. 
> Even if it is an env var, but a config setting would be ok too.

I would actually think that it should not be an env var or config setting 
if piping it to "cat" does what you want: if the output is a tty, I think 
it is safe to assume that you want to see the progress, and if you don't, 
"| cat" is not an unreasonable thing to ask for.

Ciao,
Dscho

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: CR codes from git commands
  2009-01-21 15:38         ` Johannes Schindelin
@ 2009-01-22  4:00           ` Brent Goodrick
  0 siblings, 0 replies; 24+ messages in thread
From: Brent Goodrick @ 2009-01-22  4:00 UTC (permalink / raw)
  To: Johannes Schindelin; +Cc: git

On Wed, Jan 21, 2009 at 7:38 AM, Johannes Schindelin
> In my test I performed one minute ago, "git pull | cat" did not show any
> CR.  Maybe it is the "git" instead of "get" :-)
<snip>
> > Thanks.  The fix should be to arrange it so that I can set something so
> > that a bare call such as (but just "git pull"):
> >
> >   git pull
> >
> > will emit no CR codes at all, ever, regardless of if there is a tty.
> > Even if it is an env var, but a config setting would be ok too.
>
> I would actually think that it should not be an env var or config setting
> if piping it to "cat" does what you want: if the output is a tty, I think
> it is safe to assume that you want to see the progress, and if you don't,
> "| cat" is not an unreasonable thing to ask for.

You might not be able to see those CR codes from your terminal,
because ... well ... its a terminal which will process them.  And if
you can't reproduce it in your environment, you'll have to duplicate
my environment, and at that is well beyond what I would ask anyone to
do. Thanks for your effort in looking into it.  If I get annoyed
enough with it, I'll debug the code myself and propose a patch (but
don't hold your breath because I'm still learning this complex tool).

Thanks!
Brent

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: CR codes from git commands
  2009-01-20 16:26 CR codes from git commands Brent Goodrick
  2009-01-20 17:08 ` Johannes Schindelin
@ 2009-01-22  4:47 ` Daniel Barkalow
  2009-01-22  7:34   ` Brent Goodrick
  1 sibling, 1 reply; 24+ messages in thread
From: Daniel Barkalow @ 2009-01-22  4:47 UTC (permalink / raw)
  To: Brent Goodrick; +Cc: git

On Tue, 20 Jan 2009, Brent Goodrick wrote:

> 
> Hi,
> 
> I am considering converting from CVS over to using git. I'm currently
> using git version 1.5.6.5 on Debian Linux "testing". One of the first
> things I ran into was having to set PAGER to "cat" to avoid the
> problems when running git from anything other than a terminal.  The
> second thing is that "git pull" (and possibly other commands) are
> emitting ^M (octal 013) codes on output, possibly caused by the same
> assumption as causes the problem that is fixed by setting PAGER to
> "cat".  This is not a big deal on small repos, but on larger ones I
> actually do want to see status line output (or be given some option to
> see them), so that I can then run "tail -1lf" on the log file that is
> written during a long "git pull" operation.

It's kind of unclear what you're trying to do here. I'm guessing that 
you're trying to run git with stdio directed to a /dev/tty device, where 
isatty() is true, but which doesn't interpret ASCII control characters as 
such. We're not detecting that you can't use a pager on this, and so you 
have to use PAGER=cat (which might not be a bad idea for things like 
"man", either). With some clues about the environment, we should be able 
to do something about this.

You're also trying to send the progress output to a log file that you can 
look at the end of (presumably in a more capable terminal). It should be 
possible (with an option) to get git to output progress info to a non-tty, 
and not use the CRs if the output isn't a tty.

Or do you want to use a tty that can't handle CRs, and get newlines 
instead of CRs? (If I'd git on the first computer I used, it would have 
printed the progress bar over and over in place and probably torn a hole 
in the paper, but I haven't used that one in over 20 years.)

	-Daniel
*This .sig left intentionally blank*

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: CR codes from git commands
  2009-01-22  4:47 ` Daniel Barkalow
@ 2009-01-22  7:34   ` Brent Goodrick
  2009-01-22  7:46     ` Daniel Barkalow
  0 siblings, 1 reply; 24+ messages in thread
From: Brent Goodrick @ 2009-01-22  7:34 UTC (permalink / raw)
  To: Daniel Barkalow; +Cc: git

On Wed, Jan 21, 2009 at 8:47 PM, Daniel Barkalow <barkalow@iabervon.org> wrote:
> It's kind of unclear what you're trying to do here. I'm guessing that
> you're trying to run git with stdio directed to a /dev/tty device, where
> isatty() is true, but which doesn't interpret ASCII control characters as
> such. We're not detecting that you can't use a pager on this, and so you
> have to use PAGER=cat (which might not be a bad idea for things like
> "man", either). With some clues about the environment, we should be able
> to do something about this.
>
> You're also trying to send the progress output to a log file that you can
> look at the end of (presumably in a more capable terminal). It should be
> possible (with an option) to get git to output progress info to a non-tty,
> and not use the CRs if the output isn't a tty.
>
> Or do you want to use a tty that can't handle CRs, and get newlines
> instead of CRs? (If I'd git on the first computer I used, it would have
> printed the progress bar over and over in place and probably torn a hole
> in the paper, but I haven't used that one in over 20 years.)

Hi Daniel,

Ideally, yes I would want no CR's but LF's instead (but others who do
not use my environment may actually like the way it is now, and I seek
not to disturb that use case).  I could live without the progress
lines (lines that print repeatedly over in one place on normal
terminals), but adding " 2>&1 | cat" to every command line just to get
the CR's to go away, is non-workable for me.

The environment I'm running git under is the Shell mode inside GNU
Emacs. I can't tell you what type of terminal it is, because I believe
that is defined deep in the guts of Emacs. Having read your reply
above, I'm now wondering whether this is an Emacs issue versus a git
issue. If it is an Emacs issue, then I am truly embarrassed for having
wasted everyones time with it.

Brent

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: CR codes from git commands
  2009-01-22  7:34   ` Brent Goodrick
@ 2009-01-22  7:46     ` Daniel Barkalow
  2009-01-22  8:04       ` Junio C Hamano
  0 siblings, 1 reply; 24+ messages in thread
From: Daniel Barkalow @ 2009-01-22  7:46 UTC (permalink / raw)
  To: Brent Goodrick; +Cc: git

On Wed, 21 Jan 2009, Brent Goodrick wrote:

> On Wed, Jan 21, 2009 at 8:47 PM, Daniel Barkalow <barkalow@iabervon.org> wrote:
> > It's kind of unclear what you're trying to do here. I'm guessing that
> > you're trying to run git with stdio directed to a /dev/tty device, where
> > isatty() is true, but which doesn't interpret ASCII control characters as
> > such. We're not detecting that you can't use a pager on this, and so you
> > have to use PAGER=cat (which might not be a bad idea for things like
> > "man", either). With some clues about the environment, we should be able
> > to do something about this.
> >
> > You're also trying to send the progress output to a log file that you can
> > look at the end of (presumably in a more capable terminal). It should be
> > possible (with an option) to get git to output progress info to a non-tty,
> > and not use the CRs if the output isn't a tty.
> >
> > Or do you want to use a tty that can't handle CRs, and get newlines
> > instead of CRs? (If I'd git on the first computer I used, it would have
> > printed the progress bar over and over in place and probably torn a hole
> > in the paper, but I haven't used that one in over 20 years.)
> 
> Hi Daniel,
> 
> Ideally, yes I would want no CR's but LF's instead (but others who do
> not use my environment may actually like the way it is now, and I seek
> not to disturb that use case).  I could live without the progress
> lines (lines that print repeatedly over in one place on normal
> terminals), but adding " 2>&1 | cat" to every command line just to get
> the CR's to go away, is non-workable for me.
> 
> The environment I'm running git under is the Shell mode inside GNU
> Emacs. I can't tell you what type of terminal it is, because I believe
> that is defined deep in the guts of Emacs. Having read your reply
> above, I'm now wondering whether this is an Emacs issue versus a git
> issue. If it is an Emacs issue, then I am truly embarrassed for having
> wasted everyones time with it.

The terminal type, at least in my version of Emacs, is "dumb", which ought 
to be sufficient to tell git that a pager isn't going to be useful is most 
cases (might be worthwhile to keep "git log" from eating all your memory, 
though), and that using CR to rewrite lines isn't going to work.

	-Daniel
*This .sig left intentionally blank*

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: CR codes from git commands
  2009-01-22  7:46     ` Daniel Barkalow
@ 2009-01-22  8:04       ` Junio C Hamano
  2009-01-22 10:04         ` Mike Ralphson
  0 siblings, 1 reply; 24+ messages in thread
From: Junio C Hamano @ 2009-01-22  8:04 UTC (permalink / raw)
  To: Daniel Barkalow; +Cc: Brent Goodrick, git

Daniel Barkalow <barkalow@iabervon.org> writes:

> The terminal type, at least in my version of Emacs, is "dumb", which ought 
> to be sufficient to tell git that a pager isn't going to be useful is most 
> cases (might be worthwhile to keep "git log" from eating all your memory, 
> though), and that using CR to rewrite lines isn't going to work.

I think we pay attention to "dumb" when deciding if pager is useful and if
we can do color, but I do not think we check anything beyond "is it a tty"
when deciding to show progress or not.  The only thing we do differently
for "dumb" terminal is if we use ANSI clear-to-eol escape sequence or fill
with a run of SPs to overwrite trailing part of a line, and we assume even
dumb terminals know how to do a carriage-return.

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: CR codes from git commands
  2009-01-22  8:04       ` Junio C Hamano
@ 2009-01-22 10:04         ` Mike Ralphson
  2009-01-22 16:13           ` Brent Goodrick
  0 siblings, 1 reply; 24+ messages in thread
From: Mike Ralphson @ 2009-01-22 10:04 UTC (permalink / raw)
  To: Brent Goodrick; +Cc: Daniel Barkalow, Junio C Hamano, git

>2009/1/22 Brent Goodrick <bgoodr@gmail.com>:
> The environment I'm running git under is the Shell mode inside GNU
> Emacs. I can't tell you what type of terminal it is, because I believe
> that is defined deep in the guts of Emacs. Having read your reply
> above, I'm now wondering whether this is an Emacs issue versus a git
> issue. If it is an Emacs issue, then I am truly embarrassed for having
> wasted everyones time with it.

2009/1/22 Junio C Hamano <gitster@pobox.com>:
> I think we pay attention to "dumb" when deciding if pager is useful and if
> we can do color, but I do not think we check anything beyond "is it a tty"
> when deciding to show progress or not.  The only thing we do differently
> for "dumb" terminal is if we use ANSI clear-to-eol escape sequence or fill
> with a run of SPs to overwrite trailing part of a line, and we assume even
> dumb terminals know how to do a carriage-return.

I think this earlier discussion is probably relevant... I'm guessing
though, $EDITOR is set correctly here 8-)

2008/12/17 Junio C Hamano <gitster@pobox.com>:
> Any semi-good emacs users (let alone hackers) export PAGER=cat to be used
> in compilation mode (and possibly shell mode), so this is not a problem in
> practice.
>
> I have something like this in my .emacs:
>
>    (setenv "PAGER" "cat")
>
> I suspect (I am just a user not a hacker) this will have bad interaction
> with emacs terminal emulation mode, but I do not use the mode, so it is
> enough for me.

Mike

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: CR codes from git commands
  2009-01-22 10:04         ` Mike Ralphson
@ 2009-01-22 16:13           ` Brent Goodrick
  2009-01-22 16:41             ` Mike Ralphson
  2009-01-22 16:44             ` Daniel Barkalow
  0 siblings, 2 replies; 24+ messages in thread
From: Brent Goodrick @ 2009-01-22 16:13 UTC (permalink / raw)
  To: Mike Ralphson; +Cc: Brent Goodrick, Daniel Barkalow, Junio C Hamano, git


Mike Ralphson writes:
 > >2009/1/22 Brent Goodrick <bgoodr@gmail.com>:
 > > The environment I'm running git under is the Shell mode inside GNU
 > > Emacs. I can't tell you what type of terminal it is, because I believe
 > > that is defined deep in the guts of Emacs. Having read your reply
 > > above, I'm now wondering whether this is an Emacs issue versus a git
 > > issue. If it is an Emacs issue, then I am truly embarrassed for having
 > > wasted everyones time with it.
 > 
 > 2009/1/22 Junio C Hamano <gitster@pobox.com>:
 > > I think we pay attention to "dumb" when deciding if pager is useful and if
 > > we can do color, but I do not think we check anything beyond "is it a tty"
 > > when deciding to show progress or not.  The only thing we do differently
 > > for "dumb" terminal is if we use ANSI clear-to-eol escape sequence or fill
 > > with a run of SPs to overwrite trailing part of a line, and we assume even
 > > dumb terminals know how to do a carriage-return.
 > 
 > I think this earlier discussion is probably relevant... I'm guessing
 > though, $EDITOR is set correctly here 8-)

I do have EDITOR set to a home-built version of gnuclient, and git
talks to Emacs by way of that gnuclient just fine when I'm not using the
-m "commit_message" git-commit option.

 > 
 > 2008/12/17 Junio C Hamano <gitster@pobox.com>:
 > > Any semi-good emacs users (let alone hackers) export PAGER=cat to be used
 > > in compilation mode (and possibly shell mode), so this is not a problem in
 > > practice.
 > >
 > > I have something like this in my .emacs:
 > >
 > >    (setenv "PAGER" "cat")
 > >
 > > I suspect (I am just a user not a hacker) this will have bad interaction
 > > with emacs terminal emulation mode, but I do not use the mode, so it is
 > > enough for me.

I have PAGER set to "cat" in the environment before I run Emacs for
the same reason.

Unfortunately, this morning when I rebooted and reloaded from scratch,
I am now unable to reproduce the CR codes output from "git pull" no
matter what I do. I even tried the older git installed on Debian Linux
"testing", and tried unsetting PAGER and GIT_PAGER, and saw the pager
prompts and the terminal escape sequence output as I expected to
(which is not the issue here).  I can't expect anyone else to help me
debug this problem further if I can't even reproduce it
anymore. Frustrating.

I do have automatic updates turned on, so perhaps something changed in
the termcap or how terminal I/O is being done outside of git in my
system.  Emacs would not have changed since I build Emacs from top of
trunk CVS, and it only uses local Elisp packages AFAIK.

I don't suppose git has any logic that emits the progress messages
based upon some estimate of amount of work it has to do, or has done,
does it?

Thanks,
Brent

P.S., for your reference, below is my evaluation script that
previously showed the CR code from git pull output. I even increased
the number of files added to the second repo up to 50 to see if the
quantity of files being pulled had any effect on the progress messages
output, but that didn't seem to have any effect. If anyone sees
anything bone-headed there, I'm all ears:

--- cut below this line --- 
#!/bin/sh
# -*-mode: Shell-script; indent-tabs-mode: nil; -*-

# I could have simply used "set -x" here but then I wouldn't see the
# redirection syntax like ">file1", so instead use a PrintRun
# function:
PrintRun ()
{
    echo "COMMAND: $*"
    eval "$*; exitcode=\$?"
    if [ $exitcode != 0 ]
    then
        echo "ERROR: Command failed: $*"
        exit 1
    fi
}

git_term_redirect=""
if [ "$USE_GIT_TERM_REDIRECT" = 1 ]
then
    git_term_redirect=" 2>&1 | cat"
    echo "Note: using git redirect on some git git commands: \"$git_term_redirect\""
fi

if [ "$USE_LOCALLY_BUILT_GIT" = 1 ]
then
    git_bin_dir="$HOME/git_from_source/install/bin"
    if [ -d "$git_bin_dir" ]
    then
        PATH="$HOME/git_from_source/install/bin:$PATH"; export PATH
    fi
fi

if [ "$SKIP_PAGER_HACK" = 1 ]
then
    unset PAGER
    echo "Note: setting PAGER to $PAGER"
else
    echo "Note: unsetting PAGER"
    PAGER=cat; export PAGER
fi

# Print out the git version as a double check on the above logic:
PrintRun git --version
# Clear out the scratch areas:
PrintRun rm -rf /tmp/git_area1
PrintRun rm -rf /tmp/git_area2
# Populate the initial area:
PrintRun mkdir -p /tmp/git_area1
PrintRun cd /tmp/git_area1
PrintRun git init
PrintRun "echo a new file 1 >file1"
PrintRun "echo a new file 2 >file2"
PrintRun git add file1
PrintRun git add file2
PrintRun git status
PrintRun "git commit -m \"first commit in git_area1\""
PrintRun find .
# Clone from the first area into a second area and add files there:
PrintRun rm -rf /tmp/git_area2
PrintRun cd /tmp
PrintRun git clone /tmp/git_area1 git_area2

PrintRun cd /tmp/git_area2
PrintRun find .
i=1
while [ $i -le 50 ]
do
    file="file_$i"
    echo "file==\"${file}\""
    PrintRun "echo a new file >$file"
    PrintRun git add $file
    PrintRun git status
    PrintRun "git commit -m \"committing new file $file but in git_area2\""
    #    PrintRun "git status; true" # true means don't fail inside PrintRun
    i=`expr $i + 1`
done

# Now attempt to pull the second repo changes back into into the first repo with a "git pull" operation:

PrintRun cd /tmp/git_area1
PrintRun "git status; true" # true means don't fail inside PrintRun
PrintRun "git diff; true" # true means don't fail inside PrintRun
if [ "$INJECT_TERM" != "" ]
then
    echo "Note: Exporting environment variable: TERM=\"$INJECT_TERM\""
    TERM="$INJECT_TERM"; export TERM
fi
PrintRun "git pull /tmp/git_area2 master $git_term_redirect"
# if [ "$STOP_AFTER_FIRST_GIT_PULL" = 1 ]
# then
#     echo "Note: Stopping after first git pull"
#     env | grep -i term
#     exit 0
# fi
PrintRun "git status; true" # true means don't fail inside PrintRun
PrintRun cat file_3
PrintRun "echo conflict1 >>file_3"
PrintRun git add file_3
PrintRun git status
PrintRun "git commit -m \"conflict1 added in git_area1\""

PrintRun cd /tmp/git_area2
PrintRun "echo conflict2 >>file_3"
PrintRun git add file_3
PrintRun git status
PrintRun "git commit -m \"conflict2 added in git_area2\""


PrintRun cd /tmp/git_area1
PrintRun "git status; true" # true means don't fail inside PrintRun
PrintRun "git diff; true" # true means don't fail inside PrintRun
# This git pull should show the conflict:
PrintRun "git pull /tmp/git_area2 master $git_term_redirect"
PrintRun cat file_3
PrintRun "echo conflict resolved > file_3"
# Running git commit now will fail:
### PrintRun "git commit -m \"conflict resolved\""
# Running git add on the file I just "resolved" by editing it directly above
PrintRun git add file_3
PrintRun "git status; true" # true means don't fail inside PrintRun
PrintRun "git commit -m \"conflict resolved\""
PrintRun "git log"
--- cut above this line --- 

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: CR codes from git commands
  2009-01-22 16:13           ` Brent Goodrick
@ 2009-01-22 16:41             ` Mike Ralphson
  2009-01-22 16:50               ` Johannes Schindelin
  2009-01-22 16:44             ` Daniel Barkalow
  1 sibling, 1 reply; 24+ messages in thread
From: Mike Ralphson @ 2009-01-22 16:41 UTC (permalink / raw)
  To: Brent Goodrick; +Cc: Daniel Barkalow, Junio C Hamano, git

2009/1/22 Brent Goodrick <bgoodr@gmail.com>:
> Mike Ralphson writes:
>  > I think this earlier discussion is probably relevant... I'm guessing
>  > though, $EDITOR is set correctly here 8-)
>
> I do have EDITOR set to a home-built version of gnuclient...

Sorry, I was being too subtle. My $EDITOR is set to vim, as god intended. 8-)

Mike

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: CR codes from git commands
  2009-01-22 16:13           ` Brent Goodrick
  2009-01-22 16:41             ` Mike Ralphson
@ 2009-01-22 16:44             ` Daniel Barkalow
  2009-01-22 16:52               ` Johannes Schindelin
  1 sibling, 1 reply; 24+ messages in thread
From: Daniel Barkalow @ 2009-01-22 16:44 UTC (permalink / raw)
  To: Brent Goodrick; +Cc: Mike Ralphson, Junio C Hamano, git

On Thu, 22 Jan 2009, Brent Goodrick wrote:

> Mike Ralphson writes:
>  > >2009/1/22 Brent Goodrick <bgoodr@gmail.com>:
>  > > The environment I'm running git under is the Shell mode inside GNU
>  > > Emacs. I can't tell you what type of terminal it is, because I believe
>  > > that is defined deep in the guts of Emacs. Having read your reply
>  > > above, I'm now wondering whether this is an Emacs issue versus a git
>  > > issue. If it is an Emacs issue, then I am truly embarrassed for having
>  > > wasted everyones time with it.
>  > 
>  > 2009/1/22 Junio C Hamano <gitster@pobox.com>:
>  > > I think we pay attention to "dumb" when deciding if pager is useful and if
>  > > we can do color, but I do not think we check anything beyond "is it a tty"
>  > > when deciding to show progress or not.  The only thing we do differently
>  > > for "dumb" terminal is if we use ANSI clear-to-eol escape sequence or fill
>  > > with a run of SPs to overwrite trailing part of a line, and we assume even
>  > > dumb terminals know how to do a carriage-return.
>  > 
>  > I think this earlier discussion is probably relevant... I'm guessing
>  > though, $EDITOR is set correctly here 8-)
> 
> I do have EDITOR set to a home-built version of gnuclient, and git
> talks to Emacs by way of that gnuclient just fine when I'm not using the
> -m "commit_message" git-commit option.
> 
>  > 
>  > 2008/12/17 Junio C Hamano <gitster@pobox.com>:
>  > > Any semi-good emacs users (let alone hackers) export PAGER=cat to be used
>  > > in compilation mode (and possibly shell mode), so this is not a problem in
>  > > practice.
>  > >
>  > > I have something like this in my .emacs:
>  > >
>  > >    (setenv "PAGER" "cat")
>  > >
>  > > I suspect (I am just a user not a hacker) this will have bad interaction
>  > > with emacs terminal emulation mode, but I do not use the mode, so it is
>  > > enough for me.
> 
> I have PAGER set to "cat" in the environment before I run Emacs for
> the same reason.
> 
> Unfortunately, this morning when I rebooted and reloaded from scratch,
> I am now unable to reproduce the CR codes output from "git pull" no
> matter what I do. I even tried the older git installed on Debian Linux
> "testing", and tried unsetting PAGER and GIT_PAGER, and saw the pager
> prompts and the terminal escape sequence output as I expected to
> (which is not the issue here).  I can't expect anyone else to help me
> debug this problem further if I can't even reproduce it
> anymore. Frustrating.
> 
> I do have automatic updates turned on, so perhaps something changed in
> the termcap or how terminal I/O is being done outside of git in my
> system.  Emacs would not have changed since I build Emacs from top of
> trunk CVS, and it only uses local Elisp packages AFAIK.
> 
> I don't suppose git has any logic that emits the progress messages
> based upon some estimate of amount of work it has to do, or has done,
> does it?

It does have logic to only emit progress messages at a reasonable rate 
(otherwise, you might be waiting for the progress messages to be printed 
instead of just waiting for the data to arrive). So it's possible that you 
now have things going fast enough that it only needs to print one message. 
It can also estimate that something hasn't taken long enough for the user 
to get impatient yet, and therefore not show progress at all (so the 
output won't be littered with progress output for every operation that 
could have taken a long time for some data, but didn't for this data).

In any case, it's all done in progress.c, so it should be easy enough to 
make changes to if you can come up with something better to do with 
progress messages and some way to determine when it should be done.

	-Daniel
*This .sig left intentionally blank*

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: CR codes from git commands
  2009-01-22 16:41             ` Mike Ralphson
@ 2009-01-22 16:50               ` Johannes Schindelin
  0 siblings, 0 replies; 24+ messages in thread
From: Johannes Schindelin @ 2009-01-22 16:50 UTC (permalink / raw)
  To: Mike Ralphson; +Cc: Brent Goodrick, Daniel Barkalow, Junio C Hamano, git

Hi,

On Thu, 22 Jan 2009, Mike Ralphson wrote:

> My $EDITOR is set to vim, as god intended. 8-)

Sorry, that is not true: from

http://www.biblegateway.com/passage/?book_id=50&chapter=1&verse=1&version=31&context=verse

we know that in the beginning was the Word.

Ciao,
Dscho

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: CR codes from git commands
  2009-01-22 16:44             ` Daniel Barkalow
@ 2009-01-22 16:52               ` Johannes Schindelin
  2009-01-23 16:12                 ` Brent Goodrick
  0 siblings, 1 reply; 24+ messages in thread
From: Johannes Schindelin @ 2009-01-22 16:52 UTC (permalink / raw)
  To: Daniel Barkalow; +Cc: Brent Goodrick, Mike Ralphson, Junio C Hamano, git

Hi,

On Thu, 22 Jan 2009, Daniel Barkalow wrote:

> In any case, it's all done in progress.c, so it should be easy enough to 
> make changes to if you can come up with something better to do with 
> progress messages and some way to determine when it should be done.

Maybe "git --no-progress <program>" would be a sensible user interface?

Ciao,
Dscho

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: CR codes from git commands
  2009-01-22 16:52               ` Johannes Schindelin
@ 2009-01-23 16:12                 ` Brent Goodrick
  2009-01-23 16:59                   ` Junio C Hamano
  2009-01-23 18:41                   ` Johannes Schindelin
  0 siblings, 2 replies; 24+ messages in thread
From: Brent Goodrick @ 2009-01-23 16:12 UTC (permalink / raw)
  To: Johannes Schindelin
  Cc: Daniel Barkalow, Brent Goodrick, Mike Ralphson, Junio C Hamano, git



Johannes Schindelin writes:
 > Hi,
 > 
 > On Thu, 22 Jan 2009, Daniel Barkalow wrote:
 > 
 > > In any case, it's all done in progress.c, so it should be easy enough to 
 > > make changes to if you can come up with something better to do with 
 > > progress messages and some way to determine when it should be done.
 > 
 > Maybe "git --no-progress <program>" would be a sensible user
 > interface?

Thanks. I now see the \r reference inside the "display" file-static
function inside progress.c.

However, I propose to add two options, the first being, IMO, the
minimal one to implement, and the second being "nice-to-have":

 - Bare minimum: Add a new --no-cr option (e.g., "git --no-cr
   <program>") that would prevent any git code (inside progress.c or
   elsewhere) from emitting a CR code from stdout or stderr.  This has
   the effect of allowing progress messages, but not asking too much
   of terminals-that-are-not-really-terminals such as the GNU Emacs
   shell mode.

 - Nice-to-have: Add a "git --no-progress" message that would never
   show progress at all (e.g., perhaps by not installing a signal
   handler inside progress.c such that no messages would not be
   emitted at all.

Both options are intended to be independent of each other.

And for both options, I would like there to be a config option to
allow the user to enable said behavior globally across all git
operations covered by that config file.

I might be willing to take a swipe at this myself and submit a patch,
provided I receive adequate noobie hand-holding (or hand-slapping) on
patch submission and test case development.

bg

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: CR codes from git commands
  2009-01-23 16:12                 ` Brent Goodrick
@ 2009-01-23 16:59                   ` Junio C Hamano
  2009-01-23 18:41                   ` Johannes Schindelin
  1 sibling, 0 replies; 24+ messages in thread
From: Junio C Hamano @ 2009-01-23 16:59 UTC (permalink / raw)
  To: Brent Goodrick; +Cc: Johannes Schindelin, Daniel Barkalow, Mike Ralphson, git

Brent Goodrick <bgoodr@gmail.com> writes:

>  - Bare minimum: Add a new --no-cr option (e.g., "git --no-cr
>    ...
>  - Nice-to-have: Add a "git --no-progress" message that would never
>  ...
> Both options are intended to be independent of each other.

I do not think so.  --no-progress should imply --no-cr ;-)

I do not think it makes much sense to pollute your non-terminal with 100
lines of 1%,2%,3%,...100% if it cannot sensibly do carriage-returns.  It
may be another knob to tweak, but it's a kind of thing you implement
because you could, not because it makes sense.  I would be mildly against
no-cr.

I suspect we may even be able to solve it without adding --no-progress.
Perhaps some commands do not have --quiet to squelch progress and teaching
them --quiet will solve the issue for you?

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: CR codes from git commands
  2009-01-23 16:12                 ` Brent Goodrick
  2009-01-23 16:59                   ` Junio C Hamano
@ 2009-01-23 18:41                   ` Johannes Schindelin
  2009-01-24 20:54                     ` Brent Goodrick
  1 sibling, 1 reply; 24+ messages in thread
From: Johannes Schindelin @ 2009-01-23 18:41 UTC (permalink / raw)
  To: Brent Goodrick; +Cc: Daniel Barkalow, Mike Ralphson, Junio C Hamano, git

Hi,

On Fri, 23 Jan 2009, Brent Goodrick wrote:

>  - Bare minimum: Add a new --no-cr option

I do not see any value of this over "--progress | tr '\r' '\n'".  (The 
--progress option being the natural counterpart to --no-progress, 
_forcing_ the display of the progress.)

And I disagree that --no-progress would be hard to implement.  Just have a 
look at 7d1864c(Introduce is_bare_repository() and core.bare configuration 
variable).

Basically, you'll have to

- introduce a global variable to both environment.c and cache.h,

- set it to -1 by default,

- handle a "--progress" and "--no-progress" option in git.c, setting the 
  global variable git_show_progress to 1 or 0, respectively,

- teach start_progress_delay() to return NULL if git_show_progress == 0,

- modify all users of start_progress*() to respect git_show_progress == 1,
  which probably means to look for "isatty" in builtin-pack-objects.c and 
  builtin-unpack-objects.c

- add documentation to Documentation/git.txt what --progress and 
  --no-progress do,

- add a simple test script to t/ (maybe t/t0005-progress.sh) that tests 
  that --progress works -- maybe you find a clever way to test 
  --no-progress, too, but that would be harder, as the progress is turned 
  off by default for the scripts anyway...)

Hth,
Dscho

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: CR codes from git commands
  2009-01-23 18:41                   ` Johannes Schindelin
@ 2009-01-24 20:54                     ` Brent Goodrick
  2009-01-24 21:14                       ` Johannes Schindelin
                                         ` (2 more replies)
  0 siblings, 3 replies; 24+ messages in thread
From: Brent Goodrick @ 2009-01-24 20:54 UTC (permalink / raw)
  To: Johannes Schindelin
  Cc: Brent Goodrick, Daniel Barkalow, Mike Ralphson, Junio C Hamano, git


Junio C Hamano writes:
 > I do not think so.  --no-progress should imply --no-cr ;-)
 > 
 > I do not think it makes much sense to pollute your non-terminal with 100
 > lines of 1%,2%,3%,...100% if it cannot sensibly do carriage-returns.  It
 > may be another knob to tweak, but it's a kind of thing you implement
 > because you could, not because it makes sense.  I would be mildly against
 > no-cr.

Good point. I'll drop the --no-cr as redundant.

Johannes Schindelin writes:
 > Hi,
 > 
 > On Fri, 23 Jan 2009, Brent Goodrick wrote:
 > 
 > >  - Bare minimum: Add a new --no-cr option
 > 
 > I do not see any value of this over "--progress | tr '\r' '\n'".  (The 
 > --progress option being the natural counterpart to --no-progress, 
 > _forcing_ the display of the progress.)

Agreed. Both --progress and --no-progress are the only options to be
implemented for this.  

 > Just have a 
 > look at 7d1864c(Introduce is_bare_repository() and core.bare configuration 
 > variable).

Note that I'm coming from a CVS and Perforce user background but am
still new to git usage. How do I "take a look" at "7d1864c"?

I will take a closer look at the list of things you explained in your
"Basically, you'll have to" list.

While I'm at it, what is the standard procedure for submitting git
patches for review once I've cooked up and validated it on my end? I'm
guessing posting the patch into this mailing list is part of the
answer to that question.

Thanks,
Brent

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: CR codes from git commands
  2009-01-24 20:54                     ` Brent Goodrick
@ 2009-01-24 21:14                       ` Johannes Schindelin
  2009-01-25  9:19                       ` Boyd Stephen Smith Jr.
  2009-01-25 20:35                       ` The lifecycle of a patch and the maintainer involvement Junio C Hamano
  2 siblings, 0 replies; 24+ messages in thread
From: Johannes Schindelin @ 2009-01-24 21:14 UTC (permalink / raw)
  To: Brent Goodrick; +Cc: Daniel Barkalow, Mike Ralphson, Junio C Hamano, git

Hi,

On Sat, 24 Jan 2009, Brent Goodrick wrote:

> Note that I'm coming from a CVS and Perforce user background but am 
> still new to git usage. How do I "take a look" at "7d1864c"?

Do this in a checkout of git.git:

$ git show 7d1864c

Alternatively, you can follow this URL:

	http://repo.or.cz/w/git.git?a=commitdiff;h=7d1864c

Ciao,
Dscho

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: CR codes from git commands
  2009-01-24 20:54                     ` Brent Goodrick
  2009-01-24 21:14                       ` Johannes Schindelin
@ 2009-01-25  9:19                       ` Boyd Stephen Smith Jr.
  2009-01-25 18:47                         ` Brent Goodrick
  2009-02-02  7:09                         ` Brent Goodrick
  2009-01-25 20:35                       ` The lifecycle of a patch and the maintainer involvement Junio C Hamano
  2 siblings, 2 replies; 24+ messages in thread
From: Boyd Stephen Smith Jr. @ 2009-01-25  9:19 UTC (permalink / raw)
  To: Brent Goodrick; +Cc: git

[-- Attachment #1: Type: text/plain, Size: 683 bytes --]

On Saturday 24 January 2009, Brent Goodrick <bgoodr@gmail.com> wrote 
about 'Re: CR codes from git commands':
>While I'm at it, what is the standard procedure for submitting git
>patches for review once I've cooked up and validated it on my end? I'm
>guessing posting the patch into this mailing list is part of the
>answer to that question.

If you've got a patch, I assume you've got a checkout.  Look in 
Documentation/SubmittingPatches.
-- 
Boyd Stephen Smith Jr.                     ,= ,-_-. =. 
bss@iguanasuicide.net                     ((_/)o o(\_))
ICQ: 514984 YM/AIM: DaTwinkDaddy           `-'(. .)`-' 
http://iguanasuicide.net/                      \_/     

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 197 bytes --]

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: CR codes from git commands
  2009-01-25  9:19                       ` Boyd Stephen Smith Jr.
@ 2009-01-25 18:47                         ` Brent Goodrick
  2009-02-02  7:09                         ` Brent Goodrick
  1 sibling, 0 replies; 24+ messages in thread
From: Brent Goodrick @ 2009-01-25 18:47 UTC (permalink / raw)
  To: Boyd Stephen Smith Jr.; +Cc: Brent Goodrick, git


Boyd Stephen Smith Jr. writes:
 > On Saturday 24 January 2009, Brent Goodrick <bgoodr@gmail.com> wrote 
 > about 'Re: CR codes from git commands':
 > >While I'm at it, what is the standard procedure for submitting git
 > >patches for review once I've cooked up and validated it on my end? I'm
 > >guessing posting the patch into this mailing list is part of the
 > >answer to that question.
 > 
 > If you've got a patch, I assume you've got a checkout.  Look in 
 > Documentation/SubmittingPatches.

Thanks I see that now. No, I don't have a patch yet, was struggling to
find that basic info that really should be front and center somewhere
on the wiki (and also access to the wiki is very slow).

bg

^ permalink raw reply	[flat|nested] 24+ messages in thread

* The lifecycle of a patch and the maintainer involvement
  2009-01-24 20:54                     ` Brent Goodrick
  2009-01-24 21:14                       ` Johannes Schindelin
  2009-01-25  9:19                       ` Boyd Stephen Smith Jr.
@ 2009-01-25 20:35                       ` Junio C Hamano
  2 siblings, 0 replies; 24+ messages in thread
From: Junio C Hamano @ 2009-01-25 20:35 UTC (permalink / raw)
  To: Brent Goodrick; +Cc: Johannes Schindelin, Daniel Barkalow, Mike Ralphson, git

Brent Goodrick <bgoodr@gmail.com> writes:

> While I'm at it, what is the standard procedure for submitting git
> patches for review once I've cooked up and validated it on my end? I'm
> guessing posting the patch into this mailing list is part of the
> answer to that question.

Yes, a guideline is in Documentation/SubmittingPatches for the initial
submission.  After that, the lifecycle of a patch submitted on the list
goes like this:

 (1) A patch is shown to the list participants.

 (2) People may like it, or may have issues with it, and responds with
     their comments describing problems, suggestions for improvements,
     etc.  People who are not interested in the topic may stay silent.

 (3) The original author responds with updated patch.  Sometimes people
     who commented on in step 2 may even send "here is how I would do this
     one; don't you think this is better?", and the original author may
     say "Yeah, let's use yours instead".

 (4) After steps 2 and 3 repeats zero or more times, the latest patch may
     become one that everyone likes, or at least nobody has trouble with
     inclusion.  The author sends such a patch saying "this is meant for
     inclusion based on discussion and refinements in these threads...".

 (5) The maintainer picks it up when it looks polished enough.

Your patch may appear in the periodical "What's cooking" or "What's in"
summary with zero iteration of steps 2 and 3 if it is obvious enough.

I act as just one of the list participant during steps 1-3.  I may stay
silent during this period but that only means the topic is not interesting
to me and nothing more.  It does not mean that the topic has no chance of
getting included.

I act as the maintainer for steps 4 and 5.  If you do not hear from me
after step 4, then I am either being lazy, busy, or sick, or the patch got
lost in the noise and I need a reminder.  Note that I may reject or ask
further refinement at step 4 to ensure overall quality throughout the
system even in areas I am not interested in and didn't say anything during
steps 1-3.

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: CR codes from git commands
  2009-01-25  9:19                       ` Boyd Stephen Smith Jr.
  2009-01-25 18:47                         ` Brent Goodrick
@ 2009-02-02  7:09                         ` Brent Goodrick
  1 sibling, 0 replies; 24+ messages in thread
From: Brent Goodrick @ 2009-02-02  7:09 UTC (permalink / raw)
  To: git

I'm nearing completion on the patch for the --progress and
--no-progress command-line options.  I am able to manually validate
the behavior, but am a bit stumped as to how to efficiently code up
the test script.  My manual test involves doing a git clone of the git
repository, which produces the volume of I/O sufficiently bulky to
trigger the progress message code.  But that bulk means that the test
case will take a long time to complete, hence making using a git clone
of the git code in the test case impractical.

Also, in order for the script to do its job, it will need to tell the
difference between a git run that has progress from one that does not.
 The first idea would be to simply use shell command redirection on
the git command itself, but that defeats the tty detection logic, so I
don't think that is an option either.

Does anyone have any recommendations here? If not, then I guess I will
have to forgo the test script and just submit the patch without it.

Thanks,
Brent

On Sun, Jan 25, 2009 at 1:19 AM, Boyd Stephen Smith Jr.
<bss@iguanasuicide.net> wrote:
>
> On Saturday 24 January 2009, Brent Goodrick <bgoodr@gmail.com> wrote
> about 'Re: CR codes from git commands':
> >While I'm at it, what is the standard procedure for submitting git
> >patches for review once I've cooked up and validated it on my end? I'm
> >guessing posting the patch into this mailing list is part of the
> >answer to that question.
>
> If you've got a patch, I assume you've got a checkout.  Look in
> Documentation/SubmittingPatches.
> --
> Boyd Stephen Smith Jr.                     ,= ,-_-. =.
> bss@iguanasuicide.net                     ((_/)o o(\_))
> ICQ: 514984 YM/AIM: DaTwinkDaddy           `-'(. .)`-'
> http://iguanasuicide.net/                      \_/

^ permalink raw reply	[flat|nested] 24+ messages in thread

end of thread, other threads:[~2009-02-02  7:12 UTC | newest]

Thread overview: 24+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-01-20 16:26 CR codes from git commands Brent Goodrick
2009-01-20 17:08 ` Johannes Schindelin
     [not found]   ` <18806.44057.477379.215492@hungover.brentg.com>
     [not found]     ` <alpine.DEB.1.00.0901210930370.7929@racer>
2009-01-21 14:42       ` Brent Goodrick
2009-01-21 15:38         ` Johannes Schindelin
2009-01-22  4:00           ` Brent Goodrick
2009-01-22  4:47 ` Daniel Barkalow
2009-01-22  7:34   ` Brent Goodrick
2009-01-22  7:46     ` Daniel Barkalow
2009-01-22  8:04       ` Junio C Hamano
2009-01-22 10:04         ` Mike Ralphson
2009-01-22 16:13           ` Brent Goodrick
2009-01-22 16:41             ` Mike Ralphson
2009-01-22 16:50               ` Johannes Schindelin
2009-01-22 16:44             ` Daniel Barkalow
2009-01-22 16:52               ` Johannes Schindelin
2009-01-23 16:12                 ` Brent Goodrick
2009-01-23 16:59                   ` Junio C Hamano
2009-01-23 18:41                   ` Johannes Schindelin
2009-01-24 20:54                     ` Brent Goodrick
2009-01-24 21:14                       ` Johannes Schindelin
2009-01-25  9:19                       ` Boyd Stephen Smith Jr.
2009-01-25 18:47                         ` Brent Goodrick
2009-02-02  7:09                         ` Brent Goodrick
2009-01-25 20:35                       ` The lifecycle of a patch and the maintainer involvement Junio C Hamano

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.