test-pkg: by default only test a subset of toolchains
Get series with b4
b4 am [email protected]Versions newest first · linked by author and title
- v2 Latest3 patches 0 repliesNew
- v11 of 5 patches 0 repliesSuperseded
Changes in v2 from the cover letter
- Drop the first two patches, since they have been merged.
- In the CSV list of toolchains, add better comments to explain the two "sections" of the toolchain list, as suggested by Yann E. Morin.
- In the test-pkg script changes themselves:
- Keep alphabetic sorting in the definition of the 'o' variable that contains the list of short options. Suggested by Yann E. Morin. 'h' for help remains a special case and therefore remains at the first position.
- Keep alphabetic sorting when handling options. Here as well, --help remains first, as it already was. Suggested by Yann E. Morin.
- Use spaces and not tabs for indentation. Suggested by Yann E. Morin.
Patches in v2 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/3 | toolchain-configs.csv: re-organize for test-pkgsupport/ | +26 | 1 reply | |
| 1/3 | toolchain-configs.csv: re-organize for test-pkg | +0 | 0 replies | |
| 2/3 | test-pkg: test a subset of toolchains by default, add -a and -n optionsutils/ | +49 | 2 replies | |
| 2/3 | test-pkg: test a subset of toolchains by default, add -a and -n options | +0 | 1 reply | |
| 3/3 | docs/manual: update the documentation about test-pkgdocs/ | +8 | 1 reply |
Reviews trailers per patch
All 5 patches have a Reviewed-by or Acked-by.
| Patch | Matt Weber | Yann E. MORIN |
|---|---|---|
| 1/3 | R-b | |
| 1/3 | R-b | |
| 2/3 | T-b | R-b |
| 2/3 | T-b | R-b |
| 3/3 | A-b |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [Buildroot] [PATCH v2 0/3] test-pkg: by default only test a subset of toolchains
During the Buildroot Developers meeting at ELCE 2017, we discussed how to make test-pkg more usable for our contributors. Our idea is that test-pkg should not test all toolchains by default, but only a smaller subset of "interesting" toolchains, that exhibit the most common problems we encounter with new packages.
This series does that by:
- Adjust the toolchain CSV file to have the interesting toolchains first, adding comments and empty lines along the way to clarify what is going on.
- Update test-pkg to test only the first 7 toolchains, add a --all option to test all toolchains, and a --number option to test only the first N toolchains.
- Update the Buildroot manual accordingly.
Thanks!
Thomas
Changes since v1:
- Drop the first two patches, since they have been merged.
- In the CSV list of toolchains, add better comments to explain the two "sections" of the toolchain list, as suggested by Yann E. Morin.
- In the test-pkg script changes themselves:
- Keep alphabetic sorting in the definition of the 'o' variable that contains the list of short options. Suggested by Yann E. Morin. 'h' for help remains a special case and therefore remains at the first position.
- Keep alphabetic sorting when handling options. Here as well, --help remains first, as it already was. Suggested by Yann E. Morin.
- Use spaces and not tabs for indentation. Suggested by Yann E. Morin.
Thomas Petazzoni (3):
toolchain-configs.csv: re-organize for test-pkg
test-pkg: test a subset of toolchains by default, add -a and -n
options
docs/manual: update the documentation about test-pkg
docs/manual/adding-packages-tips.txt | 11 +++-- .../autobuild/toolchain-configs.csv | 32 ++++++++++--- utils/test-pkg | 56 +++++++++++++++++++--- 3 files changed, 83 insertions(+), 16 deletions(-)
Signature · 1 line
2.14.3