Attention is currently required from: fixeria. msuraev has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-trx/+/31840 ) Change subject: tests: Makefile.am: move -I flags from AM_CFLAGS to AM_CPPFLAGS ...................................................................... Patch Set 1: Code-Review+1 (1 comment) Commit Message: https://gerrit.osmocom.org/c/osmo-trx/+/31840/comment/9780142d_749e64ce PS1, Line 11: * AM_CXXFLAGS is for C++ specific compiler flags like '-Wall'. I think having some actually C++ specific example would make commit message more convincing :) -- To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/31840 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-trx Gerrit-Branch: master Gerrit-Change-Id: I7b3504a01e3350834b35c42d8d76d5d88d84a4b9 Gerrit-Change-Number: 31840 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: msuraev <msuraev@sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy@sysmocom.de> Gerrit-Comment-Date: Sun, 12 Mar 2023 19:58:58 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment