Re: [OE-core][PATCH] rng-tools: Change "Requires" to "WantedBy" for dev-hwrng.device

[Claudius Heine]

Hi Dew.

On 2022-08-15 20:25, drew.mose...@gmail.com wrote:

From: Drew Moseley 

The previous change to remove the dependency on systemd-udev-settle
caused boot delays at least on qemu. It seems that change required
a hwrng device but that's not necessarily available on all platforms.

This changes the "Requires" to a "WantedBy" as modeled after the
version of this file on Ubuntu 20.04.

Signed-off-by: Drew Moseley 
---
  meta/recipes-support/rng-tools/rng-tools/rng-tools.service | 3 +--
  1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/meta/recipes-support/rng-tools/rng-tools/rng-tools.service 
b/meta/recipes-support/rng-tools/rng-tools/rng-tools.service
index 0f50890dcb..b013d95932 100644
--- a/meta/recipes-support/rng-tools/rng-tools/rng-tools.service
+++ b/meta/recipes-support/rng-tools/rng-tools/rng-tools.service
@@ -3,8 +3,6 @@ Description=Hardware RNG Entropy Gatherer Daemon
  DefaultDependencies=no
  After=systemd-udev-settle.service
  Before=sysinit.target shutdown.target
-Requires=dev-hwrng.device
-After=dev-hwrng.device
  Conflicts=shutdown.target
  
  [Service]

@@ -32,3 +30,4 @@ SystemCallFilter=@system-service
  
  [Install]

  WantedBy=sysinit.target
+WantedBy=dev-hwrng.device


This looses the order. Have you tried just replacing `Requires` with 
`Wants`. So something like this:



```

Before=sysinit.target shutdown.target
Wants=dev-hwrng.device
After=dev-hwrng.device
Conflicts=shutdown.target

```

If that still would cause the long waiting time on qemu?

I haven't investigated the qemu issue, but maybe qemu just doesn't 
provide a hardware random source? If it doesn't provide a hardware 
random source, then installing rng-tools on there might also not make sense.


Alternatively maybe just adding a start timeout might solve it?

regards,
Claudius

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#169618): 
https://lists.openembedded.org/g/openembedded-core/message/169618
Mute This Topic: https://lists.openembedded.org/mt/93042904/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] rng-tools: Replace obsolete "wants systemd-udev-settle"

[Claudius Heine]

Hi Drew,

On 2022-08-01 20:44, Drew Moseley wrote:

On 7/26/22 9:17 AM, drew.mose...@gmail.com wrote:


From: Drew Moseley 

The systemd-udev-settle service is listed as obsolete and does in
some cases result in a significant boot time.  Replace that with
"after systemd-modules-load" to allow the random source hardware
to be loaded in the proper sequence.

Before this change:

    Startup finished in 4.025s (kernel) + 2min 13.236s (userspace) = 
2min 17.262s

    multi-user.target reached after 2min 8.153s in userspace

After this change:

    Startup finished in 3.956s (kernel) + 13.294s (userspace) = 17.250s
    multi-user.target reached after 9.522s in userspace

Signed-off-by: Drew Moseley 
---
  meta/recipes-support/rng-tools/rng-tools/rngd.service | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-support/rng-tools/rng-tools/rngd.service 
b/meta/recipes-support/rng-tools/rng-tools/rngd.service

index 568686e80e..e818cfcf78 100644
--- a/meta/recipes-support/rng-tools/rng-tools/rngd.service
+++ b/meta/recipes-support/rng-tools/rng-tools/rngd.service
@@ -3,7 +3,7 @@ Description=Hardware RNG Entropy Gatherer Daemon
  DefaultDependencies=no
  After=systemd-udev-settle.service
  Before=sysinit.target shutdown.target
-Wants=systemd-udev-settle.service
+After=systemd-modules-load.service
  Conflicts=shutdown.target
  [Service]



Hi Khem, Claudius,

Any thoughts on this change? Obviously I cannot verify if the original 
hardware combination issue is resolved and unfortunately it sounds like 
you cannot either but I'm using this successfully to work around the 
delayed boots I was seeing here.


Well as I said, `systemd-modules-load.service` just waits until all 
statically configured modules are loaded, however not until the 
`/dev/hwrng` device node is available. `systemd-udev-settle.service` 
waits until all udev rules are applied, and that means it includes 
loading of all dynamically loaded modules as well as executing code that 
creates symlinks, etc.


Instead of waiting for modules to be loaded, that might be unrelated or 
waiting for all (mostly unrelated) udev rules to be settled, I think 
waiting directly for the `/dev/hwrng` device node might be more explicit 
and specific.


regards,
Claudius

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#168769): 
https://lists.openembedded.org/g/openembedded-core/message/168769
Mute This Topic: https://lists.openembedded.org/mt/92626958/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] rng-tools: add systemd-udev-settle wants to service

[Claudius Heine]

Hi Khem,

On 2022-07-21 17:29, Khem Raj wrote:



On 7/21/22 11:17 AM, Drew Moseley wrote:

On 2/3/22 9:12 AM, Kyle Russell wrote:

Thanks, Claudius.  I really appreciate your responses.  I'm not 
trying to be pedantic.  Since I don't have your test setup, I was 
just trying to make sure I understood the context of the problem as I 
figure out how to deal with issues this is causing in our setup.


I was also hoping one of the recipe maintainers of either systemd or 
rng-tools would comment on systemd-udev-settle.


I'll take a look at the caam module to see if I can understand how it 
works.


On Thu, Feb 3, 2022 at 3:35 AM Claudius Heine  wrote:

    On 2022-02-02 17:26, Kyle Russell wrote:
    > Thanks, Claudius.
    >
    > On Wed, Feb 2, 2022 at 8:08 AM Claudius Heine  <mailto:c...@denx.de>> wrote:
    >
    >     Hi Kyle,
    >
    >     On 2022-02-02 13:38, Kyle Russell wrote:
    >      > Is this the correct approach?  Even the
    >     systemd-udev-settle.service man
    >      > pages recommends not using its service.  Were the kernel
    modules
    >     really
    >      > not loaded when rngd started?  Or is the original problem
    just a
    >     matter
    >      > of waiting for sufficient entropy?
    >
    >     IIRC, the rngd could not find any random source device node
    (/dev/hwrng
    >     in that case), so the service failed to start.
    >
    >
    > If /dev/hwrng didn't exist, this feels like the original problem
    was a
    > misconfigured
    > kernel or module that wasn't being loaded properly.

    Yes, however it is a timing issue. The module was loaded properly at
    bootup, however at the time rngd was started the module was not
    loaded
    yet and thus the service fails to start. If it would be delayed 
until

    the module is loaded everything would be fine.

    It does not happen if the module is compiled into the kernel or if a
    initramfs is used which loads the module (I think). I our case it
    happend with the caam module as an external module loaded on boot
    from
    the real root file system.

    >     The patch you are commenting on only adds `Wants` weak
    dependency to
    >     make sure `systemd-udev-settle.service` is pulled in to the
    job queue,
    >     the `After` ordering rule was already there.
    >
    >
    > Correct.  Just because an `After` exists does not mean the
    service gets
    > pulled into
    > the job queue, so prior to this change no other service was
    causing the
    > deprecated
    > systemd-udev-settle.service to be run during boot.  But now, every
    > device including
    > openssh (which has a default PACKAGECONFIG option for rng-tools)
    now depends
    > on this deprecated service, which may cause unexpected boot 
delays.

    >
    >     So changing this service file to be triggered by a udev
    event or maybe
    >     wrap it in a script, which makes sure the right modules are
    loaded and
    >     device nodes are available, could be an improvement, but it
    would be
    >     out
    >     of scope of this patch IMO.
    >
    >
    > I'm more curious whether this change should be reverted from
    upstream.
    > It seems
    > like a drop-in file could have been applied to your distro
    instead of
    > adding a dependency
    > on a deprecated service for all openssh users.

    This patch just adds a missing entry into the service file. If you
    have
    solved the described issue in some way and can revert this patch and
    remove the `Wants=systemd-udev-settle.service` then you can also
    remove
    the `After=systemd-udev-settle.service` at the same time and at that
    point you can just remove both of those entries directly in the 
patch

    that solved the timing issue.

    I agree that `systemd-udev-settle.service` should probably not be
    used
    anymore, however that file already used it in a non-functional way
    and
    all this patch did was make it fulfill its intended function.

    In retrospect I probably should have tried to find a way to remove
    the
    usage of `systemd-udev-settle.service` completely, when I looked 
into

    the issue, however all this patch in essence does is revive dead
    code,
    which was already in place.

    Also I think at that time I couldn't find a more precise
    instrument in
    systemds massive toolbox to delay the start of rngd and services 
that

    should be started in succession until the just the hardware random
    generator device is ready and `After=systemd-udev-settle.service` 
was

    already there. I guess some `ExecStartPre=` script which delays the
    start until the conditions are met could be implemented, but that
    seems
    a bit hackish.

    regards,
    Claudius



We are getting report from our users that adding th

Re: [OE-core] [PATCH v3] classes: rootfs-postcommands: add skip option to overlayfs_qa_check

[Claudius Heine]

Hi,

On 2022-05-03 10:22, Claudius Heine wrote:

The overlayfs_qa_check checks if the current root file system has a
mount configured for each overlayfs, when the overlayfs class is used.

However there are certain instances where this mount point is created at
runtime and not static in a fstab entry or systemd mount unit.

One such case would be if overlayfs-etc is used, where the device is
mounted in the preinit script and not via a mount unit or fstab entry.

However there are other possibilities for this as well, like startup
scripts that support a dynamic partition layout. For instance when
systemd-repart is used.

This adds the `OVERLAYFS_QA_SKIP` variable, which allows to define QA
skips via its flags. In principle it supports multiple QA skip flags
separated by whitespace, but only one (`mount-configured`) is
implemented here. To skip this QA check simply add `mount-configured` to
the flag of `OVERLAYFS_QA_SKIP` with the same name. For instance if a
overlayfs is configured as:

OVERLAYFS_MOUNT_POINT[data] = "/data"

Skipping this QA check can be done by setting:

OVERLAYFS_QA_SKIP[data] = "mount-configured"

Also add a testcase and fix a typo (fstat -> fstab).


Would it be possible to merge this patch into the kirkstone stable branch?

Skipping a qa check should be possible in most cases IMO, and isn't 
really a new feature.


regards,
Claudius



Signed-off-by: Claudius Heine 
---

Changes from v2:
- Fix typo 'fstat' -> 'fstab' in testcase
- Add testcase for successful build when QA check is skipped.

Changes from v1:
- `getVarFag` -> `getVarFlag`

---
  meta/classes/rootfs-postcommands.bbclass  | 10 --
  meta/lib/oeqa/selftest/cases/overlayfs.py | 20 +++-
  2 files changed, 27 insertions(+), 3 deletions(-)

