mirror of
https://github.com/golang/go
synced 2024-09-05 07:54:46 +00:00
![Dmitri Shuralyov](/assets/img/avatar_default.png)
This is a followup to CL 96495. It should be simpler and more robust to achieve .bat files having CRLF line endings by treating it as a binary file, like all other files, and checking it in with the desired CRLF line endings. A test is used to check the entire Go tree, short of directories starting with "." and named "testdata", for any .bat files that have anything other than strict CRLF line endings. This will help catch any accidental modifications to existing .bat files or check ins of new .bat files. Importantly, this is compatible with how Gerrit serves .tar.gz files, making it so that CRLF line endings are preserved. The Go project is supported on many different environments, some of which may have limited git implementations available, or none at all. Relying on fewer git features and special rules makes it easier to have confidence in the exact content of all files. Additionally, Go development started in Subversion, moved to Perforce, then Mercurial, and now uses Git.¹ Reducing its reliance on git-specific features will help if there will be another transition in the project's future. There are only 5 .bat files in the entire Go source tree, so a new one being added is a rare event, and we prefer to do things in Go instead. We still have the option of improving the experience for developers by adding a pre-commit converter for .bat files to the git-codereview tool. ¹ https://groups.google.com/d/msg/golang-dev/sckirqOWepg/YmyT7dWJiocJ Fixes #39391. For #37791. Change-Id: I6e202216322872f0307ac96f1b8d3f57cb901e6b Reviewed-on: https://go-review.googlesource.com/c/go/+/236437 Reviewed-by: Bryan C. Mills <bcmills@google.com>
17 lines
639 B
Plaintext
17 lines
639 B
Plaintext
# Treat all files in the Go repo as binary, with no git magic updating
|
|
# line endings. This produces predictable results in different environments.
|
|
#
|
|
# Windows users contributing to Go will need to use a modern version
|
|
# of git and editors capable of LF line endings.
|
|
#
|
|
# Windows .bat files are known to have multiple bugs when run with LF
|
|
# endings, and so they are checked in with CRLF endings, with a test
|
|
# in test/winbatch.go to catch problems. (See golang.org/issue/37791.)
|
|
#
|
|
# We'll prevent accidental CRLF line endings from entering the repo
|
|
# via the git-codereview gofmt checks and tests.
|
|
#
|
|
# See golang.org/issue/9281.
|
|
|
|
* -text
|