Re: [OE-core] [PATCH 07/17] gstreamer1.0: upgrade 1.18.5 -> 1.20.0

2022-02-17 Thread Alexander Kanavin 
Simply send the patch, make sure the commit message explains what it's
for, and we'll figure it out. RP regularly has such test-only patches
in master-next, or I can run dedicated ptest builds with it.

Alex

On Thu, 17 Feb 2022 at 00:41, Jose Quaresma  wrote:
>
>
>
> Alexander Kanavin  escreveu no dia quarta, 9/02/2022 
> à(s) 10:34:
>>
>> Yes, it seems to be sporadic. Here's a sample fail:
>>
>> https://autobuilder.yoctoproject.org/typhoon/#/builders/81/builds/3083
>> https://autobuilder.yocto.io/pub/non-release/20220208-14/testresults/qemux86-64-ptest/gstreamer1.0.log
>
>
>
> I ask for help upstream but I will try to replicate this with GST_DEBUG 
> enabled.
> Which is the best option to send a patch not for merge but only to run on the 
> autobuilder to generate some logs?
>
> https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/778
>
>>
>>
>> Alex
>>
>> On Wed, 9 Feb 2022 at 10:30, Claudius Heine  wrote:
>> >
>> > Hi Jose
>> >
>> > On 2022-02-09 10:15, Jose Quaresma wrote:
>> > > Hi Alexander,
>> > >
>> > > Can you please clarify in what target arch is this happening and I can
>> > > try to replicate it with qemu?
>> >
>> > I think we are hitting:
>> > https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/778
>> >
>> > This testcase seems to be flaky and IIUC upstream marked it as such in
>> > their test infrastructure.
>> >
>> > In my patchset, I added a patch to disable that testcase.
>> >
>> > regards,
>> > Claudius
>> >
>> > >
>> > > It works on my side on qemux86-64.
>> > >
>> > > root@qemux86-64:~# /usr/libexec/installed-tests/gstreamer/gst_gstbin.sh
>> > > + CK_DEFAULT_TIMEOUT=20
>> > > + GST_PLUGIN_LOADING_WHITELIST=gstreamer
>> > > + GST_REGISTRY=/home/root/.cache/gstreamer-1.0/gst_gstbin.registry
>> > > + GST_STATE_IGNORE_ELEMENTS=
>> > > + exec /usr/libexec/installed-tests/gstreamer/gst_gstbin
>> > > Running suite(s): GstBin
>> > > 100%: Checks: 26, Failures: 0, Errors: 0
>> > > Check suite gst_bin ran in 6.239s (tests failed: 0)
>> > >
>> > >
>> > > Alexander Kanavin > > > > escreveu no dia terça, 8/02/2022 à(s)
>> > > 22:01:
>> > >
>> > > Unfortunately this patchset (which is now in master) does not address
>> > > the ptest fail. Please resolve this as soon as possible.
>> > >
>> > > + exec /usr/libexec/installed-tests/gstreamer/gst_gstbin
>> > > Running suite(s): GstBin
>> > > Executing: gstreamer/gst_gstbin.test
>> > > Executing: gstreamer/gst_gstbin.test
>> > > 
>> > > Executing: gstreamer/gst_gstbin.test
>> > > Executing: gstreamer/gst_gstbin.test
>> > > Test timed out after 300 seconds
>> > > FAIL: gstreamer/gst_gstbin.test (Child process killed by signal 9)
>> > >
>> > > Alex
>> > >
>> > > On Mon, 7 Feb 2022 at 02:08, Jose Quaresma > > > > wrote:
>> > >  >
>> > >  > - License update:
>> > >  >   The code actually is LGPLv2.1 but the COPYING file is wrong.
>> > >  >
>> > > 
>> > > https://gitlab.freedesktop.org/gstreamer/gstreamer/-/commit/d157b23fb0ce175b167611a4cb019599d9017042
>> > > 
>> > > 
>> > >  >
>> > >  > - Drop backported patches
>> > >  >   0001-gst-gstpluginloader.c-when-env-var-is-set-do-not-fal.patch
>> > >  >   0002-Remove-unused-valgrind-detection.patch
>> > >  >   0003-tests-seek-Don-t-use-too-strict-timeout-for-validati.patch
>> > >  >
>> > >  > - Update submitted patches
>> > >  >   0001-tests-respect-the-idententaion-used-in-meson.patch
>> > >  >   0002-tests-add-support-for-install-the-tests.patch
>> > >  >   0003-tests-use-a-dictionaries-for-environment.patch
>> > >  >   0004-tests-add-helper-script-to-run-the-installed_tests.patch
>> > >  >
>> > >  > Signed-off-by: Jose Quaresma > > > >
>> > >  > ---
>> > >  >  ...der.c-when-env-var-is-set-do-not-fal.patch |  69 ---
>> > >  >  ...pect-the-idententaion-used-in-meson.patch} |  14 +--
>> > >  >  ...002-Remove-unused-valgrind-detection.patch | 112
>> > > --
>> > >  >  ...s-add-support-for-install-the-tests.patch} |  67 ++-
>> > >  >  ...-use-too-strict-timeout-for-validati.patch |  32 -
>> > >  >  ...-use-a-dictionaries-for-environment.patch} |  28 +++--
>> > >  >  ...er-script-to-run-the-installed_tests.patch |  74 
>> > >  >  ...-the-environment-for-installed_tests.patch |  58 -
>> > >  >  ...er1.0_1.18.5.bb  =>
>> > > gstreamer1.0_1.20.0.bb } |  17 ++-
>> > >  >  9 files changed, 136 insertions(+), 335 deletions(-)
>> > >  >  delete mode 100644
>> > > 
>> > > meta/recipes-multimedia/gstreamer/gstreamer1.0/0001-gst-gstpluginloader.c-when-env-var-is-set-do-not-fal.patch
>> > >  >  rename
>> > >

Re: [OE-core] [PATCH 07/17] gstreamer1.0: upgrade 1.18.5 -> 1.20.0

2022-02-16 Thread Jose Quaresma 
Claudius Heine  escreveu no dia quarta, 9/02/2022 à(s) 09:30:

> Hi Jose
>
> On 2022-02-09 10:15, Jose Quaresma wrote:
> > Hi Alexander,
> >
> > Can you please clarify in what target arch is this happening and I can
> > try to replicate it with qemu?
>
> I think we are hitting:
> https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/778
>
> This testcase seems to be flaky and IIUC upstream marked it as such in
> their test infrastructure.
>
> In my patchset, I added a patch to disable that testcase.
>

Hi Claudius,

Your patch Is a possible solution but I think we need to investigate the
root cause
of the test failure and send a fix for that if possible.


> regards,
> Claudius
>
> >
> > It works on my side on qemux86-64.
> >
> > root@qemux86-64:~# /usr/libexec/installed-tests/gstreamer/gst_gstbin.sh
> > + CK_DEFAULT_TIMEOUT=20
> > + GST_PLUGIN_LOADING_WHITELIST=gstreamer
> > + GST_REGISTRY=/home/root/.cache/gstreamer-1.0/gst_gstbin.registry
> > + GST_STATE_IGNORE_ELEMENTS=
> > + exec /usr/libexec/installed-tests/gstreamer/gst_gstbin
> > Running suite(s): GstBin
> > 100%: Checks: 26, Failures: 0, Errors: 0
> > Check suite gst_bin ran in 6.239s (tests failed: 0)
> >
> >
> > Alexander Kanavin  > > escreveu no dia terça, 8/02/2022 à(s)
> > 22:01:
> >
> > Unfortunately this patchset (which is now in master) does not address
> > the ptest fail. Please resolve this as soon as possible.
> >
> > + exec /usr/libexec/installed-tests/gstreamer/gst_gstbin
> > Running suite(s): GstBin
> > Executing: gstreamer/gst_gstbin.test
> > Executing: gstreamer/gst_gstbin.test
> > 
> > Executing: gstreamer/gst_gstbin.test
> > Executing: gstreamer/gst_gstbin.test
> > Test timed out after 300 seconds
> > FAIL: gstreamer/gst_gstbin.test (Child process killed by signal 9)
> >
> > Alex
> >
> > On Mon, 7 Feb 2022 at 02:08, Jose Quaresma  > > wrote:
> >  >
> >  > - License update:
> >  >   The code actually is LGPLv2.1 but the COPYING file is wrong.
> >  >
> >
> https://gitlab.freedesktop.org/gstreamer/gstreamer/-/commit/d157b23fb0ce175b167611a4cb019599d9017042
> > <
> https://gitlab.freedesktop.org/gstreamer/gstreamer/-/commit/d157b23fb0ce175b167611a4cb019599d9017042
> >
> >  >
> >  > - Drop backported patches
> >  >   0001-gst-gstpluginloader.c-when-env-var-is-set-do-not-fal.patch
> >  >   0002-Remove-unused-valgrind-detection.patch
> >  >   0003-tests-seek-Don-t-use-too-strict-timeout-for-validati.patch
> >  >
> >  > - Update submitted patches
> >  >   0001-tests-respect-the-idententaion-used-in-meson.patch
> >  >   0002-tests-add-support-for-install-the-tests.patch
> >  >   0003-tests-use-a-dictionaries-for-environment.patch
> >  >   0004-tests-add-helper-script-to-run-the-installed_tests.patch
> >  >
> >  > Signed-off-by: Jose Quaresma  > >
> >  > ---
> >  >  ...der.c-when-env-var-is-set-do-not-fal.patch |  69 ---
> >  >  ...pect-the-idententaion-used-in-meson.patch} |  14 +--
> >  >  ...002-Remove-unused-valgrind-detection.patch | 112
> > --
> >  >  ...s-add-support-for-install-the-tests.patch} |  67 ++-
> >  >  ...-use-too-strict-timeout-for-validati.patch |  32 -
> >  >  ...-use-a-dictionaries-for-environment.patch} |  28 +++--
> >  >  ...er-script-to-run-the-installed_tests.patch |  74 
> >  >  ...-the-environment-for-installed_tests.patch |  58 -
> >  >  ...er1.0_1.18.5.bb  =>
> > gstreamer1.0_1.20.0.bb } |  17 ++-
> >  >  9 files changed, 136 insertions(+), 335 deletions(-)
> >  >  delete mode 100644
> >
>  
> meta/recipes-multimedia/gstreamer/gstreamer1.0/0001-gst-gstpluginloader.c-when-env-var-is-set-do-not-fal.patch
> >  >  rename
> >
>  
> meta/recipes-multimedia/gstreamer/gstreamer1.0/{0004-tests-respect-the-idententaion-used-in-meson.patch
> > => 0001-tests-respect-the-idententaion-used-in-meson.patch} (71%)
> >  >  delete mode 100644
> >
>  
> meta/recipes-multimedia/gstreamer/gstreamer1.0/0002-Remove-unused-valgrind-detection.patch
> >  >  rename
> >
>  
> meta/recipes-multimedia/gstreamer/gstreamer1.0/{0005-tests-add-support-for-install-the-tests.patch
> > => 0002-tests-add-support-for-install-the-tests.patch} (65%)
> >  >  delete mode 100644
> >
>  
> meta/recipes-multimedia/gstreamer/gstreamer1.0/0003-tests-seek-Don-t-use-too-strict-timeout-for-validati.patch
> >  >  rename
> >
>  
> meta/recipes-multimedia/gstreamer/gstreamer1.0/{0006-tests-use-a-dictionaries-for-environment.patch
> > => 0003-tests-use-a-dictionaries-for-environment.patch} (61%)
> >  >  create mode 100644
> >
>  
>

Re: [OE-core] [PATCH 07/17] gstreamer1.0: upgrade 1.18.5 -> 1.20.0

2022-02-16 Thread Jose Quaresma 
Alexander Kanavin  escreveu no dia quarta,
9/02/2022 à(s) 10:34:

> Yes, it seems to be sporadic. Here's a sample fail:
>
> https://autobuilder.yoctoproject.org/typhoon/#/builders/81/builds/3083
>
> https://autobuilder.yocto.io/pub/non-release/20220208-14/testresults/qemux86-64-ptest/gstreamer1.0.log



I ask for help upstream but I will try to replicate this with GST_DEBUG
enabled.
Which is the best option to send a patch not for merge but only to run on
the autobuilder to generate some logs?

https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/778


>
> Alex
>
> On Wed, 9 Feb 2022 at 10:30, Claudius Heine  wrote:
> >
> > Hi Jose
> >
> > On 2022-02-09 10:15, Jose Quaresma wrote:
> > > Hi Alexander,
> > >
> > > Can you please clarify in what target arch is this happening and I can
> > > try to replicate it with qemu?
> >
> > I think we are hitting:
> > https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/778
> >
> > This testcase seems to be flaky and IIUC upstream marked it as such in
> > their test infrastructure.
> >
> > In my patchset, I added a patch to disable that testcase.
> >
> > regards,
> > Claudius
> >
> > >
> > > It works on my side on qemux86-64.
> > >
> > > root@qemux86-64:~#
> /usr/libexec/installed-tests/gstreamer/gst_gstbin.sh
> > > + CK_DEFAULT_TIMEOUT=20
> > > + GST_PLUGIN_LOADING_WHITELIST=gstreamer
> > > + GST_REGISTRY=/home/root/.cache/gstreamer-1.0/gst_gstbin.registry
> > > + GST_STATE_IGNORE_ELEMENTS=
> > > + exec /usr/libexec/installed-tests/gstreamer/gst_gstbin
> > > Running suite(s): GstBin
> > > 100%: Checks: 26, Failures: 0, Errors: 0
> > > Check suite gst_bin ran in 6.239s (tests failed: 0)
> > >
> > >
> > > Alexander Kanavin  > > > escreveu no dia terça, 8/02/2022 à(s)
> > > 22:01:
> > >
> > > Unfortunately this patchset (which is now in master) does not
> address
> > > the ptest fail. Please resolve this as soon as possible.
> > >
> > > + exec /usr/libexec/installed-tests/gstreamer/gst_gstbin
> > > Running suite(s): GstBin
> > > Executing: gstreamer/gst_gstbin.test
> > > Executing: gstreamer/gst_gstbin.test
> > > 
> > > Executing: gstreamer/gst_gstbin.test
> > > Executing: gstreamer/gst_gstbin.test
> > > Test timed out after 300 seconds
> > > FAIL: gstreamer/gst_gstbin.test (Child process killed by signal 9)
> > >
> > > Alex
> > >
> > > On Mon, 7 Feb 2022 at 02:08, Jose Quaresma <
> quaresma.j...@gmail.com
> > > > wrote:
> > >  >
> > >  > - License update:
> > >  >   The code actually is LGPLv2.1 but the COPYING file is wrong.
> > >  >
> > >
> https://gitlab.freedesktop.org/gstreamer/gstreamer/-/commit/d157b23fb0ce175b167611a4cb019599d9017042
> > > <
> https://gitlab.freedesktop.org/gstreamer/gstreamer/-/commit/d157b23fb0ce175b167611a4cb019599d9017042
> >
> > >  >
> > >  > - Drop backported patches
> > >  >
>  0001-gst-gstpluginloader.c-when-env-var-is-set-do-not-fal.patch
> > >  >   0002-Remove-unused-valgrind-detection.patch
> > >  >
>  0003-tests-seek-Don-t-use-too-strict-timeout-for-validati.patch
> > >  >
> > >  > - Update submitted patches
> > >  >   0001-tests-respect-the-idententaion-used-in-meson.patch
> > >  >   0002-tests-add-support-for-install-the-tests.patch
> > >  >   0003-tests-use-a-dictionaries-for-environment.patch
> > >  >   0004-tests-add-helper-script-to-run-the-installed_tests.patch
> > >  >
> > >  > Signed-off-by: Jose Quaresma  > > >
> > >  > ---
> > >  >  ...der.c-when-env-var-is-set-do-not-fal.patch |  69 ---
> > >  >  ...pect-the-idententaion-used-in-meson.patch} |  14 +--
> > >  >  ...002-Remove-unused-valgrind-detection.patch | 112
> > > --
> > >  >  ...s-add-support-for-install-the-tests.patch} |  67 ++-
> > >  >  ...-use-too-strict-timeout-for-validati.patch |  32 -
> > >  >  ...-use-a-dictionaries-for-environment.patch} |  28 +++--
> > >  >  ...er-script-to-run-the-installed_tests.patch |  74
> 
> > >  >  ...-the-environment-for-installed_tests.patch |  58 -
> > >  >  ...er1.0_1.18.5.bb  =>
> > > gstreamer1.0_1.20.0.bb } |  17 ++-
> > >  >  9 files changed, 136 insertions(+), 335 deletions(-)
> > >  >  delete mode 100644
> > >
>  
> meta/recipes-multimedia/gstreamer/gstreamer1.0/0001-gst-gstpluginloader.c-when-env-var-is-set-do-not-fal.patch
> > >  >  rename
> > >
>  
> meta/recipes-multimedia/gstreamer/gstreamer1.0/{0004-tests-respect-the-idententaion-used-in-meson.patch
> > > => 0001-tests-respect-the-idententaion-used-in-meson.patch} (71%)
> > >  >  delete mode 100644
> > >
>  
> meta/recipes-multimedia/gstreamer/gstreamer1.0/0002-Remove-unused-valgrind-detection.patch
> > >  >  rename
> > >
>  
>

Re: [OE-core] [PATCH 07/17] gstreamer1.0: upgrade 1.18.5 -> 1.20.0

2022-02-16 Thread Jose Quaresma 
Peter Kjellerstedt  escreveu no dia quarta,
9/02/2022 à(s) 19:09:

> > -Original Message-
> > From: openembedded-core@lists.openembedded.org  > c...@lists.openembedded.org> On Behalf Of Jose Quaresma
> > Sent: den 7 februari 2022 02:08
> > To: openembedded-core@lists.openembedded.org
> > Cc: Jose Quaresma 
> > Subject: [OE-core] [PATCH 07/17] gstreamer1.0: upgrade 1.18.5 -> 1.20.0
> >
> > - License update:
> >   The code actually is LGPLv2.1 but the COPYING file is wrong.
> >
> https://gitlab.freedesktop.org/gstreamer/gstreamer/-/commit/d157b23fb0ce175b167611a4cb019599d9017042
>
> Since the COPYING file for the GStreamer sub-projects now states
> that the used license is LGPL-2.1, shouldn't the LICENSE for these
> recipes be updated to say "LGPL-2.1-or-later"?
>
> As for -bad and -ugly, they should really have individual LICENSE:foo
> for each generated package, so that only the packages that are affected
> by the GPL license includes it, and the others only specify LGPL.
>
> //Peter
>
>
I have sent a patch for that with your suggestion.
I change the LICENSE_FLAGS based on the new gpl as well.
https://patchwork.yoctoproject.org/project/oe-core/patch/20220216232803.1868265-1-quaresma.j...@gmail.com/

Jose

-- 
Best regards,

José Quaresma

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#161787): 
https://lists.openembedded.org/g/openembedded-core/message/161787
Mute This Topic: https://lists.openembedded.org/mt/88962313/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [OE-core] [PATCH 07/17] gstreamer1.0: upgrade 1.18.5 -> 1.20.0

2022-02-09 Thread Peter Kjellerstedt 
> -Original Message-
> From: openembedded-core@lists.openembedded.org  c...@lists.openembedded.org> On Behalf Of Jose Quaresma
> Sent: den 7 februari 2022 02:08
> To: openembedded-core@lists.openembedded.org
> Cc: Jose Quaresma 
> Subject: [OE-core] [PATCH 07/17] gstreamer1.0: upgrade 1.18.5 -> 1.20.0
> 
> - License update:
>   The code actually is LGPLv2.1 but the COPYING file is wrong.
>   
> https://gitlab.freedesktop.org/gstreamer/gstreamer/-/commit/d157b23fb0ce175b167611a4cb019599d9017042

Since the COPYING file for the GStreamer sub-projects now states 
that the used license is LGPL-2.1, shouldn't the LICENSE for these 
recipes be updated to say "LGPL-2.1-or-later"?

As for -bad and -ugly, they should really have individual LICENSE:foo 
for each generated package, so that only the packages that are affected 
by the GPL license includes it, and the others only specify LGPL.

//Peter


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#161586): 
https://lists.openembedded.org/g/openembedded-core/message/161586
Mute This Topic: https://lists.openembedded.org/mt/88962313/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [OE-core] [PATCH 07/17] gstreamer1.0: upgrade 1.18.5 -> 1.20.0

2022-02-09 Thread Alexander Kanavin 
Yes, it seems to be sporadic. Here's a sample fail:

https://autobuilder.yoctoproject.org/typhoon/#/builders/81/builds/3083
https://autobuilder.yocto.io/pub/non-release/20220208-14/testresults/qemux86-64-ptest/gstreamer1.0.log

Alex

