-
Notifications
You must be signed in to change notification settings - Fork 985
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
Should fix the tests that depends on certain locale #3450
Comments
I actually get tons of errors regarding to the tests for
I rarely use The full test output on Windows 10 (the locale is: Chinese (Simplified)_China.936)
|
I see... It's because Git auto converted the |
When running |
The English-v-native part will become a lot easier after #3935 is eventually merged. EOL issues are another thing... |
Test 1445 seems to work with R 3.6.1 but not 3.5.3 on windows. |
As metioned above, if you are using Git to clone the repo to your Windows computer, you will find tests failures. This is because when Git clones the repo, it automatically convert the |
I am getting the same issue as mentioned above.
|
using |
@shrektan could you please check if #4431 will resolve all problems here? not sure but fresh git clone may be needed to verify that (to avoid windows newline conversion): mkdir tmp
cd tmp
git clone --depth=1 -o upstream https://github.com/Rdatatable/data.table.git
cd data.table
git fetch upstream ci2:ci2
git checkout ci2
make build && make install && make test |
1445 still produces the error.
@jangorecki I expect it will fail, actually. I see you changed the ".gitattributes" in the branch ci2. But when I execute |
Uh, yes. I had the same issue on windows CI, and using gitattributes helped. |
I confirm that there are still problems,
but an easy fix is to just run the tests with LC_ALL=C,
|
@shrektan could you confirm that issue is resolved for you? |
For example, test 168.1, I have to set the locale to "C" in order to parse the
datetimes
correctly (otherwise they are all NAs) because my default locale is Chinese (Simplified)_People's Republic of China.936.Another example is the test 1966, it simply throws an error on my computer complains the file name is invalid.
data.table/inst/tests/tests.Rraw
Lines 468 to 472 in 676c9b2
data.table/inst/tests/tests.Rraw
Lines 13012 to 13014 in 676c9b2
The text was updated successfully, but these errors were encountered: