Fix windows ability to checkout project #201#246
Open
fredsa wants to merge 4 commits intosehugg:masterfrom
Open
Fix windows ability to checkout project #201#246fredsa wants to merge 4 commits intosehugg:masterfrom
fredsa wants to merge 4 commits intosehugg:masterfrom
Conversation
Contributor
Author
|
Owner
|
The "remote:" prefix is used several places in the code, it's probably easier to change the skeleton-loading code than change the prefix. Actually, we can probably strip off the entire remote: prefix when loading the file. |
Recommend MSYS2 `UCRT64` environment.
Avoid ':' in filenames by stripping 'remote:' file extension prefix: - Rename `skeleton.remote:llvm-mos` to `skeleton.llvm-mos`, which allow Windows users to once again checkout the repo. - Update `getSkeletonFile` to strip 'remote:'.
Contributor
Author
|
Sg. Force pushed new version that does this. |
Fix broken C projects on Windows. Ensure C build tools that require LF line endings always get them, regardless of checked in profile file line endings and the developer's local git config.
Contributor
Author
|
Fixed an issue with C based projects failing to compile & run: added line ending fix. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
skeleton.remote:llvm-mosto Windows compatiable filenameskeleton.remote_llvm-mos:llvm-mossuffix with_llvm-mossuffix