[OE-core] [PATCH v2] toolchain-shar-relocate.sh: Add check for missing command 'file'

2024-03-21 Thread Tom Hochstein
On a machine without the file command, the SDK install fails with a
cryptic error message.

```
xargs: file: No such file or directory
sed: no input files
Failed to replace perl. Relocate script failed. Abort!
```

Add a test for 'file' to print a clear error message.

```
The command 'file' is required by the relocation script, please install it 
first. Abort!
```

Signed-off-by: Tom Hochstein 
---
 meta/files/toolchain-shar-relocate.sh | 10 ++
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/meta/files/toolchain-shar-relocate.sh 
b/meta/files/toolchain-shar-relocate.sh
index e8ab357717..b017714df0 100644
--- a/meta/files/toolchain-shar-relocate.sh
+++ b/meta/files/toolchain-shar-relocate.sh
@@ -1,7 +1,9 @@
-if ! xargs --version > /dev/null 2>&1; then
-   echo "xargs is required by the relocation script, please install it 
first. Abort!"
-   exit 1
-fi
+for cmd in xargs file; do
+   if ! command -v $cmd > /dev/null 2>&1; then
+   echo "The command '$cmd' is required by the relocation script, 
please install it first. Abort!"
+   exit 1
+   fi
+done
 
 # fix dynamic loader paths in all ELF SDK binaries
 # allow symlinks to be accessed via the find command too
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#197425): 
https://lists.openembedded.org/g/openembedded-core/message/197425
Mute This Topic: https://lists.openembedded.org/mt/105070397/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]] toolchain-shar-relocate.sh: Add check for missing command 'file'

2024-03-21 Thread Tom Hochstein
On a machine without the file command, the SDK install fails with a
cryptic error message.

```
xargs: file: No such file or directory
sed: no input files
Failed to replace perl. Relocate script failed. Abort!
```

Add a test for 'file' to print a clear error message.

```
The command 'file' is required by the relocation script, please install it 
first. Abort!
```

Signed-off-by: Tom Hochstein 
---
 meta/files/toolchain-shar-relocate.sh | 10 ++
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/meta/files/toolchain-shar-relocate.sh 
b/meta/files/toolchain-shar-relocate.sh
index e8ab357717..b017714df0 100644
--- a/meta/files/toolchain-shar-relocate.sh
+++ b/meta/files/toolchain-shar-relocate.sh
@@ -1,7 +1,9 @@
-if ! xargs --version > /dev/null 2>&1; then
-   echo "xargs is required by the relocation script, please install it 
first. Abort!"
-   exit 1
-fi
+for cmd in xargs file; do
+   if ! command -v $cmd > /dev/null 2>&1; then
+   echo "The command '$cmd' is required by the relocation script, 
please install it first. Abort!"
+   exit 1
+   fi
+done
 
 # fix dynamic loader paths in all ELF SDK binaries
 # allow symlinks to be accessed via the find command too
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#197424): 
https://lists.openembedded.org/g/openembedded-core/message/197424
Mute This Topic: https://lists.openembedded.org/mt/105070379/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] toolchain-shar-relocate.sh: Add check for missing command 'file'

2024-03-21 Thread Tom Hochstein

Thanks Alexandre,

On 3/21/2024 10:19 AM, Alexandre Belloni wrote:

Hello Tom,

On 16/03/2024 12:35:58-0500, Tom Hochstein wrote:

On a machine without the file command, the SDK install fails with a
cryptic error message.

```
xargs: file: No such file or directory
sed: no input files
Failed to replace perl. Relocate script failed. Abort!
```

Add a test for 'file' to print a clear error message.

```
The command 'file' is required by the relocation script, please install it 
first. Abort!
```

Signed-off-by: Tom Hochstein 
---
  meta/files/toolchain-shar-relocate.sh | 10 ++
  1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/meta/files/toolchain-shar-relocate.sh 
b/meta/files/toolchain-shar-relocate.sh
index e8ab357717..3d55e38102 100644
--- a/meta/files/toolchain-shar-relocate.sh
+++ b/meta/files/toolchain-shar-relocate.sh
@@ -1,7 +1,9 @@
-if ! xargs --version > /dev/null 2>&1; then
-   echo "xargs is required by the relocation script, please install it first. 
Abort!"
-   exit 1
-fi
+for cmd in xargs file; do
+   if ! command -v $cmd &> /dev/null; then

This fails on some of our builders:
https://autobuilder.yoctoproject.org/typhoon/#/builders/20/builds/9148/steps/12/logs/stdio


Aha, looks like &> is a bashism. V2 coming.




+   echo "The command '$cmd' is required by the relocation script, 
please install it first. Abort!"
+   exit 1
+   fi
+done
  
  # fix dynamic loader paths in all ELF SDK binaries

  # allow symlinks to be accessed via the find command too
--
2.25.1






-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#197423): 
https://lists.openembedded.org/g/openembedded-core/message/197423
Mute This Topic: https://lists.openembedded.org/mt/104970783/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] toolchain-shar-relocate.sh: Add check for missing command 'file'

2024-03-16 Thread Tom Hochstein
On a machine without the file command, the SDK install fails with a
cryptic error message.

```
xargs: file: No such file or directory
sed: no input files
Failed to replace perl. Relocate script failed. Abort!
```

Add a test for 'file' to print a clear error message.

```
The command 'file' is required by the relocation script, please install it 
first. Abort!
```

Signed-off-by: Tom Hochstein 
---
 meta/files/toolchain-shar-relocate.sh | 10 ++
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/meta/files/toolchain-shar-relocate.sh 
b/meta/files/toolchain-shar-relocate.sh
index e8ab357717..3d55e38102 100644
--- a/meta/files/toolchain-shar-relocate.sh
+++ b/meta/files/toolchain-shar-relocate.sh
@@ -1,7 +1,9 @@
-if ! xargs --version > /dev/null 2>&1; then
-   echo "xargs is required by the relocation script, please install it 
first. Abort!"
-   exit 1
-fi
+for cmd in xargs file; do
+   if ! command -v $cmd &> /dev/null; then
+   echo "The command '$cmd' is required by the relocation script, 
please install it first. Abort!"
+   exit 1
+   fi
+done
 
 # fix dynamic loader paths in all ELF SDK binaries
 # allow symlinks to be accessed via the find command too
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#197256): 
https://lists.openembedded.org/g/openembedded-core/message/197256
Mute This Topic: https://lists.openembedded.org/mt/104970783/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 v3] bmaptool: Add bmap-tools runtime alias for compatibility

2024-03-07 Thread Tom Hochstein
The rename of bmap-tools to bmaptool creates an incompatibility that
will break package feeds. Restore package feed compatibility by adding
a bmap-tools runtime alias.

Acked-by: Otavio Salvador 
Signed-off-by: Tom Hochstein 
---
 meta/recipes-support/bmaptool/bmaptool_git.bb | 4 
 1 file changed, 4 insertions(+)

diff --git a/meta/recipes-support/bmaptool/bmaptool_git.bb 
b/meta/recipes-support/bmaptool/bmaptool_git.bb
index 87328af8c6..f706477323 100644
--- a/meta/recipes-support/bmaptool/bmaptool_git.bb
+++ b/meta/recipes-support/bmaptool/bmaptool_git.bb
@@ -28,4 +28,8 @@ RDEPENDS:${PN} = "python3-core python3-compression 
python3-misc python3-mmap pyt
 
 inherit setuptools3
 
+# For compatibility with layers before scarthgap
+RREPLACES:${PN} = "bmap-tools"
+RCONFLICTS:${PN} = "bmap-tools"
+
 BBCLASSEXTEND = "native nativesdk"
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#196817): 
https://lists.openembedded.org/g/openembedded-core/message/196817
Mute This Topic: https://lists.openembedded.org/mt/104793996/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] bmaptool: Add bmap-tools runtime alias for compatibility

2024-03-07 Thread Tom Hochstein
The rename of bmap-tools to bmaptool creates an incompatibility that
will break package feeds. Restore compatibility by adding bmap-tools as
a runtime alias.

Acked-by: Otavio Salvador 
Signed-off-by: Tom Hochstein 
---
 meta/recipes-support/bmaptool/bmaptool_git.bb | 5 +
 1 file changed, 5 insertions(+)

diff --git a/meta/recipes-support/bmaptool/bmaptool_git.bb 
b/meta/recipes-support/bmaptool/bmaptool_git.bb
index 87328af8c6..f0f3b85902 100644
--- a/meta/recipes-support/bmaptool/bmaptool_git.bb
+++ b/meta/recipes-support/bmaptool/bmaptool_git.bb
@@ -28,4 +28,9 @@ RDEPENDS:${PN} = "python3-core python3-compression 
python3-misc python3-mmap pyt
 
 inherit setuptools3
 
+# For compatibility with layers before scarthgap
+RPROVIDES:${PN} = "bmap-tools"
+RREPLACES:${PN} = "bmap-tools"
+RCONFLICTS:${PN} = "bmap-tools"
+
 BBCLASSEXTEND = "native nativesdk"
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#196791): 
https://lists.openembedded.org/g/openembedded-core/message/196791
Mute This Topic: https://lists.openembedded.org/mt/104787184/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] bmaptool: Add bmap-tools alias for compatibility

2024-03-05 Thread Tom Hochstein
The rename of bmap-tools to bmaptool creates an incompatibility for
pre-scarthgap layers. Restore compatibility by adding bmap-tools as an
alias.

Acked-by: Otavio Salvador 
Signed-off-by: Tom Hochstein 
---
 meta/recipes-support/bmaptool/bmaptool_git.bb | 8 
 1 file changed, 8 insertions(+)

diff --git a/meta/recipes-support/bmaptool/bmaptool_git.bb 
b/meta/recipes-support/bmaptool/bmaptool_git.bb
index 87328af8c6..33f46d7070 100644
--- a/meta/recipes-support/bmaptool/bmaptool_git.bb
+++ b/meta/recipes-support/bmaptool/bmaptool_git.bb
@@ -9,6 +9,9 @@ SECTION = "console/utils"
 LICENSE = "GPL-2.0-only"
 LIC_FILES_CHKSUM = "file://LICENSE;md5=b234ee4d69f5fce4486a80fdaf4a4263"
 
+# For compatibility with layers before scarthgap
+PROVIDES += "bmap-tools"
+
 FILESEXTRAPATHS:prepend := "${THISDIR}/files:"
 SRC_URI = "git://github.com/yoctoproject/${BPN};branch=main;protocol=https \
file://0001-BmapCopy.py-fix-error-message.patch \
@@ -28,4 +31,9 @@ RDEPENDS:${PN} = "python3-core python3-compression 
python3-misc python3-mmap pyt
 
 inherit setuptools3
 
+# For compatibility with layers before scarthgap
+RPROVIDES:${PN} = "bmap-tools"
+RREPLACES:${PN} = "bmap-tools"
+RCONFLICTS:${PN} = "bmap-tools"
+
 BBCLASSEXTEND = "native nativesdk"
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#196650): 
https://lists.openembedded.org/g/openembedded-core/message/196650
Mute This Topic: https://lists.openembedded.org/mt/104753355/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] linux-firmware: add firmware files for NXP BT chipsets

2023-09-05 Thread Tom Hochstein
These BT only firmware files are submitted for NXP chips.

88w8997-bt:
uartuart8997_bt_v4.bin version: 16.92.21.p81
helper_uart_300.bin version: 16.92.21.p81

88w8987-bt:
uartuart8987_bt.bin version: 16.92.21.p76.5

88w9098-bt:
uartuart9098_bt_v1.bin version: 17.92.1.p136.24

iw416-bt:
uartiw416_bt_v0.bin version: 16.92.21.p81

iw611-bt / iw612-bt:
uartspi_n61x_v1.bin.se version: 18.99.1.p154.40

These firmware files are downloaded over UART interface. The
helper_uart_300.bin needs to be downloaded before
uartuart8997_bt_v4.bin for 88w8997 chip to set download baudrate
to 300.

The above commit message is copied from the source commit message:
https://git.kernel.org/pub/scm/linux/kernel/git/firmware/linux-firmware.git/commit/nxp?id=1f82dd25c3ea59b6bed80ee685d860f0c5f19875

Signed-off-by: Tom Hochstein 
---
 .../linux-firmware/linux-firmware_20230804.bb | 50 +++
 1 file changed, 50 insertions(+)

diff --git a/meta/recipes-kernel/linux-firmware/linux-firmware_20230804.bb 
b/meta/recipes-kernel/linux-firmware/linux-firmware_20230804.bb
index d761673db3..ee14630ce0 100644
--- a/meta/recipes-kernel/linux-firmware/linux-firmware_20230804.bb
+++ b/meta/recipes-kernel/linux-firmware/linux-firmware_20230804.bb
@@ -41,6 +41,7 @@ LICENSE = "\
 & Firmware-myri10ge_firmware \
 & Firmware-netronome \
 & Firmware-nvidia \
+& Firmware-nxp \
 & Firmware-OLPC \
 & Firmware-ath9k-htc \
 & Firmware-phanfw \
@@ -107,6 +108,7 @@ LIC_FILES_CHKSUM = 
"file://LICENCE.Abilis;md5=b5ee3f410780e56711ad48eadc22b8bc \
 
file://LICENCE.Netronome;md5=4add08f2577086d7996503cddf5f \
 file://LICENCE.nvidia;md5=4428a922ed3ba2ceec95f076a488ce07 
\
 file://LICENCE.NXP;md5=58bb8ba632cd729b9ba6183bc6aed36f \
+file://LICENSE.nxp;md5=cca321ca1524d6a1e4fed87486cd82dc \
 file://LICENCE.OLPC;md5=5b917f9d8c061991be4f6f5f108719cd \
 
file://LICENCE.open-ath9k-htc-firmware;md5=1b33c9f4d17bc4d457bdb23727046837 \
 file://LICENCE.phanfw;md5=954dcec0e051f9409812b561ea743bfa 
\
@@ -177,6 +179,7 @@ NO_GENERIC_LICENSE[Firmware-moxa] = "LICENCE.moxa"
 NO_GENERIC_LICENSE[Firmware-myri10ge_firmware] = "LICENCE.myri10ge_firmware"
 NO_GENERIC_LICENSE[Firmware-netronome] = "LICENCE.Netronome"
 NO_GENERIC_LICENSE[Firmware-nvidia] = "LICENCE.nvidia"
+NO_GENERIC_LICENSE[Firmware-nxp] = "LICENSE.nxp"
 NO_GENERIC_LICENSE[Firmware-OLPC] = "LICENCE.OLPC"
 NO_GENERIC_LICENSE[Firmware-ath9k-htc] = "LICENCE.open-ath9k-htc-firmware"
 NO_GENERIC_LICENSE[Firmware-phanfw] = "LICENCE.phanfw"
@@ -312,6 +315,16 @@ PACKAGES =+ "${PN}-ralink-license ${PN}-ralink \
  ${PN}-nvidia-license \
  ${PN}-nvidia-tegra-k1 ${PN}-nvidia-tegra \
  ${PN}-nvidia-gpu \
+ ${PN}-nxp-license \
+ ${PN}-nxp8987-sdio \
+ ${PN}-nxp8997-common \
+ ${PN}-nxp8997-pcie \
+ ${PN}-nxp8997-sdio \
+ ${PN}-nxp9098-common \
+ ${PN}-nxp9098-pcie \
+ ${PN}-nxp9098-sdio \
+ ${PN}-nxpiw416-sdio \
+ ${PN}-nxpiw612-sdio \
  ${PN}-netronome-license ${PN}-netronome \
  ${PN}-qat ${PN}-qat-license \
  ${PN}-qcom-license ${PN}-qcom-yamato-license \
@@ -547,6 +560,42 @@ FILES:${PN}-netronome = " \
 
 RDEPENDS:${PN}-netronome += "${PN}-netronome-license"
 
+# For NXP
+LICENSE:${PN}-nxp8987-sdio = "Firmware-nxp"
+LICENSE:${PN}-nxp8997-common = "Firmware-nxp"
+LICENSE:${PN}-nxp8997-pcie = "Firmware-nxp"
+LICENSE:${PN}-nxp8997-sdio = "Firmware-nxp"
+LICENSE:${PN}-nxp9098-common = "Firmware-nxp"
+LICENSE:${PN}-nxp9098-pcie = "Firmware-nxp"
+LICENSE:${PN}-nxp9098-sdio = "Firmware-nxp"
+LICENSE:${PN}-nxpiw416-sdio = "Firmware-nxp"
+LICENSE:${PN}-nxpiw612-sdio = "Firmware-nxp"
+LICENSE:${PN}-nxp-license = "Firmware-nxp"
+
+FILES:${PN}-nxp8987-sdio = "${nonarch_base_libdir}/firmware/nxp/*8987*"
+FILES:${PN}-nxp8997-common = " \
+${nonarch_base_libdir}/firmware/nxp/uartuart8997_bt_v4.bin \
+${nonarch_base_libdir}/firmware/nxp/helper_uart_300.bin \
+"
+ALLOW_EMPTY:${PN}-nxp8997-pcie = "1"
+ALLOW_EMPTY:${PN}-nxp8997-sdio = "1"
+FILES:${PN}-nxp9098-common = 
"${nonarch_base_libdir}/firmware/nxp/uartuart9098_bt_v1.bin"
+ALLOW_EMPTY:${PN}-nxp9098-pcie = "1"
+ALLOW_EMPTY:${PN}-nxp9098-sdio = "1"
+FILES:${PN}-nxpiw416-sdio = "${nonarch_base_libdir}/firmware/nxp/*iw416*"
+FILES:${PN}-nxpiw612-sdio = 
"${nonarch_base_libdir}/firmware/nxp/uartspi_n61x_v1.bin.se"
+FILES:${PN}-nxp-license = "${nonarch_base_libdir}/firmwar

Re: [OE-core] [PATCH] linux-firmware: add firmware files for NXP BT chipsets

2023-09-05 Thread Tom Hochstein
On Tue, Aug 29, 2023 at 05:08 AM, Tom Hochstein wrote:

> 
> On Fri, Aug 25, 2023 at 07:37 AM, Tom Hochstein wrote:
> 
>> Please hold off on this. The -common design is not working correctly and
>> is causing those firmware packages to be registered as a runtime
>> dependency of the main package.
> 
> 
> 
> Actually, the -common design is working fine, there was just a
> misunderstanding.
> 
> Please do consider this patch as-is.
> 
> Tom
> 
> 
> 

Ping

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#187250): 
https://lists.openembedded.org/g/openembedded-core/message/187250
Mute This Topic: https://lists.openembedded.org/mt/100943055/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] linux-firmware: add firmware files for NXP BT chipsets

2023-08-29 Thread Tom Hochstein
On Fri, Aug 25, 2023 at 07:37 AM, Tom Hochstein wrote:

> 
> Please hold off on this. The -common design is not working correctly and
> is causing those firmware packages to be registered as a runtime
> dependency of the main package.

Actually, the -common design is working fine, there was just a misunderstanding.

Please do consider this patch as-is.

Tom

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#186856): 
https://lists.openembedded.org/g/openembedded-core/message/186856
Mute This Topic: https://lists.openembedded.org/mt/100943055/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] linux-firmware: add firmware files for NXP BT chipsets

2023-08-25 Thread Tom Hochstein
Please hold off on this. The -common design is not working correctly and is 
causing those firmware packages to be registered as a runtime dependency of the 
main package.

Tom

-Original Message-
From: openembedded-core@lists.openembedded.org 
 On Behalf Of Tom Hochstein via 
lists.openembedded.org
Sent: Thursday, August 24, 2023 3:04 PM
To: openembedded-core@lists.openembedded.org; otavio.salva...@ossystems.com.br
Cc: Tom Hochstein 
Subject: [OE-core] [PATCH] linux-firmware: add firmware files for NXP BT 
chipsets

These BT only firmware files are submitted for NXP chips.

88w8997-bt:
uartuart8997_bt_v4.bin version: 16.92.21.p81
helper_uart_300.bin version: 16.92.21.p81

88w8987-bt:
uartuart8987_bt.bin version: 16.92.21.p76.5

88w9098-bt:
uartuart9098_bt_v1.bin version: 17.92.1.p136.24

iw416-bt:
uartiw416_bt_v0.bin version: 16.92.21.p81

iw611-bt / iw612-bt:
uartspi_n61x_v1.bin.se version: 18.99.1.p154.40

These firmware files are downloaded over UART interface. The
helper_uart_300.bin needs to be downloaded before
uartuart8997_bt_v4.bin for 88w8997 chip to set download baudrate
to 300.

The above commit message is copied from the source commit message:
https://git.kernel.org/pub/scm/linux/kernel/git/firmware/linux-firmware.git/commit/nxp?id=1f82dd25c3ea59b6bed80ee685d860f0c5f19875

Signed-off-by: Tom Hochstein 
---
 .../linux-firmware/linux-firmware_20230625.bb | 50 +++
 1 file changed, 50 insertions(+)

diff --git a/meta/recipes-kernel/linux-firmware/linux-firmware_20230625.bb 
b/meta/recipes-kernel/linux-firmware/linux-firmware_20230625.bb
index 6765226b9d..45556d419e 100644
--- a/meta/recipes-kernel/linux-firmware/linux-firmware_20230625.bb
+++ b/meta/recipes-kernel/linux-firmware/linux-firmware_20230625.bb
@@ -40,6 +40,7 @@ LICENSE = "\
 & Firmware-myri10ge_firmware \
 & Firmware-netronome \
 & Firmware-nvidia \
+& Firmware-nxp \
 & Firmware-OLPC \
 & Firmware-ath9k-htc \
 & Firmware-phanfw \
@@ -105,6 +106,7 @@ LIC_FILES_CHKSUM = 
"file://LICENCE.Abilis;md5=b5ee3f410780e56711ad48eadc22b8bc \
 
file://LICENCE.Netronome;md5=4add08f2577086d7996503cddf5f \
 file://LICENCE.nvidia;md5=4428a922ed3ba2ceec95f076a488ce07 
\
 file://LICENCE.NXP;md5=58bb8ba632cd729b9ba6183bc6aed36f \
+file://LICENSE.nxp;md5=cca321ca1524d6a1e4fed87486cd82dc \
 file://LICENCE.OLPC;md5=5b917f9d8c061991be4f6f5f108719cd \
 
file://LICENCE.open-ath9k-htc-firmware;md5=1b33c9f4d17bc4d457bdb23727046837 \
 file://LICENCE.phanfw;md5=954dcec0e051f9409812b561ea743bfa 
\
@@ -174,6 +176,7 @@ NO_GENERIC_LICENSE[Firmware-moxa] = "LICENCE.moxa"
 NO_GENERIC_LICENSE[Firmware-myri10ge_firmware] = "LICENCE.myri10ge_firmware"
 NO_GENERIC_LICENSE[Firmware-netronome] = "LICENCE.Netronome"
 NO_GENERIC_LICENSE[Firmware-nvidia] = "LICENCE.nvidia"
+NO_GENERIC_LICENSE[Firmware-nxp] = "LICENSE.nxp"
 NO_GENERIC_LICENSE[Firmware-OLPC] = "LICENCE.OLPC"
 NO_GENERIC_LICENSE[Firmware-ath9k-htc] = "LICENCE.open-ath9k-htc-firmware"
 NO_GENERIC_LICENSE[Firmware-phanfw] = "LICENCE.phanfw"
@@ -309,6 +312,16 @@ PACKAGES =+ "${PN}-ralink-license ${PN}-ralink \
  ${PN}-nvidia-license \
  ${PN}-nvidia-tegra-k1 ${PN}-nvidia-tegra \
  ${PN}-nvidia-gpu \
+ ${PN}-nxp-license \
+ ${PN}-nxp8987-sdio \
+ ${PN}-nxp8997-common \
+ ${PN}-nxp8997-pcie \
+ ${PN}-nxp8997-sdio \
+ ${PN}-nxp9098-common \
+ ${PN}-nxp9098-pcie \
+ ${PN}-nxp9098-sdio \
+ ${PN}-nxpiw416-sdio \
+ ${PN}-nxpiw612-sdio \
  ${PN}-netronome-license ${PN}-netronome \
  ${PN}-qat ${PN}-qat-license \
  ${PN}-qcom-license ${PN}-qcom-yamato-license \
@@ -543,6 +556,42 @@ FILES:${PN}-netronome = " \

 RDEPENDS:${PN}-netronome += "${PN}-netronome-license"

+# For NXP
+LICENSE:${PN}-nxp8987-sdio = "Firmware-nxp"
+LICENSE:${PN}-nxp8997-common = "Firmware-nxp"
+LICENSE:${PN}-nxp8997-pcie = "Firmware-nxp"
+LICENSE:${PN}-nxp8997-sdio = "Firmware-nxp"
+LICENSE:${PN}-nxp9098-common = "Firmware-nxp"
+LICENSE:${PN}-nxp9098-pcie = "Firmware-nxp"
+LICENSE:${PN}-nxp9098-sdio = "Firmware-nxp"
+LICENSE:${PN}-nxpiw416-sdio = "Firmware-nxp"
+LICENSE:${PN}-nxpiw612-sdio = "Firmware-nxp"
+LICENSE:${PN}-nxp-license = "Firmware-nxp"
+
+FILES:${PN}-nxp8987-sdio = "${nonarch_base_libdir}/firmware/nxp/*8987*"
+FILES:${PN}-nxp8997-common = " \
+${nonarch_base_libdir}/firmware/nxp/uartuart8997_bt_v4.bin \
+${nonarch_base_libdir}/firmware/nxp/helper_uart_300.bin \
+"
+ALLOW_EMPTY

[OE-core] [PATCH] linux-firmware: add firmware files for NXP BT chipsets

2023-08-24 Thread Tom Hochstein
These BT only firmware files are submitted for NXP chips.

88w8997-bt:
uartuart8997_bt_v4.bin version: 16.92.21.p81
helper_uart_300.bin version: 16.92.21.p81

88w8987-bt:
uartuart8987_bt.bin version: 16.92.21.p76.5

88w9098-bt:
uartuart9098_bt_v1.bin version: 17.92.1.p136.24

iw416-bt:
uartiw416_bt_v0.bin version: 16.92.21.p81

iw611-bt / iw612-bt:
uartspi_n61x_v1.bin.se version: 18.99.1.p154.40

These firmware files are downloaded over UART interface. The
helper_uart_300.bin needs to be downloaded before
uartuart8997_bt_v4.bin for 88w8997 chip to set download baudrate
to 300.

The above commit message is copied from the source commit message:
https://git.kernel.org/pub/scm/linux/kernel/git/firmware/linux-firmware.git/commit/nxp?id=1f82dd25c3ea59b6bed80ee685d860f0c5f19875

Signed-off-by: Tom Hochstein 
---
 .../linux-firmware/linux-firmware_20230625.bb | 50 +++
 1 file changed, 50 insertions(+)

diff --git a/meta/recipes-kernel/linux-firmware/linux-firmware_20230625.bb 
b/meta/recipes-kernel/linux-firmware/linux-firmware_20230625.bb
index 6765226b9d..45556d419e 100644
--- a/meta/recipes-kernel/linux-firmware/linux-firmware_20230625.bb
+++ b/meta/recipes-kernel/linux-firmware/linux-firmware_20230625.bb
@@ -40,6 +40,7 @@ LICENSE = "\
 & Firmware-myri10ge_firmware \
 & Firmware-netronome \
 & Firmware-nvidia \
+& Firmware-nxp \
 & Firmware-OLPC \
 & Firmware-ath9k-htc \
 & Firmware-phanfw \
@@ -105,6 +106,7 @@ LIC_FILES_CHKSUM = 
"file://LICENCE.Abilis;md5=b5ee3f410780e56711ad48eadc22b8bc \
 
file://LICENCE.Netronome;md5=4add08f2577086d7996503cddf5f \
 file://LICENCE.nvidia;md5=4428a922ed3ba2ceec95f076a488ce07 
\
 file://LICENCE.NXP;md5=58bb8ba632cd729b9ba6183bc6aed36f \
+file://LICENSE.nxp;md5=cca321ca1524d6a1e4fed87486cd82dc \
 file://LICENCE.OLPC;md5=5b917f9d8c061991be4f6f5f108719cd \
 
file://LICENCE.open-ath9k-htc-firmware;md5=1b33c9f4d17bc4d457bdb23727046837 \
 file://LICENCE.phanfw;md5=954dcec0e051f9409812b561ea743bfa 
\
@@ -174,6 +176,7 @@ NO_GENERIC_LICENSE[Firmware-moxa] = "LICENCE.moxa"
 NO_GENERIC_LICENSE[Firmware-myri10ge_firmware] = "LICENCE.myri10ge_firmware"
 NO_GENERIC_LICENSE[Firmware-netronome] = "LICENCE.Netronome"
 NO_GENERIC_LICENSE[Firmware-nvidia] = "LICENCE.nvidia"
+NO_GENERIC_LICENSE[Firmware-nxp] = "LICENSE.nxp"
 NO_GENERIC_LICENSE[Firmware-OLPC] = "LICENCE.OLPC"
 NO_GENERIC_LICENSE[Firmware-ath9k-htc] = "LICENCE.open-ath9k-htc-firmware"
 NO_GENERIC_LICENSE[Firmware-phanfw] = "LICENCE.phanfw"
@@ -309,6 +312,16 @@ PACKAGES =+ "${PN}-ralink-license ${PN}-ralink \
  ${PN}-nvidia-license \
  ${PN}-nvidia-tegra-k1 ${PN}-nvidia-tegra \
  ${PN}-nvidia-gpu \
+ ${PN}-nxp-license \
+ ${PN}-nxp8987-sdio \
+ ${PN}-nxp8997-common \
+ ${PN}-nxp8997-pcie \
+ ${PN}-nxp8997-sdio \
+ ${PN}-nxp9098-common \
+ ${PN}-nxp9098-pcie \
+ ${PN}-nxp9098-sdio \
+ ${PN}-nxpiw416-sdio \
+ ${PN}-nxpiw612-sdio \
  ${PN}-netronome-license ${PN}-netronome \
  ${PN}-qat ${PN}-qat-license \
  ${PN}-qcom-license ${PN}-qcom-yamato-license \
@@ -543,6 +556,42 @@ FILES:${PN}-netronome = " \
 
 RDEPENDS:${PN}-netronome += "${PN}-netronome-license"
 
+# For NXP
+LICENSE:${PN}-nxp8987-sdio = "Firmware-nxp"
+LICENSE:${PN}-nxp8997-common = "Firmware-nxp"
+LICENSE:${PN}-nxp8997-pcie = "Firmware-nxp"
+LICENSE:${PN}-nxp8997-sdio = "Firmware-nxp"
+LICENSE:${PN}-nxp9098-common = "Firmware-nxp"
+LICENSE:${PN}-nxp9098-pcie = "Firmware-nxp"
+LICENSE:${PN}-nxp9098-sdio = "Firmware-nxp"
+LICENSE:${PN}-nxpiw416-sdio = "Firmware-nxp"
+LICENSE:${PN}-nxpiw612-sdio = "Firmware-nxp"
+LICENSE:${PN}-nxp-license = "Firmware-nxp"
+
+FILES:${PN}-nxp8987-sdio = "${nonarch_base_libdir}/firmware/nxp/*8987*"
+FILES:${PN}-nxp8997-common = " \
+${nonarch_base_libdir}/firmware/nxp/uartuart8997_bt_v4.bin \
+${nonarch_base_libdir}/firmware/nxp/helper_uart_300.bin \
+"
+ALLOW_EMPTY:${PN}-nxp8997-pcie = "1"
+ALLOW_EMPTY:${PN}-nxp8997-sdio = "1"
+FILES:${PN}-nxp9098-common = 
"${nonarch_base_libdir}/firmware/nxp/uartuart9098_bt_v1.bin"
+ALLOW_EMPTY:${PN}-nxp9098-pcie = "1"
+ALLOW_EMPTY:${PN}-nxp9098-sdio = "1"
+FILES:${PN}-nxpiw416-sdio = "${nonarch_base_libdir}/firmware/nxp/*iw416*"
+FILES:${PN}-nxpiw612-sdio = 
"${nonarch_base_libdir}/firmware/nxp/uartspi_n61x_v1.bin.se"
+FILES:${PN}-nxp-license = "${nonarch_base_libdir}/firmwar

[OE-core] [PATCH] weston: Cleanup and fix x11 and xwayland dependencies

2023-06-22 Thread Tom Hochstein
For the x11 backend package config, drop the redundant dependencies
libxcb and cairo. The former is listed twice in the package config,
while the latter is also listed globally.

For the xwayland package config, add the missing dependencies libxcb
and libxcursor. These dependencies are hidden when the x11 backend is
enabled by default. When the default setting is overridden to disable
the x11 backend, the following error occurs:
```
| Run-time dependency xcursor found: NO (tried pkgconfig and cmake)
|
| ../git/xwayland/meson.build:34:2: ERROR: Problem encountered: xwayland 
requires xcursor which was not found. Or, you can use '-Dxwayland=false'.
```

Signed-off-by: Tom Hochstein 
---
 meta/recipes-graphics/wayland/weston_11.0.1.bb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-graphics/wayland/weston_11.0.1.bb 
b/meta/recipes-graphics/wayland/weston_11.0.1.bb
index 4f6ce19915..0838791a6b 100644
--- a/meta/recipes-graphics/wayland/weston_11.0.1.bb
+++ b/meta/recipes-graphics/wayland/weston_11.0.1.bb
@@ -57,7 +57,7 @@ PACKAGECONFIG[kms] = 
"-Dbackend-drm=true,-Dbackend-drm=false,drm udev virtual/eg
 # Weston on Wayland (nested Weston)
 PACKAGECONFIG[wayland] = 
"-Dbackend-wayland=true,-Dbackend-wayland=false,virtual/egl virtual/libgles2"
 # Weston on X11
-PACKAGECONFIG[x11] = "-Dbackend-x11=true,-Dbackend-x11=false,virtual/libx11 
libxcb libxcb libxcursor cairo"
+PACKAGECONFIG[x11] = "-Dbackend-x11=true,-Dbackend-x11=false,virtual/libx11 
libxcb libxcursor"
 # Headless Weston
 PACKAGECONFIG[headless] = "-Dbackend-headless=true,-Dbackend-headless=false"
 # Weston on RDP
@@ -73,7 +73,7 @@ PACKAGECONFIG[webp] = 
"-Dimage-webp=true,-Dimage-webp=false,libwebp"
 # Weston with systemd-login support
 PACKAGECONFIG[systemd] = "-Dsystemd=true 
-Dlauncher-logind=true,-Dsystemd=false -Dlauncher-logind=false,systemd dbus"
 # Weston with Xwayland support (requires X11 and Wayland)
-PACKAGECONFIG[xwayland] = "-Dxwayland=true,-Dxwayland=false,xwayland"
+PACKAGECONFIG[xwayland] = "-Dxwayland=true,-Dxwayland=false,libxcb libxcursor 
xwayland"
 # colord CMS support
 PACKAGECONFIG[colord] = 
"-Ddeprecated-color-management-colord=true,-Ddeprecated-color-management-colord=false,colord"
 # Clients support
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#183289): 
https://lists.openembedded.org/g/openembedded-core/message/183289
Mute This Topic: https://lists.openembedded.org/mt/99705852/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] piglit: Add missing glslang dependencies

2023-05-16 Thread Tom Hochstein
The vulkan configuration requires glslangValidator for build and
runtime.

Fixes configure error:
```
| CMake Error at CMakeLists.txt:191 (message):
|   glslangValidator required by vulkan tests not found
```

Fixes runtime error:
```
name: vulkan@shaders@fs-large-local-array
result: fail
returncode: 1
out:
err: glslangValidator: No such file or directory
glslangValidator failed
```

Signed-off-by: Tom Hochstein 
---
 meta/recipes-graphics/piglit/piglit_git.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-graphics/piglit/piglit_git.bb 
b/meta/recipes-graphics/piglit/piglit_git.bb
index 116d2f5d81..6264483bf6 100644
--- a/meta/recipes-graphics/piglit/piglit_git.bb
+++ b/meta/recipes-graphics/piglit/piglit_git.bb
@@ -41,7 +41,7 @@ PACKAGECONFIG[freeglut] = 
"-DPIGLIT_USE_GLUT=1,-DPIGLIT_USE_GLUT=0,freeglut,"
 PACKAGECONFIG[glx] = "-DPIGLIT_BUILD_GLX_TESTS=ON,-DPIGLIT_BUILD_GLX_TESTS=OFF"
 PACKAGECONFIG[opencl] = 
"-DPIGLIT_BUILD_CL_TESTS=ON,-DPIGLIT_BUILD_CL_TESTS=OFF,virtual/opencl-icd"
 PACKAGECONFIG[x11] = 
"-DPIGLIT_BUILD_GL_TESTS=ON,-DPIGLIT_BUILD_GL_TESTS=OFF,${X11_DEPS}, 
${X11_RDEPS}"
-PACKAGECONFIG[vulkan] = 
"-DPIGLIT_BUILD_VK_TESTS=ON,-DPIGLIT_BUILD_VK_TESTS=OFF,vulkan-loader"
+PACKAGECONFIG[vulkan] = 
"-DPIGLIT_BUILD_VK_TESTS=ON,-DPIGLIT_BUILD_VK_TESTS=OFF,glslang-native 
vulkan-loader,glslang"
 
 export PIGLIT_BUILD_DIR = "../../../../git"
 
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#181423): 
https://lists.openembedded.org/g/openembedded-core/message/181423
Mute This Topic: https://lists.openembedded.org/mt/98935286/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/2] oeqa/sdk: Improve Meson test

2023-03-03 Thread Tom Hochstein
The meson wrapper setup command detection is broken in the case of an
implicit setup command with an option with a space-separated argument,
but the test was not detecting it since the case was not covered.

Add the option `--warnlevel 1` to the meson command line to cover this
case.

Signed-off-by: Tom Hochstein 
---
 meta/lib/oeqa/sdk/cases/buildepoxy.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/lib/oeqa/sdk/cases/buildepoxy.py 
b/meta/lib/oeqa/sdk/cases/buildepoxy.py
index ee515be188..147ee3e0ee 100644
--- a/meta/lib/oeqa/sdk/cases/buildepoxy.py
+++ b/meta/lib/oeqa/sdk/cases/buildepoxy.py
@@ -35,7 +35,7 @@ class EpoxyTest(OESDKTestCase):
 self.assertTrue(os.path.isdir(dirs["source"]))
 os.makedirs(dirs["build"])
 
-log = self._run("meson -Degl=no -Dglx=no -Dx11=false {build} 
{source}".format(**dirs))
+log = self._run("meson --warnlevel 1 -Degl=no -Dglx=no -Dx11=false 
{build} {source}".format(**dirs))
 # Check that Meson thinks we're doing a cross build and not a 
native
 self.assertIn("Build type: cross build", log)
 self._run("ninja -C {build} -v".format(**dirs))
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#178025): 
https://lists.openembedded.org/g/openembedded-core/message/178025
Mute This Topic: https://lists.openembedded.org/mt/97371489/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/2] meson: Fix wrapper handling of implicit setup command

2023-03-03 Thread Tom Hochstein
>From an SDK, running a meson setup build without an explicit setup
command can result in a native build when a cross build is expected.

The problem is in meson-wrapper where it tries to detect whether a
setup command is being used. The logic looks through all arguments for
a command, and the first argument it finds that doesn't start with a -
is treated as the command. This doesn't work for an implicit setup
command if any option with a space-separated argument exists. In this
case, the argument is incorrectly selected as the command, causing the
setup command options for the cross build to be excluded from the
command line, and thus a native build.

Improve the logic by just looking at the first argument. If it is
a known comand, then record it. Otherwise just assume it is the
implicit setup command.

Note that this fix does not address the possibility of a new meson
command. Two new echo statements are included to help the user in case
of trouble:

```
~/git/weston-imx$ meson --warnlevel 3 --prefix=/usr -Ddoc=false 
-Dbackend-drm-screencast-vaapi=false -Dcolor-management-lcms=false 
-Dpipewire=false -Dbackend-x11=false -Dxwayland=true -Dsimple-clients=all 
-Dbackend-wayland=false  -Dbackend-default=drm -Dbackend-rdp=false 
-Dtest-junit-xml=false -Dlauncher-libseat=false -Dimage-jpeg=false 
-Dimage-webp=false -Drenderer-g2d=true build
meson-wrapper: Implicit setup command assumed
meson-wrapper: Running meson with setup options: " 
--cross-file=/opt/fsl-imx-internal-xwayland/6.1-langdale/sysroots/x86_64-pokysdk-linux/usr/share/meson/aarch64-poky-linux-meson.cross
 
--native-file=/opt/fsl-imx-internal-xwayland/6.1-langdale/sysroots/x86_64-pokysdk-linux/usr/share/meson/meson.native
 "
The Meson build system
Version: 0.63.3
```

Signed-off-by: Tom Hochstein 
---
 meta/recipes-devtools/meson/meson/meson-wrapper | 17 -
 1 file changed, 8 insertions(+), 9 deletions(-)

diff --git a/meta/recipes-devtools/meson/meson/meson-wrapper 
b/meta/recipes-devtools/meson/meson/meson-wrapper
index fca64a5692..7455985297 100755
--- a/meta/recipes-devtools/meson/meson/meson-wrapper
+++ b/meta/recipes-devtools/meson/meson/meson-wrapper
@@ -13,20 +13,19 @@ fi
 # config is already in meson.cross.
 unset CC CXX CPP LD AR NM STRIP
 
-for arg in "$@"; do
-case "$arg" in
--*) continue ;;
-*) SUBCMD="$arg"; break ;;
-esac
-done
+case "$1" in
+setup|configure|dist|install|introspect|init|test|wrap|subprojects|rewrite|compile|devenv|env2mfile|help)
 MESON_CMD="$1" ;;
+*) echo meson-wrapper: Implicit setup command assumed; MESON_CMD=setup ;;
+esac
 
-if [ "$SUBCMD" = "setup" ] || [ -d "$SUBCMD" ]; then
-MESON_SUB_OPTS=" \
+if [ "$MESON_CMD" = "setup" ]; then
+MESON_SETUP_OPTS=" \
 
--cross-file="$OECORE_NATIVE_SYSROOT/usr/share/meson/${TARGET_PREFIX}meson.cross"
 \
 --native-file="$OECORE_NATIVE_SYSROOT/usr/share/meson/meson.native" \
 "
+echo meson-wrapper: Running meson with setup options: \"$MESON_SETUP_OPTS\"
 fi
 
 exec "$OECORE_NATIVE_SYSROOT/usr/bin/meson.real" \
 "$@" \
-$MESON_SUB_OPTS
+$MESON_SETUP_OPTS
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#178024): 
https://lists.openembedded.org/g/openembedded-core/message/178024
Mute This Topic: https://lists.openembedded.org/mt/97371487/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] meson: Fix wrapper handling of implicit setup command

2023-03-03 Thread Tom Hochstein
On Fri, Mar 3, 2023 at 11:08 AM, Alexander Kanavin wrote:

> 
> This should have been caught by meta/lib/oeqa/sdk/cases/buildepoxy.py
> - why wasn't it? Does the test need to be fixed?

The simple test case doesn't have the problematic syntax. Should I add this to 
the patch?

diff --git a/meta/lib/oeqa/sdk/cases/buildepoxy.py 
b/meta/lib/oeqa/sdk/cases/buildepoxy.py
index ee515be188..147ee3e0ee 100644
--- a/meta/lib/oeqa/sdk/cases/buildepoxy.py
+++ b/meta/lib/oeqa/sdk/cases/buildepoxy.py
@@ -35,7 +35,7 @@ class EpoxyTest(OESDKTestCase):
self.assertTrue(os.path.isdir(dirs["source"]))
os.makedirs(dirs["build"])

-            log = self._run("meson -Degl=no -Dglx=no -Dx11=false {build} 
{source}".format(**dirs))
+            log = self._run("meson --warnlevel 1 -Degl=no -Dglx=no -Dx11=false 
{build} {source}".format(**dirs))
# Check that Meson thinks we're doing a cross build and not a native
self.assertIn("Build type: cross build", log)
self._run("ninja -C {build} -v".format(**dirs))

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#178023): 
https://lists.openembedded.org/g/openembedded-core/message/178023
Mute This Topic: https://lists.openembedded.org/mt/97369067/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] meson: Fix wrapper handling of implicit setup command

2023-03-03 Thread Tom Hochstein
>From an SDK, running a meson setup build without an explicit setup
command can result in a native build when a cross build is expected.

The problem is in meson-wrapper where it tries to detect whether a
setup command is being used. The logic looks through all arguments for
a command, and the first argument it finds that doesn't start with a -
is treated as the command. This doesn't work for an implicit setup
command if any option with a space-separated argument exists. In this
case, the argument is incorrectly selected as the command, causing the
setup command options for the cross build to be excluded from the
command line, and thus a native build.

Improve the logic by just looking at the first argument. If it is
a known comand, then record it. Otherwise just assume it is the
implicit setup command.

Note that this fix does not address the possibility of a new meson
command. Two new echo statements are included to help the user in case
of trouble:

```
~/git/weston-imx$ meson --warnlevel 3 --prefix=/usr -Ddoc=false 
-Dbackend-drm-screencast-vaapi=false -Dcolor-management-lcms=false 
-Dpipewire=false -Dbackend-x11=false -Dxwayland=true -Dsimple-clients=all 
-Dbackend-wayland=false  -Dbackend-default=drm -Dbackend-rdp=false 
-Dtest-junit-xml=false -Dlauncher-libseat=false -Dimage-jpeg=false 
-Dimage-webp=false -Drenderer-g2d=true build
meson-wrapper: Implicit setup command assumed
meson-wrapper: Running meson with setup options: " 
--cross-file=/opt/fsl-imx-internal-xwayland/6.1-langdale/sysroots/x86_64-pokysdk-linux/usr/share/meson/aarch64-poky-linux-meson.cross
 
--native-file=/opt/fsl-imx-internal-xwayland/6.1-langdale/sysroots/x86_64-pokysdk-linux/usr/share/meson/meson.native
 "
The Meson build system
Version: 0.63.3
```

Signed-off-by: Tom Hochstein 
---
 meta/recipes-devtools/meson/meson/meson-wrapper | 17 -
 1 file changed, 8 insertions(+), 9 deletions(-)

diff --git a/meta/recipes-devtools/meson/meson/meson-wrapper 
b/meta/recipes-devtools/meson/meson/meson-wrapper
index fca64a5692..7455985297 100755
--- a/meta/recipes-devtools/meson/meson/meson-wrapper
+++ b/meta/recipes-devtools/meson/meson/meson-wrapper
@@ -13,20 +13,19 @@ fi
 # config is already in meson.cross.
 unset CC CXX CPP LD AR NM STRIP
 
-for arg in "$@"; do
-case "$arg" in
--*) continue ;;
-*) SUBCMD="$arg"; break ;;
-esac
-done
+case "$1" in
+setup|configure|dist|install|introspect|init|test|wrap|subprojects|rewrite|compile|devenv|env2mfile|help)
 MESON_CMD="$1" ;;
+*) echo meson-wrapper: Implicit setup command assumed; MESON_CMD=setup ;;
+esac
 
-if [ "$SUBCMD" = "setup" ] || [ -d "$SUBCMD" ]; then
-MESON_SUB_OPTS=" \
+if [ "$MESON_CMD" = "setup" ]; then
+MESON_SETUP_OPTS=" \
 
--cross-file="$OECORE_NATIVE_SYSROOT/usr/share/meson/${TARGET_PREFIX}meson.cross"
 \
 --native-file="$OECORE_NATIVE_SYSROOT/usr/share/meson/meson.native" \
 "
+echo meson-wrapper: Running meson with setup options: \"$MESON_SETUP_OPTS\"
 fi
 
 exec "$OECORE_NATIVE_SYSROOT/usr/bin/meson.real" \
 "$@" \
-$MESON_SUB_OPTS
+$MESON_SETUP_OPTS
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#178021): 
https://lists.openembedded.org/g/openembedded-core/message/178021
Mute This Topic: https://lists.openembedded.org/mt/97369067/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] [DISTRO|MACHINE]_FEATURES_BACKFILL_CONSIDERED: confusing variable names?

2023-02-28 Thread Tom Hochstein


> -Original Message-
> From: Richard Purdie 
> Sent: Tuesday, February 28, 2023 4:04 AM
> To: michael.opdenac...@bootlin.com; Tom Hochstein
> ; OE-core  c...@lists.openembedded.org>
> Cc: YP docs mailing list 
> Subject: Re: [OE-core] [DISTRO|MACHINE]_FEATURES_BACKFILL_CONSIDERED:
> confusing variable names?
> 
> On Tue, 2023-02-28 at 10:52 +0100, Michael Opdenacker via
> lists.openembedded.org wrote:
> > Greetings,
> >
> > Thank you all for your replies and comments!
> >
> > On 24.02.23 at 00:24, Tom Hochstein wrote:
> > > I agree. The problem I have with these variable names is that it's 
> > > ambiguous
> whether the features will be included or excluded from the backfill. All you 
> can
> tell from the names is that the values were 'considered'. What does that mean?
> You have to go to the manual to find the full meaning.
> > >
> > > What's important about the variables is that they will be 'excluded' from 
> > > the
> backfill, so this does look much clearer to me:
> > >
> > >   DISTRO_FEATURES_BACKFILL_EXCLUDED += "some-feature"
> >
> >
> > Richard, would you accept it if I sent a patch replacing
> > "BACKFILL_CONSIDERED" by "BACKFILL_EXCLUDED" in the two variables?
> >
> > Of course, I can understand if that's too late for the next release, and
> > instead document this oddity, at least before we can make a change again.
> 
> I don't want to rush into this. The idea of the variable was that the
> list were things that distro maintainers would need to consider when
> upgrading and then "mark as considered" so in that context, excluding
> them makes less sense. I'm sure there was discussion about the name at
> the time.

Not sure I understand. When you say 'idea of the variable', are you referring 
to the BACKFILL variable? I assume so since that seems to be the one that the 
distro maintainers will 'consider', right? After they've 'considered' the 
BACKFILL, then they will add the ones they _don't want_ to 
BACKFILL_CONSIDERED(_AND_NOT_WANTED). The ones they _do want_ are not marked.

> 
> I'm nervous about renaming variables in general as it does create pain
> for users upgrading and some community feedback has been that we're
> bordering on frustrating users. I know we have mechanisms now to help
> but it isn't pain free.

This I totally understand 

> 
> > > Is there a good reason why we have this separate variable and can't just 
> > > do
> this?
> > >
> > >   DISTRO_FEATURES_BACKFILL:remove = "some-feature"
> >
> >
> > Good question. Maybe because ":remove" is final, and cannot be
> > overridden, if I understood correctly?
> 
> Right, the concern would have been it couldn't be undone and the distro
> setups from when this dates from were more complex.

Since every variable is so afflicted by :remove, perhaps this isn't a good 
argument, particularly when there is a more general pattern for this: to add an 
extra variable at the point of the removal:

MY_DISTRO_FEATURES_BACKFILL_REMOVALS ??= "some-feature"
DISTRO_FEATURES_BACKFILL:remove = "${ MY_DISTRO_FEATURES_BACKFILL_REMOVALS}"

> 
> Cheers,
> 
> Richard
> 


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#177856): 
https://lists.openembedded.org/g/openembedded-core/message/177856
Mute This Topic: https://lists.openembedded.org/mt/97187180/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] [DISTRO|MACHINE]_FEATURES_BACKFILL_CONSIDERED: confusing variable names?

2023-02-23 Thread Tom Hochstein
I agree. The problem I have with these variable names is that it's ambiguous 
whether the features will be included or excluded from the backfill. All you 
can tell from the names is that the values were 'considered'. What does that 
mean? You have to go to the manual to find the full meaning.

What's important about the variables is that they will be 'excluded' from the 
backfill, so this does look much clearer to me:

DISTRO_FEATURES_BACKFILL_EXCLUDED += "some-feature"

Is there a good reason why we have this separate variable and can't just do 
this?

DISTRO_FEATURES_BACKFILL:remove = "some-feature"

Tom

-Original Message-
From: openembedded-core@lists.openembedded.org 
 On Behalf Of Michael Opdenacker via 
lists.openembedded.org
Sent: Thursday, February 23, 2023 11:00 AM
To: OE-core 
Cc: YP docs mailing list 
Subject: [OE-core] [DISTRO|MACHINE]_FEATURES_BACKFILL_CONSIDERED: confusing 
variable names?

Greetings

Anybody else finding the DISTRO_FEATURES_BACKFILL_CONSIDERED and 
MACHINE_FEATURES_BACKFILL_CONSIDERED variable names confusing?

According to the documentation about "feature backfilling" 
(https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.yoctoproject.org%2Fref-manual%2Ffeatures.html%23ref-features-backfill=05%7C01%7Ctom.hochstein%40nxp.com%7Cf4e5b1f4523a469612dc08db15bf5d8e%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C638127683879148065%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C=4i2IX0GpQlQXAh8Jausfvb2o1oF6MBZaU144KHKjcKg%3D=0),
 
these variables allow to exclude some features from being backfilled.

I'm not a native English speaker, but for me, "considered" carries the 
intention to include a feature, not to exclude one.

Shouldn't we call these variables DISTRO_FEATURES_BACKFILL_EXCLUDED and 
MACHINE_FEATURES_BACKFILL_EXCLUDED instead ?

Thanks in advance
Cheers
Michael.

-- 
Michael Opdenacker, Bootlin
Embedded Linux and Kernel engineering
https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fbootlin.com%2F=05%7C01%7Ctom.hochstein%40nxp.com%7Cf4e5b1f4523a469612dc08db15bf5d8e%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C638127683879148065%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C=S4rZtW35Uxk1ww8OkrPlCJdlI2FWgIpxHcq7PDq2M1o%3D=0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#177638): 
https://lists.openembedded.org/g/openembedded-core/message/177638
Mute This Topic: https://lists.openembedded.org/mt/97187180/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] [meta-oe][PATCH] nlohmann-json: Allow empty main package for SDK

2023-02-07 Thread Tom Hochstein


> -Original Message-
> From: Khem Raj 
> Sent: Monday, February 6, 2023 8:06 PM
> 
> > > -Original Message-
> > > From: Martin Jansa 
> > > Sent: Monday, February 6, 2023 3:13 PM
> > >
> > > Wrong ML and why do you want to install empty package? Add 
> > > nlohmann-json-dev to imx-gpu-sdk recipe instead.
> >
> > On Mon, Feb 6, 2023 at 1:36 PM Tom Hochstein  wrote:
> >
> > It has the advantage of not installing the header in the image. Should the 
> > -dev
> > solution be preferred?
> 
> if these are development headers and libs then yes its better since it
> goes with the general philosophy of putting these files in -dev pkgs.

This change does not change the -dev packaging.

To make my comment more clear, this change has the advantage of not installing 
the -dev packages in the rootfs, which is an unintended side-effect of Martin's 
suggestion to "add nlohmann-json-dev to imx-gpu-sdk recipe instead."


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#176831): 
https://lists.openembedded.org/g/openembedded-core/message/176831
Mute This Topic: https://lists.openembedded.org/mt/96793459/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] [meta-oe][PATCH] nlohmann-json: Allow empty main package for SDK

2023-02-06 Thread Tom Hochstein
Oops, I don't often send for non OE-Core and I forgot there was a separate 
mailing list for that. I usually just cut and paste from here:

http://www.openembedded.org/wiki/How_to_submit_a_patch_to_OpenEmbedded#Sending_patches

Which makes it easy to overlook the mailing list requirement.

There are several examples in the repo that use this same pattern. Here's one:

https://github.com/openembedded/meta-openembedded/commit/7163946b56539725d5a5868a9318e56e713a4a95

It has the advantage of not installing the header in the image. Should the -dev 
solution be preferred?

Tom

From: Martin Jansa 
Sent: Monday, February 6, 2023 3:13 PM
To: Tom Hochstein 
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [OE-core] [meta-oe][PATCH] nlohmann-json: Allow empty main package 
for SDK

Wrong ML and why do you want to install empty package? Add nlohmann-json-dev to 
imx-gpu-sdk recipe instead.

