* [PATCH][dunfell] perl: fix ptest test count
@ 2020-10-05 8:38 Lee Chee Yang
2020-10-05 9:37 ` [OE-core] " Alexander Kanavin
2020-10-05 14:07 ` Steve Sakoman
0 siblings, 2 replies; 4+ messages in thread
From: Lee Chee Yang @ 2020-10-05 8:38 UTC (permalink / raw)
To: openembedded-core
From: Chee Yang Lee <chee.yang.lee@intel.com>
patche added in commit c2710903859a5545aa9dd56251f61667ee8d80bd
failed ptest perl.t/re/pat. The patch updates incorrect test cases
count causes the ptest to fail.
864 test cases before apply the patch while the patch added only 4
test cases so it should be 868 instead of 873.
ptest-runner log :-
PASS: t/re/overload
# Looks like you planned 873 tests but ran 868.
FAIL: t/re/pat
PASS: t/re/pat_advanced
[YOCTO #14071]
Signed-off-by: Chee Yang Lee <chee.yang.lee@intel.com>
---
meta/recipes-devtools/perl/files/CVE-2020-12723.patch | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/meta/recipes-devtools/perl/files/CVE-2020-12723.patch b/meta/recipes-devtools/perl/files/CVE-2020-12723.patch
index ad195cffab..d25f4d4a74 100644
--- a/meta/recipes-devtools/perl/files/CVE-2020-12723.patch
+++ b/meta/recipes-devtools/perl/files/CVE-2020-12723.patch
@@ -265,7 +265,7 @@ index 6a868f4..2869b58 100644
skip_all_without_unicode_tables();
-plan tests => 864; # Update this when adding/deleting tests.
-+plan tests => 873; # Update this when adding/deleting tests.
++plan tests => 868; # Update this when adding/deleting tests.
run_tests() unless caller;
--
2.17.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [OE-core] [PATCH][dunfell] perl: fix ptest test count
2020-10-05 8:38 [PATCH][dunfell] perl: fix ptest test count Lee Chee Yang
@ 2020-10-05 9:37 ` Alexander Kanavin
2020-10-05 9:53 ` Lee Chee Yang
2020-10-05 14:07 ` Steve Sakoman
1 sibling, 1 reply; 4+ messages in thread
From: Alexander Kanavin @ 2020-10-05 9:37 UTC (permalink / raw)
To: Lee Chee Yang; +Cc: OE-core
[-- Attachment #1: Type: text/plain, Size: 1549 bytes --]
Hello,
Is this fix meant for dunfell, or master or both? The bug is filed against
master.
Alex
On Mon, 5 Oct 2020 at 10:41, Lee Chee Yang <chee.yang.lee@intel.com> wrote:
> From: Chee Yang Lee <chee.yang.lee@intel.com>
>
> patche added in commit c2710903859a5545aa9dd56251f61667ee8d80bd
> failed ptest perl.t/re/pat. The patch updates incorrect test cases
> count causes the ptest to fail.
> 864 test cases before apply the patch while the patch added only 4
> test cases so it should be 868 instead of 873.
>
> ptest-runner log :-
> PASS: t/re/overload
> # Looks like you planned 873 tests but ran 868.
> FAIL: t/re/pat
> PASS: t/re/pat_advanced
>
> [YOCTO #14071]
>
> Signed-off-by: Chee Yang Lee <chee.yang.lee@intel.com>
> ---
> meta/recipes-devtools/perl/files/CVE-2020-12723.patch | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta/recipes-devtools/perl/files/CVE-2020-12723.patch
> b/meta/recipes-devtools/perl/files/CVE-2020-12723.patch
> index ad195cffab..d25f4d4a74 100644
> --- a/meta/recipes-devtools/perl/files/CVE-2020-12723.patch
> +++ b/meta/recipes-devtools/perl/files/CVE-2020-12723.patch
> @@ -265,7 +265,7 @@ index 6a868f4..2869b58 100644
> skip_all_without_unicode_tables();
>
> -plan tests => 864; # Update this when adding/deleting tests.
> -+plan tests => 873; # Update this when adding/deleting tests.
> ++plan tests => 868; # Update this when adding/deleting tests.
>
> run_tests() unless caller;
>
> --
> 2.17.1
>
>
>
>
>
[-- Attachment #2: Type: text/html, Size: 2131 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [OE-core] [PATCH][dunfell] perl: fix ptest test count
2020-10-05 9:37 ` [OE-core] " Alexander Kanavin
@ 2020-10-05 9:53 ` Lee Chee Yang
0 siblings, 0 replies; 4+ messages in thread
From: Lee Chee Yang @ 2020-10-05 9:53 UTC (permalink / raw)
To: Alexander Kanavin; +Cc: OE-core
[-- Attachment #1: Type: text/plain, Size: 1848 bytes --]
For Dunfell only
From: Alexander Kanavin <alex.kanavin@gmail.com>
Sent: Monday, 5 October, 2020 5:37 PM
To: Lee, Chee Yang <chee.yang.lee@intel.com>
Cc: OE-core <openembedded-core@lists.openembedded.org>
Subject: Re: [OE-core] [PATCH][dunfell] perl: fix ptest test count
Hello,
Is this fix meant for dunfell, or master or both? The bug is filed against master.
Alex
On Mon, 5 Oct 2020 at 10:41, Lee Chee Yang <chee.yang.lee@intel.com<mailto:chee.yang.lee@intel.com>> wrote:
From: Chee Yang Lee <chee.yang.lee@intel.com<mailto:chee.yang.lee@intel.com>>
patche added in commit c2710903859a5545aa9dd56251f61667ee8d80bd
failed ptest perl.t/re/pat. The patch updates incorrect test cases
count causes the ptest to fail.
864 test cases before apply the patch while the patch added only 4
test cases so it should be 868 instead of 873.
ptest-runner log :-
PASS: t/re/overload
# Looks like you planned 873 tests but ran 868.
FAIL: t/re/pat
PASS: t/re/pat_advanced
[YOCTO #14071]
Signed-off-by: Chee Yang Lee <chee.yang.lee@intel.com<mailto:chee.yang.lee@intel.com>>
---
meta/recipes-devtools/perl/files/CVE-2020-12723.patch | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/meta/recipes-devtools/perl/files/CVE-2020-12723.patch b/meta/recipes-devtools/perl/files/CVE-2020-12723.patch
index ad195cffab..d25f4d4a74 100644
--- a/meta/recipes-devtools/perl/files/CVE-2020-12723.patch
+++ b/meta/recipes-devtools/perl/files/CVE-2020-12723.patch
@@ -265,7 +265,7 @@ index 6a868f4..2869b58 100644
skip_all_without_unicode_tables();
-plan tests => 864; # Update this when adding/deleting tests.
-+plan tests => 873; # Update this when adding/deleting tests.
++plan tests => 868; # Update this when adding/deleting tests.
run_tests() unless caller;
--
2.17.1
[-- Attachment #2: Type: text/html, Size: 4916 bytes --]
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [OE-core] [PATCH][dunfell] perl: fix ptest test count
2020-10-05 8:38 [PATCH][dunfell] perl: fix ptest test count Lee Chee Yang
2020-10-05 9:37 ` [OE-core] " Alexander Kanavin
@ 2020-10-05 14:07 ` Steve Sakoman
1 sibling, 0 replies; 4+ messages in thread
From: Steve Sakoman @ 2020-10-05 14:07 UTC (permalink / raw)
To: Lee Chee Yang; +Cc: Patches and discussions about the oe-core layer
Thanks! I tested this fix on Friday and can confirm that it fixes the problem.
I'll include this in the next batch of dunfell patches.
Steve
On Sun, Oct 4, 2020 at 10:41 PM Lee Chee Yang <chee.yang.lee@intel.com> wrote:
>
> From: Chee Yang Lee <chee.yang.lee@intel.com>
>
> patche added in commit c2710903859a5545aa9dd56251f61667ee8d80bd
> failed ptest perl.t/re/pat. The patch updates incorrect test cases
> count causes the ptest to fail.
> 864 test cases before apply the patch while the patch added only 4
> test cases so it should be 868 instead of 873.
>
> ptest-runner log :-
> PASS: t/re/overload
> # Looks like you planned 873 tests but ran 868.
> FAIL: t/re/pat
> PASS: t/re/pat_advanced
>
> [YOCTO #14071]
>
> Signed-off-by: Chee Yang Lee <chee.yang.lee@intel.com>
> ---
> meta/recipes-devtools/perl/files/CVE-2020-12723.patch | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta/recipes-devtools/perl/files/CVE-2020-12723.patch b/meta/recipes-devtools/perl/files/CVE-2020-12723.patch
> index ad195cffab..d25f4d4a74 100644
> --- a/meta/recipes-devtools/perl/files/CVE-2020-12723.patch
> +++ b/meta/recipes-devtools/perl/files/CVE-2020-12723.patch
> @@ -265,7 +265,7 @@ index 6a868f4..2869b58 100644
> skip_all_without_unicode_tables();
>
> -plan tests => 864; # Update this when adding/deleting tests.
> -+plan tests => 873; # Update this when adding/deleting tests.
> ++plan tests => 868; # Update this when adding/deleting tests.
>
> run_tests() unless caller;
>
> --
> 2.17.1
>
>
>
>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-10-05 14:07 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-05 8:38 [PATCH][dunfell] perl: fix ptest test count Lee Chee Yang
2020-10-05 9:37 ` [OE-core] " Alexander Kanavin
2020-10-05 9:53 ` Lee Chee Yang
2020-10-05 14:07 ` Steve Sakoman
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.