pkg-config: add <includedir>/libzrtcpp to CFLAGS

[tmatth]

Also remove broken lines that are not required.

[tmatth]

Fixes issue #16

[Lekensteyn]

LGTM.

[wernerd]

This fix will be in the next release.

Werner

Am 11.01.2015 um 14:59 schrieb Peter Wu:

LGTM.

—
Reply to this email directly or view it on GitHub #17 (comment).

Werner Dittmann
email: [email protected]
cell: +49 173 44 37 659
PGP key: 82EF5E8B

[keszybz]

This patch does not seem right:

• previous way to include files (#include <libzrtpcpp/xxx.h>) still works, but now one can also use #include <xxx.h>. Some people will use one some the other.
• more importantly, not all header files installed by libzrtpcpp are properly namespaced, and this might lead to collisions (for this to work they should all start with Zrtp prefix or something). For example there's common/osSpecifics.h which is too generic to put in the include path.

I think it would be much better to internally include all headers using #include "libzrtpcpp/xxx.h", and only allow users of this library to use #include <libzrtpcpp/xxx.h>, and not modify cflags.