On Mon, Feb 6, 2023 at 10:09 PM Tom Hochstein 
mailto:tom.hochst...@nxp.com>> wrote:
The header-only package cannot be included in the SDK without marking
the main package with ALLOW_EMPTY.

Fixes rootfs problem:
```
The following packages have unmet dependencies:
 imx-gpu-sdk : Depends: nlohmann-json but it is not installable
E: Unable to correct problems, you have held broken packages.
```

Signed-off-by: Tom Hochstein 
mailto:tom.hochst...@nxp.com>>
---
 
meta-oe/recipes-devtools/nlohmann-json/nlohmann-json_3.11.2.bb<https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fnlohmann-json_3.11.2.bb%2F=05%7C01%7Ctom.hochstein%40nxp.com%7Ccd434fb892594f67b4df08db0886f900%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C638113148012774729%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C=vhZ3gJZfb%2FF4qX25hkN%2BQIA6jUK6jBFLgsIFAX6dQAQ%3D=0>
 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/meta-oe/recipes-devtools/nlohmann-json/nlohmann-json_3.11.2.bb<https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fnlohmann-json_3.11.2.bb%2F=05%7C01%7Ctom.hochstein%40nxp.com%7Ccd434fb892594f67b4df08db0886f900%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C638113148012774729%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C=vhZ3gJZfb%2FF4qX25hkN%2BQIA6jUK6jBFLgsIFAX6dQAQ%3D=0>
 
b/meta-oe/recipes-devtools/nlohmann-json/nlohmann-json_3.11.2.bb<https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fnlohmann-json_3.11.2.bb%2F=05%7C01%7Ctom.hochstein%40nxp.com%7Ccd434fb892594f67b4df08db0886f900%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C638113148012774729%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C=vhZ3gJZfb%2FF4qX25hkN%2BQIA6jUK6jBFLgsIFAX6dQAQ%3D=0>
index 502262820..6cf27755e 100644
--- 
a/meta-oe/recipes-devtools/nlohmann-json/nlohmann-json_3.11.2.bb<https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fnlohmann-json_3.11.2.bb%2F=05%7C01%7Ctom.hochstein%40nxp.com%7Ccd434fb892594f67b4df08db0886f900%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C638113148012774729%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C=vhZ3gJZfb%2FF4qX25hkN%2BQIA6jUK6jBFLgsIFAX6dQAQ%3D=0>
+++ 
b/meta-oe/recipes-devtools/nlohmann-json/nlohmann-json_3.11.2.bb<https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fnlohmann-json_3.11.2.bb%2F=05%7C01%7Ctom.hochstein%40nxp.com%7Ccd434fb892594f67b4df08db0886f900%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C638113148012930975%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C=svBLqUNpOl3rYXMGmHR6RfR9il6dkPxnxENyv6IUCME%3D=0>
@@ -18,7 +18,7 @@ inherit cmake
 EXTRA_OECMAKE += "-DJSON_BuildTests=OFF"

 # nlohmann-json is a header only C++ library, so the main package will be 
empty.
-
+ALLOW_EMPTY:${PN} = "1"
 RDEPENDS:${PN}-dev = ""

 BBCLASSEXTEND = "native nativesdk"
--
2.25.1




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



[OE-core] [meta-oe][PATCH] nlohmann-json: Allow empty main package for SDK

2023-02-06 Thread Tom Hochstein
The header-only package cannot be included in the SDK without marking
the main package with ALLOW_EMPTY.

Fixes rootfs problem:
```
The following packages have unmet dependencies:
 imx-gpu-sdk : Depends: nlohmann-json but it is not installable
E: Unable to correct problems, you have held broken packages.
```

Signed-off-by: Tom Hochstein 
---
 meta-oe/recipes-devtools/nlohmann-json/nlohmann-json_3.11.2.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta-oe/recipes-devtools/nlohmann-json/nlohmann-json_3.11.2.bb 
b/meta-oe/recipes-devtools/nlohmann-json/nlohmann-json_3.11.2.bb
index 502262820..6cf27755e 100644
--- a/meta-oe/recipes-devtools/nlohmann-json/nlohmann-json_3.11.2.bb
+++ b/meta-oe/recipes-devtools/nlohmann-json/nlohmann-json_3.11.2.bb
@@ -18,7 +18,7 @@ inherit cmake
 EXTRA_OECMAKE += "-DJSON_BuildTests=OFF"
 
 # nlohmann-json is a header only C++ library, so the main package will be 
empty.
-
+ALLOW_EMPTY:${PN} = "1"
 RDEPENDS:${PN}-dev = ""
 
 BBCLASSEXTEND = "native nativesdk"
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#176797): 
https://lists.openembedded.org/g/openembedded-core/message/176797
Mute This Topic: https://lists.openembedded.org/mt/96793459/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 v3] gstreamer1.0-plugins-good: Fix libsoup runtime dependency

2023-01-15 Thread Tom Hochstein
The runtime dependency on libsoup set via PACKAGECONFIG does not work.
The problem is the dependency is placed on the main package, but the
soup package has no dependency on the main package.

I considered modifying the call to do_split_packages from
gstreamer1.0-plugins-packaging.inc, changing extra_depends from '' to
None:

```
extra_depends  -- extra runtime dependencies (RDEPENDS) to be set for
  all packages. The default value of None causes a
  dependency on the main package (${PN}) - if you do
  not want this, pass '' for this parameter.
```

However, the problem with this solution is that it does add the runtime
dependency to _all_ such split packages.

So, fix the problem with an explicit runtime dependency.

Signed-off-by: Tom Hochstein 
---
 .../gstreamer/gstreamer1.0-plugins-good_1.20.5.bb  | 10 +++---
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.20.5.bb 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.20.5.bb
index f7ade89ef8..0bd953743e 100644
--- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.20.5.bb
+++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.20.5.bb
@@ -19,6 +19,7 @@ LIC_FILES_CHKSUM = 
"file://COPYING;md5=a6f89e2100d9b6cdffcea4f398e37343 \
 DEPENDS += "gstreamer1.0-plugins-base libcap zlib"
 RPROVIDES:${PN}-pulseaudio += "${PN}-pulse"
 RPROVIDES:${PN}-soup += "${PN}-souphttpsrc"
+RDEPENDS:${PN}-soup += "${MLPREFIX}${@bb.utils.contains('PACKAGECONFIG', 
'soup2', 'libsoup-2.4', 'libsoup', d)}"
 
 PACKAGECONFIG_SOUP ?= "soup3"
 
@@ -52,13 +53,8 @@ PACKAGECONFIG[libv4l2]= 
"-Dv4l2-libv4l2=enabled,-Dv4l2-libv4l2=disabled,v4l-
 PACKAGECONFIG[mpg123] = "-Dmpg123=enabled,-Dmpg123=disabled,mpg123"
 PACKAGECONFIG[pulseaudio] = "-Dpulse=enabled,-Dpulse=disabled,pulseaudio"
 PACKAGECONFIG[qt5]= "-Dqt5=enabled,-Dqt5=disabled,qtbase qtdeclarative 
qtbase-native ${QT5WAYLANDDEPENDS}"
-# Starting with version 1.20, the GStreamer soup plugin loads libsoup with 
dlopen()
-# instead of linking to it. And instead of using the default libsoup C 
headers, it
-# uses its own stub header. Consequently, objdump will not show the libsoup 
.so as
-# a dependency, and libsoup won't be added to an image. Fix this by setting 
libsoup
-# as RDEPEND.
-PACKAGECONFIG[soup2] = "-Dsoup=enabled,,libsoup-2.4,libsoup-2.4,,soup3"
-PACKAGECONFIG[soup3] = "-Dsoup=enabled,,libsoup,libsoup,,soup2"
+PACKAGECONFIG[soup2]  = "-Dsoup=enabled,,libsoup-2.4,,,soup3"
+PACKAGECONFIG[soup3]  = "-Dsoup=enabled,,libsoup,,,soup2"
 PACKAGECONFIG[speex]  = "-Dspeex=enabled,-Dspeex=disabled,speex"
 PACKAGECONFIG[rpi]= "-Drpicamsrc=enabled,-Drpicamsrc=disabled,userland"
 PACKAGECONFIG[taglib] = "-Dtaglib=enabled,-Dtaglib=disabled,taglib"
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#175889): 
https://lists.openembedded.org/g/openembedded-core/message/175889
Mute This Topic: https://lists.openembedded.org/mt/96288096/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] gstreamer1.0-plugins-good: Fix libsoup runtime dependency

2023-01-15 Thread Tom Hochstein
Okay, will this do the trick?

+RDEPENDS:${PN}-soup += "${MLPREFIX}${@bb.utils.contains('PACKAGECONFIG', 
'soup2', 'libsoup-2.4', 'libsoup', d)}"

Is this a general rule that explicit RDEPENDS require explicit MLPREFIX?

Tom

-Original Message-
From: Richard Purdie  
Sent: Sunday, January 15, 2023 9:09 AM
To: Tom Hochstein ; 
openembedded-core@lists.openembedded.org
Subject: Re: [OE-Core][PATCH v2] gstreamer1.0-plugins-good: Fix libsoup runtime 
dependency

On Sat, 2023-01-14 at 17:49 -0600, Tom Hochstein wrote:
> The runtime dependency on libsoup set via PACKAGECONFIG does not work.
> The problem is the dependency is placed on the main package, but the
> soup package has no dependency on the main package.
> 
> I considered modifying the call to do_split_packages from
> gstreamer1.0-plugins-packaging.inc, changing extra_depends from '' to
> None:
> 
> ```
> extra_depends  -- extra runtime dependencies (RDEPENDS) to be set for
>   all packages. The default value of None causes a
>   dependency on the main package (${PN}) - if you do
>   not want this, pass '' for this parameter.
> ```
> 
> However, the problem with this solution is that it does add the runtime
> dependency to _all_ such split packages.
> 
> So, fix the problem with an explicit runtime dependency.
> 
> Signed-off-by: Tom Hochstein 
> ---
>  .../gstreamer/gstreamer1.0-plugins-good_1.20.5.bb  | 10 +++---
>  1 file changed, 3 insertions(+), 7 deletions(-)

Better but there are still multilib issues:

https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fautobuilder.yoctoproject.org%2Ftyphoon%2F%23%2Fbuilders%2F44%2Fbuilds%2F6551=05%7C01%7Ctom.hochstein%40nxp.com%7C2519f1815a0e4f78782c08daf70a6dc4%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C638093921412421859%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C=tr%2BSVqrVpQ1zPOMB8bDURNFWyurODBuk8SZk0o%2BOvCo%3D=0
https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fautobuilder.yoctoproject.org%2Ftyphoon%2F%23%2Fbuilders%2F52%2Fbuilds%2F6425=05%7C01%7Ctom.hochstein%40nxp.com%7C2519f1815a0e4f78782c08daf70a6dc4%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C638093921412421859%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C=dE5w5xWhVoVARs9hl9FGrzL1ckK2N9ARJ84hmdAuqOo%3D=0

Cheers,

Richard

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#175887): 
https://lists.openembedded.org/g/openembedded-core/message/175887
Mute This Topic: https://lists.openembedded.org/mt/96277460/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] gstreamer1.0-plugins-good: Fix libsoup runtime dependency

2023-01-14 Thread Tom Hochstein
The runtime dependency on libsoup set via PACKAGECONFIG does not work.
The problem is the dependency is placed on the main package, but the
soup package has no dependency on the main package.

I considered modifying the call to do_split_packages from
gstreamer1.0-plugins-packaging.inc, changing extra_depends from '' to
None:

```
extra_depends  -- extra runtime dependencies (RDEPENDS) to be set for
  all packages. The default value of None causes a
  dependency on the main package (${PN}) - if you do
  not want this, pass '' for this parameter.
```

However, the problem with this solution is that it does add the runtime
dependency to _all_ such split packages.

So, fix the problem with an explicit runtime dependency.

Signed-off-by: Tom Hochstein 
---
 .../gstreamer/gstreamer1.0-plugins-good_1.20.5.bb  | 10 +++---
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.20.5.bb 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.20.5.bb
index f7ade89ef8..ce2a796928 100644
--- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.20.5.bb
+++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.20.5.bb
@@ -19,6 +19,7 @@ LIC_FILES_CHKSUM = 
"file://COPYING;md5=a6f89e2100d9b6cdffcea4f398e37343 \
 DEPENDS += "gstreamer1.0-plugins-base libcap zlib"
 RPROVIDES:${PN}-pulseaudio += "${PN}-pulse"
 RPROVIDES:${PN}-soup += "${PN}-souphttpsrc"
+RDEPENDS:${PN}-soup += "${@bb.utils.contains('PACKAGECONFIG', 'soup2', 
'libsoup-2.4', 'libsoup', d)}"
 
 PACKAGECONFIG_SOUP ?= "soup3"
 
@@ -52,13 +53,8 @@ PACKAGECONFIG[libv4l2]= 
"-Dv4l2-libv4l2=enabled,-Dv4l2-libv4l2=disabled,v4l-
 PACKAGECONFIG[mpg123] = "-Dmpg123=enabled,-Dmpg123=disabled,mpg123"
 PACKAGECONFIG[pulseaudio] = "-Dpulse=enabled,-Dpulse=disabled,pulseaudio"
 PACKAGECONFIG[qt5]= "-Dqt5=enabled,-Dqt5=disabled,qtbase qtdeclarative 
qtbase-native ${QT5WAYLANDDEPENDS}"
-# Starting with version 1.20, the GStreamer soup plugin loads libsoup with 
dlopen()
-# instead of linking to it. And instead of using the default libsoup C 
headers, it
-# uses its own stub header. Consequently, objdump will not show the libsoup 
.so as
-# a dependency, and libsoup won't be added to an image. Fix this by setting 
libsoup
-# as RDEPEND.
-PACKAGECONFIG[soup2] = "-Dsoup=enabled,,libsoup-2.4,libsoup-2.4,,soup3"
-PACKAGECONFIG[soup3] = "-Dsoup=enabled,,libsoup,libsoup,,soup2"
+PACKAGECONFIG[soup2]  = "-Dsoup=enabled,,libsoup-2.4,,,soup3"
+PACKAGECONFIG[soup3]  = "-Dsoup=enabled,,libsoup,,,soup2"
 PACKAGECONFIG[speex]  = "-Dspeex=enabled,-Dspeex=disabled,speex"
 PACKAGECONFIG[rpi]= "-Drpicamsrc=enabled,-Drpicamsrc=disabled,userland"
 PACKAGECONFIG[taglib] = "-Dtaglib=enabled,-Dtaglib=disabled,taglib"
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#175860): 
https://lists.openembedded.org/g/openembedded-core/message/175860
Mute This Topic: https://lists.openembedded.org/mt/96277460/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] gstreamer1.0-plugins-good: Fix libsoup runtime dependency

2023-01-14 Thread Tom Hochstein
Thanks. An earlier commit [1] did add libsoup back to the DEPENDS list, though 
it was not mentioned in the commit message, and I overlooked it in the code 
what with the dropping of the -Dsoup=disabled.

[1] 
https://github.com/openembedded/openembedded-core/commit/88d30848981db2cb7b1f2021ba07e40976cd789f

V2 coming.

-Original Message-
From: Richard Purdie  
Sent: Saturday, January 14, 2023 11:02 AM
To: Tom Hochstein ; 
openembedded-core@lists.openembedded.org
Subject: Re: [OE-core] [PATCH] gstreamer1.0-plugins-good: Fix libsoup runtime 
dependency

On Fri, 2023-01-13 at 16:51 -0600, Tom Hochstein wrote:
> The runtime dependency on libsoup set via PACKAGECONFIG does not work.
> The problem is the dependency is placed on the main package, but the
> soup package has no dependency on the main package.
> 
> I considered modifying the call to do_split_packages from
> gstreamer1.0-plugins-packaging.inc, changing extra_depends from '' to
> None:
> 
> ```
> extra_depends  -- extra runtime dependencies (RDEPENDS) to be set for
>   all packages. The default value of None causes a
>   dependency on the main package (${PN}) - if you do
>   not want this, pass '' for this parameter.
> ```
> 
> However, the problem with this solution is that it does add the runtime
> dependency to _all_ such split packages.
> 
> So, fix the problem with an explicit runtime dependency.
> 
> Signed-off-by: Tom Hochstein 
> ---
>  .../gstreamer/gstreamer1.0-plugins-good_1.20.5.bb  | 10 +++---
>  1 file changed, 3 insertions(+), 7 deletions(-)

This threw a load of warnings/errors, see:

https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fautobuilder.yoctoproject.org%2Ftyphoon%2F%23%2Fbuilders%2F83%2Fbuilds%2F4792=05%7C01%7Ctom.hochstein%40nxp.com%7C2a738415ebef4c494dd108daf6511d1f%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C638093125482952300%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C=cfdxmtUBAwPNqIcKtqSxdABCkPq6O6S42ZEkn1CrDAg%3D=0

Cheers,

Richard

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#175859): 
https://lists.openembedded.org/g/openembedded-core/message/175859
Mute This Topic: https://lists.openembedded.org/mt/96257892/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] gstreamer1.0-plugins-good: Fix libsoup runtime dependency

2023-01-13 Thread Tom Hochstein
The runtime dependency on libsoup set via PACKAGECONFIG does not work.
The problem is the dependency is placed on the main package, but the
soup package has no dependency on the main package.

I considered modifying the call to do_split_packages from
gstreamer1.0-plugins-packaging.inc, changing extra_depends from '' to
None:

```
extra_depends  -- extra runtime dependencies (RDEPENDS) to be set for
  all packages. The default value of None causes a
  dependency on the main package (${PN}) - if you do
  not want this, pass '' for this parameter.
```

However, the problem with this solution is that it does add the runtime
dependency to _all_ such split packages.

So, fix the problem with an explicit runtime dependency.

Signed-off-by: Tom Hochstein 
---
 .../gstreamer/gstreamer1.0-plugins-good_1.20.5.bb  | 10 +++---
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.20.5.bb 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.20.5.bb
index f7ade89ef8..ed48c57311 100644
--- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.20.5.bb
+++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.20.5.bb
@@ -19,6 +19,7 @@ LIC_FILES_CHKSUM = 
"file://COPYING;md5=a6f89e2100d9b6cdffcea4f398e37343 \
 DEPENDS += "gstreamer1.0-plugins-base libcap zlib"
 RPROVIDES:${PN}-pulseaudio += "${PN}-pulse"
 RPROVIDES:${PN}-soup += "${PN}-souphttpsrc"
+RDEPENDS:${PN}-soup += "${@bb.utils.contains('PACKAGECONFIG', 'soup2', 
'libsoup-2.4', 'libsoup', d)}"
 
 PACKAGECONFIG_SOUP ?= "soup3"
 
@@ -52,13 +53,8 @@ PACKAGECONFIG[libv4l2]= 
"-Dv4l2-libv4l2=enabled,-Dv4l2-libv4l2=disabled,v4l-
 PACKAGECONFIG[mpg123] = "-Dmpg123=enabled,-Dmpg123=disabled,mpg123"
 PACKAGECONFIG[pulseaudio] = "-Dpulse=enabled,-Dpulse=disabled,pulseaudio"
 PACKAGECONFIG[qt5]= "-Dqt5=enabled,-Dqt5=disabled,qtbase qtdeclarative 
qtbase-native ${QT5WAYLANDDEPENDS}"
-# Starting with version 1.20, the GStreamer soup plugin loads libsoup with 
dlopen()
-# instead of linking to it. And instead of using the default libsoup C 
headers, it
-# uses its own stub header. Consequently, objdump will not show the libsoup 
.so as
-# a dependency, and libsoup won't be added to an image. Fix this by setting 
libsoup
-# as RDEPEND.
-PACKAGECONFIG[soup2] = "-Dsoup=enabled,,libsoup-2.4,libsoup-2.4,,soup3"
-PACKAGECONFIG[soup3] = "-Dsoup=enabled,,libsoup,libsoup,,soup2"
+PACKAGECONFIG[soup2]  = "-Dsoup=enabled,soup3"
+PACKAGECONFIG[soup3]  = "-Dsoup=enabled,soup2"
 PACKAGECONFIG[speex]  = "-Dspeex=enabled,-Dspeex=disabled,speex"
 PACKAGECONFIG[rpi]= "-Drpicamsrc=enabled,-Drpicamsrc=disabled,userland"
 PACKAGECONFIG[taglib] = "-Dtaglib=enabled,-Dtaglib=disabled,taglib"
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#175847): 
https://lists.openembedded.org/g/openembedded-core/message/175847
Mute This Topic: https://lists.openembedded.org/mt/96257892/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] vulkan-loader: Move libvulkan.so to main package

2022-10-18 Thread Tom Hochstein
Got it, thank you Ross!

-Original Message-
From: Ross Burton  
Sent: Tuesday, October 18, 2022 4:52 AM
To: Tom Hochstein 
Cc: Martin Jansa ; 
openembedded-core@lists.openembedded.org
Subject: Re: [OE-core] [PATCH] vulkan-loader: Move libvulkan.so to main package

On 17 Oct 2022, at 17:12, Tom Hochstein  wrote:
> 
> I found a few examples online for both ways. In SaschaWillems examples, .so 
> is used:
>  
> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FSaschaWillems%2FVulkan%2Fblob%2Fmaster%2Fbase%2FVulkanAndroid.cpp%23L142data=05%7C01%7Ctom.hochstein%40nxp.com%7C9c135c0d4b16464e586208dab0ee669d%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C638016835220670988%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7Csdata=D5UDy3Fd%2F%2Fq8Hby31cqj4vxn2g8anMNBTZJJMbBdVHo%3Dreserved=0
>  
> In Vulkan-Tools there's a mix. Here the implementation falls back to .so.1 if 
> .so is not found:
>  
> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FKhronosGroup%2FVulkan-Tools%2Fblob%2Fmaster%2Fvulkaninfo%2Fvulkaninfo.h%23L256-L257data=05%7C01%7Ctom.hochstein%40nxp.com%7C9c135c0d4b16464e586208dab0ee669d%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C638016835220670988%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7Csdata=fNOFnmZUbDGZAXGgU5Cq5N4cKmRBgLA7Od7FXcGXpLY%3Dreserved=0
>  
> While here only .so is used:
>  
> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FKhronosGroup%2FVulkan-Tools%2Fblob%2Fmaster%2Fcommon%2Fvulkan_wrapper.cpp%23L25data=05%7C01%7Ctom.hochstein%40nxp.com%7C9c135c0d4b16464e586208dab0ee669d%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C638016835220827298%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7Csdata=mlM7Mr%2FEt9LTalhmHo5Js1UliBABxqpkXHC4V8KfUUk%3Dreserved=0
> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FKhronosGroup%2FVulkan-Tools%2Fblob%2Fmaster%2Fscripts%2Fgenerate_vulkan_wrapper.py%23L2043data=05%7C01%7Ctom.hochstein%40nxp.com%7C9c135c0d4b16464e586208dab0ee669d%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C638016835220827298%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7Csdata=PqV8hitdAseUDE4BnKeqvwOT0B9iQ03h77LCzJbxLj8%3Dreserved=0
>  
> With no consistency or guidance I'm not sure what the right solution is. At 
> first I thought perhaps it is better for the caller to be more specific so 
> that a compatibility problem is detected with a dlopen failure. Complicating 
> that, though, is that vulkan-loader 1.2 is not compatible with 1.3. Plus why 
> does Vulkan-Tools use .so.1 second? Probably to get around the fact that the 
> .so is in the dev package?

I did some research.

libvulkan.so* is the Vulkan Loader, which is responsible for finding the 
correct driver and bridging the application with the driver.

As per the documentation at 
https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fvulkan.lunarg.com%2Fdoc%2Fsdk%2F1.3.224.1%2Flinux%2FLoaderApplicationInterface.htmldata=05%7C01%7Ctom.hochstein%40nxp.com%7C9c135c0d4b16464e586208dab0ee669d%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C638016835220827298%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7Csdata=mZURlOcefEkwdI0VUUUi8i3M8zdvZGXVNsg6I75VfyU%3Dreserved=0,
 you can either directly dynamically link (-lvulkan to the compiler) to the 
loader , or indirectly link (dlopen()) to the loader.  Quoting from the ABI 
Versioning section:

"""
On Linux, applications that have a hard dependency on Vulkan should request 
linking to the unversioned name libvulkan.so in their build system. For example 
by importing the CMake target Vulkan::Vulkan or by using the output of 
pkg-config --cflags --libs vulkan as compiler flags. As usual for Linux 
libraries, the compiler and linker will resolve this to a dependency on the 
correct versioned SONAME, currently libvulkan.so.1. Linux applications that 
load Vulkan-Loader dynamically at runtime do not benefit from this mechanism, 
and should instead make sure to pass the versioned name such as libvulkan.so.1 
to dlopen(), to ensure that they load a compatible version.
"""

Applications which dlopen("libvulkan.so") without a version are broken and 
should be fixed.  The vulkan tools doing exactly what the documentation says 
not to do is just embarrassing to be honest...

So my position is that we don't mess around with the vulkan-loader recipe, but 
instead fix the callers.

Ross

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#171939): 
https://lists.openembedded.org/g/openembedded-core/message/171939
Mute This Topic: ht

Re: [OE-core] [PATCH] vulkan-loader: Move libvulkan.so to main package

2022-10-17 Thread Tom Hochstein
I found a few examples online for both ways. In SaschaWillems examples, .so is 
used:

https://github.com/SaschaWillems/Vulkan/blob/master/base/VulkanAndroid.cpp#L142

In Vulkan-Tools there's a mix. Here the implementation falls back to .so.1 if 
.so is not found:

https://github.com/KhronosGroup/Vulkan-Tools/blob/master/vulkaninfo/vulkaninfo.h#L256-L257

While here only .so is used:

https://github.com/KhronosGroup/Vulkan-Tools/blob/master/common/vulkan_wrapper.cpp#L25
https://github.com/KhronosGroup/Vulkan-Tools/blob/master/scripts/generate_vulkan_wrapper.py#L2043

With no consistency or guidance I'm not sure what the right solution is. At 
first I thought perhaps it is better for the caller to be more specific so that 
a compatibility problem is detected with a dlopen failure. Complicating that, 
though, is that vulkan-loader 1.2 is not compatible with 1.3. Plus why does 
Vulkan-Tools use .so.1 second? Probably to get around the fact that the .so is 
in the dev package?

Is it possible and/or reasonable for a binary to load the .so then check the 
version and handle that at runtime? If so, then including the .so in the main 
package makes sense. Or should the user in this case be expected to use 
RDEPENDS to install the .so?

From: Martin Jansa 
Sent: Monday, October 17, 2022 7:29 AM
To: Ross Burton 
Cc: Tom Hochstein ; 
openembedded-core@lists.openembedded.org
Subject: Re: [OE-core] [PATCH] vulkan-loader: Move libvulkan.so to main package


On Mon, Oct 17, 2022 at 2:10 PM Ross Burton 
mailto:ross.bur...@arm.com>> wrote:
On 16 Oct 2022, at 22:43, Tom Hochstein via 
lists.openembedded.org<https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Flists.openembedded.org%2F=05%7C01%7Ctom.hochstein%40nxp.com%7Cb79ae30157f142e2468e08dab03b378b%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C638016065621027269%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C=ldUoyzxYOQMI33kzqUh9CJ15MGI2n0YRf8rwKopwkJw%3D=0>
 
mailto:nxp@lists.openembedded.org>>
 wrote:
>
> Since libvulkan.so is commonly loaded dynamically, put it in the main
> package.

Can you give an example?

I'm wondering the same, because e.g.  tensorflow-lite. arm-compute-library were 
also trying to dlopen libOpenCL.so and we've patched them to dlopen 
libOpenCL.so.1 instead, so why should libvulkan.so be different.

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#171915): 
https://lists.openembedded.org/g/openembedded-core/message/171915
Mute This Topic: https://lists.openembedded.org/mt/94372555/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] vulkan-loader: Move libvulkan.so to main package

2022-10-16 Thread Tom Hochstein
Since libvulkan.so is commonly loaded dynamically, put it in the main
package.

Signed-off-by: Tom Hochstein 
---
 meta/recipes-graphics/vulkan/vulkan-loader_1.3.216.0.bb | 5 +
 1 file changed, 5 insertions(+)

diff --git a/meta/recipes-graphics/vulkan/vulkan-loader_1.3.216.0.bb 
b/meta/recipes-graphics/vulkan/vulkan-loader_1.3.216.0.bb
index 15a079879b..698ce112dc 100644
--- a/meta/recipes-graphics/vulkan/vulkan-loader_1.3.216.0.bb
+++ b/meta/recipes-graphics/vulkan/vulkan-loader_1.3.216.0.bb
@@ -35,6 +35,11 @@ PACKAGECONFIG ??= "${@bb.utils.filter('DISTRO_FEATURES', 
'wayland x11', d)}"
 PACKAGECONFIG[x11] = "-DBUILD_WSI_XLIB_SUPPORT=ON -DBUILD_WSI_XCB_SUPPORT=ON, 
-DBUILD_WSI_XLIB_SUPPORT=OFF -DBUILD_WSI_XCB_SUPPORT=OFF, libxcb libx11 
libxrandr"
 PACKAGECONFIG[wayland] = "-DBUILD_WSI_WAYLAND_SUPPORT=ON, 
-DBUILD_WSI_WAYLAND_SUPPORT=OFF, wayland"
 
+# libvulkan.so is commonly loaded dynamically, so put it in the main package
+SOLIBS = ".so*"
+FILES_SOLIBSDEV = ""
+INSANE_SKIP:${PN} += "dev-so"
+
 RRECOMMENDS:${PN} = "mesa-vulkan-drivers"
 
 UPSTREAM_CHECK_GITTAGREGEX = "sdk-(?P\d+(\.\d+)+)"
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#171900): 
https://lists.openembedded.org/g/openembedded-core/message/171900
Mute This Topic: https://lists.openembedded.org/mt/94372555/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] cmake: Fix CMAKE_SYSTEM_PROCESSOR setting for SDK

2022-09-07 Thread Tom Hochstein
When building using an SDK, cmake complains that the target
architecture 'cortexa53-crypto' is unknown. The same build in bitbake
uses the target architecture 'aarch64'.

Set CMAKE_SYSTEM_PROCESSOR the same as for bitbake.

Signed-off-by: Tom Hochstein 
---
 meta/recipes-devtools/cmake/cmake/OEToolchainConfig.cmake | 5 +
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/meta/recipes-devtools/cmake/cmake/OEToolchainConfig.cmake 
b/meta/recipes-devtools/cmake/cmake/OEToolchainConfig.cmake
index 3ddef12c83..d6a1e0464c 100644
--- a/meta/recipes-devtools/cmake/cmake/OEToolchainConfig.cmake
+++ b/meta/recipes-devtools/cmake/cmake/OEToolchainConfig.cmake
@@ -11,10 +11,7 @@ set( CMAKE_FIND_ROOT_PATH_MODE_PACKAGE ONLY )
 
 set(CMAKE_FIND_LIBRARY_CUSTOM_LIB_SUFFIX 
"$ENV{OE_CMAKE_FIND_LIBRARY_CUSTOM_LIB_SUFFIX}")
 
-# Set CMAKE_SYSTEM_PROCESSOR from the sysroot name (assuming 
processor-distro-os).
-if ($ENV{SDKTARGETSYSROOT} MATCHES "/sysroots/([a-zA-Z0-9_-]+)-.+-.+")
-  set(CMAKE_SYSTEM_PROCESSOR ${CMAKE_MATCH_1})
-endif()
+set( CMAKE_SYSTEM_PROCESSOR $ENV{OECORE_TARGET_ARCH} )
 
 # Include the toolchain configuration subscripts
 file( GLOB toolchain_config_files "${CMAKE_CURRENT_LIST_FILE}.d/*.cmake" )
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#170432): 
https://lists.openembedded.org/g/openembedded-core/message/170432
Mute This Topic: https://lists.openembedded.org/mt/93536574/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] connman: Drop redundant nfsroot handling

2022-09-07 Thread Tom Hochstein
connman has nfsroot support built in since version 1.34 [1], so the
nfsroot handling in the init script is redundant.

[1] 
https://git.kernel.org/pub/scm/network/connman/connman.git/commit/?id=ef0d26e6ef2b883193469f016117d8238c1c9658

Signed-off-by: Tom Hochstein 
---
 .../connman/connman/connman   | 39 +--
 1 file changed, 1 insertion(+), 38 deletions(-)

diff --git a/meta/recipes-connectivity/connman/connman/connman 
b/meta/recipes-connectivity/connman/connman/connman
index 310a696863..a021fd4655 100644
--- a/meta/recipes-connectivity/connman/connman/connman
+++ b/meta/recipes-connectivity/connman/connman/connman
@@ -10,48 +10,11 @@ fi
 
 set -e
 
-nfsroot=0
-
-exec 9<&0 < /proc/mounts
-while read dev mtpt fstype rest; do
-   if test $mtpt = "/" ; then
-   case $fstype in
-   nfs | nfs4)
-   nfsroot=1
-   break
-   ;;
-   *)
-   ;;
-   esac
-   fi
-done
-
 do_start() {
-   if test $nfsroot -eq 1 ; then
-   NET_DEVS=`cat /proc/net/dev | sed -ne 's/^\([a-zA-Z0-9 
]*\):.*$/\1/p'`
-   NET_ADDR=`cat /proc/cmdline | sed -ne 's/^.*ip=\([^ :]*\).*$/\1/p'`
-
-   if [ ! -z "$NET_ADDR" ]; then
-   if [ "$NET_ADDR" = dhcp ]; then
-   ethn=`ifconfig | grep "^eth" | sed -e 
"s/\(eth[0-9]\)\(.*\)/\1/"`
-   if [ ! -z "$ethn" ]; then
-   EXTRA_PARAM="$EXTRA_PARAM -I $ethn"
-   fi
-   else
-   for i in $NET_DEVS; do
-   ADDR=`ifconfig $i | sed 's/addr://g' | sed -ne 
's/^.*inet \([0-9.]*\) .*$/\1/p'`
-   if [ "$NET_ADDR" = "$ADDR" ]; then
-   EXTRA_PARAM="$EXTRA_PARAM -I $i"
-   break
-   fi
-   done
-   fi
-   fi
-   fi
if [ -f @DATADIR@/connman/wired-setup ] ; then
. @DATADIR@/connman/wired-setup
fi
-   $DAEMON $EXTRA_PARAM
+   $DAEMON
 }
 
 do_stop() {
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#170425): 
https://lists.openembedded.org/g/openembedded-core/message/170425
Mute This Topic: https://lists.openembedded.org/mt/93531638/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] piglit: Add PACKAGECONFIG for glx and opencl

2022-08-22 Thread Tom Hochstein
- Allow GLX tests to be disabled for systems that don't support it.

- Allow OpenCL tests to be enabled.

Signed-off-by: Tom Hochstein 
---
 ...roper-WAYLAND_INCLUDE_DIRS-variable.patch} |  0
 ...-shader.c-do-not-hardcode-build-pat.patch} |  0
 ...ists.txt-add-missing-endian.h-check.patch} |  0
 ...n-t-enable-GLX-if-tests-are-disabled.patch | 32 +++
 meta/recipes-graphics/piglit/piglit_git.bb| 12 ---
 5 files changed, 39 insertions(+), 5 deletions(-)
 rename 
meta/recipes-graphics/piglit/piglit/{0001-cmake-use-proper-WAYLAND_INCLUDE_DIRS-variable.patch
 => 0002-cmake-use-proper-WAYLAND_INCLUDE_DIRS-variable.patch} (100%)
 rename 
meta/recipes-graphics/piglit/piglit/{0002-tests-util-piglit-shader.c-do-not-hardcode-build-pat.patch
 => 0003-tests-util-piglit-shader.c-do-not-hardcode-build-pat.patch} (100%)
 rename 
meta/recipes-graphics/piglit/piglit/{0001-CMakeLists.txt-add-missing-endian.h-check.patch
 => 0004-CMakeLists.txt-add-missing-endian.h-check.patch} (100%)
 create mode 100644 
meta/recipes-graphics/piglit/piglit/0005-cmake-Don-t-enable-GLX-if-tests-are-disabled.patch

diff --git 
a/meta/recipes-graphics/piglit/piglit/0001-cmake-use-proper-WAYLAND_INCLUDE_DIRS-variable.patch
 
b/meta/recipes-graphics/piglit/piglit/0002-cmake-use-proper-WAYLAND_INCLUDE_DIRS-variable.patch
similarity index 100%
rename from 
meta/recipes-graphics/piglit/piglit/0001-cmake-use-proper-WAYLAND_INCLUDE_DIRS-variable.patch
rename to 
meta/recipes-graphics/piglit/piglit/0002-cmake-use-proper-WAYLAND_INCLUDE_DIRS-variable.patch
diff --git 
a/meta/recipes-graphics/piglit/piglit/0002-tests-util-piglit-shader.c-do-not-hardcode-build-pat.patch
 
b/meta/recipes-graphics/piglit/piglit/0003-tests-util-piglit-shader.c-do-not-hardcode-build-pat.patch
similarity index 100%
rename from 
meta/recipes-graphics/piglit/piglit/0002-tests-util-piglit-shader.c-do-not-hardcode-build-pat.patch
rename to 
meta/recipes-graphics/piglit/piglit/0003-tests-util-piglit-shader.c-do-not-hardcode-build-pat.patch
diff --git 
a/meta/recipes-graphics/piglit/piglit/0001-CMakeLists.txt-add-missing-endian.h-check.patch
 
b/meta/recipes-graphics/piglit/piglit/0004-CMakeLists.txt-add-missing-endian.h-check.patch
similarity index 100%
rename from 
meta/recipes-graphics/piglit/piglit/0001-CMakeLists.txt-add-missing-endian.h-check.patch
rename to 
meta/recipes-graphics/piglit/piglit/0004-CMakeLists.txt-add-missing-endian.h-check.patch
diff --git 
a/meta/recipes-graphics/piglit/piglit/0005-cmake-Don-t-enable-GLX-if-tests-are-disabled.patch
 
b/meta/recipes-graphics/piglit/piglit/0005-cmake-Don-t-enable-GLX-if-tests-are-disabled.patch
new file mode 100644
index 00..ef6fda0f4e
--- /dev/null
+++ 
b/meta/recipes-graphics/piglit/piglit/0005-cmake-Don-t-enable-GLX-if-tests-are-disabled.patch
@@ -0,0 +1,32 @@
+From 13ff43fe760ac343b33d8e8c84b89886aac07116 Mon Sep 17 00:00:00 2001
+From: Tom Hochstein 
+Date: Fri, 3 Jun 2022 10:44:29 -0500
+Subject: [PATCH] cmake: Don't enable GLX if tests are disabled
+
+Allow building for systems that don't support GLX.
+
+Upstream-Status: Submitted 
[https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/720]
+Signed-off-by: Tom Hochstein 
+---
+ CMakeLists.txt | 5 +
+ 1 file changed, 1 insertion(+), 4 deletions(-)
+
+diff --git a/CMakeLists.txt b/CMakeLists.txt
+index e1aeb5ddf..85e171aba 100644
+--- a/CMakeLists.txt
 b/CMakeLists.txt
+@@ -134,10 +134,7 @@ if(PIGLIT_BUILD_CL_TESTS)
+ endif(PIGLIT_BUILD_CL_TESTS)
+ 
+ IF(${CMAKE_SYSTEM_NAME} MATCHES "Linux")
+-  if(X11_FOUND AND OPENGL_gl_LIBRARY)
+-  # Assume the system has GLX. In the future, systems may exist
+-  # with libGL and libX11 but no GLX, but that world hasn't
+-  # arrived yet.
++  if(X11_FOUND AND OPENGL_gl_LIBRARY AND PIGLIT_BUILD_GLX_TESTS)
+   set(PIGLIT_HAS_GLX True)
+   add_definitions(-DPIGLIT_HAS_GLX)
+   endif()
+-- 
+2.17.1
+
diff --git a/meta/recipes-graphics/piglit/piglit_git.bb 
b/meta/recipes-graphics/piglit/piglit_git.bb
index 7c1bc89c22..29360a246b 100644
--- a/meta/recipes-graphics/piglit/piglit_git.bb
+++ b/meta/recipes-graphics/piglit/piglit_git.bb
@@ -8,10 +8,10 @@ LIC_FILES_CHKSUM = 
"file://COPYING;md5=b2beded7103a3d8a442a2a0391d607b0"
 
 SRC_URI = 
"git://gitlab.freedesktop.org/mesa/piglit.git;protocol=https;branch=main \
file://0001-cmake-install-bash-completions-in-the-right-place.patch 
\
-   file://0001-cmake-use-proper-WAYLAND_INCLUDE_DIRS-variable.patch \
-   
file://0002-tests-util-piglit-shader.c-do-not-hardcode-build-pat.patch \
-   file://0001-CMakeLists.txt-add-missing-endian.h-check.patch \
-   "
+   file://0002-cmake-use-proper-WAYLAND_INCLUDE_DIRS-variable.patch \
+   
file://0003-tests-util-piglit-shader.c-do-not-hardcode-build-pat.patch \
+   file://0004-CMakeLists.txt-add-missing-endian.h-check.patch \
+  

[OE-core] [PATCH] uboot-config.bbclass: Raise error for bad key

2022-07-28 Thread Tom Hochstein
If an invalid key is used, the class ignores the error, with an
indeterminate result. In my case, the problem surfaced in do_deploy:

```
| cp: cannot stat 
'/.../build/tmp/work/imx6qdlsabresd-fsl-linux-gnueabi/u-boot-imx-mfgtool/2022.04-r0/deploy-u-boot-imx-mfgtool/u-boot.imx':
 No such file or
| directory
```

The root cause of this was that the uboot config key did not match
a valid option. With the fix, the error is caught by the class:

```
ERROR: Nothing PROVIDES 'u-boot-imx-mfgtool'
u-boot-imx-mfgtool was skipped: The selected UBOOT_CONFIG key ['mfgtool'] has 
no match in dict_keys(['sd-fslc', 'sd-imx', 'sd-optee-imx', 'sata-imx', 
'mfgtool-imx']).
```

Signed-off-by: Tom Hochstein 
---
 meta/classes/uboot-config.bbclass | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/classes/uboot-config.bbclass 
b/meta/classes/uboot-config.bbclass
index b9ad35821a..e8da8c7452 100644
--- a/meta/classes/uboot-config.bbclass
+++ b/meta/classes/uboot-config.bbclass
@@ -125,5 +125,6 @@ python () {
 else:
 bb.debug(1, "Appending '%s' to UBOOT_BINARIES." % 
ubootbinary)
 d.appendVar('UBOOT_BINARIES', ' ' + ubootbinary)
-break
+return
+raise bb.parse.SkipRecipe("The selected UBOOT_CONFIG key %s has no 
match in %s." % (ubootconfig, ubootconfigflags.keys()))
 }
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#168614): 
https://lists.openembedded.org/g/openembedded-core/message/168614
Mute This Topic: https://lists.openembedded.org/mt/92676883/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] gobject-introspection-data: Disable cache for g-ir-scanner

2022-07-18 Thread Tom Hochstein
An intermittent failure occurs in libical-native do_compile:

| Traceback (most recent call last):
|   File 
"/.../build/tmp/work/x86_64-linux/libical-native/3.0.14-r0/recipe-sysroot-native/usr/lib/python3.10/shutil.py",
 line 813, in move
| os.rename(src, real_dst)
| OSError: [Errno 18] Invalid cross-device link: 
'/tmp/g-ir-scanner-cache-adxo_2bq' -> 
'/home/bamboo/.cache/g-ir-scanner/bab9a83d2cd93e62ed005a2c1d4f89ae75c67251'
|
| During handling of the above exception, another exception occurred:
|
| Traceback (most recent call last):
|   File 
"/.../tmp/work/x86_64-linux/libical-native/3.0.14-r0/recipe-sysroot-native/usr/lib/pkgconfig/../../../usr/bin/g-ir-scanner",
 line 99, in 
| sys.exit(scanner_main(sys.argv))
|   File 
"/.../tmp/work/x86_64-linux/libical-native/3.0.14-r0/recipe-sysroot-native/usr/lib/gobject-introspection/giscanner/scannermain.py",
 line 590, in scanner_main
| transformer = create_transformer(namespace, options)
|   File 
"/.../tmp/work/x86_64-linux/libical-native/3.0.14-r0/recipe-sysroot-native/usr/lib/gobject-introspection/giscanner/scannermain.py",
 line 409, in create_transformer
| transformer.register_include(include_obj)
|   File 
"/.../tmp/work/x86_64-linux/libical-native/3.0.14-r0/recipe-sysroot-native/usr/lib/gobject-introspection/giscanner/transformer.py",
 line 140, in register_include
| self._parse_include(filename)
|   File 
"/.../tmp/work/x86_64-linux/libical-native/3.0.14-r0/recipe-sysroot-native/usr/lib/gobject-introspection/giscanner/transformer.py",
 line 230, in _parse_include
| self._parse_include(dep_filename)
|   File 
"/.../tmp/work/x86_64-linux/libical-native/3.0.14-r0/recipe-sysroot-native/usr/lib/gobject-introspection/giscanner/transformer.py",
 line 225, in _parse_include
| self._cachestore.store(filename, parser)
|   File 
"/.../tmp/work/x86_64-linux/libical-native/3.0.14-r0/recipe-sysroot-native/usr/lib/gobject-introspection/giscanner/cachestore.py",
 line 153, in store
| shutil.move(tmp_filename, store_filename)
|   File 
"/.../tmp/work/x86_64-linux/libical-native/3.0.14-r0/recipe-sysroot-native/usr/lib/python3.10/shutil.py",
 line 833, in move
| copy_function(src, real_dst)
|   File 
"/.../tmp/work/x86_64-linux/libical-native/3.0.14-r0/recipe-sysroot-native/usr/lib/python3.10/shutil.py",
 line 435, in copy2
| copystat(src, dst, follow_symlinks=follow_symlinks)
|   File 
"/.../tmp/work/x86_64-linux/libical-native/3.0.14-r0/recipe-sysroot-native/usr/lib/python3.10/shutil.py",
 line 374, in copystat
| lookup("utime")(dst, ns=(st.st_atime_ns, st.st_mtime_ns),
| FileNotFoundError: [Errno 2] No such file or directory

A similar issue is described in a fix for gobject-introspection-native.

https://github.com/openembedded/openembedded-core/commit/d3c48ff7d19e86b2338b1778f9563969bba3d336

The problem was fixed there by setting the environment variable
GI_SCANNER_DISABLE_CACHE to disable the use of $HOME/.cache.

Extend the fix to users of gobject-instropection by promoting the fix
to the bbclass.

Signed-off-by: Tom Hochstein 
---
 meta/classes/gobject-introspection-data.bbclass  | 5 +
 .../gobject-introspection/gobject-introspection_1.72.0.bb| 3 ---
 2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/meta/classes/gobject-introspection-data.bbclass 
b/meta/classes/gobject-introspection-data.bbclass
index 2ef684626a..d90cdb4839 100644
--- a/meta/classes/gobject-introspection-data.bbclass
+++ b/meta/classes/gobject-introspection-data.bbclass
@@ -5,3 +5,8 @@
 # so that qemu use can be avoided when necessary.
 GI_DATA_ENABLED ?= "${@bb.utils.contains('DISTRO_FEATURES', 
'gobject-introspection-data', \
   bb.utils.contains('MACHINE_FEATURES', 'qemu-usermode', 
'True', 'False', d), 'False', d)}"
+
+do_compile:prepend() {
+# This prevents g-ir-scanner from writing cache data to $HOME
+export GI_SCANNER_DISABLE_CACHE=1
+}
diff --git 
a/meta/recipes-gnome/gobject-introspection/gobject-introspection_1.72.0.bb 
b/meta/recipes-gnome/gobject-introspection/gobject-introspection_1.72.0.bb
index 355e77d107..9a47e908b7 100644
--- a/meta/recipes-gnome/gobject-introspection/gobject-introspection_1.72.0.bb
+++ b/meta/recipes-gnome/gobject-introspection/gobject-introspection_1.72.0.bb
@@ -113,9 +113,6 @@ EOF
 }
 
 do_compile:prepend() {
-# This prevents g-ir-scanner from writing cache data to $HOME
-export GI_SCANNER_DISABLE_CACHE=1
-
 # Needed to run g-ir unit tests, which won't be able to find the built 
libraries otherwise
 export GIR_EXTRA_LIBS_PATH=$B/.libs
 }
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#168197): 
https://lists.openembedded.org/g/openembedded-core/message/168197
Mute This Topic: https://lists.openembedded.org/mt/92458797/21656
Gro

[OE-core] [meta-python][PATCH] python3-pybind11: Override pip install variables

2022-02-28 Thread Tom Hochstein
Fix the pip install variables since the default values don't work
here.

Signed-off-by: Tom Hochstein 
---
 meta-python/recipes-devtools/python/python3-pybind11_2.8.1.bb | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/meta-python/recipes-devtools/python/python3-pybind11_2.8.1.bb 
b/meta-python/recipes-devtools/python/python3-pybind11_2.8.1.bb
index dfacb41f1..4d94e2033 100644
--- a/meta-python/recipes-devtools/python/python3-pybind11_2.8.1.bb
+++ b/meta-python/recipes-devtools/python/python3-pybind11_2.8.1.bb
@@ -19,6 +19,9 @@ EXTRA_OECMAKE =  "-DPYBIND11_TEST=OFF"
 
 inherit cmake setuptools3 python3native
 
+PIP_INSTALL_DIST_PATH = "${S}/dist"
+PIP_INSTALL_PACKAGE = "pybind11"
+
 do_configure() {
cmake_do_configure
 }
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#162513): 
https://lists.openembedded.org/g/openembedded-core/message/162513
Mute This Topic: https://lists.openembedded.org/mt/89457175/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] bitbake.conf: add BB_NUMBER_THREADS to BB_HASHEXCLUDE_COMMON

2022-02-28 Thread Tom Hochstein


> -Original Message-
> From: openembedded-core@lists.openembedded.org  c...@lists.openembedded.org> On Behalf Of Richard Purdie via
> lists.openembedded.org
> Sent: Monday, February 28, 2022 8:57 AM
> To: Rasmus Villemoes ; openembedded-
> c...@lists.openembedded.org
> Subject: Re: [OE-core] [PATCH] bitbake.conf: add BB_NUMBER_THREADS to
> BB_HASHEXCLUDE_COMMON
> 
> 
> Just to check, are the meta-imx people aware of it and able to fix the real
> problem?

Yes, I will fix it. Thanks!

Tom


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#162506): 
https://lists.openembedded.org/g/openembedded-core/message/162506
Mute This Topic: https://lists.openembedded.org/mt/89446852/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] xwayland: Add xkbcomp runtime dependency

2022-01-31 Thread Tom Hochstein
Ping.

> -Original Message-
> From: Tom Hochstein
> Sent: Wednesday, January 19, 2022 5:49 PM
> To: openembedded-core@lists.openembedded.org
> Cc: Tom Hochstein 
> Subject: [PATCH] xwayland: Add xkbcomp runtime dependency
> 
> Trying to run an xterm fails with the error:
> xterm: Xt error: Can't open display: :0
> 
> Checking systemctl status weston shows an error:
> Jan 19 21:24:16 imx8mq-evk weston[396]: sh: line 1: /usr/bin/xkbcomp: No such
> file or directory
> 
> Adding xkbcomp to the rootfs fixes these errors. Checking the history one 
> finds
> that the runtime dependency for the old xserver was removed because it wasn't
> in the correct location [1], then restored because it was still needed [2].
> 
> [1] https://github.com/openembedded/openembedded-
> core/commit/bdcc5e8f1286d288baf410458efc39a59b68d751
> [2] https://github.com/openembedded/openembedded-
> core/commit/f2330ebc3071d780cbc6d1ddab5c54bfadf8fffc
> 
> Signed-off-by: Tom Hochstein 
> ---
>  meta/recipes-graphics/xwayland/xwayland_21.1.3.bb | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/meta/recipes-graphics/xwayland/xwayland_21.1.3.bb
> b/meta/recipes-graphics/xwayland/xwayland_21.1.3.bb
> index 5d083e8ada..7ea40d9486 100644
> --- a/meta/recipes-graphics/xwayland/xwayland_21.1.3.bb
> +++ b/meta/recipes-graphics/xwayland/xwayland_21.1.3.bb
> @@ -42,3 +42,4 @@ do_install:append() {
> 
>  FILES:${PN} += "${libdir}/xorg/protocol.txt"
> 
> +RDEPENDS:${PN} += "xkbcomp"
> --
> 2.17.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#161131): 
https://lists.openembedded.org/g/openembedded-core/message/161131
Mute This Topic: https://lists.openembedded.org/mt/88548155/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] xwayland: Add xkbcomp runtime dependency

2022-01-19 Thread Tom Hochstein
Trying to run an xterm fails with the error:
xterm: Xt error: Can't open display: :0

Checking systemctl status weston shows an error:
Jan 19 21:24:16 imx8mq-evk weston[396]: sh: line 1: /usr/bin/xkbcomp: No such 
file or directory

Adding xkbcomp to the rootfs fixes these errors. Checking the history
one finds that the runtime dependency for the old xserver was
removed because it wasn't in the correct location [1], then restored
because it was still needed [2].

[1] 
https://github.com/openembedded/openembedded-core/commit/bdcc5e8f1286d288baf410458efc39a59b68d751
[2] 
https://github.com/openembedded/openembedded-core/commit/f2330ebc3071d780cbc6d1ddab5c54bfadf8fffc

Signed-off-by: Tom Hochstein 
---
 meta/recipes-graphics/xwayland/xwayland_21.1.3.bb | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-graphics/xwayland/xwayland_21.1.3.bb 
b/meta/recipes-graphics/xwayland/xwayland_21.1.3.bb
index 5d083e8ada..7ea40d9486 100644
--- a/meta/recipes-graphics/xwayland/xwayland_21.1.3.bb
+++ b/meta/recipes-graphics/xwayland/xwayland_21.1.3.bb
@@ -42,3 +42,4 @@ do_install:append() {
 
 FILES:${PN} += "${libdir}/xorg/protocol.txt"
 
+RDEPENDS:${PN} += "xkbcomp"
-- 
2.17.1


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



[OE-core] CMAKE_SYSTEM_PROCESSOR inconsistent in SDK

2021-12-16 Thread Tom Hochstein
A colleague notes a cmake problem in the SDK due to CMAKE_SYSTEM_PROCESSOR 
being set to 'cortexa53-crypto', which the package doesn't recognize. The 
package builds fine with bitbake because CMAKE_SYSTEM_PROCESSOR is set to 
'aarch64', which the package does recognize.

The value for the SDK comes from this commit:

https://github.com/openembedded/openembedded-core/commit/4143f3b0ce0d0c52f5b0babc1bb16ac0ac9610eb#diff-68a2f5f7520be29f69417255d4067d5e97b8681596cd851d946d241c8a1dda76

Is there a problem with this commit? Or is there something missing that 
prevents 'cortexa53-crypto' to resolve to the more general 'aarch64'?

Tom

<>
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#159803): 
https://lists.openembedded.org/g/openembedded-core/message/159803
Mute This Topic: https://lists.openembedded.org/mt/87773810/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] pls comment on weston non-PAM patch (link inside)

2021-11-19 Thread Tom Hochstein
Thanks Alex. @Otavio<mailto:otavio.salva...@ossystems.com.br> pushed the 
hardest for the use case and I don't really have anything to add. Given that 
Pekka has suggested that there is a better approach without weston-launch, it 
seems that removing the patch is fine. In the worst case we can backport the 
patch to meta-freescale until we have a new solution.

Tom

From: Alexander Kanavin 
Sent: Friday, November 19, 2021 11:06 AM
To: OE-core ; Tom Hochstein 
; Denys Dmytriyenko 
Cc: ppaala...@gmail.com
Subject: Re: pls comment on weston non-PAM patch (link inside)

Hello guys,

reminding you to speak up in the patch submission; if you don't, upstream will 
close it without merging, and I will start working towards removing the patch 
from Yocto as well.

Alex

On Fri, 12 Nov 2021 at 10:48, Alexander Kanavin 
mailto:alex.kana...@gmail.com>> wrote:
Hello folks,

I sent the weston non-PAM patch upstream and they have raised questions about 
its validity, can you please comment on it?
https://gitlab.freedesktop.org/wayland/weston/-/merge_requests/725<https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitlab.freedesktop.org%2Fwayland%2Fweston%2F-%2Fmerge_requests%2F725=04%7C01%7Ctom.hochstein%40nxp.com%7C6a003f79d8a44b3beaf008d9ab7ee8ad%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C637729383818649026%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000=mOGax0pBnQjJBMrDsfv6bv%2FgZ96u%2Fd5UcP3wZDph36Q%3D=0>

Thanks,
Alex




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



[OE-core] [honister][PATCH 3/3] bitbake.conf: Use wayland distro feature for native builds

2021-11-08 Thread Tom Hochstein
The wayland-scanner is missing from SDKs with weston, but the weston build
requires wayland-scanner. Allow the distro feature in order to include
the wayland-scanner packages via nativesdk-packagegroup-sdk-host.bb.

Signed-off-by: Tom Hochstein 
Signed-off-by: Richard Purdie 
(cherry picked from commit 858cc6f257e22e39df83f4808ea27c6d12cd1b80)
---
 meta/conf/bitbake.conf | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
index 9274322e3a..c2cb2f0d9d 100644
--- a/meta/conf/bitbake.conf
+++ b/meta/conf/bitbake.conf
@@ -850,8 +850,8 @@ DISTRO_FEATURES_NATIVESDK ?= "x11"
 
 # Normally target distro features will not be applied to native builds:
 # Native distro features on this list will use the target feature value
-DISTRO_FEATURES_FILTER_NATIVE ?= "api-documentation debuginfod opengl"
-DISTRO_FEATURES_FILTER_NATIVESDK ?= "api-documentation debuginfod opengl"
+DISTRO_FEATURES_FILTER_NATIVE ?= "api-documentation debuginfod opengl wayland"
+DISTRO_FEATURES_FILTER_NATIVESDK ?= "api-documentation debuginfod opengl 
wayland"
 
 DISTRO_FEATURES_BACKFILL = "pulseaudio sysvinit gobject-introspection-data 
ldconfig"
 MACHINE_FEATURES_BACKFILL = "rtc qemu-usermode"
-- 
2.17.1


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



[OE-core] [honister][PATCH 1/3] wayland: Fix wayland-tools packaging

2021-11-08 Thread Tom Hochstein
There are some packaging problems due to the wayland-tools packaging
implementation. The wayland-tools package currently looks like this:

wayland-tools
└── usr
├── bin
│   └── wayland-scanner
└── share
└── wayland
├── wayland.dtd
├── wayland-scanner.mk
└── wayland.xml

The files wayland.dtd and wayland.xml belong in the main package,
while wayland-scanner.mk belongs in wayland-dev.

Fix the wayland.dtd and wayland.xml packaging by prepending the
wayland-tools package and dropping the main package FILES variable
override. The file wayland-scanner.mk is included in the main
package by default, and so must be explicitly added to wayland-dev.

Signed-off-by: Tom Hochstein 
Signed-off-by: Richard Purdie 
(cherry picked from commit a31fbec45d24df5b74091940d0e0b2daf34d8492)
---
 meta/recipes-graphics/wayland/wayland_1.19.0.bb | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/meta/recipes-graphics/wayland/wayland_1.19.0.bb 
b/meta/recipes-graphics/wayland/wayland_1.19.0.bb
index d6e468497d..5f8b972f76 100644
--- a/meta/recipes-graphics/wayland/wayland_1.19.0.bb
+++ b/meta/recipes-graphics/wayland/wayland_1.19.0.bb
@@ -52,10 +52,10 @@ sysroot_stage_all:append:class-target () {
cp ${STAGING_DATADIR_NATIVE}/aclocal/wayland-scanner.m4 
${SYSROOT_DESTDIR}/${datadir}/aclocal/
 }
 
-PACKAGES += "${PN}-tools"
+PACKAGES =+ "${PN}-tools"
 
-FILES:${PN} = "${libdir}/*${SOLIBS}"
-FILES:${PN}-tools += "${bindir} ${datadir}/wayland"
+FILES:${PN}-tools = "${bindir}/wayland-scanner"
+FILES:${PN}-dev += "${datadir}/${BPN}/wayland-scanner.mk"
 
 BBCLASSEXTEND = "native nativesdk"
 
-- 
2.17.1


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



[OE-core] [honister][PATCH 2/3] nativesdk-packagegroup-sdk-host.bb: Update host tools for wayland

2021-11-08 Thread Tom Hochstein
The wayland-scanner host tool required to build weston is moved to the
wayland-tools package, so update the SDK host tools list accordingly.

Also, the weston build requires wayland-scanner.pc to find wayland-scanner,
so add wayland-dev.

Signed-off-by: Tom Hochstein 
Signed-off-by: Richard Purdie 
(cherry picked from commit 31ed91bdbb0ec05730fb98d7cc523bb46aca50e3)
---
 .../packagegroups/nativesdk-packagegroup-sdk-host.bb| 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-core/packagegroups/nativesdk-packagegroup-sdk-host.bb 
b/meta/recipes-core/packagegroups/nativesdk-packagegroup-sdk-host.bb
index 39e5002bb7..9166a0851f 100644
--- a/meta/recipes-core/packagegroups/nativesdk-packagegroup-sdk-host.bb
+++ b/meta/recipes-core/packagegroups/nativesdk-packagegroup-sdk-host.bb
@@ -25,7 +25,7 @@ RDEPENDS:${PN} = "\
 nativesdk-makedevs \
 nativesdk-cmake \
 nativesdk-meson \
-${@bb.utils.contains('DISTRO_FEATURES', 'wayland', 'nativesdk-wayland', 
'', d)} \
+${@bb.utils.contains('DISTRO_FEATURES', 'wayland', 
'nativesdk-wayland-tools nativesdk-wayland-dev', '', d)} \
 nativesdk-sdk-provides-dummy \
 nativesdk-bison \
 nativesdk-flex \
-- 
2.17.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#157992): 
https://lists.openembedded.org/g/openembedded-core/message/157992
Mute This Topic: https://lists.openembedded.org/mt/86919164/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/3] bitbake.conf: Use wayland distro feature for native builds

2021-11-07 Thread Tom Hochstein
The wayland-scanner is missing from SDKs with weston, but the weston build
requires wayland-scanner. Allow the distro feature in order to include
the wayland-scanner packages via nativesdk-packagegroup-sdk-host.bb.

Signed-off-by: Tom Hochstein 
---
 meta/conf/bitbake.conf | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
index 790f2f7a8c..71c1e52ad6 100644
--- a/meta/conf/bitbake.conf
+++ b/meta/conf/bitbake.conf
@@ -869,8 +869,8 @@ DISTRO_FEATURES_NATIVESDK ?= "x11"
 
 # Normally target distro features will not be applied to native builds:
 # Native distro features on this list will use the target feature value
-DISTRO_FEATURES_FILTER_NATIVE ?= "api-documentation debuginfod opengl"
-DISTRO_FEATURES_FILTER_NATIVESDK ?= "api-documentation debuginfod opengl"
+DISTRO_FEATURES_FILTER_NATIVE ?= "api-documentation debuginfod opengl wayland"
+DISTRO_FEATURES_FILTER_NATIVESDK ?= "api-documentation debuginfod opengl 
wayland"
 
 DISTRO_FEATURES_BACKFILL = "pulseaudio sysvinit gobject-introspection-data 
ldconfig"
 MACHINE_FEATURES_BACKFILL = "rtc qemu-usermode"
-- 
2.17.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#157960): 
https://lists.openembedded.org/g/openembedded-core/message/157960
Mute This Topic: https://lists.openembedded.org/mt/86888902/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/3] nativesdk-packagegroup-sdk-host.bb: Update host tools for wayland

2021-11-07 Thread Tom Hochstein
The wayland-scanner host tool required to build weston is moved to the
wayland-tools package, so update the SDK host tools list accordingly.

Also, the weston build requires wayland-scanner.pc to find wayland-scanner,
so add wayland-dev.

Signed-off-by: Tom Hochstein 
---
 .../packagegroups/nativesdk-packagegroup-sdk-host.bb| 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-core/packagegroups/nativesdk-packagegroup-sdk-host.bb 
b/meta/recipes-core/packagegroups/nativesdk-packagegroup-sdk-host.bb
index 39e5002bb7..9166a0851f 100644
--- a/meta/recipes-core/packagegroups/nativesdk-packagegroup-sdk-host.bb
+++ b/meta/recipes-core/packagegroups/nativesdk-packagegroup-sdk-host.bb
@@ -25,7 +25,7 @@ RDEPENDS:${PN} = "\
 nativesdk-makedevs \
 nativesdk-cmake \
 nativesdk-meson \
-${@bb.utils.contains('DISTRO_FEATURES', 'wayland', 'nativesdk-wayland', 
'', d)} \
+${@bb.utils.contains('DISTRO_FEATURES', 'wayland', 
'nativesdk-wayland-tools nativesdk-wayland-dev', '', d)} \
 nativesdk-sdk-provides-dummy \
 nativesdk-bison \
 nativesdk-flex \
-- 
2.17.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#157959): 
https://lists.openembedded.org/g/openembedded-core/message/157959
Mute This Topic: https://lists.openembedded.org/mt/86888900/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/3] wayland: Fix wayland-tools packaging

2021-11-07 Thread Tom Hochstein
There are some packaging problems due to the wayland-tools packaging
implementation. The wayland-tools package currently looks like this:

wayland-tools
└── usr
├── bin
│   └── wayland-scanner
└── share
└── wayland
├── wayland.dtd
├── wayland-scanner.mk
└── wayland.xml

The files wayland.dtd and wayland.xml belong in the main package,
while wayland-scanner.mk belongs in wayland-dev.

Fix the wayland.dtd and wayland.xml packaging by prepending the
wayland-tools package and dropping the main package FILES variable
override. The file wayland-scanner.mk is included in the main
package by default, and so must be explicitly added to wayland-dev.

Signed-off-by: Tom Hochstein 
---
 meta/recipes-graphics/wayland/wayland_1.19.0.bb | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/meta/recipes-graphics/wayland/wayland_1.19.0.bb 
b/meta/recipes-graphics/wayland/wayland_1.19.0.bb
index d6e468497d..5f8b972f76 100644
--- a/meta/recipes-graphics/wayland/wayland_1.19.0.bb
+++ b/meta/recipes-graphics/wayland/wayland_1.19.0.bb
@@ -52,10 +52,10 @@ sysroot_stage_all:append:class-target () {
cp ${STAGING_DATADIR_NATIVE}/aclocal/wayland-scanner.m4 
${SYSROOT_DESTDIR}/${datadir}/aclocal/
 }
 
-PACKAGES += "${PN}-tools"
+PACKAGES =+ "${PN}-tools"
 
-FILES:${PN} = "${libdir}/*${SOLIBS}"
-FILES:${PN}-tools += "${bindir} ${datadir}/wayland"
+FILES:${PN}-tools = "${bindir}/wayland-scanner"
+FILES:${PN}-dev += "${datadir}/${BPN}/wayland-scanner.mk"
 
 BBCLASSEXTEND = "native nativesdk"
 
-- 
2.17.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#157958): 
https://lists.openembedded.org/g/openembedded-core/message/157958
Mute This Topic: https://lists.openembedded.org/mt/8691/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 3/3] bitbake.conf: Use wayland distro feature for native builds

2021-11-07 Thread Tom Hochstein
The wayland-scanner is missing from SDKs with weston, but the weston build
requires wayland-scanner. Allow the distro feature in order to include
the wayland-scanner packages via nativesdk-packagegroup-sdk-host.bb.

Signed-off-by: Tom Hochstein 
---
 meta/conf/bitbake.conf | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
index 790f2f7a8c..71c1e52ad6 100644
--- a/meta/conf/bitbake.conf
+++ b/meta/conf/bitbake.conf
@@ -869,8 +869,8 @@ DISTRO_FEATURES_NATIVESDK ?= "x11"
 
 # Normally target distro features will not be applied to native builds:
 # Native distro features on this list will use the target feature value
-DISTRO_FEATURES_FILTER_NATIVE ?= "api-documentation debuginfod opengl"
-DISTRO_FEATURES_FILTER_NATIVESDK ?= "api-documentation debuginfod opengl"
+DISTRO_FEATURES_FILTER_NATIVE ?= "api-documentation debuginfod opengl wayland"
+DISTRO_FEATURES_FILTER_NATIVESDK ?= "api-documentation debuginfod opengl 
wayland"
 
 DISTRO_FEATURES_BACKFILL = "pulseaudio sysvinit gobject-introspection-data 
ldconfig"
 MACHINE_FEATURES_BACKFILL = "rtc qemu-usermode"
-- 
2.17.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#157956): 
https://lists.openembedded.org/g/openembedded-core/message/157956
Mute This Topic: https://lists.openembedded.org/mt/86885059/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/3] nativesdk-packagegroup-sdk-host.bb: Update host tools for wayland

2021-11-07 Thread Tom Hochstein
The wayland-scanner host tool required to build weston is moved to the
wayland-tools package, so update the SDK host tools list accordingly.

Also, the weston build requires wayland-scanner.pc to find wayland-scanner,
so add wayland-tools-dev.

Signed-off-by: Tom Hochstein 
---
 .../packagegroups/nativesdk-packagegroup-sdk-host.bb| 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-core/packagegroups/nativesdk-packagegroup-sdk-host.bb 
b/meta/recipes-core/packagegroups/nativesdk-packagegroup-sdk-host.bb
index 39e5002bb7..16752026d0 100644
--- a/meta/recipes-core/packagegroups/nativesdk-packagegroup-sdk-host.bb
+++ b/meta/recipes-core/packagegroups/nativesdk-packagegroup-sdk-host.bb
@@ -25,7 +25,7 @@ RDEPENDS:${PN} = "\
 nativesdk-makedevs \
 nativesdk-cmake \
 nativesdk-meson \
-${@bb.utils.contains('DISTRO_FEATURES', 'wayland', 'nativesdk-wayland', 
'', d)} \
+${@bb.utils.contains('DISTRO_FEATURES', 'wayland', 
'nativesdk-wayland-tools nativesdk-wayland-tools-dev', '', d)} \
 nativesdk-sdk-provides-dummy \
 nativesdk-bison \
 nativesdk-flex \
-- 
2.17.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#157955): 
https://lists.openembedded.org/g/openembedded-core/message/157955
Mute This Topic: https://lists.openembedded.org/mt/86885057/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/3] wayland: Fix wayland-tools packaging problems

2021-11-07 Thread Tom Hochstein
There are several packaging problems due to the wayland-tools packaging
implementation. The wayland-tools package currently looks like this:

wayland-tools
└── usr
├── bin
│   └── wayland-scanner
└── share
└── wayland
├── wayland.dtd
├── wayland-scanner.mk
└── wayland.xml

The files wayland-scanner.pc and wayland-scanner.m4 are incorrectly
located in the main package. The files wayland.dtd and wayland.xml
are incorrectly located here but belong in the main package.

Fix the problems by moving wayland-tools before the other packages,
adding wayland-tools-dev, and dropping the main package FILES
variable override as no longer needed.

After the fix wayland-tools and wayland-tools-dev look like this:

wayland-tools
└── usr
└── bin
└── wayland-scanner
wayland-tools-dev
└── usr
├── lib
│   └── pkgconfig
│   └── wayland-scanner.pc
└── share
├── aclocal
│   └── wayland-scanner.m4
└── wayland
└── wayland-scanner.mk

Signed-off-by: Tom Hochstein 
---
 meta/recipes-graphics/wayland/wayland_1.19.0.bb | 10 +++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/meta/recipes-graphics/wayland/wayland_1.19.0.bb 
b/meta/recipes-graphics/wayland/wayland_1.19.0.bb
index d6e468497d..74d6d65e9e 100644
--- a/meta/recipes-graphics/wayland/wayland_1.19.0.bb
+++ b/meta/recipes-graphics/wayland/wayland_1.19.0.bb
@@ -52,10 +52,14 @@ sysroot_stage_all:append:class-target () {
cp ${STAGING_DATADIR_NATIVE}/aclocal/wayland-scanner.m4 
${SYSROOT_DESTDIR}/${datadir}/aclocal/
 }
 
-PACKAGES += "${PN}-tools"
+PACKAGES =+ "${PN}-tools ${PN}-tools-dev"
 
-FILES:${PN} = "${libdir}/*${SOLIBS}"
-FILES:${PN}-tools += "${bindir} ${datadir}/wayland"
+FILES:${PN}-tools = "${bindir}/wayland-scanner"
+FILES:${PN}-tools-dev = " \
+${libdir}/pkgconfig/wayland-scanner.pc \
+${datadir}/aclocal/wayland-scanner.m4 \
+${datadir}/wayland/wayland-scanner.mk \
+"
 
 BBCLASSEXTEND = "native nativesdk"
 
-- 
2.17.1


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



[OE-core] weston systemd watchdog

2021-08-23 Thread Tom Hochstein
Hi Khem,

This is regarding the weston systemd watchdog timeout of 20 seconds:

https://github.com/openembedded/openembedded-core/commit/c21fa5a291ab207a084285935ab73a0b4225c965#diff-ac8a4b56ade4a43a070c93486fb1d7606573da5d44de96bc1529d15b0b216660R36

For GPU-limited parts, we see watchdog timeouts during intense testing, though 
the part is not hung, it's just going to take longer than 20 seconds.

I stumbled across the following systemd commit, which looks like the watchdog 
used to default to 180 seconds. Yet even then was under scrutiny because it was 
causing un-necessary failures more often than not:

https://github.com/systemd/systemd/commit/21d0dd5a89fe0ef259ca51ebea9f39dd79a341c2#diff-9d3c1cd071b0f173aaf53a7799ac5765bb3a58d0d07125f075b6b49112f0a425

Do you think the default should be raised? Or should we just make a special 
case in our layer?

Tom

<>
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#155193): 
https://lists.openembedded.org/g/openembedded-core/message/155193
Mute This Topic: https://lists.openembedded.org/mt/85096163/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 2/2] gstreamer1.0-plugins-base: Use bb.utils.filter to reduce code

2021-04-15 Thread Tom Hochstein


> -Original Message-
> From: openembedded-core@lists.openembedded.org 
>  On Behalf Of Otavio Salvador via 
> lists.openembedded.org
> Sent: Thursday, April 15, 2021 9:59 AM


> +OPENGL_WINSYS_append = "${@bb.utils.filter('PACKAGECONFIG', 'x11 gbm wayland 
> dispmanx egl viv-fb', d)}"

Needs a leading space?

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#150588): 
https://lists.openembedded.org/g/openembedded-core/message/150588
Mute This Topic: https://lists.openembedded.org/mt/82118763/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] [meta-oe][PATCH] glm: Fix packaging for header-only recipe

2021-01-29 Thread Tom Hochstein

On Thu, Jan 28, 2021 at 10:57 PM Khem Raj <mailto:raj.k...@gmail.com> wrote:
> 
> On Thu, Jan 28, 2021 at 5:44 PM Tom Hochstein <mailto:tom.hochst...@nxp.com> 
> wrote:
> > Set ALLOW_EMPTY for the main package to "1" for this header-only recipe. 
> > This
> > allows the SDK to be created properly when there is an RDEPENDS on glm.
> 
> Runtime dependency on a header only package seems a bit iffy it's perhaps 
> better to depend on glm-dev perhaps ?

Well, it's all iffy in my mind since these are just different workarounds for 
the root problem that header-only packages are not included in the SDK. If you 
think including the headers in the image is the better workaround, then I will 
switch to that.

Tom

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



[OE-core] [meta-oe][PATCH] glm: Fix packaging for header-only recipe

2021-01-28 Thread Tom Hochstein
Set ALLOW_EMPTY for the main package to "1" for this header-only recipe. This
allows the SDK to be created properly when there is an RDEPENDS on glm.

Signed-off-by: Tom Hochstein 
---
 meta-oe/recipes-graphics/glm/glm_0.9.9.6.bb | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta-oe/recipes-graphics/glm/glm_0.9.9.6.bb 
b/meta-oe/recipes-graphics/glm/glm_0.9.9.6.bb
index e2f4dbebc..019e6257d 100644
--- a/meta-oe/recipes-graphics/glm/glm_0.9.9.6.bb
+++ b/meta-oe/recipes-graphics/glm/glm_0.9.9.6.bb
@@ -34,6 +34,7 @@ do_install() {
 
 }
 
-RDEPENDS_${PN}-dev = ""
+# This is a header-only library, so the main package will be empty.
+ALLOW_EMPTY_${PN} = "1"
 
 BBCLASSEXTEND = "native"
-- 
2.17.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#147422): 
https://lists.openembedded.org/g/openembedded-core/message/147422
Mute This Topic: https://lists.openembedded.org/mt/80200725/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] [RFC] openssl: Enable cryptodev-linux by default

2021-01-28 Thread Tom Hochstein
This is a Request for Comment. Would it be a good idea to enable cryptodev-linux
by default, gaining hardware acceleration where supported? Are there any
unacceptable drawbacks? What happens on hardware without acceleration?

Signed-off-by: Tom Hochstein 
---
 meta/recipes-connectivity/openssl/openssl_1.1.1i.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-connectivity/openssl/openssl_1.1.1i.bb 
b/meta/recipes-connectivity/openssl/openssl_1.1.1i.bb
index 5617f337e0..d4c19f1a52 100644
--- a/meta/recipes-connectivity/openssl/openssl_1.1.1i.bb
+++ b/meta/recipes-connectivity/openssl/openssl_1.1.1i.bb
@@ -28,7 +28,7 @@ SRC_URI[sha256sum] = 
"e8be6a35fe41d10603c3cc635e93289ed00bf34b79671a3a4de64fcee0
 inherit lib_package multilib_header multilib_script ptest
 MULTILIB_SCRIPTS = "${PN}-bin:${bindir}/c_rehash"
 
-PACKAGECONFIG ?= ""
+PACKAGECONFIG ?= "cryptodev-linux"
 PACKAGECONFIG_class-native = ""
 PACKAGECONFIG_class-nativesdk = ""
 
-- 
2.17.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#147412): 
https://lists.openembedded.org/g/openembedded-core/message/147412
Mute This Topic: https://lists.openembedded.org/mt/80195674/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] mesa: Add xcb-fixes to loader when using x11 and dri3

2020-11-12 Thread Tom Hochstein
> -Original Message-
> From: Andrey Zhizhikin 
> Sent: Thursday, November 12, 2020 3:23 PM
> 
> I believe all patch files are located in `files` folder, while this
> patch tries to put it into `mesa` folder which does not exist yet? Is
> it intended here?
> 

It was a mistake, thanks for catching it. V2 submitted.

Tom

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#144537): 
https://lists.openembedded.org/g/openembedded-core/message/144537
Mute This Topic: https://lists.openembedded.org/mt/78060312/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] mesa: Add xcb-fixes to loader when using x11 and dri3

2020-11-12 Thread Tom Hochstein
Backport fix for undefined symbol for xcb_xfixes_create_region in 
loader_dri3_helper.c.

Signed-off-by: Tom Hochstein 
---
 ...xes-to-loader-when-using-x11-and-dri.patch | 36 +++
 meta/recipes-graphics/mesa/mesa.inc   |  1 +
 2 files changed, 37 insertions(+)
 create mode 100644 
meta/recipes-graphics/mesa/files/0001-meson-Add-xcb-fixes-to-loader-when-using-x11-and-dri.patch

diff --git 
a/meta/recipes-graphics/mesa/files/0001-meson-Add-xcb-fixes-to-loader-when-using-x11-and-dri.patch
 
b/meta/recipes-graphics/mesa/files/0001-meson-Add-xcb-fixes-to-loader-when-using-x11-and-dri.patch
new file mode 100644
index 00..9ee72880a2
--- /dev/null
+++ 
b/meta/recipes-graphics/mesa/files/0001-meson-Add-xcb-fixes-to-loader-when-using-x11-and-dri.patch
@@ -0,0 +1,36 @@
+From cf17d6251653f4a98e7c4f904ea2f0bc0ecedd5c Mon Sep 17 00:00:00 2001
+From: Duncan Hopkins 
+Date: Thu, 15 Oct 2020 12:14:57 +0100
+Subject: [PATCH] meson: Add xcb-fixes to loader when using x11 and dri3. Fixes
+ undefined symbol for xcb_xfixes_create_region in loader_dri3_helper.c
+
+loader_dr3_helper.c uses xcb_xfixes_create_region() that requires 
dep_xcb_xfixes to link. This is dependent on with_platform_x11 and with_dri3.
+But the source meson file does not set this up dependent on with_dri3.
+The build was initialsed using platforms=x11 and gallium-drivers=zink,swrast.
+
+Reviewed-by: Eric Anholt 
+Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/7164>
+
+Upstream-Status: Backport [cf17d6251653f4a98e7c4f904ea2f0bc0ecedd5c]
+
+---
+ meson.build | 3 ++-
+ 1 file changed, 2 insertions(+), 1 deletion(-)
+
+diff --git a/meson.build b/meson.build
+index cfe02fa6373..3cb3c904927 100644
+--- a/meson.build
 b/meson.build
+@@ -1782,7 +1782,8 @@ if with_platform_x11
+   dep_xxf86vm = dependency('xxf86vm')
+ endif
+   endif
+-  if (with_egl or (
++  if (with_egl or 
++  with_dri3 or (
+   with_gallium_vdpau or with_gallium_xvmc or with_gallium_xa or
+   with_gallium_omx != 'disabled'))
+ dep_xcb_xfixes = dependency('xcb-xfixes')
+-- 
+2.17.1
+
diff --git a/meta/recipes-graphics/mesa/mesa.inc 
b/meta/recipes-graphics/mesa/mesa.inc
index 5a6caf2662..ca593b8b1f 100644
--- a/meta/recipes-graphics/mesa/mesa.inc
+++ b/meta/recipes-graphics/mesa/mesa.inc
@@ -22,6 +22,7 @@ SRC_URI = 
"https://mesa.freedesktop.org/archive/mesa-${PV}.tar.xz \
file://0001-meson-misdetects-64bit-atomics-on-mips-clang.patch \
file://0001-futex.h-Define-__NR_futex-if-it-does-not-exist.patch \

file://0001-anv-fix-a-build-race-between-generating-a-header-and.patch \
+   
file://0001-meson-Add-xcb-fixes-to-loader-when-using-x11-and-dri.patch \
"
 
 SRC_URI[sha256sum] = 
"d1a46d9a3f291bc0e0374600bdcb59844fa3eafaa50398e472a36fc65fd0244a"
-- 
2.17.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#144536): 
https://lists.openembedded.org/g/openembedded-core/message/144536
Mute This Topic: https://lists.openembedded.org/mt/78217060/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] gtest and gmock not added to SDK

2020-05-22 Thread Tom Hochstein
On Fri, May 22, 2020 at 03:33 PM, Andre McCurdy wrote:

> 
> On Fri, May 22, 2020 at 2:29 PM Otavio Salvador
>  wrote:
> 
>> On Fri, May 22, 2020 at 5:48 PM Tom Hochstein 
>> wrote:
>> 
>>> The gtest and gmock packages provided by googletest are not added to the
>>> SDK by default. I assume it is because the googletest main package is
>>> empty?
>> 
>> 
> 
> Are you including googletest in the image used to create the SDK or
> gmock and gtest?
> 
> It looks like gmock and gtest are only mentioned in the googletest
> recipe as PROVIDES, so they are aliases for build time dependencies
> only. For run time dependencies you should use googletest.

Right, I should have mentioned that. I have a DEPENDS on gtest for a package in 
my image. To fix the SDK, I added an RDEPENDS on googletest from the same 
package.

> 
> 
> 
>> 
>>> I added an RDEPENDS_${PN} from the dependent package to fix it. Is there a
>>> better solution?
>> 
>> You can change the packages to have their `-dev` to rdepends on
>> `gtest/gmock-dev` for example.
>> 
>> --
>> Otavio Salvador O.S. Systems
>> http://www.ossystems.com.br http://code.ossystems.com.br
>> Mobile: +55 (53) 9 9981-7854 Mobile: +1 (347) 903-9750
> 
>
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#138615): 
https://lists.openembedded.org/g/openembedded-core/message/138615
Mute This Topic: https://lists.openembedded.org/mt/74408257/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


[OE-core] gtest and gmock not added to SDK

2020-05-22 Thread Tom Hochstein
The gtest and gmock packages provided by googletest are not added to the SDK by 
default. I assume it is because the googletest main package is empty?

I added an RDEPENDS_${PN} from the dependent package to fix it. Is there a 
better solution?

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

View/Reply Online (#138610): 
https://lists.openembedded.org/g/openembedded-core/message/138610
Mute This Topic: https://lists.openembedded.org/mt/74408257/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


[OE-core] Cannot disable weston

2020-05-06 Thread Tom Hochstein
We are no longer able to disable weston service with standard `systemctl 
disable weston@root.service`. On boot the service is started again. It seems 
likely this started with the introduction of a starting udev rule:

https://github.com/openembedded/openembedded-core/commit/aa3bced2e1de2f4ba507aa014835b06edccc138a#diff-d92aa97566c1bd0dd1eb817bb4511cda

Is this behaving correctly? Can the rule be made to honor the service 
disablement?

Tom

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

View/Reply Online (#137989): 
https://lists.openembedded.org/g/openembedded-core/message/137989
Mute This Topic: https://lists.openembedded.org/mt/74037529/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


[OE-core][zeus][PATCH] security_flags.inc: fix flags missing from SDK toolchain

2020-03-25 Thread Tom Hochstein
The security flags were missing from the SDK toolchain
because they were added specifically to class-target.
Add them to class-cross-canadian as well (since the SDK environment
file is created from cross-canadian target flags).

Signed-off-by: Tom Hochstein 
Signed-off-by: Antoine Manache 
Signed-off-by: Richard Purdie 
---
 meta/conf/distro/include/security_flags.inc | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/meta/conf/distro/include/security_flags.inc 
b/meta/conf/distro/include/security_flags.inc
index aaf04e9e59..568d03693c 100644
--- a/meta/conf/distro/include/security_flags.inc
+++ b/meta/conf/distro/include/security_flags.inc
@@ -57,6 +57,8 @@ SECURITY_STRINGFORMAT_pn-gcc = ""
 
 TARGET_CC_ARCH_append_class-target = " ${SECURITY_CFLAGS}"
 TARGET_LDFLAGS_append_class-target = " ${SECURITY_LDFLAGS}"
+TARGET_CC_ARCH_append_class-cross-canadian = " ${SECURITY_CFLAGS}"
+TARGET_LDFLAGS_append_class-cross-canadian = " ${SECURITY_LDFLAGS}"
 
 SECURITY_STACK_PROTECTOR_pn-gcc-runtime = ""
 SECURITY_STACK_PROTECTOR_pn-glibc = ""
-- 
2.17.1

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

View/Reply Online (#136692): 
https://lists.openembedded.org/g/openembedded-core/message/136692
Mute This Topic: https://lists.openembedded.org/mt/72540579/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] security_flags.inc: fix flags missing from SDK toolchain

2020-03-24 Thread Tom Hochstein


> -Original Message-
> From: Richard Purdie 
> Sent: Tuesday, March 24, 2020 5:58 PM
> To: Antoine Manache ; Tom Hochstein 
> 
> Cc: openembedded-core@lists.openembedded.org
> Subject: Re: [OE-core][PATCH] security_flags.inc: fix flags missing from SDK 
> toolchain
> 
> I've therefore taken Tom's version with some tweaks to the commit
> message.

Thanks. Can we get this on zeus? Do I need to cherry-pick and submit the patch?

Tom

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

View/Reply Online (#136685): 
https://lists.openembedded.org/g/openembedded-core/message/136685
Mute This Topic: https://lists.openembedded.org/mt/72521987/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] security_flags.inc: fix flags missing from SDK toolchain

2020-03-24 Thread Tom Hochstein


> -Original Message-
> From: Richard Purdie 
> Sent: Tuesday, March 24, 2020 1:03 PM
> To: Antoine Manache ; Tom Hochstein 
> 
> Cc: openembedded-core@lists.openembedded.org
> Subject: Re: [OE-core][PATCH] security_flags.inc: fix flags missing from SDK 
> toolchain
> 
> On Tue, 2020-03-24 at 18:25 +0100, Antoine Manache wrote:
> > I've already sent this patch few months ago but not taken into
> > account yet.
> 
> Can someone explain more about the issue here please?
> 
> Shouldn't these flags be added to the SDK toolchain environment files
> rather than coded into the compiler? Why aren't they being coded in? or
> is gcc being misconfigured?

The recipe meta-environment inherits cross-canadian and toolchain-scripts. The 
latter generates the SDK toolchain environment file using TARGET_CC_ARCH and 
TARGET_LDFLAGS like this:

echo 'export CC="${TARGET_PREFIX}gcc ${TARGET_CC_ARCH} 
--sysroot=$SDKTARGETSYSROOT"' >> $script
echo 'export LDFLAGS="${TARGET_LDFLAGS}"' >> $script

https://github.com/openembedded/openembedded-core/blob/master/meta/classes/toolchain-scripts.bbclass#L82

With class-cross-canadian as the applicable override, updating the security 
flags for class-target does nothing for the SDK toolchain, hence the proposed 
fix:

TARGET_CC_ARCH_append_class-target = " ${SECURITY_CFLAGS}"
+TARGET_CC_ARCH_append_class-cross-canadian = " ${SECURITY_CFLAGS}"
 TARGET_LDFLAGS_append_class-target = " ${SECURITY_LDFLAGS}"
+TARGET_LDFLAGS_append_class-cross-canadian = " ${SECURITY_LDFLAGS}"

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

View/Reply Online (#136665): 
https://lists.openembedded.org/g/openembedded-core/message/136665
Mute This Topic: https://lists.openembedded.org/mt/72521987/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] security_flags.inc: fix flags missing from SDK toolchain

2020-03-24 Thread Tom Hochstein
The security flags were missing from the SDK toolchain
because they were added specifically to class-target.
Add them to class-cross-canadian as well.

Signed-off-by: Tom Hochstein 
---
 meta/conf/distro/include/security_flags.inc | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/meta/conf/distro/include/security_flags.inc 
b/meta/conf/distro/include/security_flags.inc
index aaf04e9e59..568d03693c 100644
--- a/meta/conf/distro/include/security_flags.inc
+++ b/meta/conf/distro/include/security_flags.inc
@@ -57,6 +57,8 @@ SECURITY_STRINGFORMAT_pn-gcc = ""
 
 TARGET_CC_ARCH_append_class-target = " ${SECURITY_CFLAGS}"
 TARGET_LDFLAGS_append_class-target = " ${SECURITY_LDFLAGS}"
+TARGET_CC_ARCH_append_class-cross-canadian = " ${SECURITY_CFLAGS}"
+TARGET_LDFLAGS_append_class-cross-canadian = " ${SECURITY_LDFLAGS}"
 
 SECURITY_STACK_PROTECTOR_pn-gcc-runtime = ""
 SECURITY_STACK_PROTECTOR_pn-glibc = ""
-- 
2.17.1

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

View/Reply Online (#136657): 
https://lists.openembedded.org/g/openembedded-core/message/136657
Mute This Topic: https://lists.openembedded.org/mt/72521987/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/2] bitbake.conf: Fix nativesdk-wayland missing from SDK

2020-03-20 Thread Tom Hochstein
nativesdk-wayland was not being added to the SDK because it
relies on checking the distro feature. This fixes that by
filtering the wayland feature to the SDK build.

Signed-off-by: Tom Hochstein 
---
 meta/conf/bitbake.conf | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
index 4b544a22cd..e44de34a00 100644
--- a/meta/conf/bitbake.conf
+++ b/meta/conf/bitbake.conf
@@ -830,7 +830,7 @@ DISTRO_FEATURES_NATIVESDK ?= "x11"
 # Normally target distro features will not be applied to native builds:
 # Native distro features on this list will use the target feature value
 DISTRO_FEATURES_FILTER_NATIVE ?= "api-documentation"
-DISTRO_FEATURES_FILTER_NATIVESDK ?= "api-documentation"
+DISTRO_FEATURES_FILTER_NATIVESDK ?= "api-documentation wayland"
 
 DISTRO_FEATURES_BACKFILL = "pulseaudio sysvinit gobject-introspection-data 
ldconfig"
 MACHINE_FEATURES_BACKFILL = "rtc qemu-usermode"
-- 
2.17.1


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

View/Reply Online (#136573): 
https://lists.openembedded.org/g/Openembedded-core/message/136573
Mute This Topic: https://lists.openembedded.org/mt/72439818/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/2] wayland: Fix wayland-scanner missing from SDK

2020-03-20 Thread Tom Hochstein
The FILES variable was customized to package wayland-scanner
in the -dev package. However, this is only correct for the
target package and causes wayland-scanner to be missing from
the SDK.

Fix the nativesdk packaging by properly limiting the
wayland-scanner -dev packaging to the target.

Signed-off-by: Tom Hochstein 
---
 meta/recipes-graphics/wayland/wayland_1.18.0.bb | 6 --
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-graphics/wayland/wayland_1.18.0.bb 
b/meta/recipes-graphics/wayland/wayland_1.18.0.bb
index 00be3aac27..4d110a6e1f 100644
--- a/meta/recipes-graphics/wayland/wayland_1.18.0.bb
+++ b/meta/recipes-graphics/wayland/wayland_1.18.0.bb
@@ -54,8 +54,10 @@ sysroot_stage_all_append_class-target () {
cp ${STAGING_DATADIR_NATIVE}/aclocal/wayland-scanner.m4 
${SYSROOT_DESTDIR}/${datadir}/aclocal/
 }
 
-FILES_${PN} = "${libdir}/*${SOLIBS}"
-FILES_${PN}-dev += "${bindir} ${datadir}/wayland"
+# Place wayland-scanner in the -dev package for the target.
+# Leave it in the main package for native so it is included
+# in the SDK.
+FILES_${PN}-dev_append_class-target = " ${bindir} ${datadir}/wayland"
 
 BBCLASSEXTEND = "native nativesdk"
 
-- 
2.17.1


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

View/Reply Online (#136574): 
https://lists.openembedded.org/g/Openembedded-core/message/136574
Mute This Topic: https://lists.openembedded.org/mt/72439819/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] bitbake.conf: Fix nativesdk-wayland missing from SDK

2020-03-19 Thread Tom Hochstein
nativesdk-wayland was not being added to the SDK because it
relies on checking the distro feature. This fixes that by
filtering the wayland feature to the SDK build.

Signed-off-by: Tom Hochstein 
---
 meta/conf/bitbake.conf | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
index 4b544a22cd..e44de34a00 100644
--- a/meta/conf/bitbake.conf
+++ b/meta/conf/bitbake.conf
@@ -830,7 +830,7 @@ DISTRO_FEATURES_NATIVESDK ?= "x11"
 # Normally target distro features will not be applied to native builds:
 # Native distro features on this list will use the target feature value
 DISTRO_FEATURES_FILTER_NATIVE ?= "api-documentation"
-DISTRO_FEATURES_FILTER_NATIVESDK ?= "api-documentation"
+DISTRO_FEATURES_FILTER_NATIVESDK ?= "api-documentation wayland"
 
 DISTRO_FEATURES_BACKFILL = "pulseaudio sysvinit gobject-introspection-data 
ldconfig"
 MACHINE_FEATURES_BACKFILL = "rtc qemu-usermode"
-- 
2.17.1

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


[OE-core] [PATCH 2/2] wayland: Fix wayland-scanner missing from SDK

2020-03-19 Thread Tom Hochstein
The FILES variable is customized to package wayland-scanner
in the target -dev package. However, this is not correct for
the nativesdk packages and causes wayland-scanner to be
missing from the SDK.

Fix the nativesdk packaging by properly limiting the
wayland-scanner target packaging.

Signed-off-by: Tom Hochstein 
---
 meta/recipes-graphics/wayland/wayland_1.18.0.bb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-graphics/wayland/wayland_1.18.0.bb 
b/meta/recipes-graphics/wayland/wayland_1.18.0.bb
index 00be3aac27..c7098483bc 100644
--- a/meta/recipes-graphics/wayland/wayland_1.18.0.bb
+++ b/meta/recipes-graphics/wayland/wayland_1.18.0.bb
@@ -54,8 +54,8 @@ sysroot_stage_all_append_class-target () {
cp ${STAGING_DATADIR_NATIVE}/aclocal/wayland-scanner.m4 
${SYSROOT_DESTDIR}/${datadir}/aclocal/
 }
 
-FILES_${PN} = "${libdir}/*${SOLIBS}"
-FILES_${PN}-dev += "${bindir} ${datadir}/wayland"
+FILES_${PN}_class-target = "${libdir}/*${SOLIBS}"
+FILES_${PN}-dev_append_class-target = " ${bindir} ${datadir}/wayland"
 
 BBCLASSEXTEND = "native nativesdk"
 
-- 
2.17.1

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


[OE-core] [PATCH 2/2] wayland: Fix wayland-scanner missing from SDK

2020-02-08 Thread Tom Hochstein
The FILES variable is customized to package wayland-scanner
in the target -dev package. However, this is not correct for
the nativesdk packages and causes wayland-scanner to be
missing from the SDK.

Fix the nativesdk packaging by properly limiting the
wayland-scanner target packaging.

Signed-off-by: Tom Hochstein 
---
 meta/recipes-graphics/wayland/wayland_1.17.0.bb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-graphics/wayland/wayland_1.17.0.bb 
b/meta/recipes-graphics/wayland/wayland_1.17.0.bb
index 12916a0c40..cb4ff2c20b 100644
--- a/meta/recipes-graphics/wayland/wayland_1.17.0.bb
+++ b/meta/recipes-graphics/wayland/wayland_1.17.0.bb
@@ -43,7 +43,7 @@ sysroot_stage_all_append_class-target () {
cp ${STAGING_DATADIR_NATIVE}/aclocal/wayland-scanner.m4 
${SYSROOT_DESTDIR}/${datadir}/aclocal/
 }
 
-FILES_${PN} = "${libdir}/*${SOLIBS}"
-FILES_${PN}-dev += "${bindir} ${datadir}/wayland"
+FILES_${PN}_class-target = "${libdir}/*${SOLIBS}"
+FILES_${PN}-dev_append_class-target = " ${bindir} ${datadir}/wayland"
 
 BBCLASSEXTEND = "native nativesdk"
-- 
2.17.1

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


[OE-core] [PATCH 1/2] bitbake.conf: Add wayland to DISTRO_FEATURES_FILTER_NATIVESDK

2020-02-08 Thread Tom Hochstein
nativesdk-wayland was not being added to the SDK. This fixes that.

Signed-off-by: Tom Hochstein 
---
 meta/conf/bitbake.conf | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
index 2da7c5849a..a93099bf0c 100644
--- a/meta/conf/bitbake.conf
+++ b/meta/conf/bitbake.conf
@@ -826,7 +826,7 @@ DISTRO_FEATURES_NATIVESDK ?= "x11"
 # Normally target distro features will not be applied to native builds:
 # Native distro features on this list will use the target feature value
 DISTRO_FEATURES_FILTER_NATIVE ?= "api-documentation"
-DISTRO_FEATURES_FILTER_NATIVESDK ?= "api-documentation"
+DISTRO_FEATURES_FILTER_NATIVESDK ?= "api-documentation wayland"
 
 DISTRO_FEATURES_BACKFILL = "pulseaudio sysvinit gobject-introspection-data 
ldconfig"
 MACHINE_FEATURES_BACKFILL = "rtc qemu-usermode"
-- 
2.17.1

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


Re: [OE-core] [PATCH 1/2] devtool/standard.py: Allow recipe to disable menuconfig logic

2019-12-13 Thread Tom Hochstein
I made a PR as well:

https://github.com/openembedded/openembedded-core/pull/57

Tom

> -Original Message-
> From: Schrempf Frieder 
> Sent: Wednesday, December 4, 2019 10:44 AM
> To: Tom Hochstein ; 
> openembedded-core@lists.openembedded.org
> Subject: Re: [OE-core] [PATCH 1/2] devtool/standard.py: Allow recipe to 
> disable menuconfig logic
> 
> Hi,
> 
> On 20.11.19 20:25, Tom Hochstein wrote:
> > u-boot.inc supports u-boot recipes with or without menuconfig [1].
> > However, running devtool on a u-boot recipe that does not support menuconfig
> > results in an error:
> >
> > cp: cannot stat 
> > '/home/r60874/upstream/fsl-xwayland/tmp/work/imx8mmevk-fsl-linux/u-boot-imx/2018.03-r0/u-boot-imx-
> 2018.03//.config': No such file or directory
> >
> > The problem is the devtool logic assumes that any recipe with a 
> > do_menuconfig task
> > will generate a .config in do_configure().
> >
> > Fix the problem by removing the assumption with a flag that the recipe can 
> > control,
> > like this:
> >
> > do_configure() {
> >  if [ menuconfig-supported ]; then
> >  ...
> >  else
> >  DEVTOOL_DISABLE_MENUCONFIG=true
> >  fi
> > }
> >
> > [1] 
> > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fopenembedded%2Fopenembedded-
> core%2Fcommit%2F11278e3b2c75be80645b9841763a97dbb35daadcdata=02%7C01%7Ctom.hochstein%40nxp.com%7C5d668f64
> d3a044abb11e08d778d92bef%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C637110746452407756sdata=AHjvw91G8N7
> Wuf%2BORB%2B5E7cxE5cciCpnEktIqkoFKMY%3Dreserved=0
> >
> > Signed-off-by: Tom Hochstein 
> 
> I'm experiencing this issue since moving to zeus. Your fix in this
> series works fine for me and I would really like to see this merged in zeus.
> 
> Thanks,
> Frieder
> 
> > ---
> >   scripts/lib/devtool/standard.py | 6 --
> >   1 file changed, 4 insertions(+), 2 deletions(-)
> >
> > diff --git a/scripts/lib/devtool/standard.py 
> > b/scripts/lib/devtool/standard.py
> > index 8d9c1a3022..66bd1415c3 100644
> > --- a/scripts/lib/devtool/standard.py
> > +++ b/scripts/lib/devtool/standard.py
> > @@ -940,8 +940,10 @@ def modify(args, config, basepath, workspace):
> >   '}\n')
> >   if rd.getVarFlag('do_menuconfig','task'):
> >   f.write('\ndo_configure_append() {\n'
> > -'cp ${B}/.config ${S}/.config.baseline\n'
> > -'ln -sfT ${B}/.config ${S}/.config.new\n'
> > +'if [ ! ${DEVTOOL_DISABLE_MENUCONFIG} ]; then\n'
> > +'cp ${B}/.config ${S}/.config.baseline\n'
> > +'ln -sfT ${B}/.config ${S}/.config.new\n'
> > +'fi\n'
> >   '}\n')
> >   if initial_rev:
> >   f.write('\n# initial_rev: %s\n' % initial_rev)
> >
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 1/2] devtool/standard.py: Allow recipe to disable menuconfig logic

2019-11-21 Thread Tom Hochstein



> -Original Message-
> From: Peter Kjellerstedt 
> Sent: Thursday, November 21, 2019 4:24 AM
> To: Tom Hochstein ; 
> openembedded-core@lists.openembedded.org
> Subject: RE: [OE-core] [PATCH 1/2] devtool/standard.py: Allow recipe to 
> disable menuconfig logic
> 
> > -Original Message-
> > From: openembedded-core-boun...@lists.openembedded.org  > boun...@lists.openembedded.org> On Behalf Of Tom Hochstein
> > Sent: den 20 november 2019 20:26
> > To: openembedded-core@lists.openembedded.org
> > Subject: [OE-core] [PATCH 1/2] devtool/standard.py: Allow recipe to
> > disable menuconfig logic
> >
> > @@ -940,8 +940,10 @@ def modify(args, config, basepath, workspace):
> >  '}\n')
> >  if rd.getVarFlag('do_menuconfig','task'):
> >  f.write('\ndo_configure_append() {\n'
> > -'cp ${B}/.config ${S}/.config.baseline\n'
> > -'ln -sfT ${B}/.config ${S}/.config.new\n'
> > +'if [ ! ${DEVTOOL_DISABLE_MENUCONFIG} ]; then\n'
> > +'cp ${B}/.config ${S}/.config.baseline\n'
> > +'ln -sfT ${B}/.config ${S}/.config.new\n'
> > +'fi\n'
> 
> Why do you need the extra variable? Why not just check if the .config
> file exists before copying it:
> 
> 'if -e ${B}/.config; then\n'
> 'cp ${B}/.config ${S}/.config.baseline\n'
> 'ln -sfT ${B}/.config ${S}/.config.new\n'
> 'fi\n'
> 
> >  '}\n')
> >  if initial_rev:
> >  f.write('\n# initial_rev: %s\n' % initial_rev)
> > --
> > 2.17.1
> 
> //Peter

I wanted to preserve the existing error handling in the case that menuconfig is 
supported and .config is unexpectedly missing. Having the cp fail immediately 
seems best.

Tom
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH 2/2] u-boot.inc: Fix devtool build u-boot for u-boot without menuconfig

2019-11-20 Thread Tom Hochstein
For u-boot recipes without menuconfig support, running devtool results
in a do_configure error:

cp: cannot stat 
'/home/r60874/upstream/fsl-xwayland/tmp/work/imx8mmevk-fsl-linux/u-boot-imx/2018.03-r0/u-boot-imx-2018.03//.config':
 No such file or directory

The problem arises because u-boot.inc supports recipes with and without
menuconfig.

Fix the problem by properly setting DEVTOOL_DISABLE_MENUCONFIG so that devtool
can control logic that applies only for menuconfig support.

Signed-off-by: Tom Hochstein 
---
 meta/recipes-bsp/u-boot/u-boot.inc | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/meta/recipes-bsp/u-boot/u-boot.inc 
b/meta/recipes-bsp/u-boot/u-boot.inc
index 9a754fd09b..d241347bf7 100644
--- a/meta/recipes-bsp/u-boot/u-boot.inc
+++ b/meta/recipes-bsp/u-boot/u-boot.inc
@@ -87,6 +87,8 @@ do_configure () {
 fi
 merge_config.sh -m .config ${@" ".join(find_cfgs(d))}
 cml1_do_configure
+else
+DEVTOOL_DISABLE_MENUCONFIG=true
 fi
 }
 
-- 
2.17.1

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


[OE-core] [PATCH 1/2] devtool/standard.py: Allow recipe to disable menuconfig logic

2019-11-20 Thread Tom Hochstein
u-boot.inc supports u-boot recipes with or without menuconfig [1].
However, running devtool on a u-boot recipe that does not support menuconfig
results in an error:

cp: cannot stat 
'/home/r60874/upstream/fsl-xwayland/tmp/work/imx8mmevk-fsl-linux/u-boot-imx/2018.03-r0/u-boot-imx-2018.03//.config':
 No such file or directory

The problem is the devtool logic assumes that any recipe with a do_menuconfig 
task
will generate a .config in do_configure().

Fix the problem by removing the assumption with a flag that the recipe can 
control,
like this:

do_configure() {
if [ menuconfig-supported ]; then
...
else
DEVTOOL_DISABLE_MENUCONFIG=true
fi
}

[1] 
https://github.com/openembedded/openembedded-core/commit/11278e3b2c75be80645b9841763a97dbb35daadc

Signed-off-by: Tom Hochstein 
---
 scripts/lib/devtool/standard.py | 6 --
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/scripts/lib/devtool/standard.py b/scripts/lib/devtool/standard.py
index 8d9c1a3022..66bd1415c3 100644
--- a/scripts/lib/devtool/standard.py
+++ b/scripts/lib/devtool/standard.py
@@ -940,8 +940,10 @@ def modify(args, config, basepath, workspace):
 '}\n')
 if rd.getVarFlag('do_menuconfig','task'):
 f.write('\ndo_configure_append() {\n'
-'cp ${B}/.config ${S}/.config.baseline\n'
-'ln -sfT ${B}/.config ${S}/.config.new\n'
+'if [ ! ${DEVTOOL_DISABLE_MENUCONFIG} ]; then\n'
+'cp ${B}/.config ${S}/.config.baseline\n'
+'ln -sfT ${B}/.config ${S}/.config.new\n'
+'fi\n'
 '}\n')
 if initial_rev:
 f.write('\n# initial_rev: %s\n' % initial_rev)
-- 
2.17.1

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


Re: [OE-core] python3-native certificate failure

2018-04-19 Thread Tom Hochstein
Thanks, Ross. That is indeed what we are doing now.

Tom

-Original Message-
From: Burton, Ross [mailto:ross.bur...@intel.com] 
Sent: Thursday, April 19, 2018 5:36 AM
To: Tom Hochstein <tom.hochst...@nxp.com>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [OE-core] python3-native certificate failure

Why don't you just use the host Python?

This is probably due to the certificate path not being correctly remapped as it 
goes in/out of sstate, you'll have to tell the SSL code explicitly where to 
find the certificates.

Ross

On 4 April 2018 at 21:09, Tom Hochstein <tom.hochst...@nxp.com> wrote:
> I am trying to download a URL from a source package using 
> python3-native, but it is failing with a certificate failure. I added 
> DEPENDS on ca-certificates-native and python3-certifi-native, but it 
> doesn’t seem to help. A colleague with a corresponding SDK took the 
> aarch64 sysroot certificates and dropped them in x86_64 sysroot and the 
> problem was fixed.
>
>
>
> Tom
>
>
>
> r60874@tx30imx-01:~/rocko/build-xwayland$ bitbake imx-gpu-sdk -c 
> devshell
>
> root@tx30imx-01:~/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-lin
> ux/imx-gpu-sdk/5.0.2-r0/git#
> python3
>
> Python 3.5.3 (default, Mar  6 2018, 14:15:44)
>
> [GCC 5.4.0 20160609] on linux
>
> Type "help", "copyright", "credits" or "license" for more information.
>
>>>> import urllib.request
>
>>>>
>>>> urllib.request.urlretrieve("https://emea01.safelinks.protection.out
>>>> look.com/?url=https%3A%2F%2Fsourceforge.net%2Fprojects%2Ftclap%2Ffi
>>>> les%2Ftclap-1.2.2.tar.gz=02%7C01%7Ctom.hochstein%40nxp.com%7C7
>>>> 096802f05364674162908d5a5e161a0%7C686ea1d3bc2b4c6fa92cd99c5c301635%
>>>> 7C0%7C0%7C636597309792188874=sM3F9XsfnAyJALQv3zOjm4fOiLiV6DuY
>>>> jCg%2FOoD9GHM%3D=0",
>>>> "~")
>
> Traceback (most recent call last):
>
>   File
> "/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/i
> mx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/urllib/req
> uest.py",
> line 1254, in do_open
>
> h.request(req.get_method(), req.selector, req.data, headers)
>
>   File
> "/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/i
> mx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/http/clien
> t.py",
> line 1107, in request
>
> self._send_request(method, url, body, headers)
>
>   File
> "/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/i
> mx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/http/clien
> t.py",
> line 1152, in _send_request
>
> self.endheaders(body)
>
>   File
> "/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/i
> mx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/http/clien
> t.py",
> line 1103, in endheaders
>
> self._send_output(message_body)
>
>   File
> "/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/i
> mx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/http/clien
> t.py",
> line 934, in _send_output
>
> self.send(msg)
>
>   File
> "/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/i
> mx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/http/clien
> t.py",
> line 877, in send
>
> self.connect()
>
>   File
> "/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/i
> mx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/http/clien
> t.py",
> line 1261, in connect
>
> server_hostname=server_hostname)
>
>   File
> "/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/i
> mx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/ssl.py",
> line 385, in wrap_socket
>
> _context=self)
>
>   File
> "/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/i
> mx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/ssl.py",
> line 760, in __init__
>
> self.do_handshake()
>
>   File
> "/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/i
> mx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/ssl.py",
> line 996, in do_handshake
>
> self._sslobj.do_handshake()
>
>   File
> "/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/i
> mx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/ssl.py",
> line 641, in do_handshake
>
> self._sslobj.do_handshake()
>
> ssl.SSLError: [SSL: CERTIFICATE_VERIFY_FAILED] c

[OE-core] [PATCH v2] python3: Add recommended modules to nativesdk install

2018-04-09 Thread Tom Hochstein
The python3 installation in the SDK did not include the minimum set
of modules to be functional, particularly in the case where Python
is brought in through dependencies. Rather than requiring the user
to explicitly add the modules, it's better to pull in the modules
through RRECOMMENDS. Note that the Python 2 recipe already does
this.

Signed-off-by: Tom Hochstein <tom.hochst...@nxp.com>
---
 meta/recipes-devtools/python/python3_3.5.5.bb | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-devtools/python/python3_3.5.5.bb 
b/meta/recipes-devtools/python/python3_3.5.5.bb
index d458d32..f893b84 100644
--- a/meta/recipes-devtools/python/python3_3.5.5.bb
+++ b/meta/recipes-devtools/python/python3_3.5.5.bb
@@ -211,6 +211,7 @@ py_package_preprocess () {
 
 # manual dependency additions
 RPROVIDES_${PN}-modules = "${PN}"
+RRECOMMENDS_${PN}-core_append_class-nativesdk = " nativesdk-python3-modules"
 RRECOMMENDS_${PN}-crypt = "openssl"
 RRECOMMENDS_${PN}-crypt_class-nativesdk = "nativesdk-openssl"
 
-- 
2.7.4

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


Re: [OE-core] [PATCH] python3: Add recommended modules to nativesdk install

2018-04-07 Thread Tom Hochstein
A colleague pointed out that what we installed in the SDK for python 3 was not 
functional, and that a minimal set of modules was required.  I'm not an expert, 
but I found a similar line in the python 2 recipe and thought it might be what 
was needed here.

Tom

-Original Message-
From: Richard Purdie [mailto:richard.pur...@linuxfoundation.org] 
Sent: Thursday, April 5, 2018 8:56 AM
To: Tom Hochstein <tom.hochst...@nxp.com>; 
openembedded-core@lists.openembedded.org
Subject: Re: [OE-core] [PATCH] python3: Add recommended modules to nativesdk 
install

On Wed, 2018-04-04 at 14:25 -0500, Tom Hochstein wrote:
> The python3 installation in the SDK did not include the minimum set of 
> modules.
> 
> Signed-off-by: Tom Hochstein <tom.hochst...@nxp.com>
> ---
>  meta/recipes-devtools/python/python3_3.5.5.bb | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/meta/recipes-devtools/python/python3_3.5.5.bb
> b/meta/recipes-devtools/python/python3_3.5.5.bb
> index d458d32..f893b84 100644
> --- a/meta/recipes-devtools/python/python3_3.5.5.bb
> +++ b/meta/recipes-devtools/python/python3_3.5.5.bb
> @@ -211,6 +211,7 @@ py_package_preprocess () {
>  
>  # manual dependency additions
>  RPROVIDES_${PN}-modules = "${PN}"
> +RRECOMMENDS_${PN}-core_append_class-nativesdk = " nativesdk-python3-
> modules"
>  RRECOMMENDS_${PN}-crypt = "openssl"
>  RRECOMMENDS_${PN}-crypt_class-nativesdk = "nativesdk-openssl"

This doesn't look correct, if you want the SDK to contain all python modules, 
surely you'd just add nativesdk-python3-modules rather than forcing this 
everywhere?

Cheers,

Richard
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] python3-native certificate failure

2018-04-05 Thread Tom Hochstein
But I am not using an SDK, I'm doing a straight bitbake. Using the native 
Python3 urllib.request.urlopen (via inherit python3native) from do_compile I 
see the certificate error. My original email duplicated the issue manually 
using -c devshell.

Tom

-Original Message-
From: Khem Raj [mailto:raj.k...@gmail.com] 
Sent: Wednesday, April 4, 2018 7:29 PM
To: Tom Hochstein <tom.hochst...@nxp.com>; 
openembedded-core@lists.openembedded.org
Subject: Re: [OE-core] python3-native certificate failure

On 4/4/18 1:09 PM, Tom Hochstein wrote:
> I am trying to download a URL from a source package using 
> python3-native, but it is failing with a certificate failure. I added 
> DEPENDS on ca-certificates-native and python3-certifi-native, but it 
> doesn't seem to help. A colleague with a corresponding SDK took the
> aarch64 sysroot certificates and dropped them in x86_64 sysroot and 
> the problem was fixed.
> 
>  

you might try adding nativesdk-ca-certificates to TOOLCHAIN_HOST_TASK and 
regenerate SDK

> 
> Tom
> 
>  
> 
> r60874@tx30imx-01:~/rocko/build-xwayland$ bitbake imx-gpu-sdk -c 
> devshell
> 
> root@tx30imx-01:~/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-lin
> ux/imx-gpu-sdk/5.0.2-r0/git#
> python3
> 
> Python 3.5.3 (default, Mar  6 2018, 14:15:44)
> 
> [GCC 5.4.0 20160609] on linux
> 
> Type "help", "copyright", "credits" or "license" for more information.
> 
>>>> import urllib.request
> 
>>>>
> urllib.request.urlretrieve("https://emea01.safelinks.protection.outloo
> k.com/?url=https%3A%2F%2Fsourceforge.net%2Fprojects%2Ftclap%2Ffiles%2F
> tclap-1.2.2.tar.gz=02%7C01%7Ctom.hochstein%40nxp.com%7Ca40b2e05d7
> aa43d50fc108d59a8c3bf6%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C63
> 6584849450785202=2BmKu3tXThT6%2FsHG0CzQJNPvXQ3gq2GJpahu2t8PNCY%3
> D=0",
> "~")
> 
> Traceback (most recent call last):
> 
>   File
> "/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/i
> mx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/urllib/req
> uest.py",
> line 1254, in do_open
> 
>     h.request(req.get_method(), req.selector, req.data, headers)
> 
>   File
> "/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/i
> mx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/http/clien
> t.py",
> line 1107, in request
> 
>     self._send_request(method, url, body, headers)
> 
>   File
> "/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/i
> mx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/http/clien
> t.py",
> line 1152, in _send_request
> 
>     self.endheaders(body)
> 
>   File
> "/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/i
> mx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/http/clien
> t.py",
> line 1103, in endheaders
> 
>     self._send_output(message_body)
> 
>   File
> "/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/i
> mx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/http/clien
> t.py",
> line 934, in _send_output
> 
>     self.send(msg)
> 
>   File
> "/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/i
> mx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/http/clien
> t.py",
> line 877, in send
> 
>     self.connect()
> 
>   File
> "/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/i
> mx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/http/clien
> t.py",
> line 1261, in connect
> 
>     server_hostname=server_hostname)
> 
>   File
> "/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/i
> mx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/ssl.py",
> line 385, in wrap_socket
> 
>     _context=self)
> 
>   File
> "/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/i
> mx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/ssl.py",
> line 760, in __init__
> 
>     self.do_handshake()
> 
>   File
> "/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/i
> mx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/ssl.py",
> line 996, in do_handshake
> 
>     self._sslobj.do_handshake()
> 
>   File
> "/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/i
> mx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/ssl.py",
> line 641, in do_handshake
> 
>     self._sslobj.do_handshake()
> 
> ssl.SSLError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify 
> failed
>

[OE-core] python3-native certificate failure

2018-04-04 Thread Tom Hochstein
I am trying to download a URL from a source package using python3-native, but 
it is failing with a certificate failure. I added DEPENDS on 
ca-certificates-native and python3-certifi-native, but it doesn't seem to help. 
A colleague with a corresponding SDK took the aarch64 sysroot certificates and 
dropped them in x86_64 sysroot and the problem was fixed.

Tom

r60874@tx30imx-01:~/rocko/build-xwayland$ bitbake imx-gpu-sdk -c devshell
root@tx30imx-01:~/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/imx-gpu-sdk/5.0.2-r0/git#
 python3
Python 3.5.3 (default, Mar  6 2018, 14:15:44)
[GCC 5.4.0 20160609] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import urllib.request
>>> urllib.request.urlretrieve("https://sourceforge.net/projects/tclap/files/tclap-1.2.2.tar.gz;,
>>>  "~")
Traceback (most recent call last):
  File 
"/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/imx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/urllib/request.py",
 line 1254, in do_open
h.request(req.get_method(), req.selector, req.data, headers)
  File 
"/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/imx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/http/client.py",
 line 1107, in request
self._send_request(method, url, body, headers)
  File 
"/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/imx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/http/client.py",
 line 1152, in _send_request
self.endheaders(body)
  File 
"/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/imx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/http/client.py",
 line 1103, in endheaders
self._send_output(message_body)
  File 
"/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/imx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/http/client.py",
 line 934, in _send_output
self.send(msg)
  File 
"/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/imx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/http/client.py",
 line 877, in send
self.connect()
  File 
"/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/imx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/http/client.py",
 line 1261, in connect
server_hostname=server_hostname)
  File 
"/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/imx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/ssl.py",
 line 385, in wrap_socket
_context=self)
  File 
"/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/imx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/ssl.py",
 line 760, in __init__
self.do_handshake()
  File 
"/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/imx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/ssl.py",
 line 996, in do_handshake
self._sslobj.do_handshake()
  File 
"/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/imx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/ssl.py",
 line 641, in do_handshake
self._sslobj.do_handshake()
ssl.SSLError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed 
(_ssl.c:720)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "", line 1, in 
  File 
"/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/imx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/urllib/request.py",
 line 188, in urlretrieve
with contextlib.closing(urlopen(url, data)) as fp:
  File 
"/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/imx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/urllib/request.py",
 line 163, in urlopen
return opener.open(url, data, timeout)
  File 
"/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/imx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/urllib/request.py",
 line 466, in open
response = self._open(req, data)
  File 
"/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/imx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/urllib/request.py",
 line 484, in _open
'_open', req)
  File 
"/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/imx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/urllib/request.py",
 line 444, in _call_chain
result = func(*args)
  File 
"/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/imx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/urllib/request.py",
 line 1297, in https_open
context=self._context, check_hostname=self._check_hostname)
  File 
"/home/r60874/rocko/build-xwayland/tmp/work/aarch64-mx8qm-poky-linux/imx-gpu-sdk/5.0.2-r0/recipe-sysroot-native/usr/lib/python3.5/urllib/request.py",
 line 1256, in do_open
raise URLError(err)
urllib.error.URLError: 
-- 
___

[OE-core] [PATCH] python3: Add recommended modules to nativesdk install

2018-04-04 Thread Tom Hochstein
The python3 installation in the SDK did not include the minimum set
of modules.

Signed-off-by: Tom Hochstein <tom.hochst...@nxp.com>
---
 meta/recipes-devtools/python/python3_3.5.5.bb | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-devtools/python/python3_3.5.5.bb 
