Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GH1025: Replace the command line argument switches for GitVersion #1021

Merged

Conversation

asbjornu
Copy link
Contributor

This PR replaces slash with dash in the command line argument switches for gitversion.exe, so it is more Unix friendly. It will still fail exeucting on OS X due to GitTools/GitVersion#890, but once that has been fixed, this should ensure that GitVersion executes successfully on OS X.

@dnfclas
Copy link

dnfclas commented Jun 29, 2016

Hi @asbjornu, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!

The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.

TTYL, DNFBOT;

Replace slash with dash in the command line argument switches for gitversion.exe, so it is more Unix friendly. It will still fail exeucting on OS X due to GitTools/GitVersion#890, but once that has been fixed, this should ensure that GitVersion executes successfully on OS X.
@asbjornu asbjornu force-pushed the feature/gitversion-command-line-switches branch from ffdb7fb to 4b0b47b Compare June 30, 2016 11:35
asbjornu added a commit to asbjornu/NUnit.System.Linq that referenced this pull request Jun 30, 2016
@gep13
Copy link
Member

gep13 commented Jun 30, 2016

This is related to #1025

@gep13
Copy link
Member

gep13 commented Jun 30, 2016

@asbjornu this makes sense to me, and explains why I was seeing errors in the GitVersion build on Travis.

asbjornu added a commit to asbjornu/NUnit.System.Linq that referenced this pull request Jun 30, 2016
@gep13
Copy link
Member

gep13 commented Jun 30, 2016

@cake-build/team I am going to go ahead and merge this, even though the Travis build failed, as there is a known issue with Travis.

@gep13 gep13 merged commit 60629d4 into cake-build:develop Jun 30, 2016
@asbjornu
Copy link
Contributor Author

@gep13 Excellent, thanks!

@asbjornu asbjornu deleted the feature/gitversion-command-line-switches branch June 30, 2016 11:52
@gep13 gep13 changed the title Replace the command line argument switches for GitVersion GH1025: Replace the command line argument switches for GitVersion Jul 11, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants