You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
audiomanager.pc.in sets Cflags to include AUDIOMANAGER_CMAKE_CXX_FLAGS, which in turn brings in the -pedantic flag. This is intrusive to the user of that pkg-config, enforcing to provide own code to be compatible - and sometimes impossible if using incompatible 3rd party code.
What is the intention of including extraneous flags here? -pedantic should not be enforced in the interface.
The text was updated successfully, but these errors were encountered:
This is most likely a relict from the early days of the audiomanager. Although the AM itself claims conformity with ISO C++, it seems reasonable not strictly requiring this from users of the accompanying libraries.
So I propose removing this flag from the projects exports (both pkgconfig and cmake).
audiomanager.pc.in sets Cflags to include AUDIOMANAGER_CMAKE_CXX_FLAGS, which in turn brings in the -pedantic flag. This is intrusive to the user of that pkg-config, enforcing to provide own code to be compatible - and sometimes impossible if using incompatible 3rd party code.
What is the intention of including extraneous flags here? -pedantic should not be enforced in the interface.
The text was updated successfully, but these errors were encountered: