bazel
/
bazel-skylib
mirror of https://github.com/bazelbuild/bazel-skylib
2
0
Fork 0
Code Issues Releases Activity

Remove `$` from grep regexes (#311) 

Due to grep having dropped support for handling line-ending matches in a cross-platform way, grepping for `...$` will now fail on Windows, as it no longer ignores the CR part of the CRLF line endings on Windows.

This should turn this project green again on Bazel CI.
This commit is contained in:
Philipp Wollermann 2021-08-09 17:15:50 +02:00 committed by GitHub
parent b053a5ae11
commit 0c57e74606
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
tests/write_file/write_file_tests.sh
View File

@ -52,8 +52,8 @@ function test_write_empty_text() {
function test_write_nonempty_text() {
cat "$(rlocation bazel_skylib/tests/write_file/out/nonempty.txt)" >"$TEST_log"
expect_log '^aaa$'
expect_log '^bbb$'
expect_log '^aaa'
expect_log '^bbb'
}
function test_write_empty_bin() {
@ -62,7 +62,7 @@ function test_write_empty_bin() {
function test_write_nonempty_bin() {
cat "$(rlocation bazel_skylib/tests/write_file/nonempty-bin-out.txt)" >"$TEST_log"
expect_log '^potato$'
expect_log '^potato'
}
run_suite "write_file_tests test suite"