forked from project-chip/connectedhomeip
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Fix some configuration differences between GN & automake (project-chi…
…p#1582) The standalone project configs now apply unless specifically overriden, which matches configure's behavior. Previously the project configs were disabled by default, so the defaults inline in CHIPConfig.h applied. Note there are at 3 sources of defaults here which lead to this confusion: (1) The defaults used by scripts/build/bootstrap.sh, which calls configure with very few arguments. This uses the standalone project configs and builds tests. (2) The defaults used by config/standalone/standalone-chip.mk, which calls configure and overrides a few arguments. This uses the standalone project configs, but disables some components, and does not build tests. (3) The defaults specified directly in CHIPConfig.h et al. These defaults may have no effect because configure always overrides them using BuildConfig.h. GN currently doesn't use BuildConfig.h, but uses the command line to get the same result.
- Loading branch information
Showing
3 changed files
with
21 additions
and
8 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters