add addon_config.mk, update README about dylib setup on osx

nariakiiwatani / ofxNDI

NewTek NDI SDK Wrapper for openFrameworks.

https://ndi.tv/

MIT License

103 stars 19 forks source link

add addon_config.mk, update README about dylib setup on osx #2

Closed 2bbb closed 6 years ago

2bbb commented 6 years ago

maybe this is related to #1

2bbb commented 6 years ago

// also i added config about vs, ios // but, i tested on osx only... // maybe those are basic templates and need update.

nariakiiwatani commented 6 years ago

Thank you! Actually I can build and run without these settings, but anyway it'll be good to add addon_config.mk. I will merge this PR after I test with mac, ios, and vs. I hope this helps #1, too.

nariakiiwatani commented 6 years ago

about addon_config.mk

Couldn't build on Windows; There should be ofxNDI/src and ofxNDI/utils also to ADDON_INCLUDES.
ADDON_DATA seems not to be working.

I don't know much about ADDON_DATA but the addon_config.mk seems to do nothing more than default PG settings.

about readme

I couldn't see any change in result with or without the additional settings. Does it depend on Xcode, macOS, or oF's versions?

Xcode 8.3.3
macOS 10.12.2
oF 0.9.8

Could you note about your environments for my information?

2bbb commented 6 years ago

@nariakiiwatani

about win

i will test with win/vs.

my env

Xcode 9.3
10.13.3
oF 0.9.8 osx release (PG: v0.30.0)

2bbb commented 6 years ago

about win

i misunderstood about copy dll. we need to copy to bin/, not bin/data/. ADDON_DATA is not correct option. (and not works...) maybe copying manually is correct solution...

2bbb commented 6 years ago

config works on windows. with no need to copy files manually.

env:

windows 10
vs2017
of_v0.9.8_vs_release

nariakiiwatani commented 6 years ago

seems perfect! thank you so much!