Date: Sun, 19 Aug 2012 03:29:10 +0200 From: magnum <john.magnum@...hmail.com> To: john-dev@...ts.openwall.com Subject: Re: OSX 10.8 (GT 650M) build fail On 2012-08-19 02:57, Solar Designer wrote: > On Sun, Aug 19, 2012 at 02:22:18AM +0200, magnum wrote: >> On 2012-08-19 00:03, magnum wrote: >>> On 2012-08-18 20:06, Solar Designer wrote: >>>> I think we shouldn't be changing these settings in the Makefile globally >>>> unless we truly have to. A better way may be to do this just for the >>>> sed invocations, like this: >>>> >>>> LANG=C sed ... >>> >>> Yes. I'll fix this too. >> >> BTW I still used LC_ALL as opposed to LANG. Is LC_ALL not supported on >> all platforms? Maybe I should set both? > > I think both are supported, and LC_ALL is supposed to have higher > precedence than LANG. So I think we should try: > > LC_ALL=C sed ... > > (which I guess is what you implemented now), and see if this works. That is what I did now, but I still don't get why my previous try did not work. I had it like this: SED = sed NULL = /dev/null CPPFLAGS = -E + LC_ALL = C That may have downsides but I'm puzzled why it did not work. Erik, myrice, does the current git version work out of the box? magnum
Powered by blists - more mailing lists
Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.