This is the first split of patches coming from a previous series[1] I submitted. Compared to that version, the changes here are refactors limited to the core scripts/patchtest script itself, and do the following:
- Make sure that the user has enabled the meta-selftest layer before running patchtest, as otherwise selftest results can be misleading - Avoid duplicating code for test results generation - Remove excess whitespace, unused variables, redundant function calls, and unreachable code Note that this was driven partially by Claude Code (Sonnet 4.6 and 4.7), but I've gone back and reworked some of its changes before submission. Selftests look OK after these changes: |============================================================================ |Testsuite summary for patchtest |============================================================================ |# TOTAL: 38 |# XPASS: 18 |# XFAIL: 18 |# XSKIP: 2 |# PASS: 0 |# FAIL: 0 |# SKIP: 0 |# ERROR: 0 |============================================================================ [1]: https://lists.openembedded.org/g/openembedded-core/message/237047 Trevor Gamblin (7): scripts/patchtest: refactor results methods scripts/patchtest: clean up startTestRun() scripts/patchtest: simplify _runner() scripts/patchtest: simplify traceback logging, remove whitespace scripts/patchtest: simplify run() scripts/patchtest: clean up main() scripts/patchtest: check for meta-selftest scripts/patchtest | 147 ++++++++++++++++++++-------------------------- 1 file changed, 65 insertions(+), 82 deletions(-) -- 2.54.0
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#239101): https://lists.openembedded.org/g/openembedded-core/message/239101 Mute This Topic: https://lists.openembedded.org/mt/119873958/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
