from:"Vyacheslav Yurkov"

You sure? Because I remember the test in grpc to check for 
CROSS_COMPILING in cmake to avoid that.


Slava

On 10.11.2023 08:12, Samuli Piippo wrote:

This is not strictly a nativesdk issue.
If you enable PACKAGECONFIG = "compiler" for protobuf, it will then 
build protoc for the target.
This will then cause grpc target build to fail in the same way as 
nativesdk build.


On Fri, 10 Nov 2023 at 08:40, Vyacheslav Yurkov  
wrote:


FTR, I suggested oe-core solution in the corresponding mailing list.

Slava

On 10.11.2023 07:19, Vyacheslav Yurkov via lists.openembedded.org
<http://lists.openembedded.org> wrote:

On 10.11.2023 02:23, Clément Péron wrote:

Hi Samuli,

Le ven. 10 nov. 2023, 00:05, Vyacheslav Yurkov
 a écrit :

Thanks for the explanation, that was the missing piece for
me to get an understanding of the problem :)

I think it's not only me, but also others, were confused by
the fact that we are talking about native(sdk), but try to
fix something with a non-native variable.

I've just gave it a try and have two suggestions how this
can be addressed.

1. Recipe based solution:
SYSROOT_DIRS:append:class-nativesdk = " $bindir}"


Is it not just missing something like

DEPENDS:append:class-nativesdk= " protobuf-native "
Regards
Clement


This is already implicit by DEPENDS += "protobuf". Take a look at
my second suggestion. You can see that ${bindir} is staged for
native recipes, but not for nativesdk.



2. oe-core based solution:
--- a/meta/classes-global/staging.bbclass
+++ b/meta/classes-global/staging.bbclass
@@ -26,6 +26,7 @@ SYSROOT_DIRS_NATIVE = " \
 ${localstatedir} \
 "
 SYSROOT_DIRS:append:class-native = " ${SYSROOT_DIRS_NATIVE}"
+SYSROOT_DIRS:append:class-nativesdk = " ${SYSROOT_DIRS_NATIVE}"
 SYSROOT_DIRS:append:class-cross = " ${SYSROOT_DIRS_NATIVE}"
 SYSROOT_DIRS:append:class-crosssdk = " ${SYSROOT_DIRS_NATIVE}"

The second one will invalidate nativesdk sstate, but might
cover not only grpc issue (I would assume gdbus-codegen or
any other generator has the same issue when used for SDK build).

What are your thoughs?

Thanks,
Slava

On 09.11.2023 14:35, Samuli Piippo wrote:

DEPENDS += "protobuf" is enough to have dependencies resolved.

Problem is that bitbake's do_populate_sysroot does not take
anything from bindir.
This leads to nativesdk-(qt)grpc's
do_prepare_recipe_sysroot to not get the protoc binary into
the recipe_sysroot as expected by CMake.

This can be fixed with SYSROOT_DIRS.

On Thu, 9 Nov 2023 at 12:19, Vyacheslav Yurkov
 wrote:

Hi Clément,
No, the issue is when you just build nativesdk recipe
(we are not at the stage to use the SDK yet).

And that was exactly my question how to propagate the
dependency correctly for nativesdk build.

Regards,
Slava


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

Re: [oe] [PATCH] Revert "protobuf: stage protoc binary to sysroot"


FTR, I suggested oe-core solution in the corresponding mailing list.

Slava

On 10.11.2023 07:19, Vyacheslav Yurkov via lists.openembedded.org wrote:

On 10.11.2023 02:23, Clément Péron wrote:

Hi Samuli,

Le ven. 10 nov. 2023, 00:05, Vyacheslav Yurkov  a 
écrit :


Thanks for the explanation, that was the missing piece for me to
get an understanding of the problem :)

I think it's not only me, but also others, were confused by the
fact that we are talking about native(sdk), but try to fix
something with a non-native variable.

I've just gave it a try and have two suggestions how this can be
addressed.

1. Recipe based solution:
SYSROOT_DIRS:append:class-nativesdk = " $bindir}"


Is it not just missing something like

DEPENDS:append:class-nativesdk= " protobuf-native "
Regards
Clement


This is already implicit by DEPENDS += "protobuf". Take a look at my 
second suggestion. You can see that ${bindir} is staged for native 
recipes, but not for nativesdk.




2. oe-core based solution:
--- a/meta/classes-global/staging.bbclass
+++ b/meta/classes-global/staging.bbclass
@@ -26,6 +26,7 @@ SYSROOT_DIRS_NATIVE = " \
 ${localstatedir} \
 "
 SYSROOT_DIRS:append:class-native = " ${SYSROOT_DIRS_NATIVE}"
+SYSROOT_DIRS:append:class-nativesdk = " ${SYSROOT_DIRS_NATIVE}"
 SYSROOT_DIRS:append:class-cross = " ${SYSROOT_DIRS_NATIVE}"
 SYSROOT_DIRS:append:class-crosssdk = " ${SYSROOT_DIRS_NATIVE}"

The second one will invalidate nativesdk sstate, but might cover
not only grpc issue (I would assume gdbus-codegen or any other
generator has the same issue when used for SDK build).

What are your thoughs?

Thanks,
Slava

On 09.11.2023 14:35, Samuli Piippo wrote:

DEPENDS += "protobuf" is enough to have dependencies resolved.

Problem is that bitbake's do_populate_sysroot does not take
anything from bindir.
This leads to nativesdk-(qt)grpc's do_prepare_recipe_sysroot to
not get the protoc binary into the recipe_sysroot as expected by
CMake.

    This can be fixed with SYSROOT_DIRS.

On Thu, 9 Nov 2023 at 12:19, Vyacheslav Yurkov
 wrote:

Hi Clément,
No, the issue is when you just build nativesdk recipe (we
are not at the stage to use the SDK yet).

And that was exactly my question how to propagate the
dependency correctly for nativesdk build.

Regards,
Slava




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

Re: [oe] [PATCH] Revert "protobuf: stage protoc binary to sysroot"


On 10.11.2023 02:23, Clément Péron wrote:

Hi Samuli,

Le ven. 10 nov. 2023, 00:05, Vyacheslav Yurkov  a 
écrit :


Thanks for the explanation, that was the missing piece for me to
get an understanding of the problem :)

I think it's not only me, but also others, were confused by the
fact that we are talking about native(sdk), but try to fix
something with a non-native variable.

I've just gave it a try and have two suggestions how this can be
addressed.

1. Recipe based solution:
SYSROOT_DIRS:append:class-nativesdk = " $bindir}"


Is it not just missing something like

DEPENDS:append:class-nativesdk= " protobuf-native "
Regards
Clement


This is already implicit by DEPENDS += "protobuf". Take a look at my 
second suggestion. You can see that ${bindir} is staged for native 
recipes, but not for nativesdk.




2. oe-core based solution:
--- a/meta/classes-global/staging.bbclass
+++ b/meta/classes-global/staging.bbclass
@@ -26,6 +26,7 @@ SYSROOT_DIRS_NATIVE = " \
 ${localstatedir} \
 "
 SYSROOT_DIRS:append:class-native = " ${SYSROOT_DIRS_NATIVE}"
+SYSROOT_DIRS:append:class-nativesdk = " ${SYSROOT_DIRS_NATIVE}"
 SYSROOT_DIRS:append:class-cross = " ${SYSROOT_DIRS_NATIVE}"
 SYSROOT_DIRS:append:class-crosssdk = " ${SYSROOT_DIRS_NATIVE}"

The second one will invalidate nativesdk sstate, but might cover
not only grpc issue (I would assume gdbus-codegen or any other
generator has the same issue when used for SDK build).

What are your thoughs?

Thanks,
Slava

On 09.11.2023 14:35, Samuli Piippo wrote:

DEPENDS += "protobuf" is enough to have dependencies resolved.

Problem is that bitbake's do_populate_sysroot does not take
anything from bindir.
This leads to nativesdk-(qt)grpc's do_prepare_recipe_sysroot to
not get the protoc binary into the recipe_sysroot as expected by
CMake.

    This can be fixed with SYSROOT_DIRS.

On Thu, 9 Nov 2023 at 12:19, Vyacheslav Yurkov
 wrote:

Hi Clément,
No, the issue is when you just build nativesdk recipe (we are
not at the stage to use the SDK yet).

And that was exactly my question how to propagate the
dependency correctly for nativesdk build.

Regards,
Slava






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

Re: [oe] [PATCH] Revert "protobuf: stage protoc binary to sysroot"

Thanks for the explanation, that was the missing piece for me to get an 
understanding of the problem :)


I think it's not only me, but also others, were confused by the fact 
that we are talking about native(sdk), but try to fix something with a 
non-native variable.


I've just gave it a try and have two suggestions how this can be addressed.

1. Recipe based solution:
SYSROOT_DIRS:append:class-nativesdk = " $bindir}"

2. oe-core based solution:
--- a/meta/classes-global/staging.bbclass
+++ b/meta/classes-global/staging.bbclass
@@ -26,6 +26,7 @@ SYSROOT_DIRS_NATIVE = " \
 ${localstatedir} \
 "
 SYSROOT_DIRS:append:class-native = " ${SYSROOT_DIRS_NATIVE}"
+SYSROOT_DIRS:append:class-nativesdk = " ${SYSROOT_DIRS_NATIVE}"
 SYSROOT_DIRS:append:class-cross = " ${SYSROOT_DIRS_NATIVE}"
 SYSROOT_DIRS:append:class-crosssdk = " ${SYSROOT_DIRS_NATIVE}"

The second one will invalidate nativesdk sstate, but might cover not 
only grpc issue (I would assume gdbus-codegen or any other generator has 
the same issue when used for SDK build).


What are your thoughs?

Thanks,
Slava

On 09.11.2023 14:35, Samuli Piippo wrote:

DEPENDS += "protobuf" is enough to have dependencies resolved.

Problem is that bitbake's do_populate_sysroot does not take anything 
from bindir.
This leads to nativesdk-(qt)grpc's do_prepare_recipe_sysroot to not 
get the protoc binary into the recipe_sysroot as expected by CMake.


This can be fixed with SYSROOT_DIRS.

On Thu, 9 Nov 2023 at 12:19, Vyacheslav Yurkov  wrote:

Hi Clément,
No, the issue is when you just build nativesdk recipe (we are not
at the stage to use the SDK yet).

And that was exactly my question how to propagate the dependency
correctly for nativesdk build.

Regards,
Slava


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

Re: [oe] [PATCH] Revert "protobuf: stage protoc binary to sysroot"


Hi Clément,
No, the issue is when you just build nativesdk recipe (we are not at the 
stage to use the SDK yet).


And that was exactly my question how to propagate the dependency 
correctly for nativesdk build.


Regards,
Slava

On 09.11.2023 01:13, Clément Péron wrote:

Hi Samuli,

Le mer. 8 nov. 2023, 22:30, Samuli Piippo  a 
écrit :


Hi Clément,

Were you able to reproduce the issue?


Sorry I'm on vacation without access to a laptop at the moment.

But if I understand the issue is not related to a target or native 
recipes issue, right?


It's only when building the SDK and then try to build an application 
with it right?


Maybe there is a missing dependency when building the SDK.

Regards,
Clement


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

Re: [oe] [PATCH] Revert "protobuf: stage protoc binary to sysroot"

2023-10-22 Thread Vyacheslav Yurkov

I'm a bit confused here.

I just did 'find
tmp/work/x86_64-nativesdk-pokysdk-linux/nativesdk-grpc/1.56.2/ -name
"protoc*"' and I see protoc binary only in recipe-sysroot-native. That
tells me that protoc, which is a part of nativesdk-protobuf-compiler, is
not installed to recipe-sysroot.

What am I missing here?

Slava

On 20.10.2023 09:35, Martin Jansa wrote:
do_prepare_recipe_sysroot doesn't install any packages to RSS. Only
package manager on target or in do_rootfs uses packages.

It installs the files from dependency populate_sysroot task.

On Fri, Oct 20, 2023 at 9:11 AM Samuli Piippo
wrote:

Dependency to the recipe name will always install all packages to RSS.
You can verify this by checking that the libprotoc.so from the
nativesdk-protobuf-compile package is in the sysroot.
The problem you are facing is the fact that binaries are not
installed into the sysroot (for nativesdk builds), so you are only
missing the bin/protoc-23.4.0 file.
The proposed and twice reverted fix for this is the aforementioned
SYSROOT_DIRS.

-samuli

On Fri, 20 Oct 2023 at 09:17, Vyacheslav Yurkov
wrote:

I think the problem goes down to how dependencies are
populated in the sysroot. I raised this question in oe-core
mailing list
https://lists.openembedded.org/g/openembedded-core/message/189324
, but didn't get an answer yet.

DEPENDS += "protobuf" will only install the main package of
protobuf to the sysroot, because it assumes that
nativesdk-protobuf-compiler is also part of the main package
(which is not). I guess the part of the system populating
sysroot deals with recipes only, and not with packages. I'm
still trying to understand how that should be addressed properly.
My understanding is that either all packages should be
installed to sysroot (which recipe claims it provides), or we
need a way to indicate a package level dependency (i.e.
nativesdk-protobuf-compiler in this case)

Any ideas?

Slava

On 18.10.2023 14:28, Samuli Piippo wrote:

Following simple test recipe will fail now when trying to use
Protobuf with CMake.

inherit cmake
DEPENDS += "protobuf"

do_configure:prepend() {
echo "find_package(Protobuf CONFIG)" > ${S}/CMakeLists.txt
}

BBCLASSEXTEND = "nativesdk"

---
CMake Error at

tmp/work/x86_64-nativesdk-pokysdk-linux/nativesdk-test/1.0/recipe-sysroot/usr/local/oe-sdk-hardcoded-buildpath/sysroots/x86_64-pokysdk-linux/usr/lib/cmake/protobuf/protobuf-targets.cmake:107
(message):
The imported target "protobuf::protoc" references the file

"tmp/work/x86_64-nativesdk-pokysdk-linux/nativesdk-test/1.0/recipe-sysroot/usr/local/oe-sdk-hardcoded-buildpath/sysroots/x86_64-pokysdk-linux/usr/bin/protoc-23.4.0"

but this file does not exist.
---

I don't quite see why every project/recipe should fix this
independently when the simple workaround for this
yocto-specific issue is available (namely the use
of SYSROOT_DIRS).

-samuli

ps. qtgrpc recipe and other Qt6 recipes are now visible in
the layerindex
https://layers.openembedded.org/layerindex/recipe/348210/

On Thu, 12 Oct 2023 at 15:06, Vyacheslav Yurkov
wrote:

I'd like to follow-up on this and say that I see this
issue now with nativesdk build, in particular
nativesdk-grpc recipe fails in master with the same
error. I hope that's partially related to the issue
everybody is confused about, but I'd like understand how
to properly fix it.

| The imported target "protobuf::protoc" references the
file
|
|

"/home/uvv/projects/poky/build/tmp/work/x86_64-nativesdk-pokysdk-linux/nativesdk-grpc/1.56.2/recipe-sysroot/usr/local/oe-sdk-hardcoded-buildpath/sysroots/x86_64-pokysdk-linux/usr/bin/protoc-23.4.0"
|
| but this file does not exist. Possible reasons include:
|
| * The file was deleted, renamed, or moved to another
location.

The protoc-23.4.0 file already exists in the
recipe-sysroot-native.

And we do want protoc to be present in the SDK, but
nativesdk build is a cross-compilation, because
SDKMACHINE and HOST might be different.
In this particular grpc case there's an option to say
where the actual protoc executable is (

-D_gRPC_PROTOBUF_PROTOC_EXECUTABLE=/home/uvv/projects/poky/build/tmp/work/x86_64-nativesdk-pokysdk-linux/nativesdk-grpc/1.56.2/recipe-sysroot-native/usr/

Re: [oe] [PATCH] Revert "protobuf: stage protoc binary to sysroot"

2023-10-20 Thread Vyacheslav Yurkov

I think the problem goes down to how dependencies are populated in the 
sysroot. I raised this question in oe-core mailing list 
https://lists.openembedded.org/g/openembedded-core/message/189324 , but 
didn't get an answer yet.


DEPENDS += "protobuf" will only install the main package of protobuf to 
the sysroot, because it assumes that nativesdk-protobuf-compiler is also 
part of the main package (which is not). I guess the part of the system 
populating sysroot deals with recipes only, and not with packages. I'm 
still trying to understand how that should be addressed properly.
My understanding is that either all packages should be installed to 
sysroot (which recipe claims it provides), or we need a way to indicate 
a package level dependency (i.e. nativesdk-protobuf-compiler in this case)


Any ideas?

Slava

On 18.10.2023 14:28, Samuli Piippo wrote:
Following simple test recipe will fail now when trying to use Protobuf 
with CMake.


inherit cmake
DEPENDS += "protobuf"

do_configure:prepend() {
    echo "find_package(Protobuf CONFIG)" > ${S}/CMakeLists.txt
}

BBCLASSEXTEND = "nativesdk"

---
CMake Error at 
tmp/work/x86_64-nativesdk-pokysdk-linux/nativesdk-test/1.0/recipe-sysroot/usr/local/oe-sdk-hardcoded-buildpath/sysroots/x86_64-pokysdk-linux/usr/lib/cmake/protobuf/protobuf-targets.cmake:107 
(message):

  The imported target "protobuf::protoc" references the file

 
"tmp/work/x86_64-nativesdk-pokysdk-linux/nativesdk-test/1.0/recipe-sysroot/usr/local/oe-sdk-hardcoded-buildpath/sysroots/x86_64-pokysdk-linux/usr/bin/protoc-23.4.0"

  but this file does not exist.
---

I don't quite see why every project/recipe should fix this 
independently when the simple workaround for this yocto-specific issue 
is available (namely the use of SYSROOT_DIRS).


-samuli

ps. qtgrpc recipe and other Qt6 recipes are now visible in the 
layerindex https://layers.openembedded.org/layerindex/recipe/348210/



On Thu, 12 Oct 2023 at 15:06, Vyacheslav Yurkov  
wrote:


I'd like to follow-up on this and say that I see this issue now
with nativesdk build, in particular nativesdk-grpc recipe fails in
master with the same error. I hope that's partially related to the
issue everybody is confused about, but I'd like understand how to
properly fix it.

|   The imported target "protobuf::protoc" references the file
|
|

"/home/uvv/projects/poky/build/tmp/work/x86_64-nativesdk-pokysdk-linux/nativesdk-grpc/1.56.2/recipe-sysroot/usr/local/oe-sdk-hardcoded-buildpath/sysroots/x86_64-pokysdk-linux/usr/bin/protoc-23.4.0"
|
|   but this file does not exist.  Possible reasons include:
|
|   * The file was deleted, renamed, or moved to another location.

The protoc-23.4.0 file already exists in the recipe-sysroot-native.

And we do want protoc to be present in the SDK, but nativesdk
build is a cross-compilation, because SDKMACHINE and HOST might be
different.
In this particular grpc case there's an option to say where the
actual protoc executable is (

-D_gRPC_PROTOBUF_PROTOC_EXECUTABLE=/home/uvv/projects/poky/build/tmp/work/x86_64-nativesdk-pokysdk-linux/nativesdk-grpc/1.56.2/recipe-sysroot-native/usr/bin/protoc
is passed to grpc).

So there shouldn't be any issues here, but nativesdk build still
fails.

Looking into it further I see that protoc is actually packaged by
protobuf-compiler package, but when I try to add a package
dependency to grpc recipe I get this:

nativesdk-protobuf RPROVIDES nativesdk-protobuf-compiler

Moreover, if I go into devshell and forcefully install
nativesdk-libprotobuf-compiler, then `bitbake nativesdk-grpc` is
happy and compiles!

So something wrong with the dependencies in protobuf recipe, but I
can't figure out yet what exactly. Any ideas?

Regards,
Slava


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

Re: [oe] [PATCH] Revert "protobuf: stage protoc binary to sysroot"

2023-10-12 Thread Vyacheslav Yurkov

I'd like to follow-up on this and say that I see this issue now with 
nativesdk build, in particular nativesdk-grpc recipe fails in master 
with the same error. I hope that's partially related to the issue 
everybody is confused about, but I'd like understand how to properly fix it.


|   The imported target "protobuf::protoc" references the file
|
| 
"/home/uvv/projects/poky/build/tmp/work/x86_64-nativesdk-pokysdk-linux/nativesdk-grpc/1.56.2/recipe-sysroot/usr/local/oe-sdk-hardcoded-buildpath/sysroots/x86_64-pokysdk-linux/usr/bin/protoc-23.4.0"

|
|   but this file does not exist.  Possible reasons include:
|
|   * The file was deleted, renamed, or moved to another location.

The protoc-23.4.0 file already exists in the recipe-sysroot-native.

And we do want protoc to be present in the SDK, but nativesdk build is a 
cross-compilation, because SDKMACHINE and HOST might be different.
In this particular grpc case there's an option to say where the actual 
protoc executable is ( 
-D_gRPC_PROTOBUF_PROTOC_EXECUTABLE=/home/uvv/projects/poky/build/tmp/work/x86_64-nativesdk-pokysdk-linux/nativesdk-grpc/1.56.2/recipe-sysroot-native/usr/bin/protoc 
is passed to grpc).


So there shouldn't be any issues here, but nativesdk build still fails.

Looking into it further I see that protoc is actually packaged by 
protobuf-compiler package, but when I try to add a package dependency to 
grpc recipe I get this:


nativesdk-protobuf RPROVIDES nativesdk-protobuf-compiler

Moreover, if I go into devshell and forcefully install 
nativesdk-libprotobuf-compiler, then `bitbake nativesdk-grpc` is happy 
and compiles!


So something wrong with the dependencies in protobuf recipe, but I can't 
figure out yet what exactly. Any ideas?


Regards,
Slava

On 02.10.2023 18:25, Vyacheslav Yurkov via lists.openembedded.org wrote:

On 01.10.2023 20:22, Martin Jansa wrote:


Issue with .cmake file from protobuf, right?


No, it's the issue with the cmake of the project that uses protobuf. 
But that depends on a user case, perhaps I didn't get it properly.



If we can properly fix it at the recipe level, then why should we
keep this workaround/hack?

Also if you really need it, you can have a .bbappend for
protoc/grpc in the downstream layer. But I would be in favor to
avoid this by default no?


I haven't seen any proper fix for this exact issue. Yes you can patch 
.cmake generation not to include bindir when building for target 
(while it needs it for native build), but then that change will break 
possible use of protobuf on target, so the SYSROOT_DIRS still seems 
like lesser evil here.


Here are some examples from meta-ros:
https://github.com/ros/meta-ros/commit/875ceed4b1c693c2eb0f2b7ea85385be576a62cc
https://github.com/ros/meta-ros/commit/67c03755543655ad0634e078bc30d8ab9969a16c
https://github.com/ros/meta-ros/commit/5e61eb0520d2841c0da7e14eecc833b3c4104533
https://github.com/ros/meta-ros/commit/a53bb7cc8b45a72b704e75311d68b37a6db50cdf

I don't care about protobuf, but the explanation from Samuli looks 
like the same issue as what I remember from above and I don't see how 
SYSROOT_DIRS causes mixing executables or any issues for installing 
them for target.


I didn't look at the original issue of Samuli, but your issue is also 
fixed by a workaround. IMHO, the proper fix would be not to make CMake 
"happy" by providing a binary it can't use, but a proper export of 
cross-compiled tools for 
https://github.com/geographiclib/geographiclib/blob/main/CMakeLists.txt 
library. I'd say the workflow should be similar to protobuf/grpc 
recipes, i.e. a native recipe builds the libraries only, and the 
target recipe builds tools and libraries.


Regards,
Slava

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

Re: [oe] [meta-filesystems][PATCH 1/7] Add static-passwd and static-group files

2023-10-12 Thread Vyacheslav Yurkov


Thanks for the patch.

Is there any way to ensure (have a QA?) this file is updated / not 
forgotten when somebody else adds new id in the recipe?


Slava

On 10.10.2023 11:39, Fabien Thomas wrote:

In order to pass reproducible tests, recipes that use the
useradd class must have static ids configured.

Signed-off-by: Fabien Thomas 
Reviewed-by: Yoann Congal 
---
  meta-filesystems/files/static-passwd-meta-filesystems | 3 +++
  1 file changed, 3 insertions(+)
  create mode 100644 meta-filesystems/files/static-passwd-meta-filesystems

diff --git a/meta-filesystems/files/static-passwd-meta-filesystems 
b/meta-filesystems/files/static-passwd-meta-filesystems
new file mode 100644
index 0..157697f3e
--- /dev/null
+++ b/meta-filesystems/files/static-passwd-meta-filesystems
@@ -0,0 +1,3 @@
+fsgqa:x:700:nogroup::/:/bin/nologin
+123456-fsgqa:x:701:nogroup::/:/bin/nologin
+fsgqa2:x:702:nogroup::/:/bin/nologin
\ No newline at end of file



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

Re: [oe] [PATCH] Revert "protobuf: stage protoc binary to sysroot"

2023-10-02 Thread Vyacheslav Yurkov


On 01.10.2023 20:22, Martin Jansa wrote:


Issue with .cmake file from protobuf, right?


No, it's the issue with the cmake of the project that uses protobuf. But 
that depends on a user case, perhaps I didn't get it properly.



If we can properly fix it at the recipe level, then why should we
keep this workaround/hack?

Also if you really need it, you can have a .bbappend for
protoc/grpc in the downstream layer. But I would be in favor to
avoid this by default no?


I haven't seen any proper fix for this exact issue. Yes you can patch 
.cmake generation not to include bindir when building for target 
(while it needs it for native build), but then that change will break 
possible use of protobuf on target, so the SYSROOT_DIRS still seems 
like lesser evil here.


Here are some examples from meta-ros:
https://github.com/ros/meta-ros/commit/875ceed4b1c693c2eb0f2b7ea85385be576a62cc
https://github.com/ros/meta-ros/commit/67c03755543655ad0634e078bc30d8ab9969a16c
https://github.com/ros/meta-ros/commit/5e61eb0520d2841c0da7e14eecc833b3c4104533
https://github.com/ros/meta-ros/commit/a53bb7cc8b45a72b704e75311d68b37a6db50cdf

I don't care about protobuf, but the explanation from Samuli looks 
like the same issue as what I remember from above and I don't see how 
SYSROOT_DIRS causes mixing executables or any issues for installing 
them for target.


I didn't look at the original issue of Samuli, but your issue is also 
fixed by a workaround. IMHO, the proper fix would be not to make CMake 
"happy" by providing a binary it can't use, but a proper export of 
cross-compiled tools for 
https://github.com/geographiclib/geographiclib/blob/main/CMakeLists.txt 
library. I'd say the workflow should be similar to protobuf/grpc 
recipes, i.e. a native recipe builds the libraries only, and the target 
recipe builds tools and libraries.


Regards,
Slava

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

Re: [oe] [PATCH] Revert "protobuf: stage protoc binary to sysroot"

2023-09-30 Thread Vyacheslav Yurkov

I agree, protoc should _not_ be in sysroot, because target binary can't 
run on host system.


Vyacheslav

On 30.09.2023 19:30, Clément Péron wrote:

This reverts commit a0557fe5433620717eeb00d3b16801711337b1a4.

As said by Ross[Ø]:
"Putting the _target_ protoc into the sysroot for executation at _build_
time isn't useful because even if it has the right architecture, the
tune might be incompatible.  Recipes which want protoc should just depend
on protobuf-native."

This has been reverted recently by Samuli[1]:
"If protoc is enabled for the build, recipes using protobuf will
fail when protoc is not available in the recipe sysroot"

Be the revert is incorret as This is an issue coming from qtgrpc
other recipes that use protobuf or gRPC compiler, proplery looks for
the binary in the correct sysroot folder.

Qtgrpc recipe should fix this issue at the recipe level, for example this
is what I've done for "etcd-cpp-apiv3" recipe[2] that doesn't need this
patch to properly compile.

So keeping this hack doesn't seems to be a correct fix.

Note that qtgrpc recipe isn't available on meta-oe nor any other public
layers.

0: 
https://patchwork.yoctoproject.org/project/oe/patch/20230904161230.377450-1-ross.bur...@arm.com/
1: 
https://patchwork.yoctoproject.org/project/oe/patch/20230927051101.3088498-1-samuli.pii...@qt.io/
2: 
https://github.com/etcd-cpp-apiv3/etcd-cpp-apiv3/commit/47f0d9e0326f3cc31c801a0ecf7312d1049ece3e

CC: Samuli Piippo 
CC: Ross Burton 
Signed-off-by: Clément Péron 
---
  meta-oe/recipes-devtools/protobuf/protobuf_4.23.4.bb | 3 ---
  1 file changed, 3 deletions(-)

diff --git a/meta-oe/recipes-devtools/protobuf/protobuf_4.23.4.bb 
b/meta-oe/recipes-devtools/protobuf/protobuf_4.23.4.bb
index 06d73d648..1edc21cdf 100644
--- a/meta-oe/recipes-devtools/protobuf/protobuf_4.23.4.bb
+++ b/meta-oe/recipes-devtools/protobuf/protobuf_4.23.4.bb
@@ -101,9 +101,6 @@ PACKAGE_BEFORE_PN = "${PN}-compiler ${PN}-lite"
  FILES:${PN}-compiler = "${bindir} ${libdir}/libprotoc${SOLIBS}"
  FILES:${PN}-lite = "${libdir}/libprotobuf-lite${SOLIBS}"
  
-# CMake requires binaries to exist in sysroot, even if they have wrong architecture.

-SYSROOT_DIRS += "${bindir}"
-
  RDEPENDS:${PN}-compiler = "${PN}"
  RDEPENDS:${PN}-dev += "${PN}-compiler"
  RDEPENDS:${PN}-ptest = "bash ${@bb.utils.contains('PACKAGECONFIG', 'python', 
'python3-protobuf', '', d)}"






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

[oe] [meta-filesystems][PATCH v2 3/3] overlayfs-tools: Install fsck binary

Two projects, overlayfs-tools and overlayfs-progs, were combined
upstream.

Signed-off-by: Vyacheslav Yurkov 
---
 ...-Makefile-proper-location-of-LDFLAGS.patch | 32 ---
 .../overlayfs/overlayfs-progs_git.bb  | 32 ---
 .../overlayfs/overlayfs-tools_git.bb  |  5 +--
 .../xfstests/xfstests_2023.03.05.bb   |  2 +-
 4 files changed, 2 insertions(+), 69 deletions(-)
 delete mode 100644 
meta-filesystems/recipes-utils/overlayfs/overlayfs-progs/0001-Makefile-proper-location-of-LDFLAGS.patch
 delete mode 100644 
meta-filesystems/recipes-utils/overlayfs/overlayfs-progs_git.bb

diff --git 
a/meta-filesystems/recipes-utils/overlayfs/overlayfs-progs/0001-Makefile-proper-location-of-LDFLAGS.patch
 
b/meta-filesystems/recipes-utils/overlayfs/overlayfs-progs/0001-Makefile-proper-location-of-LDFLAGS.patch
deleted file mode 100644
index 2c001834b..0
--- 
a/meta-filesystems/recipes-utils/overlayfs/overlayfs-progs/0001-Makefile-proper-location-of-LDFLAGS.patch
+++ /dev/null
@@ -1,32 +0,0 @@
-From 8eeaee82dcfdf47f16ad880e416b722827f41bdb Mon Sep 17 00:00:00 2001
-From: Vyacheslav Yurkov 
-Date: Mon, 23 May 2022 19:37:32 +0200
-Subject: [PATCH] Makefile: proper location of LDFLAGS
-
-Signed-off-by: Vyacheslav Yurkov 

-Upstream-Status: Pending
-
- Makefile | 4 ++--
- 1 file changed, 2 insertions(+), 2 deletions(-)
-
-diff --git a/Makefile b/Makefile
-index e3c5207..14b155e 100644
 a/Makefile
-+++ b/Makefile
-@@ -1,5 +1,5 @@
- CFLAGS = -Wall -g
--LFLAGS = -lm
-+LDFLAGS = -lm
- CC = gcc
- 
- all: overlay
-@@ -7,7 +7,7 @@ all: overlay
- objects = fsck.o common.o lib.o check.o mount.o path.o overlayfs.o
- 
- overlay: $(objects)
--  $(CC) $(LFLAGS) $(objects) -o fsck.overlay
-+  $(CC) $(objects) -o fsck.overlay $(LDFLAGS) 
- 
- .c.o:
-   $(CC) $(CFLAGS) -c $<
diff --git a/meta-filesystems/recipes-utils/overlayfs/overlayfs-progs_git.bb 
b/meta-filesystems/recipes-utils/overlayfs/overlayfs-progs_git.bb
deleted file mode 100644
index 9d58b9a63..0
--- a/meta-filesystems/recipes-utils/overlayfs/overlayfs-progs_git.bb
+++ /dev/null
@@ -1,32 +0,0 @@
-SUMMARY = "File system check utility for OverlayFS"
-HOMEPAGE = "https://github.com/hisilicon/overlayfs-progs;
-LICENSE = "PD"
-LIC_FILES_CHKSUM = 
"file://${COREBASE}/meta/files/common-licenses/PD;md5=b3597d12946881e13cb3b548d1173851"
-
-SRC_URI = "\
-
git://github.com/hisilicon/overlayfs-progs.git;protocol=https;branch=master \
-file://0001-Makefile-proper-location-of-LDFLAGS.patch \
-"
-
-PV = "1.0+git${SRCPV}"
-SRCREV = "e10ef686570d9c7eff42f52461593a5c15da56bd"
-
-S = "${WORKDIR}/git"
-B = "${S}"
-
-# Required to have the fts.h header for musl
-DEPENDS:append:libc-musl = " fts"
-# Fix the missing fts libs when using musl
-EXTRA_OEMAKE:append:libc-musl = " LDFLAGS='-lfts'"
-
-EXTRA_OEMAKE += "'CC=${CC} -O2' "
-TARGET_CC_ARCH += "${LDFLAGS}"
-
-do_compile () {
-oe_runmake
-}
-
-do_install () {
-install -d ${D}${bindir}
-install -m 0755 ${B}/fsck.overlay ${D}${bindir}
-}
diff --git a/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools_git.bb 
b/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools_git.bb
index de5155133..ca2ec9cc4 100644
--- a/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools_git.bb
+++ b/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools_git.bb
@@ -21,11 +21,8 @@ EXTRA_OEMAKE += "'CC=${CC} -O2'"
 EXTRA_OEMAKE:append:libc-musl = " LDLIBS=-lfts"
 TARGET_CC_ARCH += "${LDFLAGS}"
 
-do_compile () {
-oe_runmake
-}
-
 do_install () {
 install -d ${D}${bindir}
 install -m 0755 ${B}/overlay ${D}${bindir}
+install -m 0755 ${B}/fsck.overlay ${D}${bindir}
 }
diff --git a/meta-filesystems/recipes-utils/xfstests/xfstests_2023.03.05.bb 
b/meta-filesystems/recipes-utils/xfstests/xfstests_2023.03.05.bb
index fdfe04574..cfb6e4fd8 100644
--- a/meta-filesystems/recipes-utils/xfstests/xfstests_2023.03.05.bb
+++ b/meta-filesystems/recipes-utils/xfstests/xfstests_2023.03.05.bb
@@ -28,7 +28,7 @@ RDEPENDS:${PN} += "\
 e2fsprogs-resize2fs \
 libaio \
 libcap-bin \
-overlayfs-progs \
+overlayfs-tools \
 perl \
 python3 \
 python3-core \
-- 
2.35.1


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

[oe] [meta-filesystems][PATCH v2 2/3] overlayfs-tools: Bump up the version

Drop the patches, they have been merged upstream.

Signed-off-by: Vyacheslav Yurkov 
---
 ...-so-that-it-compiles-on-Ubuntu-20.04.patch | 43 ---
 .../0002-makefile-fix-linking-flags.patch | 33 --
 .../overlayfs/overlayfs-tools_git.bb  |  4 +-
 3 files changed, 1 insertion(+), 79 deletions(-)
 delete mode 100644 
meta-filesystems/recipes-utils/overlayfs/overlayfs-tools/0001-Fixed-includes-so-that-it-compiles-on-Ubuntu-20.04.patch
 delete mode 100644 
meta-filesystems/recipes-utils/overlayfs/overlayfs-tools/0002-makefile-fix-linking-flags.patch

diff --git 
a/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools/0001-Fixed-includes-so-that-it-compiles-on-Ubuntu-20.04.patch
 
b/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools/0001-Fixed-includes-so-that-it-compiles-on-Ubuntu-20.04.patch
deleted file mode 100644
index ed84d92c7..0
--- 
a/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools/0001-Fixed-includes-so-that-it-compiles-on-Ubuntu-20.04.patch
+++ /dev/null
@@ -1,43 +0,0 @@
-From 81b4fbb5f52044cb348534c23f10b3884972b09b Mon Sep 17 00:00:00 2001
-From: Beat Schaer 
-Date: Fri, 19 Mar 2021 08:18:58 +0100
-Subject: [PATCH] Fixed includes so that it compiles on Ubuntu 20.04
-

-Upstream-Status: Pending
-
- logic.c | 3 +--
- main.c  | 3 ++-
- 2 files changed, 3 insertions(+), 3 deletions(-)
-
-diff --git a/logic.c b/logic.c
-index 97767f5..47ebfaa 100644
 a/logic.c
