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

Unable to build ff_meters with latest version of Juce #12

Open
mediawizrd opened this issue Jun 19, 2020 · 6 comments
Open

Unable to build ff_meters with latest version of Juce #12

mediawizrd opened this issue Jun 19, 2020 · 6 comments

Comments

@mediawizrd
Copy link

I get errors on the following header files when I try to build ff_meters using the latest version of the Juce library:

#include <juce_audio_basics/juce_audio_basics.h>
#include <juce_gui_basics/juce_gui_basics.h>
#include <juce_events/juce_events.h>

@ffAudio
Copy link
Owner

ffAudio commented Jun 19, 2020

Thanks for using ff_meters.

Do you mind sharing a little more detail?

  • what's the error message?
  • which latest juce: develop branch, master branch or juce6?
  • are the module paths valid?

I must admit, that currently I don't have an active project using the meters, but I'm still happy to fix it, if we can get to the bottom of it.

@ffAudio
Copy link
Owner

ffAudio commented Jun 21, 2020

I tried and didn't have a problem. If it was related to issue #13 this should be fixed now. Can you please try again? (you will need to resave with Projucer or use whatever method to update your project files)

@mediawizrd
Copy link
Author

mediawizrd commented Jun 22, 2020 via email

@ffAudio
Copy link
Owner

ffAudio commented Jun 22, 2020

Yes, a buildlog would help massively. Unfortunately your attachment didn't go through, I guess github doesn't support attachments. Just copy it in the post...

Thanks!

@mediawizrd
Copy link
Author

mediawizrd commented Jun 22, 2020 via email

@ffAudio
Copy link
Owner

ffAudio commented Jun 22, 2020

Oh, I see. I wasn't aware that maybe_unused is a C++17 feature.
You can easily fix that by switching your project to C++17, but I will move back to use ignoreUnused() so you don't have to switch just for that feature...

Thanks for the report

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

No branches or pull requests

2 participants