commit | 5c853ef21b9247d2ad3d60736b3dea77b8681ed8 | [log] [tgz] |
---|---|---|
author | Ben Chan <benchan@chromium.org> | Fri Oct 05 00:05:37 2012 -0700 |
committer | Gerrit <chrome-bot@google.com> | Fri Oct 05 04:26:27 2012 -0700 |
tree | 394a170e992e4584f2afe84fba3eb8d2df840c76 | |
parent | 19f83971a35d7f2c7b08aae6eb0abdc8e52ba057 [diff] |
shill: cellular: Fix ModemManager header inclusion. The upstream ModemManager now disallows including ModemManager-enums.h and ModemManager-names.h directly. This CL fixes the ModemManager header inclusion as follows: - Use pkg-config in Makefile to specify CFLAGS for including ModemManager headers - Include ModemManager.h instead of ModemManager-enums.h and ModemManager-names.h - Move ModemManager1 related code from modem_manager.cc to a new modem_manager_1.cc in order to avoid including both mm-modem.h and ModemManager.h BUG=chromium-os:35075 TEST=Build and run unit tests. Change-Id: Ice180f673562c8b71500e2bec861de80f45148fc Reviewed-on: https://gerrit.chromium.org/gerrit/34734 Tested-by: Ben Chan <benchan@chromium.org> Reviewed-by: Darin Petkov <petkov@chromium.org> Commit-Ready: Ben Chan <benchan@chromium.org>