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.

Sign up for GitHub

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

Jump to bottom

Improve BUILDING.md instructions on how to use autotools. #143

Closed
turon opened this issue Mar 27, 2020 · 3 comments · Fixed by #178
Closed

Improve BUILDING.md instructions on how to use autotools. #143

turon opened this issue Mar 27, 2020 · 3 comments · Fixed by #178
Assignees
@turon
Labels
documentation Improvements or additions to documentation
Milestone
M4.1

Comments

@turon
Copy link
Contributor

turon commented Mar 27, 2020

Problem

Could someone with experience update the BUILDING.md doc with workflow best practices?

  • how to build the entire source,
  • how to build a specific set of sub-folders / modules, add new source files,
  • run & debug unit tests,
  • run and analyze code coverage results,
  • run formatting checks, etc.
This was referenced Mar 28, 2020
Closed
Closed
@rwalker-apple rwalker-apple added this to the M4.1 milestone Mar 30, 2020
@woody-apple woody-apple added the documentation Improvements or additions to documentation label Mar 31, 2020
This was referenced Mar 31, 2020
Merged
Merged
@turon
Copy link
Contributor Author

turon commented Mar 31, 2020

Just to calibrate this work, I'm linking to existing instructions for each item:

@rwalker-apple
Copy link
Contributor

maybe make checkboxes?

@rwalker-apple
Copy link
Contributor

nice links

@turon turon mentioned this issue Mar 31, 2020
Merged
woody-apple added a commit that referenced this issue Mar 31, 2020
@woody-apple
Merge pull request #178 from turon/pr/143/build-doc-improvement
0203b67 
[doc] Improve BUILDING.md per #143.
jmartinez-silabs referenced this issue in SiliconLabs/matter Oct 13, 2022
@mkardous-silabs @jmartinez-silabs
Pull request #143: In emberAfEndpointConfigure() change endpoint ID t…
fa2b9ec 
…o a uint16_t (project-chip#22998)

Merge in WMN_TOOLS/matter from cherry_pick/unify_build_fix to silabs_1.0

Squashed commit of the following:

commit 77f4fef74e58480c79bfb8dcfa67ffbdd57c61f3
Author: Sergei Lissianoi <54454955+selissia@users.noreply.github.com>
Date:   Mon Oct 3 16:51:28 2022 -0400

    In emberAfEndpointConfigure() change endpoint ID to a uint16_t (project-chip#22998)

    * Test added march 8 (project-chip#15957)

    * Added new manual scripts

    * Added Auto generated File

    * [OTA] Fix OTARequestorDriverImpl inclusion (project-chip#15981)

    * Regen to fix CI failures (project-chip#15990)

    * [ota] Store Default OTA Providers in flash (project-chip#15970)

    * [ota] Store Default OTA Providers in flash

    Store Default OTA Providers in flash each time the attribute
    is modified and load it back on the application startup.

    * Restyled by clang-format

    * Fix build and reduce flash usage

    Co-authored-by: Restyled.io <commits@restyled.io>

    * Remove merge artifacts

    * Cast boolean to uint8_t as setOnOffValue expects, fixes GCC 8.3 error

    * Make endpoint id a uint16_t as per spec

    * Restyled by clang-format

    * Update the signature of OnOffServer::setOnOffValue() to avoid casts

    * Fix printf argument and formatting

    Co-authored-by: kowsisoundhar12 <57476670+kowsisoundhar12@users.noreply.github.com>
    Co-authored-by: Carol Yang <clyang@apple.com>
    Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>
    Co-authored-by: Damian Królik <66667989+Damian-Nordic@users.noreply.github.com>
    Co-authored-by: Restyled.io <commits@restyled.io>
nipatel-silabs pushed a commit to nipatel-silabs/connectedhomeip that referenced this issue Oct 19, 2022
@mkardous-silabs @jmartinez-silabs
Pull request project-chip#143: In emberAfEndpointConfigure() change e…
665805a 
…ndpoint ID to a uint16_t (project-chip#22998)

Merge in WMN_TOOLS/matter from cherry_pick/unify_build_fix to silabs_1.0

Squashed commit of the following:

commit 77f4fef74e58480c79bfb8dcfa67ffbdd57c61f3
Author: Sergei Lissianoi <54454955+selissia@users.noreply.github.com>
Date:   Mon Oct 3 16:51:28 2022 -0400

    In emberAfEndpointConfigure() change endpoint ID to a uint16_t (project-chip#22998)

    * Test added march 8 (project-chip#15957)

    * Added new manual scripts

    * Added Auto generated File

    * [OTA] Fix OTARequestorDriverImpl inclusion (project-chip#15981)

    * Regen to fix CI failures (project-chip#15990)

    * [ota] Store Default OTA Providers in flash (project-chip#15970)

    * [ota] Store Default OTA Providers in flash

    Store Default OTA Providers in flash each time the attribute
    is modified and load it back on the application startup.

    * Restyled by clang-format

    * Fix build and reduce flash usage

    Co-authored-by: Restyled.io <commits@restyled.io>

    * Remove merge artifacts

    * Cast boolean to uint8_t as setOnOffValue expects, fixes GCC 8.3 error

    * Make endpoint id a uint16_t as per spec

    * Restyled by clang-format

    * Update the signature of OnOffServer::setOnOffValue() to avoid casts

    * Fix printf argument and formatting

    Co-authored-by: kowsisoundhar12 <57476670+kowsisoundhar12@users.noreply.github.com>
    Co-authored-by: Carol Yang <clyang@apple.com>
    Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>
    Co-authored-by: Damian Królik <66667989+Damian-Nordic@users.noreply.github.com>
    Co-authored-by: Restyled.io <commits@restyled.io>
mkardous-silabs referenced this issue in mkardous-silabs/connectedhomeip Nov 2, 2022
@mkardous-silabs @jmartinez-silabs
Pull request #143: In emberAfEndpointConfigure() change endpoint ID t…
12c147b 
…o a uint16_t (project-chip#22998)

Merge in WMN_TOOLS/matter from cherry_pick/unify_build_fix to silabs_1.0

Squashed commit of the following:

commit 77f4fef74e58480c79bfb8dcfa67ffbdd57c61f3
Author: Sergei Lissianoi <54454955+selissia@users.noreply.github.com>
Date:   Mon Oct 3 16:51:28 2022 -0400

    In emberAfEndpointConfigure() change endpoint ID to a uint16_t (project-chip#22998)

    * Test added march 8 (project-chip#15957)

    * Added new manual scripts

    * Added Auto generated File

    * [OTA] Fix OTARequestorDriverImpl inclusion (project-chip#15981)

    * Regen to fix CI failures (project-chip#15990)

    * [ota] Store Default OTA Providers in flash (project-chip#15970)

    * [ota] Store Default OTA Providers in flash

    Store Default OTA Providers in flash each time the attribute
    is modified and load it back on the application startup.

    * Restyled by clang-format

    * Fix build and reduce flash usage

    Co-authored-by: Restyled.io <commits@restyled.io>

    * Remove merge artifacts

    * Cast boolean to uint8_t as setOnOffValue expects, fixes GCC 8.3 error

    * Make endpoint id a uint16_t as per spec

    * Restyled by clang-format

    * Update the signature of OnOffServer::setOnOffValue() to avoid casts

    * Fix printf argument and formatting

    Co-authored-by: kowsisoundhar12 <57476670+kowsisoundhar12@users.noreply.github.com>
    Co-authored-by: Carol Yang <clyang@apple.com>
    Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>
    Co-authored-by: Damian Królik <66667989+Damian-Nordic@users.noreply.github.com>
    Co-authored-by: Restyled.io <commits@restyled.io>
rerasool referenced this issue in SiliconLabs/matter Nov 2, 2022
@mkardous-silabs @jmartinez-silabs
Pull request #143: In emberAfEndpointConfigure() change endpoint ID t…
4c9c27c 
…o a uint16_t (project-chip#22998)

Merge in WMN_TOOLS/matter from cherry_pick/unify_build_fix to silabs_1.0

Squashed commit of the following:

commit 77f4fef74e58480c79bfb8dcfa67ffbdd57c61f3
Author: Sergei Lissianoi <54454955+selissia@users.noreply.github.com>
Date:   Mon Oct 3 16:51:28 2022 -0400

    In emberAfEndpointConfigure() change endpoint ID to a uint16_t (project-chip#22998)

    * Test added march 8 (project-chip#15957)

    * Added new manual scripts

    * Added Auto generated File

    * [OTA] Fix OTARequestorDriverImpl inclusion (project-chip#15981)

    * Regen to fix CI failures (project-chip#15990)

    * [ota] Store Default OTA Providers in flash (project-chip#15970)

    * [ota] Store Default OTA Providers in flash

    Store Default OTA Providers in flash each time the attribute
    is modified and load it back on the application startup.

    * Restyled by clang-format

    * Fix build and reduce flash usage

    Co-authored-by: Restyled.io <commits@restyled.io>

    * Remove merge artifacts

    * Cast boolean to uint8_t as setOnOffValue expects, fixes GCC 8.3 error

    * Make endpoint id a uint16_t as per spec

    * Restyled by clang-format

    * Update the signature of OnOffServer::setOnOffValue() to avoid casts

    * Fix printf argument and formatting

    Co-authored-by: kowsisoundhar12 <57476670+kowsisoundhar12@users.noreply.github.com>
    Co-authored-by: Carol Yang <clyang@apple.com>
    Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>
    Co-authored-by: Damian Królik <66667989+Damian-Nordic@users.noreply.github.com>
    Co-authored-by: Restyled.io <commits@restyled.io>
chrisdecenzo pushed a commit that referenced this issue Apr 5, 2023
@sharadb-amazon
android/tv-casting-app: Adding ConfigurationManager as AppParameter (#…
e4faf94 
…143) (#25976)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@turon turon

Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
M4.1
Development

Successfully merging a pull request may close this issue.

[doc] Improve BUILDING.md per #143. turon/connectedhomeip-priv
3 participants
@turon @woody-apple @rwalker-apple