diff --git a/meta/classes/rootfs-postcommands.bbclass 
b/meta/classes/rootfs-postcommands.bbclass
index 7b92df69c5..9b6824043c 100644
--- a/meta/classes/rootfs-postcommands.bbclass
+++ b/meta/classes/rootfs-postcommands.bbclass
@@ -398,6 +398,10 @@ python overlayfs_qa_check() {
  
  allUnitExist = True;

  for mountPoint in overlayMountPoints:
+qaSkip = (d.getVarFlag("OVERLAYFS_QA_SKIP", mountPoint) or "").split()
+if "mount-configured" in qaSkip:
+continue
+
  mountPath = d.getVarFlag('OVERLAYFS_MOUNT_POINT', mountPoint)
  if mountPath in fstabDevices:
  continue
@@ -407,8 +411,10 @@ python overlayfs_qa_check() {
 for dirpath in searchpaths):
  continue
  
-bb.warn('Mount path %s not found in fstat and unit %s not found '

-'in systemd unit directories' % (mountPath, mountUnit))
+bb.warn(f'Mount path {mountPath} not found in fstab and unit '
+f'{mountUnit} not found in systemd unit directories.')
+bb.warn(f'Skip this check by setting OVERLAYFS_QA_SKIP[{mountPoint}] = 
'
+'"mount-configured"')
  allUnitExist = False;
  
  if not allUnitExist:

diff --git a/meta/lib/oeqa/selftest/cases/overlayfs.py 
b/meta/lib/oeqa/selftest/cases/overlayfs.py
index 472746a64f..ce1d2f1ec3 100644
--- a/meta/lib/oeqa/selftest/cases/overlayfs.py
+++ b/meta/lib/oeqa/selftest/cases/overlayfs.py
@@ -62,11 +62,29 @@ DISTRO_FEATURES += "systemd overlayfs"
  self.add_overlay_conf_to_machine()
  
  res = bitbake('core-image-minimal', ignore_status=True)

-line = getline(res, " Mount path /mnt/overlay not found in fstat and unit 
mnt-overlay.mount not found in systemd unit directories")
+line = getline(res, " Mount path /mnt/overlay not found in fstab and unit 
mnt-overlay.mount not found in systemd unit directories")
  self.assertTrue(line and line.startswith("WARNING:"), msg=res.output)
  line = getline(res, "Not all mount paths and units are installed in the 
image")
  self.assertTrue(line and line.startswith("ERROR:"), msg=res.output)
  
+def test_not_all_units_installed_but_qa_skipped(self):

+"""
+Summary:   Test skipping the QA check
+Expected:  Image is created successfully
+Author:Claudius Heine 
+"""
+
+config = """
+IMAGE_INSTALL:append = " overlayfs-user"
+DISTRO_FEATURES += "systemd overlayfs"
+OVERLAYFS_QA_SKIP[mnt-overlay] = "mount-configured"
+"""
+
+self.write_config(config)
+self.add_overlay_conf_to_machine()
+
+bitbake('core-image-minimal')
+
  def test_mount_unit_not_set(self):
  """
  Summary:   Test whether mount unit was set properly







--
DENX Software Engineering GmbH,  Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-81

[OE-core] [PATCH] overlayfs: add docs about skipping QA check & service dependencies

[Claudius Heine]

Add some documentation about skipping the QA check related to missing
fstab entries or mount units for base mount points where the overlayfs
is mounted from.

Also add a short paragraph about adding a systemd unit dependency to
services in recipes, so that they are started only after the overlayfs
is mounted and ready.

No functional change.

Signed-off-by: Claudius Heine 
---
 meta/classes/overlayfs.bbclass | 18 +++---
 1 file changed, 15 insertions(+), 3 deletions(-)

diff --git a/meta/classes/overlayfs.bbclass b/meta/classes/overlayfs.bbclass
index 29fced2ca7..f7069edd41 100644
--- a/meta/classes/overlayfs.bbclass
+++ b/meta/classes/overlayfs.bbclass
@@ -16,10 +16,18 @@
 #
 #   OVERLAYFS_MOUNT_POINT[data] ?= "/data"
 #
-# The class assumes you have a data.mount systemd unit defined in your
-# systemd-machine-units recipe and installed to the image.
+# Per default the class assumes you have a corresponding fstab entry or systemd
+# mount unit (data.mount in this case) for this mount point installed on the
+# image, for instance via a wks script or the systemd-machine-units recipe.
 #
-# Then you can specify writable directories on a recipe base
+# If the mount point is handled somewhere else, e.g. custom boot or preinit
+# scripts or in a initramfs, then this QA check can be skipped by adding
+# mount-configured to the related OVERLAYFS_QA_SKIP flag:
+#
+#   OVERLAYFS_QA_SKIP[data] = "mount-configured"
+#
+# To use the overlayfs, you just have to specify writable directories inside
+# their recipe:
 #
 #   OVERLAYFS_WRITABLE_PATHS[data] = "/usr/share/my-custom-application"
 #
@@ -30,6 +38,10 @@
 #   OVERLAYFS_MOUNT_POINT[mnt-overlay] = "/mnt/overlay"
 #   OVERLAYFS_WRITABLE_PATHS[mnt-overlay] = "/usr/share/another-application"
 #
+# If your recipe deploys a systemd service, then it should require and be
+# started after the ${PN}-overlays.service to make sure that all overlays are
+# mounted beforehand.
+#
 # Note: the class does not support /etc directory itself, because systemd 
depends on it
 # For /etc directory use overlayfs-etc class
 
-- 
2.33.3


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#165411): 
https://lists.openembedded.org/g/openembedded-core/message/165411
Mute This Topic: https://lists.openembedded.org/mt/91008083/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 v3] classes: rootfs-postcommands: add skip option to overlayfs_qa_check

[Claudius Heine]

On 2022-05-09 07:02, Vyacheslav Yurkov wrote:

On 03.05.2022 10:22, Claudius Heine wrote:

The overlayfs_qa_check checks if the current root file system has a
mount configured for each overlayfs, when the overlayfs class is used.

However there are certain instances where this mount point is created at
runtime and not static in a fstab entry or systemd mount unit.

One such case would be if overlayfs-etc is used, where the device is
mounted in the preinit script and not via a mount unit or fstab entry.

However there are other possibilities for this as well, like startup
scripts that support a dynamic partition layout. For instance when
systemd-repart is used.

This adds the `OVERLAYFS_QA_SKIP` variable, which allows to define QA
skips via its flags. In principle it supports multiple QA skip flags
separated by whitespace, but only one (`mount-configured`) is
implemented here. To skip this QA check simply add `mount-configured` to
the flag of `OVERLAYFS_QA_SKIP` with the same name. For instance if a
overlayfs is configured as:

OVERLAYFS_MOUNT_POINT[data] = "/data"

Skipping this QA check can be done by setting:

OVERLAYFS_QA_SKIP[data] = "mount-configured"

Also add a testcase and fix a typo (fstat -> fstab).

Signed-off-by: Claudius Heine 
---

Changes from v2:
- Fix typo 'fstat' -> 'fstab' in testcase
- Add testcase for successful build when QA check is skipped.

Changes from v1:
- `getVarFag` -> `getVarFlag`

---
  meta/classes/rootfs-postcommands.bbclass  | 10 --
  meta/lib/oeqa/selftest/cases/overlayfs.py | 20 +++-
  2 files changed, 27 insertions(+), 3 deletions(-)

diff --git a/meta/classes/rootfs-postcommands.bbclass 
b/meta/classes/rootfs-postcommands.bbclass

index 7b92df69c5..9b6824043c 100644
--- a/meta/classes/rootfs-postcommands.bbclass
+++ b/meta/classes/rootfs-postcommands.bbclass
@@ -398,6 +398,10 @@ python overlayfs_qa_check() {
  allUnitExist = True;
  for mountPoint in overlayMountPoints:
+    qaSkip = (d.getVarFlag("OVERLAYFS_QA_SKIP", mountPoint) or 
"").split()

+    if "mount-configured" in qaSkip:
+    continue
+
  mountPath = d.getVarFlag('OVERLAYFS_MOUNT_POINT', mountPoint)
  if mountPath in fstabDevices:
  continue
@@ -407,8 +411,10 @@ python overlayfs_qa_check() {
 for dirpath in searchpaths):
  continue
-    bb.warn('Mount path %s not found in fstat and unit %s not 
found '

-    'in systemd unit directories' % (mountPath, mountUnit))
+    bb.warn(f'Mount path {mountPath} not found in fstab and unit '
+    f'{mountUnit} not found in systemd unit directories.')
+    bb.warn(f'Skip this check by setting 
OVERLAYFS_QA_SKIP[{mountPoint}] = '

+    '"mount-configured"')
  allUnitExist = False;
  if not allUnitExist:
diff --git a/meta/lib/oeqa/selftest/cases/overlayfs.py 
b/meta/lib/oeqa/selftest/cases/overlayfs.py

index 472746a64f..ce1d2f1ec3 100644
--- a/meta/lib/oeqa/selftest/cases/overlayfs.py
+++ b/meta/lib/oeqa/selftest/cases/overlayfs.py
@@ -62,11 +62,29 @@ DISTRO_FEATURES += "systemd overlayfs"
  self.add_overlay_conf_to_machine()
  res = bitbake('core-image-minimal', ignore_status=True)
-    line = getline(res, " Mount path /mnt/overlay not found in 
fstat and unit mnt-overlay.mount not found in systemd unit directories")
+    line = getline(res, " Mount path /mnt/overlay not found in 
fstab and unit mnt-overlay.mount not found in systemd unit directories")
  self.assertTrue(line and line.startswith("WARNING:"), 
msg=res.output)
  line = getline(res, "Not all mount paths and units are 
installed in the image")
  self.assertTrue(line and line.startswith("ERROR:"), 
msg=res.output)

+    def test_not_all_units_installed_but_qa_skipped(self):
+    """
+    Summary:   Test skipping the QA check
+    Expected:  Image is created successfully
+    Author:    Claudius Heine 
+    """
+
+    config = """
+IMAGE_INSTALL:append = " overlayfs-user"
+DISTRO_FEATURES += "systemd overlayfs"
+OVERLAYFS_QA_SKIP[mnt-overlay] = "mount-configured"
+"""
+
+    self.write_config(config)
+    self.add_overlay_conf_to_machine()
+
+    bitbake('core-image-minimal')
+
  def test_mount_unit_not_set(self):
  """
  Summary:   Test whether mount unit was set properly

Hi Claudius,
Thanks a lot for the patch

Could you please also extend the header of overlayfs.bbclass to describe 
the use case? I will update the documentation accordingly after that.


Hi Vyacheslav,

sure. I will try to work that in.

regards,
Claudius

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

[OE-core] [PATCH v3] classes: rootfs-postcommands: add skip option to overlayfs_qa_check

[Claudius Heine]

The overlayfs_qa_check checks if the current root file system has a
mount configured for each overlayfs, when the overlayfs class is used.

However there are certain instances where this mount point is created at
runtime and not static in a fstab entry or systemd mount unit.

One such case would be if overlayfs-etc is used, where the device is
mounted in the preinit script and not via a mount unit or fstab entry.

However there are other possibilities for this as well, like startup
scripts that support a dynamic partition layout. For instance when
systemd-repart is used.

This adds the `OVERLAYFS_QA_SKIP` variable, which allows to define QA
skips via its flags. In principle it supports multiple QA skip flags
separated by whitespace, but only one (`mount-configured`) is
implemented here. To skip this QA check simply add `mount-configured` to
the flag of `OVERLAYFS_QA_SKIP` with the same name. For instance if a
overlayfs is configured as:

OVERLAYFS_MOUNT_POINT[data] = "/data"

Skipping this QA check can be done by setting:

OVERLAYFS_QA_SKIP[data] = "mount-configured"

Also add a testcase and fix a typo (fstat -> fstab).

Signed-off-by: Claudius Heine 
---

Changes from v2:
- Fix typo 'fstat' -> 'fstab' in testcase
- Add testcase for successful build when QA check is skipped.

Changes from v1:
- `getVarFag` -> `getVarFlag`

---
 meta/classes/rootfs-postcommands.bbclass  | 10 --
 meta/lib/oeqa/selftest/cases/overlayfs.py | 20 +++-
 2 files changed, 27 insertions(+), 3 deletions(-)

diff --git a/meta/classes/rootfs-postcommands.bbclass 
b/meta/classes/rootfs-postcommands.bbclass
index 7b92df69c5..9b6824043c 100644
--- a/meta/classes/rootfs-postcommands.bbclass
+++ b/meta/classes/rootfs-postcommands.bbclass
@@ -398,6 +398,10 @@ python overlayfs_qa_check() {
 
 allUnitExist = True;
 for mountPoint in overlayMountPoints:
+qaSkip = (d.getVarFlag("OVERLAYFS_QA_SKIP", mountPoint) or "").split()
+if "mount-configured" in qaSkip:
+continue
+
 mountPath = d.getVarFlag('OVERLAYFS_MOUNT_POINT', mountPoint)
 if mountPath in fstabDevices:
 continue
@@ -407,8 +411,10 @@ python overlayfs_qa_check() {
for dirpath in searchpaths):
 continue
 
-bb.warn('Mount path %s not found in fstat and unit %s not found '
-'in systemd unit directories' % (mountPath, mountUnit))
+bb.warn(f'Mount path {mountPath} not found in fstab and unit '
+f'{mountUnit} not found in systemd unit directories.')
+bb.warn(f'Skip this check by setting OVERLAYFS_QA_SKIP[{mountPoint}] = 
'
+'"mount-configured"')
 allUnitExist = False;
 
 if not allUnitExist:
diff --git a/meta/lib/oeqa/selftest/cases/overlayfs.py 
b/meta/lib/oeqa/selftest/cases/overlayfs.py
index 472746a64f..ce1d2f1ec3 100644
--- a/meta/lib/oeqa/selftest/cases/overlayfs.py
+++ b/meta/lib/oeqa/selftest/cases/overlayfs.py
@@ -62,11 +62,29 @@ DISTRO_FEATURES += "systemd overlayfs"
 self.add_overlay_conf_to_machine()
 
 res = bitbake('core-image-minimal', ignore_status=True)
-line = getline(res, " Mount path /mnt/overlay not found in fstat and 
unit mnt-overlay.mount not found in systemd unit directories")
+line = getline(res, " Mount path /mnt/overlay not found in fstab and 
unit mnt-overlay.mount not found in systemd unit directories")
 self.assertTrue(line and line.startswith("WARNING:"), msg=res.output)
 line = getline(res, "Not all mount paths and units are installed in 
the image")
 self.assertTrue(line and line.startswith("ERROR:"), msg=res.output)
 
+def test_not_all_units_installed_but_qa_skipped(self):
+"""
+Summary:   Test skipping the QA check
+Expected:  Image is created successfully
+Author:Claudius Heine 
+"""
+
+config = """
+IMAGE_INSTALL:append = " overlayfs-user"
+DISTRO_FEATURES += "systemd overlayfs"
+OVERLAYFS_QA_SKIP[mnt-overlay] = "mount-configured"
+"""
+
+self.write_config(config)
+self.add_overlay_conf_to_machine()
+
+bitbake('core-image-minimal')
+
 def test_mount_unit_not_set(self):
 """
 Summary:   Test whether mount unit was set properly
-- 
2.33.3


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#165196): 
https://lists.openembedded.org/g/openembedded-core/message/165196
Mute This Topic: https://lists.openembedded.org/mt/90854996/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 v2] classes: rootfs-postcommands: add skip option to overlayfs_qa_check

[Claudius Heine]

On 2022-05-02 21:09, Luca Ceresoli wrote:

Hi Claudius,



Hi Luca,


Il giorno Fri, 29 Apr 2022 10:17:33 +0200
"Claudius Heine"  ha scritto:


The overlayfs_qa_check checks if the current root file system has a
mount configured for each overlayfs, when the overlayfs class is used.

However there are certain instances where this mount point is created
at runtime and not static in a fstab entry or systemd mount unit.

One such case would be if overlayfs-etc is used, where the device is
mounted in the preinit script and not via a mount unit or fstab entry.

However there are other possibilities for this as well, like startup
scripts that support a dynamic partition layout. For instance when
systemd-repart is used.

This adds the `OVERLAYFS_QA_SKIP` variable, which allows to define QA
skips via its flags. In principle it supports multiple QA skip flags
separated by whitespace, but only one (`mount-configured`) is
implemented here. To skip this QA check simply add `mount-configured`
to the flag of `OVERLAYFS_QA_SKIP` with the same name. For instance
if a overlayfs is configured as:

OVERLAYFS_MOUNT_POINT[data] = "/data"

Skipping this QA check can be done by setting:

OVERLAYFS_QA_SKIP[data] = "mount-configured"

Signed-off-by: Claudius Heine 


Failures appeared on the autobuilders with your patch applied:

https://autobuilder.yoctoproject.org/typhoon/#/builders/87/builds/3530/steps/15/logs/stdio
https://autobuilder.yoctoproject.org/typhoon/#/builders/87/builds/3530/steps/15/logs/stdio
https://autobuilder.yoctoproject.org/typhoon/#/builders/80/builds/3470/steps/14/logs/stdio
https://autobuilder.yoctoproject.org/typhoon/#/builders/79/builds/3521/steps/15/logs/stdio


Thanks! It seems that failed because I also fixed a typo in the warning 
message ('fstat -> fstab').


I will change that in v3 and also add a testcase for the 
OVERLAYFS_QA_SKIP functionality.


regards,
Claudius

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

[OE-core] [PATCH v2] classes: rootfs-postcommands: add skip option to overlayfs_qa_check

[Claudius Heine]

The overlayfs_qa_check checks if the current root file system has a
mount configured for each overlayfs, when the overlayfs class is used.

However there are certain instances where this mount point is created at
runtime and not static in a fstab entry or systemd mount unit.

One such case would be if overlayfs-etc is used, where the device is
mounted in the preinit script and not via a mount unit or fstab entry.

However there are other possibilities for this as well, like startup
scripts that support a dynamic partition layout. For instance when
systemd-repart is used.

This adds the `OVERLAYFS_QA_SKIP` variable, which allows to define QA
skips via its flags. In principle it supports multiple QA skip flags
separated by whitespace, but only one (`mount-configured`) is
implemented here. To skip this QA check simply add `mount-configured` to
the flag of `OVERLAYFS_QA_SKIP` with the same name. For instance if a
overlayfs is configured as:

OVERLAYFS_MOUNT_POINT[data] = "/data"

Skipping this QA check can be done by setting:

OVERLAYFS_QA_SKIP[data] = "mount-configured"

Signed-off-by: Claudius Heine 
---

Sorry, I was a bit to quick when sending the last patch and forgot to
amend a typo fix before sending.

Difference between v2:
 - `getVarFag` -> `getVarFlag`

---
 meta/classes/rootfs-postcommands.bbclass | 10 --
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/meta/classes/rootfs-postcommands.bbclass 
b/meta/classes/rootfs-postcommands.bbclass
index 7b92df69c5..9b6824043c 100644
--- a/meta/classes/rootfs-postcommands.bbclass
+++ b/meta/classes/rootfs-postcommands.bbclass
@@ -398,6 +398,10 @@ python overlayfs_qa_check() {
 
 allUnitExist = True;
 for mountPoint in overlayMountPoints:
+qaSkip = (d.getVarFlag("OVERLAYFS_QA_SKIP", mountPoint) or "").split()
+if "mount-configured" in qaSkip:
+continue
+
 mountPath = d.getVarFlag('OVERLAYFS_MOUNT_POINT', mountPoint)
 if mountPath in fstabDevices:
 continue
@@ -407,8 +411,10 @@ python overlayfs_qa_check() {
for dirpath in searchpaths):
 continue
 
-bb.warn('Mount path %s not found in fstat and unit %s not found '
-'in systemd unit directories' % (mountPath, mountUnit))
+bb.warn(f'Mount path {mountPath} not found in fstab and unit '
+f'{mountUnit} not found in systemd unit directories.')
+bb.warn(f'Skip this check by setting OVERLAYFS_QA_SKIP[{mountPoint}] = 
'
+'"mount-configured"')
 allUnitExist = False;
 
 if not allUnitExist:
-- 
2.33.3


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

[OE-core] [PATCH] classes: rootfs-postcommands: add skip option to overlayfs_qa_check

[Claudius Heine]

The overlayfs_qa_check checks if the current root file system has a
mount configured for each overlayfs, when the overlayfs class is used.

However there are certain instances where this mount point is created at
runtime and not static in a fstab entry or systemd mount unit.

One such case would be if overlayfs-etc is used, where the device is
mounted in the preinit script and not via a mount unit or fstab entry.

However there are other possibilities for this as well, like startup
scripts that support a dynamic partition layout. For instance when
systemd-repart is used.

This adds the `OVERLAYFS_QA_SKIP` variable, which allows to define QA
skips via its flags. In principle it supports multiple QA skip flags
separated by whitespace, but only one (`mount-configured`) is
implemented here. To skip this QA check simply add `mount-configured` to
the flag of `OVERLAYFS_QA_SKIP` with the same name. For instance if a
overlayfs is configured as:

OVERLAYFS_MOUNT_POINT[data] = "/data"

Skipping this QA check can be done by setting:

OVERLAYFS_QA_SKIP[data] = "mount-configured"

Signed-off-by: Claudius Heine 
---
 meta/classes/rootfs-postcommands.bbclass | 10 --
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/meta/classes/rootfs-postcommands.bbclass 
b/meta/classes/rootfs-postcommands.bbclass
index 7b92df69c5..8df175e42e 100644
--- a/meta/classes/rootfs-postcommands.bbclass
+++ b/meta/classes/rootfs-postcommands.bbclass
@@ -398,6 +398,10 @@ python overlayfs_qa_check() {
 
 allUnitExist = True;
 for mountPoint in overlayMountPoints:
+qaSkip = (d.getVarFag("OVERLAYFS_QA_SKIP", mountPoint) or "").split()
+if "mount-configured" in qaSkip:
+continue
+
 mountPath = d.getVarFlag('OVERLAYFS_MOUNT_POINT', mountPoint)
 if mountPath in fstabDevices:
 continue
@@ -407,8 +411,10 @@ python overlayfs_qa_check() {
for dirpath in searchpaths):
 continue
 
-bb.warn('Mount path %s not found in fstat and unit %s not found '
-'in systemd unit directories' % (mountPath, mountUnit))
+bb.warn(f'Mount path {mountPath} not found in fstab and unit '
+f'{mountUnit} not found in systemd unit directories.')
+bb.warn(f'Skip this check by setting OVERLAYFS_QA_SKIP[{mountPoint}] = 
'
+'"mount-configured"')
 allUnitExist = False;
 
 if not allUnitExist:
-- 
2.33.3


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#165000): 
https://lists.openembedded.org/g/openembedded-core/message/165000
Mute This Topic: https://lists.openembedded.org/mt/90771865/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] [RFC PATCH 1/1] kernel: add kernel-image-fitimage-initramfs

[Claudius Heine]

Hi Richard,

On 2022-04-13 09:35, Richard Purdie wrote:

On Wed, 2022-04-13 at 09:18 +0200, Claudius Heine wrote:

Hi,

On 2022-03-30 11:15, Claudius Heine wrote:

When creating an initramfs bundled into a kernel fitimage, the resulting
fitimage will only be placed into the deploy directory and not packaged
by the kernel recipe/class.

Changing the kernel recipe/class to produce a package with the fitimage
containing the initramfs is not possible, because build dependencies
require the kernel packages to be already build before the initramfs can
be created.

The kernel-image-fitimage-initramfs recipe solves this dependency cycle
by packaging the fitimage with the embedded initramfs from the deploy
directory and replaces the `kernel-image` and `kernel-image-fitimage`
packages from the kernel recipe/class.


This patch was on master-next for a short while, any feedback on it?

In the cover letter I stated the only downside of this solution is, that
in case of package based updating, it would require manually changing
the version of this recipe.

Is this an acceptable downside, or is there a good way to fix this?


There were a few things which concern me. It is late in the release cycle for
this to make it into kirkstone so I was deferring it until 4.1 and the next
development cycle.

I was worried that it doesn't have a maintainers entry. I was also worried that
it doesn't have any tests and doesn't get used at all on the autobuilder. There
doesn't seem to have been much other feedback/review on the mailing list either.
There also isn't any documentation. This makes it likely to become a piece of
untested and potentially dead code going forward.

People are fixing up bits of the fitimage and initramfs code adhoc with just
enough changes to sovle their use case. We really need docs and tests and
ideally a clear plan on how this all fits together.

I suspect that the downside you mention, the fact that the kernel is removed
from package management will also catch people out. It is different to our
normal behaviour and I don't know how we highlight to users that this issue is
present.

I'm also worried that we don't have enough people looking at rewiew and that
there are probably issues here we need to discuss but we simply don't have the
right people with time to look at and think about it. I'm not the subject matter
expert here :(.

I wish I had something I could say as to how to make it acceptable to merge
(definitely needs tests) but I really don't know this area well enough and I
haven't had the time to think enough about it. It hovered in -next for a while
as I was trying my best to do so but with the release breaking I needed to clear
out -next for my own sanity and this ended in my defer pile. Even writing the
email explaining why takes 10 minutes.


Thanks a lot for the explanation! I might come back to this at a more 
relaxed period with the suggested improvements.


regards,
Claudius

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#164307): 
https://lists.openembedded.org/g/openembedded-core/message/164307
Mute This Topic: https://lists.openembedded.org/mt/90127688/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] [RFC PATCH 1/1] kernel: add kernel-image-fitimage-initramfs

[Claudius Heine]

Hi,

On 2022-03-30 11:15, Claudius Heine wrote:

When creating an initramfs bundled into a kernel fitimage, the resulting
fitimage will only be placed into the deploy directory and not packaged
by the kernel recipe/class.

Changing the kernel recipe/class to produce a package with the fitimage
containing the initramfs is not possible, because build dependencies
require the kernel packages to be already build before the initramfs can
be created.

The kernel-image-fitimage-initramfs recipe solves this dependency cycle
by packaging the fitimage with the embedded initramfs from the deploy
directory and replaces the `kernel-image` and `kernel-image-fitimage`
packages from the kernel recipe/class.


This patch was on master-next for a short while, any feedback on it?

In the cover letter I stated the only downside of this solution is, that 
in case of package based updating, it would require manually changing 
the version of this recipe.


Is this an acceptable downside, or is there a good way to fix this?

regards,
Claudius



Signed-off-by: Claudius Heine 
---
  .../kernel-image-fitimage-initramfs_0.1.0.bb  | 50 +++
  1 file changed, 50 insertions(+)
  create mode 100644 
meta/recipes-kernel/kernel-image-fitimage-initramfs/kernel-image-fitimage-initramfs_0.1.0.bb

diff --git 
a/meta/recipes-kernel/kernel-image-fitimage-initramfs/kernel-image-fitimage-initramfs_0.1.0.bb
 
b/meta/recipes-kernel/kernel-image-fitimage-initramfs/kernel-image-fitimage-initramfs_0.1.0.bb
new file mode 100644
index 00..c25168301b
--- /dev/null
+++ 
b/meta/recipes-kernel/kernel-image-fitimage-initramfs/kernel-image-fitimage-initramfs_0.1.0.bb
@@ -0,0 +1,50 @@
+# Copyright (C) 2022 Claudius Heine 
+# Released under the MIT license (see COPYING.MIT for the terms)
+
+SUMMARY = "Packages the fitimage with embedded initramfs"
+LICENSE = "GPL-2.0-only"
+DEPENDS = "virtual/kernel"
+
+SRC_URI = ""
+
+inherit kernel-artifact-names
+INITRAMFS_IMAGE_NAME ?= "${@['${INITRAMFS_IMAGE}-${MACHINE}', 
''][d.getVar('INITRAMFS_IMAGE') == '']}"
+KERNEL_PACKAGE_NAME ?= "kernel"
+
+PN = "${KERNEL_PACKAGE_NAME}-image-fitimage-initramfs"
+
+do_configure[noexec] = "1"
+do_compile[noexec] = "1"
+
+do_install[deptask] = "do_deploy"
+do_install() {
+fitimage="$(readlink 
"${DEPLOY_DIR_IMAGE}/fitImage-${INITRAMFS_IMAGE_NAME}-${KERNEL_FIT_LINK_NAME}")"
+install -d ${D}/boot
+install -m 0644 \
+${DEPLOY_DIR_IMAGE}/${fitimage} \
+${D}/boot/
+ln -snf \
+${fitimage} \
+${D}/boot/fitImage
+}
+
+PACKAGES += "${KERNEL_PACKAGE_NAME}-image-initramfs"
+
+FILES:${PN} = "/boot/fitImage /boot/fitImage-*"
+RPROVIDES:${PN} = "${KERNEL_PACKAGE_NAME}-image-fitimage"
+RCONFLICTS:${PN} = "${KERNEL_PACKAGE_NAME}-image-fitimage"
+RREPLACES:${PN} = "${KERNEL_PACKAGE_NAME}-image-fitimage"
+
+RDEPENDS:${KERNEL_PACKAGE_NAME}-image-initramfs = 
"${KERNEL_PACKAGE_NAME}-image-fitimage-initramfs (= ${EXTENDPGKV})"
+RPROVIDES:${KERNEL_PACKAGE_NAME}-image-initramfs = 
"${KERNEL_PACKAGE_NAME}-image"
+RCONFLICTS:${KERNEL_PACKAGE_NAME}-image-initramfs = 
"${KERNEL_PACKAGE_NAME}-image"
+RREPLACES:${KERNEL_PACKAGE_NAME}-image-initramfs = 
"${KERNEL_PACKAGE_NAME}-image"
+ALLOW_EMPTY:${KERNEL_PACKAGE_NAME}-image-initramfs = "1"
+
+python() {
+if (not 'fitImage' in d.getVar('KERNEL_IMAGETYPES') or
+bb.utils.to_boolean(d.getVar('INITRAMFS_IMAGE_BUNDLE'))):
+raise bb.parse.SkipRecipe('Requires generation of fitImage and bundled 
initramfs.')
+if not d.getVar('KERNEL_FIT_LINK_NAME'):
+raise bb.parse.SkipRecipe('Requires generation of fitImage symlink in 
deploy dir.')
+}

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

[OE-core] [PATCH] os-release: add os-release-initrd package

[Claudius Heine]

When systemd is used inside an initramfs, it uses the presence of
`/etc/initrd-release` to detect when the system is in the initrd phase.
Otherwise `/etc/initrd-release` has the same format as
`/etc/os-release`.

Add `os-release-initrd` package to provide `/etc/initrd-release` as a
symlink to the os-release file. To avoid adding this file to the
`os-release` package, explicitly only add `/etc/os-release` to the
`os-release` package.

Signed-off-by: Claudius Heine 
---
 meta/recipes-core/os-release/os-release.bb | 7 ++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-core/os-release/os-release.bb 
b/meta/recipes-core/os-release/os-release.bb
index 637d059e8b..caa2ed25aa 100644
--- a/meta/recipes-core/os-release/os-release.bb
+++ b/meta/recipes-core/os-release/os-release.bb
@@ -50,6 +50,11 @@ do_install () {
 install -d ${D}${nonarch_libdir} ${D}${sysconfdir}
 install -m 0644 os-release ${D}${nonarch_libdir}/
 ln -rs ${D}${nonarch_libdir}/os-release ${D}${sysconfdir}/os-release
+ln -rs ${D}${nonarch_libdir}/os-release ${D}${sysconfdir}/initrd-release
 }
 
-FILES:${PN} += "${nonarch_libdir}/os-release"
+FILES:${PN} = "${sysconfdir}/os-release ${nonarch_libdir}/os-release"
+
+PACKAGES += "${PN}-initrd"
+FILES:${PN}-initrd = "${sysconfdir}/initrd-release"
+RDEPENDS:${PN}-initrd += "${PN}"
-- 
2.33.1


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

[OE-core] [RFC PATCH 1/1] kernel: add kernel-image-fitimage-initramfs

[Claudius Heine]

When creating an initramfs bundled into a kernel fitimage, the resulting
fitimage will only be placed into the deploy directory and not packaged
by the kernel recipe/class.

Changing the kernel recipe/class to produce a package with the fitimage
containing the initramfs is not possible, because build dependencies
require the kernel packages to be already build before the initramfs can
be created.

The kernel-image-fitimage-initramfs recipe solves this dependency cycle
by packaging the fitimage with the embedded initramfs from the deploy
directory and replaces the `kernel-image` and `kernel-image-fitimage`
packages from the kernel recipe/class.

Signed-off-by: Claudius Heine 
---
 .../kernel-image-fitimage-initramfs_0.1.0.bb  | 50 +++
 1 file changed, 50 insertions(+)
 create mode 100644 
meta/recipes-kernel/kernel-image-fitimage-initramfs/kernel-image-fitimage-initramfs_0.1.0.bb

diff --git 
a/meta/recipes-kernel/kernel-image-fitimage-initramfs/kernel-image-fitimage-initramfs_0.1.0.bb
 
b/meta/recipes-kernel/kernel-image-fitimage-initramfs/kernel-image-fitimage-initramfs_0.1.0.bb
new file mode 100644
index 00..c25168301b
--- /dev/null
+++ 
b/meta/recipes-kernel/kernel-image-fitimage-initramfs/kernel-image-fitimage-initramfs_0.1.0.bb
@@ -0,0 +1,50 @@
+# Copyright (C) 2022 Claudius Heine 
+# Released under the MIT license (see COPYING.MIT for the terms)
+
+SUMMARY = "Packages the fitimage with embedded initramfs"
+LICENSE = "GPL-2.0-only"
+DEPENDS = "virtual/kernel"
+
+SRC_URI = ""
+
+inherit kernel-artifact-names
+INITRAMFS_IMAGE_NAME ?= "${@['${INITRAMFS_IMAGE}-${MACHINE}', 
''][d.getVar('INITRAMFS_IMAGE') == '']}"
+KERNEL_PACKAGE_NAME ?= "kernel"
+
+PN = "${KERNEL_PACKAGE_NAME}-image-fitimage-initramfs"
+
+do_configure[noexec] = "1"
+do_compile[noexec] = "1"
+
+do_install[deptask] = "do_deploy"
+do_install() {
+fitimage="$(readlink 
"${DEPLOY_DIR_IMAGE}/fitImage-${INITRAMFS_IMAGE_NAME}-${KERNEL_FIT_LINK_NAME}")"
+install -d ${D}/boot
+install -m 0644 \
+${DEPLOY_DIR_IMAGE}/${fitimage} \
+${D}/boot/
+ln -snf \
+${fitimage} \
+${D}/boot/fitImage
+}
+
+PACKAGES += "${KERNEL_PACKAGE_NAME}-image-initramfs"
+
+FILES:${PN} = "/boot/fitImage /boot/fitImage-*"
+RPROVIDES:${PN} = "${KERNEL_PACKAGE_NAME}-image-fitimage"
+RCONFLICTS:${PN} = "${KERNEL_PACKAGE_NAME}-image-fitimage"
+RREPLACES:${PN} = "${KERNEL_PACKAGE_NAME}-image-fitimage"
+
+RDEPENDS:${KERNEL_PACKAGE_NAME}-image-initramfs = 
"${KERNEL_PACKAGE_NAME}-image-fitimage-initramfs (= ${EXTENDPGKV})"
+RPROVIDES:${KERNEL_PACKAGE_NAME}-image-initramfs = 
"${KERNEL_PACKAGE_NAME}-image"
+RCONFLICTS:${KERNEL_PACKAGE_NAME}-image-initramfs = 
"${KERNEL_PACKAGE_NAME}-image"
+RREPLACES:${KERNEL_PACKAGE_NAME}-image-initramfs = 
"${KERNEL_PACKAGE_NAME}-image"
+ALLOW_EMPTY:${KERNEL_PACKAGE_NAME}-image-initramfs = "1"
+
+python() {
+if (not 'fitImage' in d.getVar('KERNEL_IMAGETYPES') or
+bb.utils.to_boolean(d.getVar('INITRAMFS_IMAGE_BUNDLE'))):
+raise bb.parse.SkipRecipe('Requires generation of fitImage and bundled 
initramfs.')
+if not d.getVar('KERNEL_FIT_LINK_NAME'):
+raise bb.parse.SkipRecipe('Requires generation of fitImage symlink in 
deploy dir.')
+}
-- 
2.33.1


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

[OE-core] [RFC PATCH 0/1] Packaging a fitimage with initramfs

[Claudius Heine]

Hi,

I am currently investigating how to package a fitimage with embedded
initramfs and came up with the following recipe.

The recipe will have issues because its version is not linked to the
kernel version, so updating via package repos will not work.
But that would not be important for my use-case.

Is there some other way to solve this or any ideas how to get around its
limitations?

My goal is to have the fitImage embedded into the rootfs partition
reachable via /boot/fitImage.

thanks and regards,
Claudius

Claudius Heine (1):
  kernel: add kernel-image-fitimage-initramfs

 .../kernel-image-fitimage-initramfs_0.1.0.bb  | 50 +++
 1 file changed, 50 insertions(+)
 create mode 100644 
meta/recipes-kernel/kernel-image-fitimage-initramfs/kernel-image-fitimage-initramfs_0.1.0.bb

-- 
2.33.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#163779): 
https://lists.openembedded.org/g/openembedded-core/message/163779
Mute This Topic: https://lists.openembedded.org/mt/90127689/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 v2 0/4] Patches and bugfixes for overlayfs

[Claudius Heine]

Hi Vyacheslav,

On 2022-03-10 15:28, Vyacheslav Yurkov wrote:

On 10.03.2022 14:13, Claudius Heine wrote:

Hi,

I bundled my patches to overlayfs and overlayfs-etc togehter in this
patchset and fixed the whitespace issues.

regards,
Claudius

--
Changes from v1:
   - fixed whitespace issues: converted tabs to 4 spaces.
   - changed git commit title of patch 4

Claudius Heine (4):
   overlayfs-etc: add condition to package-management feature conflict
   overlayfs: add systemd unit path prefix to FILES:${PN} array
   files: overlayfs-etc: wrap long lines of preinit file
   files: overlayfs-etc: add overlay mount options to preinit

  meta/classes/overlayfs-etc.bbclass |  3 ++-
  meta/classes/overlayfs.bbclass |  3 ++-
  meta/files/overlayfs-etc-preinit.sh.in | 12 ++--
  3 files changed, 14 insertions(+), 4 deletions(-)



Hi Claudius,
Thanks for the patches. It seems like the first 2 are already in master 
branch or I'm missing something?


Yes, that seems to be the case, I missed that.

There are only non-essential white space differences between those and 
mine, so my first two patches can be ignored.


Thanks for spotting that.

regards,
Claudius

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

[OE-core] [PATCH v2 4/4] files: overlayfs-etc: add overlay mount options to preinit

[Claudius Heine]

Overlayfs-etc is useful if the rootfs is read-only. One reason to have
the rootfs read-only is to allow image based updating.

Image based updating will change the underlying root file system, which
is unsupported by overlayfs when with some mount options [1].

This disables those options.

[1] 
https://www.kernel.org/doc/html/latest/filesystems/overlayfs.html?highlight=overlayfs#changes-to-underlying-filesystems

Signed-off-by: Claudius Heine 
---
 meta/files/overlayfs-etc-preinit.sh.in | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/files/overlayfs-etc-preinit.sh.in 
b/meta/files/overlayfs-etc-preinit.sh.in
index e5f5001a05..43c9b04eb9 100644
--- a/meta/files/overlayfs-etc-preinit.sh.in
+++ b/meta/files/overlayfs-etc-preinit.sh.in
@@ -26,6 +26,7 @@ then
 -o upperdir={OVERLAYFS_ETC_MOUNT_POINT}/overlay-etc/upper \
 -o lowerdir=/etc \
 -o workdir={OVERLAYFS_ETC_MOUNT_POINT}/overlay-etc/work \
+-o index=off,xino=off,redirect_dir=off,metacopy=off \
 {OVERLAYFS_ETC_MOUNT_POINT}/overlay-etc/upper /etc || \
 echo "PREINIT: Mounting etc-overlay failed!"
 else
-- 
2.35.1


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

[OE-core] [PATCH v2 3/4] files: overlayfs-etc: wrap long lines of preinit file

[Claudius Heine]

Make it easier to see what is happening by wrapping the overly long
lines in the preinit file for the overlayfs-etc.

Signed-off-by: Claudius Heine 
---
 meta/files/overlayfs-etc-preinit.sh.in | 11 +--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/meta/files/overlayfs-etc-preinit.sh.in 
b/meta/files/overlayfs-etc-preinit.sh.in
index 2ebb6c9224..e5f5001a05 100644
--- a/meta/files/overlayfs-etc-preinit.sh.in
+++ b/meta/files/overlayfs-etc-preinit.sh.in
@@ -16,11 +16,18 @@ mount -t sysfs sysfs /sys
 [ -z "$CONSOLE" ] && CONSOLE="/dev/console"
 
 mkdir -p {OVERLAYFS_ETC_MOUNT_POINT}
-if mount -n -t {OVERLAYFS_ETC_FSTYPE} -o {OVERLAYFS_ETC_MOUNT_OPTIONS} 
{OVERLAYFS_ETC_DEVICE} {OVERLAYFS_ETC_MOUNT_POINT}
+if mount -n -t {OVERLAYFS_ETC_FSTYPE} \
+-o {OVERLAYFS_ETC_MOUNT_OPTIONS} \
+{OVERLAYFS_ETC_DEVICE} {OVERLAYFS_ETC_MOUNT_POINT}
 then
 mkdir -p {OVERLAYFS_ETC_MOUNT_POINT}/overlay-etc/upper
 mkdir -p {OVERLAYFS_ETC_MOUNT_POINT}/overlay-etc/work
-mount -n -t overlay -o 
upperdir={OVERLAYFS_ETC_MOUNT_POINT}/overlay-etc/upper,lowerdir=/etc,workdir={OVERLAYFS_ETC_MOUNT_POINT}/overlay-etc/work
 {OVERLAYFS_ETC_MOUNT_POINT}/overlay-etc/upper /etc || echo "PREINIT: Mounting 
etc-overlay failed!"
+mount -n -t overlay \
+-o upperdir={OVERLAYFS_ETC_MOUNT_POINT}/overlay-etc/upper \
+-o lowerdir=/etc \
+-o workdir={OVERLAYFS_ETC_MOUNT_POINT}/overlay-etc/work \
+{OVERLAYFS_ETC_MOUNT_POINT}/overlay-etc/upper /etc || \
+echo "PREINIT: Mounting etc-overlay failed!"
 else
 echo "PREINIT: Mounting  failed!"
 fi
-- 
2.35.1


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

[OE-core] [PATCH v2 2/4] overlayfs: add systemd unit path prefix to FILES:${PN} array

[Claudius Heine]

The 'FILES:${PN}' array is missing the systemd unit dir prefix causing
them to not be packaged and the build fails with the
`installed-vs-shipped` error.

This adds the `systemd_system_unitdir` variable in front of every unit
file added with this class.

Signed-off-by: Claudius Heine 
---
 meta/classes/overlayfs.bbclass | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/classes/overlayfs.bbclass b/meta/classes/overlayfs.bbclass
index 4a860f7308..25e30aee83 100644
--- a/meta/classes/overlayfs.bbclass
+++ b/meta/classes/overlayfs.bbclass
@@ -103,7 +103,8 @@ python () {
 unitList = unitFileList(d)
 for unit in unitList:
 d.appendVar('SYSTEMD_SERVICE:' + d.getVar('PN'), ' ' + unit)
-d.appendVar('FILES:' + d.getVar('PN'), ' ' + strForBash(unit))
+d.appendVar('FILES:' + d.getVar('PN'), ' ' +
+d.getVar('systemd_system_unitdir') + '/' + strForBash(unit))
 
 d.setVar('OVERLAYFS_UNIT_LIST', ' '.join([strForBash(s) for s in 
unitList]))
 }
-- 
2.35.1


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

[OE-core] [PATCH v2 0/4] Patches and bugfixes for overlayfs

[Claudius Heine]

Hi,

I bundled my patches to overlayfs and overlayfs-etc togehter in this
patchset and fixed the whitespace issues.

regards,
Claudius

--
Changes from v1:
  - fixed whitespace issues: converted tabs to 4 spaces.
  - changed git commit title of patch 4

Claudius Heine (4):
  overlayfs-etc: add condition to package-management feature conflict
  overlayfs: add systemd unit path prefix to FILES:${PN} array
  files: overlayfs-etc: wrap long lines of preinit file
  files: overlayfs-etc: add overlay mount options to preinit

 meta/classes/overlayfs-etc.bbclass |  3 ++-
 meta/classes/overlayfs.bbclass |  3 ++-
 meta/files/overlayfs-etc-preinit.sh.in | 12 ++--
 3 files changed, 14 insertions(+), 4 deletions(-)

-- 
2.35.1


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

[OE-core] [PATCH v2 1/4] overlayfs-etc: add condition to package-management feature conflict

[Claudius Heine]

The conflict with the `package-managment` only happens if the
`overlayfs-etc` class overwrites `/sbin/init`, which is not the case
when `OVERLAYFS_ETC_USE_ORIG_INIT_NAME` is set to `0`.

Signed-off-by: Claudius Heine 
---
 meta/classes/overlayfs-etc.bbclass | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/classes/overlayfs-etc.bbclass 
b/meta/classes/overlayfs-etc.bbclass
index 4ced07ba11..b593a4d9eb 100644
--- a/meta/classes/overlayfs-etc.bbclass
+++ b/meta/classes/overlayfs-etc.bbclass
@@ -26,7 +26,8 @@
 # overlay is out of scope of this class
 
 ROOTFS_POSTPROCESS_COMMAND += '${@bb.utils.contains("IMAGE_FEATURES", 
"overlayfs-etc", "create_overlayfs_etc_preinit;", "", d)}'
-IMAGE_FEATURES_CONFLICTS_overlayfs-etc = "package-management"
+IMAGE_FEATURES_CONFLICTS_overlayfs-etc = " \
+${@ 'package-management' if 
bb.utils.to_boolean(d.getVar('OVERLAYFS_ETC_USE_ORIG_INIT_NAME'), True) else 
''}"
 
 OVERLAYFS_ETC_MOUNT_POINT ??= ""
 OVERLAYFS_ETC_FSTYPE ??= ""
-- 
2.35.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#163019): 
https://lists.openembedded.org/g/openembedded-core/message/163019
Mute This Topic: https://lists.openembedded.org/mt/89685526/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] overlayfs: add systemd unit path prefix to FILES:${PN} array

[Claudius Heine]

Hi Richard,

On 2022-03-10 11:11, Richard Purdie wrote:

On Wed, 2022-03-09 at 15:29 +0100, Claudius Heine wrote:

The 'FILES:${PN}' array is missing the systemd unit dir prefix causing
them to not be packaged and the build fails with the
`installed-vs-shipped` error.

This adds the `systemd_system_unitdir` variable in front of every unit
file added with this class.

Signed-off-by: Claudius Heine 
---
  meta/classes/overlayfs.bbclass | 3 ++-
  1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/classes/overlayfs.bbclass b/meta/classes/overlayfs.bbclass
index 4a860f7308..f525ea6625 100644
--- a/meta/classes/overlayfs.bbclass
+++ b/meta/classes/overlayfs.bbclass
@@ -103,7 +103,8 @@ python () {
  unitList = unitFileList(d)
  for unit in unitList:
  d.appendVar('SYSTEMD_SERVICE:' + d.getVar('PN'), ' ' + unit)
-d.appendVar('FILES:' + d.getVar('PN'), ' ' + strForBash(unit))
+d.appendVar('FILES:' + d.getVar('PN'), ' ' +
+   d.getVar('systemd_system_unitdir') + '/' + strForBash(unit))
  
  d.setVar('OVERLAYFS_UNIT_LIST', ' '.join([strForBash(s) for s in unitList]))

  }


I have to wonder how this is being tested since this patch injects tabs into a
python function and that triggers warnings about tabs in python code.

That caused selftest failures:

https://autobuilder.yoctoproject.org/typhoon/#/builders/80/builds/3227
https://autobuilder.yoctoproject.org/typhoon/#/builders/86/builds/3240
https://autobuilder.yoctoproject.org/typhoon/#/builders/87/builds/3271

I've tweaked the patch to fix the whitespace here and in the other patch but I
notice more patches on the list also with what I suspect are whitespace issues.


Ouch, I will check my editor config. Sorry for this.

regards,
Claudius

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

[OE-core] [PATCH 1/2] files: overlayfs-etc: wrap long lines of preinit file

[Claudius Heine]

Make it easier to see what is happening by wrapping the overly long
lines in the preinit file for the overlayfs-etc.

Signed-off-by: Claudius Heine 
---
 meta/files/overlayfs-etc-preinit.sh.in | 11 +--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/meta/files/overlayfs-etc-preinit.sh.in 
b/meta/files/overlayfs-etc-preinit.sh.in
index 2ebb6c9224..1daf83513e 100644
--- a/meta/files/overlayfs-etc-preinit.sh.in
+++ b/meta/files/overlayfs-etc-preinit.sh.in
@@ -16,11 +16,18 @@ mount -t sysfs sysfs /sys
 [ -z "$CONSOLE" ] && CONSOLE="/dev/console"
 
 mkdir -p {OVERLAYFS_ETC_MOUNT_POINT}
-if mount -n -t {OVERLAYFS_ETC_FSTYPE} -o {OVERLAYFS_ETC_MOUNT_OPTIONS} 
{OVERLAYFS_ETC_DEVICE} {OVERLAYFS_ETC_MOUNT_POINT}
+if mount -n -t {OVERLAYFS_ETC_FSTYPE} \
+   -o {OVERLAYFS_ETC_MOUNT_OPTIONS} \
+   {OVERLAYFS_ETC_DEVICE} {OVERLAYFS_ETC_MOUNT_POINT}
 then
 mkdir -p {OVERLAYFS_ETC_MOUNT_POINT}/overlay-etc/upper
 mkdir -p {OVERLAYFS_ETC_MOUNT_POINT}/overlay-etc/work
-mount -n -t overlay -o 
upperdir={OVERLAYFS_ETC_MOUNT_POINT}/overlay-etc/upper,lowerdir=/etc,workdir={OVERLAYFS_ETC_MOUNT_POINT}/overlay-etc/work
 {OVERLAYFS_ETC_MOUNT_POINT}/overlay-etc/upper /etc || echo "PREINIT: Mounting 
etc-overlay failed!"
+mount -n -t overlay \
+   -o upperdir={OVERLAYFS_ETC_MOUNT_POINT}/overlay-etc/upper \
+   -o lowerdir=/etc \
+   -o workdir={OVERLAYFS_ETC_MOUNT_POINT}/overlay-etc/work \
+   {OVERLAYFS_ETC_MOUNT_POINT}/overlay-etc/upper /etc || \
+   echo "PREINIT: Mounting etc-overlay failed!"
 else
 echo "PREINIT: Mounting  failed!"
 fi
-- 
2.35.1


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

[OE-core] [PATCH 2/2] files: overlayfs-etc: add index=off and xino=off mount option to preinit

[Claudius Heine]

Overlayfs-etc is useful if the rootfs is read-only. One reason to have
the rootfs read-only is to allow image based updating.

Image based updating will change the underlying root file system, which
is unsupported by overlayfs when with some mount options [1].

This disables those options.

[1] 
https://www.kernel.org/doc/html/latest/filesystems/overlayfs.html?highlight=overlayfs#changes-to-underlying-filesystems

Signed-off-by: Claudius Heine 
---
 meta/files/overlayfs-etc-preinit.sh.in | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/files/overlayfs-etc-preinit.sh.in 
b/meta/files/overlayfs-etc-preinit.sh.in
index 1daf83513e..51cbdf25a8 100644
--- a/meta/files/overlayfs-etc-preinit.sh.in
+++ b/meta/files/overlayfs-etc-preinit.sh.in
@@ -26,6 +26,7 @@ then
-o upperdir={OVERLAYFS_ETC_MOUNT_POINT}/overlay-etc/upper \
-o lowerdir=/etc \
-o workdir={OVERLAYFS_ETC_MOUNT_POINT}/overlay-etc/work \
+   -o index=off,xino=off,redirect_dir=off,metacopy=off \
{OVERLAYFS_ETC_MOUNT_POINT}/overlay-etc/upper /etc || \
echo "PREINIT: Mounting etc-overlay failed!"
 else
-- 
2.35.1


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

[OE-core] [PATCH] overlayfs: add systemd unit path prefix to FILES:${PN} array

[Claudius Heine]

The 'FILES:${PN}' array is missing the systemd unit dir prefix causing
them to not be packaged and the build fails with the
`installed-vs-shipped` error.

This adds the `systemd_system_unitdir` variable in front of every unit
file added with this class.

Signed-off-by: Claudius Heine 
---
 meta/classes/overlayfs.bbclass | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/classes/overlayfs.bbclass b/meta/classes/overlayfs.bbclass
index 4a860f7308..f525ea6625 100644
--- a/meta/classes/overlayfs.bbclass
+++ b/meta/classes/overlayfs.bbclass
@@ -103,7 +103,8 @@ python () {
 unitList = unitFileList(d)
 for unit in unitList:
 d.appendVar('SYSTEMD_SERVICE:' + d.getVar('PN'), ' ' + unit)
-d.appendVar('FILES:' + d.getVar('PN'), ' ' + strForBash(unit))
+d.appendVar('FILES:' + d.getVar('PN'), ' ' +
+   d.getVar('systemd_system_unitdir') + '/' + strForBash(unit))
 
 d.setVar('OVERLAYFS_UNIT_LIST', ' '.join([strForBash(s) for s in 
unitList]))
 }
-- 
2.35.1


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

[OE-core] [PATCH] overlayfs-etc: add condition to package-management feature conflict

[Claudius Heine]

The conflict with the `package-managment` only happens if the
`overlayfs-etc` class overwrites `/sbin/init`, which is not the case
when `OVERLAYFS_ETC_USE_ORIG_INIT_NAME` is set to `0`.

Signed-off-by: Claudius Heine 
---
 meta/classes/overlayfs-etc.bbclass | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/classes/overlayfs-etc.bbclass 
b/meta/classes/overlayfs-etc.bbclass
index 4ced07ba11..bf107b67e7 100644
--- a/meta/classes/overlayfs-etc.bbclass
+++ b/meta/classes/overlayfs-etc.bbclass
@@ -26,7 +26,8 @@
 # overlay is out of scope of this class
 
 ROOTFS_POSTPROCESS_COMMAND += '${@bb.utils.contains("IMAGE_FEATURES", 
"overlayfs-etc", "create_overlayfs_etc_preinit;", "", d)}'
-IMAGE_FEATURES_CONFLICTS_overlayfs-etc = "package-management"
+IMAGE_FEATURES_CONFLICTS_overlayfs-etc = " \
+   ${@ 'package-management' if 
bb.utils.to_boolean(d.getVar('OVERLAYFS_ETC_USE_ORIG_INIT_NAME'), True) else 
''}"
 
 OVERLAYFS_ETC_MOUNT_POINT ??= ""
 OVERLAYFS_ETC_FSTYPE ??= ""
-- 
2.33.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#162959): 
https://lists.openembedded.org/g/openembedded-core/message/162959
Mute This Topic: https://lists.openembedded.org/mt/89660257/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

[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 01/33] seatd: add recipe

[Claudius Heine]

On 2022-02-03 14:46, Alexander Kanavin wrote:
The current recipe doesn't enable (or allow to enable) those 
alternatives, so unless someone needs them, and tests that they work, 
there is still no point in splitting.




That is reasonable. I was just a bit concerned about possible RDPENDS 
for other software packages. Depending on `libseat` seems to be more 
future-proof than `seatd` if only the library is required, whatever the 
backend is.


But I don't feel too strongly about this, so your call ;)


Alex

On Thu, 3 Feb 2022 at 14:36, Claudius Heine <mailto:c...@denx.de>> wrote:


On 2022-02-03 14:28, Alexander Kanavin wrote:
 > As far as I understand the library is not useful without the
daemon, so
 > there is no point in packaging them separately. -dev stuff is split
 > automatically.

That is not the impression I got from the readme:

  > # libseat
  >
  > A seat management library allowing applications to use whatever
seat
management is available.
  >
  > Supports:
  >
  >     seatd
  >     (e)logind
  >     embedded seatd for standalone operation
  >
  > Each backend can be compile-time included and is runtime
auto-detected or manually selected with the LIBSEAT_BACKEND environment
variable.
  >
  > Which backend is in use is transparent to the application,
providing
a simple common interface.

https://sr.ht/~kennylevinsen/seatd/#libseat
<https://sr.ht/~kennylevinsen/seatd/#libseat>

I guess the general question is if it makes sense to split them up
for OE.

regards,
Claudius

 >
     > Alex
 >
 > On Thu, 3 Feb 2022 at 13:55, Claudius Heine mailto:c...@denx.de>
 > <mailto:c...@denx.de <mailto:c...@denx.de>>> wrote:
 >
 >     Hi Alexander,
 >
 >     On 2022-02-02 19:27, Alexander Kanavin wrote:
 >      > This is needed to run weston properly as non-root in the
absence
 >      > of systemd-logind, and other compositors will likely
require seatd
 >      > as well.
 >      >
 >      > Signed-off-by: Alexander Kanavin mailto:a...@linutronix.de>
 >     <mailto:a...@linutronix.de <mailto:a...@linutronix.de>>>
 >      > ---
 >      >   meta/conf/distro/include/maintainers.inc |  1 +
 >      >   meta/recipes-core/seatd/seatd/init       | 45
 >     
 >      >   meta/recipes-core/seatd/seatd_0.6.3.bb
<http://seatd_0.6.3.bb> <http://seatd_0.6.3.bb <http://seatd_0.6.3.bb>>
 >       | 29 +++
 >      >   3 files changed, 75 insertions(+)
 >      >   create mode 100644 meta/recipes-core/seatd/seatd/init
 >      >   create mode 100644
meta/recipes-core/seatd/seatd_0.6.3.bb <http://seatd_0.6.3.bb>
 >     <http://seatd_0.6.3.bb <http://seatd_0.6.3.bb>>
 >      >
 >      > diff --git a/meta/conf/distro/include/maintainers.inc
 >     b/meta/conf/distro/include/maintainers.inc
 >      > index ae25287c11..cb289a0eee 100644
 >      > --- a/meta/conf/distro/include/maintainers.inc
 >      > +++ b/meta/conf/distro/include/maintainers.inc
 >      > @@ -684,6 +684,7 @@ RECIPE_MAINTAINER:pn-rxvt-unicode =
 >     "Unassigned mailto:unassig...@yoctoproject.org>
 >     <mailto:unassig...@yoctoproject.org
<mailto:unassig...@yoctoproject.org>>>"
 >      >   RECIPE_MAINTAINER:pn-sato-screenshot = "Ross Burton
 >     mailto:ross.bur...@arm.com>
<mailto:ross.bur...@arm.com <mailto:ross.bur...@arm.com>>>"
 >      >   RECIPE_MAINTAINER:pn-sbc = "Unassigned
 >     mailto:unassig...@yoctoproject.org>
<mailto:unassig...@yoctoproject.org
<mailto:unassig...@yoctoproject.org>>>"
 >      >   RECIPE_MAINTAINER:pn-screen = "Anuj Mittal
 >     mailto:anuj.mit...@intel.com>
<mailto:anuj.mit...@intel.com <mailto:anuj.mit...@intel.com>>>"
 >      > +RECIPE_MAINTAINER:pn-seatd = "Alexander Kanavin
 >     mailto:alex.kana...@gmail.com>
<mailto:alex.kana...@gmail.com <mailto:alex.kana...@gmail.com>>>"
 >      >   RECIPE_MAINTAINER:pn-sed = "Chen Qi
mailto:qi.c...@windriver.com>
 >     <mailto:qi.c...@windriver.com <mailto:qi.c...@windriver.com>>>"
 >      >   RECIPE_MAINTAINER:pn-serf = "Anuj Mittal
mailto:anuj.mit...@intel.com>
 >     <mailto:anuj.mit...@intel.com <mailto:anuj.mit...@intel.com>>>"
 >      >   RECIPE_MAINTAIN

Re: [OE-core] [PATCH 01/33] seatd: add recipe

[Claudius Heine]

On 2022-02-03 14:28, Alexander Kanavin wrote:
As far as I understand the library is not useful without the daemon, so 
there is no point in packaging them separately. -dev stuff is split 
automatically.


That is not the impression I got from the readme:

> # libseat
>
> A seat management library allowing applications to use whatever seat 
management is available.

>
> Supports:
>
> seatd
> (e)logind
> embedded seatd for standalone operation
>
> Each backend can be compile-time included and is runtime 
auto-detected or manually selected with the LIBSEAT_BACKEND environment 
variable.

>
> Which backend is in use is transparent to the application, providing 
a simple common interface.


https://sr.ht/~kennylevinsen/seatd/#libseat

I guess the general question is if it makes sense to split them up for OE.

regards,
Claudius



Alex

On Thu, 3 Feb 2022 at 13:55, Claudius Heine <mailto:c...@denx.de>> wrote:


Hi Alexander,

On 2022-02-02 19:27, Alexander Kanavin wrote:
 > This is needed to run weston properly as non-root in the absence
 > of systemd-logind, and other compositors will likely require seatd
 > as well.
 >
 > Signed-off-by: Alexander Kanavin mailto:a...@linutronix.de>>
 > ---
 >   meta/conf/distro/include/maintainers.inc |  1 +
 >   meta/recipes-core/seatd/seatd/init       | 45

 >   meta/recipes-core/seatd/seatd_0.6.3.bb <http://seatd_0.6.3.bb> 
  | 29 +++

 >   3 files changed, 75 insertions(+)
 >   create mode 100644 meta/recipes-core/seatd/seatd/init
 >   create mode 100644 meta/recipes-core/seatd/seatd_0.6.3.bb
<http://seatd_0.6.3.bb>
 >
 > diff --git a/meta/conf/distro/include/maintainers.inc
b/meta/conf/distro/include/maintainers.inc
 > index ae25287c11..cb289a0eee 100644
 > --- a/meta/conf/distro/include/maintainers.inc
 > +++ b/meta/conf/distro/include/maintainers.inc
 > @@ -684,6 +684,7 @@ RECIPE_MAINTAINER:pn-rxvt-unicode =
"Unassigned mailto:unassig...@yoctoproject.org>>"
 >   RECIPE_MAINTAINER:pn-sato-screenshot = "Ross Burton
mailto:ross.bur...@arm.com>>"
 >   RECIPE_MAINTAINER:pn-sbc = "Unassigned
mailto:unassig...@yoctoproject.org>>"
 >   RECIPE_MAINTAINER:pn-screen = "Anuj Mittal
mailto:anuj.mit...@intel.com>>"
 > +RECIPE_MAINTAINER:pn-seatd = "Alexander Kanavin
mailto:alex.kana...@gmail.com>>"
 >   RECIPE_MAINTAINER:pn-sed = "Chen Qi mailto:qi.c...@windriver.com>>"
 >   RECIPE_MAINTAINER:pn-serf = "Anuj Mittal mailto:anuj.mit...@intel.com>>"
 >   RECIPE_MAINTAINER:pn-setserial = "Yi Zhao
mailto:yi.z...@windriver.com>>"
 > diff --git a/meta/recipes-core/seatd/seatd/init
b/meta/recipes-core/seatd/seatd/init
 > new file mode 100644
 > index 00..0589c765ac
 > --- /dev/null
 > +++ b/meta/recipes-core/seatd/seatd/init
 > @@ -0,0 +1,45 @@
 > +#!/bin/sh
 > +#
 > +### BEGIN INIT INFO
 > +# Provides: seatd
 > +# Required-Start: $local_fs $remote_fs
 > +# Required-Stop: $local_fs $remote_fs
 > +# Default-Start:     2 3 4 5
 > +# Default-Stop:      0 1 6
 > +### END INIT INFO
 > +
 > +killproc() {
 > +        pid=`/bin/pidof $1`
 > +        [ "$pid" != "" ] && kill $pid
 > +}
 > +
 > +case "$1" in
 > +  start)
 > +        seatd -g video -n 1 > /tmp/seatd-start-notify &
 > +        [ -s /tmp/seatd-start-notify ] && exit 0
 > +        sleep 0.1
 > +        [ -s /tmp/seatd-start-notify ] && exit 0
 > +        sleep 0.5
 > +        [ -s /tmp/seatd-start-notify ] && exit 0
 > +        sleep 5
 > +        [ -s /tmp/seatd-start-notify ] && exit 0
 > +        exit 1
 > +  ;;
 > +
 > +  stop)
 > +        echo "Stopping seatd"
 > +        killproc seatd
 > +  ;;
 > +
 > +  restart)
 > +     $0 stop
 > +        sleep 1
 > +        $0 start
 > +  ;;
 > +
 > +  *)
 > +        echo "usage: $0 { start | stop | restart }"
 > +  ;;
 > +esac
 > +
 > +exit 0
 > diff --git a/meta/recipes-core/seatd/seatd_0.6.3.bb
<http://seatd_0.6.3.bb> b/meta/recipes-core/seatd/seatd_0.6.3.bb
<http://seatd_0.6.3.bb>
 > new file mode 100644
 > index 00..0e1a79dddf
 > --- /dev/null
 > +++ b/meta/recipes-core/seatd/seatd_0.6.3.bb <http:

Re: [OE-core] [PATCH 01/33] seatd: add recipe

[Claudius Heine]

Hi Alexander,

On 2022-02-02 19:27, Alexander Kanavin wrote:

This is needed to run weston properly as non-root in the absence
of systemd-logind, and other compositors will likely require seatd
as well.

Signed-off-by: Alexander Kanavin 
---
  meta/conf/distro/include/maintainers.inc |  1 +
  meta/recipes-core/seatd/seatd/init   | 45 
  meta/recipes-core/seatd/seatd_0.6.3.bb   | 29 +++
  3 files changed, 75 insertions(+)
  create mode 100644 meta/recipes-core/seatd/seatd/init
  create mode 100644 meta/recipes-core/seatd/seatd_0.6.3.bb

diff --git a/meta/conf/distro/include/maintainers.inc 
b/meta/conf/distro/include/maintainers.inc
index ae25287c11..cb289a0eee 100644
--- a/meta/conf/distro/include/maintainers.inc
+++ b/meta/conf/distro/include/maintainers.inc
@@ -684,6 +684,7 @@ RECIPE_MAINTAINER:pn-rxvt-unicode = "Unassigned 
"
  RECIPE_MAINTAINER:pn-sato-screenshot = "Ross Burton "
  RECIPE_MAINTAINER:pn-sbc = "Unassigned "
  RECIPE_MAINTAINER:pn-screen = "Anuj Mittal "
+RECIPE_MAINTAINER:pn-seatd = "Alexander Kanavin "
  RECIPE_MAINTAINER:pn-sed = "Chen Qi "
  RECIPE_MAINTAINER:pn-serf = "Anuj Mittal "
  RECIPE_MAINTAINER:pn-setserial = "Yi Zhao "
diff --git a/meta/recipes-core/seatd/seatd/init 
b/meta/recipes-core/seatd/seatd/init
new file mode 100644
index 00..0589c765ac
--- /dev/null
+++ b/meta/recipes-core/seatd/seatd/init
@@ -0,0 +1,45 @@
+#!/bin/sh
+#
+### BEGIN INIT INFO
+# Provides: seatd
+# Required-Start: $local_fs $remote_fs
+# Required-Stop: $local_fs $remote_fs
+# Default-Start: 2 3 4 5
+# Default-Stop:  0 1 6
+### END INIT INFO
+
+killproc() {
+pid=`/bin/pidof $1`
+[ "$pid" != "" ] && kill $pid
+}
+
+case "$1" in
+  start)
+seatd -g video -n 1 > /tmp/seatd-start-notify &
+[ -s /tmp/seatd-start-notify ] && exit 0
+sleep 0.1
+[ -s /tmp/seatd-start-notify ] && exit 0
+sleep 0.5
+[ -s /tmp/seatd-start-notify ] && exit 0
+sleep 5
+[ -s /tmp/seatd-start-notify ] && exit 0
+exit 1
+  ;;
+
+  stop)
+echo "Stopping seatd"
+killproc seatd
+  ;;
+
+  restart)
+   $0 stop
+sleep 1
+$0 start
+  ;;
+
+  *)
+echo "usage: $0 { start | stop | restart }"
+  ;;
+esac
+
+exit 0
diff --git a/meta/recipes-core/seatd/seatd_0.6.3.bb 
b/meta/recipes-core/seatd/seatd_0.6.3.bb
new file mode 100644
index 00..0e1a79dddf
--- /dev/null
+++ b/meta/recipes-core/seatd/seatd_0.6.3.bb
@@ -0,0 +1,29 @@
+SUMMARY = "A minimal seat management daemon, and a universal seat management 
library."
+DESCRIPTION = "Seat management takes care of mediating access to shared devices 
(graphics, input), without requiring the applications needing access to be root."
+HOMEPAGE = "https://git.sr.ht/~kennylevinsen/seatd;
+
+LICENSE = "MIT"
+
+LIC_FILES_CHKSUM = "file://LICENSE;md5=715a99d2dd552e6188e74d4ed2914d5a"
+
+SRC_URI = "git://git.sr.ht/~kennylevinsen/seatd;protocol=https;branch=master \
+   file://init"
+SRCREV = "88db55f6068c1c01d85b61aa6adff0a6b2a8dce8"
+S = "${WORKDIR}/git"
+
+inherit meson pkgconfig update-rc.d
+
+PACKAGECONFIG ?= "libseat-builtin"
+
+PACKAGECONFIG[libseat-builtin] = 
"-Dlibseat-builtin=enabled,-Dlibseat-builtin=disabled"
+
+do_install:append() {
+if [ "${VIRTUAL-RUNTIME_init_manager}" != "systemd" ]; then
+install -Dm755 ${WORKDIR}/init ${D}/${sysconfdir}/init.d/seatd
+fi
+}
+
+INITSCRIPT_NAME = "seatd"
+INITSCRIPT_PARAMS = "start 9 5 2 . stop 20 0 1 6 ."
+INHIBIT_UPDATERCD_BBCLASS = 
"${@oe.utils.conditional('VIRTUAL-RUNTIME_init_manager', 'systemd', '1', '', 
d)}"
+



I was looking into seatd as well, in my recipe I split it up a bit:

```
PACKAGES =+ "${PN}-launch libseat libseat-dev"

FILES:${PN} = "${bindir}/seatd"
FILES:${PN}-launch = "${bindir}/seatd-launch"
FILES:libseat = "${libdir}/lib*${SOLIBS}"
FILES:libseat-dev = " \
   ${includedir} ${FILES_SOLIBSDEV} \
   ${libdir}/pkgconfig"

RDEPENDS:${PN} = "libseat (= ${EXTENDPKGV})"
RDEPENDS:${PN}-launch = "${PN} (= ${EXTENDPKGV})"
RDEPENDS:libseat-dev = "libseat (= ${EXTENDPKGV})"
```

regards,
Claudius

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#161277): 
https://lists.openembedded.org/g/openembedded-core/message/161277
Mute This Topic: https://lists.openembedded.org/mt/88865275/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 05/12] gstreamer1.0-plugins-ugly: 1.18.5 -> 1.19.90

[Claudius Heine]

Hi Jose,

On 2022-02-02 18:58, Jose Quaresma wrote:
[...]

@@ -13,7 +13,7 @@ LICENSE_FLAGS = "commercial"
  SRC_URI = " \

https://gstreamer.freedesktop.org/src/gst-plugins-ugly/gst-plugins-ugly-${PV}.tar.xz


\
              "
-SRC_URI[sha256sum] =
"df32803e98f8a9979373fa2ca7e05e62f977b1097576d3a80619d9f5c69f66d9"
+SRC_URI[sha256sum] =
"5242a42c3fad209030bd2e8630b08ae93821384c2278965764a039d31ab2a246"

  S = "${WORKDIR}/gst-plugins-ugly-${PV}"

@@ -23,7 +23,7 @@ GST_PLUGIN_SET_HAS_EXAMPLES = "0"

  PACKAGECONFIG ??= " \
      ${GSTREAMER_ORC} \
-    a52dec mpeg2dec \
+    a52dec mpeg2dec gpl \


This will require commercial licence support
https://docs.yoctoproject.org/dev-manual/common-tasks.html?highlight=license_flags#enabling-commercially-licensed-recipes 



Yes. I am not sure if I understand your point. The `LICENSE_FLAGS = 
"commercial"` is already set in the recipe, see first line of this diff.


regards,
Claudius

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#161270): 
https://lists.openembedded.org/g/openembedded-core/message/161270
Mute This Topic: https://lists.openembedded.org/mt/88858022/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 04/12] gstreamer1.0-plugins-bad: 1.18.5 -> 1.19.90

[Claudius Heine]

On 2022-02-02 18:57, Jose Quaresma wrote:

Hi, Claudius

There are some new options on the gstreamer bad that are not updated on 
this patch.

more precisely what will be done to use the new gpl option ?


Hmm... I am not sure. `gpl` probably needs to be added because we 
already have `faad` and others which require it. I will at least that in v2.


For all other features I am open to suggestions about which should be added.

regards,
Claudius

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#161269): 
https://lists.openembedded.org/g/openembedded-core/message/161269
Mute This Topic: https://lists.openembedded.org/mt/88858014/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 02/12] gstreamer1.0-plugins-base: 1.18.5 -> 1.19.90

[Claudius Heine]

Hi Jose,

On 2022-02-02 18:57, Jose Quaresma wrote:
[...]

  SRC_URI =

"https://gstreamer.freedesktop.org/src/gst-plugins-base/gst-plugins-base-${PV}.tar.xz


\

file://0001-ENGR00312515-get-caps-from-src-pad-when-query-caps.patch \

             file://0003-viv-fb-Make-sure-config.h-is-included.patch \
             file://0002-ssaparse-enhance-SSA-text-lines-parsing.patch \
-           file://0004-glimagesink-Downrank-to-marginal.patch \


Why is this patch dropped? As Far as I know it is not merged upstream.


Good catch! Thanks. Not sure why I dropped it. I will add a rebased 
version to v2.

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#161267): 
https://lists.openembedded.org/g/openembedded-core/message/161267
Mute This Topic: https://lists.openembedded.org/mt/88858012/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] rng-tools: add systemd-udev-settle wants to service

[Claudius Heine]

On 2022-02-02 17:26, Kyle Russell wrote:

Thanks, Claudius.

On Wed, Feb 2, 2022 at 8:08 AM Claudius Heine <mailto:c...@denx.de>> wrote:


Hi Kyle,

On 2022-02-02 13:38, Kyle Russell wrote:
 > Is this the correct approach?  Even the
systemd-udev-settle.service man
 > pages recommends not using its service.  Were the kernel modules
really
 > not loaded when rngd started?  Or is the original problem just a
matter
 > of waiting for sufficient entropy?

IIRC, the rngd could not find any random source device node (/dev/hwrng
in that case), so the service failed to start.


If /dev/hwrng didn't exist, this feels like the original problem was a 
misconfigured

kernel or module that wasn't being loaded properly.


Yes, however it is a timing issue. The module was loaded properly at 
bootup, however at the time rngd was started the module was not loaded 
yet and thus the service fails to start. If it would be delayed until 
the module is loaded everything would be fine.


It does not happen if the module is compiled into the kernel or if a 
initramfs is used which loads the module (I think). I our case it 
happend with the caam module as an external module loaded on boot from 
the real root file system.



The patch you are commenting on only adds `Wants` weak dependency to
make sure `systemd-udev-settle.service` is pulled in to the job queue,
the `After` ordering rule was already there.


Correct.  Just because an `After` exists does not mean the service gets 
pulled into
the job queue, so prior to this change no other service was causing the 
deprecated
systemd-udev-settle.service to be run during boot.  But now, every 
device including

openssh (which has a default PACKAGECONFIG option for rng-tools) now depends
on this deprecated service, which may cause unexpected boot delays.

So changing this service file to be triggered by a udev event or maybe
wrap it in a script, which makes sure the right modules are loaded and
device nodes are available, could be an improvement, but it would be
out
of scope of this patch IMO.


I'm more curious whether this change should be reverted from upstream.  
It seems
like a drop-in file could have been applied to your distro instead of 
adding a dependency

on a deprecated service for all openssh users.


This patch just adds a missing entry into the service file. If you have 
solved the described issue in some way and can revert this patch and 
remove the `Wants=systemd-udev-settle.service` then you can also remove 
the `After=systemd-udev-settle.service` at the same time and at that 
point you can just remove both of those entries directly in the patch 
that solved the timing issue.


I agree that `systemd-udev-settle.service` should probably not be used 
anymore, however that file already used it in a non-functional way and 
all this patch did was make it fulfill its intended function.


In retrospect I probably should have tried to find a way to remove the 
usage of `systemd-udev-settle.service` completely, when I looked into 
the issue, however all this patch in essence does is revive dead code, 
which was already in place.


Also I think at that time I couldn't find a more precise instrument in 
systemds massive toolbox to delay the start of rngd and services that 
should be started in succession until the just the hardware random 
generator device is ready and `After=systemd-udev-settle.service` was 
already there. I guess some `ExecStartPre=` script which delays the 
start until the conditions are met could be implemented, but that seems 
a bit hackish.


regards,
Claudius

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

[OE-core] [PATCH 12/12] gst-devtools: 1.18.5 -> 1.19.90

[Claudius Heine]

Signed-off-by: Claudius Heine 
---
 .../{gst-devtools_1.18.5.bb => gst-devtools_1.19.90.bb} | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
 rename meta/recipes-multimedia/gstreamer/{gst-devtools_1.18.5.bb => 
gst-devtools_1.19.90.bb} (94%)

diff --git a/meta/recipes-multimedia/gstreamer/gst-devtools_1.18.5.bb 
b/meta/recipes-multimedia/gstreamer/gst-devtools_1.19.90.bb
similarity index 94%
rename from meta/recipes-multimedia/gstreamer/gst-devtools_1.18.5.bb
rename to meta/recipes-multimedia/gstreamer/gst-devtools_1.19.90.bb
index 1b46b89cb9..fc3ecd46a1 100644
--- a/meta/recipes-multimedia/gstreamer/gst-devtools_1.18.5.bb
+++ b/meta/recipes-multimedia/gstreamer/gst-devtools_1.19.90.bb
@@ -12,7 +12,7 @@ SRC_URI = 
"https://gstreamer.freedesktop.org/src/gst-devtools/gst-devtools-${PV}
file://0001-connect-has-a-different-signature-on-musl.patch \
"
 
-SRC_URI[sha256sum] = 
"fecffc86447daf5c2a06843c757a991d745caa2069446a0d746e99b13f7cb079"
+SRC_URI[sha256sum] = 
"a51061e4a799a24dcc6ccea5350ab1366fcae53331f8067bd6f52d50c6d822f3"
 
 DEPENDS = "json-glib glib-2.0 glib-2.0-native gstreamer1.0 
gstreamer1.0-plugins-base"
 RRECOMMENDS:${PN} = "git"
-- 
2.34.1


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

[OE-core] [PATCH 09/12] gstreamer1.0-vaapi: 1.18.5 -> 1.19.90

[Claudius Heine]

Signed-off-by: Claudius Heine 
---
 ...treamer1.0-vaapi_1.18.5.bb => gstreamer1.0-vaapi_1.19.90.bb} | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
 rename meta/recipes-multimedia/gstreamer/{gstreamer1.0-vaapi_1.18.5.bb => 
gstreamer1.0-vaapi_1.19.90.bb} (95%)

diff --git a/meta/recipes-multimedia/gstreamer/gstreamer1.0-vaapi_1.18.5.bb 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-vaapi_1.19.90.bb
similarity index 95%
rename from meta/recipes-multimedia/gstreamer/gstreamer1.0-vaapi_1.18.5.bb
rename to meta/recipes-multimedia/gstreamer/gstreamer1.0-vaapi_1.19.90.bb
index 9a68a3fadf..aa3509c7bb 100644
--- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-vaapi_1.18.5.bb
+++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-vaapi_1.19.90.bb
@@ -11,7 +11,7 @@ LIC_FILES_CHKSUM = 
"file://COPYING.LIB;md5=4fbd65380cdd255951079008b364516c"
 
 SRC_URI = 
"https://gstreamer.freedesktop.org/src/${REALPN}/${REALPN}-${PV}.tar.xz;
 
-SRC_URI[sha256sum] = 
"4a460fb95559f41444eb24864ad2d9e37922b6eea941510310319fc3e0ba727b"
+SRC_URI[sha256sum] = 
"437f614d6fc8904f97feb6a297e42f09d9cd9c6c05f1ebc96a95f1c1b89f2f69"
 
 S = "${WORKDIR}/${REALPN}-${PV}"
 DEPENDS = "libva gstreamer1.0 gstreamer1.0-plugins-base 
gstreamer1.0-plugins-bad"
-- 
2.34.1


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

[OE-core] [PATCH 08/12] gstreamer1.0-omx: 1.18.5 -> 1.19.90

[Claudius Heine]

Signed-off-by: Claudius Heine 
---
 .../{gstreamer1.0-omx_1.18.5.bb => gstreamer1.0-omx_1.19.90.bb} | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
 rename meta/recipes-multimedia/gstreamer/{gstreamer1.0-omx_1.18.5.bb => 
gstreamer1.0-omx_1.19.90.bb} (95%)

diff --git a/meta/recipes-multimedia/gstreamer/gstreamer1.0-omx_1.18.5.bb 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-omx_1.19.90.bb
similarity index 95%
rename from meta/recipes-multimedia/gstreamer/gstreamer1.0-omx_1.18.5.bb
rename to meta/recipes-multimedia/gstreamer/gstreamer1.0-omx_1.19.90.bb
index b2c1eb3ea0..45a304f624 100644
--- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-omx_1.18.5.bb
+++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-omx_1.19.90.bb
@@ -10,7 +10,7 @@ LIC_FILES_CHKSUM = 
"file://COPYING;md5=4fbd65380cdd255951079008b364516c \
 
 SRC_URI = "https://gstreamer.freedesktop.org/src/gst-omx/gst-omx-${PV}.tar.xz;
 
-SRC_URI[sha256sum] = 
"2cd457c1e8deb1a9b39608048fb36a44f6c9a864a6b6115b1453a32e7be93b42"
+SRC_URI[sha256sum] = 
"1699520169e5fb61e4583dfbc18c0674485592ca5bba1da34fd25bb8bed0a0c7"
 
 S = "${WORKDIR}/gst-omx-${PV}"
 
-- 
2.34.1


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

[OE-core] [PATCH 05/12] gstreamer1.0-plugins-ugly: 1.18.5 -> 1.19.90

[Claudius Heine]

Feature `gpl` was added, which allows building gpl licensed codecs.

Signed-off-by: Claudius Heine 
---
 ...s-ugly_1.18.5.bb => gstreamer1.0-plugins-ugly_1.19.90.bb} | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)
 rename meta/recipes-multimedia/gstreamer/{gstreamer1.0-plugins-ugly_1.18.5.bb 
=> gstreamer1.0-plugins-ugly_1.19.90.bb} (89%)

diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-ugly_1.18.5.bb 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-ugly_1.19.90.bb
similarity index 89%
rename from 
meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-ugly_1.18.5.bb
rename to meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-ugly_1.19.90.bb
index 9777aaee19..53d8d0ed07 100644
--- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-ugly_1.18.5.bb
+++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-ugly_1.19.90.bb
@@ -13,7 +13,7 @@ LICENSE_FLAGS = "commercial"
 SRC_URI = " \
 
https://gstreamer.freedesktop.org/src/gst-plugins-ugly/gst-plugins-ugly-${PV}.tar.xz
 \
 "
-SRC_URI[sha256sum] = 
"df32803e98f8a9979373fa2ca7e05e62f977b1097576d3a80619d9f5c69f66d9"
+SRC_URI[sha256sum] = 
"5242a42c3fad209030bd2e8630b08ae93821384c2278965764a039d31ab2a246"
 
 S = "${WORKDIR}/gst-plugins-ugly-${PV}"
 
@@ -23,7 +23,7 @@ GST_PLUGIN_SET_HAS_EXAMPLES = "0"
 
 PACKAGECONFIG ??= " \
 ${GSTREAMER_ORC} \
-a52dec mpeg2dec \
+a52dec mpeg2dec gpl \
 "
 
 PACKAGECONFIG[a52dec]   = "-Da52dec=enabled,-Da52dec=disabled,liba52"
@@ -33,6 +33,7 @@ PACKAGECONFIG[cdio] = 
"-Dcdio=enabled,-Dcdio=disabled,libcdio"
 PACKAGECONFIG[dvdread]  = "-Ddvdread=enabled,-Ddvdread=disabled,libdvdread"
 PACKAGECONFIG[mpeg2dec] = "-Dmpeg2dec=enabled,-Dmpeg2dec=disabled,mpeg2dec"
 PACKAGECONFIG[x264] = "-Dx264=enabled,-Dx264=disabled,x264"
+PACKAGECONFIG[gpl]  = "-Dgpl=enabled,-Dgpl=disabled"
 
 EXTRA_OEMESON += " \
 -Ddoc=disabled \
-- 
2.34.1


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

[OE-core] [PATCH 11/12] gst-examples: 1.18.5 -> 1.19.90

[Claudius Heine]

Signed-off-by: Claudius Heine 
---
 .../gst-examples/0001-Make-player-examples-installable.patch| 2 +-
 .../{gst-examples_1.18.5.bb => gst-examples_1.19.90.bb} | 0
 2 files changed, 1 insertion(+), 1 deletion(-)
 rename meta/recipes-multimedia/gstreamer/{gst-examples_1.18.5.bb => 
gst-examples_1.19.90.bb} (100%)

diff --git 
a/meta/recipes-multimedia/gstreamer/gst-examples/0001-Make-player-examples-installable.patch
 
b/meta/recipes-multimedia/gstreamer/gst-examples/0001-Make-player-examples-installable.patch
index ab93c13244..cae00045a3 100644
--- 
a/meta/recipes-multimedia/gstreamer/gst-examples/0001-Make-player-examples-installable.patch
+++ 
b/meta/recipes-multimedia/gstreamer/gst-examples/0001-Make-player-examples-installable.patch
@@ -32,6 +32,6 @@ index f7a7419..6281130 100644
  gtk_play_resources,
 'gtk-video-renderer.h',
 'gtk-video-renderer.c'],
-+install: true,
++  install: true,
dependencies : [glib_dep, gobject_dep, gmodule_dep, gst_dep, 
gsttag_dep, gstplayer_dep, gtk_dep, x11_dep])
  endif
diff --git a/meta/recipes-multimedia/gstreamer/gst-examples_1.18.5.bb 
b/meta/recipes-multimedia/gstreamer/gst-examples_1.19.90.bb
similarity index 100%
rename from meta/recipes-multimedia/gstreamer/gst-examples_1.18.5.bb
rename to meta/recipes-multimedia/gstreamer/gst-examples_1.19.90.bb
-- 
2.34.1


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

[OE-core] [PATCH 10/12] gstreamer1.0-rtsp-server: 1.18.5 -> 1.19.90

[Claudius Heine]

Signed-off-by: Claudius Heine 
---
 ...p-server_1.18.5.bb => gstreamer1.0-rtsp-server_1.19.90.bb} | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
 rename meta/recipes-multimedia/gstreamer/{gstreamer1.0-rtsp-server_1.18.5.bb 
=> gstreamer1.0-rtsp-server_1.19.90.bb} (82%)

diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-rtsp-server_1.18.5.bb 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-rtsp-server_1.19.90.bb
similarity index 82%
rename from meta/recipes-multimedia/gstreamer/gstreamer1.0-rtsp-server_1.18.5.bb
rename to meta/recipes-multimedia/gstreamer/gstreamer1.0-rtsp-server_1.19.90.bb
index 50426ad46d..1b877edd6d 100644
--- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-rtsp-server_1.18.5.bb
+++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-rtsp-server_1.19.90.bb
@@ -2,7 +2,7 @@ SUMMARY = "A library on top of GStreamer for building an RTSP 
server"
 HOMEPAGE = "http://cgit.freedesktop.org/gstreamer/gst-rtsp-server/;
 SECTION = "multimedia"
 LICENSE = "LGPLv2"
-LIC_FILES_CHKSUM = "file://COPYING;md5=6762ed442b3822387a51c92d928ead0d"
+LIC_FILES_CHKSUM = "file://COPYING;md5=69333daa044cb77e486cc36129f7a770"
 
 DEPENDS = "gstreamer1.0 gstreamer1.0-plugins-base"
 
@@ -10,7 +10,7 @@ PNREAL = "gst-rtsp-server"
 
 SRC_URI = 
"https://gstreamer.freedesktop.org/src/${PNREAL}/${PNREAL}-${PV}.tar.xz;
 
-SRC_URI[sha256sum] = 
"04d63bf48816c6f41c73f6de0f912a7cef0aab39c44162a7bcece1923dfc9d1f"
+SRC_URI[sha256sum] = 
"eb610cad65101c1df68ec8771a60ff3a3ba0d00a7f81b4b0048d7f49dd02d06c"
 
 S = "${WORKDIR}/${PNREAL}-${PV}"
 
-- 
2.34.1


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

[OE-core] [PATCH 06/12] gstreamer1.0-libav: 1.18.5 -> 1.19.90

[Claudius Heine]

Signed-off-by: Claudius Heine 
---
 ...st-libav-fix-build-with-ffmpeg-5.0.0.patch | 346 --
 18.5.bb => gstreamer1.0-libav_1.19.90.bb} |   5 +-
 2 files changed, 2 insertions(+), 349 deletions(-)
 delete mode 100644 
meta/recipes-multimedia/gstreamer/gstreamer1.0-libav/0001-gst-libav-fix-build-with-ffmpeg-5.0.0.patch
 rename meta/recipes-multimedia/gstreamer/{gstreamer1.0-libav_1.18.5.bb => 
gstreamer1.0-libav_1.19.90.bb} (78%)

diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-libav/0001-gst-libav-fix-build-with-ffmpeg-5.0.0.patch
 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-libav/0001-gst-libav-fix-build-with-ffmpeg-5.0.0.patch
deleted file mode 100644
index 022ff9af29..00
--- 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-libav/0001-gst-libav-fix-build-with-ffmpeg-5.0.0.patch
+++ /dev/null
@@ -1,346 +0,0 @@
-From 38d10ee800e42afeacc6bee714216e4c974c11f5 Mon Sep 17 00:00:00 2001
-From: Xi Ruoyao 
-Date: Mon, 17 Jan 2022 01:33:47 +0800
-Subject: [PATCH] gst-libav: fix build with ffmpeg-5.0.0
-
-Latest ffmpeg has removed avcodec_get_context_defaults(), and its
-documentation says a new AVCodecContext should be allocated for this
-purpose.  The pointer returned by avcodec_find_decoder() is now
-const-qualified so we also need to adjust for it.  And, AVCOL_RANGE_MPEG
-is now rejected with strict_std_compliance > FF_COMPLIANCE_UNOFFICIAL.
-
-Part-of: 
<https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1531>
-Upstream-Status: Backport
-Signed-off-by: Alexander Kanavin 

- ext/libav/gstavauddec.c   | 22 -
- ext/libav/gstavaudenc.c   | 40 +++
- ext/libav/gstavcodecmap.c |  7 ---
- ext/libav/gstavutils.c|  2 +-
- ext/libav/gstavviddec.c   | 28 +++
- ext/libav/gstavvidenc.c   | 21 ++--
- 6 files changed, 54 insertions(+), 66 deletions(-)
-
-diff --git a/ext/libav/gstavauddec.c b/ext/libav/gstavauddec.c
-index baf7aa5..b03a724 100644
 a/ext/libav/gstavauddec.c
-+++ b/ext/libav/gstavauddec.c
-@@ -168,12 +168,7 @@ gst_ffmpegauddec_finalize (GObject * object)
-   GstFFMpegAudDec *ffmpegdec = (GstFFMpegAudDec *) object;
- 
-   av_frame_free (>frame);
--
--  if (ffmpegdec->context != NULL) {
--gst_ffmpeg_avcodec_close (ffmpegdec->context);
--av_free (ffmpegdec->context);
--ffmpegdec->context = NULL;
--  }
-+  avcodec_free_context (>context);
- 
-   G_OBJECT_CLASS (parent_class)->finalize (object);
- }
-@@ -193,14 +188,12 @@ gst_ffmpegauddec_close (GstFFMpegAudDec * ffmpegdec, 
gboolean reset)
-   gst_ffmpeg_avcodec_close (ffmpegdec->context);
-   ffmpegdec->opened = FALSE;
- 
--  if (ffmpegdec->context->extradata) {
--av_free (ffmpegdec->context->extradata);
--ffmpegdec->context->extradata = NULL;
--  }
-+  av_freep (>context->extradata);
- 
-   if (reset) {
--if (avcodec_get_context_defaults3 (ffmpegdec->context,
--oclass->in_plugin) < 0) {
-+avcodec_free_context (>context);
-+ffmpegdec->context = avcodec_alloc_context3 (oclass->in_plugin);
-+if (ffmpegdec->context == NULL) {
-   GST_DEBUG_OBJECT (ffmpegdec, "Failed to set context defaults");
-   return FALSE;
- }
-@@ -219,8 +212,9 @@ gst_ffmpegauddec_start (GstAudioDecoder * decoder)
-   oclass = (GstFFMpegAudDecClass *) (G_OBJECT_GET_CLASS (ffmpegdec));
- 
-   GST_OBJECT_LOCK (ffmpegdec);
--  gst_ffmpeg_avcodec_close (ffmpegdec->context);
--  if (avcodec_get_context_defaults3 (ffmpegdec->context, oclass->in_plugin) < 
0) {
-+  avcodec_free_context (>context);
-+  ffmpegdec->context = avcodec_alloc_context3 (oclass->in_plugin);
-+  if (ffmpegdec->context == NULL) {
- GST_DEBUG_OBJECT (ffmpegdec, "Failed to set context defaults");
- GST_OBJECT_UNLOCK (ffmpegdec);
- return FALSE;
-diff --git a/ext/libav/gstavaudenc.c b/ext/libav/gstavaudenc.c
-index 3ff6432..689982f 100644
 a/ext/libav/gstavaudenc.c
-+++ b/ext/libav/gstavaudenc.c
-@@ -175,10 +175,8 @@ gst_ffmpegaudenc_finalize (GObject * object)
- 
-   /* clean up remaining allocated data */
-   av_frame_free (>frame);
--  gst_ffmpeg_avcodec_close (ffmpegaudenc->context);
--  gst_ffmpeg_avcodec_close (ffmpegaudenc->refcontext);
--  av_free (ffmpegaudenc->context);
--  av_free (ffmpegaudenc->refcontext);
-+  avcodec_free_context (>context);
-+  avcodec_free_context (>refcontext);
- 
-   G_OBJECT_CLASS (parent_class)->finalize (object);
- }
-@@ -193,9 +191,9 @@ gst_ffmpegaudenc_start (GstAudioEncoder * encoder)
-   ffmpegaudenc->opened = FALSE;
-   ffmpegaudenc->need_reopen = FALSE;
- 
--  gst_ffmpeg_avcodec_close (ffmpegaudenc->context);
--  if (avcodec_get_context_defaults3 (ffmpegaudenc->context,
--  oclass->in_plugin) < 0) {
-+  avcodec_free_context (>context);
-+  ffmpegaudenc->c

[OE-core] [PATCH 07/12] gstreamer1.0-python: 1.18.5 -> 1.19.90

[Claudius Heine]

Signed-off-by: Claudius Heine 
---
 ...eamer1.0-python_1.18.5.bb => gstreamer1.0-python_1.19.90.bb} | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
 rename meta/recipes-multimedia/gstreamer/{gstreamer1.0-python_1.18.5.bb => 
gstreamer1.0-python_1.19.90.bb} (91%)

diff --git a/meta/recipes-multimedia/gstreamer/gstreamer1.0-python_1.18.5.bb 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-python_1.19.90.bb
similarity index 91%
rename from meta/recipes-multimedia/gstreamer/gstreamer1.0-python_1.18.5.bb
rename to meta/recipes-multimedia/gstreamer/gstreamer1.0-python_1.19.90.bb
index 1dd7d0d09a..448bfa8bea 100644
--- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-python_1.18.5.bb
+++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-python_1.19.90.bb
@@ -8,7 +8,7 @@ LICENSE = "LGPLv2.1"
 LIC_FILES_CHKSUM = "file://COPYING;md5=c34deae4e395ca07e725ab0076a5f740"
 
 SRC_URI = 
"https://gstreamer.freedesktop.org/src/${PNREAL}/${PNREAL}-${PV}.tar.xz;
-SRC_URI[sha256sum] = 
"533685871305959d6db89507f3b3aa6c765c2f2b0dacdc32c5a6543e72e5bc52"
+SRC_URI[sha256sum] = 
"5945f2295318e76e9102a77efabccb440e820c5de83a094e2dc5ad0e92d2d78d"
 
 DEPENDS = "gstreamer1.0 gstreamer1.0-plugins-base python3-pygobject"
 RDEPENDS:${PN} += "gstreamer1.0 gstreamer1.0-plugins-base python3-pygobject"
-- 
2.34.1


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

[OE-core] [PATCH 04/12] gstreamer1.0-plugins-bad: 1.18.5 -> 1.19.90

[Claudius Heine]

Features `libmms` and `ofa` where removed upstream.

Patches where rebased.

Signed-off-by: Claudius Heine 
---
 ...missing-opencv-data-dir-in-yocto-buil.patch | 17 -
 .../0005-msdk-fix-includedir-path.patch| 18 +-
 bb => gstreamer1.0-plugins-bad_1.19.90.bb} |  4 +---
 3 files changed, 18 insertions(+), 21 deletions(-)
 rename meta/recipes-multimedia/gstreamer/{gstreamer1.0-plugins-bad_1.18.5.bb 
=> gstreamer1.0-plugins-bad_1.19.90.bb} (97%)

diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad/0004-opencv-resolve-missing-opencv-data-dir-in-yocto-buil.patch
 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad/0004-opencv-resolve-missing-opencv-data-dir-in-yocto-buil.patch
index b816709066..2b9854b45e 100644
--- 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad/0004-opencv-resolve-missing-opencv-data-dir-in-yocto-buil.patch
+++ 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad/0004-opencv-resolve-missing-opencv-data-dir-in-yocto-buil.patch
@@ -19,31 +19,30 @@ diff --git a/ext/opencv/meson.build b/ext/opencv/meson.build
 index 0b0b3fc..0ed3344 100644
 --- a/ext/opencv/meson.build
 +++ b/ext/opencv/meson.build
-@@ -78,20 +78,21 @@ else
+@@ -84,20 +84,21 @@ if not opencv_found
  endif
  
  if opencv_found
 +  pkgconf_sysroot = run_command(python3, '-c', 'import os; 
print(os.environ.get("PKG_CONFIG_SYSROOT_DIR"))').stdout().strip()
-   opencv_prefix = opencv_dep.get_pkgconfig_variable('prefix')
+   opencv_prefix = opencv_dep.get_variable('prefix')
gstopencv_cargs += ['-DOPENCV_PREFIX="' + opencv_prefix + '"']
  
# Check the data dir used by opencv for its xml data files
# Use prefix from pkg-config to be compatible with cross-compilation
--  r = run_command('test', '-d', opencv_prefix + '/share/opencv')
-+  r = run_command('test', '-d', pkgconf_sysroot + opencv_prefix + 
'/share/opencv')
+-  r = run_command('test', '-d', opencv_prefix + '/share/opencv', check: false)
++  r = run_command('test', '-d', pkgconf_sysroot + opencv_prefix + 
'/share/opencv', check: false)
if r.returncode() == 0
  gstopencv_cargs += '-DOPENCV_PATH_NAME="opencv"'
else
--r = run_command('test', '-d', opencv_prefix + '/share/OpenCV')
-+r = run_command('test', '-d', pkgconf_sysroot + opencv_prefix + 
'/share/OpenCV')
+-r = run_command('test', '-d', opencv_prefix + '/share/OpenCV', check: 
false)
++r = run_command('test', '-d', pkgconf_sysroot + opencv_prefix + 
'/share/OpenCV', check: false)
  if r.returncode() == 0
gstopencv_cargs += '-DOPENCV_PATH_NAME="OpenCV"'
  else
--  r = run_command('test', '-d', opencv_prefix + '/share/opencv4')
-+  r = run_command('test', '-d', pkgconf_sysroot + opencv_prefix + 
'/share/opencv4')
+-  r = run_command('test', '-d', opencv_prefix + '/share/opencv4', check: 
false)
++  r = run_command('test', '-d', pkgconf_sysroot + opencv_prefix + 
'/share/opencv4', check: false)
if r.returncode() == 0
  gstopencv_cargs += '-DOPENCV_PATH_NAME="opencv4"'
else
 -- 
 2.28.0
-
diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad/0005-msdk-fix-includedir-path.patch
 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad/0005-msdk-fix-includedir-path.patch
index cb3bb7d361..5a89be79d0 100644
--- 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad/0005-msdk-fix-includedir-path.patch
+++ 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad/0005-msdk-fix-includedir-path.patch
@@ -16,16 +16,16 @@ diff --git a/sys/msdk/meson.build b/sys/msdk/meson.build
 index 6346c9451..068f38548 100644
 --- a/sys/msdk/meson.build
 +++ b/sys/msdk/meson.build
-@@ -40,7 +40,9 @@ endif
+@@ -46,7 +46,9 @@ if mfx_api != 'oneVPL'
+   mfx_dep = dependency('libmfx', version: ['>= 1.0', '<= 1.99'], required: 
false)
  
- mfx_dep = dependency('libmfx', required: false)
- if mfx_dep.found()
-+  pkgconf_sysroot = run_command(python3, '-c', 'import os; 
print(os.environ.get("PKG_CONFIG_SYSROOT_DIR"))').stdout().strip()
-   mfx_incdir = mfx_dep.get_pkgconfig_variable('includedir')
-+  mfx_incdir = pkgconf_sysroot +  mfx_incdir
-   mfx_inc = []
- else
-   # Old versions of MediaSDK don't provide a pkg-config file
+   if mfx_dep.found()
++pkgconf_sysroot = run_command(python3, '-c', 'import os; 
print(os.environ.get("PKG_CONFIG_SYSROOT_DIR"))').stdout().strip()
+ mfx_incdir = mfx_dep.get_variable('includedir')
++mfx_incdir = pkgconf_sysroot +  mfx_incdir
+ mfx_inc = []
+ use_msdk = true
+   else
 -- 
 2.17.1
 
diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad_1.18.5.bb 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad_1.19.90.bb
similarity index 97%
rename from meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad_1.18.5.bb
rename to meta/recipes-multimedia/gstreamer/gstreame

[OE-core] [PATCH 01/12] gstreamer1.0: 1.18.5 -> 1.19.90

[Claudius Heine]

Backported patches where removed, others updated to their current
version.

This also adds a patch to remove the
gstbin:test_watch_for_state_change testcase, since it seems flaky:
https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/778

Signed-off-by: Claudius Heine 
---
 ...der.c-when-env-var-is-set-do-not-fal.patch |  69 ---
 ...pect-the-idententaion-used-in-meson.patch} |   7 +-
 ...002-Remove-unused-valgrind-detection.patch | 112 --
 ...s-add-support-for-install-the-tests.patch} |  48 
 ...-use-too-strict-timeout-for-validati.patch |  32 -
 ...-use-a-dictionaries-for-environment.patch} |  24 ++--
 ...er-script-to-run-the-installed_tests.patch |  72 +++
 ...bin-test_watch_for_state_change-test.patch | 107 +
 ...-the-environment-for-installed_tests.patch |  58 -
 ...r1.0_1.18.5.bb => gstreamer1.0_1.19.90.bb} |  18 ++-
 10 files changed, 225 insertions(+), 322 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} (87%)
 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} (79%)
 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} (70%)
 create mode 100644 
meta/recipes-multimedia/gstreamer/gstreamer1.0/0004-tests-add-helper-script-to-run-the-installed_tests.patch
 create mode 100644 
meta/recipes-multimedia/gstreamer/gstreamer1.0/0005-tests-remove-gstbin-test_watch_for_state_change-test.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.19.90.bb} (77%)

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: 
<https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/669>
-
-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 @@ gst_plugin_loader_spawn (GstPluginLoader * loader)
-   if (loader->child_running)
- return TRUE;
- 
--  /* Find the gst-plugin-scanner: first try the env-var if it is set,
--   * otherwise use the installed version */
-+  /* Find the gst-plugin-scanner */
-   env = g_getenv ("GST_PLUGIN_SCANNER_1_0");
-   if (env == NULL)
- env = g_getenv ("GST_PLUGIN_SCANNER");
- 
-   if (env != NULL && *env != '\0') {
-+/* use the env-var if it is set */
- GST_LOG ("Trying GST_PLUGIN_SCANNER env var: %s", env);
- helper_bin = g_strdup (env);
- res = gst_plugin_loader_try_helper (loader, helper_bin);
- g_free (helper_bin);
--  }
--
--  if (!res) {
-+  } else {
-+/* use the installed version */
- GST_LOG ("Trying installed plugin scanner");
- 
- #ifdef G_OS_WIN32
-@@ -497,10 +496,10 @@ gst_plugin_loader_spawn (GstPluginLo

[OE-core] [PATCH 03/12] gstreamer1.0-plugins-good: 1.18.5 -> 1.19.90

[Claudius Heine]

Signed-off-by: Claudius Heine 
---
 ...gins-good_1.18.5.bb => gstreamer1.0-plugins-good_1.19.90.bb} | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
 rename meta/recipes-multimedia/gstreamer/{gstreamer1.0-plugins-good_1.18.5.bb 
=> gstreamer1.0-plugins-good_1.19.90.bb} (97%)

diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.18.5.bb 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.19.90.bb
similarity index 97%
rename from 
meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.18.5.bb
rename to meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.19.90.bb
index ade935df9e..fbf521e4cf 100644
--- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.18.5.bb
+++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.19.90.bb
@@ -8,7 +8,7 @@ SRC_URI = 
"https://gstreamer.freedesktop.org/src/gst-plugins-good/gst-plugins-go

file://0001-qt-include-ext-qt-gstqtgl.h-instead-of-gst-gl-gstglf.patch \
"
 
-SRC_URI[sha256sum] = 
"3aaeeea7765fbf8801acce4a503a9b05f73f04e8a35352e9d00232cfd555796b"
+SRC_URI[sha256sum] = 
"76c74ad1f2b62ee751c40d7d24544c3c0c580e2b7153d85272bd8b9dafa1a43e"
 
 S = "${WORKDIR}/gst-plugins-good-${PV}"
 
-- 
2.34.1


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

[OE-core] [PATCH 02/12] gstreamer1.0-plugins-base: 1.18.5 -> 1.19.90

[Claudius Heine]

Signed-off-by: Claudius Heine 
---
 ...004-glimagesink-Downrank-to-marginal.patch | 32 ---
 ...b => gstreamer1.0-plugins-base_1.19.90.bb} |  5 ++-
 2 files changed, 2 insertions(+), 35 deletions(-)
 delete mode 100644 
meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0004-glimagesink-Downrank-to-marginal.patch
 rename meta/recipes-multimedia/gstreamer/{gstreamer1.0-plugins-base_1.18.5.bb 
=> gstreamer1.0-plugins-base_1.19.90.bb} (94%)

diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0004-glimagesink-Downrank-to-marginal.patch
 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0004-glimagesink-Downrank-to-marginal.patch
deleted file mode 100644
index f45ea65ef3..00
--- 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0004-glimagesink-Downrank-to-marginal.patch
+++ /dev/null
@@ -1,32 +0,0 @@
-From 79841a02ee754eba736cb32aaf2ed2b2fc0483d4 Mon Sep 17 00:00:00 2001
-From: Alexander Kanavin 
-Date: Thu, 24 Sep 2015 19:47:32 +0300
-Subject: [PATCH 4/4] glimagesink: Downrank to marginal
-
-On desktop, where there is good OpenGL, xvimagesink will come up first,
-on other platforms, OpenGL can't be trusted because it's either software (like
-in a VM) or broken (like on embedded)., so let ximagesink come above.
-
-Upstream-Status: Submitted [https://bugzilla.gnome.org/show_bug.cgi?id=751684]
-
-Signed-off-by: Alexander Kanavin 

- ext/gl/gstopengl.c | 2 +-
- 1 file changed, 1 insertion(+), 1 deletion(-)
-
-diff --git a/ext/gl/gstopengl.c b/ext/gl/gstopengl.c
-index 302e845..463be9c 100644
 a/ext/gl/gstopengl.c
-+++ b/ext/gl/gstopengl.c
-@@ -127,7 +127,7 @@ plugin_init (GstPlugin * plugin)
- #endif
- 
-   if (!gst_element_register (plugin, "glimagesink",
--  GST_RANK_SECONDARY, gst_gl_image_sink_bin_get_type ())) {
-+  GST_RANK_MARGINAL, gst_gl_image_sink_bin_get_type ())) {
- return FALSE;
-   }
- 
--- 
-2.28.0
-
diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base_1.18.5.bb 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base_1.19.90.bb
similarity index 94%
rename from 
meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base_1.18.5.bb
rename to meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base_1.19.90.bb
index 180241828f..5b02d4565a 100644
--- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base_1.18.5.bb
+++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base_1.19.90.bb
@@ -4,15 +4,14 @@ DESCRIPTION = "'Base' GStreamer plugins and helper libraries"
 HOMEPAGE = "https://gstreamer.freedesktop.org/;
 BUGTRACKER = 
"https://gitlab.freedesktop.org/gstreamer/gst-plugins-base/-/issues;
 LICENSE = "GPLv2+ & LGPLv2+"
-LIC_FILES_CHKSUM = "file://COPYING;md5=6762ed442b3822387a51c92d928ead0d"
+LIC_FILES_CHKSUM = "file://COPYING;md5=69333daa044cb77e486cc36129f7a770"
 
 SRC_URI = 
"https://gstreamer.freedesktop.org/src/gst-plugins-base/gst-plugins-base-${PV}.tar.xz
 \

file://0001-ENGR00312515-get-caps-from-src-pad-when-query-caps.patch \
file://0003-viv-fb-Make-sure-config.h-is-included.patch \
file://0002-ssaparse-enhance-SSA-text-lines-parsing.patch \
-   file://0004-glimagesink-Downrank-to-marginal.patch \
"
-SRC_URI[sha256sum] = 
"960b7af4585700db0fdd5b843554e11e2564fed9e061f591fae88a7be6446fa3"
+SRC_URI[sha256sum] = 
"667c5cbfae1873c8291b5911bedbf20ce92ee607b8b1b3d37120ed54e452db00"
 
 S = "${WORKDIR}/gst-plugins-base-${PV}"
 
-- 
2.34.1


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

[OE-core] [PATCH 00/12] Update gstreamer recipes to 1.19.90

[Claudius Heine]

Hi,

this patchset update gstreamer to 1.19.90, the rc1 of the 1.20.0 release.

Changes from the RFC patchset:
- Use split up tarballs, instead of combined one
- Use 1.19.90 release instead of commit id
- Added patch to remove the gstbin:test_watch_for_state_change testcase, since 
it seems to be flaky:
  https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/778

regards,
Claudius

Claudius Heine (12):
  gstreamer1.0: 1.18.5 -> 1.19.90
  gstreamer1.0-plugins-base: 1.18.5 -> 1.19.90
  gstreamer1.0-plugins-good: 1.18.5 -> 1.19.90
  gstreamer1.0-plugins-bad: 1.18.5 -> 1.19.90
  gstreamer1.0-plugins-ugly: 1.18.5 -> 1.19.90
  gstreamer1.0-libav: 1.18.5 -> 1.19.90
  gstreamer1.0-python: 1.18.5 -> 1.19.90
  gstreamer1.0-omx: 1.18.5 -> 1.19.90
  gstreamer1.0-vaapi: 1.18.5 -> 1.19.90
  gstreamer1.0-rtsp-server: 1.18.5 -> 1.19.90
  gst-examples: 1.18.5 -> 1.19.90
  gst-devtools: 1.18.5 -> 1.19.90

 ...ools_1.18.5.bb => gst-devtools_1.19.90.bb} |   2 +-
 ...001-Make-player-examples-installable.patch |   2 +-
 ...ples_1.18.5.bb => gst-examples_1.19.90.bb} |   0
 ...st-libav-fix-build-with-ffmpeg-5.0.0.patch | 346 --
 18.5.bb => gstreamer1.0-libav_1.19.90.bb} |   5 +-
 ..._1.18.5.bb => gstreamer1.0-omx_1.19.90.bb} |   2 +-
 ...issing-opencv-data-dir-in-yocto-buil.patch |  17 +-
 .../0005-msdk-fix-includedir-path.patch   |  18 +-
 ...bb => gstreamer1.0-plugins-bad_1.19.90.bb} |   4 +-
 ...004-glimagesink-Downrank-to-marginal.patch |  32 --
 ...b => gstreamer1.0-plugins-base_1.19.90.bb} |   5 +-
 ...b => gstreamer1.0-plugins-good_1.19.90.bb} |   2 +-
 ...b => gstreamer1.0-plugins-ugly_1.19.90.bb} |   5 +-
 ...18.5.bb => gstreamer1.0-python_1.19.90.bb} |   2 +-
 ...bb => gstreamer1.0-rtsp-server_1.19.90.bb} |   4 +-
 18.5.bb => gstreamer1.0-vaapi_1.19.90.bb} |   2 +-
 ...der.c-when-env-var-is-set-do-not-fal.patch |  69 
 ...pect-the-idententaion-used-in-meson.patch} |   7 +-
 ...002-Remove-unused-valgrind-detection.patch | 112 --
 ...s-add-support-for-install-the-tests.patch} |  48 ++-
 ...-use-too-strict-timeout-for-validati.patch |  32 --
 ...-use-a-dictionaries-for-environment.patch} |  24 +-
 ...er-script-to-run-the-installed_tests.patch |  72 
 ...bin-test_watch_for_state_change-test.patch | 107 ++
 ...-the-environment-for-installed_tests.patch |  58 ---
 ...r1.0_1.18.5.bb => gstreamer1.0_1.19.90.bb} |  18 +-
 26 files changed, 258 insertions(+), 737 deletions(-)
 rename meta/recipes-multimedia/gstreamer/{gst-devtools_1.18.5.bb => 
gst-devtools_1.19.90.bb} (94%)
 rename meta/recipes-multimedia/gstreamer/{gst-examples_1.18.5.bb => 
gst-examples_1.19.90.bb} (100%)
 delete mode 100644 
meta/recipes-multimedia/gstreamer/gstreamer1.0-libav/0001-gst-libav-fix-build-with-ffmpeg-5.0.0.patch
 rename meta/recipes-multimedia/gstreamer/{gstreamer1.0-libav_1.18.5.bb => 
gstreamer1.0-libav_1.19.90.bb} (78%)
 rename meta/recipes-multimedia/gstreamer/{gstreamer1.0-omx_1.18.5.bb => 
gstreamer1.0-omx_1.19.90.bb} (95%)
 rename meta/recipes-multimedia/gstreamer/{gstreamer1.0-plugins-bad_1.18.5.bb 
=> gstreamer1.0-plugins-bad_1.19.90.bb} (97%)
 delete mode 100644 
meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0004-glimagesink-Downrank-to-marginal.patch
 rename meta/recipes-multimedia/gstreamer/{gstreamer1.0-plugins-base_1.18.5.bb 
=> gstreamer1.0-plugins-base_1.19.90.bb} (94%)
 rename meta/recipes-multimedia/gstreamer/{gstreamer1.0-plugins-good_1.18.5.bb 
=> gstreamer1.0-plugins-good_1.19.90.bb} (97%)
 rename meta/recipes-multimedia/gstreamer/{gstreamer1.0-plugins-ugly_1.18.5.bb 
=> gstreamer1.0-plugins-ugly_1.19.90.bb} (89%)
 rename meta/recipes-multimedia/gstreamer/{gstreamer1.0-python_1.18.5.bb => 
gstreamer1.0-python_1.19.90.bb} (91%)
 rename meta/recipes-multimedia/gstreamer/{gstreamer1.0-rtsp-server_1.18.5.bb 
=> gstreamer1.0-rtsp-server_1.19.90.bb} (82%)
 rename meta/recipes-multimedia/gstreamer/{gstreamer1.0-vaapi_1.18.5.bb => 
gstreamer1.0-vaapi_1.19.90.bb} (95%)
 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} (87%)
 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} (79%)
 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.pa

Re: [OE-core] [PATCH] rng-tools: add systemd-udev-settle wants to service

[Claudius Heine]

Hi Kyle,

On 2022-02-02 13:38, Kyle Russell wrote:
Is this the correct approach?  Even the systemd-udev-settle.service man 
pages recommends not using its service.  Were the kernel modules really 
not loaded when rngd started?  Or is the original problem just a matter 
of waiting for sufficient entropy?


IIRC, the rngd could not find any random source device node (/dev/hwrng 
in that case), so the service failed to start.


The patch you are commenting on only adds `Wants` weak dependency to 
make sure `systemd-udev-settle.service` is pulled in to the job queue, 
the `After` ordering rule was already there.


So changing this service file to be triggered by a udev event or maybe 
wrap it in a script, which makes sure the right modules are loaded and 
device nodes are available, could be an improvement, but it would be out 
of scope of this patch IMO.


regards,
Claudius



On Fri, Sep 17, 2021 at 4:08 AM Claudius Heine <mailto:c...@denx.de>> wrote:


rngd needs to start after `systemd-udev-settle` in order for the kernel
modules of the random source hardware to be loaded before it is started.

However, since the `rngd.service` does not require or want
`systemd-udev-settle.service` it might not be scheduled for start and
the `After=systemd-udev-settle.service` there has no effect.

Adding `Wants=systemd-udev-settle.service` provides a weak requirement
to it, so that the `rngd` is started after it, if possible.

Signed-off-by: Claudius Heine mailto:c...@denx.de>>
---

Hi,

this is a fix, which should probably be backported to the maintained
releases.

regards,
Claudius

  meta/recipes-support/rng-tools/rng-tools/rngd.service | 1 +
  1 file changed, 1 insertion(+)

diff --git a/meta/recipes-support/rng-tools/rng-tools/rngd.service
b/meta/recipes-support/rng-tools/rng-tools/rngd.service
index 0559b97991..568686e80e 100644
--- a/meta/recipes-support/rng-tools/rng-tools/rngd.service
+++ b/meta/recipes-support/rng-tools/rng-tools/rngd.service
@@ -3,6 +3,7 @@ Description=Hardware RNG Entropy Gatherer Daemon
  DefaultDependencies=no
  After=systemd-udev-settle.service
  Before=sysinit.target shutdown.target
+Wants=systemd-udev-settle.service
  Conflicts=shutdown.target

  [Service]
-- 
2.33.0






-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#161195): 
https://lists.openembedded.org/g/openembedded-core/message/161195
Mute This Topic: https://lists.openembedded.org/mt/85671578/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][RFC PATCH 00/12] Update to gstreamer 1.20 for kirkstone

[Claudius Heine]

Hi Jose,

On 2022-01-26 11:52, Jose Quaresma wrote:


Hi Claudius,

I think we can do that switch for the monorepo in a follow up patch 
after the 1.20 integration.
I have an internal update with most ready for 19.3 that misses some of 
the existing downstream patches that we have.


The gstreamer project will provide the archives for different components 
as they had done so far,
so we don't need to change the monorepo to integrate the gstreamer 1.20 
for now.


Ok that sounds good. I didn't know about that.

So that means you will update gstreamer to 1.20 when it is ready for 
kirkstone?


thanks,
Claudius



Cheers,
Jose

Claudius Heine mailto:c...@denx.de>> escreveu no dia quarta, 
26/01/2022 à(s) 10:28:


Hi,

since the release of gstreamer 1.20 is getting close, and it might
make sense to
include it into the kirkstone release. I prepared preliminary update
patchset
for it.

Gstreamer changed to use monorepos with 1.20 so I went ahead I
changed all
recipes to use one SRC_URI from `gstreamer1.0-source.inc`. Currently
I just use
a tarball from a unrelease commit id, but that will be changed to
the correct
one, after gstreamer is released.

I have some ideas which could be implemented with this change, but I
wanted
feedback for this first:
  - remove the version from the filename, and define the PV in the
    `gstreamer1.0-source.inc`, this might make future updates easier.
  - put all recipes together into one recipe, which provides
packages for all
    subprojects, this improves build time and makes maintenance
easier, but also
    breaks downstream bbappends.

Any feedback on this is welcome.

kind regards,
Claudius

Claudius Heine (12):
   gstreamer1.0: 1.18.5 -> 1.20.0
   gstreamer1.0-plugins-base: 1.18.5 -> 1.20.0
   gstreamer1.0-plugins-good: 1.18.5 -> 1.20.0
   gstreamer1.0-plugins-bad: 1.18.5 -> 1.20.0
   gstreamer1.0-plugins-ugly: 1.18.5 -> 1.20.0
   gstreamer1.0-libav: 1.18.5 -> 1.20.0
   gstreamer1.0-python: 1.18.5 -> 1.20.0
   gstreamer1.0-omx: 1.18.5 -> 1.20.0
   gstreamer1.0-vaapi: 1.18.5 -> 1.20.0
   gstreamer1.0-rtsp-server: 1.18.5 -> 1.20.0
   gst-examples: 1.18.5 -> 1.20.0
   gst-devtools: 1.18.5 -> 1.20.0

  ...ct-has-a-different-signature-on-musl.patch |   8 +-
  ...tools_1.18.5.bb <http://tools_1.18.5.bb> =>
gst-devtools_1.20.0.bb <http://gst-devtools_1.20.0.bb>} |   9 +-
  ...001-Make-player-examples-installable.patch |  22 +-
  ...mples_1.18.5.bb <http://mples_1.18.5.bb> =>
gst-examples_1.20.0.bb <http://gst-examples_1.20.0.bb>} |  11 +-
  ...st-libav-fix-build-with-ffmpeg-5.0.0.patch | 346 --
  ...1.18.5.bb <http://1.18.5.bb> => gstreamer1.0-libav_1.20.0.bb
<http://gstreamer1.0-libav_1.20.0.bb>} |   9 +-
  ...x_1.18.5.bb <http://x_1.18.5.bb> => gstreamer1.0-omx_1.20.0.bb
<http://gstreamer1.0-omx_1.20.0.bb>} |   6 +-
  ...ialized-warnings-when-compiling-with.patch |   8 +-
  ...-avoid-including-sys-poll.h-directly.patch |   8 +-
  ...-sentinals-for-gst_structure_get-etc.patch |  24 +-
  ...issing-opencv-data-dir-in-yocto-buil.patch |  25 +-
  .../0005-msdk-fix-includedir-path.patch       |  26 +-
  bb => gstreamer1.0-plugins-bad_1.20.0.bb
<http://gstreamer1.0-plugins-bad_1.20.0.bb>} |  20 +-
  ...et-caps-from-src-pad-when-query-caps.patch |  10 +-
  ...parse-enhance-SSA-text-lines-parsing.patch |  10 +-
  ...iv-fb-Make-sure-config.h-is-included.patch |   8 +-
  ...004-glimagesink-Downrank-to-marginal.patch |  32 --
  ...bb => gstreamer1.0-plugins-base_1.20.0.bb
<http://gstreamer1.0-plugins-base_1.20.0.bb>} |  17 +-
  ...t-gstqtgl.h-instead-of-gst-gl-gstglf.patch |  12 +-
  ...bb => gstreamer1.0-plugins-good_1.20.0.bb
<http://gstreamer1.0-plugins-good_1.20.0.bb>} |   9 +-
  ...bb => gstreamer1.0-plugins-ugly_1.20.0.bb
<http://gstreamer1.0-plugins-ugly_1.20.0.bb>} |  10 +-
  18.5.bb <http://18.5.bb> => gstreamer1.0-python_1.20.0.bb
<http://gstreamer1.0-python_1.20.0.bb>} |   7 +-
  bb => gstreamer1.0-rtsp-server_1.20.0.bb
<http://gstreamer1.0-rtsp-server_1.20.0.bb>} |   8 +-
  .../gstreamer/gstreamer1.0-source.inc         |   5 +
  ...1.18.5.bb <http://1.18.5.bb> => gstreamer1.0-vaapi_1.20.0.bb
<http://gstreamer1.0-vaapi_1.20.0.bb>} |   5 +-
  ...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-

Re: [OE-core][RFC PATCH 00/12] Update to gstreamer 1.20 for kirkstone

[Claudius Heine]

Hi Alex,

On 2022-01-26 12:36, Alexander Kanavin wrote:
Thanks, I guess RP or I should spin this on the autobuilder so you can 
see where the issues are?


Sure that sounds good.

Thanks!



Alex

On Wed, 26 Jan 2022 at 11:52, Jose Quaresma <mailto:quaresma.j...@gmail.com>> wrote:



Hi Claudius,

I think we can do that switch for the monorepo in a follow up patch
after the 1.20 integration.
I have an internal update with most ready for 19.3 that misses some
of the existing downstream patches that we have.

The gstreamer project will provide the archives for
different components as they had done so far,
so we don't need to change the monorepo to integrate the gstreamer
1.20 for now.

Cheers,
Jose

    Claudius Heine mailto:c...@denx.de>> escreveu no dia
quarta, 26/01/2022 à(s) 10:28:

Hi,

since the release of gstreamer 1.20 is getting close, and it
might make sense to
include it into the kirkstone release. I prepared preliminary
update patchset
for it.

Gstreamer changed to use monorepos with 1.20 so I went ahead I
changed all
recipes to use one SRC_URI from `gstreamer1.0-source.inc`.
Currently I just use
a tarball from a unrelease commit id, but that will be changed
to the correct
one, after gstreamer is released.

I have some ideas which could be implemented with this change,
but I wanted
feedback for this first:
  - remove the version from the filename, and define the PV in the
    `gstreamer1.0-source.inc`, this might make future updates
easier.
  - put all recipes together into one recipe, which provides
packages for all
    subprojects, this improves build time and makes maintenance
easier, but also
    breaks downstream bbappends.

Any feedback on this is welcome.

kind regards,
Claudius

    Claudius Heine (12):
   gstreamer1.0: 1.18.5 -> 1.20.0
   gstreamer1.0-plugins-base: 1.18.5 -> 1.20.0
   gstreamer1.0-plugins-good: 1.18.5 -> 1.20.0
   gstreamer1.0-plugins-bad: 1.18.5 -> 1.20.0
   gstreamer1.0-plugins-ugly: 1.18.5 -> 1.20.0
   gstreamer1.0-libav: 1.18.5 -> 1.20.0
   gstreamer1.0-python: 1.18.5 -> 1.20.0
   gstreamer1.0-omx: 1.18.5 -> 1.20.0
   gstreamer1.0-vaapi: 1.18.5 -> 1.20.0
   gstreamer1.0-rtsp-server: 1.18.5 -> 1.20.0
   gst-examples: 1.18.5 -> 1.20.0
   gst-devtools: 1.18.5 -> 1.20.0

  ...ct-has-a-different-signature-on-musl.patch |   8 +-
  ...tools_1.18.5.bb <http://tools_1.18.5.bb> =>
gst-devtools_1.20.0.bb <http://gst-devtools_1.20.0.bb>} |   9 +-
  ...001-Make-player-examples-installable.patch |  22 +-
  ...mples_1.18.5.bb <http://mples_1.18.5.bb> =>
gst-examples_1.20.0.bb <http://gst-examples_1.20.0.bb>} |  11 +-
  ...st-libav-fix-build-with-ffmpeg-5.0.0.patch | 346
--
  ...1.18.5.bb <http://1.18.5.bb> =>
gstreamer1.0-libav_1.20.0.bb
<http://gstreamer1.0-libav_1.20.0.bb>} |   9 +-
  ...x_1.18.5.bb <http://x_1.18.5.bb> =>
gstreamer1.0-omx_1.20.0.bb <http://gstreamer1.0-omx_1.20.0.bb>}
|   6 +-
  ...ialized-warnings-when-compiling-with.patch |   8 +-
  ...-avoid-including-sys-poll.h-directly.patch |   8 +-
  ...-sentinals-for-gst_structure_get-etc.patch |  24 +-
  ...issing-opencv-data-dir-in-yocto-buil.patch |  25 +-
  .../0005-msdk-fix-includedir-path.patch       |  26 +-
  bb => gstreamer1.0-plugins-bad_1.20.0.bb
<http://gstreamer1.0-plugins-bad_1.20.0.bb>} |  20 +-
  ...et-caps-from-src-pad-when-query-caps.patch |  10 +-
  ...parse-enhance-SSA-text-lines-parsing.patch |  10 +-
  ...iv-fb-Make-sure-config.h-is-included.patch |   8 +-
  ...004-glimagesink-Downrank-to-marginal.patch |  32 --
  ...bb => gstreamer1.0-plugins-base_1.20.0.bb
<http://gstreamer1.0-plugins-base_1.20.0.bb>} |  17 +-
  ...t-gstqtgl.h-instead-of-gst-gl-gstglf.patch |  12 +-
  ...bb => gstreamer1.0-plugins-good_1.20.0.bb
<http://gstreamer1.0-plugins-good_1.20.0.bb>} |   9 +-
  ...bb => gstreamer1.0-plugins-ugly_1.20.0.bb
<http://gstreamer1.0-plugins-ugly_1.20.0.bb>} |  10 +-
  18.5.bb <http://18.5.bb> => gstreamer1.0-python_1.20.0.bb
<http://gstreamer1.0-python_1.20.0.bb>} |   7 +-
  bb => gstreamer1.0-rtsp-server_1.20.0.bb
<http://gstreamer1.0-rtsp-server_1.20.0.bb>} |   8 +-
  .../gstreamer/gstreamer1.0-source.inc         |   5 +
  ...1.18.5.bb <h

[OE-core][RFC PATCH 12/12] gst-devtools: 1.18.5 -> 1.20.0

[Claudius Heine]

Signed-off-by: Claudius Heine 
---
 .../0001-connect-has-a-different-signature-on-musl.patch | 8 
 .../{gst-devtools_1.18.5.bb => gst-devtools_1.20.0.bb}   | 9 -
 2 files changed, 8 insertions(+), 9 deletions(-)
 rename meta/recipes-multimedia/gstreamer/{gst-devtools_1.18.5.bb => 
gst-devtools_1.20.0.bb} (83%)

diff --git 
a/meta/recipes-multimedia/gstreamer/gst-devtools/0001-connect-has-a-different-signature-on-musl.patch
 
b/meta/recipes-multimedia/gstreamer/gst-devtools/0001-connect-has-a-different-signature-on-musl.patch
index c0e4581358..ed0a6d387b 100644
--- 
a/meta/recipes-multimedia/gstreamer/gst-devtools/0001-connect-has-a-different-signature-on-musl.patch
+++ 
b/meta/recipes-multimedia/gstreamer/gst-devtools/0001-connect-has-a-different-signature-on-musl.patch
@@ -12,13 +12,13 @@ Upstream-Status: Pending
 
 Signed-off-by: Khem Raj 
 ---
- validate/plugins/fault_injection/socket_interposer.c | 7 ++-
+ subprojects/gst-devtools/validate/plugins/fault_injection/socket_interposer.c 
| 7 ++-
  1 file changed, 6 insertions(+), 1 deletion(-)
 
-diff --git a/validate/plugins/fault_injection/socket_interposer.c 
b/validate/plugins/fault_injection/socket_interposer.c
+diff --git 
a/subprojects/gst-devtools/validate/plugins/fault_injection/socket_interposer.c 
b/subprojects/gst-devtools/validate/plugins/fault_injection/socket_interposer.c
 index 53c1ebb..ad7adf8 100644
 a/validate/plugins/fault_injection/socket_interposer.c
-+++ b/validate/plugins/fault_injection/socket_interposer.c
+--- 
a/subprojects/gst-devtools/validate/plugins/fault_injection/socket_interposer.c
 
b/subprojects/gst-devtools/validate/plugins/fault_injection/socket_interposer.c
 @@ -100,10 +100,15 @@ socket_interposer_set_callback (struct sockaddr_in 
*addrin,
  }
  
diff --git a/meta/recipes-multimedia/gstreamer/gst-devtools_1.18.5.bb 
b/meta/recipes-multimedia/gstreamer/gst-devtools_1.20.0.bb
similarity index 83%
rename from meta/recipes-multimedia/gstreamer/gst-devtools_1.18.5.bb
rename to meta/recipes-multimedia/gstreamer/gst-devtools_1.20.0.bb
index 1b46b89cb9..a8954a577b 100644
--- a/meta/recipes-multimedia/gstreamer/gst-devtools_1.18.5.bb
+++ b/meta/recipes-multimedia/gstreamer/gst-devtools_1.20.0.bb
@@ -6,13 +6,12 @@ SECTION = "multimedia"
 LICENSE = "LGPLv2.1"
 LIC_FILES_CHKSUM = 
"file://validate/COPYING;md5=a6f89e2100d9b6cdffcea4f398e37343"
 
-#S = "${WORKDIR}/gst-devtools-${PV}"
+require gstreamer1.0-source.inc
 
-SRC_URI = 
"https://gstreamer.freedesktop.org/src/gst-devtools/gst-devtools-${PV}.tar.xz \
-   file://0001-connect-has-a-different-signature-on-musl.patch \
-   "
+S = "${SRC_BASE}/subprojects/gst-devtools"
 
-SRC_URI[sha256sum] = 
"fecffc86447daf5c2a06843c757a991d745caa2069446a0d746e99b13f7cb079"
+SRC_URI += 
"file://0001-connect-has-a-different-signature-on-musl.patch;patchdir=${SRC_BASE}
 \
+   "
 
 DEPENDS = "json-glib glib-2.0 glib-2.0-native gstreamer1.0 
gstreamer1.0-plugins-base"
 RRECOMMENDS:${PN} = "git"
-- 
2.34.1


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

[OE-core][RFC PATCH 10/12] gstreamer1.0-rtsp-server: 1.18.5 -> 1.20.0

[Claudius Heine]

Signed-off-by: Claudius Heine 
---
 ...erver_1.18.5.bb => gstreamer1.0-rtsp-server_1.20.0.bb} | 8 +++-
 1 file changed, 3 insertions(+), 5 deletions(-)
 rename meta/recipes-multimedia/gstreamer/{gstreamer1.0-rtsp-server_1.18.5.bb 
=> gstreamer1.0-rtsp-server_1.20.0.bb} (70%)

diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-rtsp-server_1.18.5.bb 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-rtsp-server_1.20.0.bb
similarity index 70%
rename from meta/recipes-multimedia/gstreamer/gstreamer1.0-rtsp-server_1.18.5.bb
rename to meta/recipes-multimedia/gstreamer/gstreamer1.0-rtsp-server_1.20.0.bb
index 50426ad46d..a8cc8f0a09 100644
--- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-rtsp-server_1.18.5.bb
+++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-rtsp-server_1.20.0.bb
@@ -2,17 +2,15 @@ SUMMARY = "A library on top of GStreamer for building an RTSP 
server"
 HOMEPAGE = "http://cgit.freedesktop.org/gstreamer/gst-rtsp-server/;
 SECTION = "multimedia"
 LICENSE = "LGPLv2"
-LIC_FILES_CHKSUM = "file://COPYING;md5=6762ed442b3822387a51c92d928ead0d"
+LIC_FILES_CHKSUM = "file://COPYING;md5=69333daa044cb77e486cc36129f7a770"
 
 DEPENDS = "gstreamer1.0 gstreamer1.0-plugins-base"
 
 PNREAL = "gst-rtsp-server"
 
-SRC_URI = 
"https://gstreamer.freedesktop.org/src/${PNREAL}/${PNREAL}-${PV}.tar.xz;
+require gstreamer1.0-source.inc
 
-SRC_URI[sha256sum] = 
"04d63bf48816c6f41c73f6de0f912a7cef0aab39c44162a7bcece1923dfc9d1f"
-
-S = "${WORKDIR}/${PNREAL}-${PV}"
+S = "${SRC_BASE}/subprojects/gst-rtsp-server"
 
 inherit meson pkgconfig upstream-version-is-even gobject-introspection
 
-- 
2.34.1


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

[OE-core][RFC PATCH 11/12] gst-examples: 1.18.5 -> 1.20.0

[Claudius Heine]

Signed-off-by: Claudius Heine 
---
 ...001-Make-player-examples-installable.patch | 22 +--
 ...mples_1.18.5.bb => gst-examples_1.20.0.bb} | 11 +-
 2 files changed, 16 insertions(+), 17 deletions(-)
 rename meta/recipes-multimedia/gstreamer/{gst-examples_1.18.5.bb => 
gst-examples_1.20.0.bb} (82%)

diff --git 
a/meta/recipes-multimedia/gstreamer/gst-examples/0001-Make-player-examples-installable.patch
 
b/meta/recipes-multimedia/gstreamer/gst-examples/0001-Make-player-examples-installable.patch
index ab93c13244..96baab2415 100644
--- 
a/meta/recipes-multimedia/gstreamer/gst-examples/0001-Make-player-examples-installable.patch
+++ 
b/meta/recipes-multimedia/gstreamer/gst-examples/0001-Make-player-examples-installable.patch
@@ -9,29 +9,29 @@ Upstream-Status: Denied [Upstream considers these code 
examples, for now a least
 https://bugzilla.gnome.org/show_bug.cgi?id=777827
 
 ---
- playback/player/gst-play/meson.build | 1 +
- playback/player/gtk/meson.build  | 1 +
+ subprojects/gst-examples/playback/player/gst-play/meson.build | 1 +
+ subprojects/gst-examples/playback/player/gtk/meson.build  | 1 +
  2 files changed, 2 insertions(+)
 
-diff --git a/playback/player/gst-play/meson.build 
b/playback/player/gst-play/meson.build
+diff --git a/subprojects/gst-examples/playback/player/gst-play/meson.build 
b/subprojects/gst-examples/playback/player/gst-play/meson.build
 index 8ec021d..977cc5c 100644
 a/playback/player/gst-play/meson.build
-+++ b/playback/player/gst-play/meson.build
+--- a/subprojects/gst-examples/playback/player/gst-play/meson.build
 b/subprojects/gst-examples/playback/player/gst-play/meson.build
 @@ -2,5 +2,6 @@ executable('gst-play',
  ['gst-play.c',
   'gst-play-kb.c',
   'gst-play-kb.h'],
 +install: true,
- dependencies : [gst_dep, gstplayer_dep, m_dep])
+ dependencies : [gst_dep, dependency('gstreamer-play-1.0'), m_dep])
  
-diff --git a/playback/player/gtk/meson.build b/playback/player/gtk/meson.build
+diff --git a/subprojects/gst-examples/playback/player/gtk/meson.build 
b/subprojects/gst-examples/playback/player/gtk/meson.build
 index f7a7419..6281130 100644
 a/playback/player/gtk/meson.build
-+++ b/playback/player/gtk/meson.build
+--- a/subprojects/gst-examples/playback/player/gtk/meson.build
 b/subprojects/gst-examples/playback/player/gtk/meson.build
 @@ -13,5 +13,6 @@ if gtk_dep.found()
  gtk_play_resources,
 'gtk-video-renderer.h',
 'gtk-video-renderer.c'],
-+install: true,
-   dependencies : [glib_dep, gobject_dep, gmodule_dep, gst_dep, 
gsttag_dep, gstplayer_dep, gtk_dep, x11_dep])
++  install: true,
+   dependencies : [glib_dep, gobject_dep, gmodule_dep, gst_dep, 
gsttag_dep, gstplay_dep, gtk_dep, x11_dep])
  endif
diff --git a/meta/recipes-multimedia/gstreamer/gst-examples_1.18.5.bb 
b/meta/recipes-multimedia/gstreamer/gst-examples_1.20.0.bb
similarity index 82%
rename from meta/recipes-multimedia/gstreamer/gst-examples_1.18.5.bb
rename to meta/recipes-multimedia/gstreamer/gst-examples_1.20.0.bb
index a720bb73ff..ef1b457882 100644
--- a/meta/recipes-multimedia/gstreamer/gst-examples_1.18.5.bb
+++ b/meta/recipes-multimedia/gstreamer/gst-examples_1.20.0.bb
@@ -7,14 +7,13 @@ LIC_FILES_CHKSUM = 
"file://playback/player/gtk/gtk-play.c;beginline=1;endline=20
 
 DEPENDS = "glib-2.0 gstreamer1.0 gstreamer1.0-plugins-base 
gstreamer1.0-plugins-bad gtk+3 libsoup-2.4 json-glib glib-2.0-native"
 
-SRC_URI = 
"git://gitlab.freedesktop.org/gstreamer/gst-examples.git;protocol=https;branch=1.18
 \
-   file://0001-Make-player-examples-installable.patch \
-   file://gst-player.desktop \
-   "
+require gstreamer1.0-source.inc
 
-SRCREV = "fe9a365dc0f1ff632abcfe3322ac5527a2cf30a0"
+S = "${SRC_BASE}/subprojects/gst-examples"
 
-S = "${WORKDIR}/git"
+SRC_URI += 
"file://0001-Make-player-examples-installable.patch;patchdir=${SRC_BASE} \
+file://gst-player.desktop \
+"
 
 inherit meson pkgconfig features_check
 
-- 
2.34.1


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

[OE-core][RFC PATCH 09/12] gstreamer1.0-vaapi: 1.18.5 -> 1.20.0

[Claudius Heine]

Signed-off-by: Claudius Heine 
---
 ...eamer1.0-vaapi_1.18.5.bb => gstreamer1.0-vaapi_1.20.0.bb} | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)
 rename meta/recipes-multimedia/gstreamer/{gstreamer1.0-vaapi_1.18.5.bb => 
gstreamer1.0-vaapi_1.20.0.bb} (90%)

diff --git a/meta/recipes-multimedia/gstreamer/gstreamer1.0-vaapi_1.18.5.bb 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-vaapi_1.20.0.bb
similarity index 90%
rename from meta/recipes-multimedia/gstreamer/gstreamer1.0-vaapi_1.18.5.bb
rename to meta/recipes-multimedia/gstreamer/gstreamer1.0-vaapi_1.20.0.bb
index 9a68a3fadf..49e9d93837 100644
--- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-vaapi_1.18.5.bb
+++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-vaapi_1.20.0.bb
@@ -9,11 +9,10 @@ REALPN = "gstreamer-vaapi"
 LICENSE = "LGPLv2.1+"
 LIC_FILES_CHKSUM = "file://COPYING.LIB;md5=4fbd65380cdd255951079008b364516c"
 
-SRC_URI = 
"https://gstreamer.freedesktop.org/src/${REALPN}/${REALPN}-${PV}.tar.xz;
+require gstreamer1.0-source.inc
 
-SRC_URI[sha256sum] = 
"4a460fb95559f41444eb24864ad2d9e37922b6eea941510310319fc3e0ba727b"
+S = "${SRC_BASE}/subprojects/gstreamer-vaapi"
 
-S = "${WORKDIR}/${REALPN}-${PV}"
 DEPENDS = "libva gstreamer1.0 gstreamer1.0-plugins-base 
gstreamer1.0-plugins-bad"
 
 inherit meson pkgconfig features_check upstream-version-is-even
-- 
2.34.1


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

[OE-core][RFC PATCH 08/12] gstreamer1.0-omx: 1.18.5 -> 1.20.0

[Claudius Heine]

Signed-off-by: Claudius Heine 
---
 ...streamer1.0-omx_1.18.5.bb => gstreamer1.0-omx_1.20.0.bb} | 6 ++
 1 file changed, 2 insertions(+), 4 deletions(-)
 rename meta/recipes-multimedia/gstreamer/{gstreamer1.0-omx_1.18.5.bb => 
gstreamer1.0-omx_1.20.0.bb} (89%)

diff --git a/meta/recipes-multimedia/gstreamer/gstreamer1.0-omx_1.18.5.bb 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-omx_1.20.0.bb
similarity index 89%
rename from meta/recipes-multimedia/gstreamer/gstreamer1.0-omx_1.18.5.bb
rename to meta/recipes-multimedia/gstreamer/gstreamer1.0-omx_1.20.0.bb
index b2c1eb3ea0..1ff5bf6f74 100644
--- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-omx_1.18.5.bb
+++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-omx_1.20.0.bb
@@ -8,11 +8,9 @@ LICENSE_FLAGS = "commercial"
 LIC_FILES_CHKSUM = "file://COPYING;md5=4fbd65380cdd255951079008b364516c \
 
file://omx/gstomx.h;beginline=1;endline=21;md5=5c8e1fca32704488e76d2ba9ddfa935f"
 
-SRC_URI = "https://gstreamer.freedesktop.org/src/gst-omx/gst-omx-${PV}.tar.xz;
+require gstreamer1.0-source.inc
 
-SRC_URI[sha256sum] = 
"2cd457c1e8deb1a9b39608048fb36a44f6c9a864a6b6115b1453a32e7be93b42"
-
-S = "${WORKDIR}/gst-omx-${PV}"
+S = "${SRC_BASE}/subprojects/gst-omx"
 
 DEPENDS = "gstreamer1.0 gstreamer1.0-plugins-base gstreamer1.0-plugins-bad 
virtual/libomxil"
 
-- 
2.34.1


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

[OE-core][RFC PATCH 07/12] gstreamer1.0-python: 1.18.5 -> 1.20.0

[Claudius Heine]

Signed-off-by: Claudius Heine 
---
 ...r1.0-python_1.18.5.bb => gstreamer1.0-python_1.20.0.bb} | 7 +++
 1 file changed, 3 insertions(+), 4 deletions(-)
 rename meta/recipes-multimedia/gstreamer/{gstreamer1.0-python_1.18.5.bb => 
gstreamer1.0-python_1.20.0.bb} (79%)

diff --git a/meta/recipes-multimedia/gstreamer/gstreamer1.0-python_1.18.5.bb 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-python_1.20.0.bb
similarity index 79%
rename from meta/recipes-multimedia/gstreamer/gstreamer1.0-python_1.18.5.bb
rename to meta/recipes-multimedia/gstreamer/gstreamer1.0-python_1.20.0.bb
index 1dd7d0d09a..eaba1e23bc 100644
--- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-python_1.18.5.bb
+++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-python_1.20.0.bb
@@ -7,16 +7,15 @@ SECTION = "multimedia"
 LICENSE = "LGPLv2.1"
 LIC_FILES_CHKSUM = "file://COPYING;md5=c34deae4e395ca07e725ab0076a5f740"
 
-SRC_URI = 
"https://gstreamer.freedesktop.org/src/${PNREAL}/${PNREAL}-${PV}.tar.xz;
-SRC_URI[sha256sum] = 
"533685871305959d6db89507f3b3aa6c765c2f2b0dacdc32c5a6543e72e5bc52"
+require gstreamer1.0-source.inc
+
+S = "${SRC_BASE}/subprojects/gst-python"
 
 DEPENDS = "gstreamer1.0 gstreamer1.0-plugins-base python3-pygobject"
 RDEPENDS:${PN} += "gstreamer1.0 gstreamer1.0-plugins-base python3-pygobject"
 
 PNREAL = "gst-python"
 
-S = "${WORKDIR}/${PNREAL}-${PV}"
-
 EXTRA_OEMESON += "-Dlibpython-dir=${libdir}"
 
 # gobject-introspection is mandatory and cannot be configured
-- 
2.34.1


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

[OE-core][RFC PATCH 05/12] gstreamer1.0-plugins-ugly: 1.18.5 -> 1.20.0

[Claudius Heine]

Feature `gpl` was added, since it is required to be enabled for gpl
licensed codecs.

Signed-off-by: Claudius Heine 
---
 ...y_1.18.5.bb => gstreamer1.0-plugins-ugly_1.20.0.bb} | 10 --
 1 file changed, 4 insertions(+), 6 deletions(-)
 rename meta/recipes-multimedia/gstreamer/{gstreamer1.0-plugins-ugly_1.18.5.bb 
=> gstreamer1.0-plugins-ugly_1.20.0.bb} (82%)

diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-ugly_1.18.5.bb 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-ugly_1.20.0.bb
similarity index 82%
rename from 
meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-ugly_1.18.5.bb
rename to meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-ugly_1.20.0.bb
index 9777aaee19..c3b2066ca3 100644
--- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-ugly_1.18.5.bb
+++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-ugly_1.20.0.bb
@@ -10,12 +10,9 @@ LIC_FILES_CHKSUM = 
"file://COPYING;md5=a6f89e2100d9b6cdffcea4f398e37343 \
 LICENSE = "GPLv2+ & LGPLv2.1+ & LGPLv2+"
 LICENSE_FLAGS = "commercial"
 
-SRC_URI = " \
-
https://gstreamer.freedesktop.org/src/gst-plugins-ugly/gst-plugins-ugly-${PV}.tar.xz
 \
-"
-SRC_URI[sha256sum] = 
"df32803e98f8a9979373fa2ca7e05e62f977b1097576d3a80619d9f5c69f66d9"
+require gstreamer1.0-source.inc
 
-S = "${WORKDIR}/gst-plugins-ugly-${PV}"
+S = "${SRC_BASE}/subprojects/gst-plugins-ugly"
 
 DEPENDS += "gstreamer1.0-plugins-base"
 
@@ -23,7 +20,7 @@ GST_PLUGIN_SET_HAS_EXAMPLES = "0"
 
 PACKAGECONFIG ??= " \
 ${GSTREAMER_ORC} \
-a52dec mpeg2dec \
+a52dec mpeg2dec gpl \
 "
 
 PACKAGECONFIG[a52dec]   = "-Da52dec=enabled,-Da52dec=disabled,liba52"
@@ -33,6 +30,7 @@ PACKAGECONFIG[cdio] = 
"-Dcdio=enabled,-Dcdio=disabled,libcdio"
 PACKAGECONFIG[dvdread]  = "-Ddvdread=enabled,-Ddvdread=disabled,libdvdread"
 PACKAGECONFIG[mpeg2dec] = "-Dmpeg2dec=enabled,-Dmpeg2dec=disabled,mpeg2dec"
 PACKAGECONFIG[x264] = "-Dx264=enabled,-Dx264=disabled,x264"
+PACKAGECONFIG[gpl]  = "-Dgpl=enabled,-Dgpl=disabled"
 
 EXTRA_OEMESON += " \
 -Ddoc=disabled \
-- 
2.34.1


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

[OE-core][RFC PATCH 04/12] gstreamer1.0-plugins-bad: 1.18.5 -> 1.20.0

[Claudius Heine]

Feature `ofa` and `libmms` are no longer available and where removed.

Signed-off-by: Claudius Heine 
---
 ...ialized-warnings-when-compiling-with.patch |  8 +++---
 ...-avoid-including-sys-poll.h-directly.patch |  8 +++---
 ...-sentinals-for-gst_structure_get-etc.patch | 24 -
 ...issing-opencv-data-dir-in-yocto-buil.patch | 25 +-
 .../0005-msdk-fix-includedir-path.patch   | 26 +--
 bb => gstreamer1.0-plugins-bad_1.20.0.bb} | 20 +++---
 6 files changed, 54 insertions(+), 57 deletions(-)
 rename meta/recipes-multimedia/gstreamer/{gstreamer1.0-plugins-bad_1.18.5.bb 
=> gstreamer1.0-plugins-bad_1.20.0.bb} (91%)

diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad/0001-fix-maybe-uninitialized-warnings-when-compiling-with.patch
 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad/0001-fix-maybe-uninitialized-warnings-when-compiling-with.patch
index 13a673cd50..ae8af094cb 100644
--- 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad/0001-fix-maybe-uninitialized-warnings-when-compiling-with.patch
+++ 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad/0001-fix-maybe-uninitialized-warnings-when-compiling-with.patch
@@ -7,13 +7,13 @@ Upstream-Status: Pending
 
 Signed-off-by: Andre McCurdy 
 ---
- gst-libs/gst/codecparsers/gstvc1parser.c | 2 +-
+ subprojects/gst-plugins-bad/gst-libs/gst/codecparsers/gstvc1parser.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
 
-diff --git a/gst-libs/gst/codecparsers/gstvc1parser.c 
b/gst-libs/gst/codecparsers/gstvc1parser.c
+diff --git 
a/subprojects/gst-plugins-bad/gst-libs/gst/codecparsers/gstvc1parser.c 
b/subprojects/gst-plugins-bad/gst-libs/gst/codecparsers/gstvc1parser.c
 index 2c60ced..e8226d8 100644
 a/gst-libs/gst/codecparsers/gstvc1parser.c
-+++ b/gst-libs/gst/codecparsers/gstvc1parser.c
+--- a/subprojects/gst-plugins-bad/gst-libs/gst/codecparsers/gstvc1parser.c
 b/subprojects/gst-plugins-bad/gst-libs/gst/codecparsers/gstvc1parser.c
 @@ -1730,7 +1730,7 @@ gst_vc1_parse_sequence_layer (const guint8 * data, gsize 
size,
  GstVC1SeqLayer * seqlayer)
  {
diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad/0002-avoid-including-sys-poll.h-directly.patch
 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad/0002-avoid-including-sys-poll.h-directly.patch
index ead6897f67..b1aaff864b 100644
--- 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad/0002-avoid-including-sys-poll.h-directly.patch
+++ 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad/0002-avoid-including-sys-poll.h-directly.patch
@@ -9,13 +9,13 @@ Upstream-Status: Pending
 
 Signed-off-by: Andre McCurdy 
 ---
- sys/dvb/gstdvbsrc.c | 2 +-
+ subprojects/gst-plugins-bad/sys/dvb/gstdvbsrc.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
 
-diff --git a/sys/dvb/gstdvbsrc.c b/sys/dvb/gstdvbsrc.c
+diff --git a/subprojects/gst-plugins-bad/sys/dvb/gstdvbsrc.c 
b/subprojects/gst-plugins-bad/sys/dvb/gstdvbsrc.c
 index ca6b92a..b2772db 100644
 a/sys/dvb/gstdvbsrc.c
-+++ b/sys/dvb/gstdvbsrc.c
+--- a/subprojects/gst-plugins-bad/sys/dvb/gstdvbsrc.c
 b/subprojects/gst-plugins-bad/sys/dvb/gstdvbsrc.c
 @@ -97,7 +97,7 @@
  #include 
  #include 
diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad/0003-ensure-valid-sentinals-for-gst_structure_get-etc.patch
 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad/0003-ensure-valid-sentinals-for-gst_structure_get-etc.patch
index 88fbc40dcd..a53a840e98 100644
--- 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad/0003-ensure-valid-sentinals-for-gst_structure_get-etc.patch
+++ 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad/0003-ensure-valid-sentinals-for-gst_structure_get-etc.patch
@@ -18,15 +18,15 @@ Upstream-Status: Pending
 
 Signed-off-by: Andre McCurdy 
 ---
- sys/decklink/gstdecklink.cpp  | 10 +-
- sys/decklink/gstdecklinkaudiosrc.cpp  |  2 +-
- sys/decklink/gstdecklinkvideosink.cpp |  2 +-
+ subprojects/gst-plugins-bad/sys/decklink/gstdecklink.cpp  | 10 
+-
+ subprojects/gst-plugins-bad/sys/decklink/gstdecklinkaudiosrc.cpp  |  2 +-
+ subprojects/gst-plugins-bad/sys/decklink/gstdecklinkvideosink.cpp |  2 +-
  3 files changed, 7 insertions(+), 7 deletions(-)
 
-diff --git a/sys/decklink/gstdecklink.cpp b/sys/decklink/gstdecklink.cpp
+diff --git a/subprojects/gst-plugins-bad/sys/decklink/gstdecklink.cpp 
b/subprojects/gst-plugins-bad/sys/decklink/gstdecklink.cpp
 index 4dac7e1..43762ce 100644
 a/sys/decklink/gstdecklink.cpp
-+++ b/sys/decklink/gstdecklink.cpp
+--- a/subprojects/gst-plugins-bad/sys/decklink/gstdecklink.cpp
 b/subprojects/gst-plugins-bad/sys/decklink/gstdecklink.cpp
 @@ -674,7 +674,7 @@ gst_decklink_mode_get_generic_structure 
(GstDecklinkModeEnum e)
"pixel-aspect-ratio", GST_TYPE_FRACTION, mode->par_n, mode->par_d,
"interlace-mode",

[OE-core][RFC PATCH 06/12] gstreamer1.0-libav: 1.18.5 -> 1.20.0

[Claudius Heine]

Signed-off-by: Claudius Heine 
---
 ...st-libav-fix-build-with-ffmpeg-5.0.0.patch | 346 --
 ...1.18.5.bb => gstreamer1.0-libav_1.20.0.bb} |   9 +-
 2 files changed, 3 insertions(+), 352 deletions(-)
 delete mode 100644 
meta/recipes-multimedia/gstreamer/gstreamer1.0-libav/0001-gst-libav-fix-build-with-ffmpeg-5.0.0.patch
 rename meta/recipes-multimedia/gstreamer/{gstreamer1.0-libav_1.18.5.bb => 
gstreamer1.0-libav_1.20.0.bb} (66%)

diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-libav/0001-gst-libav-fix-build-with-ffmpeg-5.0.0.patch
 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-libav/0001-gst-libav-fix-build-with-ffmpeg-5.0.0.patch
deleted file mode 100644
index 022ff9af29..00
--- 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-libav/0001-gst-libav-fix-build-with-ffmpeg-5.0.0.patch
+++ /dev/null
@@ -1,346 +0,0 @@
-From 38d10ee800e42afeacc6bee714216e4c974c11f5 Mon Sep 17 00:00:00 2001
-From: Xi Ruoyao 
-Date: Mon, 17 Jan 2022 01:33:47 +0800
-Subject: [PATCH] gst-libav: fix build with ffmpeg-5.0.0
-
-Latest ffmpeg has removed avcodec_get_context_defaults(), and its
-documentation says a new AVCodecContext should be allocated for this
-purpose.  The pointer returned by avcodec_find_decoder() is now
-const-qualified so we also need to adjust for it.  And, AVCOL_RANGE_MPEG
-is now rejected with strict_std_compliance > FF_COMPLIANCE_UNOFFICIAL.
-
-Part-of: 
<https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1531>
-Upstream-Status: Backport
-Signed-off-by: Alexander Kanavin 

- ext/libav/gstavauddec.c   | 22 -
- ext/libav/gstavaudenc.c   | 40 +++
- ext/libav/gstavcodecmap.c |  7 ---
- ext/libav/gstavutils.c|  2 +-
- ext/libav/gstavviddec.c   | 28 +++
- ext/libav/gstavvidenc.c   | 21 ++--
- 6 files changed, 54 insertions(+), 66 deletions(-)
-
-diff --git a/ext/libav/gstavauddec.c b/ext/libav/gstavauddec.c
-index baf7aa5..b03a724 100644
 a/ext/libav/gstavauddec.c
-+++ b/ext/libav/gstavauddec.c
-@@ -168,12 +168,7 @@ gst_ffmpegauddec_finalize (GObject * object)
-   GstFFMpegAudDec *ffmpegdec = (GstFFMpegAudDec *) object;
- 
-   av_frame_free (>frame);
--
--  if (ffmpegdec->context != NULL) {
--gst_ffmpeg_avcodec_close (ffmpegdec->context);
--av_free (ffmpegdec->context);
--ffmpegdec->context = NULL;
--  }
-+  avcodec_free_context (>context);
- 
-   G_OBJECT_CLASS (parent_class)->finalize (object);
- }
-@@ -193,14 +188,12 @@ gst_ffmpegauddec_close (GstFFMpegAudDec * ffmpegdec, 
gboolean reset)
-   gst_ffmpeg_avcodec_close (ffmpegdec->context);
-   ffmpegdec->opened = FALSE;
- 
--  if (ffmpegdec->context->extradata) {
--av_free (ffmpegdec->context->extradata);
--ffmpegdec->context->extradata = NULL;
--  }
-+  av_freep (>context->extradata);
- 
-   if (reset) {
--if (avcodec_get_context_defaults3 (ffmpegdec->context,
--oclass->in_plugin) < 0) {
-+avcodec_free_context (>context);
-+ffmpegdec->context = avcodec_alloc_context3 (oclass->in_plugin);
-+if (ffmpegdec->context == NULL) {
-   GST_DEBUG_OBJECT (ffmpegdec, "Failed to set context defaults");
-   return FALSE;
- }
-@@ -219,8 +212,9 @@ gst_ffmpegauddec_start (GstAudioDecoder * decoder)
-   oclass = (GstFFMpegAudDecClass *) (G_OBJECT_GET_CLASS (ffmpegdec));
- 
-   GST_OBJECT_LOCK (ffmpegdec);
--  gst_ffmpeg_avcodec_close (ffmpegdec->context);
--  if (avcodec_get_context_defaults3 (ffmpegdec->context, oclass->in_plugin) < 
0) {
-+  avcodec_free_context (>context);
-+  ffmpegdec->context = avcodec_alloc_context3 (oclass->in_plugin);
-+  if (ffmpegdec->context == NULL) {
- GST_DEBUG_OBJECT (ffmpegdec, "Failed to set context defaults");
- GST_OBJECT_UNLOCK (ffmpegdec);
- return FALSE;
-diff --git a/ext/libav/gstavaudenc.c b/ext/libav/gstavaudenc.c
-index 3ff6432..689982f 100644
 a/ext/libav/gstavaudenc.c
-+++ b/ext/libav/gstavaudenc.c
-@@ -175,10 +175,8 @@ gst_ffmpegaudenc_finalize (GObject * object)
- 
-   /* clean up remaining allocated data */
-   av_frame_free (>frame);
--  gst_ffmpeg_avcodec_close (ffmpegaudenc->context);
--  gst_ffmpeg_avcodec_close (ffmpegaudenc->refcontext);
--  av_free (ffmpegaudenc->context);
--  av_free (ffmpegaudenc->refcontext);
-+  avcodec_free_context (>context);
-+  avcodec_free_context (>refcontext);
- 
-   G_OBJECT_CLASS (parent_class)->finalize (object);
- }
-@@ -193,9 +191,9 @@ gst_ffmpegaudenc_start (GstAudioEncoder * encoder)
-   ffmpegaudenc->opened = FALSE;
-   ffmpegaudenc->need_reopen = FALSE;
- 
--  gst_ffmpeg_avcodec_close (ffmpegaudenc->context);
--  if (avcodec_get_context_defaults3 (ffmpegaudenc->context,
--  oclass->in_plugin) < 0) {
-+  avcodec_free_context (>context);
-+  ffmpegaudenc->context

[OE-core][RFC PATCH 03/12] gstreamer1.0-plugins-good: 1.18.5 -> 1.20.0

[Claudius Heine]

Signed-off-by: Claudius Heine 
---
 ...e-ext-qt-gstqtgl.h-instead-of-gst-gl-gstglf.patch | 12 ++--
 ...1.18.5.bb => gstreamer1.0-plugins-good_1.20.0.bb} |  9 -
 2 files changed, 10 insertions(+), 11 deletions(-)
 rename meta/recipes-multimedia/gstreamer/{gstreamer1.0-plugins-good_1.18.5.bb 
=> gstreamer1.0-plugins-good_1.20.0.bb} (91%)

diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good/0001-qt-include-ext-qt-gstqtgl.h-instead-of-gst-gl-gstglf.patch
 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good/0001-qt-include-ext-qt-gstqtgl.h-instead-of-gst-gl-gstglf.patch
index 788d752058..11f0deba97 100644
--- 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good/0001-qt-include-ext-qt-gstqtgl.h-instead-of-gst-gl-gstglf.patch
+++ 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good/0001-qt-include-ext-qt-gstqtgl.h-instead-of-gst-gl-gstglf.patch
@@ -25,10 +25,10 @@ Signed-off-by: Khem Raj 
  ext/qt/qtwindow.cc  | 2 +-
  2 files changed, 2 insertions(+), 2 deletions(-)
 
-diff --git a/ext/qt/gstqsgtexture.cc b/ext/qt/gstqsgtexture.cc
+diff --git a/subprojects/gst-plugins-good/ext/qt/gstqsgtexture.cc 
b/subprojects/gst-plugins-good/ext/qt/gstqsgtexture.cc
 index a05d26e..4cc9fc6 100644
 a/ext/qt/gstqsgtexture.cc
-+++ b/ext/qt/gstqsgtexture.cc
+--- a/subprojects/gst-plugins-good/ext/qt/gstqsgtexture.cc
 b/subprojects/gst-plugins-good/ext/qt/gstqsgtexture.cc
 @@ -27,7 +27,7 @@
  
  #include 
@@ -38,10 +38,10 @@ index a05d26e..4cc9fc6 100644
  #include "gstqsgtexture.h"
  
  #define GST_CAT_DEFAULT gst_qsg_texture_debug
-diff --git a/ext/qt/qtwindow.cc b/ext/qt/qtwindow.cc
+diff --git a/subprojects/gst-plugins-good/ext/qt/qtwindow.cc 
b/subprojects/gst-plugins-good/ext/qt/qtwindow.cc
 index 9360c33..0dfd3f1 100644
 a/ext/qt/qtwindow.cc
-+++ b/ext/qt/qtwindow.cc
+--- a/subprojects/gst-plugins-good/ext/qt/qtwindow.cc
 b/subprojects/gst-plugins-good/ext/qt/qtwindow.cc
 @@ -25,7 +25,7 @@
  #include 
  
diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.18.5.bb 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.20.0.bb
similarity index 91%
rename from 
meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.18.5.bb
rename to meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.20.0.bb
index ade935df9e..baba628df4 100644
--- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.18.5.bb
+++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.20.0.bb
@@ -4,13 +4,12 @@ DESCRIPTION = "'Good' GStreamer plugins"
 HOMEPAGE = "https://gstreamer.freedesktop.org/;
 BUGTRACKER = 
"https://gitlab.freedesktop.org/gstreamer/gst-plugins-good/-/issues;
 
-SRC_URI = 
"https://gstreamer.freedesktop.org/src/gst-plugins-good/gst-plugins-good-${PV}.tar.xz
 \
-   
file://0001-qt-include-ext-qt-gstqtgl.h-instead-of-gst-gl-gstglf.patch \
-   "
+require gstreamer1.0-source.inc
 
-SRC_URI[sha256sum] = 
"3aaeeea7765fbf8801acce4a503a9b05f73f04e8a35352e9d00232cfd555796b"
+S = "${SRC_BASE}/subprojects/gst-plugins-good"
 
-S = "${WORKDIR}/gst-plugins-good-${PV}"
+SRC_URI += 
"file://0001-qt-include-ext-qt-gstqtgl.h-instead-of-gst-gl-gstglf.patch;patchdir=${SRC_BASE}
 \
+   "
 
 LICENSE = "GPLv2+ & LGPLv2.1+"
 LIC_FILES_CHKSUM = "file://COPYING;md5=a6f89e2100d9b6cdffcea4f398e37343 \
-- 
2.34.1


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

[OE-core][RFC PATCH 01/12] gstreamer1.0: 1.18.5 -> 1.20.0

[Claudius Heine]

Gstreamer moved to a monorepo structure, this commit changed to use one
SRC_URI defined in gstreamer1.0-source.inc, which all recipes use and
set their own `S`ource path to their subproject path.

Backported patches where remove and the other, which still apply where
updated.

Signed-off-by: Claudius Heine 
---
 .../gstreamer/gstreamer1.0-source.inc |   5 +
 ...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 |  72 +++
 ...-the-environment-for-installed_tests.patch |  58 -
 ...er1.0_1.18.5.bb => gstreamer1.0_1.20.0.bb} |  25 ++--
 10 files changed, 140 insertions(+), 342 deletions(-)
 create mode 100644 meta/recipes-multimedia/gstreamer/gstreamer1.0-source.inc
 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} (72%)

diff --git a/meta/recipes-multimedia/gstreamer/gstreamer1.0-source.inc 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-source.inc
new file mode 100644
index 00..ca38322576
--- /dev/null
+++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-source.inc
@@ -0,0 +1,5 @@
+REVISION = "4f7e881fcc2e6df3ce04584cf5edb07b2682891a"
+
+SRC_BASE = "${WORKDIR}/gstreamer-${REVISION}"
+SRC_URI = 
"https://gitlab.freedesktop.org/gstreamer/gstreamer/-/archive/${REVISION}/gstreamer-${REVISION}.tar.gz;
+SRC_URI[sha256sum] = 
"bc9f6b9402d7575d8a7490f0aae347e6b524741cc459cb7aa25040fb55fcb606"
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: 
<https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/669>
-
-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 @@ gst_plugin_loader_spawn (GstPluginLoader * loader)
-   if (loader->child_running)
- return TRUE;
- 
--  /* Find the gst-plugin-scanner: first try the env-var if it is set,
--   * otherwise use the installed version */
-+  /* Find the gst-plugin-scanner */
-   env = g_getenv ("GST_PLUGIN_SCANNER_1_0&q

[OE-core][RFC PATCH 02/12] gstreamer1.0-plugins-base: 1.18.5 -> 1.20.0

[Claudius Heine]

Signed-off-by: Claudius Heine 
---
 ...et-caps-from-src-pad-when-query-caps.patch | 10 +++---
 ...parse-enhance-SSA-text-lines-parsing.patch | 10 +++---
 ...iv-fb-Make-sure-config.h-is-included.patch |  8 ++---
 ...004-glimagesink-Downrank-to-marginal.patch | 32 ---
 ...bb => gstreamer1.0-plugins-base_1.20.0.bb} | 17 +-
 5 files changed, 22 insertions(+), 55 deletions(-)
 delete mode 100644 
meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0004-glimagesink-Downrank-to-marginal.patch
 rename meta/recipes-multimedia/gstreamer/{gstreamer1.0-plugins-base_1.18.5.bb 
=> gstreamer1.0-plugins-base_1.20.0.bb} (85%)

diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0001-ENGR00312515-get-caps-from-src-pad-when-query-caps.patch
 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0001-ENGR00312515-get-caps-from-src-pad-when-query-caps.patch
index d5d9838372..bbc24b3e84 100644
--- 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0001-ENGR00312515-get-caps-from-src-pad-when-query-caps.patch
+++ 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0001-ENGR00312515-get-caps-from-src-pad-when-query-caps.patch
@@ -9,16 +9,16 @@ Upstream-Status: Pending
 
 Signed-off-by: zhouming 
 ---
- gst-libs/gst/tag/gsttagdemux.c | 13 +
+ subprojects/gst-plugins-base/gst-libs/gst/tag/gsttagdemux.c | 13 +
  1 file changed, 13 insertions(+)
- mode change 100644 => 100755 gst-libs/gst/tag/gsttagdemux.c
+ mode change 100644 => 100755 
subprojects/gst-plugins-base/gst-libs/gst/tag/gsttagdemux.c
 
-diff --git a/gst-libs/gst/tag/gsttagdemux.c b/gst-libs/gst/tag/gsttagdemux.c
+diff --git a/subprojects/gst-plugins-base/gst-libs/gst/tag/gsttagdemux.c 
b/subprojects/gst-plugins-base/gst-libs/gst/tag/gsttagdemux.c
 old mode 100644
 new mode 100755
 index f545857..62d10ef
 a/gst-libs/gst/tag/gsttagdemux.c
-+++ b/gst-libs/gst/tag/gsttagdemux.c
+--- a/subprojects/gst-plugins-base/gst-libs/gst/tag/gsttagdemux.c
 b/subprojects/gst-plugins-base/gst-libs/gst/tag/gsttagdemux.c
 @@ -1777,6 +1777,19 @@ gst_tag_demux_pad_query (GstPad * pad, GstObject * 
parent, GstQuery * query)
}
break;
diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0002-ssaparse-enhance-SSA-text-lines-parsing.patch
 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0002-ssaparse-enhance-SSA-text-lines-parsing.patch
index e453a500c9..c6babd4ec9 100644
--- 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0002-ssaparse-enhance-SSA-text-lines-parsing.patch
+++ 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0002-ssaparse-enhance-SSA-text-lines-parsing.patch
@@ -10,16 +10,16 @@ Upstream-Status: Submitted 
[https://bugzilla.gnome.org/show_bug.cgi?id=747496]
 
 Signed-off-by: Mingke Wang 
 ---
- gst/subparse/gstssaparse.c | 150 +
+ subprojects/gst-plugins-base/gst/subparse/gstssaparse.c | 150 
+
  1 file changed, 134 insertions(+), 16 deletions(-)
- mode change 100644 => 100755 gst/subparse/gstssaparse.c
+ mode change 100644 => 100755 
subprojects/gst-plugins-base/gst/subparse/gstssaparse.c
 
-diff --git a/gst/subparse/gstssaparse.c b/gst/subparse/gstssaparse.c
+diff --git a/subprojects/gst-plugins-base/gst/subparse/gstssaparse.c 
b/subprojects/gst-plugins-base/gst/subparse/gstssaparse.c
 old mode 100644
 new mode 100755
 index c849c08..4b9636c
 a/gst/subparse/gstssaparse.c
-+++ b/gst/subparse/gstssaparse.c
+--- a/subprojects/gst-plugins-base/gst/subparse/gstssaparse.c
 b/subprojects/gst-plugins-base/gst/subparse/gstssaparse.c
 @@ -262,6 +262,7 @@ gst_ssa_parse_remove_override_codes (GstSsaParse * parse, 
gchar * txt)
   * gst_ssa_parse_push_line:
   * @parse: caller element
diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0003-viv-fb-Make-sure-config.h-is-included.patch
 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0003-viv-fb-Make-sure-config.h-is-included.patch
index 2af83ff8b9..1bf21c2e4d 100644
--- 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0003-viv-fb-Make-sure-config.h-is-included.patch
+++ 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0003-viv-fb-Make-sure-config.h-is-included.patch
@@ -9,13 +9,13 @@ Upstream-Status: Pending
 
 Signed-off-by: Carlos Rafael Giani 
 ---
- gst-libs/gst/gl/gl-prelude.h | 4 
+ subprojects/gst-plugins-base/gst-libs/gst/gl/gl-prelude.h | 4 
  1 file changed, 4 insertions(+)
 
-diff --git a/gst-libs/gst/gl/gl-prelude.h b/gst-libs/gst/gl/gl-prelude.h
+diff --git a/subprojects/gst-plugins-base/gst-libs/gst/gl/gl-prelude.h 
b/subprojects/gst-plugins-base/gst-libs/gst/gl/gl-prelude.h
 index 05e1f62..96ce5e6 100644
 a/gst-libs/gst/gl/gl-prelude.h
-+++ b/gst-libs/gst/gl/gl-prelude.h
+--- a/subprojects/gst-plugins-base/gst-libs/gst/gl/gl-prelude.h
 b/subprojects/gst-plu

[OE-core][RFC PATCH 00/12] Update to gstreamer 1.20 for kirkstone

[Claudius Heine]

Hi,

since the release of gstreamer 1.20 is getting close, and it might make sense to
include it into the kirkstone release. I prepared preliminary update patchset
for it.

Gstreamer changed to use monorepos with 1.20 so I went ahead I changed all
recipes to use one SRC_URI from `gstreamer1.0-source.inc`. Currently I just use
a tarball from a unrelease commit id, but that will be changed to the correct
one, after gstreamer is released.

I have some ideas which could be implemented with this change, but I wanted
feedback for this first:
 - remove the version from the filename, and define the PV in the
   `gstreamer1.0-source.inc`, this might make future updates easier.
 - put all recipes together into one recipe, which provides packages for all
   subprojects, this improves build time and makes maintenance easier, but also
   breaks downstream bbappends.

Any feedback on this is welcome.

kind regards,
Claudius

Claudius Heine (12):
  gstreamer1.0: 1.18.5 -> 1.20.0
  gstreamer1.0-plugins-base: 1.18.5 -> 1.20.0
  gstreamer1.0-plugins-good: 1.18.5 -> 1.20.0
  gstreamer1.0-plugins-bad: 1.18.5 -> 1.20.0
  gstreamer1.0-plugins-ugly: 1.18.5 -> 1.20.0
  gstreamer1.0-libav: 1.18.5 -> 1.20.0
  gstreamer1.0-python: 1.18.5 -> 1.20.0
  gstreamer1.0-omx: 1.18.5 -> 1.20.0
  gstreamer1.0-vaapi: 1.18.5 -> 1.20.0
  gstreamer1.0-rtsp-server: 1.18.5 -> 1.20.0
  gst-examples: 1.18.5 -> 1.20.0
  gst-devtools: 1.18.5 -> 1.20.0

 ...ct-has-a-different-signature-on-musl.patch |   8 +-
 ...tools_1.18.5.bb => gst-devtools_1.20.0.bb} |   9 +-
 ...001-Make-player-examples-installable.patch |  22 +-
 ...mples_1.18.5.bb => gst-examples_1.20.0.bb} |  11 +-
 ...st-libav-fix-build-with-ffmpeg-5.0.0.patch | 346 --
 ...1.18.5.bb => gstreamer1.0-libav_1.20.0.bb} |   9 +-
 ...x_1.18.5.bb => gstreamer1.0-omx_1.20.0.bb} |   6 +-
 ...ialized-warnings-when-compiling-with.patch |   8 +-
 ...-avoid-including-sys-poll.h-directly.patch |   8 +-
 ...-sentinals-for-gst_structure_get-etc.patch |  24 +-
 ...issing-opencv-data-dir-in-yocto-buil.patch |  25 +-
 .../0005-msdk-fix-includedir-path.patch   |  26 +-
 bb => gstreamer1.0-plugins-bad_1.20.0.bb} |  20 +-
 ...et-caps-from-src-pad-when-query-caps.patch |  10 +-
 ...parse-enhance-SSA-text-lines-parsing.patch |  10 +-
 ...iv-fb-Make-sure-config.h-is-included.patch |   8 +-
 ...004-glimagesink-Downrank-to-marginal.patch |  32 --
 ...bb => gstreamer1.0-plugins-base_1.20.0.bb} |  17 +-
 ...t-gstqtgl.h-instead-of-gst-gl-gstglf.patch |  12 +-
 ...bb => gstreamer1.0-plugins-good_1.20.0.bb} |   9 +-
 ...bb => gstreamer1.0-plugins-ugly_1.20.0.bb} |  10 +-
 18.5.bb => gstreamer1.0-python_1.20.0.bb} |   7 +-
 bb => gstreamer1.0-rtsp-server_1.20.0.bb} |   8 +-
 .../gstreamer/gstreamer1.0-source.inc |   5 +
 ...1.18.5.bb => gstreamer1.0-vaapi_1.20.0.bb} |   5 +-
 ...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 |  72 
 ...-the-environment-for-installed_tests.patch |  58 ---
 ...er1.0_1.18.5.bb => gstreamer1.0_1.20.0.bb} |  25 +-
 34 files changed, 267 insertions(+), 865 deletions(-)
 rename meta/recipes-multimedia/gstreamer/{gst-devtools_1.18.5.bb => 
gst-devtools_1.20.0.bb} (83%)
 rename meta/recipes-multimedia/gstreamer/{gst-examples_1.18.5.bb => 
gst-examples_1.20.0.bb} (82%)
 delete mode 100644 
meta/recipes-multimedia/gstreamer/gstreamer1.0-libav/0001-gst-libav-fix-build-with-ffmpeg-5.0.0.patch
 rename meta/recipes-multimedia/gstreamer/{gstreamer1.0-libav_1.18.5.bb => 
gstreamer1.0-libav_1.20.0.bb} (66%)
 rename meta/recipes-multimedia/gstreamer/{gstreamer1.0-omx_1.18.5.bb => 
gstreamer1.0-omx_1.20.0.bb} (89%)
 rename meta/recipes-multimedia/gstreamer/{gstreamer1.0-plugins-bad_1.18.5.bb 
=> gstreamer1.0-plugins-bad_1.20.0.bb} (91%)
 delete mode 100644 
meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0004-glimagesink-Downrank-to-marginal.patch
 rename meta/recipes-multimedia/gstreamer/{gstreamer1.0-plugins-base_1.18.5.bb 
=> gstreamer1.0-plugins-base_1.20.0.bb} (85%)
 rename meta/recipes-multimedia/gstreamer/{gstreamer1.0-plugins-good_1.18.5.bb 
=> gstreamer1.0-plugins-good_1.20.0.bb} (91%)
 rename meta/recipes-multimedia/gstreamer/{gstreamer1.0-plugins-ugly_1.18.5.bb 
=> gstreamer1.0-plugins-ugly_1.20.0.bb} (82%)
 rename meta/recipes-multimedia/gstreamer/{gstreamer1.0-python_1.18.5.bb => 
gstreamer1.0-python_1.20.0.bb} (79%)
 rename meta/recipes-multimedia/gstreamer/{gstreamer1.0-rtsp-server_1.18.5.bb 
=> gstreamer1.0-rtsp-server_1.20.0.bb} (70%)
 create mode 100644 meta/recipes-multimedia/gstreamer/gstreamer1

[OE-core] [PATCH] rng-tools: add systemd-udev-settle wants to service

[Claudius Heine]

rngd needs to start after `systemd-udev-settle` in order for the kernel
modules of the random source hardware to be loaded before it is started.

However, since the `rngd.service` does not require or want
`systemd-udev-settle.service` it might not be scheduled for start and
the `After=systemd-udev-settle.service` there has no effect.

Adding `Wants=systemd-udev-settle.service` provides a weak requirement
to it, so that the `rngd` is started after it, if possible.

Signed-off-by: Claudius Heine 
---

Hi,

this is a fix, which should probably be backported to the maintained releases.

regards,
Claudius

 meta/recipes-support/rng-tools/rng-tools/rngd.service | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-support/rng-tools/rng-tools/rngd.service 
b/meta/recipes-support/rng-tools/rng-tools/rngd.service
index 0559b97991..568686e80e 100644
--- a/meta/recipes-support/rng-tools/rng-tools/rngd.service
+++ b/meta/recipes-support/rng-tools/rng-tools/rngd.service
@@ -3,6 +3,7 @@ Description=Hardware RNG Entropy Gatherer Daemon
 DefaultDependencies=no
 After=systemd-udev-settle.service
 Before=sysinit.target shutdown.target
+Wants=systemd-udev-settle.service
 Conflicts=shutdown.target
 
 [Service]
-- 
2.33.0


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