Skip to content

Allow specification of std=c++17 for tests #219

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

Closed
jgfoster opened this issue Nov 17, 2020 · 3 comments
Closed

Allow specification of std=c++17 for tests #219

jgfoster opened this issue Nov 17, 2020 · 3 comments

Comments

@jgfoster
Copy link
Member

I see that cpp_library.rb:401 specifies -std=c++0x as the version of C++. I'd like to use some features that are in std=c++17 (I want the filesystem features). Would it be reasonable to bump up to version 17 or make it optional?

@ianfixes
Copy link
Collaborator

I'm actually falling somewhere in between, see #146

Summary: I think this project should be using the same hard-coded c++ standard as the official Arduino compiler(s). It's not doing that yet.

So to be blunt, in my opinion this is either a duplicate of #146 or a won't-fix.

@jgfoster
Copy link
Member Author

We found it pretty difficult to mock SD in a cross-platform way without having filesystem. So, I'll make a gentle appeal for at least an option to use a later library at your own risk. But I'm sure we can find a work-around if needed...

@jgfoster
Copy link
Member Author

I've had further thoughts on this and will present them in a new issue.

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