Re: [Buildroot] [PATCH] package/avahi: fix python install issue

["Yann E. MORIN" <yann.morin.1998@free.fr>]

Waldemar, All,

On 2024-08-17 15:18 +0200, Waldemar Brodkorb spake thusly:
> Since the update of Python to 3.12.4 in commit
> 76cd14167f5eb3ffe22e131ddbbecf9bc61f2ef1 avahi fails to install
> python subcomponents. The reason is that the obsolete imp
> python modul is used. Add a patch to remove the imp dependency.
> 
> Fixes:
>  - http://autobuild.buildroot.net/results/2bf/2bf391b93362204917a560705bc402585334ab3c
> 
> Signed-off-by: Waldemar Brodkorb <wbx@openadk.org>

Applied to master, thanks.

But see below

> ---
>  .../0002-remove-imp-from-py-compile.patch     | 50 +++++++++++++++++++
>  1 file changed, 50 insertions(+)
>  create mode 100644 package/avahi/0002-remove-imp-from-py-compile.patch
> 
> diff --git a/package/avahi/0002-remove-imp-from-py-compile.patch b/package/avahi/0002-remove-imp-from-py-compile.patch
> new file mode 100644
> index 0000000000..37534a3f7a
> --- /dev/null
> +++ b/package/avahi/0002-remove-imp-from-py-compile.patch
> @@ -0,0 +1,50 @@
> +Remove optional imp python module dependency.
> +
> +Upstream already removed py-compile entirely in their git repository,
> +so the patch is not appropiate for upstream.

This is really weird: the git tree removed the py-comple in 2005, and it
is not present in the v0.8 tag, but then the release tarball, that we
use, does carry that file.

Could you see with upstream how they end up with py-compile in the
released archive, while it is not in git?

There is a risk that that file gets added somehow by the autotools
packagng process, so it will eventually be present in the next release
as well, and we'd have to carry that patch forever and ever...

Regards,
Yann E. MORIN.

> +Signed-off-by: Waldemar Brodkorb <wbx@openadk.org>
> +Upstream: not appropiate
> +
> +diff -Nur avahi-0.8.orig/py-compile avahi-0.8/py-compile
> +--- avahi-0.8.orig/py-compile	2020-02-18 07:58:21.192417644 +0100
> ++++ avahi-0.8/py-compile	2024-08-17 15:03:07.328733849 +0200
> +@@ -116,7 +116,7 @@
> + fi
> + 
> + $PYTHON -c "
> +-import sys, os, py_compile, imp
> ++import sys, os, py_compile
> + 
> + files = '''$files'''
> + 
> +@@ -129,15 +129,12 @@
> + 	    continue
> +     sys.stdout.write(file)
> +     sys.stdout.flush()
> +-    if hasattr(imp, 'get_tag'):
> +-        py_compile.compile(filepath, imp.cache_from_source(filepath), path)
> +-    else:
> +-        py_compile.compile(filepath, filepath + 'c', path)
> ++    py_compile.compile(filepath, filepath + 'c', path)
> + sys.stdout.write('\n')" || exit $?
> + 
> + # this will fail for python < 1.5, but that doesn't matter ...
> + $PYTHON -O -c "
> +-import sys, os, py_compile, imp
> ++import sys, os, py_compile
> + 
> + # pypy does not use .pyo optimization
> + if hasattr(sys, 'pypy_translation_info'):
> +@@ -153,10 +150,7 @@
> + 	    continue
> +     sys.stdout.write(file)
> +     sys.stdout.flush()
> +-    if hasattr(imp, 'get_tag'):
> +-        py_compile.compile(filepath, imp.cache_from_source(filepath, False), path)
> +-    else:
> +-        py_compile.compile(filepath, filepath + 'o', path)
> ++    py_compile.compile(filepath, filepath + 'o', path)
> + sys.stdout.write('\n')" 2>/dev/null || :
> + 
> + # Local Variables:
> -- 
> 2.30.2
> 
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot