Install locale in Dockerfile

gdevenyi commented 4 years ago

Running perl based packages currently throws a warning:

perl: warning: Setting locale failed.
perl: warning: Please check that your locale settings:
        LANGUAGE = (unset),
        LC_ALL = (unset),
        LANG = "en_US.UTF-8"
    are supported and installed on your system.
perl: warning: Falling back to the standard locale ("C").

Stolen from https://daten-und-bass.io/blog/fixing-missing-locale-setting-in-ubuntu-docker-image/

prioux commented 4 years ago

We could also simply set the LANG environment variable to 'C'....

gdevenyi commented 4 years ago

It should at least be C.UTF-8 if we go with the C locale.

prioux commented 4 years ago

Well, if installing the locales package works, let's go with it. Xindi, want to merge this?

sandywang commented 4 years ago

For Docker version, I think it's great to just install locales because we do not need change any other part of CIVET. I will merge it now!

Thank you for your contribution, Gabriel and Pierre.

Sincerely, Xindi Wang

PhD, Postdoctoral Researcher, McGill Centre for Integrative Neuroscience (MCIN), Montreal Neurological Institute (MNI), 3801 University St., Montreal, QC., Canada, H3A 2B4

On Thu, Feb 27, 2020 at 12:44 PM Pierre Rioux notifications@github.com wrote:

Well, if installing the locales package works, let's go with it. Xindi, want to merge this?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/aces/CIVET_Full_Project/pull/5?email_source=notifications&email_token=AARPERK7OGVJSGSLI3RR4GDRE73YNA5CNFSM4K4TK2PKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENFIYXI#issuecomment-592088157, or unsubscribe https://github.com/notifications/unsubscribe-auth/AARPERN2NXOQNYJ5HQVXJO3RE73YNANCNFSM4K4TK2PA .

aces / CIVET_Full_Project

Install locale in Dockerfile #5

Thank you for your contribution, Gabriel and Pierre.