-+++ b/logic.c
-@@ -7,8 +7,7 @@
- #include 
- #include 
- #include 
--#include 
--#include 
-+#include 
- #include 
- #include 
- #include "logic.h"
-diff --git a/main.c b/main.c
-index aa11239..f462b98 100644
 a/main.c
-+++ b/main.c
-@@ -12,7 +12,8 @@
- #include 
- #include 
- #include 
--#include 
-+#include 
-+#include 
- #ifndef _SYS_STAT_H
-   #include 
- #endif
--- 
-2.25.1
-
diff --git 
a/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools/0002-makefile-fix-linking-flags.patch
 
b/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools/0002-makefile-fix-linking-flags.patch
deleted file mode 100644
index 8fbb250f0..0
--- 
a/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools/0002-makefile-fix-linking-flags.patch
+++ /dev/null
@@ -1,33 +0,0 @@
-From b4ff5886797e72d1c21da43261ca7648412f3186 Mon Sep 17 00:00:00 2001
-From: Vyacheslav Yurkov 
-Date: Mon, 23 May 2022 19:53:21 +0200
-Subject: [PATCH] makefile: fix linking flags
-
-LDLIBS should be placed at the end according to
-https://www.gnu.org/software/make/manual/html_node/Catalogue-of-Rules.html
-
-Signed-off-by: Vyacheslav Yurkov 

-Upstream-Status: Pending
-
- makefile | 4 ++--
- 1 file changed, 2 insertions(+), 2 deletions(-)
-
-diff --git a/makefile b/makefile
-index fb1bed4..038c7ce 100644
 a/makefile
-+++ b/makefile
-@@ -1,11 +1,11 @@
- CFLAGS = -Wall -std=c99
--LFLAGS = -lm
-+LDLIBS = -lm
- CC = gcc
- 
- all: overlay
- 
- overlay: main.o logic.o sh.o
--  $(CC) $(LFLAGS) main.o logic.o sh.o -o overlay
-+  $(CC) main.o logic.o sh.o -o overlay $(LDLIBS) 
- 
- main.o: main.c logic.h
-   $(CC) $(CFLAGS) -c main.c
diff --git a/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools_git.bb 
b/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools_git.bb
index c638be9af..de5155133 100644
--- a/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools_git.bb
+++ b/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools_git.bb
@@ -5,12 +5,10 @@ LIC_FILES_CHKSUM = 
"file://LICENSE.txt;md5=f312a7c4d02230e8f2b537295d375c69"
 
 SRC_URI = "\
 git://github.com/kmxz/overlayfs-tools.git;protocol=https;branch=master \
-file://0001-Fixed-includes-so-that-it-compiles-on-Ubuntu-20.04.patch \
-file://0002-makefile-fix-linking-flags.patch \
 "
 
 PV = "1.0+git${SRCPV}"
-SRCREV = "291c7f4a3fb548d06c572700650c2e3bccb0cd27"
+SRCREV = "b5e5a829895ac98ccfe4629fbfbd8b819262bd00"
 
 S = "${WORKDIR}/git"
 B = "${S}"
-- 
2.35.1


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

[oe] [meta-filesystems][PATCH v2 1/3] overlayfs-tools: Drop unneeded dependency

Signed-off-by: Vyacheslav Yurkov 
---
 meta-filesystems/recipes-utils/overlayfs/overlayfs-tools_git.bb | 1 -
 1 file changed, 1 deletion(-)

diff --git a/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools_git.bb 
b/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools_git.bb
index eb6e4963a..c638be9af 100644
--- a/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools_git.bb
+++ b/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools_git.bb
@@ -15,7 +15,6 @@ SRCREV = "291c7f4a3fb548d06c572700650c2e3bccb0cd27"
 S = "${WORKDIR}/git"
 B = "${S}"
 
-DEPENDS += "attr"
 # Required to have the fts.h header for musl
 DEPENDS:append:libc-musl = " fts"
 
-- 
2.35.1


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

Re: [oe] [meta-filesystems][PATCH 3/3] overlayfs-tools: Install fsck binary

Thanks for catching this. I'd rather change the dependency in V2.

Slava

On Fri, Sep 29, 2023, 10:22 Khem Raj  wrote:

> On Fri, Sep 29, 2023 at 6:38 AM Vyacheslav Yurkov 
> wrote:
> >
> > Two projects, overlayfs-tools and overlayfs-progs, were combined
> > upstream.
>
> Merging into one is fine however there are dependencies on
> overlayfs-progs so perhaps we need to add
> RPROVIDES = "overlayfs-progs" to overlayfs-tools_git.bb and then
> update the recipes e.g. xftests to depend on overayfs-tools
>
> >
> > Signed-off-by: Vyacheslav Yurkov 
> > ---
> >  ...-Makefile-proper-location-of-LDFLAGS.patch | 32 ---
> >  .../overlayfs/overlayfs-progs_git.bb  | 32 ---
> >  .../overlayfs/overlayfs-tools_git.bb  |  5 +--
> >  3 files changed, 1 insertion(+), 68 deletions(-)
> >  delete mode 100644
> meta-filesystems/recipes-utils/overlayfs/overlayfs-progs/0001-Makefile-proper-location-of-LDFLAGS.patch
> >  delete mode 100644 meta-filesystems/recipes-utils/overlayfs/
> overlayfs-progs_git.bb
> >
> > diff --git
> a/meta-filesystems/recipes-utils/overlayfs/overlayfs-progs/0001-Makefile-proper-location-of-LDFLAGS.patch
> b/meta-filesystems/recipes-utils/overlayfs/overlayfs-progs/0001-Makefile-proper-location-of-LDFLAGS.patch
> > deleted file mode 100644
> > index 2c001834b..0
> > ---
> a/meta-filesystems/recipes-utils/overlayfs/overlayfs-progs/0001-Makefile-proper-location-of-LDFLAGS.patch
> > +++ /dev/null
> > @@ -1,32 +0,0 @@
> > -From 8eeaee82dcfdf47f16ad880e416b722827f41bdb Mon Sep 17 00:00:00 2001
> > -From: Vyacheslav Yurkov 
> > -Date: Mon, 23 May 2022 19:37:32 +0200
> > -Subject: [PATCH] Makefile: proper location of LDFLAGS
> > -
> > -Signed-off-by: Vyacheslav Yurkov 
> > 
> > -Upstream-Status: Pending
> > -
> > - Makefile | 4 ++--
> > - 1 file changed, 2 insertions(+), 2 deletions(-)
> > -
> > -diff --git a/Makefile b/Makefile
> > -index e3c5207..14b155e 100644
> >  a/Makefile
> > -+++ b/Makefile
> > -@@ -1,5 +1,5 @@
> > - CFLAGS = -Wall -g
> > --LFLAGS = -lm
> > -+LDFLAGS = -lm
> > - CC = gcc
> > -
> > - all: overlay
> > -@@ -7,7 +7,7 @@ all: overlay
> > - objects = fsck.o common.o lib.o check.o mount.o path.o overlayfs.o
> > -
> > - overlay: $(objects)
> > --  $(CC) $(LFLAGS) $(objects) -o fsck.overlay
> > -+  $(CC) $(objects) -o fsck.overlay $(LDFLAGS)
> > -
> > - .c.o:
> > -   $(CC) $(CFLAGS) -c $<
> > diff --git a/meta-filesystems/recipes-utils/overlayfs/
> overlayfs-progs_git.bb b/meta-filesystems/recipes-utils/overlayfs/
> overlayfs-progs_git.bb
> > deleted file mode 100644
> > index 9d58b9a63..0
> > --- a/meta-filesystems/recipes-utils/overlayfs/overlayfs-progs_git.bb
> > +++ /dev/null
> > @@ -1,32 +0,0 @@
> > -SUMMARY = "File system check utility for OverlayFS"
> > -HOMEPAGE = "https://github.com/hisilicon/overlayfs-progs;
> > -LICENSE = "PD"
> > -LIC_FILES_CHKSUM =
> "file://${COREBASE}/meta/files/common-licenses/PD;md5=b3597d12946881e13cb3b548d1173851"
> > -
> > -SRC_URI = "\
> > -git://
> github.com/hisilicon/overlayfs-progs.git;protocol=https;branch=master \
> > -file://0001-Makefile-proper-location-of-LDFLAGS.patch \
> > -"
> > -
> > -PV = "1.0+git${SRCPV}"
> > -SRCREV = "e10ef686570d9c7eff42f52461593a5c15da56bd"
> > -
> > -S = "${WORKDIR}/git"
> > -B = "${S}"
> > -
> > -# Required to have the fts.h header for musl
> > -DEPENDS:append:libc-musl = " fts"
> > -# Fix the missing fts libs when using musl
> > -EXTRA_OEMAKE:append:libc-musl = " LDFLAGS='-lfts'"
> > -
> > -EXTRA_OEMAKE += "'CC=${CC} -O2' "
> > -TARGET_CC_ARCH += "${LDFLAGS}"
> > -
> > -do_compile () {
> > -oe_runmake
> > -}
> > -
> > -do_install () {
> > -install -d ${D}${bindir}
> > -install -m 0755 ${B}/fsck.overlay ${D}${bindir}
> > -}
> > diff --git a/meta-filesystems/recipes-utils/overlayfs/
> overlayfs-tools_git.bb b/meta-filesystems/recipes-utils/overlayfs/
> overlayfs-tools_git.bb
> > index de5155133..ca2ec9cc4 100644
> > --- a/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools_git.bb
> > +++ b/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools_git.bb
> > @@ -21,11 +21,8 @@ EXTRA_OEMAKE += "'CC=${CC} -O2'"
> >  EXTRA_OEMAKE:append:libc-musl = " LDLIBS=-lfts"
> >  TARGET_CC_ARCH += "${LDFLAGS}"
> >
> > -do_compile () {
> > -oe_runmake
> > -}
> > -
> >  do_install () {
> >  install -d ${D}${bindir}
> >  install -m 0755 ${B}/overlay ${D}${bindir}
> > +install -m 0755 ${B}/fsck.overlay ${D}${bindir}
> >  }
> > --
> > 2.35.1
> >
> >
> > 
> >
>

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