On Wed, 9 Feb 2022 at 10:30, Claudius Heine  wrote:
>
> Hi Jose
>
> On 2022-02-09 10:15, Jose Quaresma wrote:
> > Hi Alexander,
> >
> > Can you please clarify in what target arch is this happening and I can
> > try to replicate it with qemu?
>
> I think we are hitting:
> https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/778
>
> This testcase seems to be flaky and IIUC upstream marked it as such in
> their test infrastructure.
>
> In my patchset, I added a patch to disable that testcase.
>
> regards,
> Claudius
>
> >
> > It works on my side on qemux86-64.
> >
> > root@qemux86-64:~# /usr/libexec/installed-tests/gstreamer/gst_gstbin.sh
> > + CK_DEFAULT_TIMEOUT=20
> > + GST_PLUGIN_LOADING_WHITELIST=gstreamer
> > + GST_REGISTRY=/home/root/.cache/gstreamer-1.0/gst_gstbin.registry
> > + GST_STATE_IGNORE_ELEMENTS=
> > + exec /usr/libexec/installed-tests/gstreamer/gst_gstbin
> > Running suite(s): GstBin
> > 100%: Checks: 26, Failures: 0, Errors: 0
> > Check suite gst_bin ran in 6.239s (tests failed: 0)
> >
> >
> > Alexander Kanavin  > > escreveu no dia terça, 8/02/2022 à(s)
> > 22:01:
> >
> > Unfortunately this patchset (which is now in master) does not address
> > the ptest fail. Please resolve this as soon as possible.
> >
> > + exec /usr/libexec/installed-tests/gstreamer/gst_gstbin
> > Running suite(s): GstBin
> > Executing: gstreamer/gst_gstbin.test
> > Executing: gstreamer/gst_gstbin.test
> > 
> > Executing: gstreamer/gst_gstbin.test
> > Executing: gstreamer/gst_gstbin.test
> > Test timed out after 300 seconds
> > FAIL: gstreamer/gst_gstbin.test (Child process killed by signal 9)
> >
> > Alex
> >
> > On Mon, 7 Feb 2022 at 02:08, Jose Quaresma  > > wrote:
> >  >
> >  > - License update:
> >  >   The code actually is LGPLv2.1 but the COPYING file is wrong.
> >  >
> > 
> > https://gitlab.freedesktop.org/gstreamer/gstreamer/-/commit/d157b23fb0ce175b167611a4cb019599d9017042
> > 
> > 
> >  >
> >  > - Drop backported patches
> >  >   0001-gst-gstpluginloader.c-when-env-var-is-set-do-not-fal.patch
> >  >   0002-Remove-unused-valgrind-detection.patch
> >  >   0003-tests-seek-Don-t-use-too-strict-timeout-for-validati.patch
> >  >
> >  > - Update submitted patches
> >  >   0001-tests-respect-the-idententaion-used-in-meson.patch
> >  >   0002-tests-add-support-for-install-the-tests.patch
> >  >   0003-tests-use-a-dictionaries-for-environment.patch
> >  >   0004-tests-add-helper-script-to-run-the-installed_tests.patch
> >  >
> >  > Signed-off-by: Jose Quaresma  > >
> >  > ---
> >  >  ...der.c-when-env-var-is-set-do-not-fal.patch |  69 ---
> >  >  ...pect-the-idententaion-used-in-meson.patch} |  14 +--
> >  >  ...002-Remove-unused-valgrind-detection.patch | 112
> > --
> >  >  ...s-add-support-for-install-the-tests.patch} |  67 ++-
> >  >  ...-use-too-strict-timeout-for-validati.patch |  32 -
> >  >  ...-use-a-dictionaries-for-environment.patch} |  28 +++--
> >  >  ...er-script-to-run-the-installed_tests.patch |  74 
> >  >  ...-the-environment-for-installed_tests.patch |  58 -
> >  >  ...er1.0_1.18.5.bb  =>
> > gstreamer1.0_1.20.0.bb } |  17 ++-
> >  >  9 files changed, 136 insertions(+), 335 deletions(-)
> >  >  delete mode 100644
> > 
> > meta/recipes-multimedia/gstreamer/gstreamer1.0/0001-gst-gstpluginloader.c-when-env-var-is-set-do-not-fal.patch
> >  >  rename
> > 
> > meta/recipes-multimedia/gstreamer/gstreamer1.0/{0004-tests-respect-the-idententaion-used-in-meson.patch
> > => 0001-tests-respect-the-idententaion-used-in-meson.patch} (71%)
> >  >  delete mode 100644
> > 
> > meta/recipes-multimedia/gstreamer/gstreamer1.0/0002-Remove-unused-valgrind-detection.patch
> >  >  rename
> > 
> > meta/recipes-multimedia/gstreamer/gstreamer1.0/{0005-tests-add-support-for-install-the-tests.patch
> > => 0002-tests-add-support-for-install-the-tests.patch} (65%)
> >  >  delete mode 100644
> > 
> > meta/recipes-multimedia/gstreamer/gstreamer1.0/0003-tests-seek-Don-t-use-too-strict-timeout-for-validati.patch
> >  >  rename
> > 
> > meta/recipes-multimedia/gstreamer/gstreamer1.0/{0006-tests-use-a-dictionaries-for-environment.patch
> > => 0003-tests-use-a-dictionaries-for-environment.patch} (61%)
> >  >  create mode 100644
> > 
> >

Re: [OE-core] [PATCH 07/17] gstreamer1.0: upgrade 1.18.5 -> 1.20.0

2022-02-09 Thread Claudius Heine 

Hi Jose

On 2022-02-09 10:15, Jose Quaresma wrote:

Hi Alexander,

Can you please clarify in what target arch is this happening and I can 
try to replicate it with qemu?


I think we are hitting:
https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/778

This testcase seems to be flaky and IIUC upstream marked it as such in 
their test infrastructure.


In my patchset, I added a patch to disable that testcase.

regards,
Claudius



It works on my side on qemux86-64.

root@qemux86-64:~# /usr/libexec/installed-tests/gstreamer/gst_gstbin.sh
+ CK_DEFAULT_TIMEOUT=20
+ GST_PLUGIN_LOADING_WHITELIST=gstreamer
+ GST_REGISTRY=/home/root/.cache/gstreamer-1.0/gst_gstbin.registry
+ GST_STATE_IGNORE_ELEMENTS=
+ exec /usr/libexec/installed-tests/gstreamer/gst_gstbin
Running suite(s): GstBin
100%: Checks: 26, Failures: 0, Errors: 0
Check suite gst_bin ran in 6.239s (tests failed: 0)


Alexander Kanavin > escreveu no dia terça, 8/02/2022 à(s) 
22:01:


Unfortunately this patchset (which is now in master) does not address
the ptest fail. Please resolve this as soon as possible.

+ exec /usr/libexec/installed-tests/gstreamer/gst_gstbin
Running suite(s): GstBin
Executing: gstreamer/gst_gstbin.test
Executing: gstreamer/gst_gstbin.test

