All the mail mirrored from lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] python3targetconfig.bbclass: Extend PYTHONPATH instead of overwriting
@ 2023-04-25  9:15 dev
  2023-04-25 14:25 ` [OE-core] " Tim Orling
  0 siblings, 1 reply; 3+ messages in thread
From: dev @ 2023-04-25  9:15 UTC (permalink / raw
  To: openembedded-core; +Cc: Johannes Schrimpf

From: Johannes Schrimpf <dev@loewen-email.de>

[YOCTO #15108] -- https://bugzilla.yoctoproject.org/show_bug.cgi?id=15108

Since the latest change, the PYTHONPATH is overwritten instead of extended.
This leads to changed behavior and build errors of recipes where the PYTHONPATH
is set before setup_target_config is run.

Signed-off-by: Johannes Schrimpf <dev@loewen-email.de>
---
 meta/classes-recipe/python3targetconfig.bbclass | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/classes-recipe/python3targetconfig.bbclass b/meta/classes-recipe/python3targetconfig.bbclass
index 22305fe176..08bc619398 100644
--- a/meta/classes-recipe/python3targetconfig.bbclass
+++ b/meta/classes-recipe/python3targetconfig.bbclass
@@ -12,7 +12,7 @@ DEPENDS:append = " ${EXTRA_PYTHON_DEPENDS}"
 
 setup_target_config() {
         export _PYTHON_SYSCONFIGDATA_NAME="_sysconfigdata"
-        export PYTHONPATH=${STAGING_LIBDIR}/python-sysconfigdata
+        export PYTHONPATH=${STAGING_LIBDIR}/python-sysconfigdata:$PYTHONPATH
         export PATH=${STAGING_EXECPREFIXDIR}/python-target-config/:$PATH
 }
 
-- 
2.37.1 (Apple Git-137.1)



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [OE-core] [PATCH] python3targetconfig.bbclass: Extend PYTHONPATH instead of overwriting
  2023-04-25  9:15 [PATCH] python3targetconfig.bbclass: Extend PYTHONPATH instead of overwriting dev
@ 2023-04-25 14:25 ` Tim Orling
  2023-04-26  8:08   ` Luca Ceresoli
  0 siblings, 1 reply; 3+ messages in thread
From: Tim Orling @ 2023-04-25 14:25 UTC (permalink / raw
  To: dev; +Cc: openembedded-core

[-- Attachment #1: Type: text/plain, Size: 1900 bytes --]

On Tue, Apr 25, 2023 at 2:15 AM <dev@loewen-email.de> wrote:

> From: Johannes Schrimpf <dev@loewen-email.de>
>
> [YOCTO #15108] -- https://bugzilla.yoctoproject.org/show_bug.cgi?id=15108
>
> Since the latest change,


We should mention the commit where the change was made:
https://git.openembedded.org/openembedded-core/commit/meta/classes-recipe/python3targetconfig.bbclass?id=c9617c03bceee54dc540318cada392799b137bd5

the PYTHONPATH is overwritten instead of extended.
> This leads to changed behavior and build errors of recipes where the
> PYTHONPATH
> is set before setup_target_config is run.
>
> Signed-off-by: Johannes Schrimpf <dev@loewen-email.de>
> ---
>  meta/classes-recipe/python3targetconfig.bbclass | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta/classes-recipe/python3targetconfig.bbclass
> b/meta/classes-recipe/python3targetconfig.bbclass
> index 22305fe176..08bc619398 100644
> --- a/meta/classes-recipe/python3targetconfig.bbclass
> +++ b/meta/classes-recipe/python3targetconfig.bbclass
> @@ -12,7 +12,7 @@ DEPENDS:append = " ${EXTRA_PYTHON_DEPENDS}"
>
>  setup_target_config() {
>          export _PYTHON_SYSCONFIGDATA_NAME="_sysconfigdata"
> -        export PYTHONPATH=${STAGING_LIBDIR}/python-sysconfigdata
> +        export
> PYTHONPATH=${STAGING_LIBDIR}/python-sysconfigdata:$PYTHONPATH
>          export PATH=${STAGING_EXECPREFIXDIR}/python-target-config/:$PATH
>  }
>
> --
> 2.37.1 (Apple Git-137.1)
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#180375):
> https://lists.openembedded.org/g/openembedded-core/message/180375
> Mute This Topic: https://lists.openembedded.org/mt/98489203/924729
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [
> ticotimo@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
>

[-- Attachment #2: Type: text/html, Size: 3493 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [OE-core] [PATCH] python3targetconfig.bbclass: Extend PYTHONPATH instead of overwriting
  2023-04-25 14:25 ` [OE-core] " Tim Orling
@ 2023-04-26  8:08   ` Luca Ceresoli
  0 siblings, 0 replies; 3+ messages in thread
From: Luca Ceresoli @ 2023-04-26  8:08 UTC (permalink / raw
  To: Tim Orling; +Cc: dev, openembedded-core

Hi Tim, Johannes,

On Tue, 25 Apr 2023 07:25:58 -0700
"Tim Orling" <ticotimo@gmail.com> wrote:

> On Tue, Apr 25, 2023 at 2:15 AM <dev@loewen-email.de> wrote:
> 
> > From: Johannes Schrimpf <dev@loewen-email.de>
> >
> > [YOCTO #15108] -- https://bugzilla.yoctoproject.org/show_bug.cgi?id=15108
> >
> > Since the latest change,  
> 
> 
> We should mention the commit where the change was made:
> https://git.openembedded.org/openembedded-core/commit/meta/classes-recipe/python3targetconfig.bbclass?id=c9617c03bceee54dc540318cada392799b137bd5

Good idea. I'm taking this patch for testing and adding a 'Fixes' tag.
No need to resend.

Best regards,
Luca

-- 
Luca Ceresoli, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2023-04-26  8:08 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-04-25  9:15 [PATCH] python3targetconfig.bbclass: Extend PYTHONPATH instead of overwriting dev
2023-04-25 14:25 ` [OE-core] " Tim Orling
2023-04-26  8:08   ` Luca Ceresoli

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.