b/meta/recipes-devtools/python/python3_3.5.5.bb
index d458d32..f893b84 100644
--- a/meta/recipes-devtools/python/python3_3.5.5.bb
+++ b/meta/recipes-devtools/python/python3_3.5.5.bb
@@ -211,6 +211,7 @@ py_package_preprocess () {
 
 # manual dependency additions
 RPROVIDES_${PN}-modules = "${PN}"
+RRECOMMENDS_${PN}-core_append_class-nativesdk = " nativesdk-python3-modules"
 RRECOMMENDS_${PN}-crypt = "openssl"
 RRECOMMENDS_${PN}-crypt_class-nativesdk = "nativesdk-openssl"
 
-- 
2.7.4

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


[OE-core] [PATCH 2/2] archiver: Fix archive for linked kernel source

2018-04-03 Thread Tom Hochstein
When archiving a kernel, if S is ${WORKDIR}/git, then
${WORKDIR}/git is in fact a soft link into work-shared,
and the archive contains just the soft link. Fix by
archiving the real directory.

Signed-off-by: Tom Hochstein <tom.hochst...@nxp.com>
---
 meta/classes/archiver.bbclass | 5 +
 1 file changed, 5 insertions(+)

diff --git a/meta/classes/archiver.bbclass b/meta/classes/archiver.bbclass
index c107615..31c9b7e 100644
--- a/meta/classes/archiver.bbclass
+++ b/meta/classes/archiver.bbclass
@@ -277,6 +277,11 @@ def create_tarball(d, srcdir, suffix, ar_outdir):
 if (d.getVar('SRC_URI') == ""):
 return
 
+# For the kernel archive, srcdir may just be a link to the
+# work-shared location. Use os.path.realpath to make sure
+# that we archive the actual directory and not just the link.
+srcdir = os.path.realpath(srcdir)
+
 bb.utils.mkdirhier(ar_outdir)
 if suffix:
 filename = '%s-%s.tar.gz' % (d.getVar('PF'), suffix)
-- 
2.7.4

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


[OE-core] [PATCH 1/2] archiver: Fix build errors for kernels that don't use kernel-yocto.bbclass

2018-04-03 Thread Tom Hochstein
Special handling for kernel source in work-shared was
incorrectly limited to kernel-yocto.bbclass instead
of kernel.bbclass.

Signed-off-by: Tom Hochstein <tom.hochst...@nxp.com>
---
 meta/classes/archiver.bbclass | 8 ++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/meta/classes/archiver.bbclass b/meta/classes/archiver.bbclass
index e17e6b4..c107615 100644
--- a/meta/classes/archiver.bbclass
+++ b/meta/classes/archiver.bbclass
@@ -319,6 +319,10 @@ def create_diff_gz(d, src_orig, src, ar_outdir):
 finally:
 os.chdir(cwd)
 
+def is_work_shared(d):
+pn = d.getVar('PN')
+return bb.data.inherits_class('kernel', d) or pn.startswith('gcc-source')
+
 # Run do_unpack and do_patch
 python do_unpack_and_patch() {
 if d.getVarFlag('ARCHIVER_MODE', 'src') not in \
@@ -331,7 +335,7 @@ python do_unpack_and_patch() {
 pn = d.getVar('PN')
 
 # The kernel class functions require it to be on work-shared, so we dont 
change WORKDIR
-if not (bb.data.inherits_class('kernel-yocto', d) or 
pn.startswith('gcc-source')):
+if not is_work_shared(d):
 # Change the WORKDIR to make do_unpack do_patch run in another dir.
 d.setVar('WORKDIR', ar_workdir)
 # Restore the original path to recipe's native sysroot (it's relative 
to WORKDIR).
@@ -351,7 +355,7 @@ python do_unpack_and_patch() {
 oe.path.copytree(src, src_orig)
 
 # Make sure gcc and kernel sources are patched only once
-if not (d.getVar('SRC_URI') == "" or 
(bb.data.inherits_class('kernel-yocto', d) or pn.startswith('gcc-source'))):
+if not (d.getVar('SRC_URI') == "" or is_work_shared(d)):
 bb.build.exec_func('do_patch', d)
 
 # Create the patches
-- 
2.7.4

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


[OE-core] [PATCH v2] libepoxy: Fix build break for EGL_CAST dependency

2017-11-18 Thread Tom Hochstein
This is a backport [1] to fix the following build error:

| In file included from 
/home/r60874/upstream/xwayland/tmp/work/armv7at2hf-neon-mx6qdl-fslc-linux-gnueabi/gtk+3/3.22.17-r0/recipe-sysroot/usr/include/epoxy/egl.h:46:0,
|  from 
../../../gtk+-3.22.17/gdk/wayland/gdkglcontext-wayland.h:32,
|  from 
../../../gtk+-3.22.17/gdk/wayland/gdkglcontext-wayland.c:24:
| ../../../gtk+-3.22.17/gdk/wayland/gdkglcontext-wayland.c: In function 
'gdk_wayland_gl_context_realize':
| ../../../gtk+-3.22.17/gdk/wayland/gdkglcontext-wayland.c:179:43: error: 
expected expression before 'EGLContext'
|  : EGL_NO_CONTEXT,
|^

[1] 
https://github.com/anholt/libepoxy/commit/ebe3a53db1c0bb34e1ca963b95d1f222115f93f8

Signed-off-by: Tom Hochstein <tom.hochst...@nxp.com>
---
 .../Add-fallback-definition-for-EGL-CAST.patch | 33 ++
 meta/recipes-graphics/libepoxy/libepoxy_1.4.3.bb   |  3 +-
 2 files changed, 35 insertions(+), 1 deletion(-)
 create mode 100644 
meta/recipes-graphics/libepoxy/libepoxy/Add-fallback-definition-for-EGL-CAST.patch

diff --git 
a/meta/recipes-graphics/libepoxy/libepoxy/Add-fallback-definition-for-EGL-CAST.patch
 
b/meta/recipes-graphics/libepoxy/libepoxy/Add-fallback-definition-for-EGL-CAST.patch
new file mode 100644
index 000..b929725
--- /dev/null
+++ 
b/meta/recipes-graphics/libepoxy/libepoxy/Add-fallback-definition-for-EGL-CAST.patch
@@ -0,0 +1,33 @@
+Add fallback definition for EGL_CAST
+
+The EGL API update from d11104f introduced a dependency on the
+EGL_CAST() macro, provided by an updated eglplatform.h. Given that we
+don't provide eglplatform.h, add a fallback definition for if we're
+building against Mesa 17.0.x or similar.
+
+https://bugs.gentoo.org/show_bug.cgi?id=623926
+
+Upstream-Status: Backport 
[https://github.com/anholt/libepoxy/commit/ebe3a53db1c0bb34e1ca963b95d1f222115f93f8]
+
+Signed-off-by: Tom Hochstein <tom.hochst...@nxp.com>
+
+Index: libepoxy-1.4.3/src/gen_dispatch.py
+===
+--- libepoxy-1.4.3.orig/src/gen_dispatch.py2017-06-06 04:24:13.0 
-0500
 libepoxy-1.4.3/src/gen_dispatch.py 2017-11-06 12:45:43.594966473 -0600
+@@ -491,6 +491,15 @@
+ self.outln('#include "epoxy/gl.h"')
+ if self.target == "egl":
+ self.outln('#include "EGL/eglplatform.h"')
++# Account for older eglplatform.h, which doesn't define
++# the EGL_CAST macro.
++self.outln('#ifndef EGL_CAST')
++self.outln('#if defined(__cplusplus)')
++self.outln('#define EGL_CAST(type, value) 
(static_cast(value))')
++self.outln('#else')
++self.outln('#define EGL_CAST(type, value) ((type) (value))')
++self.outln('#endif')
++self.outln('#endif')
+ else:
+ # Add some ridiculous inttypes.h redefinitions that are
+ # from khrplatform.h and not included in the XML.  We
diff --git a/meta/recipes-graphics/libepoxy/libepoxy_1.4.3.bb 
b/meta/recipes-graphics/libepoxy/libepoxy_1.4.3.bb
index c8b398f..0172322 100644
--- a/meta/recipes-graphics/libepoxy/libepoxy_1.4.3.bb
+++ b/meta/recipes-graphics/libepoxy/libepoxy_1.4.3.bb
@@ -5,7 +5,8 @@ SECTION = "libs"
 LICENSE = "MIT"
 LIC_FILES_CHKSUM = "file://COPYING;md5=58ef4c80d401e07bd9ee8b6b58cf464b"
 
-SRC_URI = 
"https://github.com/anholt/${BPN}/releases/download/${PV}/${BP}.tar.xz;
+SRC_URI = 
"https://github.com/anholt/${BPN}/releases/download/${PV}/${BP}.tar.xz \
+   file://Add-fallback-definition-for-EGL-CAST.patch"
 SRC_URI[md5sum] = "af4c3ce0fb1143bdc4e43f85695a9bed"
 SRC_URI[sha256sum] = 
"0b808a06c9685a62fca34b680abb8bc7fb2fda074478e329b063c1f872b826f6"
 UPSTREAM_CHECK_URI = "https://github.com/anholt/libepoxy/releases;
-- 
1.9.1

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


[OE-core] [PATCH] libepoxy: Fix build break for EGL_CAST dependency

2017-11-16 Thread Tom Hochstein
This is a backport [1] to fix the following build error:

| In file included from 
/home/r60874/upstream/xwayland/tmp/work/armv7at2hf-neon-mx6qdl-fslc-linux-gnueabi/gtk+3/3.22.17-r0/recipe-sysroot/usr/include/epoxy/egl.h:46:0,
|  from 
../../../gtk+-3.22.17/gdk/wayland/gdkglcontext-wayland.h:32,
|  from 
../../../gtk+-3.22.17/gdk/wayland/gdkglcontext-wayland.c:24:
| ../../../gtk+-3.22.17/gdk/wayland/gdkglcontext-wayland.c: In function 
'gdk_wayland_gl_context_realize':
| ../../../gtk+-3.22.17/gdk/wayland/gdkglcontext-wayland.c:179:43: error: 
expected expression before 'EGLContext'
|  : EGL_NO_CONTEXT,
|^

[1] 
https://github.com/anholt/libepoxy/commit/ebe3a53db1c0bb34e1ca963b95d1f222115f93f8

Signed-off-by: Tom Hochstein <tom.hochst...@nxp.com>
---
 .../Add-fallback-definition-for-EGL-CAST.patch | 31 ++
 meta/recipes-graphics/libepoxy/libepoxy_1.4.3.bb   |  3 ++-
 2 files changed, 33 insertions(+), 1 deletion(-)
 create mode 100644 
meta/recipes-graphics/libepoxy/libepoxy/Add-fallback-definition-for-EGL-CAST.patch

diff --git 
a/meta/recipes-graphics/libepoxy/libepoxy/Add-fallback-definition-for-EGL-CAST.patch
 
b/meta/recipes-graphics/libepoxy/libepoxy/Add-fallback-definition-for-EGL-CAST.patch
new file mode 100644
index 000..daa1785
--- /dev/null
+++ 
b/meta/recipes-graphics/libepoxy/libepoxy/Add-fallback-definition-for-EGL-CAST.patch
@@ -0,0 +1,31 @@
+Add fallback definition for EGL_CAST
+
+The EGL API update from d11104f introduced a dependency on the
+EGL_CAST() macro, provided by an updated eglplatform.h. Given that we
+don't provide eglplatform.h, add a fallback definition for if we're
+building against Mesa 17.0.x or similar.
+
+https://bugs.gentoo.org/show_bug.cgi?id=623926
+
+Upstream-Status: Backport 
[https://github.com/anholt/libepoxy/commit/ebe3a53db1c0bb34e1ca963b95d1f222115f93f8]
+
+Index: libepoxy-1.4.3/src/gen_dispatch.py
+===
+--- libepoxy-1.4.3.orig/src/gen_dispatch.py2017-06-06 04:24:13.0 
-0500
 libepoxy-1.4.3/src/gen_dispatch.py 2017-11-06 12:45:43.594966473 -0600
+@@ -491,6 +491,15 @@
+ self.outln('#include "epoxy/gl.h"')
+ if self.target == "egl":
+ self.outln('#include "EGL/eglplatform.h"')
++# Account for older eglplatform.h, which doesn't define
++# the EGL_CAST macro.
++self.outln('#ifndef EGL_CAST')
++self.outln('#if defined(__cplusplus)')
++self.outln('#define EGL_CAST(type, value) 
(static_cast(value))')
++self.outln('#else')
++self.outln('#define EGL_CAST(type, value) ((type) (value))')
++self.outln('#endif')
++self.outln('#endif')
+ else:
+ # Add some ridiculous inttypes.h redefinitions that are
+ # from khrplatform.h and not included in the XML.  We
diff --git a/meta/recipes-graphics/libepoxy/libepoxy_1.4.3.bb 
b/meta/recipes-graphics/libepoxy/libepoxy_1.4.3.bb
index c8b398f..0172322 100644
--- a/meta/recipes-graphics/libepoxy/libepoxy_1.4.3.bb
+++ b/meta/recipes-graphics/libepoxy/libepoxy_1.4.3.bb
@@ -5,7 +5,8 @@ SECTION = "libs"
 LICENSE = "MIT"
 LIC_FILES_CHKSUM = "file://COPYING;md5=58ef4c80d401e07bd9ee8b6b58cf464b"
 
-SRC_URI = 
"https://github.com/anholt/${BPN}/releases/download/${PV}/${BP}.tar.xz;
+SRC_URI = 
"https://github.com/anholt/${BPN}/releases/download/${PV}/${BP}.tar.xz \
+   file://Add-fallback-definition-for-EGL-CAST.patch"
 SRC_URI[md5sum] = "af4c3ce0fb1143bdc4e43f85695a9bed"
 SRC_URI[sha256sum] = 
"0b808a06c9685a62fca34b680abb8bc7fb2fda074478e329b063c1f872b826f6"
 UPSTREAM_CHECK_URI = "https://github.com/anholt/libepoxy/releases;
-- 
1.9.1

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


[OE-core] [PATCH v3] xserver-xorg: Fix X server 1.19 crash

2017-02-01 Thread Tom Hochstein
Commit 6a5a4e60373c1386b311b2a8bb666c32d68a9d99 removes the configure of
useSIGIO option:

xfree86 SIGIO support is reworked to use internal versions of
OsBlockSIGIO and OsReleaseSIGIO

The check for useSIGIO is no longer needed.

Signed-off-by: Tom Hochstein <tom.hochst...@nxp.com>
---
 .../0003-Remove-check-for-useSIGIO-option.patch| 47 ++
 .../xorg-xserver/xserver-xorg_1.19.1.bb|  1 +
 2 files changed, 48 insertions(+)
 create mode 100644 
meta/recipes-graphics/xorg-xserver/xserver-xorg/0003-Remove-check-for-useSIGIO-option.patch

diff --git 
a/meta/recipes-graphics/xorg-xserver/xserver-xorg/0003-Remove-check-for-useSIGIO-option.patch
 
b/meta/recipes-graphics/xorg-xserver/xserver-xorg/0003-Remove-check-for-useSIGIO-option.patch
new file mode 100644
index 000..beed6cb
--- /dev/null
+++ 
b/meta/recipes-graphics/xorg-xserver/xserver-xorg/0003-Remove-check-for-useSIGIO-option.patch
@@ -0,0 +1,47 @@
+From cf407b16cd65ad6e26a9c8e5984e163409a5c0f7 Mon Sep 17 00:00:00 2001
+From: Prabhu Sundararaj <prabhu.sundara...@nxp.com>
+Date: Mon, 30 Jan 2017 16:32:06 -0600
+Subject: [PATCH] Remove check for useSIGIO option
+
+Commit 6a5a4e60373c1386b311b2a8bb666c32d68a9d99 removes the configure of 
useSIGIO
+option.
+
+As the xfree86 SIGIO support is reworked to use internal versions of 
OsBlockSIGIO
+and OsReleaseSIGIO.
+
+No longer the check for useSIGIO is needed
+
+Upstream-Status: Pending
+
+Signed-off-by: Prabhu Sundararaj <prabhu.sundara...@nxp.com>
+---
+ hw/xfree86/os-support/shared/sigio.c | 6 --
+ 1 file changed, 6 deletions(-)
+
+diff --git a/hw/xfree86/os-support/shared/sigio.c 
b/hw/xfree86/os-support/shared/sigio.c
+index 884a71c..be76498 100644
+--- a/hw/xfree86/os-support/shared/sigio.c
 b/hw/xfree86/os-support/shared/sigio.c
+@@ -185,9 +185,6 @@ xf86InstallSIGIOHandler(int fd, void (*f) (int, void *), 
void *closure)
+ int i;
+ int installed = FALSE;
+ 
+-if (!xf86Info.useSIGIO)
+-return 0;
+-
+ for (i = 0; i < MAX_FUNCS; i++) {
+ if (!xf86SigIOFuncs[i].f) {
+ if (xf86IsPipe(fd))
+@@ -256,9 +253,6 @@ xf86RemoveSIGIOHandler(int fd)
+ int max;
+ int ret;
+ 
+-if (!xf86Info.useSIGIO)
+-return 0;
+-
+ max = 0;
+ ret = 0;
+ for (i = 0; i < MAX_FUNCS; i++) {
+-- 
+2.7.4
+
diff --git a/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.19.1.bb 
b/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.19.1.bb
index 987a2be..5a657e0 100644
--- a/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.19.1.bb
+++ b/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.19.1.bb
@@ -3,6 +3,7 @@ require xserver-xorg.inc
 SRC_URI += "file://musl-arm-inb-outb.patch \
 file://0001-configure.ac-Fix-check-for-CLOCK_MONOTONIC.patch \
 
file://0002-configure.ac-Fix-wayland-scanner-and-protocols-locat.patch \
+file://0003-Remove-check-for-useSIGIO-option.patch \
 "
 SRC_URI[md5sum] = "caa8ee7b2950abbf734347d137529fb6"
 SRC_URI[sha256sum] = 
"79ae2cf39d3f6c4a91201d8dad549d1d774b3420073c5a70d390040aa965a7fb"
-- 
1.9.1

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


Re: [OE-core] [PATCH v2] xserver-xorg: Fix X server 1.19 crash

2017-02-01 Thread Tom Hochstein
https://lists.yoctoproject.org/pipermail/meta-freescale/2017-January/019980.html


From: Jussi Kukkonen [mailto:jussi.kukko...@intel.com]
Sent: Wednesday, February 01, 2017 5:38 AM
To: Tom Hochstein <tom.hochst...@nxp.com>
Cc: Patches and discussions about the oe-core layer 
<openembedded-core@lists.openembedded.org>
Subject: Re: [OE-core] [PATCH v2] xserver-xorg: Fix X server 1.19 crash

On 1 February 2017 at 03:22, Tom Hochstein 
<tom.hochst...@nxp.com<mailto:tom.hochst...@nxp.com>> wrote:
Commit 6a5a4e60373c1386b311b2a8bb666c32d68a9d99 removes the configure of
useSIGIO option:

xfree86 SIGIO support is reworked to use internal versions of
OsBlockSIGIO and OsReleaseSIGIO

The check for useSIGIO is no longer needed.

Can you explain what the crash is or how to reproduce, I can't figure it out 
from the patch. Link to possible upstream ML discussion is fine as well.

Thanks,
  Jussi
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH v2] xserver-xorg: Fix X server 1.19 crash

2017-01-31 Thread Tom Hochstein
Commit 6a5a4e60373c1386b311b2a8bb666c32d68a9d99 removes the configure of
useSIGIO option:

xfree86 SIGIO support is reworked to use internal versions of
OsBlockSIGIO and OsReleaseSIGIO

The check for useSIGIO is no longer needed.

Upstream-Status: Pending

Signed-off-by: Tom Hochstein <tom.hochst...@nxp.com>
---
 .../0003-Remove-check-for-useSIGIO-option.patch| 47 ++
 .../xorg-xserver/xserver-xorg_1.19.1.bb|  1 +
 2 files changed, 48 insertions(+)
 create mode 100644 
meta/recipes-graphics/xorg-xserver/xserver-xorg/0003-Remove-check-for-useSIGIO-option.patch

diff --git 
a/meta/recipes-graphics/xorg-xserver/xserver-xorg/0003-Remove-check-for-useSIGIO-option.patch
 
b/meta/recipes-graphics/xorg-xserver/xserver-xorg/0003-Remove-check-for-useSIGIO-option.patch
new file mode 100644
index 000..beed6cb
--- /dev/null
+++ 
b/meta/recipes-graphics/xorg-xserver/xserver-xorg/0003-Remove-check-for-useSIGIO-option.patch
@@ -0,0 +1,47 @@
+From cf407b16cd65ad6e26a9c8e5984e163409a5c0f7 Mon Sep 17 00:00:00 2001
+From: Prabhu Sundararaj <prabhu.sundara...@nxp.com>
+Date: Mon, 30 Jan 2017 16:32:06 -0600
+Subject: [PATCH] Remove check for useSIGIO option
+
+Commit 6a5a4e60373c1386b311b2a8bb666c32d68a9d99 removes the configure of 
useSIGIO
+option.
+
+As the xfree86 SIGIO support is reworked to use internal versions of 
OsBlockSIGIO
+and OsReleaseSIGIO.
+
+No longer the check for useSIGIO is needed
+
+Upstream-Status: Pending
+
+Signed-off-by: Prabhu Sundararaj <prabhu.sundara...@nxp.com>
+---
+ hw/xfree86/os-support/shared/sigio.c | 6 --
+ 1 file changed, 6 deletions(-)
+
+diff --git a/hw/xfree86/os-support/shared/sigio.c 
b/hw/xfree86/os-support/shared/sigio.c
+index 884a71c..be76498 100644
+--- a/hw/xfree86/os-support/shared/sigio.c
 b/hw/xfree86/os-support/shared/sigio.c
+@@ -185,9 +185,6 @@ xf86InstallSIGIOHandler(int fd, void (*f) (int, void *), 
void *closure)
+ int i;
+ int installed = FALSE;
+ 
+-if (!xf86Info.useSIGIO)
+-return 0;
+-
+ for (i = 0; i < MAX_FUNCS; i++) {
+ if (!xf86SigIOFuncs[i].f) {
+ if (xf86IsPipe(fd))
+@@ -256,9 +253,6 @@ xf86RemoveSIGIOHandler(int fd)
+ int max;
+ int ret;
+ 
+-if (!xf86Info.useSIGIO)
+-return 0;
+-
+ max = 0;
+ ret = 0;
+ for (i = 0; i < MAX_FUNCS; i++) {
+-- 
+2.7.4
+
diff --git a/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.19.1.bb 
b/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.19.1.bb
index 987a2be..5a657e0 100644
--- a/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.19.1.bb
+++ b/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.19.1.bb
@@ -3,6 +3,7 @@ require xserver-xorg.inc
 SRC_URI += "file://musl-arm-inb-outb.patch \
 file://0001-configure.ac-Fix-check-for-CLOCK_MONOTONIC.patch \
 
file://0002-configure.ac-Fix-wayland-scanner-and-protocols-locat.patch \
+file://0003-Remove-check-for-useSIGIO-option.patch \
 "
 SRC_URI[md5sum] = "caa8ee7b2950abbf734347d137529fb6"
 SRC_URI[sha256sum] = 
"79ae2cf39d3f6c4a91201d8dad549d1d774b3420073c5a70d390040aa965a7fb"
-- 
1.9.1

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


[OE-core] [PATCH] xserver-xorg: Fix X server 1.19 crash

2017-01-31 Thread Tom Hochstein
Commit 6a5a4e60373c1386b311b2a8bb666c32d68a9d99 removes the configure of
useSIGIO option:

xfree86 SIGIO support is reworked to use internal versions of
OsBlockSIGIO and OsReleaseSIGIO

The check for useSIGIO is no longer needed.

Upstream-Status: Pending

Signed-off-by: Tom Hochstein <tom.hochst...@nxp.com>
---
 .../0003-Remove-check-for-useSIGIO-option.patch| 45 ++
 .../xorg-xserver/xserver-xorg_1.19.1.bb|  1 +
 2 files changed, 46 insertions(+)
 create mode 100644 
meta/recipes-graphics/xorg-xserver/xserver-xorg/0003-Remove-check-for-useSIGIO-option.patch

diff --git 
a/meta/recipes-graphics/xorg-xserver/xserver-xorg/0003-Remove-check-for-useSIGIO-option.patch
 
b/meta/recipes-graphics/xorg-xserver/xserver-xorg/0003-Remove-check-for-useSIGIO-option.patch
new file mode 100644
index 000..db2d845
--- /dev/null
+++ 
b/meta/recipes-graphics/xorg-xserver/xserver-xorg/0003-Remove-check-for-useSIGIO-option.patch
@@ -0,0 +1,45 @@
+From cf407b16cd65ad6e26a9c8e5984e163409a5c0f7 Mon Sep 17 00:00:00 2001
+From: Prabhu Sundararaj <prabhu.sundara...@nxp.com>
+Date: Mon, 30 Jan 2017 16:32:06 -0600
+Subject: [PATCH] Remove check for useSIGIO option
+
+Commit 6a5a4e60373c1386b311b2a8bb666c32d68a9d99 removes the configure of 
useSIGIO
+option.
+
+As the xfree86 SIGIO support is reworked to use internal versions of 
OsBlockSIGIO
+and OsReleaseSIGIO.
+
+No longer the check for useSIGIO is needed
+
+Signed-off-by: Prabhu Sundararaj <prabhu.sundara...@nxp.com>
+---
+ hw/xfree86/os-support/shared/sigio.c | 6 --
+ 1 file changed, 6 deletions(-)
+
+diff --git a/hw/xfree86/os-support/shared/sigio.c 
b/hw/xfree86/os-support/shared/sigio.c
+index 884a71c..be76498 100644
+--- a/hw/xfree86/os-support/shared/sigio.c
 b/hw/xfree86/os-support/shared/sigio.c
+@@ -185,9 +185,6 @@ xf86InstallSIGIOHandler(int fd, void (*f) (int, void *), 
void *closure)
+ int i;
+ int installed = FALSE;
+ 
+-if (!xf86Info.useSIGIO)
+-return 0;
+-
+ for (i = 0; i < MAX_FUNCS; i++) {
+ if (!xf86SigIOFuncs[i].f) {
+ if (xf86IsPipe(fd))
+@@ -256,9 +253,6 @@ xf86RemoveSIGIOHandler(int fd)
+ int max;
+ int ret;
+ 
+-if (!xf86Info.useSIGIO)
+-return 0;
+-
+ max = 0;
+ ret = 0;
+ for (i = 0; i < MAX_FUNCS; i++) {
+-- 
+2.7.4
+
diff --git a/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.19.1.bb 
b/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.19.1.bb
index 987a2be..5a657e0 100644
--- a/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.19.1.bb
+++ b/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.19.1.bb
@@ -3,6 +3,7 @@ require xserver-xorg.inc
 SRC_URI += "file://musl-arm-inb-outb.patch \
 file://0001-configure.ac-Fix-check-for-CLOCK_MONOTONIC.patch \
 
file://0002-configure.ac-Fix-wayland-scanner-and-protocols-locat.patch \
+file://0003-Remove-check-for-useSIGIO-option.patch \
 "
 SRC_URI[md5sum] = "caa8ee7b2950abbf734347d137529fb6"
 SRC_URI[sha256sum] = 
"79ae2cf39d3f6c4a91201d8dad549d1d774b3420073c5a70d390040aa965a7fb"
-- 
1.9.1

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


Re: [OE-core] weston startup problem

2016-09-10 Thread Tom Hochstein
Hi Jussi,

Thanks for the help.

Here are the relevant steps of the problem:


1.  weston.service sets XDG_RUNTIME_DIR, then starts Weston

2.  Weston starts up and registers its Wayland server socket in 
XDG_RUNTIME_DIR

3.  pam starts and sets XDG_RUNTIME_DIR to another location

4.  Wayland client fails to find the Wayland server socket.

At this point, I realize I’m not exactly sure what will happen. Looking at the 
Weston man page, it appears possible that the Wayland client will still be able 
to connect through the default Wayland server socket. I can’t immediately test 
this to know for sure, but regardless, there is still an issue except for the 
default case.

Tom

From: Jussi Kukkonen [mailto:jussi.kukko...@intel.com]
Sent: Friday, September 09, 2016 1:41 AM
To: Tom Hochstein <tom.hochst...@nxp.com>
Cc: Patches and discussions about the oe-core layer 
(openembedded-core@lists.openembedded.org) 
<openembedded-core@lists.openembedded.org>
Subject: Re: [OE-core] weston startup problem

On 24 August 2016 at 20:08, Tom Hochstein 
<tom.hochst...@nxp.com<mailto:tom.hochst...@nxp.com>> wrote:
Hi All,

Hi, sorry for missing this in the originally,

The weston systemd startup implementation is not working correctly and I need 
some help fixing it.

With systemd and pam on the image, the expected behavior is that 
XDG_RUNTIME_DIR would already be set when weston is launched from 
weston.service. Turns out XDG_RUNTIME_DIR is not set, and weston.service sets 
it itself. This is already a problem, and the problem would be worse except 
that weston happens to use a different folder. If weston.service is modified to 
use the same folder name, the folder ends up getting replaced later by pam.


I can see our XDG_RUNTIME_DIR handling indeed should be cleaned up but just so 
I understand the whole issue: can you explain the problem you have in detail? 
It seems one issue is that the variable isn't set outside of  Xsession (it 
really should) but why is it a problem if weston.service decides a runtime 
directory for itself? How does pam interfere with that?


Thanks,
 Jussi


From looking at the weston-launch code and its pam handling, I thought to try 
the weston-launch option '--user=root' to force the creation of the pam 
session. This does seem to help, and I was able to get it mostly working by 
modifying the openvt args in weston-start, removing -e and adding -w:

openvt -v -s -w -- weston-launch --user=root -- --modules=xwayland.so 
--log=/var/log/weston.log

The problem I have now is that keyboard input goes to both weston and the 
desktop. I tried with and without the -s option, and there is no difference.

I'm stuck at this point and would appreciate some help.

Tom

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


Re: [OE-core] weston startup problem

2016-09-08 Thread Tom Hochstein
Ping.

-Original Message-
From: openembedded-core-boun...@lists.openembedded.org 
[mailto:openembedded-core-boun...@lists.openembedded.org] On Behalf Of Tom 
Hochstein
Sent: Wednesday, August 24, 2016 12:08 PM
To: Patches and discussions about the oe-core layer 
(openembedded-core@lists.openembedded.org) 
<openembedded-core@lists.openembedded.org>
Subject: [OE-core] weston startup problem

Hi All,

The weston systemd startup implementation is not working correctly and I need 
some help fixing it.

With systemd and pam on the image, the expected behavior is that 
XDG_RUNTIME_DIR would already be set when weston is launched from 
weston.service. Turns out XDG_RUNTIME_DIR is not set, and weston.service sets 
it itself. This is already a problem, and the problem would be worse except 
that weston happens to use a different folder. If weston.service is modified to 
use the same folder name, the folder ends up getting replaced later by pam.

>From looking at the weston-launch code and its pam handling, I thought to try 
>the weston-launch option '--user=root' to force the creation of the pam 
>session. This does seem to help, and I was able to get it mostly working by 
>modifying the openvt args in weston-start, removing -e and adding -w:

openvt -v -s -w -- weston-launch --user=root -- --modules=xwayland.so 
--log=/var/log/weston.log

The problem I have now is that keyboard input goes to both weston and the 
desktop. I tried with and without the -s option, and there is no difference.

I'm stuck at this point and would appreciate some help.

Tom

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


[OE-core] weston startup problem

2016-08-24 Thread Tom Hochstein
Hi All,

The weston systemd startup implementation is not working correctly and I need 
some help fixing it.

With systemd and pam on the image, the expected behavior is that 
XDG_RUNTIME_DIR would already be set when weston is launched from 
weston.service. Turns out XDG_RUNTIME_DIR is not set, and weston.service sets 
it itself. This is already a problem, and the problem would be worse except 
that weston happens to use a different folder. If weston.service is modified to 
use the same folder name, the folder ends up getting replaced later by pam.

>From looking at the weston-launch code and its pam handling, I thought to try 
>the weston-launch option '--user=root' to force the creation of the pam 
>session. This does seem to help, and I was able to get it mostly working by 
>modifying the openvt args in weston-start, removing -e and adding -w:

openvt -v -s -w -- weston-launch --user=root -- --modules=xwayland.so 
--log=/var/log/weston.log

The problem I have now is that keyboard input goes to both weston and the 
desktop. I tried with and without the -s option, and there is no difference.

I'm stuck at this point and would appreciate some help.

Tom
<>-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [krogoth][PATCH] mesa-demos: Fix OpenGL ES configurability

2016-08-09 Thread Tom Hochstein
No, master does not need this fix. The problematic patch 
0011-drop-demos-dependant-on-obsolete-MESA_screen_surface.patch does not exist 
on master because the underlying issue was addressed upstream (without 
introducing this bug).

Tom

From: Burton, Ross [mailto:ross.bur...@intel.com]
Sent: Tuesday, August 09, 2016 5:10 AM
To: Tom Hochstein <tom.hochst...@nxp.com>
Cc: OE-core <openembedded-core@lists.openembedded.org>
Subject: Re: [OE-core] [krogoth][PATCH] mesa-demos: Fix OpenGL ES 
configurability


On 8 August 2016 at 16:45, Tom Hochstein 
<tom.hochst...@nxp.com<mailto:tom.hochst...@nxp.com>> wrote:
The most recent patch 
0011-drop-demos-dependant-on-obsolete-MESA_screen_surface.patch
incorrectly removed the configuration constructs that allowed the
package to be configured without OpenGL ES support.

Does this also need to be applied to master?

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


[OE-core] [krogoth][PATCH] mesa-demos: Fix OpenGL ES configurability

2016-08-09 Thread Tom Hochstein
The most recent patch 
0011-drop-demos-dependant-on-obsolete-MESA_screen_surface.patch
incorrectly removed the configuration constructs that allowed the
package to be configured without OpenGL ES support.

Signed-off-by: Tom Hochstein <tom.hochst...@nxp.com>
---
 .../mesa-demos/0012-Fix-gles-configurability.patch | 39 ++
 meta/recipes-graphics/mesa/mesa-demos_8.2.0.bb |  1 +
 2 files changed, 40 insertions(+)
 create mode 100644 
meta/recipes-graphics/mesa/mesa-demos/0012-Fix-gles-configurability.patch

diff --git 
a/meta/recipes-graphics/mesa/mesa-demos/0012-Fix-gles-configurability.patch 
b/meta/recipes-graphics/mesa/mesa-demos/0012-Fix-gles-configurability.patch
new file mode 100644
index 000..0b437a4
--- /dev/null
+++ b/meta/recipes-graphics/mesa/mesa-demos/0012-Fix-gles-configurability.patch
@@ -0,0 +1,39 @@
+Index: mesa-demos-8.2.0/src/egl/opengles1/Makefile.am
+===
+--- mesa-demos-8.2.0.orig/src/egl/opengles1/Makefile.am2016-07-29 
10:52:50.131729547 -0500
 mesa-demos-8.2.0/src/egl/opengles1/Makefile.am 2016-07-29 
10:59:26.0 -0500
+@@ -36,6 +36,8 @@
+   $(EGL_LIBS) \
+   -lm
+ 
++if HAVE_EGL
++if HAVE_GLESV1
+ bin_PROGRAMS =
+ 
+ if HAVE_X11
+@@ -76,3 +78,5 @@
+ torus_x11_LDADD = ../eglut/libeglut_x11.la
+ tri_x11_LDADD = ../eglut/libeglut_x11.la
+ endif
++endif
++endif
+Index: mesa-demos-8.2.0/src/egl/opengles2/Makefile.am
+===
+--- mesa-demos-8.2.0.orig/src/egl/opengles2/Makefile.am2016-07-29 
10:52:50.131729547 -0500
 mesa-demos-8.2.0/src/egl/opengles2/Makefile.am 2016-07-29 
10:59:03.0 -0500
+@@ -33,6 +33,8 @@
+   $(EGL_LIBS) \
+   -lm
+ 
++if HAVE_EGL
++if HAVE_GLESV2
+ bin_PROGRAMS =
+ 
+ if HAVE_WAYLAND
+@@ -53,4 +55,5 @@
+ es2gears_x11_LDADD = ../eglut/libeglut_x11.la
+ es2tri_LDADD = $(X11_LIBS)
+ endif
+-
++endif
++endif
diff --git a/meta/recipes-graphics/mesa/mesa-demos_8.2.0.bb 
b/meta/recipes-graphics/mesa/mesa-demos_8.2.0.bb
index 2df9bba..8e12d82 100644
--- a/meta/recipes-graphics/mesa/mesa-demos_8.2.0.bb
+++ b/meta/recipes-graphics/mesa/mesa-demos_8.2.0.bb
@@ -21,6 +21,7 @@ SRC_URI = 
"ftp://ftp.freedesktop.org/pub/mesa/demos/${PV}/${BPN}-${PV}.tar.bz2 \
 file://0009-glsl-perf-Install-.glsl-.vert-.frag-files.patch \
 file://0010-sharedtex_mt-fix-rendering-thread-hang.patch \
 file://0011-drop-demos-dependant-on-obsolete-MESA_screen_surface.patch \
+file://0012-Fix-gles-configurability.patch \
 "
 SRC_URI[md5sum] = "72613a2c8c013716db02e3ff59d29061"
 SRC_URI[sha256sum] = 
"e4bfecb5816ddd4b7b37c1bc876b63f1f7f06fda5879221a9774d0952f90ba92"
-- 
1.9.1

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


Re: [OE-core] [PATCH v3 3/3] weston-init: Fix weston-start to allow weston args without openvt args

2016-07-18 Thread Tom Hochstein
Re-sent.

From: Burton, Ross [mailto:ross.bur...@intel.com]
Sent: Monday, July 18, 2016 11:29 AM
To: Otavio Salvador <otavio.salva...@ossystems.com.br>
Cc: Tom Hochstein <tom.hochst...@nxp.com>; Otavio Salvador 
<ota...@ossystems.com.br>; Patches and discussions about the oe-core layer 
<openembedded-core@lists.openembedded.org>
Subject: Re: [OE-core] [PATCH v3 3/3] weston-init: Fix weston-start to allow 
weston args without openvt args

I can only see 1/3 and 3/3, but no 2/3.

On 18 July 2016 at 16:13, Otavio Salvador 
<otavio.salva...@ossystems.com.br<mailto:otavio.salva...@ossystems.com.br>> 
wrote:
On Mon, Jul 18, 2016 at 11:43 AM, Tom Hochstein 
<tom.hochst...@nxp.com<mailto:tom.hochst...@nxp.com>> wrote:
> The parser didn't properly handle commands of the form
> weston-start -- .
>
> Signed-off-by: Tom Hochstein 
> <tom.hochst...@nxp.com<mailto:tom.hochst...@nxp.com>>



Acked-by: Otavio Salvador 
<ota...@ossystems.com.br<mailto:ota...@ossystems.com.br>>

--
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9981-7854<tel:%2B55%20%2853%29%209981-7854>Mobile: 
+1 (347) 903-9750<tel:%2B1%20%28347%29%20903-9750>
--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org<mailto:Openembedded-core@lists.openembedded.org>
http://lists.openembedded.org/mailman/listinfo/openembedded-core

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


[OE-core] [PATCH v3 2/3] weston-init: Fix weston-start to handle 0 or 1 args

2016-07-18 Thread Tom Hochstein
The parser incorrectly treated anything less than 2 args as an error.

Signed-off-by: Tom Hochstein <tom.hochst...@nxp.com>
---
 meta/recipes-graphics/wayland/weston-init/weston-start | 7 +--
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/meta/recipes-graphics/wayland/weston-init/weston-start 
b/meta/recipes-graphics/wayland/weston-init/weston-start
index d7358ba..cd64216 100755
--- a/meta/recipes-graphics/wayland/weston-init/weston-start
+++ b/meta/recipes-graphics/wayland/weston-init/weston-start
@@ -6,7 +6,7 @@ export PATH="/sbin:/usr/sbin:/bin:/usr/bin"
 
 usage() {
 cat < -- 
+$0 [] [-- ]
 EOF
 }
 
@@ -23,11 +23,6 @@ add_openvt_argument() {
openvt_args="$openvt_args $1"
 }
 
-if test $# -lt 2; then
-   usage
-   exit 1
-fi
-
 if [ -n "$WAYLAND_DISPLAY" ]; then
echo "ERROR: A Wayland compositor is already running, nested Weston 
instance is not supported yet."
exit 1
-- 
1.9.1

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


[OE-core] [PATCH v3 2/3] weston-init: Fix weston-start to handle 0 or 1 args

2016-07-18 Thread Tom Hochstein
The parser incorrectly treated anything less than 2 args as an error.

Signed-off-by: Tom Hochstein <tom.hochst...@nxp.com>
---
 meta/recipes-graphics/wayland/weston-init/weston-start | 7 +--
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/meta/recipes-graphics/wayland/weston-init/weston-start 
b/meta/recipes-graphics/wayland/weston-init/weston-start
index d7358ba..cd64216 100755
--- a/meta/recipes-graphics/wayland/weston-init/weston-start
+++ b/meta/recipes-graphics/wayland/weston-init/weston-start
@@ -6,7 +6,7 @@ export PATH="/sbin:/usr/sbin:/bin:/usr/bin"
 
 usage() {
 cat < -- 
+$0 [] [-- ]
 EOF
 }
 
@@ -23,11 +23,6 @@ add_openvt_argument() {
openvt_args="$openvt_args $1"
 }
 
-if test $# -lt 2; then
-   usage
-   exit 1
-fi
-
 if [ -n "$WAYLAND_DISPLAY" ]; then
echo "ERROR: A Wayland compositor is already running, nested Weston 
instance is not supported yet."
exit 1
-- 
1.9.1

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


[OE-core] [PATCH v3 1/3] weston-init: De-couple framebuffer console from Weston for systemd startup

2016-07-18 Thread Tom Hochstein
The framebuffer console was using the same I/O as Weston. We fix this
by having openvt switch to the new VT when starting weston-launch, same
as is already done for the sysvinit case.

Signed-off-by: Tom Hochstein <tom.hochst...@nxp.com>
---
 meta/recipes-graphics/wayland/weston-init/init | 2 +-
 meta/recipes-graphics/wayland/weston-init/weston-start | 3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-graphics/wayland/weston-init/init 
b/meta/recipes-graphics/wayland/weston-init/init
index 5c925f4..d3e87c6 100644
--- a/meta/recipes-graphics/wayland/weston-init/init
+++ b/meta/recipes-graphics/wayland/weston-init/init
@@ -31,7 +31,7 @@ case "$1" in
   start)
 . /etc/profile
 
-weston-start -s -- $OPTARGS
+weston-start -- $OPTARGS
   ;;
 
   stop)
diff --git a/meta/recipes-graphics/wayland/weston-init/weston-start 
b/meta/recipes-graphics/wayland/weston-init/weston-start
index 5b7604f..d7358ba 100755
--- a/meta/recipes-graphics/wayland/weston-init/weston-start
+++ b/meta/recipes-graphics/wayland/weston-init/weston-start
@@ -1,5 +1,6 @@
 #!/bin/sh
 # Copyright (C) 2016 O.S. Systems Software LTDA.
+# Copyright (C) 2016 Freescale Semiconductor
 
 export PATH="/sbin:/usr/sbin:/bin:/usr/bin"
 
@@ -37,7 +38,7 @@ else
launcher="weston-launch --"
 fi
 
-openvt_args=""
+openvt_args="-s"
 while [ -n "$1" ]; do
openvt_args="$openvt_args $1"
shift
-- 
1.9.1

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


[OE-core] [PATCH v3 3/3] weston-init: Fix weston-start to allow weston args without openvt args

2016-07-18 Thread Tom Hochstein
The parser didn't properly handle commands of the form
weston-start -- .

Signed-off-by: Tom Hochstein <tom.hochst...@nxp.com>
---
 meta/recipes-graphics/wayland/weston-init/weston-start | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/meta/recipes-graphics/wayland/weston-init/weston-start 
b/meta/recipes-graphics/wayland/weston-init/weston-start
index cd64216..e72fbaa 100755
--- a/meta/recipes-graphics/wayland/weston-init/weston-start
+++ b/meta/recipes-graphics/wayland/weston-init/weston-start
@@ -35,13 +35,12 @@ fi
 
 openvt_args="-s"
 while [ -n "$1" ]; do
-   openvt_args="$openvt_args $1"
-   shift
-
if [ "$1" = "--" ]; then
shift
break
fi
+   openvt_args="$openvt_args $1"
+   shift
 done
 
 weston_args=$*
-- 
1.9.1

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


Re: [OE-core] [PATCH v2] weston-init: De-couple framebuffer console from Weston for systemd startup

2016-07-15 Thread Tom Hochstein
Sorry again, seems that the weston-start parser is not prepared to take no 
openvt arguments. Will need to fix that as well.

Tom

-Original Message-
From: Otavio Salvador [mailto:otavio.salva...@ossystems.com.br] 
Sent: Friday, July 15, 2016 12:01 PM
To: Tom Hochstein <tom.hochst...@nxp.com>
Cc: Patches and discussions about the oe-core layer 
<openembedded-core@lists.openembedded.org>
Subject: Re: [OE-core] [PATCH v2] weston-init: De-couple framebuffer console 
from Weston for systemd startup

On Fri, Jul 15, 2016 at 1:33 PM, Tom Hochstein <tom.hochst...@nxp.com> wrote:
> The framebuffer console was using the same I/O as Weston. We fix this
> by having openvt switch to the new VT when starting weston-launch, same
> as is already done for the sysvinit case.
>
> Signed-off-by: Tom Hochstein <tom.hochst...@nxp.com>

Acked-by: Otavio Salvador <ota...@ossystems.com.br>

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


  1   2   >