Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Redirect 'git fetch' output to log#7312

Merged
naamunds merged 1 commit into
dotnet:masterfrom
naamunds:FixSyncLogging
Mar 29, 2016
Merged

Redirect 'git fetch' output to log#7312
naamunds merged 1 commit into
dotnet:masterfrom
naamunds:FixSyncLogging

Conversation

@naamunds

Copy link
Copy Markdown
Member

Much of the output from 'git fetch' is to stderr, so it should also be
redirected to the log file. The original implementation using &>> was
added to Bash in version 4, so to ensure that this script works on OS X,
an alternative syntax is used.

cc: @maririos @joperezr

Much of the output from 'git fetch' is to stderr, so it should also be
redirected to the log file.  The original implementation using &>> was
added to Bash in version 4, so to ensure that this script works on OS X,
an alternative syntax is used.
@maririos

Copy link
Copy Markdown
Member

@naamunds please make sure you test it in Linux and OS X

@naamunds

Copy link
Copy Markdown
Member Author

@maririos I verified that it works as expected in both Ubuntu and OS X.

@maririos

Copy link
Copy Markdown
Member

Cool. Thanks Nate.
Then LGTM

@joperezr

Copy link
Copy Markdown
Member

@dotnet-bot test outerloop Ubuntu14.04 debug

@joperezr

Copy link
Copy Markdown
Member

I just triggered and outerloop run as I want to make sure that the ./build.sh flow on the build machines are not broken by this.

@joperezr

Copy link
Copy Markdown
Member

If you have already tested it for Ubuntu then there is no need for the outerloop run to finish. LGTM

@naamunds naamunds merged commit 77f0b11 into dotnet:master Mar 29, 2016
@naamunds naamunds deleted the FixSyncLogging branch March 29, 2016 17:31
@karelz karelz modified the milestone: 1.0.0-rtm Dec 3, 2016
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
Redirect 'git fetch' output to log

Commit migrated from dotnet/corefx@77f0b11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants