Attention is currently required from: pespin.
fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-hlr/+/31785 )
Change subject: tests/*/Makefile.am: move -I to AM_CPPFLAGS ......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
Pau I saw your opinion, thanks, no need to post it again on each and every patch.
It's the matter of consistency (most project do have `-I` in `AM_CPPFLAGS`, with some exceptions I am fixing) and clarity: `AM_CPPFLAGS` is for preprocessor flags, `AM_CFLAGS` is for compiler flags. The fact that gcc combines both and allows to pass arguments in arbitrary order does not mean we should put stuff in wrong variables.