threads / discuss / 49040

git worktree add prints to stdout

Subject: git worktree add prints to stdout

## tl;dr

3 messages between Aug 5, 2018 and Aug 5, 2018.

replies: 2people: 3as markdown or json

Karen Arutyunov· Aug 5, 2018, 13:01 UTC · lore

Hello,

The 'git worktree add' command prints to both standard streams. So in the following example the first line is printed to stderr and the second to stdout.

$ git worktree add ../pub build2-control Preparing ../pub (identifier pub) HEAD is now at b03ea86 Update

This looks like a bug, as, for example, the checkout command prints 'HEAD is now at...' message to stderr.

Best regards, Karen

Thomas Gummerer· Aug 5, 2018, 14:52 UTC · re: Karen Arutyunov · lore

Re: git worktree add prints to stdout

On 08/05, Karen Arutyunov wrote:

Show 5 quoted lines

> Hello,
> 
> The 'git worktree add' command prints to both standard streams. So in the
> following example the first line is printed to stderr and the second to
> stdout.

git 2.18.0 should print both of those lines to stdout. This was done to match where 'git reset --hard' prints the 'HEAD is now at...' message. See also the thread at [1] where we did make that decision.

[1]: https://public-inbox.org/git/CAPig+cQ8VzDycUMo-QOexNDBgQGEGj2BPmPa-Y0vhGCt_brbhg@mail.gmail.com/

Show 6 quoted lines

> $ git worktree add ../pub build2-control
> Preparing ../pub (identifier pub)
> HEAD is now at b03ea86 Update
> 
> This looks like a bug, as, for example, the checkout command prints 'HEAD is
> now at...' message to stderr.

I think eventually it would be nice to write all those messages to 'stderr', as I think they do make more sense there. I said I may do that at some point in [2], but never got around to it yet. If you want to take a stab at it, feel free :)

[2]: https://public-inbox.org/git/xmqq604rzytx.fsf@gitster-ct.c.googlers.com/

> Best regards,
> Karen

Jonathan Nieder· Aug 5, 2018, 22:45 UTC · re: Thomas Gummerer · lore

Re: git worktree add prints to stdout

Hi,

Thomas Gummerer wrote:

Show 5 quoted lines

> git 2.18.0 should print both of those lines to stdout.  This was done
> to match where 'git reset --hard' prints the 'HEAD is now at...'
> message. See also the thread at [1] where we did make that decision.
>
> [1]: https://public-inbox.org/git/CAPig+cQ8VzDycUMo-QOexNDBgQGEGj2BPmPa-Y0vhGCt_brbhg@mail.gmail.com/

Thanks for the pointer!  And especially:

[...]

Show 6 quoted lines

> I think eventually it would be nice to write all those messages to
> 'stderr', as I think they do make more sense there.  I said I may do
> that at some point in [2], but never got around to it yet.  If you
> want to take a stab at it, feel free :)
>
> [2]: https://public-inbox.org/git/xmqq604rzytx.fsf@gitster-ct.c.googlers.com/

Thanks for this pointer, which captures my feelings pretty well.

Jonathan