[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openembedded-core
Subject:    Re: [OE-core] [PATCH v2] glib-2.0: fix ptest failures
From:       "Burton, Ross" <ross.burton () intel ! com>
Date:       2019-04-30 13:16:15
Message-ID: CAJTo0LZnTfeC66-BWnhhXrb6w9eRsUH-hMostVgtKLY4mxwA3A () mail ! gmail ! com
[Download RAW message or body]

On Tue, 30 Apr 2019 at 08:45, <mingli.yu@windriver.com> wrote:
> From: Mingli Yu <Mingli.Yu@windriver.com>
> 
> * Install 7 translation packages to fix 7 ptest
> failures such as:
> > GLib:ERROR:../glib-2.58.3/glib/tests/gdatetime.c:1477:test_non_utf8_printf: \
> > assertion failed (__p == ("10\346\234\210")): ("Oct" == "10\346\234\210")
> > GLib:ERROR:../glib-2.58.3/glib/tests/gdatetime.c:1716:test_month_names: assertion \
> > failed (p_casefold == (o_casefold)): ("nov" == "stu")
> > GLib:ERROR:../glib-2.58.3/glib/tests/gdatetime.c:1731:test_month_names: assertion \
> > failed (p_casefold == (o_casefold)): ("jul" == "liep.")

Would be better if we had a meta-package to install 'all
translations', but we don't.

> * Increase the timeout to 15 minutes as the default
> timeout which is 5 minutes is not enough to fix below error:
> > Executing: glib/actions.test
> > Test timed out after 300 seconds
> > cleaning up pid 13050
> > FAIL: glib/actions.test (Child process killed by signal 9)

Looks like we're working around a test case that is going to fail
because it is timing out.  Let's investigate why it is taking so long
instead.

> * There are 240+ ptest cases for glib-2.0 and one
> case is gdatetime.test. And there are 53 sub-cases
> under gdatetime.test.
> 
> Among of the 53 sub-cases of gdatetime.test, there
> is one sub-case named test_month_names.
> 
> For test_month_names case, there are also 64 cases
> against it. Comment 2 cases of test_month_names
> as a workaround to fix the below test_month_names
> failure as the issue also exists in the latest version:
> > GLib:ERROR:../glib-2.60.1/glib/tests
> /gdatetime.c:1702:test_month_names: assertion failed (p_casefold == (o_casefold)): \
> ("\316\261\317\205\316\263" == "\316\261\317\215\316\263") | \
> GLib:ERROR:../glib-2.60.1/glib/tests/gdatetime.c:1765:test_month_names: assertion \
> failed (p_casefold == (o_casefold)): ("\320\274\320\260\321\217" == \
> "\320\274\320\260\320\271") 
> Have reported the above issues to upstream as
> https://gitlab.gnome.org/GNOME/glib/issues/1759
> and as the failure even exists on the latest version 2.60.1
> until now, so comment it as a workaround until the
> upstream fixs it.

The test is failing and we don't know why, so it should continue to
fail.  We don't get to 100% pass rate by disabling every test that
fails.

Can you drop the timeout and datetime test removal, leaving just the
translation additions?

Ross
-- 
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[prev in list] [next in list] [prev in thread] [next in thread]