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

Fix for https://github.com/jwood803/xfsharp/pull/1 #546

Merged
merged 1 commit into from
Sep 21, 2014

Conversation

sergey-tihon
Copy link
Member

Messages to stderr stream can break the build

Messages to stderr stream can break the builds
@forki forki merged commit ae75e40 into fsprojects:master Sep 21, 2014
@forki
Copy link
Member

forki commented Sep 21, 2014

I don't really understand. We do this everywhere in FAKE

@sergey-tihon
Copy link
Member Author

Yep, when AppVeyor configured to run build.cmd using cmd (like here) everything should be OK.

build_script:
  - cmd: build.cmd

but if you (for some reason) run build.cmd using PowerShell

build_script:
  - ps: .\build.cmd

you could not write to stderr stream.

So, probable it not a reason to modify FAKE ...
Or would be better to release some kind of guides how to configure Travis-CI and AppVeyor to run FAKE scripts, and that is not a very good idea to use PowerShell ISE for running FAKE scripts or you need to redirect stderr stream.

// cc @jwood803

@jwood803
Copy link
Contributor

I think the guide on how to configure the two to run the FAKE scripts would work well. If that works, I'll be glad to take a shot at it.

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