[oe] [meta-filesystems][PATCH 3/3] overlayfs-tools: Install fsck binary

Two projects, overlayfs-tools and overlayfs-progs, were combined
upstream.

Signed-off-by: Vyacheslav Yurkov 
---
 ...-Makefile-proper-location-of-LDFLAGS.patch | 32 ---
 .../overlayfs/overlayfs-progs_git.bb  | 32 ---
 .../overlayfs/overlayfs-tools_git.bb  |  5 +--
 3 files changed, 1 insertion(+), 68 deletions(-)
 delete mode 100644 
meta-filesystems/recipes-utils/overlayfs/overlayfs-progs/0001-Makefile-proper-location-of-LDFLAGS.patch
 delete mode 100644 
meta-filesystems/recipes-utils/overlayfs/overlayfs-progs_git.bb

diff --git 
a/meta-filesystems/recipes-utils/overlayfs/overlayfs-progs/0001-Makefile-proper-location-of-LDFLAGS.patch
 
b/meta-filesystems/recipes-utils/overlayfs/overlayfs-progs/0001-Makefile-proper-location-of-LDFLAGS.patch
deleted file mode 100644
index 2c001834b..0
--- 
a/meta-filesystems/recipes-utils/overlayfs/overlayfs-progs/0001-Makefile-proper-location-of-LDFLAGS.patch
+++ /dev/null
@@ -1,32 +0,0 @@
-From 8eeaee82dcfdf47f16ad880e416b722827f41bdb Mon Sep 17 00:00:00 2001
-From: Vyacheslav Yurkov 
-Date: Mon, 23 May 2022 19:37:32 +0200
-Subject: [PATCH] Makefile: proper location of LDFLAGS
-
-Signed-off-by: Vyacheslav Yurkov 

-Upstream-Status: Pending
-
- Makefile | 4 ++--
- 1 file changed, 2 insertions(+), 2 deletions(-)
-
-diff --git a/Makefile b/Makefile
-index e3c5207..14b155e 100644
 a/Makefile
-+++ b/Makefile
-@@ -1,5 +1,5 @@
- CFLAGS = -Wall -g
--LFLAGS = -lm
-+LDFLAGS = -lm
- CC = gcc
- 
- all: overlay
-@@ -7,7 +7,7 @@ all: overlay
- objects = fsck.o common.o lib.o check.o mount.o path.o overlayfs.o
- 
- overlay: $(objects)
--  $(CC) $(LFLAGS) $(objects) -o fsck.overlay
-+  $(CC) $(objects) -o fsck.overlay $(LDFLAGS) 
- 
- .c.o:
-   $(CC) $(CFLAGS) -c $<
diff --git a/meta-filesystems/recipes-utils/overlayfs/overlayfs-progs_git.bb 
b/meta-filesystems/recipes-utils/overlayfs/overlayfs-progs_git.bb
deleted file mode 100644
index 9d58b9a63..0
--- a/meta-filesystems/recipes-utils/overlayfs/overlayfs-progs_git.bb
+++ /dev/null
@@ -1,32 +0,0 @@
-SUMMARY = "File system check utility for OverlayFS"
-HOMEPAGE = "https://github.com/hisilicon/overlayfs-progs;
-LICENSE = "PD"
-LIC_FILES_CHKSUM = 
"file://${COREBASE}/meta/files/common-licenses/PD;md5=b3597d12946881e13cb3b548d1173851"
-
-SRC_URI = "\
-
git://github.com/hisilicon/overlayfs-progs.git;protocol=https;branch=master \
-file://0001-Makefile-proper-location-of-LDFLAGS.patch \
-"
-
-PV = "1.0+git${SRCPV}"
-SRCREV = "e10ef686570d9c7eff42f52461593a5c15da56bd"
-
-S = "${WORKDIR}/git"
-B = "${S}"
-
-# Required to have the fts.h header for musl
-DEPENDS:append:libc-musl = " fts"
-# Fix the missing fts libs when using musl
-EXTRA_OEMAKE:append:libc-musl = " LDFLAGS='-lfts'"
-
-EXTRA_OEMAKE += "'CC=${CC} -O2' "
-TARGET_CC_ARCH += "${LDFLAGS}"
-
-do_compile () {
-oe_runmake
-}
-
-do_install () {
-install -d ${D}${bindir}
-install -m 0755 ${B}/fsck.overlay ${D}${bindir}
-}
diff --git a/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools_git.bb 
b/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools_git.bb
index de5155133..ca2ec9cc4 100644
--- a/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools_git.bb
+++ b/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools_git.bb
@@ -21,11 +21,8 @@ EXTRA_OEMAKE += "'CC=${CC} -O2'"
 EXTRA_OEMAKE:append:libc-musl = " LDLIBS=-lfts"
 TARGET_CC_ARCH += "${LDFLAGS}"
 
-do_compile () {
-oe_runmake
-}
-
 do_install () {
 install -d ${D}${bindir}
 install -m 0755 ${B}/overlay ${D}${bindir}
+install -m 0755 ${B}/fsck.overlay ${D}${bindir}
 }
-- 
2.35.1


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

[oe] [meta-filesystems][PATCH 2/3] overlayfs-tools: Bump up the version

Drop the patches, they have been merged upstream.

Signed-off-by: Vyacheslav Yurkov 
---
 ...-so-that-it-compiles-on-Ubuntu-20.04.patch | 43 ---
 .../0002-makefile-fix-linking-flags.patch | 33 --
 .../overlayfs/overlayfs-tools_git.bb  |  4 +-
 3 files changed, 1 insertion(+), 79 deletions(-)
 delete mode 100644 
meta-filesystems/recipes-utils/overlayfs/overlayfs-tools/0001-Fixed-includes-so-that-it-compiles-on-Ubuntu-20.04.patch
 delete mode 100644 
meta-filesystems/recipes-utils/overlayfs/overlayfs-tools/0002-makefile-fix-linking-flags.patch

diff --git 
a/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools/0001-Fixed-includes-so-that-it-compiles-on-Ubuntu-20.04.patch
 
b/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools/0001-Fixed-includes-so-that-it-compiles-on-Ubuntu-20.04.patch
deleted file mode 100644
index ed84d92c7..0
--- 
a/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools/0001-Fixed-includes-so-that-it-compiles-on-Ubuntu-20.04.patch
+++ /dev/null
@@ -1,43 +0,0 @@
-From 81b4fbb5f52044cb348534c23f10b3884972b09b Mon Sep 17 00:00:00 2001
-From: Beat Schaer 
-Date: Fri, 19 Mar 2021 08:18:58 +0100
-Subject: [PATCH] Fixed includes so that it compiles on Ubuntu 20.04
-

-Upstream-Status: Pending
-
- logic.c | 3 +--
- main.c  | 3 ++-
- 2 files changed, 3 insertions(+), 3 deletions(-)
-
-diff --git a/logic.c b/logic.c
-index 97767f5..47ebfaa 100644
 a/logic.c
-+++ b/logic.c
-@@ -7,8 +7,7 @@
- #include 
- #include 
- #include 
--#include 
--#include 
-+#include 
- #include 
- #include 
- #include "logic.h"
-diff --git a/main.c b/main.c
-index aa11239..f462b98 100644
 a/main.c
-+++ b/main.c
-@@ -12,7 +12,8 @@
- #include 
- #include 
- #include 
--#include 
-+#include 
-+#include 
- #ifndef _SYS_STAT_H
-   #include 
- #endif
--- 
-2.25.1
-
diff --git 
a/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools/0002-makefile-fix-linking-flags.patch
 
b/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools/0002-makefile-fix-linking-flags.patch
deleted file mode 100644
index 8fbb250f0..0
--- 
a/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools/0002-makefile-fix-linking-flags.patch
+++ /dev/null
@@ -1,33 +0,0 @@
-From b4ff5886797e72d1c21da43261ca7648412f3186 Mon Sep 17 00:00:00 2001
-From: Vyacheslav Yurkov 
-Date: Mon, 23 May 2022 19:53:21 +0200
-Subject: [PATCH] makefile: fix linking flags
-
-LDLIBS should be placed at the end according to
-https://www.gnu.org/software/make/manual/html_node/Catalogue-of-Rules.html
-
-Signed-off-by: Vyacheslav Yurkov 

-Upstream-Status: Pending
-
- makefile | 4 ++--
- 1 file changed, 2 insertions(+), 2 deletions(-)
-
-diff --git a/makefile b/makefile
-index fb1bed4..038c7ce 100644
 a/makefile
-+++ b/makefile
-@@ -1,11 +1,11 @@
- CFLAGS = -Wall -std=c99
--LFLAGS = -lm
-+LDLIBS = -lm
- CC = gcc
- 
- all: overlay
- 
- overlay: main.o logic.o sh.o
--  $(CC) $(LFLAGS) main.o logic.o sh.o -o overlay
-+  $(CC) main.o logic.o sh.o -o overlay $(LDLIBS) 
- 
- main.o: main.c logic.h
-   $(CC) $(CFLAGS) -c main.c
diff --git a/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools_git.bb 
b/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools_git.bb
index c638be9af..de5155133 100644
--- a/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools_git.bb
+++ b/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools_git.bb
@@ -5,12 +5,10 @@ LIC_FILES_CHKSUM = 
"file://LICENSE.txt;md5=f312a7c4d02230e8f2b537295d375c69"
 
 SRC_URI = "\
 git://github.com/kmxz/overlayfs-tools.git;protocol=https;branch=master \
-file://0001-Fixed-includes-so-that-it-compiles-on-Ubuntu-20.04.patch \
-file://0002-makefile-fix-linking-flags.patch \
 "
 
 PV = "1.0+git${SRCPV}"
-SRCREV = "291c7f4a3fb548d06c572700650c2e3bccb0cd27"
+SRCREV = "b5e5a829895ac98ccfe4629fbfbd8b819262bd00"
 
 S = "${WORKDIR}/git"
 B = "${S}"
-- 
2.35.1


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

[oe] [meta-filesystems][PATCH 1/3] overlayfs-tools: Drop unneeded dependency

Signed-off-by: Vyacheslav Yurkov 
---
 meta-filesystems/recipes-utils/overlayfs/overlayfs-tools_git.bb | 1 -
 1 file changed, 1 deletion(-)

diff --git a/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools_git.bb 
b/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools_git.bb
index eb6e4963a..c638be9af 100644
--- a/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools_git.bb
+++ b/meta-filesystems/recipes-utils/overlayfs/overlayfs-tools_git.bb
@@ -15,7 +15,6 @@ SRCREV = "291c7f4a3fb548d06c572700650c2e3bccb0cd27"
 S = "${WORKDIR}/git"
 B = "${S}"
 
-DEPENDS += "attr"
 # Required to have the fts.h header for musl
 DEPENDS:append:libc-musl = " fts"
 
-- 
2.35.1


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

Re: [oe] [meta-oe][PATCH] protobuf: stage protoc binary to sysroot

2023-09-28 Thread Vyacheslav Yurkov

I think I was the one who originally removed protoc, because it's not 
useful in the sysroot. Even if you satisfy cmake so it can find protoc 
target, the next step is that it will try to use it, which will 
obviously fail.


The only reason to put it on the target sysroot would be if you really 
going to use it _on the target_. But I guess this can be done via 
bbappend in the user layer.


Vyacheslav

On 27.09.2023 07:19, Samuli Piippo wrote:
Re-applying this patch as it got removed in 
https://patchwork.yoctoproject.org/project/oe/patch/20230904161230.377450-1-ross.bur...@arm.com/


On Wed, 27 Sept 2023 at 08:11, Samuli Piippo via 
lists.openembedded.org  
 wrote:


From: Samuli Piippo 

If protoc is enabled for the build, recipes using protobuf will
fail when protoc is not available in the recipe sysroot:

|   The imported target "protobuf::protoc" references the file
|
|

".../recipe-sysroot/usr/local/oe-sdk-hardcoded-buildpath/sysroots/x86_64-pokysdk-linux/usr/bin/protoc-3.21.5.0"
|
|   but this file does not exist.  Possible reasons include:
|
|   * The file was deleted, renamed, or moved to another location.
|
|   * An install or uninstall procedure did not complete successfully.
|
|   * The installation package was faulty and contained
|
|

".../recipe-sysroot/usr/local/oe-sdk-hardcoded-buildpath/sysroots/x86_64-pokysdk-linux/usr/lib/cmake/protobuf/protobuf-targets.cmake"
|
|   but not all the files it references.

Use SYSROOT_DIRS to stage the binary to sysroot so it's always
available for other recipes.

Signed-off-by: Samuli Piippo 
Signed-off-by: Khem Raj 
---
 meta-oe/recipes-devtools/protobuf/protobuf_3.21.12.bb
 | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/meta-oe/recipes-devtools/protobuf/protobuf_3.21.12.bb

b/meta-oe/recipes-devtools/protobuf/protobuf_3.21.12.bb

index d95e1c20f..06b21a0d6 100644
--- a/meta-oe/recipes-devtools/protobuf/protobuf_3.21.12.bb

+++ b/meta-oe/recipes-devtools/protobuf/protobuf_3.21.12.bb

@@ -92,6 +92,9 @@ PACKAGE_BEFORE_PN = "${PN}-compiler ${PN}-lite"
 FILES:${PN}-compiler = "${bindir} ${libdir}/libprotoc${SOLIBS}"
 FILES:${PN}-lite = "${libdir}/libprotobuf-lite${SOLIBS}"

+# CMake requires binaries to exist in sysroot, even if they have
wrong architecture.
+SYSROOT_DIRS += "${bindir}"
+
 RDEPENDS:${PN}-compiler = "${PN}"
 RDEPENDS:${PN}-dev += "${PN}-compiler"
 RDEPENDS:${PN}-ptest = "bash
${@bb.utils.contains('PACKAGECONFIG', 'python',
'python3-protobuf', '', d)}"
-- 
2.25.1






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

[oe] [meta-networking][PATCH] opcua: Add new recipe

2022-10-03 Thread Vyacheslav Yurkov

From: Vyacheslav Yurkov 

open62541 (http://open62541.org) is an open source and free implementation
of OPC Unified Architecture according to IEC62541 standard

The patch exclude git-related files from installation directory
Upstream-Status: Accepted
https://github.com/open62541/open62541/commit/a0328d4cb527f9778a262fc4a6b42bfbdffc38e9

Signed-off-by: Vyacheslav Yurkov 
---
 ...1-fix-build-do-not-install-git-files.patch | 22 +++
 .../opcua/open62541_1.3.3.bb  | 38 +++
 2 files changed, 60 insertions(+)
 create mode 100644 
meta-networking/recipes-protocols/opcua/open62541/0001-fix-build-do-not-install-git-files.patch
 create mode 100644 meta-networking/recipes-protocols/opcua/open62541_1.3.3.bb

diff --git 
a/meta-networking/recipes-protocols/opcua/open62541/0001-fix-build-do-not-install-git-files.patch
 
b/meta-networking/recipes-protocols/opcua/open62541/0001-fix-build-do-not-install-git-files.patch
new file mode 100644
index 0..44ee7b92c
--- /dev/null
+++ 
b/meta-networking/recipes-protocols/opcua/open62541/0001-fix-build-do-not-install-git-files.patch
@@ -0,0 +1,22 @@
+From a490e82dca5a669b0af27a13d74759d8f77e2333 Mon Sep 17 00:00:00 2001
+From: Vyacheslav Yurkov 
+Date: Mon, 3 Oct 2022 18:25:15 +0200
+Subject: [PATCH] fix(build): do not install git files
+
+Signed-off-by: Vyacheslav Yurkov 
+---
+ CMakeLists.txt | 1 +
+ 1 file changed, 1 insertion(+)
+
+diff --git a/CMakeLists.txt b/CMakeLists.txt
+index 1934374e..bc5c8bad 100644
+--- a/CMakeLists.txt
 b/CMakeLists.txt
+@@ -1703,6 +1703,7 @@ install(DIRECTORY ${UA_install_tools_dirs}
+ FILES_MATCHING
+ PATTERN "*"
+ PATTERN "*.pyc" EXCLUDE
++PATTERN ".git*" EXCLUDE
+ )
+ 
+ install(FILES ${UA_install_tools_files} DESTINATION 
${open62541_install_tools_dir})
diff --git a/meta-networking/recipes-protocols/opcua/open62541_1.3.3.bb 
b/meta-networking/recipes-protocols/opcua/open62541_1.3.3.bb
new file mode 100644
index 0..317697262
--- /dev/null
+++ b/meta-networking/recipes-protocols/opcua/open62541_1.3.3.bb
@@ -0,0 +1,38 @@
+DESCRIPTION = "open62541 is an implementation of OPC UA (OPC Unified 
Architecture)"
+HOMEPAGE = "https://github.com/open62541/open62541.git;
+LICENSE = "MPL-2.0 & BSD-3-Clause & MIT"
+LIC_FILES_CHKSUM = "\
+file://LICENSE;md5=815ca599c9df247a0c7f619bab123dad \
+file://deps/mdnsd/LICENSE;md5=3bb4047dc4095cd7336de3e2a9be94f0 \
+file://deps/mqtt-c/LICENSE;md5=9226377baf0b79174c89a1ab55592456 \
+"
+
+SRCREV_FORMAT = "opcua_mdnsd_ua-nodeset_mqtt-c"
+SRCREV_opcua = "ff73268829359639531ff02905c889f73a77b408"
+SRCREV_mdnsd = "3151afe5899dba5125dffa9f4cf3ae1fe2edc0f0"
+SRCREV_ua-nodeset = "f71b3f411d5cb16097c3ae0c744f67ad45535ffb"
+SRCREV_mqtt-c = "f69ce1e7fd54f3b1834c9c9137ce0ec5d703cb4d"
+
+SRC_URI = " \
+
git://github.com/open62541/open62541.git;name=opcua;branch=1.3;protocol=https \
+
git://github.com/Pro/mdnsd.git;name=mdnsd;protocol=https;branch=master;destsuffix=git/deps/mdnsd
 \
+
git://github.com/OPCFoundation/UA-Nodeset;name=ua-nodeset;protocol=https;branch=v1.04;destsuffix=git/deps/ua-nodeset
 \
+
git://github.com/LiamBindle/MQTT-C.git;name=mqtt-c;protocol=https;branch=master;destsuffix=git/deps/mqtt-c
 \
+file://0001-fix-build-do-not-install-git-files.patch \
+"
+
+S = "${WORKDIR}/git"
+
+inherit cmake python3native
+
+EXTRA_OECMAKE += "\
+-DBUILD_SHARED_LIBS=ON \
+-DUA_NAMESPACE_ZERO=FULL \
+-DUA_LOGLEVEL=600 \
+"
+
+PACKAGECONFIG ?= "encryption pubsub pubsub-eth"
+PACKAGECONFIG[amalgamation] = "-DUA_ENABLE_AMALGAMATION=ON, 
-DUA_ENABLE_AMALGAMATION=OFF"
+PACKAGECONFIG[encryption] = "-DUA_ENABLE_ENCRYPTION=ON, 
-DUA_ENABLE_ENCRYPTION=OFF, mbedtls"
+PACKAGECONFIG[pubsub] = "-DUA_ENABLE_PUBSUB=ON, -DUA_ENABLE_PUBSUB=OFF"
+PACKAGECONFIG[pubsub-eth] = "-DUA_ENABLE_PUBSUB_ETH_UADP=ON, 
-DUA_ENABLE_PUBSUB_ETH_UADP=OFF"
-- 
2.30.2


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

[oe] [meta-oe][PATCH v3] protobuf: disable protoc binary for target

From: Vyacheslav Yurkov 

When applications want to use protoc during the build, the target binary
has to be disabled in order to avoid following error:

|   The imported target "protobuf::protoc" references the file
|
|  "/recipe-sysroot/usr/bin/protoc-"
|
|   but this file does not exist.  Possible reasons include:
|
|   * The file was deleted, renamed, or moved to another location.
|
|   * An install or uninstall procedure did not complete successfully.
|
|   * The installation package was faulty and contained
|
|  "/recipe-sysroot/usr/lib/cmake/protobuf/protobuf-targets.cmake"
|
|   but not all the files it references.

Signed-off-by: Vyacheslav Yurkov 
---
 meta-oe/recipes-devtools/protobuf/protobuf_3.21.5.bb | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/meta-oe/recipes-devtools/protobuf/protobuf_3.21.5.bb 
b/meta-oe/recipes-devtools/protobuf/protobuf_3.21.5.bb
index 83dabc9f8..0d8648e2f 100644
--- a/meta-oe/recipes-devtools/protobuf/protobuf_3.21.5.bb
+++ b/meta-oe/recipes-devtools/protobuf/protobuf_3.21.5.bb
@@ -25,7 +25,9 @@ S = "${WORKDIR}/git"
 inherit cmake pkgconfig ptest
 
 PACKAGECONFIG ??= ""
+PACKAGECONFIG:class-native ?= "compiler"
 PACKAGECONFIG[python] = ",,"
+PACKAGECONFIG[compiler] = 
"-Dprotobuf_BUILD_PROTOC_BINARIES=ON,-Dprotobuf_BUILD_PROTOC_BINARIES=OFF"
 
 EXTRA_OECMAKE += "\
 -Dprotobuf_BUILD_SHARED_LIBS=ON \
-- 
2.25.1


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

Re: [oe] [meta-oe][PATCH v2] protobuf: disable protoc binary for target


On 27.08.2022 11:18, Khem Raj wrote:

On Sat, Aug 27, 2022 at 1:00 AM Vyacheslav Yurkov  wrote:

From: Vyacheslav Yurkov 

When applications want to use protoc during the build, the target binary
has to be disabled in order to avoid following error:

|   The imported target "protobuf::protoc" references the file
|
|  "/recipe-sysroot/usr/bin/protoc-"
|
|   but this file does not exist.  Possible reasons include:
|
|   * The file was deleted, renamed, or moved to another location.
|
|   * An install or uninstall procedure did not complete successfully.
|
|   * The installation package was faulty and contained
|
|  "/recipe-sysroot/usr/lib/cmake/protobuf/protobuf-targets.cmake"
|
|   but not all the files it references.

Signed-off-by: Vyacheslav Yurkov 
---
  meta-oe/recipes-devtools/protobuf/protobuf_3.21.5.bb | 7 +++
  1 file changed, 7 insertions(+)

diff --git a/meta-oe/recipes-devtools/protobuf/protobuf_3.21.5.bb 
b/meta-oe/recipes-devtools/protobuf/protobuf_3.21.5.bb
index 83dabc9f8..e36179363 100644
--- a/meta-oe/recipes-devtools/protobuf/protobuf_3.21.5.bb
+++ b/meta-oe/recipes-devtools/protobuf/protobuf_3.21.5.bb
@@ -34,6 +34,13 @@ EXTRA_OECMAKE += "\
  -Dprotobuf_BUILD_EXAMPLES=OFF \
  "

+# Configuration below allows to cross-compile protobuf applications
+# and use native protoc binary in your recipes. In order to use protoc
+# on the target, set protobuf_BUILD_PROTOC_BINARIES to ON in your bbappend
+EXTRA_OECMAKE:append:class-target = " \
+ -Dprotobuf_BUILD_PROTOC_BINARIES=OFF \
+"
+

can we make it a packageconfig perhaps ?



Makes sense. Any suggestions about the name? So far only something like 
"compiler" comes to mind.


Vyacheslav

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

[oe] [meta-oe][PATCH v2] protobuf: disable protoc binary for target

From: Vyacheslav Yurkov 

When applications want to use protoc during the build, the target binary
has to be disabled in order to avoid following error:

|   The imported target "protobuf::protoc" references the file
|
|  "/recipe-sysroot/usr/bin/protoc-"
|
|   but this file does not exist.  Possible reasons include:
|
|   * The file was deleted, renamed, or moved to another location.
|
|   * An install or uninstall procedure did not complete successfully.
|
|   * The installation package was faulty and contained
|
|  "/recipe-sysroot/usr/lib/cmake/protobuf/protobuf-targets.cmake"
|
|   but not all the files it references.

Signed-off-by: Vyacheslav Yurkov 
---
 meta-oe/recipes-devtools/protobuf/protobuf_3.21.5.bb | 7 +++
 1 file changed, 7 insertions(+)

diff --git a/meta-oe/recipes-devtools/protobuf/protobuf_3.21.5.bb 
b/meta-oe/recipes-devtools/protobuf/protobuf_3.21.5.bb
index 83dabc9f8..e36179363 100644
--- a/meta-oe/recipes-devtools/protobuf/protobuf_3.21.5.bb
+++ b/meta-oe/recipes-devtools/protobuf/protobuf_3.21.5.bb
@@ -34,6 +34,13 @@ EXTRA_OECMAKE += "\
 -Dprotobuf_BUILD_EXAMPLES=OFF \
 "
 
+# Configuration below allows to cross-compile protobuf applications
+# and use native protoc binary in your recipes. In order to use protoc
+# on the target, set protobuf_BUILD_PROTOC_BINARIES to ON in your bbappend
+EXTRA_OECMAKE:append:class-target = " \
+ -Dprotobuf_BUILD_PROTOC_BINARIES=OFF \
+"
+
 TEST_SRC_DIR = "examples"
 LANG_SUPPORT = "cpp ${@bb.utils.contains('PACKAGECONFIG', 'python', 'python', 
'', d)}"
 
-- 
2.25.1


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

Re: [oe] [meta-oe][PATCH] protobuf: disable protoc binary for targe


On master-next please

Thanks,
Vyacheslav

On 27.08.2022 09:55, Vyacheslav Yurkov via lists.openembedded.org wrote:

From: Vyacheslav Yurkov 

When applications want to use protoc during the build, the target binary
has to be disabled in order to avoid following error:

|   The imported target "protobuf::protoc" references the file
|
|  "/recipe-sysroot/usr/bin/protoc-"
|
|   but this file does not exist.  Possible reasons include:
|
|   * The file was deleted, renamed, or moved to another location.
|
|   * An install or uninstall procedure did not complete successfully.
|
|   * The installation package was faulty and contained
|
|  "/recipe-sysroot/usr/lib/cmake/protobuf/protobuf-targets.cmake"
|
|   but not all the files it references.

Signed-off-by: Vyacheslav Yurkov 
---
  meta-oe/recipes-devtools/protobuf/protobuf_3.21.5.bb | 7 +++
  1 file changed, 7 insertions(+)

diff --git a/meta-oe/recipes-devtools/protobuf/protobuf_3.21.5.bb 
b/meta-oe/recipes-devtools/protobuf/protobuf_3.21.5.bb
index 83dabc9f8..e36179363 100644
--- a/meta-oe/recipes-devtools/protobuf/protobuf_3.21.5.bb
+++ b/meta-oe/recipes-devtools/protobuf/protobuf_3.21.5.bb
@@ -34,6 +34,13 @@ EXTRA_OECMAKE += "\
  -Dprotobuf_BUILD_EXAMPLES=OFF \
  "
  
+# Configuration below allows to cross-compile protobuf applications

+# and use native protoc binary in your recipes. In order to use protoc
+# on the target, set protobuf_BUILD_PROTOC_BINARIES to ON in your bbappend
+EXTRA_OECMAKE:append:class-target = " \
+ -Dprotobuf_BUILD_PROTOC_BINARIES=OFF \
+"
+
  TEST_SRC_DIR = "examples"
  LANG_SUPPORT = "cpp ${@bb.utils.contains('PACKAGECONFIG', 'python', 'python', '', 
d)}"
  








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

[oe] [meta-oe][PATCH] protobuf: disable protoc binary for targe