Skip to content
This repository has been archived by the owner on Jul 4, 2023. It is now read-only.

add yosemite support for neko #34537

Closed
wants to merge 1 commit into from
Closed

add yosemite support for neko #34537

wants to merge 1 commit into from

Conversation

jdonaldson
Copy link
Contributor

neko works fine in yosemite. This formula works if I change the symbol reference. Is this how we're supposed to signal that?

@DomT4
Copy link
Member

DomT4 commented Nov 29, 2014

Just remove the dependency entirely.

@jacknagel
Copy link
Contributor

Doesn't work according to the build bot.

@DomT4
Copy link
Member

DomT4 commented Nov 29, 2014

Oh! I wonder about something. @jdonaldson, are you using 10.10.2? There seems to have been some Apache-based fixes in 10.10.2, at least, I assume that's the variable because of the 10.10.1 bot failure and neko only actually building successfully for me now, when it failed upwards of 15+ times on 10.10.1 and 10.10.

==> Summary
🍺  /usr/local/Cellar/neko/2.0.0-6ab8f48_1: 23 files, 4.5M, built in 107 seconds
CPU: quad-core 64-bit ivybridge
OS X: 10.10.2-x86_64
Xcode: 6.1
CLT: 6.1.0.0.1.1413057044
Clang: 6.0 build 600
X11: 2.7.7 => /opt/X11
System Ruby: 2.0.0-481

@jdonaldson
Copy link
Contributor Author

I'm still on 10.10.1, that's strange.

@DomT4
Copy link
Member

DomT4 commented Nov 30, 2014

Oh, curious. I think you're the first person I've seen manage to compile Neko on 10.10.1. It didn't require any formula modification locally, it just built? As far as I knew, the Apache2 errors were essentially waiting on an Apple fix. Upstream had only just fixed another set of fatal build errors. I'm completely stumped as to why this would compile for you on 10.10.1 or 10.10, heh 😸.

@jdonaldson
Copy link
Contributor Author

I think I had some leftovers from a previous build in the neko directory, my bad.

It looks like there's some static directory settings for neko in its build file. I have filed another pull request on neko to allow brew to set this according to its prefix. When that goes through then I'll update this pull request with the updated settings.

HaxeFoundation/neko#45

@jacknagel
Copy link
Contributor

I don't think that pull request changes anything from our perspective, as we are already passing LIB_PREFIX to make in the formula.

The difference here seems to be whether or not the CLT is installed (vs. Xcode).

@DomT4
Copy link
Member

DomT4 commented May 9, 2015

The difference here seems to be whether or not the CLT is installed (vs. Xcode).

We don't have a special dep to state this, right?

@jacknagel jacknagel closed this in 5c30287 May 10, 2015
@Homebrew Homebrew locked and limited conversation to collaborators Jul 11, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants