[OE-core] [PATCH] base/bitbake.conf: use default assigment for UNPACKDIR

2024-05-31 Thread Samuli Piippo
This lets recipes in other layers add weak default assignment as
  UNPACKDIR ??= "${WORKDIR}"
which allows the recipes to work also with older Yocto releases
without overriding the default value used in styhead.

Signed-off-by: Samuli Piippo 
---
 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 3ef2deb088..386e6e0a3e 100644
--- a/meta/conf/bitbake.conf
+++ b/meta/conf/bitbake.conf
@@ -405,7 +405,7 @@ STAMP = "${STAMPS_DIR}/${MULTIMACH_TARGET_SYS}/${PN}/${PV}"
 STAMPCLEAN = "${STAMPS_DIR}/${MULTIMACH_TARGET_SYS}/${PN}/*-*"
 BASE_WORKDIR ?= "${TMPDIR}/work"
 WORKDIR = "${BASE_WORKDIR}/${MULTIMACH_TARGET_SYS}/${PN}/${PV}"
-UNPACKDIR ??= "${WORKDIR}/sources-unpack"
+UNPACKDIR ?= "${WORKDIR}/sources-unpack"
 T = "${WORKDIR}/temp"
 D = "${WORKDIR}/image"
 S = "${WORKDIR}/${BP}"
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#200061): 
https://lists.openembedded.org/g/openembedded-core/message/200061
Mute This Topic: https://lists.openembedded.org/mt/106405273/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 v6] qemu: Split the qemu package

2023-06-12 Thread Samuli Piippo
I'm seeing following QA issue after this change:

ERROR: nativesdk-qemu-8.0.0-r0 do_package_qa: QA Issue:
nativesdk-qemu-user-mips rdepends on nativesdk-bash, but it isn't a build
dependency, missing nativesdk-bash in DEPENDS or PACKAGECONFIG? [build-deps]
ERROR: nativesdk-qemu-8.0.0-r0 do_package_qa: Fatal QA errors were found,
failing task.

On Fri, 9 Jun 2023 at 08:22, Martin Jansa  wrote:

> On Fri, Jun 9, 2023 at 5:04 AM Yu, Mingli 
> wrote:
>
>> +if userpackages:
>> +d.setVar('RDEPENDS:' + d.getVar('PN') + '-user-all', '
>> '.join(userpackages))
>> +mipspackage = d.getVar('PN') + "-user-mips"
>> +if mipspackage in ' '.join(userpackages):
>> +d.appendVar('RDEPENDS:' + mipspackage, ' ' +
>> d.getVar("MLPREFIX") + 'bash')
>>
>
> This is just a style question, you don't need to update.
>
> Is it worth making this conditional on ${PN}-user-mips existence? I would
> just use
>
> RDEPENDS:${PN}-user-mips = "bash"
>
> outside this function and I don't think you would even need to explicitly
> use ${MLPREFIX}. Nothing will break when ${PN}-user-mips doesn't exist
> (e.g. when splitting is disabled) and it's a bit easier to read than
> corresponding 3 lines in python.
>
> 
>
>

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#182672): 
https://lists.openembedded.org/g/openembedded-core/message/182672
Mute This Topic: https://lists.openembedded.org/mt/99421260/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 v6] qemu: Split the qemu package

2023-06-12 Thread Samuli Piippo
I'm seeing following QA issue after this change:


On Fri, 9 Jun 2023 at 08:22, Martin Jansa  wrote:

> On Fri, Jun 9, 2023 at 5:04 AM Yu, Mingli 
> wrote:
>
>> +if userpackages:
>> +d.setVar('RDEPENDS:' + d.getVar('PN') + '-user-all', '
>> '.join(userpackages))
>> +mipspackage = d.getVar('PN') + "-user-mips"
>> +if mipspackage in ' '.join(userpackages):
>> +d.appendVar('RDEPENDS:' + mipspackage, ' ' +
>> d.getVar("MLPREFIX") + 'bash')
>>
>
> This is just a style question, you don't need to update.
>
> Is it worth making this conditional on ${PN}-user-mips existence? I would
> just use
>
> RDEPENDS:${PN}-user-mips = "bash"
>
> outside this function and I don't think you would even need to explicitly
> use ${MLPREFIX}. Nothing will break when ${PN}-user-mips doesn't exist
> (e.g. when splitting is disabled) and it's a bit easier to read than
> corresponding 3 lines in python.
>
> 
>
>

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

2023-01-17 Thread Samuli Piippo
I think this is now a webos specific issue since they have disabled xlib
feature (
https://github.com/shr-project/meta-webosose/blob/c96010fe4df28397a33e447bde650932a3463a9e/meta-webos/recipes-qt/qt6/qtbase_git.bbappend#L56
)
While the qtbase cmake configuration could be improved to better handle
situation where x11 dependencies are found but shouldn't be used, I think
the simplest solution would be to disable egl on X11 in the webos bbappend
(-DFEATURE_egl_x11=OFF)

-samuli

On Tue, 17 Jan 2023 at 13:01, Martin Jansa  wrote:

> Samuli:
>
> Similar mesa upgrade was merged recently:
>
> https://git.openembedded.org/openembedded-core/commit/?id=3bf4341ef6a681574a1c6f393bf241f412e26eac
>
> qtbase still fails the same, more details in:
>
> https://github.com/shr-project/meta-webosose/commit/c96010fe4df28397a33e447bde650932a3463a9e
>
> The change you've mentioned is the one which actually breaks some
> PACKAGECONFIG combinations, hopefully now you'll be able to reproduce it.
>
> Regards,
>
> On Thu, Dec 8, 2022 at 8:55 AM Martin Jansa via lists.openembedded.org
>  wrote:
>
>> On Thu, Dec 8, 2022 at 8:42 AM Samuli Piippo  wrote:
>>
>>> Qt6 should handle this already with
>>> https://codereview.qt-project.org/c/qt/qtbase/+/371780 in place.
>>> I tried test build with the mesa upgrade and didn't catch any build
>>> errors.
>>>
>>
>> I've tried qemux86-64 build with 6.4.1 and 6.5.0 from yesterday's dev
>> branch and both failed after including this mesa upgrade with:
>> | In file included from
>> qtbase/6.4.1-r0/recipe-sysroot/usr/include/EGL/egl.h:20,
>> |  from
>> qtbase/6.4.1-r0/build/include/QtGui/6.4.1/QtGui/private/../../../../../../git/src/gui/opengl/platform/egl/qt_egl_p.h:43,
>> |  from
>> qtbase/6.4.1-r0/build/include/QtGui/6.4.1/QtGui/private/qt_egl_p.h:1,
>> |  from
>> qtbase/6.4.1-r0/git/src/gui/opengl/platform/egl/qeglstreamconvenience_p.h:20,
>> |  from
>> qtbase/6.4.1-r0/git/src/gui/opengl/platform/egl/qeglstreamconvenience.cpp:4:
>> | qtbase/6.4.1-r0/recipe-sysroot/usr/include/EGL/eglplatform.h:109:10:
>> fatal error: X11/Xlib.h: No such file or directory
>> |   109 | #include 
>> |   |  ^~~~
>> | compilation terminated.
>>
>>>
>>
>>
>>
> 
>
>

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

2022-12-07 Thread Samuli Piippo via lists.openembedded.org
Qt6 should handle this already with 
https://codereview.qt-project.org/c/qt/qtbase/+/371780 in place.
I tried test build with the mesa upgrade and didn't catch any build errors.

-samuli

From: Martin Jansa 
Sent: 07 December 2022 16:10
To: Otavio Salvador ; Samuli Piippo 

Cc: Markus Volk ; Khem Raj ; 
openembedded-core@lists.openembedded.org 

Subject: Re: [oe-core][PATCH] mesa: update 22.2.3 -> 22.3.0

On Mon, Dec 5, 2022 at 3:04 PM Otavio Salvador 
mailto:otavio.salva...@ossystems.com.br>> 
wrote:


Em seg., 5 de dez. de 2022 às 02:52, Markus Volk 
mailto:f_...@t-online.de>> escreveu:
Am So, 4. Dez 2022 um 09:23:49 -0800 schrieb Khem Raj 
mailto:raj.k...@gmail.com>>:
Not sure if this is related but I think its somewhere in egl headers provided 
by mesa.

This is related to an update of the egl headers in Mesa, just as you thought:
https://github.com/meta-qt5/meta-qt5/pull/491

It looks fine for me. @Martin Jansa<mailto:martin.ja...@gmail.com> could you 
check the PR above and merge if fine with it?

Thanks, I've merged this yesterday.

+Samuli Piippo<mailto:samuli.pii...@qt.io> as qtbase in meta-qt6 fails with 
this mesa upgrade as well.

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



[OE-core] [PATCH 2/2] gcc-cross-canadian: add default plugin linker

2022-09-16 Thread Samuli Piippo
Fix multilib sdk issue where gcc is unable to find linker. Previous
fix was in cdd86896c8d29135f937968e9aa07f919cf543d3 using real-ld
symlink, but that prevented switching between bfd and gold linkers.

Running compiler with debug arguments shows that collect2 tries and
fails to find linker using the multilib triples:

  $ $CC -v -Wl,-debug
  ...
  Looking for 'real-ld'
  Looking for 'collect-ld'
  Looking for 'mips-oemllib32-linux-ld'
  Looking for 'mips-oe-linux-mips-oemllib32-linux-ld'
  ...
  collect2 version 12.2.0
  ld_file_name= not found
  ...
  collect2: fatal error: cannot find ‘ld’

Using --with-plugin-ld=ld in gcc-cross-canadian builds to set default
linker name for collect2, lets it find the linker correctly:

  Looking for 'real-ld'
  Looking for 'collect-ld'
  Looking for 'ld'
  ...
  collect2 version 12.2.0
  ld_file_name= 
/usr/local/oecore-x86_64/sysroots/x86_64-oesdk-linux/usr/libexec/mips-oe-linux/gcc/mips-oe-linux/12.2.0/ld

Swith between bfd and gold linker works as expected now:

  $ $CC -v -Wl,-debug -fuse-ld=gold
  ...
  Looking for 'real-ld'
  Looking for 'collect-ld'
  Looking for 'ld.gold'
  ...
  collect2 version 12.2.0
  ld_file_name= 
/usr/local/oecore-x86_64/sysroots/x86_64-oesdk-linux/usr/libexec/i686-oe-linux/gcc/i686-oe-linux/12.2.0/ld.gold

Signed-off-by: Samuli Piippo 
---
 meta/recipes-devtools/gcc/gcc-cross-canadian.inc | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-devtools/gcc/gcc-cross-canadian.inc 
b/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
index 38c972aa7c..c36e4cba81 100644
--- a/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
+++ b/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
@@ -9,6 +9,7 @@ GCCMULTILIB = "--enable-multilib"
 
 require gcc-configure-common.inc
 
+EXTRA_OECONF += "--with-plugin-ld=ld"
 EXTRA_OECONF_PATHS = "\
 --with-gxx-include-dir=/not/exist${target_includedir}/c++/${BINV} \
 
--with-build-time-tools=${STAGING_DIR_NATIVE}${prefix_native}/${TARGET_SYS}/bin 
\
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#170781): 
https://lists.openembedded.org/g/openembedded-core/message/170781
Mute This Topic: https://lists.openembedded.org/mt/93718003/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] Revert "gcc-cross-canadian: Add symlink to real-ld alongside other symlinks"

2022-09-16 Thread Samuli Piippo
This reverts commit cdd86896c8d29135f937968e9aa07f919cf543d3.

real-ld is always used if that is found, which means you cannot
switch between bfd and gold linkers using -fuse-ld gcc option.

Signed-off-by: Samuli Piippo 
---
 meta/recipes-devtools/gcc/gcc-cross-canadian.inc | 2 --
 1 file changed, 2 deletions(-)

diff --git a/meta/recipes-devtools/gcc/gcc-cross-canadian.inc 
b/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
index a87b446c4f..38c972aa7c 100644
--- a/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
+++ b/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
@@ -134,8 +134,6 @@ do_install () {
 
ln -sf ${BINRELPATH}/${TARGET_PREFIX}$t$suffix $dest$t$suffix
done
-   t=real-ld
-   ln -sf ${BINRELPATH}/${TARGET_PREFIX}ld$suffix $dest$t$suffix
 
# libquadmath headers need to  be available in the gcc libexec dir
install -d ${D}${libdir}/gcc/${TARGET_SYS}/${BINV}/include/
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#170780): 
https://lists.openembedded.org/g/openembedded-core/message/170780
Mute This Topic: https://lists.openembedded.org/mt/93718002/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] binutils: Bump to latest 2.38 release branch

2022-05-16 Thread Samuli Piippo
Previous update brought
caa6172de4b x86: Disallow invalid relocation against protected symbol
and it was later fixed in
9c67f6382ac x86: Properly handle function pointer reference

https://bugreports.qt.io/browse/QTBUG-103465

On Mon, 16 May 2022 at 19:12, Khem Raj  wrote:

>
>
> On 5/16/22 12:11 AM, Samuli Piippo wrote:
> > Fixes Qt build with GCC 12.1.
> >
> > Brings following fixes
> >
> >   * 3d549e5ccc0 PR29142, segv in ar with empty archive and libdeps
> specified
> >   * c473aa1b9d8 M68K: avoid quadratic slowdlow in label alignment check
> >   * 8d1187516e3 Adjust ld ctf test for 32-bit targets
> >   * 9c67f6382ac x86: Properly handle function pointer reference
> >   * 9a01457e02e s390: Add DT_JMPREL pointing to .rela.[i]plt with
> static-pie
> >   * 82a5bb730a1 s390: Avoid dynamic TLS relocs in PIE
> >   * d54081c642a LoongArch: Update ABI eflag in elf header.
> >   * fb4d148004f IBM zSystems: Add support for z16 as CPU name.
> >   * 975b5540232 libctf, ld: diagnose corrupted CTF header cth_strlen
> >   * 99852365513 dlltool: Use the output name as basis for deterministic
> temp prefixes
> >   * 210bf1d6225 Updated Serbian (for binutils/) and Russian (for gprof/)
> translations
> >   * fcf60fe8482 PR28959, obdump doesn't disassemble mftb instruction
> >   * e4a35c73196 PowerPC64 DT_RELR relative reloc addresses
> >   * 7183434818e Work around gcc-4 warnings in elf64-ppc.c
>
> this looks ok. Out of curiousity which fix is particularly needed for QT
>
> >
> > Signed-off-by: Samuli Piippo 
> > ---
> >   meta/recipes-devtools/binutils/binutils-2.38.inc | 2 +-
> >   1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/meta/recipes-devtools/binutils/binutils-2.38.inc
> b/meta/recipes-devtools/binutils/binutils-2.38.inc
> > index 3544c0c7e7..dc0a2a4054 100644
> > --- a/meta/recipes-devtools/binutils/binutils-2.38.inc
> > +++ b/meta/recipes-devtools/binutils/binutils-2.38.inc
> > @@ -18,7 +18,7 @@ SRCBRANCH ?= "binutils-2_38-branch"
> >
> >   UPSTREAM_CHECK_GITTAGREGEX = "binutils-(?P\d+_(\d_?)*)"
> >
> > -SRCREV ?= "6938a3aca19044aeb9bc4b9564e04f9092a701ed"
> > +SRCREV ?= "134f17ef688ba4c72a6c4e57af7382882cc1a705"
> >   BINUTILS_GIT_URI ?= "git://
> sourceware.org/git/binutils-gdb.git;branch=${SRCBRANCH};protocol=git
> <http://sourceware.org/git/binutils-gdb.git;branch=$%7BSRCBRANCH%7D;protocol=git>
> "
> >   SRC_URI = "\
> >${BINUTILS_GIT_URI} \
> >
> >
> >
> >
> >
>
> 
>
>

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#165705): 
https://lists.openembedded.org/g/openembedded-core/message/165705
Mute This Topic: https://lists.openembedded.org/mt/91134867/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] binutils: Bump to latest 2.38 release branch

2022-05-16 Thread Samuli Piippo
Fixes Qt build with GCC 12.1.

Brings following fixes

 * 3d549e5ccc0 PR29142, segv in ar with empty archive and libdeps specified
 * c473aa1b9d8 M68K: avoid quadratic slowdlow in label alignment check
 * 8d1187516e3 Adjust ld ctf test for 32-bit targets
 * 9c67f6382ac x86: Properly handle function pointer reference
 * 9a01457e02e s390: Add DT_JMPREL pointing to .rela.[i]plt with static-pie
 * 82a5bb730a1 s390: Avoid dynamic TLS relocs in PIE
 * d54081c642a LoongArch: Update ABI eflag in elf header.
 * fb4d148004f IBM zSystems: Add support for z16 as CPU name.
 * 975b5540232 libctf, ld: diagnose corrupted CTF header cth_strlen
 * 99852365513 dlltool: Use the output name as basis for deterministic temp 
prefixes
 * 210bf1d6225 Updated Serbian (for binutils/) and Russian (for gprof/) 
translations
 * fcf60fe8482 PR28959, obdump doesn't disassemble mftb instruction
 * e4a35c73196 PowerPC64 DT_RELR relative reloc addresses
 * 7183434818e Work around gcc-4 warnings in elf64-ppc.c

Signed-off-by: Samuli Piippo 
---
 meta/recipes-devtools/binutils/binutils-2.38.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-devtools/binutils/binutils-2.38.inc 
b/meta/recipes-devtools/binutils/binutils-2.38.inc
index 3544c0c7e7..dc0a2a4054 100644
--- a/meta/recipes-devtools/binutils/binutils-2.38.inc
+++ b/meta/recipes-devtools/binutils/binutils-2.38.inc
@@ -18,7 +18,7 @@ SRCBRANCH ?= "binutils-2_38-branch"
 
 UPSTREAM_CHECK_GITTAGREGEX = "binutils-(?P\d+_(\d_?)*)"
 
-SRCREV ?= "6938a3aca19044aeb9bc4b9564e04f9092a701ed"
+SRCREV ?= "134f17ef688ba4c72a6c4e57af7382882cc1a705"
 BINUTILS_GIT_URI ?= 
"git://sourceware.org/git/binutils-gdb.git;branch=${SRCBRANCH};protocol=git"
 SRC_URI = "\
  ${BINUTILS_GIT_URI} \
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#165664): 
https://lists.openembedded.org/g/openembedded-core/message/165664
Mute This Topic: https://lists.openembedded.org/mt/91134867/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] binutils-cross-canadian: enable gold for mingw

2022-03-09 Thread Samuli Piippo
This reverts commit b5a595a4be09756b88e91f3353e3b221b165ab44,
gold linker can now be built correctly for mingw SDKs.

Signed-off-by: Samuli Piippo 
---
 meta/recipes-devtools/binutils/binutils-cross-canadian.inc | 2 --
 1 file changed, 2 deletions(-)

diff --git a/meta/recipes-devtools/binutils/binutils-cross-canadian.inc 
b/meta/recipes-devtools/binutils/binutils-cross-canadian.inc
index f44f46602d..b3d591e658 100644
--- a/meta/recipes-devtools/binutils/binutils-cross-canadian.inc
+++ b/meta/recipes-devtools/binutils/binutils-cross-canadian.inc
@@ -13,8 +13,6 @@ EXTRA_OECONF += 
"--with-sysroot=${SDKPATH}/sysroots/${TUNE_PKGARCH}${TARGET_VEND
 # e.g. we switch between different machines with different tunes.
 EXTRA_OECONF[vardepsexclude] = "TUNE_PKGARCH"
 
-LDGOLD:sdkmingw32 = ""
-
 do_install () {
autotools_do_install
 
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#162953): 
https://lists.openembedded.org/g/openembedded-core/message/162953
Mute This Topic: https://lists.openembedded.org/mt/89658339/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] mesa: make sure GLES3 headers are installed

2022-03-08 Thread Samuli Piippo
Since there is no libgles3-mesa package that would pull in the headers,
add dependency to libgles2-mesa-dev. Now there no need to manually add
GLES3 headers to image or toolchain.

Signed-off-by: Samuli Piippo 
Signed-off-by: Ross Burton 
---
 meta/recipes-graphics/mesa/mesa.inc | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/meta/recipes-graphics/mesa/mesa.inc 
b/meta/recipes-graphics/mesa/mesa.inc
index 2f79e8b34d..713a82dc7d 100644
--- a/meta/recipes-graphics/mesa/mesa.inc
+++ b/meta/recipes-graphics/mesa/mesa.inc
@@ -193,6 +193,9 @@ RDEPENDS:${PN}-dev = ""
 # development package of libgles3.
 RDEPENDS:libgles3-mesa-dev += "libgles2-mesa-dev"
 
+# Add dependency so that GLES3 header don't need to be added manually
+RDEPENDS_libgles2-mesa-dev += "libgles3-mesa-dev"
+
 PACKAGES =+ "libegl-mesa libegl-mesa-dev \
  libosmesa libosmesa-dev \
  libgl-mesa libgl-mesa-dev \
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#162876): 
https://lists.openembedded.org/g/openembedded-core/message/162876
Mute This Topic: https://lists.openembedded.org/mt/89633249/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] rpm: remove tmp folder created during install

2022-01-03 Thread Samuli Piippo
nativesdk-rpm build is also affected by the stray /var/tmp
created during the rpm install. Remove it to fix QA Issue:
nativesdk-rpm installs files in 
/usr/local/oe-sdk-hardcoded-buildpath/sysroots/x86_64-pokysdk-linux/var/volatile,
 but it is expected to be empty [empty-dirs]

Signed-off-by: Samuli Piippo 
---
 meta/recipes-devtools/rpm/rpm_4.17.0.bb | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/meta/recipes-devtools/rpm/rpm_4.17.0.bb 
b/meta/recipes-devtools/rpm/rpm_4.17.0.bb
index beb03ba3dd..7769d6a836 100644
--- a/meta/recipes-devtools/rpm/rpm_4.17.0.bb
+++ b/meta/recipes-devtools/rpm/rpm_4.17.0.bb
@@ -134,6 +134,9 @@ do_install:append:class-nativesdk() {
 do_install:append:class-target() {
 rm -rf ${D}/var
 }
+do_install:append:class-nativesdk() {
+rm -rf ${D}${SDKPATHNATIVE}/var
+}
 
 do_install:append () {
sed -i -e 's:${HOSTTOOLS_DIR}/::g' \
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#160113): 
https://lists.openembedded.org/g/openembedded-core/message/160113
Mute This Topic: https://lists.openembedded.org/mt/88108682/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] [poky][master][PATCH] assimp: added patch to fix hardcoded non-existing paths in CMake modules

2021-08-24 Thread Samuli Piippo
assimpTargets-release.cmake has it hardcoded:

set_target_properties(assimp::assimp PROPERTIES
  IMPORTED_IMPLIB_RELEASE "/usr/lib/${importLibraryName}"
  IMPORTED_LOCATION_RELEASE "/usr/bin/${sharedLibraryName}"
)

before it used to be:

set_target_properties(assimp::assimp PROPERTIES
  IMPORTED_IMPLIB_RELEASE "${_IMPORT_PREFIX}/lib/${importLibraryName}"
  IMPORTED_LOCATION_RELEASE "${_IMPORT_PREFIX}/bin/${sharedLibraryName}"
)


-samuli

On Tue, 24 Aug 2021 at 13:59,  wrote:

> Hi,
>
> On Tue, Aug 24, 2021 at 12:27:09PM +0300, Samuli Piippo wrote:
> > Looks like this commit breaks something in the cmake files.
> >
> > When using assimp from another recipe (Qt3D), it will try to use the
> > library from host path:
> >
> > /usr/lib/cmake/assimp-5.0/assimpTargets-release.cmake(69):
> >  set(sharedLibraryName libassimp.so.5 )
> > /usr/lib/cmake/assimp-5.0/assimpTargets-release.cmake(71):
> >  set_target_properties(assimp::assimp PROPERTIES IMPORTED_SONAME_RELEASE
> > libassimp.so.5 IMPORTED_LOCATION_RELEASE /usr/lib/libassimp.so.5 )
>
> Can you see where the /usr/lib search path comes from?
>
> To me, the assimp CMake files try to do way too much and should instead
> rely
> on default search paths and the cross compile toolchain file.
>
> Cheers,
>
> -Mikko
>
> > Whereas, before the patch, it was correctly using it from the sysroot:
> >
> > /usr/lib/cmake/assimp-5.0/assimpTargets-release.cmake(69):
> >  set(sharedLibraryName libassimp.so.5 )
> > /usr/lib/cmake/assimp-5.0/assimpTargets-release.cmake(71):
> >  set_target_properties(assimp::assimp PROPERTIES IMPORTED_SONAME_RELEASE
> > libassimp.so.5 IMPORTED_LOCATION_RELEASE
> >
> /home/qt/work/build/build/tmp/work/core2-32-poky-linux/qt3d/6.3.0-r0/recipe-sysroot/usr/lib/libassimp.so.5
> > )
> > /usr/lib/cmake/assimp-5.0/assimpTargets-release.cmake(75):  list(APPEND
> > _IMPORT_CHECK_TARGETS assimp::assimp )
> > /usr/lib/cmake/assimp-5.0/assimpTargets-release.cmake(76):  list(APPEND
> > _IMPORT_CHECK_FILES_FOR_assimp::assimp
> >
> /home/qt/work/build/build/tmp/work/core2-32-poky-linux/qt3d/6.3.0-r0/recipe-sysroot/usr/lib/libassimp.so.5
> > )
> >
> > Probably explains why the IMPORT_CHECK_TARGETS had to be removed.
> >
> >
> >
> > On Fri, 20 Aug 2021 at 13:07, Purushottam Choudhary <
> > purushottamchoudhar...@gmail.com> wrote:
> >
> > > assimp CMake modules were adding non-existing paths to its CMake
> modules
> > > breaking builds for users of assimp. Remove the hardcoded paths with
> > > an upstream patch with small tweaks.
> > >
> > > Signed-off-by: Purushottam Choudhary  >
> > > ---
> > >  ...-ASSIMP_LIB_INSTALL_DIR-to-search-library.patch |  68 --
> > >  ...ove-shared-lib-from-_IMPORT_CHECK_TARGETS.patch |  32 +++
> > >  .../assimp/use-GNUInstallDirs-where-possible.patch | 257
> > > +
> > >  meta/recipes-graphics/vulkan/assimp_5.0.1.bb   |   3 +-
> > >  4 files changed, 291 insertions(+), 69 deletions(-)
> > >  delete mode 100644
> > >
> meta/recipes-graphics/vulkan/assimp/0001-Use-ASSIMP_LIB_INSTALL_DIR-to-search-library.patch
> > >  create mode 100644
> > >
> meta/recipes-graphics/vulkan/assimp/0001-assimp-remove-shared-lib-from-_IMPORT_CHECK_TARGETS.patch
> > >  create mode 100644
> > >
> meta/recipes-graphics/vulkan/assimp/use-GNUInstallDirs-where-possible.patch
> > >
> > > diff --git
> > >
> a/meta/recipes-graphics/vulkan/assimp/0001-Use-ASSIMP_LIB_INSTALL_DIR-to-search-library.patch
> > >
> b/meta/recipes-graphics/vulkan/assimp/0001-Use-ASSIMP_LIB_INSTALL_DIR-to-search-library.patch
> > > deleted file mode 100644
> > > index 34c5bfa..000
> > > ---
> > >
> a/meta/recipes-graphics/vulkan/assimp/0001-Use-ASSIMP_LIB_INSTALL_DIR-to-search-library.patch
> > > +++ /dev/null
> > > @@ -1,68 +0,0 @@
> > > -From 58f576d6f240b08957a402960cdf06c74201ac7b Mon Sep 17 00:00:00 2001
> > > -From: Hongxu Jia 
> > > -Date: Thu, 12 Dec 2019 22:16:15 -0800
> > > -Subject: [PATCH] Use ASSIMP_LIB_INSTALL_DIR to search library
> > > -
> > > -Use ASSIMP_LIB_INSTALL_DIR rather than hardcoded /lib to search
> library
> > > -
> > > -Upstream-Status: Inappropriate [oe specific]
> > > -
> > > -Signed-off-by: Hongxu Jia 
> > > -
> > > 
> > > - assimpTargets-release.cmake.in | 16 
> > > - 1 file changed, 8

Re: [OE-core] [poky][master][PATCH] assimp: added patch to fix hardcoded non-existing paths in CMake modules

2021-08-24 Thread Samuli Piippo
Looks like this commit breaks something in the cmake files.

When using assimp from another recipe (Qt3D), it will try to use the
library from host path:

/usr/lib/cmake/assimp-5.0/assimpTargets-release.cmake(69):
 set(sharedLibraryName libassimp.so.5 )
/usr/lib/cmake/assimp-5.0/assimpTargets-release.cmake(71):
 set_target_properties(assimp::assimp PROPERTIES IMPORTED_SONAME_RELEASE
libassimp.so.5 IMPORTED_LOCATION_RELEASE /usr/lib/libassimp.so.5 )

Whereas, before the patch, it was correctly using it from the sysroot:

/usr/lib/cmake/assimp-5.0/assimpTargets-release.cmake(69):
 set(sharedLibraryName libassimp.so.5 )
/usr/lib/cmake/assimp-5.0/assimpTargets-release.cmake(71):
 set_target_properties(assimp::assimp PROPERTIES IMPORTED_SONAME_RELEASE
libassimp.so.5 IMPORTED_LOCATION_RELEASE
/home/qt/work/build/build/tmp/work/core2-32-poky-linux/qt3d/6.3.0-r0/recipe-sysroot/usr/lib/libassimp.so.5
)
/usr/lib/cmake/assimp-5.0/assimpTargets-release.cmake(75):  list(APPEND
_IMPORT_CHECK_TARGETS assimp::assimp )
/usr/lib/cmake/assimp-5.0/assimpTargets-release.cmake(76):  list(APPEND
_IMPORT_CHECK_FILES_FOR_assimp::assimp
/home/qt/work/build/build/tmp/work/core2-32-poky-linux/qt3d/6.3.0-r0/recipe-sysroot/usr/lib/libassimp.so.5
)

Probably explains why the IMPORT_CHECK_TARGETS had to be removed.



On Fri, 20 Aug 2021 at 13:07, Purushottam Choudhary <
purushottamchoudhar...@gmail.com> wrote:

> assimp CMake modules were adding non-existing paths to its CMake modules
> breaking builds for users of assimp. Remove the hardcoded paths with
> an upstream patch with small tweaks.
>
> Signed-off-by: Purushottam Choudhary 
> ---
>  ...-ASSIMP_LIB_INSTALL_DIR-to-search-library.patch |  68 --
>  ...ove-shared-lib-from-_IMPORT_CHECK_TARGETS.patch |  32 +++
>  .../assimp/use-GNUInstallDirs-where-possible.patch | 257
> +
>  meta/recipes-graphics/vulkan/assimp_5.0.1.bb   |   3 +-
>  4 files changed, 291 insertions(+), 69 deletions(-)
>  delete mode 100644
> meta/recipes-graphics/vulkan/assimp/0001-Use-ASSIMP_LIB_INSTALL_DIR-to-search-library.patch
>  create mode 100644
> meta/recipes-graphics/vulkan/assimp/0001-assimp-remove-shared-lib-from-_IMPORT_CHECK_TARGETS.patch
>  create mode 100644
> meta/recipes-graphics/vulkan/assimp/use-GNUInstallDirs-where-possible.patch
>
> diff --git
> a/meta/recipes-graphics/vulkan/assimp/0001-Use-ASSIMP_LIB_INSTALL_DIR-to-search-library.patch
> b/meta/recipes-graphics/vulkan/assimp/0001-Use-ASSIMP_LIB_INSTALL_DIR-to-search-library.patch
> deleted file mode 100644
> index 34c5bfa..000
> ---
> a/meta/recipes-graphics/vulkan/assimp/0001-Use-ASSIMP_LIB_INSTALL_DIR-to-search-library.patch
> +++ /dev/null
> @@ -1,68 +0,0 @@
> -From 58f576d6f240b08957a402960cdf06c74201ac7b Mon Sep 17 00:00:00 2001
> -From: Hongxu Jia 
> -Date: Thu, 12 Dec 2019 22:16:15 -0800
> -Subject: [PATCH] Use ASSIMP_LIB_INSTALL_DIR to search library
> -
> -Use ASSIMP_LIB_INSTALL_DIR rather than hardcoded /lib to search library
> -
> -Upstream-Status: Inappropriate [oe specific]
> -
> -Signed-off-by: Hongxu Jia 
> -
> 
> - assimpTargets-release.cmake.in | 16 
> - 1 file changed, 8 insertions(+), 8 deletions(-)
> -
> -diff --git a/assimpTargets-release.cmake.in b/
> assimpTargets-release.cmake.in
> -index f3db8f1..b8a066a 100644
>  a/assimpTargets-release.cmake.in
> -+++ b/assimpTargets-release.cmake.in
> -@@ -42,11 +42,11 @@ if(MSVC)
> - # Import target "assimp::assimp" for configuration "Release"
> - set_property(TARGET assimp::assimp APPEND PROPERTY
> IMPORTED_CONFIGURATIONS RELEASE)
> - set_target_properties(assimp::assimp PROPERTIES
> --  IMPORTED_IMPLIB_RELEASE
> "${_IMPORT_PREFIX}/lib/${importLibraryName}"
> -+  IMPORTED_IMPLIB_RELEASE "${_IMPORT_PREFIX}/@ASSIMP_LIB_INSTALL_DIR@
> /${importLibraryName}"
> -   IMPORTED_LOCATION_RELEASE
> "${_IMPORT_PREFIX}/bin/${sharedLibraryName}"
> - )
> - list(APPEND _IMPORT_CHECK_TARGETS assimp::assimp )
> --list(APPEND _IMPORT_CHECK_FILES_FOR_assimp::assimp
> "${_IMPORT_PREFIX}/lib/${importLibraryName}")
> -+list(APPEND _IMPORT_CHECK_FILES_FOR_assimp::assimp
> "${_IMPORT_PREFIX}/@ASSIMP_LIB_INSTALL_DIR@/${importLibraryName}")
> - list(APPEND _IMPORT_CHECK_FILES_FOR_assimp::assimp
> "${_IMPORT_PREFIX}/bin/${sharedLibraryName}" )
> -   else()
> - set(staticLibraryName
> "assimp${ASSIMP_LIBRARY_SUFFIX}@CMAKE_STATIC_LIBRARY_SUFFIX@")
> -@@ -54,10 +54,10 @@ if(MSVC)
> - # Import target "assimp::assimp" for configuration "Release"
> - set_property(TARGET assimp::assimp APPEND PROPERTY
> IMPORTED_CONFIGURATIONS RELEASE)
> - set_target_properties(assimp::assimp PROPERTIES
> --  IMPORTED_LOCATION_RELEASE
> "${_IMPORT_PREFIX}/lib/${staticLibraryName}"
> -+  IMPORTED_LOCATION_RELEASE
> "${_IMPORT_PREFIX}/@ASSIMP_LIB_INSTALL_DIR@/${staticLibraryName}"
> - )
> - list(APPEND _IMPORT_CHECK_TARGETS assimp::assimp )
> --list(APPEND _IMPORT_CHECK_FILES_FOR_assimp::assimp
> 

Re: [OE-core] [PATCH v3] Revert "gcc-cross-canadian: Add symlink to real-ld alongside other symlinks"

2021-05-28 Thread Samuli Piippo
Tested this locally and gcc is trying to find the linker with the multilib
prefix, but that's not available in SDK.
[pid  3784] newfstatat(AT_FDCWD,
"/home/sapiippo/build/toolchain/sysroots/x86_64-pokysdk-linux/usr/libexec/mips-poky-linux/gcc/mips-poky-linux/11.1.0/mips-pokymllib32-linux-ld",
0x7ffe380d44a0, 0) = -1 ENOENT (No such file or directory)

The same works on bitbake builds, where lib32-gcc-cross-mips add the
symlink:
mips-pokymllib32-linux-ld ->
../../../../../bin/mips-pokymllib32-linux/mips-pokymllib32-linux-ld
(even though the binary is built in lib32-binutils-cross-mips)

So, the cross-canadian builds are missing all the multilib links in libexec
that are available in cross builds.

-samuli

On Wed, 26 May 2021 at 12:44, Richard Purdie <
richard.pur...@linuxfoundation.org> wrote:

> On Wed, 2021-05-26 at 12:23 +0300, Samuli Piippo wrote:
> > Was the "gcc-cross-canadian: add symlinks for ld.bfd and ld.gold" change
> > included in that test build?
>
> Yes, it was.
>
> Cheers,
>
> Richard
>
>

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



Re: [OE-core] [PATCH v3] Revert "gcc-cross-canadian: Add symlink to real-ld alongside other symlinks"

2021-05-26 Thread Samuli Piippo
Was the "gcc-cross-canadian: add symlinks for ld.bfd and ld.gold" change
included in that test build?


On Sat, 22 May 2021 at 08:41, Richard Purdie <
richard.pur...@linuxfoundation.org> wrote:

> On Fri, 2021-05-21 at 17:33 +0300, Samuli Piippo wrote:
> > This reverts commit cdd86896c8d29135f937968e9aa07f919cf543d3.
> >
> > real-ld is always used if that is found, which means you cannot
> > switch between bfd and gold linkers using -fuse-ld gcc option.
> >
> > Signed-off-by: Samuli Piippo 
> > ---
> >  meta/recipes-devtools/gcc/gcc-cross-canadian.inc | 2 --
> >  1 file changed, 2 deletions(-)
> >
> > diff --git a/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
> b/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
> > index 878feaf810..e8fbb6186b 100644
> > --- a/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
> > +++ b/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
> > @@ -131,8 +131,6 @@ do_install () {
> >
> >
> >
> >
> >   ln -sf ${BINRELPATH}/${TARGET_PREFIX}$t$suffix
> $dest$t$suffix
> >   done
> > - t=real-ld
> > - ln -sf ${BINRELPATH}/${TARGET_PREFIX}ld$suffix $dest$t$suffix
> >
> >
> >
> >
> >   # libquadmath headers need to  be available in the gcc libexec dir
> >   install -d ${D}${libdir}/gcc/${TARGET_SYS}/${BINV}/include/
>
> It seems the issue that commit fixed back in 2015 is still there if
> we revert this:
>
>
> https://autobuilder.yoctoproject.org/typhoon/#/builders/44/builds/3490/steps/26/logs/stdio
>
> Cheers,
>
> Richard
>
>

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#152257): 
https://lists.openembedded.org/g/openembedded-core/message/152257
Mute This Topic: https://lists.openembedded.org/mt/82987140/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] cmake-native: enabled zstd support

2021-05-25 Thread Samuli Piippo
commit messages updated in v2 patches

-samuli

On Tue, 25 May 2021 at 15:00, Alexander Kanavin 
wrote:

> Thanks, it would be beneficial to record this information in the commit
> message.
>
> Alex
>
> On Tue, 25 May 2021 at 13:27, Samuli Piippo 
> wrote:
>
>> Issue came up while building Qt6, which uses cmake packaging API:
>> https://gitlab.kitware.com/cmake/cmake/-/issues/21552
>> Things break when zstd support is not available and CMake doesn't provide
>> simple why to ask if the support is available.
>>
>> Quote from cmake dev:
>> As far as CMake's design is concerned, we have no optional formats. All
>> should be supported.
>> That's why we bundle sufficiently new versions of libarchive and libzstd.
>> If a distro builds with an older libarchive that doesn't have zstd
>> support, then that is not a proper packaging of CMake.
>>
>> For target/nativesdk CMake depends on libarchive for this support, which
>> is covered in the second patch.
>>
>> -samuli
>>
>> On Tue, 25 May 2021 at 13:07, Alexander Kanavin 
>> wrote:
>>
>>> You need to explain the benefit in doing so, I think? :)
>>>
>>> Alex
>>>
>>> On Tue, 25 May 2021 at 12:05, Samuli Piippo 
>>> wrote:
>>>
>>>> Now that zstd is in oe-core, enable support for it in native CMake.
>>>>
>>>> Signed-off-by: Samuli Piippo 
>>>> ---
>>>>  meta/recipes-devtools/cmake/cmake-native_3.20.1.bb | 2 +-
>>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>>
>>>> diff --git a/meta/recipes-devtools/cmake/cmake-native_3.20.1.bb
>>>> b/meta/recipes-devtools/cmake/cmake-native_3.20.1.bb
>>>> index d91e42ef9a..335097d387 100644
>>>> --- a/meta/recipes-devtools/cmake/cmake-native_3.20.1.bb
>>>> +++ b/meta/recipes-devtools/cmake/cmake-native_3.20.1.bb
>>>> @@ -1,7 +1,7 @@
>>>>  require cmake.inc
>>>>  inherit native
>>>>
>>>> -DEPENDS += "bzip2-replacement-native xz-native zlib-native curl-native
>>>> ncurses-native"
>>>> +DEPENDS += "bzip2-replacement-native xz-native zlib-native curl-native
>>>> ncurses-native zstd-native"
>>>>
>>>>  SRC_URI += "file://OEToolchainConfig.cmake \
>>>>  file://environment.d-cmake.sh \
>>>> --
>>>> 2.25.1
>>>>
>>>>
>>>> 
>>>>
>>>>

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#152224): 
https://lists.openembedded.org/g/openembedded-core/message/152224
Mute This Topic: https://lists.openembedded.org/mt/83072435/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] cmake-native: enabled zstd support

2021-05-25 Thread Samuli Piippo
CMake depends on having all formats supported and build issues can
arise when zstd is not available:
https://gitlab.kitware.com/cmake/cmake/-/issues/21552

Quote from a CMake dev:
"As far as CMake's design is concerned, we have no optional formats.
All should be supported. That's why we bundle sufficiently new versions
of libarchive and libzstd. If a distro builds with an older libarchive
that doesn't have zstd support, then that is not a proper packaging of CMake."

Signed-off-by: Samuli Piippo 
---
 meta/recipes-devtools/cmake/cmake-native_3.20.1.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-devtools/cmake/cmake-native_3.20.1.bb 
b/meta/recipes-devtools/cmake/cmake-native_3.20.1.bb
index d91e42ef9a..335097d387 100644
--- a/meta/recipes-devtools/cmake/cmake-native_3.20.1.bb
+++ b/meta/recipes-devtools/cmake/cmake-native_3.20.1.bb
@@ -1,7 +1,7 @@
 require cmake.inc
 inherit native
 
-DEPENDS += "bzip2-replacement-native xz-native zlib-native curl-native 
ncurses-native"
+DEPENDS += "bzip2-replacement-native xz-native zlib-native curl-native 
ncurses-native zstd-native"
 
 SRC_URI += "file://OEToolchainConfig.cmake \
 file://environment.d-cmake.sh \
-- 
2.25.1


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

2021-05-25 Thread Samuli Piippo
Now that zstd is in oe-core, add PACKAGECONFIG for it and enabled
it by default in libarchive.

zstd support is expected by CMake, which in nativesdk depends on
libarchive. CMake depends on having all formats supported and build
issues can arise when zstd is not available:
https://gitlab.kitware.com/cmake/cmake/-/issues/21552

Quote from a CMake dev:
"As far as CMake's design is concerned, we have no optional formats.
All should be supported. That's why we bundle sufficiently new versions
of libarchive and libzstd. If a distro builds with an older libarchive
that doesn't have zstd support, then that is not a proper packaging of CMake."

Signed-off-by: Samuli Piippo 
---
 meta/recipes-extended/libarchive/libarchive_3.5.1.bb | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-extended/libarchive/libarchive_3.5.1.bb 
b/meta/recipes-extended/libarchive/libarchive_3.5.1.bb
index c9e30f468f..ebecee1f3e 100644
--- a/meta/recipes-extended/libarchive/libarchive_3.5.1.bb
+++ b/meta/recipes-extended/libarchive/libarchive_3.5.1.bb
@@ -7,7 +7,7 @@ LIC_FILES_CHKSUM = 
"file://COPYING;md5=d499814247adaee08d88080841cb5665"
 
 DEPENDS = "e2fsprogs-native"
 
-PACKAGECONFIG ?= "zlib bz2 xz lzo"
+PACKAGECONFIG ?= "zlib bz2 xz lzo zstd"
 
 PACKAGECONFIG_append_class-target = "\
${@bb.utils.filter('DISTRO_FEATURES', 'acl xattr', d)} \
@@ -28,6 +28,7 @@ PACKAGECONFIG[lzo] = "--with-lzo2,--without-lzo2,lzo,"
 PACKAGECONFIG[nettle] = "--with-nettle,--without-nettle,nettle,"
 PACKAGECONFIG[lz4] = "--with-lz4,--without-lz4,lz4,"
 PACKAGECONFIG[mbedtls] = "--with-mbedtls,--without-mbedtls,mbedtls,"
+PACKAGECONFIG[zstd] = "--with-zstd,--without-zstd,zstd,"
 
 EXTRA_OECONF += "--enable-largefile"
 
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#15): 
https://lists.openembedded.org/g/openembedded-core/message/15
Mute This Topic: https://lists.openembedded.org/mt/83074750/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] cmake-native: enabled zstd support

2021-05-25 Thread Samuli Piippo
Issue came up while building Qt6, which uses cmake packaging API:
https://gitlab.kitware.com/cmake/cmake/-/issues/21552
Things break when zstd support is not available and CMake doesn't provide
simple why to ask if the support is available.

Quote from cmake dev:
As far as CMake's design is concerned, we have no optional formats. All
should be supported.
That's why we bundle sufficiently new versions of libarchive and libzstd.
If a distro builds with an older libarchive that doesn't have zstd support,
then that is not a proper packaging of CMake.

For target/nativesdk CMake depends on libarchive for this support, which is
covered in the second patch.

-samuli

On Tue, 25 May 2021 at 13:07, Alexander Kanavin 
wrote:

> You need to explain the benefit in doing so, I think? :)
>
> Alex
>
> On Tue, 25 May 2021 at 12:05, Samuli Piippo 
> wrote:
>
>> Now that zstd is in oe-core, enable support for it in native CMake.
>>
>> Signed-off-by: Samuli Piippo 
>> ---
>>  meta/recipes-devtools/cmake/cmake-native_3.20.1.bb | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/meta/recipes-devtools/cmake/cmake-native_3.20.1.bb
>> b/meta/recipes-devtools/cmake/cmake-native_3.20.1.bb
>> index d91e42ef9a..335097d387 100644
>> --- a/meta/recipes-devtools/cmake/cmake-native_3.20.1.bb
>> +++ b/meta/recipes-devtools/cmake/cmake-native_3.20.1.bb
>> @@ -1,7 +1,7 @@
>>  require cmake.inc
>>  inherit native
>>
>> -DEPENDS += "bzip2-replacement-native xz-native zlib-native curl-native
>> ncurses-native"
>> +DEPENDS += "bzip2-replacement-native xz-native zlib-native curl-native
>> ncurses-native zstd-native"
>>
>>  SRC_URI += "file://OEToolchainConfig.cmake \
>>  file://environment.d-cmake.sh \
>> --
>> 2.25.1
>>
>>
>> 
>>
>>

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

2021-05-25 Thread Samuli Piippo
Now that zstd is in oe-core, enable support for it in native CMake.

Signed-off-by: Samuli Piippo 
---
 meta/recipes-devtools/cmake/cmake-native_3.20.1.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-devtools/cmake/cmake-native_3.20.1.bb 
b/meta/recipes-devtools/cmake/cmake-native_3.20.1.bb
index d91e42ef9a..335097d387 100644
--- a/meta/recipes-devtools/cmake/cmake-native_3.20.1.bb
+++ b/meta/recipes-devtools/cmake/cmake-native_3.20.1.bb
@@ -1,7 +1,7 @@
 require cmake.inc
 inherit native
 
-DEPENDS += "bzip2-replacement-native xz-native zlib-native curl-native 
ncurses-native"
+DEPENDS += "bzip2-replacement-native xz-native zlib-native curl-native 
ncurses-native zstd-native"
 
 SRC_URI += "file://OEToolchainConfig.cmake \
 file://environment.d-cmake.sh \
-- 
2.25.1


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

2021-05-25 Thread Samuli Piippo
Now that zstd is in oe-core, add PACKAGECONFIG for it and enabled
it by default in libarchive.

zstd support is expected by CMake, which in nativesdk depends on
libarchive (https://gitlab.kitware.com/cmake/cmake/-/issues/21552)
---
 meta/recipes-extended/libarchive/libarchive_3.5.1.bb | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-extended/libarchive/libarchive_3.5.1.bb 
b/meta/recipes-extended/libarchive/libarchive_3.5.1.bb
index c9e30f468f..ebecee1f3e 100644
--- a/meta/recipes-extended/libarchive/libarchive_3.5.1.bb
+++ b/meta/recipes-extended/libarchive/libarchive_3.5.1.bb
@@ -7,7 +7,7 @@ LIC_FILES_CHKSUM = 
"file://COPYING;md5=d499814247adaee08d88080841cb5665"
 
 DEPENDS = "e2fsprogs-native"
 
-PACKAGECONFIG ?= "zlib bz2 xz lzo"
+PACKAGECONFIG ?= "zlib bz2 xz lzo zstd"
 
 PACKAGECONFIG_append_class-target = "\
${@bb.utils.filter('DISTRO_FEATURES', 'acl xattr', d)} \
@@ -28,6 +28,7 @@ PACKAGECONFIG[lzo] = "--with-lzo2,--without-lzo2,lzo,"
 PACKAGECONFIG[nettle] = "--with-nettle,--without-nettle,nettle,"
 PACKAGECONFIG[lz4] = "--with-lz4,--without-lz4,lz4,"
 PACKAGECONFIG[mbedtls] = "--with-mbedtls,--without-mbedtls,mbedtls,"
+PACKAGECONFIG[zstd] = "--with-zstd,--without-zstd,zstd,"
 
 EXTRA_OECONF += "--enable-largefile"
 
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#152204): 
https://lists.openembedded.org/g/openembedded-core/message/152204
Mute This Topic: https://lists.openembedded.org/mt/83072226/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] gcc-cross-canadian: add symlinks for ld.bfd and ld.gold

2021-05-21 Thread Samuli Piippo
When -fuse-ld gcc option is used, missing ld.bfd or ld.gold symlinks
can lead to linker error:

collect2: fatal error: cannot find 'ld'

Signed-off-by: Samuli Piippo 
---
 meta/recipes-devtools/gcc/gcc-cross-canadian.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-devtools/gcc/gcc-cross-canadian.inc 
b/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
index e8fbb6186b..2d025b54ca 100644
--- a/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
+++ b/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
@@ -124,7 +124,7 @@ do_install () {
dest=${D}${libexecdir}/gcc/${TARGET_SYS}/${BINV}/
install -d $dest
suffix=${EXEEXT}
-   for t in ar as ld nm objcopy objdump ranlib strip g77 gcc cpp gfortran; 
do
+   for t in ar as ld ld.bfd ld.gold nm objcopy objdump ranlib strip g77 
gcc cpp gfortran; do
if [ "$t" = "g77" -o "$t" = "gfortran" ] && [ ! -e 
${D}${bindir}/${TARGET_PREFIX}$t$suffix ]; then
continue
fi
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#152129): 
https://lists.openembedded.org/g/openembedded-core/message/152129
Mute This Topic: https://lists.openembedded.org/mt/82987143/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] Revert "gcc-cross-canadian: Add symlink to real-ld alongside other symlinks"

2021-05-21 Thread Samuli Piippo
This reverts commit cdd86896c8d29135f937968e9aa07f919cf543d3.

real-ld is always used if that is found, which means you cannot
switch between bfd and gold linkers using -fuse-ld gcc option.

Signed-off-by: Samuli Piippo 
---
 meta/recipes-devtools/gcc/gcc-cross-canadian.inc | 2 --
 1 file changed, 2 deletions(-)

diff --git a/meta/recipes-devtools/gcc/gcc-cross-canadian.inc 
b/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
index 878feaf810..e8fbb6186b 100644
--- a/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
+++ b/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
@@ -131,8 +131,6 @@ do_install () {
 
ln -sf ${BINRELPATH}/${TARGET_PREFIX}$t$suffix $dest$t$suffix
done
-   t=real-ld
-   ln -sf ${BINRELPATH}/${TARGET_PREFIX}ld$suffix $dest$t$suffix
 
# libquadmath headers need to  be available in the gcc libexec dir
install -d ${D}${libdir}/gcc/${TARGET_SYS}/${BINV}/include/
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#152128): 
https://lists.openembedded.org/g/openembedded-core/message/152128
Mute This Topic: https://lists.openembedded.org/mt/82987140/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] assimp: BBCLASSEXTEND to native and nativesdk

2021-04-08 Thread Samuli Piippo
At least some Qt tooling depends on assimp.

Signed-off-by: Samuli Piippo 
---
 meta/recipes-graphics/vulkan/assimp_5.0.1.bb | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/meta/recipes-graphics/vulkan/assimp_5.0.1.bb 
b/meta/recipes-graphics/vulkan/assimp_5.0.1.bb
index 5a8c62e64d..4eb0cefffc 100644
--- a/meta/recipes-graphics/vulkan/assimp_5.0.1.bb
+++ b/meta/recipes-graphics/vulkan/assimp_5.0.1.bb
@@ -21,3 +21,5 @@ S = "${WORKDIR}/git"
 inherit cmake
 
 EXTRA_OECMAKE = "-DASSIMP_BUILD_ASSIMP_TOOLS=OFF -DASSIMP_BUILD_TESTS=OFF 
-DASSIMP_LIB_INSTALL_DIR=${baselib}"
+
+BBCLASSEXTEND = "native nativesdk"
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#150291): 
https://lists.openembedded.org/g/openembedded-core/message/150291
Mute This Topic: https://lists.openembedded.org/mt/81935971/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][zeus][PATCH] populate_sdk_base.bbclass: fix warning: name not matched

2020-09-16 Thread Samuli Piippo
Hi,

As I understand it, the original zip format support was done so that there
wouldn't be any symlinks in the sdk package.
https://git.openembedded.org/openembedded-core/commit/?id=57a33048a89a422cfdc986d3489c67b2d297e1e7

With this change the symlinks are now back again which leads to broken SDK
when extracted with 7Zip.
Both commits were contributed from windriver, are you now using something
else to extract the zip?

-samuli

On Fri, 8 May 2020 at 07:34, wenlin.k...@windriver.com <
wenlin.k...@windriver.com> wrote:

> On 2020/5/8 上午11:04, Mittal, Anuj wrote:
>
> Hi,
>
> On Wed, 2020-05-06 at 01:46 -0700, wenlin.k...@windriver.com wrote:
>
> Fix below warning:
> zip warning: name not matched: sysroots/aarch64-wrs-
> linux/etc/udev/rules.d/80-net-setup-link.rules
> zip warning: name not matched: sysroots/aarch64-wrs-
> linux/etc/tmpfiles.d/etc.conf
> zip warning: name not matched: sysroots/aarch64-wrs-
> linux/etc/tmpfiles.d/home.conf
> zip warning: name not matched: sysroots/aarch64-wrs-
> linux/etc/systemd/network/80-wired.network
> zip warning: name not matched: sysroots/aarch64-wrs-
> linux/etc/resolv.conf
> zip warning: name not matched: sysroots/aarch64-wrs-linux/etc/mtab
> zip warning: name not matched: sysroots/aarch64-wrs-linux/etc/resolv-
> conf.systemd
> zip warning: name not matched: sysroots/aarch64-wrs-linux/var/lock
>
>
> Is this specific to zeus? I don't see this change in master/dunfell.
> It'd be great if you could include more details in commit message
> explaining what is happening.
>
>
> No, this issue can be seen in master too, but this patch is only to zeus,
> for master, I have sent patch too.
>
> Steps:
>
> 1. Setup poky[zeus] project
>
> 2. In local.conf, add:
> SDK_ARCHIVE_TYPE = "zip"
>
> 3. bitbake core-image-minimal -c populate_sdk
>
> 4.  check log file log.do_populate_sdk
>
>
> Thanks,
>
> Anuj
>
>
> Signed-off-by: Wenlin Kang  
> 
> ---
>  meta/classes/populate_sdk_base.bbclass | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta/classes/populate_sdk_base.bbclass
> b/meta/classes/populate_sdk_base.bbclass
> index d03465b6fc..b5c004d832 100644
> --- a/meta/classes/populate_sdk_base.bbclass
> +++ b/meta/classes/populate_sdk_base.bbclass
> @@ -55,7 +55,7 @@ python () {
> d.setVar('SDK_ARCHIVE_DEPENDS', 'zip-native')
> # SDK_ARCHIVE_CMD used to generate archived sdk
> ${TOOLCHAIN_OUTPUTNAME}.${SDK_ARCHIVE_TYPE} from input dir
> ${SDK_OUTPUT}/${SDKPATH} to output dir ${SDKDEPLOYDIR}
> # recommand to cd into input dir first to avoid archive with
> buildpath
> -   d.setVar('SDK_ARCHIVE_CMD', 'cd ${SDK_OUTPUT}/${SDKPATH}; zip
> -r ${SDKDEPLOYDIR}/${TOOLCHAIN_OUTPUTNAME}.${SDK_ARCHIVE_TYPE} .')
> +   d.setVar('SDK_ARCHIVE_CMD', 'cd ${SDK_OUTPUT}/${SDKPATH}; zip
> -r -y ${SDKDEPLOYDIR}/${TOOLCHAIN_OUTPUTNAME}.${SDK_ARCHIVE_TYPE} .')
>  else:
> d.setVar('SDK_ARCHIVE_DEPENDS', 'xz-native')
> d.setVar('SDK_ARCHIVE_CMD', 'cd ${SDK_OUTPUT}/${SDKPATH}; tar
> ${SDKTAROPTS} -cf - . | xz -T 0 -9 >
> ${SDKDEPLOYDIR}/${TOOLCHAIN_OUTPUTNAME}.${SDK_ARCHIVE_TYPE}')
>
>
>
> --
> Thanks,
> Wenlin Kang
>
> 
>
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#142604): 
https://lists.openembedded.org/g/openembedded-core/message/142604
Mute This Topic: https://lists.openembedded.org/mt/74024894/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] perl: use relative paths in the perl wrapper

2020-06-16 Thread Samuli Piippo
Use SDKPATHNATIVE so that the perl wrapper for the nativesdk can be created
using relative paths and without dependency to the environment variables.

Signed-off-by: Samuli Piippo 
---
 meta/recipes-devtools/perl/perl_5.30.2.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-devtools/perl/perl_5.30.2.bb 
b/meta/recipes-devtools/perl/perl_5.30.2.bb
index 26138ea9e5..7158102ca7 100644
--- a/meta/recipes-devtools/perl/perl_5.30.2.bb
+++ b/meta/recipes-devtools/perl/perl_5.30.2.bb
@@ -156,7 +156,7 @@ do_install_append_class-nativesdk() {
 ln -s Config_heavy.pl 
${D}${libdir}/perl5/${PV}/${TARGET_ARCH}-linux/Config_heavy-target.pl
 
 create_wrapper ${D}${bindir}/perl \
-
PERL5LIB='$PERL5LIB:$OECORE_NATIVE_SYSROOT/${libdir_nativesdk}/perl5/site_perl/${PV}:$OECORE_NATIVE_SYSROOT/${libdir_nativesdk}/perl5/vendor_perl/${PV}:$OECORE_NATIVE_SYSROOT/${libdir_nativesdk}/perl5/${PV}'
+
PERL5LIB='$PERL5LIB:${SDKPATHNATIVE}/${libdir_nativesdk}/perl5/site_perl/${PV}:${SDKPATHNATIVE}/${libdir_nativesdk}/perl5/vendor_perl/${PV}:${SDKPATHNATIVE}/${libdir_nativesdk}/perl5/${PV}'
 }
 
 do_install_append_class-native () {
-- 
2.25.1

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

View/Reply Online (#139560): 
https://lists.openembedded.org/g/openembedded-core/message/139560
Mute This Topic: https://lists.openembedded.org/mt/74917437/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: allow chainloading of the toolchain file

2020-06-16 Thread Samuli Piippo
Use path from CMAKE_CURRENT_LIST_FILE to load the cmake subscripts.
This allows the toolchain file to be chainloaded from another toolchain file.

Signed-off-by: Samuli Piippo 
---
 meta/recipes-devtools/cmake/cmake/OEToolchainConfig.cmake | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-devtools/cmake/cmake/OEToolchainConfig.cmake 
b/meta/recipes-devtools/cmake/cmake/OEToolchainConfig.cmake
index 398069eef2..8f6f3a272d 100644
--- a/meta/recipes-devtools/cmake/cmake/OEToolchainConfig.cmake
+++ b/meta/recipes-devtools/cmake/cmake/OEToolchainConfig.cmake
@@ -19,7 +19,7 @@ if ($ENV{SDKTARGETSYSROOT} MATCHES 
"/sysroots/([a-zA-Z0-9_-]+)-.+-.+")
 endif()
 
 # Include the toolchain configuration subscripts
-file( GLOB toolchain_config_files "${CMAKE_TOOLCHAIN_FILE}.d/*.cmake" )
+file( GLOB toolchain_config_files "${CMAKE_CURRENT_LIST_FILE}.d/*.cmake" )
 foreach(config ${toolchain_config_files})
 include(${config})
 endforeach()
-- 
2.25.1

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

View/Reply Online (#139557): 
https://lists.openembedded.org/g/openembedded-core/message/139557
Mute This Topic: https://lists.openembedded.org/mt/74916584/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] gcc: make sure mingw builds use mingw32-w64 configs

2020-04-27 Thread Samuli Piippo
On Mon, 27 Apr 2020 at 23:20, Joshua Watt  wrote:
>
>
> On 4/27/20 12:35 PM, Samuli Piippo wrote:
>
> On Mon, 27 Apr 2020 at 17:51, Richard Purdie
>  wrote:
>
> On Mon, 2020-04-27 at 16:13 +0300, Samuli Piippo wrote:
>
> meta-mingw is using Mingw-w64 but the host triplet does not match
> the expected pattern. config/os/mingw32-w64 enables more features
> that were not working with mingw32.
>
> Signed-off-by: Samuli Piippo 
> ---
>  meta/recipes-devtools/gcc/gcc-9.3.inc |  1 +
>  ...nfig-os-mingw32-w64-for-mingw-builds.patch | 36 +++
>  2 files changed, 37 insertions(+)
>  create mode 100644 
> meta/recipes-devtools/gcc/gcc-9.3/0040-Always-use-config-os-mingw32-w64-for-mingw-builds.patch
>
> diff --git a/meta/recipes-devtools/gcc/gcc-9.3.inc 
> b/meta/recipes-devtools/gcc/gcc-9.3.inc
> index b0411078d3..4c8275bf2a 100644
> --- a/meta/recipes-devtools/gcc/gcc-9.3.inc
> +++ b/meta/recipes-devtools/gcc/gcc-9.3.inc
> @@ -67,6 +67,7 @@ SRC_URI = "\
> 
> file://0037-CVE-2019-14250-Check-zero-value-in-simple_object_elf.patch \
> 
> file://0038-gentypes-genmodes-Do-not-use-__LINE__-for-maintainin.patch \
> 
> file://0039-process_alt_operands-Don-t-match-user-defined-regs-o.patch \
> +   
> file://0040-Always-use-config-os-mingw32-w64-for-mingw-builds.patch \
>  "
>  S = "${TMPDIR}/work-shared/gcc-${PV}-${PR}/gcc-${PV}"
>  SRC_URI[sha256sum] = 
> "71e197867611f6054aa1119b13a0c0abac12834765fe2d81f35ac57f84f742d1"
> diff --git 
> a/meta/recipes-devtools/gcc/gcc-9.3/0040-Always-use-config-os-mingw32-w64-for-mingw-builds.patch
>  
> b/meta/recipes-devtools/gcc/gcc-9.3/0040-Always-use-config-os-mingw32-w64-for-mingw-builds.patch
> new file mode 100644
> index 00..5a167f252c
> --- /dev/null
> +++ 
> b/meta/recipes-devtools/gcc/gcc-9.3/0040-Always-use-config-os-mingw32-w64-for-mingw-builds.patch
> @@ -0,0 +1,36 @@
> +From 052f7e5db0d793155015530b639c4e42d37eb7eb Mon Sep 17 00:00:00 2001
> +From: Samuli Piippo 
> +Date: Fri, 23 Aug 2019 10:10:43 +0300
> +Subject: [PATCH] Always use config/os/mingw32-w64 for mingw builds
> +
> +Make sure we use the w64 version of the configs, since that's what
> +meta-mingw is using.
> +
> +Signed-off-by: Samuli Piippo 
> +---
> + libstdc++-v3/configure.host | 12 ++--
> + 1 file changed, 2 insertions(+), 10 deletions(-)
> +
> +diff --git a/libstdc++-v3/configure.host b/libstdc++-v3/configure.host
> +index 155a3cdea..adbda8784 100644
> +--- a/libstdc++-v3/configure.host
>  b/libstdc++-v3/configure.host
> +@@ -277,16 +277,8 @@ case "${host_os}" in
> + os_include_dir="os/hpux"
> + ;;
> +   mingw32*)
> +-case "$host" in
> +-  *-w64-*)
> +-os_include_dir="os/mingw32-w64"
> +-error_constants_dir="os/mingw32-w64"
> +-;;
> +-  *)
> +-os_include_dir="os/mingw32"
> +-error_constants_dir="os/mingw32"
> +-;;
> +-esac
> ++os_include_dir="os/mingw32-w64"
> ++error_constants_dir="os/mingw32-w64"
> + OPT_LDFLAGS="${OPT_LDFLAGS} \$(lt_host_flags)"
>
> It might make things a bit more obvious and upstreamable to just change
>   *-w64-*)
> to
>   *-w64*)
> if I understand correct?
>
> With meta-mingw, the host triplet is actually either
> i686-pokysdk-mingw32 or x86_64-pokysdk-mingw32, depending on bitness
> in the SDKMACHINE value.
>
> Hmm, it does indeed look like we are not correctly setting the host in a way 
> it can be detected by GCC. It's a little surprising this hasn't caused any 
> actual problems (at least up to this point)... is this something new to gcc 
> 9.3, and do we have any idea why the AB MinGW tests aren't seeing any 
> problems?
>
> Either way, it would probably be better to figure out a way to make the host 
> look correct in meta-mingw if at all possible.
>
> Your commit message is also a little confusing because the first few times I 
> read it, I thought we were configuring for 64-bit when 32-bit is desired, but 
> based on the current GCC code being replaced and the value of the 
> host-triplets, it would seem to be the opposite (we are getting 32-bit when 
> 64-bit is desired).

It's not actually the bitness that GCC is looking for here but the
version of the MinGW. The MingGW-w64 is a fork of the original MinGW
and can be used for both 32- and 64-bit builds. With the os/mingw32
config, GCC is using subset of the features that are available in
os/mingw32-w64, so builds still work although perhaps not always
optimally.

I don't see clear way to change the host

Re: [OE-core] [PATCH] gcc: make sure mingw builds use mingw32-w64 configs

2020-04-27 Thread Samuli Piippo
On Mon, 27 Apr 2020 at 17:51, Richard Purdie
 wrote:
>
> On Mon, 2020-04-27 at 16:13 +0300, Samuli Piippo wrote:
> > meta-mingw is using Mingw-w64 but the host triplet does not match
> > the expected pattern. config/os/mingw32-w64 enables more features
> > that were not working with mingw32.
> >
> > Signed-off-by: Samuli Piippo 
> > ---
> >  meta/recipes-devtools/gcc/gcc-9.3.inc |  1 +
> >  ...nfig-os-mingw32-w64-for-mingw-builds.patch | 36 +++
> >  2 files changed, 37 insertions(+)
> >  create mode 100644 
> > meta/recipes-devtools/gcc/gcc-9.3/0040-Always-use-config-os-mingw32-w64-for-mingw-builds.patch
> >
> > diff --git a/meta/recipes-devtools/gcc/gcc-9.3.inc 
> > b/meta/recipes-devtools/gcc/gcc-9.3.inc
> > index b0411078d3..4c8275bf2a 100644
> > --- a/meta/recipes-devtools/gcc/gcc-9.3.inc
> > +++ b/meta/recipes-devtools/gcc/gcc-9.3.inc
> > @@ -67,6 +67,7 @@ SRC_URI = "\
> > 
> > file://0037-CVE-2019-14250-Check-zero-value-in-simple_object_elf.patch \
> > 
> > file://0038-gentypes-genmodes-Do-not-use-__LINE__-for-maintainin.patch \
> > 
> > file://0039-process_alt_operands-Don-t-match-user-defined-regs-o.patch \
> > +   
> > file://0040-Always-use-config-os-mingw32-w64-for-mingw-builds.patch \
> >  "
> >  S = "${TMPDIR}/work-shared/gcc-${PV}-${PR}/gcc-${PV}"
> >  SRC_URI[sha256sum] = 
> > "71e197867611f6054aa1119b13a0c0abac12834765fe2d81f35ac57f84f742d1"
> > diff --git 
> > a/meta/recipes-devtools/gcc/gcc-9.3/0040-Always-use-config-os-mingw32-w64-for-mingw-builds.patch
> >  
> > b/meta/recipes-devtools/gcc/gcc-9.3/0040-Always-use-config-os-mingw32-w64-for-mingw-builds.patch
> > new file mode 100644
> > index 000000..5a167f252c
> > --- /dev/null
> > +++ 
> > b/meta/recipes-devtools/gcc/gcc-9.3/0040-Always-use-config-os-mingw32-w64-for-mingw-builds.patch
> > @@ -0,0 +1,36 @@
> > +From 052f7e5db0d793155015530b639c4e42d37eb7eb Mon Sep 17 00:00:00 2001
> > +From: Samuli Piippo 
> > +Date: Fri, 23 Aug 2019 10:10:43 +0300
> > +Subject: [PATCH] Always use config/os/mingw32-w64 for mingw builds
> > +
> > +Make sure we use the w64 version of the configs, since that's what
> > +meta-mingw is using.
> > +
> > +Signed-off-by: Samuli Piippo 
> > +---
> > + libstdc++-v3/configure.host | 12 ++--
> > + 1 file changed, 2 insertions(+), 10 deletions(-)
> > +
> > +diff --git a/libstdc++-v3/configure.host b/libstdc++-v3/configure.host
> > +index 155a3cdea..adbda8784 100644
> > +--- a/libstdc++-v3/configure.host
> >  b/libstdc++-v3/configure.host
> > +@@ -277,16 +277,8 @@ case "${host_os}" in
> > + os_include_dir="os/hpux"
> > + ;;
> > +   mingw32*)
> > +-case "$host" in
> > +-  *-w64-*)
> > +-os_include_dir="os/mingw32-w64"
> > +-error_constants_dir="os/mingw32-w64"
> > +-;;
> > +-  *)
> > +-os_include_dir="os/mingw32"
> > +-error_constants_dir="os/mingw32"
> > +-;;
> > +-esac
> > ++os_include_dir="os/mingw32-w64"
> > ++error_constants_dir="os/mingw32-w64"
> > + OPT_LDFLAGS="${OPT_LDFLAGS} \$(lt_host_flags)"
>
> It might make things a bit more obvious and upstreamable to just change
>   *-w64-*)
> to
>   *-w64*)
> if I understand correct?

With meta-mingw, the host triplet is actually either
i686-pokysdk-mingw32 or x86_64-pokysdk-mingw32, depending on bitness
in the SDKMACHINE value.
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#137529): 
https://lists.openembedded.org/g/openembedded-core/message/137529
Mute This Topic: https://lists.openembedded.org/mt/73303435/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] gcc: make sure mingw builds use mingw32-w64 configs

2020-04-27 Thread Samuli Piippo
meta-mingw is using Mingw-w64 but the host triplet does not match
the expected pattern. config/os/mingw32-w64 enables more features
that were not working with mingw32.

Signed-off-by: Samuli Piippo 
---
 meta/recipes-devtools/gcc/gcc-9.3.inc |  1 +
 ...nfig-os-mingw32-w64-for-mingw-builds.patch | 36 +++
 2 files changed, 37 insertions(+)
 create mode 100644 
meta/recipes-devtools/gcc/gcc-9.3/0040-Always-use-config-os-mingw32-w64-for-mingw-builds.patch

diff --git a/meta/recipes-devtools/gcc/gcc-9.3.inc 
b/meta/recipes-devtools/gcc/gcc-9.3.inc
index b0411078d3..4c8275bf2a 100644
--- a/meta/recipes-devtools/gcc/gcc-9.3.inc
+++ b/meta/recipes-devtools/gcc/gcc-9.3.inc
@@ -67,6 +67,7 @@ SRC_URI = "\

file://0037-CVE-2019-14250-Check-zero-value-in-simple_object_elf.patch \

file://0038-gentypes-genmodes-Do-not-use-__LINE__-for-maintainin.patch \

file://0039-process_alt_operands-Don-t-match-user-defined-regs-o.patch \
+   file://0040-Always-use-config-os-mingw32-w64-for-mingw-builds.patch 
\
 "
 S = "${TMPDIR}/work-shared/gcc-${PV}-${PR}/gcc-${PV}"
 SRC_URI[sha256sum] = 
"71e197867611f6054aa1119b13a0c0abac12834765fe2d81f35ac57f84f742d1"
diff --git 
a/meta/recipes-devtools/gcc/gcc-9.3/0040-Always-use-config-os-mingw32-w64-for-mingw-builds.patch
 
b/meta/recipes-devtools/gcc/gcc-9.3/0040-Always-use-config-os-mingw32-w64-for-mingw-builds.patch
new file mode 100644
index 00..5a167f252c
--- /dev/null
+++ 
b/meta/recipes-devtools/gcc/gcc-9.3/0040-Always-use-config-os-mingw32-w64-for-mingw-builds.patch
@@ -0,0 +1,36 @@
+From 052f7e5db0d793155015530b639c4e42d37eb7eb Mon Sep 17 00:00:00 2001
+From: Samuli Piippo 
+Date: Fri, 23 Aug 2019 10:10:43 +0300
+Subject: [PATCH] Always use config/os/mingw32-w64 for mingw builds
+
+Make sure we use the w64 version of the configs, since that's what
+meta-mingw is using.
+
+Signed-off-by: Samuli Piippo 
+---
+ libstdc++-v3/configure.host | 12 ++--
+ 1 file changed, 2 insertions(+), 10 deletions(-)
+
+diff --git a/libstdc++-v3/configure.host b/libstdc++-v3/configure.host
+index 155a3cdea..adbda8784 100644
+--- a/libstdc++-v3/configure.host
 b/libstdc++-v3/configure.host
+@@ -277,16 +277,8 @@ case "${host_os}" in
+ os_include_dir="os/hpux"
+ ;;
+   mingw32*)
+-case "$host" in
+-  *-w64-*)
+-os_include_dir="os/mingw32-w64"
+-error_constants_dir="os/mingw32-w64"
+-;;
+-  *)
+-os_include_dir="os/mingw32"
+-error_constants_dir="os/mingw32"
+-;;
+-esac
++os_include_dir="os/mingw32-w64"
++error_constants_dir="os/mingw32-w64"
+ OPT_LDFLAGS="${OPT_LDFLAGS} \$(lt_host_flags)"
+ ;;
+   netbsd*)
-- 
2.17.1

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

View/Reply Online (#137522): 
https://lists.openembedded.org/g/openembedded-core/message/137522
Mute This Topic: https://lists.openembedded.org/mt/73303435/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] linux-firmware: update packaging for brcm files

2019-11-06 Thread Samuli Piippo
Add new package for bcm4366c and include available NVRAM config files
into the corrent bcm* packages.

Signed-off-by: Samuli Piippo 
---
 .../linux-firmware/linux-firmware_20190815.bb  | 18 +++---
 1 file changed, 11 insertions(+), 7 deletions(-)

diff --git a/meta/recipes-kernel/linux-firmware/linux-firmware_20190815.bb 
b/meta/recipes-kernel/linux-firmware/linux-firmware_20190815.bb
index 518638806f..d83000b64f 100644
--- a/meta/recipes-kernel/linux-firmware/linux-firmware_20190815.bb
+++ b/meta/recipes-kernel/linux-firmware/linux-firmware_20190815.bb
@@ -273,6 +273,7 @@ PACKAGES =+ "${PN}-ralink-license ${PN}-ralink \
  ${PN}-bcm4358 \
  ${PN}-bcm43602 \
  ${PN}-bcm4366b \
+ ${PN}-bcm4366c \
  ${PN}-bcm4371 \
  ${PN}-bcm4373 \
  ${PN}-bcm43xx \
@@ -571,7 +572,7 @@ FILES_${PN}-bcm43xx-hdr = 
"${nonarch_base_libdir}/firmware/brcm/bcm43xx_hdr-0.fw
 FILES_${PN}-bcm4329-fullmac = 
"${nonarch_base_libdir}/firmware/brcm/bcm4329-fullmac-4.bin"
 FILES_${PN}-bcm43236b = 
"${nonarch_base_libdir}/firmware/brcm/brcmfmac43236b.bin"
 FILES_${PN}-bcm4329 = 
"${nonarch_base_libdir}/firmware/brcm/brcmfmac4329-sdio.bin"
-FILES_${PN}-bcm4330 = 
"${nonarch_base_libdir}/firmware/brcm/brcmfmac4330-sdio.bin"
+FILES_${PN}-bcm4330 = 
"${nonarch_base_libdir}/firmware/brcm/brcmfmac4330-sdio.*"
 FILES_${PN}-bcm4334 = 
"${nonarch_base_libdir}/firmware/brcm/brcmfmac4334-sdio.bin"
 FILES_${PN}-bcm4335 = 
"${nonarch_base_libdir}/firmware/brcm/brcmfmac4335-sdio.bin"
 FILES_${PN}-bcm4339 = 
"${nonarch_base_libdir}/firmware/brcm/brcmfmac4339-sdio.bin"
@@ -582,8 +583,8 @@ FILES_${PN}-bcm43242a = 
"${nonarch_base_libdir}/firmware/brcm/brcmfmac43242a.bin
 FILES_${PN}-bcm43143 = "${nonarch_base_libdir}/firmware/brcm/brcmfmac43143.bin 
\
   ${nonarch_base_libdir}/firmware/brcm/brcmfmac43143-sdio.bin \
 "
-FILES_${PN}-bcm43430a0 = 
"${nonarch_base_libdir}/firmware/brcm/brcmfmac43430a0-sdio.bin"
-FILES_${PN}-bcm43455 = 
"${nonarch_base_libdir}/firmware/brcm/brcmfmac43455-sdio.bin"
+FILES_${PN}-bcm43430a0 = 
"${nonarch_base_libdir}/firmware/brcm/brcmfmac43430a0-sdio.*"
+FILES_${PN}-bcm43455 = 
"${nonarch_base_libdir}/firmware/brcm/brcmfmac43455-sdio.*"
 FILES_${PN}-bcm4350c2 = 
"${nonarch_base_libdir}/firmware/brcm/brcmfmac4350c2-pcie.bin"
 FILES_${PN}-bcm4350 = 
"${nonarch_base_libdir}/firmware/brcm/brcmfmac4350-pcie.bin"
 FILES_${PN}-bcm4356 = 
"${nonarch_base_libdir}/firmware/brcm/brcmfmac4356-sdio.bin"
@@ -594,6 +595,7 @@ FILES_${PN}-bcm43602 = 
"${nonarch_base_libdir}/firmware/brcm/brcmfmac43602-pcie.
   ${nonarch_base_libdir}/firmware/brcm/brcmfmac43602-pcie.ap.bin \
 "
 FILES_${PN}-bcm4366b = 
"${nonarch_base_libdir}/firmware/brcm/brcmfmac4366b-pcie.bin"
+FILES_${PN}-bcm4366c = 
"${nonarch_base_libdir}/firmware/brcm/brcmfmac4366c-pcie.bin"
 FILES_${PN}-bcm4371 = 
"${nonarch_base_libdir}/firmware/brcm/brcmfmac4371-pcie.bin"
 
 # for i in `grep brcm WHENCE  | grep ^File | sed 's/File: brcm.//g'`; do 
pkg=`echo $i | sed 's/-[sp40].*//g; s/\.bin//g; s/brcmfmac/bcm/g; 
s/_hdr/-hdr/g; s/BCM/bcm-0bb4-0306/g'`; echo -e "LICENSE_\${PN}-$pkg = 
\"Firmware-broadcom_bcm43xx\"\nRDEPENDS_\${PN}-$pkg += 
\"\${PN}-broadcom-license\""; done
@@ -647,6 +649,8 @@ LICENSE_${PN}-bcm43602 = "Firmware-broadcom_bcm43xx"
 RDEPENDS_${PN}-bcm43602 += "${PN}-broadcom-license"
 LICENSE_${PN}-bcm4366b = "Firmware-broadcom_bcm43xx"
 RDEPENDS_${PN}-bcm4366b += "${PN}-broadcom-license"
+LICENSE_${PN}-bcm4366c = "Firmware-broadcom_bcm43xx"
+RDEPENDS_${PN}-bcm4366c += "${PN}-broadcom-license"
 LICENSE_${PN}-bcm4371 = "Firmware-broadcom_bcm43xx"
 RDEPENDS_${PN}-bcm4371 += "${PN}-broadcom-license"
 
@@ -656,11 +660,11 @@ LICENSE_${PN}-cypress-license = "Firmware-cypress"
 FILES_${PN}-cypress-license = "${nonarch_base_libdir}/firmware/LICENCE.cypress"
 
 FILES_${PN}-bcm-0bb4-0306 = 
"${nonarch_base_libdir}/firmware/brcm/BCM-0bb4-0306.hcd"
-FILES_${PN}-bcm43340 = 
"${nonarch_base_libdir}/firmware/brcm/brcmfmac43340-sdio.bin"
-FILES_${PN}-bcm43362 = 
"${nonarch_base_libdir}/firmware/brcm/brcmfmac43362-sdio.bin"
-FILES_${PN}-bcm43430 = 
"${nonarch_base_libdir}/firmware/brcm/brcmfmac43430-sdio.bin"
+FILES_${PN}-bcm43340 = 
"${nonarch_base_libdir}/firmware/brcm/brcmfmac43340-sdio.*"
+FILES_${PN}-bcm43362 = 
"${nonarch_base_libdir}/firmware/brcm/brcmfmac43362-sdio.*"
+FILES_${PN}-bcm43430 = 
"${nonarch_base_libdir}/firmware/brcm/brcmfmac43430-sdio.*"
 FILES_${PN}-bcm4354 = 
"${nonarch_base_libdir}/firmware/brcm/brcmfmac4354-sdio.bin"
-FILES_${PN}-bcm4356-pcie = 
"$

[OE-core] [PATCH v2] Revert "gcc-cross-canadian: Add symlink to real-ld alongside other symlinks"

2019-01-31 Thread Samuli Piippo
This reverts commit cdd86896c8d29135f937968e9aa07f919cf543d3.

real-ld is always used if that is found, which means you cannot
switch between bfd and gold linkers using -fuse-ld=gold.

Signed-off-by: Samuli Piippo 
---
 meta/recipes-devtools/gcc/gcc-cross-canadian.inc | 2 --
 1 file changed, 2 deletions(-)

diff --git a/meta/recipes-devtools/gcc/gcc-cross-canadian.inc 
b/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
index e7c08d3a61..ececec4965 100644
--- a/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
+++ b/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
@@ -137,8 +137,6 @@ do_install () {
 
ln -sf ${BINRELPATH}/${TARGET_PREFIX}$t$suffix $dest$t$suffix
done
-   t=real-ld
-   ln -sf ${BINRELPATH}/${TARGET_PREFIX}ld$suffix $dest$t$suffix
 
# libquadmath headers need to  be available in the gcc libexec dir
install -d ${D}${libdir}/gcc/${TARGET_SYS}/${BINV}/include/
-- 
2.17.1

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


Re: [OE-core] [PATCH] gcc-cross-canadian: Do not create symlink to real-ld

2019-01-30 Thread Samuli Piippo
Sent v2 as a revert patch.

Simple compilation test with mips64 toolchain works fine, as does arm64/armv7.
Richard, would you happen to remember more details where this failed
in the first place?

-samuli

On Tue, 29 Jan 2019 at 20:20, Khem Raj  wrote:
>
> On Mon, Jan 28, 2019 at 11:19 PM Samuli Piippo  wrote:
> >
> > real-ld is always used if that is found, which means you cannot
> > switch between bfd and gold linkers using -fuse-ld=gold.
> >
> > Signed-off-by: Samuli Piippo 
> > ---
> >  meta/recipes-devtools/gcc/gcc-cross-canadian.inc | 2 --
> >  1 file changed, 2 deletions(-)
> >
> > diff --git a/meta/recipes-devtools/gcc/gcc-cross-canadian.inc 
> > b/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
> > index e7c08d3a61..ececec4965 100644
> > --- a/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
> > +++ b/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
> > @@ -137,8 +137,6 @@ do_install () {
> >
> > ln -sf ${BINRELPATH}/${TARGET_PREFIX}$t$suffix 
> > $dest$t$suffix
> > done
> > -   t=real-ld
> > -   ln -sf ${BINRELPATH}/${TARGET_PREFIX}ld$suffix $dest$t$suffix
> >
>
> this patch is essentially, reverting
> https://git.openembedded.org/openembedded-core/commit/?id=cdd86896c8d29135f937968e9aa07f919cf543d3
>
> so lets do a revert instead to keep the context clear
>
> secondly, please test with the mips/mips64 as those were primary drivers for
> the patch in first place.  Richard might add more context if he remembers.
>
>
> > # libquadmath headers need to  be available in the gcc libexec dir
> > install -d ${D}${libdir}/gcc/${TARGET_SYS}/${BINV}/include/
> > --
> > 2.17.1
> >
> > --
> > ___
> > Openembedded-core mailing list
> > 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
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH] gcc-cross-canadian: Do not create symlink to real-ld

2019-01-28 Thread Samuli Piippo
real-ld is always used if that is found, which means you cannot
switch between bfd and gold linkers using -fuse-ld=gold.

Signed-off-by: Samuli Piippo 
---
 meta/recipes-devtools/gcc/gcc-cross-canadian.inc | 2 --
 1 file changed, 2 deletions(-)

diff --git a/meta/recipes-devtools/gcc/gcc-cross-canadian.inc 
b/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
index e7c08d3a61..ececec4965 100644
--- a/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
+++ b/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
@@ -137,8 +137,6 @@ do_install () {
 
ln -sf ${BINRELPATH}/${TARGET_PREFIX}$t$suffix $dest$t$suffix
done
-   t=real-ld
-   ln -sf ${BINRELPATH}/${TARGET_PREFIX}ld$suffix $dest$t$suffix
 
# libquadmath headers need to  be available in the gcc libexec dir
install -d ${D}${libdir}/gcc/${TARGET_SYS}/${BINV}/include/
-- 
2.17.1

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


[OE-core] [PATCH v2] sdl2: fix sdl2.pc include paths

2019-01-17 Thread Samuli Piippo
SDL2 dumps all cflags to the sdl2.pc, which might then contain paths
to the recipe sysroot include dirs.

Signed-off-by: Samuli Piippo 
---
 meta/recipes-graphics/libsdl2/libsdl2_2.0.9.bb | 4 
 1 file changed, 4 insertions(+)

diff --git a/meta/recipes-graphics/libsdl2/libsdl2_2.0.9.bb 
b/meta/recipes-graphics/libsdl2/libsdl2_2.0.9.bb
index a5ec77728a..8941f05958 100644
--- a/meta/recipes-graphics/libsdl2/libsdl2_2.0.9.bb
+++ b/meta/recipes-graphics/libsdl2/libsdl2_2.0.9.bb
@@ -63,6 +63,10 @@ do_configure_prepend() {
 export SYSROOT=$PKG_CONFIG_SYSROOT_DIR
 }
 
+do_install_append() {
+sed -i -e "s|${STAGING_INCDIR}|$\{includedir}|g" 
${D}${libdir}/pkgconfig/sdl2.pc
+}
+
 FILES_${PN}-dev += "${libdir}/cmake"
 
 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] sdl2: fix sdl2.pc include paths

2019-01-12 Thread Samuli Piippo
SDL2 dumps all cflags to the sdl2.pc, which might then contain paths
to the recipe sysroot include dirs.

Signed-off-by: Samuli Piippo 
---
 meta/recipes-graphics/libsdl2/libsdl2_2.0.8.bb | 4 
 1 file changed, 4 insertions(+)

diff --git a/meta/recipes-graphics/libsdl2/libsdl2_2.0.8.bb 
b/meta/recipes-graphics/libsdl2/libsdl2_2.0.8.bb
index 812a9abf3c..a9c5d8a443 100644
--- a/meta/recipes-graphics/libsdl2/libsdl2_2.0.8.bb
+++ b/meta/recipes-graphics/libsdl2/libsdl2_2.0.8.bb
@@ -64,6 +64,10 @@ do_configure_prepend() {
 export SYSROOT=$PKG_CONFIG_SYSROOT_DIR
 }
 
+do_install_append() {
+sed -i -e "s|${STAGING_INCDIR}|$\{includedir}|g" 
${D}${libdir}/pkgconfig/sdl2.pc
+}
+
 FILES_${PN}-dev += "${libdir}/cmake"
 
 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] glib: remove unnecessary dependency to DISTRO_FEATURES

2018-06-27 Thread Samuli Piippo
Since DISTRO_FEATURES was expanded in the comments, it created
task dependency to the full content of DISTRO_FEATURES, instead
of just the x11 used below. This prevented reuse of sstate-cache
when unrelated feature flags were changed.

Signed-off-by: Samuli Piippo 
---
 meta/recipes-core/glib-2.0/glib.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-core/glib-2.0/glib.inc 
b/meta/recipes-core/glib-2.0/glib.inc
index 31b74362e3..d6d91335a6 100644
--- a/meta/recipes-core/glib-2.0/glib.inc
+++ b/meta/recipes-core/glib-2.0/glib.inc
@@ -117,7 +117,7 @@ do_install_append () {
 
 do_install_append_class-target () {
# Tests are only installed on targets, not native builds.  Separating 
this out
-   # keeps glib-2.0-native from depending on ${DISTRO_FEATURES}
+   # keeps glib-2.0-native from depending on DISTRO_FEATURES
if [ -f ${D}${datadir}/installed-tests/glib/gdbus-serialization.test ]; 
then
if ${@bb.utils.contains("DISTRO_FEATURES", "x11", "false", 
"true", d)}; then
rm 
${D}${datadir}/installed-tests/glib/gdbus-serialization.test
-- 
2.17.1

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


[OE-core] [PATCH] binutils: apply mingw fix only for binutils-cross-canadian

2017-09-15 Thread Samuli Piippo
Whenever SDKMACHINE is set to mingw32, sdkmingw32 override is defined
everywhere. This meant that value of LDGOLD was different also for
binutils and binutils-cross depending whether SDKMACHINE was set or not.

Signed-off-by: Samuli Piippo <samuli.pii...@qt.io>
---
 meta/recipes-devtools/binutils/binutils-cross-canadian.inc | 2 ++
 meta/recipes-devtools/binutils/binutils.inc| 1 -
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-devtools/binutils/binutils-cross-canadian.inc 
b/meta/recipes-devtools/binutils/binutils-cross-canadian.inc
index b3d591e..e98f31f 100644
--- a/meta/recipes-devtools/binutils/binutils-cross-canadian.inc
+++ b/meta/recipes-devtools/binutils/binutils-cross-canadian.inc
@@ -13,6 +13,8 @@ EXTRA_OECONF += 
"--with-sysroot=${SDKPATH}/sysroots/${TUNE_PKGARCH}${TARGET_VEND
 # e.g. we switch between different machines with different tunes.
 EXTRA_OECONF[vardepsexclude] = "TUNE_PKGARCH"
 
+LDGOLD_sdkmingw32 = ""
+
 do_install () {
autotools_do_install
 
diff --git a/meta/recipes-devtools/binutils/binutils.inc 
b/meta/recipes-devtools/binutils/binutils.inc
index 7efe13f..37813dd 100644
--- a/meta/recipes-devtools/binutils/binutils.inc
+++ b/meta/recipes-devtools/binutils/binutils.inc
@@ -78,7 +78,6 @@ EXTRA_OECONF = "--program-prefix=${TARGET_PREFIX} \
 
 LDGOLD_class-native = ""
 LDGOLD_class-crosssdk = ""
-LDGOLD_sdkmingw32 = ""
 LDGOLD ?= "${@bb.utils.contains('DISTRO_FEATURES', 'ld-is-gold', 
'--enable-gold=default --enable-threads', '--enable-gold --enable-ld=default 
--enable-threads', d)}"
 
 # This is necessary due to a bug in the binutils Makefiles
-- 
2.7.4

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


[OE-core] [PATCH] mesa: make sure GLES3 headers are installed

2016-11-23 Thread Samuli Piippo
Since there is no libgles3-mesa package that would pull in the headers,
add dependency to libgles2-mesa-dev. Now there no need to manually add
GLES3 headers to image or toolchain.

Signed-off-by: Samuli Piippo <samuli.pii...@qt.io>
---
 meta/recipes-graphics/mesa/mesa.inc | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/meta/recipes-graphics/mesa/mesa.inc 
b/meta/recipes-graphics/mesa/mesa.inc
index e4880ff..525a2d5 100644
--- a/meta/recipes-graphics/mesa/mesa.inc
+++ b/meta/recipes-graphics/mesa/mesa.inc
@@ -79,6 +79,9 @@ EXCLUDE_FROM_WORLD = "1"
 # Remove the mesa dependency on mesa-dev, as mesa is empty
 RDEPENDS_${PN}-dev = ""
 
+# Add dependency so that GLES3 header don't need to be added manually
+RDEPENDS_libgles2-mesa-dev += "libgles3-mesa-dev"
+
 PACKAGES =+ "libegl-mesa libegl-mesa-dev \
  libosmesa libosmesa-dev \
  libgl-mesa libgl-mesa-dev \
-- 
1.9.1

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


[OE-core] [PATCH] package_manager.py: correctly remove all dependent packages

2016-11-01 Thread Samuli Piippo
Do not use --force-depends when trying to remove all dependent packages,
as it removes only the selected package and not the dependent packages.

Signed-off-by: Samuli Piippo <samuli.pii...@qt.io>
---
 meta/lib/oe/package_manager.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/lib/oe/package_manager.py b/meta/lib/oe/package_manager.py
index 3cee973..a9d216a 100644
--- a/meta/lib/oe/package_manager.py
+++ b/meta/lib/oe/package_manager.py
@@ -1776,7 +1776,7 @@ class OpkgPM(OpkgDpkgPM):
 
 def remove(self, pkgs, with_dependencies=True):
 if with_dependencies:
-cmd = "%s %s --force-depends --force-remove 
--force-removal-of-dependent-packages remove %s" % \
+cmd = "%s %s --force-remove --force-removal-of-dependent-packages 
remove %s" % \
 (self.opkg_cmd, self.opkg_args, ' '.join(pkgs))
 else:
 cmd = "%s %s --force-depends remove %s" % \
-- 
1.9.1

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


Re: [OE-core] [meta-oe][PATCH] systemd-serialgetty: add dependency to agetty

2016-10-18 Thread Samuli Piippo

On 18.10.2016 02:36, Khem Raj wrote:



On Oct 11, 2016, at 8:14 AM, Samuli Piippo <samuli.pii...@qt.io> wrote:

Signed-off-by: Samuli Piippo <samuli.pii...@qt.io>
---
meta/recipes-core/systemd/systemd-serialgetty.bb | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-core/systemd/systemd-serialgetty.bb 
b/meta/recipes-core/systemd/systemd-serialgetty.bb
index 768b130..c4e5310 100644
--- a/meta/recipes-core/systemd/systemd-serialgetty.bb
+++ b/meta/recipes-core/systemd/systemd-serialgetty.bb
@@ -38,7 +38,7 @@ do_install() {
fi
}

-RDEPENDS_${PN} = "systemd"
+RDEPENDS_${PN} = "systemd util-linux-agetty”


Does it need agetty specifically ? since there are other providers of getty as 
well.
I wonder if its possible to use those too


serial-getty@.service is using /sbin/agetty, so currently it's the one.
systemd also has util-linux-agetty in RECOMMENDS, but that could perhaps 
be removed.






# This is a machine specific file
FILES_${PN} = "${systemd_unitdir}/system/*.service ${sysconfdir}"
--
1.9.1

--
___
Openembedded-core mailing list
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] [meta-oe][PATCH] systemd-serialgetty: add dependency to agetty

2016-10-11 Thread Samuli Piippo
Signed-off-by: Samuli Piippo <samuli.pii...@qt.io>
---
 meta/recipes-core/systemd/systemd-serialgetty.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-core/systemd/systemd-serialgetty.bb 
b/meta/recipes-core/systemd/systemd-serialgetty.bb
index 768b130..c4e5310 100644
--- a/meta/recipes-core/systemd/systemd-serialgetty.bb
+++ b/meta/recipes-core/systemd/systemd-serialgetty.bb
@@ -38,7 +38,7 @@ do_install() {
fi
 }
 
-RDEPENDS_${PN} = "systemd"
+RDEPENDS_${PN} = "systemd util-linux-agetty"
 
 # This is a machine specific file
 FILES_${PN} = "${systemd_unitdir}/system/*.service ${sysconfdir}"
-- 
1.9.1

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


[OE-core] [PATCH] wayland: add nativesdk support

2015-02-01 Thread Samuli Piippo
Allow building wayland-scanner for nativesdk. Extra patch required
to use the scanner from wayland-native during the build.

Signed-off-by: Samuli Piippo samuli.pii...@theqtcompany.com
---
 .../always-use-wayland-scanner-from-system.patch   | 25 ++
 meta/recipes-graphics/wayland/wayland_1.6.0.bb |  8 ++-
 2 files changed, 32 insertions(+), 1 deletion(-)
 create mode 100644 
meta/recipes-graphics/wayland/wayland/always-use-wayland-scanner-from-system.patch

diff --git 
a/meta/recipes-graphics/wayland/wayland/always-use-wayland-scanner-from-system.patch
 
b/meta/recipes-graphics/wayland/wayland/always-use-wayland-scanner-from-system.patch
new file mode 100644
index 000..9e5fe4d
--- /dev/null
+++ 
b/meta/recipes-graphics/wayland/wayland/always-use-wayland-scanner-from-system.patch
@@ -0,0 +1,25 @@
+The wayland-scanner built for the nativesdk cannot be run during the build,
+so instead use the wayland-scanner from native build.
+
+Upstream-Status: Pending
+
+Signed-off-by: Samuli Piippo samuli.pii...@theqtcompany.com
+---
+ Makefile.am | 4 +---
+ 1 file changed, 1 insertion(+), 3 deletions(-)
+diff --git a/Makefile.am b/Makefile.am
+index c15d8b8..45f7133 100644
+--- a/Makefile.am
 b/Makefile.am
+@@ -62,7 +62,7 @@ nodist_libwayland_client_la_SOURCES = \
+ pkgconfig_DATA += src/wayland-client.pc src/wayland-server.pc
+ 
+ if ENABLE_SCANNER
+-wayland_scanner = $(top_builddir)/wayland-scanner
++wayland_scanner = wayland-scanner
+ bin_PROGRAMS = wayland-scanner
+ wayland_scanner_SOURCES = src/scanner.c
+ wayland_scanner_LDADD = $(EXPAT_LIBS) libwayland-util.la
+-- 
+1.9.1
+
diff --git a/meta/recipes-graphics/wayland/wayland_1.6.0.bb 
b/meta/recipes-graphics/wayland/wayland_1.6.0.bb
index cf3c094..5c9832e 100644
--- a/meta/recipes-graphics/wayland/wayland_1.6.0.bb
+++ b/meta/recipes-graphics/wayland/wayland_1.6.0.bb
@@ -16,14 +16,20 @@ SRC_URI[sha256sum] = 
a7d5102dcf53d08c059d24bc62de491d7cd482070abeb6737a20d0d86b
 SRC_URI_append_class-native =  \
 file://disable-macro-checks-not-used-for-scanner.patch \
 
+SRC_URI_append_class-nativesdk =  \
+file://disable-macro-checks-not-used-for-scanner.patch \
+file://always-use-wayland-scanner-from-system.patch \
+
 EXTRA_OECONF_class-native = --disable-documentation --enable-scanner
+EXTRA_OECONF_class-nativesdk = --disable-documentation --enable-scanner
 
 inherit autotools pkgconfig
 
 # We need wayland-native for the wayland-scanner utility
-BBCLASSEXTEND = native
+BBCLASSEXTEND = native nativesdk
 
 DEPENDS_class-native = expat-native libffi-native
+DEPENDS_class-nativesdk = nativesdk-expat nativesdk-libffi wayland-native
 DEPENDS = expat libffi wayland-native
 
 EXTRA_OECONF = --disable-documentation --disable-scanner
-- 
1.9.1

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


[OE-core] [PATCH] gdb-cross-canadian: use correct exec-prefix path for python

2013-05-27 Thread Samuli Piippo
Incorrect exec-prefix path was given to gdb which leads to gdb
startup failure when SDK is not installed to its original destination.
Gdb relocates the exec-prefix path, so it will work for SDKs that
are installed to different location. PYTHONHOME env in no longer
neeeded for gdb.

[YOCTO #3839]
---
 meta/recipes-devtools/gdb/gdb-cross-canadian.inc |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-devtools/gdb/gdb-cross-canadian.inc 
b/meta/recipes-devtools/gdb/gdb-cross-canadian.inc
index f6f515c..3cb347b 100644
--- a/meta/recipes-devtools/gdb/gdb-cross-canadian.inc
+++ b/meta/recipes-devtools/gdb/gdb-cross-canadian.inc
@@ -19,7 +19,7 @@ cat  ${WORKDIR}/python  EOF
 case \$2 in
 --includes) echo -I${STAGING_INCDIR}/${PYTHON_DIR}/ ;;
 --ldflags) echo -Wl,-rpath-link,${STAGING_LIBDIR}/.. 
-Wl,-rpath,${libdir}/.. -lpthread -ldl -lutil -lm 
-lpython${PYTHON_BASEVERSION} ;;
---exec-prefix) echo ${exec_prefix}/bin ;;
+--exec-prefix) echo ${exec_prefix} ;;
 *) exit 1 ;;
 esac
 exit 0
-- 
1.7.10.4

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