Executing: gstreamer/gst_gstbin.test
Executing: gstreamer/gst_gstbin.test
Test timed out after 300 seconds
FAIL: gstreamer/gst_gstbin.test (Child process killed by signal 9)

Alex

On Mon, 7 Feb 2022 at 02:08, Jose Quaresma mailto:quaresma.j...@gmail.com>> wrote:
 >
 > - License update:
 >   The code actually is LGPLv2.1 but the COPYING file is wrong.
 >

https://gitlab.freedesktop.org/gstreamer/gstreamer/-/commit/d157b23fb0ce175b167611a4cb019599d9017042


 >
 > - Drop backported patches
 >   0001-gst-gstpluginloader.c-when-env-var-is-set-do-not-fal.patch
 >   0002-Remove-unused-valgrind-detection.patch
 >   0003-tests-seek-Don-t-use-too-strict-timeout-for-validati.patch
 >
 > - Update submitted patches
 >   0001-tests-respect-the-idententaion-used-in-meson.patch
 >   0002-tests-add-support-for-install-the-tests.patch
 >   0003-tests-use-a-dictionaries-for-environment.patch
 >   0004-tests-add-helper-script-to-run-the-installed_tests.patch
 >
 > Signed-off-by: Jose Quaresma mailto:quaresma.j...@gmail.com>>
 > ---
 >  ...der.c-when-env-var-is-set-do-not-fal.patch |  69 ---
 >  ...pect-the-idententaion-used-in-meson.patch} |  14 +--
 >  ...002-Remove-unused-valgrind-detection.patch | 112
--
 >  ...s-add-support-for-install-the-tests.patch} |  67 ++-
 >  ...-use-too-strict-timeout-for-validati.patch |  32 -
 >  ...-use-a-dictionaries-for-environment.patch} |  28 +++--
 >  ...er-script-to-run-the-installed_tests.patch |  74 
 >  ...-the-environment-for-installed_tests.patch |  58 -
 >  ...er1.0_1.18.5.bb  =>
gstreamer1.0_1.20.0.bb } |  17 ++-
 >  9 files changed, 136 insertions(+), 335 deletions(-)
 >  delete mode 100644

meta/recipes-multimedia/gstreamer/gstreamer1.0/0001-gst-gstpluginloader.c-when-env-var-is-set-do-not-fal.patch
 >  rename

meta/recipes-multimedia/gstreamer/gstreamer1.0/{0004-tests-respect-the-idententaion-used-in-meson.patch
=> 0001-tests-respect-the-idententaion-used-in-meson.patch} (71%)
 >  delete mode 100644

meta/recipes-multimedia/gstreamer/gstreamer1.0/0002-Remove-unused-valgrind-detection.patch
 >  rename

meta/recipes-multimedia/gstreamer/gstreamer1.0/{0005-tests-add-support-for-install-the-tests.patch
=> 0002-tests-add-support-for-install-the-tests.patch} (65%)
 >  delete mode 100644

meta/recipes-multimedia/gstreamer/gstreamer1.0/0003-tests-seek-Don-t-use-too-strict-timeout-for-validati.patch
 >  rename

meta/recipes-multimedia/gstreamer/gstreamer1.0/{0006-tests-use-a-dictionaries-for-environment.patch
=> 0003-tests-use-a-dictionaries-for-environment.patch} (61%)
 >  create mode 100644

meta/recipes-multimedia/gstreamer/gstreamer1.0/0004-tests-add-helper-script-to-run-the-installed_tests.patch
 >  delete mode 100644

meta/recipes-multimedia/gstreamer/gstreamer1.0/0007-tests-install-the-environment-for-installed_tests.patch
 >  rename meta/recipes-multimedia/gstreamer/{gstreamer1.0_1.18.5.bb
 => gstreamer1.0_1.20.0.bb
} (76%)
 >
 > diff --git

a/meta/recipes-multimedia/gstreamer/gstreamer1.0/0001-gst-gstpluginloader.c-when-env-var-is-set-do-not-fal.patch

b/meta/recipes-multimedia/gstreamer/gstreamer1.0/0001-gst-gstpluginloader.c-when-env-var-is-set-do-not-fal.patch
 >

Re: [OE-core] [PATCH 07/17] gstreamer1.0: upgrade 1.18.5 -> 1.20.0

2022-02-09 Thread Jose Quaresma 
Hi Alexander,

Can you please clarify in what target arch is this happening and I can try
to replicate it with qemu?

It works on my side on qemux86-64.

root@qemux86-64:~# /usr/libexec/installed-tests/gstreamer/gst_gstbin.sh
+ CK_DEFAULT_TIMEOUT=20
+ GST_PLUGIN_LOADING_WHITELIST=gstreamer
+ GST_REGISTRY=/home/root/.cache/gstreamer-1.0/gst_gstbin.registry
+ GST_STATE_IGNORE_ELEMENTS=
+ exec /usr/libexec/installed-tests/gstreamer/gst_gstbin
Running suite(s): GstBin
100%: Checks: 26, Failures: 0, Errors: 0
Check suite gst_bin ran in 6.239s (tests failed: 0)


Alexander Kanavin  escreveu no dia terça, 8/02/2022
à(s) 22:01:

> Unfortunately this patchset (which is now in master) does not address
> the ptest fail. Please resolve this as soon as possible.
>
> + exec /usr/libexec/installed-tests/gstreamer/gst_gstbin
> Running suite(s): GstBin
> Executing: gstreamer/gst_gstbin.test
> Executing: gstreamer/gst_gstbin.test
> 
> Executing: gstreamer/gst_gstbin.test
> Executing: gstreamer/gst_gstbin.test
> Test timed out after 300 seconds
> FAIL: gstreamer/gst_gstbin.test (Child process killed by signal 9)
>
> Alex
>
> On Mon, 7 Feb 2022 at 02:08, Jose Quaresma 
> wrote:
> >
> > - License update:
> >   The code actually is LGPLv2.1 but the COPYING file is wrong.
> >
> https://gitlab.freedesktop.org/gstreamer/gstreamer/-/commit/d157b23fb0ce175b167611a4cb019599d9017042
> >
> > - Drop backported patches
> >   0001-gst-gstpluginloader.c-when-env-var-is-set-do-not-fal.patch
> >   0002-Remove-unused-valgrind-detection.patch
> >   0003-tests-seek-Don-t-use-too-strict-timeout-for-validati.patch
> >
> > - Update submitted patches
> >   0001-tests-respect-the-idententaion-used-in-meson.patch
> >   0002-tests-add-support-for-install-the-tests.patch
> >   0003-tests-use-a-dictionaries-for-environment.patch
> >   0004-tests-add-helper-script-to-run-the-installed_tests.patch
> >
> > Signed-off-by: Jose Quaresma 
> > ---
> >  ...der.c-when-env-var-is-set-do-not-fal.patch |  69 ---
> >  ...pect-the-idententaion-used-in-meson.patch} |  14 +--
> >  ...002-Remove-unused-valgrind-detection.patch | 112 --
> >  ...s-add-support-for-install-the-tests.patch} |  67 ++-
> >  ...-use-too-strict-timeout-for-validati.patch |  32 -
> >  ...-use-a-dictionaries-for-environment.patch} |  28 +++--
> >  ...er-script-to-run-the-installed_tests.patch |  74 
> >  ...-the-environment-for-installed_tests.patch |  58 -
> >  ...er1.0_1.18.5.bb => gstreamer1.0_1.20.0.bb} |  17 ++-
> >  9 files changed, 136 insertions(+), 335 deletions(-)
> >  delete mode 100644
> meta/recipes-multimedia/gstreamer/gstreamer1.0/0001-gst-gstpluginloader.c-when-env-var-is-set-do-not-fal.patch
> >  rename
> meta/recipes-multimedia/gstreamer/gstreamer1.0/{0004-tests-respect-the-idententaion-used-in-meson.patch
> => 0001-tests-respect-the-idententaion-used-in-meson.patch} (71%)
> >  delete mode 100644
> meta/recipes-multimedia/gstreamer/gstreamer1.0/0002-Remove-unused-valgrind-detection.patch
> >  rename
> meta/recipes-multimedia/gstreamer/gstreamer1.0/{0005-tests-add-support-for-install-the-tests.patch
> => 0002-tests-add-support-for-install-the-tests.patch} (65%)
> >  delete mode 100644
> meta/recipes-multimedia/gstreamer/gstreamer1.0/0003-tests-seek-Don-t-use-too-strict-timeout-for-validati.patch
> >  rename
> meta/recipes-multimedia/gstreamer/gstreamer1.0/{0006-tests-use-a-dictionaries-for-environment.patch
> => 0003-tests-use-a-dictionaries-for-environment.patch} (61%)
> >  create mode 100644
> meta/recipes-multimedia/gstreamer/gstreamer1.0/0004-tests-add-helper-script-to-run-the-installed_tests.patch
> >  delete mode 100644
> meta/recipes-multimedia/gstreamer/gstreamer1.0/0007-tests-install-the-environment-for-installed_tests.patch
> >  rename meta/recipes-multimedia/gstreamer/{gstreamer1.0_1.18.5.bb =>
> gstreamer1.0_1.20.0.bb} (76%)
> >
> > diff --git
> a/meta/recipes-multimedia/gstreamer/gstreamer1.0/0001-gst-gstpluginloader.c-when-env-var-is-set-do-not-fal.patch
> b/meta/recipes-multimedia/gstreamer/gstreamer1.0/0001-gst-gstpluginloader.c-when-env-var-is-set-do-not-fal.patch
> > deleted file mode 100644
> > index 23ebd5c600..00
> > ---
> a/meta/recipes-multimedia/gstreamer/gstreamer1.0/0001-gst-gstpluginloader.c-when-env-var-is-set-do-not-fal.patch
> > +++ /dev/null
> > @@ -1,69 +0,0 @@
> > -From fd8f49dba8c09d47425da80f5faab3bfa4a7c962 Mon Sep 17 00:00:00 2001
> > -From: Jose Quaresma 
> > -Date: Sat, 10 Oct 2020 19:09:03 +
> > -Subject: [PATCH 1/3] gstpluginloader: when env var is set do not fall
> through
> > - to system plugin scanner
> > -
> > -If we set a custom GST_PLUGIN_SCANNER env var, then we probably want to
> use that and only that.
> > -
> > -Falling through to the one installed on the system is problamatic in
> cross-compilation
> > -environemnts, regardless of whether one pointed to by the env var
> succeeded or failed.
> > -
> > -taken from:
> > -
>

Re: [OE-core] [PATCH 07/17] gstreamer1.0: upgrade 1.18.5 -> 1.20.0

2022-02-08 Thread Alexander Kanavin 
Unfortunately this patchset (which is now in master) does not address
the ptest fail. Please resolve this as soon as possible.

+ exec /usr/libexec/installed-tests/gstreamer/gst_gstbin
Running suite(s): GstBin
Executing: gstreamer/gst_gstbin.test
Executing: gstreamer/gst_gstbin.test

Executing: gstreamer/gst_gstbin.test
Executing: gstreamer/gst_gstbin.test
Test timed out after 300 seconds
FAIL: gstreamer/gst_gstbin.test (Child process killed by signal 9)

Alex

