I think situation with locales is similar one to SIGPIPE issues we had a while ago, i.e as long as libprocess and stout are considered being libraries we can't force users to set process global flags like SIG_IGN. So I guess we should fix (or at least mark as FIXME) all locale related cases like we did with SIGPIPE.
On Wed, May 20, 2015, 10:31 Alexander Rojas <alexan...@mesosphere.io> wrote: > Hey guys, I just got a review pointing out that ‘strftime’ is dependent on > the locale used. This not only affects my patch but code already in Mesos: > process/time.hpp and encoder.hpp both make use of fields which are locale > specific. The question here is, how to deal with it? > > Locales tend to be a pain since the functions handling them are not thread > safe, so I would suggest setting a global locale for mesos programs (master > and slaves) on start up, i.e. setlocale(LC_ALL, "C");
