Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move build/config to config #426

Merged

Conversation

rwalker-apple
Copy link
Contributor

@rwalker-apple rwalker-apple commented Apr 20, 2020

Problem

build/ is a mix of git files and build output

Summary of Changes

moving git files out of build

fixes #405

@rwalker-apple rwalker-apple changed the title Move build config to config Move build/config to config Apr 20, 2020
@@ -80,7 +80,7 @@ AC_CONFIG_AUX_DIR([build/autoconf])
# Tell autoconf where to find auxilliary M4 macros
# Note that nlbuild-autotools has macros in two subdirs, for local vs third-party scripts
#
AC_CONFIG_MACRO_DIRS([build/autoconf/m4 third_party/nlbuild-autotools/repo/third_party/autoconf/m4 third_party/nlbuild-autotools/repo/autoconf/m4])
AC_CONFIG_MACRO_DIRS([build/autoconf/m4 autoconf/m4 third_party/nlbuild-autotools/repo/third_party/autoconf/m4 third_party/nlbuild-autotools/repo/autoconf/m4])
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the extra autoconf/m4 needed for?

Copy link
Contributor Author

@rwalker-apple rwalker-apple Apr 20, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we've written some CHIP-specific autoconf helpers that we add to the mix. those live in autoconf/m4
prior to this change, we were mixing those in with the autoconf-supplied --missing ones.

@woody-apple woody-apple merged commit aeec6c5 into project-chip:master Apr 20, 2020
@rwalker-apple rwalker-apple deleted the move-build_config-to-config branch April 20, 2020 17:06
rerasool pushed a commit to SiliconLabs/matter that referenced this pull request Dec 19, 2022
…e due to outdated log statement

Merge in WMN_TOOLS/matter from bugfix/wifi_window_app_build_failure_fix to RC_1.0.2-1.0

Squashed commit of the following:

commit 32d5a05f52f1cb7a177582085a9d4f75be0b6bc1
Author: Manav Mehta <Manav.Mehta@silabs.com>
Date:   Tue Dec 13 04:31:14 2022 +0000

    Fixed build failure due to outdated log statement
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
7 participants