Skip to content
This repository was archived by the owner on Apr 5, 2025. It is now read-only.

Use LANG=C in example#61

Open
toolforger wants to merge 1 commit into
agavrilov76:masterfrom
toolforger:patch-1
Open

Use LANG=C in example#61
toolforger wants to merge 1 commit into
agavrilov76:masterfrom
toolforger:patch-1

Conversation

@toolforger
Copy link
Copy Markdown

Incomplete fix: I don't know where to add the environment for the JSch example, and I haven't checked the code in the examples/ directory.

Incomplete fix: I don't know where to add the environment for the JSch example, and I haven't checked the code in the `examples/` directory.
@coveralls
Copy link
Copy Markdown

coveralls commented Apr 9, 2019

Coverage Status

Coverage decreased (-0.2%) to 91.944% when pulling 8941004 on toolforger:patch-1 into 5acbe1f on Alexey1Gavrilov:master.

@toolforger
Copy link
Copy Markdown
Author

Test failure is unrelated to the change.

@toolforger toolforger closed this Apr 16, 2019
@toolforger toolforger reopened this Apr 16, 2019
@agavrilov76
Copy link
Copy Markdown
Owner

Can you please elaborate why LANG=C is needed in the example code?

@toolforger
Copy link
Copy Markdown
Author

LANG=C eliminates internationalization.
E.g. on my machine, ls does not respond with total XXX but with insgesamt XXX, making the script fail.

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.

3 participants