Skip to content
This repository has been archived by the owner on Aug 23, 2018. It is now read-only.

Fix build with non-utf8 locale #205

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kwohlfahrt
Copy link

Had some issues building elm-reactor due to LC_ALL=C. This is standard behaviour for at least the paludis package manager, and possibly others. The changes in this PR should make the build process independent of locale.

Run of elm-make was throwing an exception with LC_ALL=C, fixed by using
readProcessWithExitCode from System.Process.ByteString instead of
System.Process.
@process-bot
Copy link

Thanks for the pull request! Make sure it satisfies this checklist. My human colleagues will appreciate it!

Here is what to expect next, and if anyone wants to comment, keep these things in mind.

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