Skip to content

updating list of core addons#182

Open
tpltnt wants to merge 9 commits into
openframeworks:masterfrom
tpltnt:remove-old-addons
Open

updating list of core addons#182
tpltnt wants to merge 9 commits into
openframeworks:masterfrom
tpltnt:remove-old-addons

Conversation

@tpltnt

@tpltnt tpltnt commented Feb 21, 2016

Copy link
Copy Markdown
Collaborator

Hi there,

I updated the list of core addons as pointed out in #177.

Cheers,
tpltnt

@tpltnt

tpltnt commented Apr 17, 2016

Copy link
Copy Markdown
Collaborator Author

ping

@arturoc

arturoc commented Apr 17, 2016

Copy link
Copy Markdown
Member

hey, thanks but not sure if ofxAndroid, ofxiOS, ofxEmscripten and ofxUnitTests should be mentioned at all. ofxUnitTests is not even included in the download and the rest are not meant to be used as normal addons but get automatically included when compiling for the specific platform

@tpltnt

tpltnt commented Apr 17, 2016

Copy link
Copy Markdown
Collaborator Author

This PR is to address the (now) non-existent addons, as mentioned in #177

@tpltnt

tpltnt commented Apr 22, 2016

Copy link
Copy Markdown
Collaborator Author

ping

@tpltnt

tpltnt commented May 25, 2016

Copy link
Copy Markdown
Collaborator Author

ping

@tpltnt

tpltnt commented Oct 30, 2016

Copy link
Copy Markdown
Collaborator Author

@arturoc the context is about the core addons, not whether or not they should/need to be used.

@tpltnt

tpltnt commented Oct 30, 2016

Copy link
Copy Markdown
Collaborator Author

This PR lingers for more than half a years for no good reason. It should either be merged or closed. #177 should be addressed if this PR is not found suitable.

@ofZach

ofZach commented Oct 30, 2016

Copy link
Copy Markdown
Contributor

I don't think arturo's comments have been addressed. I am not sure it's helpful to consider something like ofxIos in the same list as something like ofxOsc, since they can't be used in the same way. One is used automatically to extend OF per platform (with no user control) the other is a choice in the project generator, etc.

@tpltnt

tpltnt commented Oct 30, 2016

Copy link
Copy Markdown
Collaborator Author

I removed the remarks on removed ofxiOS, ofxAndroid, ofxEmscripten and ofxUnitTests.

@edap

edap commented Oct 25, 2018

Copy link
Copy Markdown
Member

@tpltnt is this PR still actual?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants