-
Notifications
You must be signed in to change notification settings - Fork 99
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
-path
flag does not appear in usage
#273
Comments
Thanks for catching that. Yes, it's likely a bad merge since the .rc file gets treated as a binary file. I've tried a few ways to fix it, but either Git or VS doesn't like it when I do. But, yes, This is a quick fix, but I want to talk with the VC folks and see what they recommend to help mitigate this issue going forward. |
That's what I thought. I encountered the same issue when I was comparing the commits. Kind of disappointing that the |
I followed up with the VC team and as of 16.3, UTF-8 .rc files should work fine now. I'll convert the file and make sure everything is working end to end in the course of fixing this. |
Upgrades the project to work with supported versions, and prepares pipelines to work with a GitHub flow i.e., only main (will rename from master). Also fixes microsoft#273
* Upgrade project, prepare GitHub flow Upgrades the project to work with supported versions, and prepares pipelines to work with a GitHub flow i.e., only main (will rename from master). Also fixes #273 * Microsoft mandatory file (#274) Co-authored-by: microsoft-github-policy-service[bot] <77245923+microsoft-github-policy-service[bot]@users.noreply.github.com> * Update .vsconfig Co-authored-by: microsoft-github-policy-service[bot] <77245923+microsoft-github-policy-service[bot]@users.noreply.github.com>
This was resolved in #279. We'll release a new version soon. |
The
-path
flag usage was removed from the RC file in this commit. This flag seems to be exactly what we need, but I saw it disappear from the help:Version 2.7.1
Version 3.0.3
I'm pretty sure it's just a bad merge or something similar because I couldn't find anything that would indicate the flag is being phased out.
The text was updated successfully, but these errors were encountered: