On Wed, Feb 15, 2023 at 9:18 AM Martin Jansa <martin.ja...@gmail.com> wrote: > > * I've noticed this, because patchreview.py reports Malformed > Upstream-Status in this one now, but the QA check in insane.bbclass > wasn't reporting it before > > * the reason why insane.bbclass doesn't report it is because: > SRC_URI:append:ptest = " > file://0001-Don-t-let-host-enviroment-to-decide-if-a-test-is-bui.patch" > doesn't work and probably never did as "ptest" is not an override > and patch-status-core checks only the .patch files which are really > in SRC_URI. > > * I've added it directly to SRC_URI and it also didn't apply: > http://errors.yoctoproject.org/Errors/Details/689955/ > as the "endif" at the end was removed in 1.10.0 version: > > https://github.com/libssh2/libssh2/commit/ecd6a74e44562797a1e92186ad4a402c5641720e#diff-32103f666ff2fb42b025a47ccf1b959bbcc6db89f217e5943b1de73c81a4f9db > > after updating the patch to apply I got expected patch-status-core failure: > http://errors.yoctoproject.org/Errors/Details/689956/ > ERROR: QA Issue: Malformed Upstream-Status in patch > > TOPDIR/openembedded-core/meta/recipes-support/libssh2/files/0001-Don-t-let-host-enviroment-to-decide-if-a-test-is-bui.patch > Please correct according to > https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines#Patch_Header_Recommendations:_Upstream-Status > : > Upstream-Status: Inappropriate[oe specific] [patch-status-core] > > * so the mystery is solved, but what should be do with this never-used > patch, I think we should just delete it together with > SRC_URI:append:ptest, but please Changqing Li confirm it's not needed > > * it was originally added by Changqing Li in meta-oe with: > > https://git.openembedded.org/meta-openembedded/commit/?id=d7aa7173405c3b36235af736cd31dbe110708787 > then imported to oe-core by Randy MacLeod with: > > https://git.openembedded.org/openembedded-core/commit/?id=57df134b1be56a688f41851e5ff014dd859c0bc3 > > Signed-off-by: Martin Jansa <martin.ja...@gmail.com> > --- > ...nviroment-to-decide-if-a-test-is-bui.patch | 22 ++++++------------- > 1 file changed, 7 insertions(+), 15 deletions(-) > > diff --git > a/meta/recipes-support/libssh2/files/0001-Don-t-let-host-enviroment-to-decide-if-a-test-is-bui.patch > > b/meta/recipes-support/libssh2/files/0001-Don-t-let-host-enviroment-to-decide-if-a-test-is-bui.patch > index b1204e49eb..e0000b2658 100644 > --- > a/meta/recipes-support/libssh2/files/0001-Don-t-let-host-enviroment-to-decide-if-a-test-is-bui.patch > +++ > b/meta/recipes-support/libssh2/files/0001-Don-t-let-host-enviroment-to-decide-if-a-test-is-bui.patch > @@ -1,24 +1,23 @@ > -From f6abce5ba41a412a247250dcd80e387e53474466 Mon Sep 17 00:00:00 2001 > -From: Your Name <y...@example.com> > +From 7ffa1773be8d262bed0f5c8bdcb4dd8f906de095 Mon Sep 17 00:00:00 2001 > +From: Changqing Li <changqing...@windriver.com> > Date: Mon, 28 Dec 2020 02:08:03 +0000 > Subject: [PATCH] Don't let host enviroment to decide if a test is build > > test ssh2.sh need sshd, for cross compile, we need it on target, so > don't use SSHD on host to decide weither to build a test > > -Upstream-Status: Inappropriate[oe specific] > +Upstream-Status: Inappropriate [oe specific]
Since we are already using a special token '[' to separate reason from status, I always wondered if having this space before '[' is something that could be ignored. > > Signed-off-by: Changqing Li <changqing...@windriver.com> > - > --- > - tests/Makefile.am | 6 +----- > - 1 file changed, 1 insertion(+), 5 deletions(-) > + tests/Makefile.am | 4 ---- > + 1 file changed, 4 deletions(-) > > diff --git a/tests/Makefile.am b/tests/Makefile.am > -index dc0922f..6cbc35d 100644 > +index 27ddc2d..13de8ab 100644 > --- a/tests/Makefile.am > +++ b/tests/Makefile.am > -@@ -1,16 +1,12 @@ > +@@ -3,16 +3,12 @@ SUBDIRS = ossfuzz > AM_CPPFLAGS = -I$(top_srcdir)/src -I$(top_srcdir)/include > -I$(top_builddir)/src > LDADD = ../src/libssh2.la > > @@ -35,10 +34,3 @@ index dc0922f..6cbc35d 100644 > check_PROGRAMS = $(ctests) > > TESTS_ENVIRONMENT = SSHD=$(SSHD) EXEEXT=$(EXEEXT) > -@@ -38,4 +34,4 @@ if OPENSSL > - # EXTRA_DIST += > test_public_key_auth_succeeds_with_correct_encrypted_ed25519_key.c > - # EXTRA_DIST += > test_public_key_auth_succeeds_with_correct_ed25519_key_from_mem.c > - EXTRA_DIST += test_public_key_auth_succeeds_with_correct_rsa_openssh_key.c > --endif > -\ No newline at end of file > -+endif > -- > 2.39.2 > > > >
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#177209): https://lists.openembedded.org/g/openembedded-core/message/177209 Mute This Topic: https://lists.openembedded.org/mt/96987532/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-