]> git.ipfire.org Git - thirdparty/openembedded/openembedded-core.git/commit
patchreview: use check_upstream_status() from oe.qa
authorMartin Jansa <Martin.Jansa@gmail.com>
Wed, 6 Nov 2024 14:54:49 +0000 (15:54 +0100)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Tue, 12 Nov 2024 11:19:42 +0000 (11:19 +0000)
commitf291c08ea6a95638c3ad1f70434678bd5e374195
tree7c36df2dc363225649499f8607236a342db3fc45
parent73bea15bb5d0f70b587d2ad1007cc0282652eed7
patchreview: use check_upstream_status() from oe.qa

* the idea was to reuse the same function as I've noticed that the
  QA check which was added to insane.bbclass in:
  https://git.openembedded.org/openembedded-core/commit/?id=76a685bfcf927593eac67157762a53259089ea8a
  is in some cases more strcit than scripts/contrib/patchreview.py

  To be honest I wasn't aware of scripts/contrib/patchreview.py
  existence when I've asked about moving check_upstream_status()
  to oe.qa in order to write standalone script just like
  patchreview.py

* I've sent this long time ago:
  https://lists.openembedded.org/g/openembedded-core/message/177207
  but didn't like the sys.path.append to find oe.qa much or the
  duplicated path to .patch file in the output, then I've forgot about
  it until today in https://github.com/OE4T/meta-tegra/pull/1749
  where checklayer found one more issue, which I haven't noticed
  with patchreview.py before (because I've accidentally used a version
  without this change). It's not perfect, but at least it will be
  consistent with checklayer and patch-status QA check.

Signed-off-by: Martin Jansa <martin.jansa@gmail.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
scripts/contrib/patchreview.py