Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
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.