Skip to content
This repository has been archived by the owner on Dec 8, 2024. It is now read-only.

Handle case of .mill-version with Windows line ends #65

Merged
merged 1 commit into from
Nov 17, 2024

Conversation

sideeffffect
Copy link
Contributor

@sideeffffect sideeffffect commented Nov 15, 2024

This is a precaution for cases when the .mill-version file can contain a Windows line end \r\n instead of the typical Unix line end \n. Without this fix, in such situation the script breaks in a spectacular and surprising way.

This is a precaution for cases when the .mill-version file can contain a Windows line end `\r\n` instead of the typical Unix line end `\n`.
Without this fix, in such situation the script breaks in a spectacular and surprising way.
@sideeffffect
Copy link
Contributor Author

@lefou
Copy link
Owner

lefou commented Nov 17, 2024

Thank you for this contribution.

Without this fix, in such situation the script breaks in a spectacular and surprising way.

Do you have an example by any chance?

@lefou lefou merged commit fbfbd39 into lefou:main Nov 17, 2024
2 checks passed
@lefou lefou added this to the 0.4.13 milestone Nov 17, 2024
@sideeffffect sideeffffect deleted the patch-1 branch November 17, 2024 23:09
@sideeffffect
Copy link
Contributor Author

The error it was giving looked like this:

$ mill __.compile
.jar ...st-0.12.1o1.maven.org/maven2/com/lihaoyi/mill-dist/0.12.1
curl: (3) URL using bad/illegal format or missing URL

Where mill is pointing to the wrapper script that I'm changing in this PR

@sideeffffect
Copy link
Contributor Author

Archive with an example
scala project.zip

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants