Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature : Windows support #3

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

elliotwoods
Copy link
Contributor

Here's a version with Windows support.

I have to say, it wasn't very straight-forwards since it's 4 years old (I managed to avoid installing Windows 7 by not following the instructions for building, and hacking it so libaudiodecoder can sit directly in the oF project rather than a separate DLL).

Often I avoid these kinds of hacks vehemently, because it means it's difficult to keep in-tow with any updates. But libaudiodecoder hasn't been updated for 4 years so that isn't really an issue here.

Notes:

  • Folder structure is rearranged to match oF style (osx example may need rebuilding)
  • Currently you need to remove the audiodecodermediafoundation.cpp on the OSX side and audiodecodercoreaudio.cpp on the windows side from your project
  • I've put all the extra settings for Windows in the .props file (which includes some required WMF libraries which need to be linked).
  • Note that the windows version doesn't support mono audio files (OSX doubles them to be stereo, Windows reports them as empty)

Remote portaudio (we don't need it apparently)
@elliotwoods
Copy link
Contributor Author

image

image

@noetits
Copy link

noetits commented Sep 11, 2016

Hello elliotwoods,
How can I get a "snapshot" of this version with windows support ?

Thanks a lot

EDIT :
image

I tried this and downloaded the zip to try the example, removed files for mac in the project to have the same files you show in your screenshot. But I still have the unresolved external symbols ...

@elliotwoods
Copy link
Contributor Author

elliotwoods commented Sep 11, 2016

Perhaps you're missing the .props file. Did you add that in the property
manager to your project?
It links some Windows libs which are required

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.

2 participants