-
Notifications
You must be signed in to change notification settings - Fork 586
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
Start of process xbuild failed #1213
Comments
Fixes encoding issues when encoding property is not set: Refs fsprojects#1212. This is a revised version of commit 2d63a45, which broke external processes when standard error/output is not redirected: Refs fsprojects#1213 This commit only sets the encoding when output is redirected.
can you please retry with 4.24.3? @juergenhoetzel submitted a second patch |
Yes, that does work for me. 👍 |
But note that I still get Chinese characters. Was the patch supposed to fix this too? |
@kentcb Yes, this should be fixed by the new release. Are you sure you have updated the FAKE package? I can reproduce invalid characters with the old release:
After
|
Hmmm, you might be right. I'm struggling to reproduce it now. I may have just confused myself when testing yesterday. My code base is not quite ready to give it a full run through right now. Let me get back to you ASAP. |
I am getting this on 4.28. Output in bad encoding:
Code:
|
/cc @nosami we really need to find the root cause for this. |
I'm looking at the Shell.Exec code and it calls asyncShellExec function. I cannot see when during asyncShellExec we specify input and output encoding. |
@juergenhoetzel I repeated your mistake, should learn history better. |
See fsprojects/FAKE#1196, fsprojects/FAKE#1212, fsprojects/FAKE#1213 & fsprojects/FAKE#1194 Not sure of the ramifications of this change, but this fixes the encoding issues in FAKE for me. l.Head.Encoding was `System.IO.StringWriter(new System.Text.StringBuilder()).Encoding` which evaluates to utf-16 /cc:@forki
Description
Since the 4.24.1 release at the weekend, my build now fails with:
Repro steps
A build that was previously working with V4.23.5 is now failing per above.
Expected behavior
Build should work as in prior versions.
Actual behavior
Build fails with above error.
Known workarounds
Use older version of FAKE.
Related information
The text was updated successfully, but these errors were encountered: