From: Daniel Turull <[email protected]> devtool upgrade's changelog extraction misattributed unrelated commits when upgrading recipes like nghttp2, whose ChangeLog is the literal output of `git log`, regenerated wholesale on every release.
Changes from v1: - Header-skipping in the structural diff now relies on the blank line git log always inserts before the commit message, instead of enumerating header prefixes with a regex. - The UTF-8 decode fix is split into its own patch and applied to both git-show call sites instead of just one. - A single-use helper that only sequenced two operations with no reuse benefit has been inlined into its caller. - The git-diff fallback is now guarded against UnicodeDecodeError too, so a bad byte in the diff hunk itself doesn't crash extraction. Changes from v2: - drop already merged patch. - as suggested in the review use subprocess.run() and drop helper function Daniel Turull (2): devtool: upgrade: call subprocess.run() instead of bb.process.run() devtool: upgrade: diff git-log-style changelogs by commit hash scripts/lib/devtool/upgrade.py | 68 +++++++++++++++++++++++++++++----- 1 file changed, 58 insertions(+), 10 deletions(-)
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#242986): https://lists.openembedded.org/g/openembedded-core/message/242986 Mute This Topic: https://lists.openembedded.org/mt/120639144/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
