Revert "Revert "use POSIX version of esrerror_r""

[breznak]

This reverts commit ecd6ba7.

Because it introduced problems on OSX
Use strerror only, simplify

Should fix #53

[breznak]

@rhyolight please test when u have time..

[rhyolight]

@breznak doing now...

[breznak]

yes :) because it's
https://github.com/breznak/nupic.core/tree/fix_OSX_fstream

[rhyolight]

Yeah I saw that as soon as I commented 😊

[breznak]

looks like travis is not eager to check this for us.. :/

[rhyolight]

@breznak There is a problem. In subsequent commits on nupic.core, files were moved into a src directory. We can merge this, but we still won't be able to fix the nupic build for OS X, because the merge with be at the tip of master, and nupic is not compatible with that until numenta/nupic-legacy#751 merges. So if this passes, we can merge it, and we might say it fixes #53, but it will not fix the nupic build for OS X.

[breznak]

@rhyolight Right, I see.. But what should we do? It has to be fixed in nupic.core and that file does not exist here anymore. That is the problem of being out-of-sync :/

I can try some git magic with time..but im not sure

[rhyolight]

The only thing I think will work is to revert numenta/nupic-legacy#768 so nupic points to https://github.com/numenta/nupic.core/tree/3ecc06a2e0b175a74381330d64a4ab803b71863d once again.

[rhyolight]

Or we can just accept the fact that nupic@master won't build with OS X until numenta/nupic-legacy#751 is merged.

[rhyolight]

Ok, if #56 passes and we merge it, we can then merge master into this PR and I will retry.

[fergalbyrne]

I'd accept a short downtime on OSX build to get it sorted. Can't have 100% uptime when you're performing brain surgery.—
Sent from Mailbox for iPhone

On Fri, Apr 4, 2014 at 9:19 PM, Matthew Taylor notifications@github.com
wrote:

Ok, if #56 passes and we merge it, we can then merge master into this PR and I will retry.

Reply to this email directly or view it on GitHub:
#54 (comment)

[david-ragazzi]

Hi @rhyolight , merge this when possible, please..