On Mon, 7 Feb 2022 at 02:08, Jose Quaresma  wrote:
>
> - License update:
>   The code actually is LGPLv2.1 but the COPYING file is wrong.
>   
> https://gitlab.freedesktop.org/gstreamer/gstreamer/-/commit/d157b23fb0ce175b167611a4cb019599d9017042
>
> - Drop backported patches
>   0001-gst-gstpluginloader.c-when-env-var-is-set-do-not-fal.patch
>   0002-Remove-unused-valgrind-detection.patch
>   0003-tests-seek-Don-t-use-too-strict-timeout-for-validati.patch
>
> - Update submitted patches
>   0001-tests-respect-the-idententaion-used-in-meson.patch
>   0002-tests-add-support-for-install-the-tests.patch
>   0003-tests-use-a-dictionaries-for-environment.patch
>   0004-tests-add-helper-script-to-run-the-installed_tests.patch
>
> Signed-off-by: Jose Quaresma 
> ---
>  ...der.c-when-env-var-is-set-do-not-fal.patch |  69 ---
>  ...pect-the-idententaion-used-in-meson.patch} |  14 +--
>  ...002-Remove-unused-valgrind-detection.patch | 112 --
>  ...s-add-support-for-install-the-tests.patch} |  67 ++-
>  ...-use-too-strict-timeout-for-validati.patch |  32 -
>  ...-use-a-dictionaries-for-environment.patch} |  28 +++--
>  ...er-script-to-run-the-installed_tests.patch |  74 
>  ...-the-environment-for-installed_tests.patch |  58 -
>  ...er1.0_1.18.5.bb => gstreamer1.0_1.20.0.bb} |  17 ++-
>  9 files changed, 136 insertions(+), 335 deletions(-)
>  delete mode 100644 
> meta/recipes-multimedia/gstreamer/gstreamer1.0/0001-gst-gstpluginloader.c-when-env-var-is-set-do-not-fal.patch
>  rename 
> meta/recipes-multimedia/gstreamer/gstreamer1.0/{0004-tests-respect-the-idententaion-used-in-meson.patch
>  => 0001-tests-respect-the-idententaion-used-in-meson.patch} (71%)
>  delete mode 100644 
> meta/recipes-multimedia/gstreamer/gstreamer1.0/0002-Remove-unused-valgrind-detection.patch
>  rename 
> meta/recipes-multimedia/gstreamer/gstreamer1.0/{0005-tests-add-support-for-install-the-tests.patch
>  => 0002-tests-add-support-for-install-the-tests.patch} (65%)
>  delete mode 100644 
> meta/recipes-multimedia/gstreamer/gstreamer1.0/0003-tests-seek-Don-t-use-too-strict-timeout-for-validati.patch
>  rename 
> meta/recipes-multimedia/gstreamer/gstreamer1.0/{0006-tests-use-a-dictionaries-for-environment.patch
>  => 0003-tests-use-a-dictionaries-for-environment.patch} (61%)
>  create mode 100644 
> meta/recipes-multimedia/gstreamer/gstreamer1.0/0004-tests-add-helper-script-to-run-the-installed_tests.patch
>  delete mode 100644 
> meta/recipes-multimedia/gstreamer/gstreamer1.0/0007-tests-install-the-environment-for-installed_tests.patch
>  rename meta/recipes-multimedia/gstreamer/{gstreamer1.0_1.18.5.bb => 
> gstreamer1.0_1.20.0.bb} (76%)
>
> diff --git 
> a/meta/recipes-multimedia/gstreamer/gstreamer1.0/0001-gst-gstpluginloader.c-when-env-var-is-set-do-not-fal.patch
>  
> b/meta/recipes-multimedia/gstreamer/gstreamer1.0/0001-gst-gstpluginloader.c-when-env-var-is-set-do-not-fal.patch
> deleted file mode 100644
> index 23ebd5c600..00
> --- 
> a/meta/recipes-multimedia/gstreamer/gstreamer1.0/0001-gst-gstpluginloader.c-when-env-var-is-set-do-not-fal.patch
> +++ /dev/null
> @@ -1,69 +0,0 @@
> -From fd8f49dba8c09d47425da80f5faab3bfa4a7c962 Mon Sep 17 00:00:00 2001
> -From: Jose Quaresma 
> -Date: Sat, 10 Oct 2020 19:09:03 +
> -Subject: [PATCH 1/3] gstpluginloader: when env var is set do not fall through
> - to system plugin scanner
> -
> -If we set a custom GST_PLUGIN_SCANNER env var, then we probably want to use 
> that and only that.
> -
> -Falling through to the one installed on the system is problamatic in 
> cross-compilation
> -environemnts, regardless of whether one pointed to by the env var succeeded 
> or failed.
> -
> -taken from:
> -http://cgit.openembedded.org/openembedded-core/commit/meta/recipes-multimedia/gstreamer/gstreamer1.0/0001-gst-gstpluginloader.c-when-env-var-is-set-do-not-fal.patch?id=0db7ba34ca41b107042306d13a6f0162885c123b
> -
> -Part-of: 
> 
> -
> -Upstream-Status: Backport 
> [https://gitlab.freedesktop.org/gstreamer/gstreamer/-/commit/9f958058697e6fbf5bde325228034572331d1a3a]
> -
> -Signed-off-by: Jose Quaresma 
> 
> - gst/gstpluginloader.c | 15 +++
> - 1 file changed, 7 insertions(+), 8 deletions(-)
> -
> -diff --git a/gst/gstpluginloader.c b/gst/gstpluginloader.c
> -index d1e404d98..c626bf263 100644
>  a/gst/gstpluginloader.c
> -+++ b/gst/gstpluginloader.c
> -@@ -464,20 +464,19 @@