Hi Etienne,

On 9/13/22 17:25, Etienne Cordonnier via lists.openembedded.org wrote:
editline is a BSD alternative for readline which is GPLv3.

Signed-off-by: Etienne Cordonnier <[email protected]>
---
  meta/recipes-devtools/python/python3_3.10.6.bb | 1 +
  1 file changed, 1 insertion(+)

diff --git a/meta/recipes-devtools/python/python3_3.10.6.bb 
b/meta/recipes-devtools/python/python3_3.10.6.bb
index 1b28728732..1f8b60a7a5 100644
--- a/meta/recipes-devtools/python/python3_3.10.6.bb
+++ b/meta/recipes-devtools/python/python3_3.10.6.bb
@@ -105,6 +105,7 @@ PACKAGECONFIG:class-target ??= "readline gdbm 
${@bb.utils.filter('DISTRO_FEATURE
  PACKAGECONFIG:class-native ??= "readline gdbm"
  PACKAGECONFIG:class-nativesdk ??= "readline gdbm"
  PACKAGECONFIG[readline] = ",,readline"
+PACKAGECONFIG[editline] = "--with-readline=editline,,libedit,,,readline"

I'm wondering if something else isn't missing just below:
https://cgit.openembedded.org/openembedded-core/tree/meta/recipes-devtools/python/python3_3.10.6.bb?h=master#n119

Considering the introducing commit: https://cgit.openembedded.org/openembedded-core/commit/?id=79ebbca6943bc66b619671f93ac283ed2cfc8e5c

I'm wondering if there isn't something we can do better here.

I see that the configure.ac script allows to disable readline entirely, c.f. https://github.com/python/cpython/blob/v3.10.6/configure.ac#L4945-L5062

--without-readline should work. I hope this can fix the issue the aforementioned commit works around by avoiding to always try to build the readline module.

Now we'll have another issue is that this without-readline should be passed only when both readline and editline PACKAGECONFIG are not used.

I guess we usually do it by having EXTRA_OECONF with the --without-readline always set (you'll need --with-readline in the readline PACKAGECONFIG though). Then if readline or editline is set, --without-readline will be overridden by either --with-readline or --with-readline=editline?

Cc'ing Alex as author of the aforementioned commit, hope you don't mind :)

Cheers,
Quentin

  # Use profile guided optimisation by running PyBench inside qemu-user
  PACKAGECONFIG[pgo] = "--enable-optimizations,,qemu-native"
  PACKAGECONFIG[tk] = ",,tk"





-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#170647): 
https://lists.openembedded.org/g/openembedded-core/message/170647
Mute This Topic: https://lists.openembedded.org/mt/93657764/21656
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to