omf2097 / openomf

One Must Fall 2097 Remake
http://www.openomf.org
MIT License
372 stars 40 forks source link

give apple OpenAL include better default location #270

Closed Sharnw closed 9 years ago

Sharnw commented 9 years ago

Hey mate,

Had to make a fix to build with OpenAL on osx. No probs if this is unnecessary but would love to contribute.

You're doing amazing work. Cheers!

katajakasa commented 9 years ago

@Vagabond I think this is for you :)

Vagabond commented 9 years ago

How did you install OpenAL on your machine? I think this might be installation-method dependant?

ericmhalvorsen commented 9 years ago

Saw the same issue, this fixed it for me. Compiled OpenAL from source downloaded from this link: http://kcat.strangesoft.net/openal.html

@Vagabond Any recommended method for MacOSX to avoid this issue?

Vagabond commented 9 years ago

I think it depends on how you installed openal, I think I installed it from macports or from homebrew.

Vagabond commented 9 years ago

I'm not sure how to solve this properly, maybe we can use cmake to look for the header files?

katajakasa commented 9 years ago

I wouldn't go changing the default seek location; i'd rather modify the cmake script to look from different places, indeed. Or, I'd rather accept a patch that does that :P

katajakasa commented 9 years ago

Possibly fixed in master with commit 23e5aa96bed9be006fac66e95bc2bb6268759213 Please test.