From: "Mattijs Korpershoek" <mkorpershoek@baylibre.com>
To: Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
Alexander Kanavin <alex.kanavin@gmail.com>
Cc: libcamera-devel@lists.libcamera.org,
openembedded-devel@lists.openembedded.org
Subject: Re: [libcamera-devel] [PATCH 19/20] libcamera: skip until upstream resolves python 3.12 compatibility
Date: Fri, 15 Dec 2023 14:21:53 +0100 [thread overview]
Message-ID: <8734w3txce.fsf@baylibre.com> (raw)
In-Reply-To: <20231215125050.GK21146@pendragon.ideasonboard.com>
Hi Laurent,
On ven., déc. 15, 2023 at 14:50, Laurent Pinchart via libcamera-devel <libcamera-devel@lists.libcamera.org> wrote:
> On Fri, Dec 15, 2023 at 01:04:02PM +0100, Alexander Kanavin via libcamera-devel wrote:
>> That's right.
>>
>> This page provides hints for how to replace usage of imp with
>> equivalents in importlib:
>> https://docs.python.org/3.11/library/imp.html
>
> The trouble is that the code in libcamera that uses imp isn't ours, it's
> copied as-is from a third-party project. If we're lucky they will have
> fixed the issue, and we can just take the latest version.
If this is in mojo, seems that chromium fixed it:
https://chromium.googlesource.com/chromium/src/+/f5f6e361d037c31630661186e7bd7b31d2784cb8
>
>> On Fri, 15 Dec 2023 at 11:18, Kieran Bingham wrote:
>> > Quoting Kieran Bingham (2023-12-15 10:16:20)
>> > > +libcamera-devel mailing list
>> > >
>> > > Quoting Alexander Kanavin (2023-12-15 07:52:38)
>> > > > I'd note that imp module has been deprecated for years, so that's what
>> > > > happens when deprecation warnings are ignored.
>
> I haven't seen any warnings anywhere in the build process. Did I miss
> them ?
>
>> > > Can you tell me more about what this issue is please? I'm not aware of
>> > > it at all.
>> > >
>> > > > Signed-off-by: Alexander Kanavin <alex@linutronix.de>
>> > > > ---
>> > > > meta-multimedia/recipes-multimedia/libcamera/libcamera_0.1.0.bb | 1 +
>> > > > .../packagegroups/packagegroup-meta-multimedia.bb | 1 -
>> > > > meta-multimedia/recipes-multimedia/pipewire/pipewire_1.0.0.bb | 2 +-
>> > > > 3 files changed, 2 insertions(+), 2 deletions(-)
>> > > >
>> > > > diff --git a/meta-multimedia/recipes-multimedia/libcamera/libcamera_0.1.0.bb b/meta-multimedia/recipes-multimedia/libcamera/libcamera_0.1.0.bb
>> > > > index 2a3c6473e..9c04d7434 100644
>> > > > --- a/meta-multimedia/recipes-multimedia/libcamera/libcamera_0.1.0.bb
>> > > > +++ b/meta-multimedia/recipes-multimedia/libcamera/libcamera_0.1.0.bb
>> > > > @@ -73,3 +73,4 @@ FILES:${PN}-gst = "${libdir}/gstreamer-1.0"
>> > > > # both 32 and 64 bit file APIs.
>> > > > GLIBC_64BIT_TIME_FLAGS = ""
>> > > >
>> > > > +SKIP_RECIPE[libcamera] ?= "Not compatible with python 3.12, see https://bugs.libcamera.org/show_bug.cgi?id=206"
>> >
>> > Aha, the clue was buried here.
>> >
>> > > > diff --git a/meta-multimedia/recipes-multimedia/packagegroups/packagegroup-meta-multimedia.bb b/meta-multimedia/recipes-multimedia/packagegroups/packagegroup-meta-multimedia.bb
>> > > > index 672c8abc9..6ca4d539f 100644
>> > > > --- a/meta-multimedia/recipes-multimedia/packagegroups/packagegroup-meta-multimedia.bb
>> > > > +++ b/meta-multimedia/recipes-multimedia/packagegroups/packagegroup-meta-multimedia.bb
>> > > > @@ -42,7 +42,6 @@ RDEPENDS:packagegroup-meta-multimedia = "\
>> > > > tearsofsteel-1080p \
>> > > > pipewire \
>> > > > ${@bb.utils.contains("LICENSE_FLAGS_ACCEPTED", "commercial", bb.utils.contains("DISTRO_FEATURES", "x11", "projucer", "", d), "", d)} \
>> > > > - libcamera \
>> > > > vorbis-tools \
>> > > > libopenmpt \
>> > > > mimic \
>> > > > diff --git a/meta-multimedia/recipes-multimedia/pipewire/pipewire_1.0.0.bb b/meta-multimedia/recipes-multimedia/pipewire/pipewire_1.0.0.bb
>> > > > index b4da781f0..4d894349a 100644
>> > > > --- a/meta-multimedia/recipes-multimedia/pipewire/pipewire_1.0.0.bb
>> > > > +++ b/meta-multimedia/recipes-multimedia/pipewire/pipewire_1.0.0.bb
>> > > > @@ -86,7 +86,7 @@ PACKAGECONFIG:class-target ??= " \
>> > > > ${@bb.utils.contains('DISTRO_FEATURES', 'systemd', 'systemd systemd-system-service systemd-user-service', '', d)} \
>> > > > ${@bb.utils.filter('DISTRO_FEATURES', 'alsa vulkan pulseaudio', d)} \
>> > > > ${PIPEWIRE_SESSION_MANAGER} \
>> > > > - ${FFMPEG_AVAILABLE} avahi flatpak gstreamer gsettings jack libusb pw-cat raop sndfile v4l2 udev volume webrtc-echo-cancelling libcamera readline \
>> > > > + ${FFMPEG_AVAILABLE} avahi flatpak gstreamer gsettings jack libusb pw-cat raop sndfile v4l2 udev volume webrtc-echo-cancelling readline \
>> > > > "
>> > > >
>> > > > # "jack" and "pipewire-jack" packageconfigs cannot be both enabled,
>
> --
> Regards,
>
> Laurent Pinchart
next prev parent reply other threads:[~2023-12-15 13:53 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-12-15 7:52 [PATCH 01/20] python3-pyinotify: remove as unmaintained Alexander Kanavin
2023-12-15 7:52 ` [PATCH 02/20] python3-supervisor: do not rely on smtpd module Alexander Kanavin
2023-12-15 7:52 ` [PATCH 03/20] python3-meld3: " Alexander Kanavin
2023-12-15 7:52 ` [PATCH 04/20] python3-m2crypto: " Alexander Kanavin
2023-12-15 7:52 ` [PATCH 05/20] python3-uinput: remove as unmaintained Alexander Kanavin
2023-12-15 7:52 ` [PATCH 06/20] python3-mcrypto: rely on setuptools for distutils copy Alexander Kanavin
2023-12-15 7:52 ` [PATCH 07/20] python3-joblib: do not rely in distutils Alexander Kanavin
2023-12-15 7:52 ` [PATCH 08/20] python3-web3: remove distutils dependency Alexander Kanavin
2023-12-15 7:52 ` [PATCH 09/20] python3-cppy: remove unused " Alexander Kanavin
2023-12-15 7:52 ` [PATCH 10/20] python3-pyroute2: " Alexander Kanavin
2023-12-15 7:52 ` [PATCH 11/20] python3-eventlet: backport a patch to remove " Alexander Kanavin
2023-12-15 17:45 ` [oe] " Khem Raj
2023-12-15 7:52 ` [PATCH 12/20] python3-unoconv: rely on setuptools to obtain distutils copy Alexander Kanavin
2023-12-15 7:52 ` [PATCH 13/20] python3-astroid: remove unneeded distutils dependency Alexander Kanavin
2023-12-15 7:52 ` [PATCH 14/20] python3-django: " Alexander Kanavin
2023-12-15 7:52 ` [PATCH 15/20] python3-pillow: " Alexander Kanavin
2023-12-15 7:52 ` [PATCH 16/20] python3-grpcio: update 1.56.2 -> 1.59.3 Alexander Kanavin
2023-12-15 7:52 ` [PATCH 17/20] gstd: correctly delete files in do_install Alexander Kanavin
2023-12-15 7:52 ` [PATCH 18/20] libplist: fix python 3.12 compatibility Alexander Kanavin
2023-12-15 7:52 ` [PATCH 19/20] libcamera: skip until upstream resolves " Alexander Kanavin
2023-12-15 10:16 ` Kieran Bingham
2023-12-15 10:18 ` Kieran Bingham
2023-12-15 12:04 ` Alexander Kanavin
2023-12-15 12:50 ` [libcamera-devel] " Laurent Pinchart
2023-12-15 13:21 ` Mattijs Korpershoek [this message]
2023-12-15 13:27 ` Laurent Pinchart
2023-12-15 7:52 ` [PATCH 20/20] nodejs: backport (partially) python 3.12 support Alexander Kanavin
2023-12-20 19:12 ` [oe] " Khem Raj
2023-12-20 20:45 ` Alexander Kanavin
2023-12-21 0:17 ` Khem Raj
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=8734w3txce.fsf@baylibre.com \
--to=mkorpershoek@baylibre.com \
--cc=alex.kanavin@gmail.com \
--cc=laurent.pinchart@ideasonboard.com \
--cc=libcamera-devel@lists.libcamera.org \
--cc=openembedded-devel@lists.openembedded.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.