From mboxrd@z Thu Jan 1 00:00:00 1970 Delivery-date: Wed, 16 Jul 2025 15:59:22 +0200 Received: from metis.whiteo.stw.pengutronix.de ([2a0a:edc0:2:b01:1d::104]) by lore.white.stw.pengutronix.de with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.96) (envelope-from ) id 1uc2fK-000Vib-1C for lore@lore.pengutronix.de; Wed, 16 Jul 2025 15:59:22 +0200 Received: from localhost ([127.0.0.1] helo=metis.whiteo.stw.pengutronix.de) by metis.whiteo.stw.pengutronix.de with esmtp (Exim 4.92) (envelope-from ) id 1uc2fK-00087u-6n; Wed, 16 Jul 2025 15:59:22 +0200 Received: from mx08-007fc201.pphosted.com ([91.207.212.40]) by metis.whiteo.stw.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1uc2ev-00087l-Q2 for ptxdist@pengutronix.de; Wed, 16 Jul 2025 15:58:58 +0200 Received: from pps.filterd (m0456229.ppops.net [127.0.0.1]) by mx08-007fc201.pphosted.com (8.18.1.8/8.18.1.8) with ESMTP id 56GDoKEY2625658; Wed, 16 Jul 2025 15:58:55 +0200 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cab.de; h=cc :content-id:content-transfer-encoding:content-type:date:from :in-reply-to:message-id:mime-version:references:subject:to; s= pp-2025; bh=id0rIY34Krea6FDiDhEeTX+Bd2FqwxFexhKCauJH0PE=; b=p2RV yJ0CK8pchuo66FT9GWvsT+sB6p8/Q3qP/e+qtOoHaxs9gaFOLnCgRCqUTKuhtPS2 CnCYMZPf/5N4J+33zGdt6RiN6+8oCSQ9cPvr6GhWjGxt3PdWkq+JJd3uoXejmjQG uS57/shOyaVpWjYUGbkUXX3iPcWGiFlsbWsamfW+42RpWF5V/NMMWD5KtQbyBgrS Ahq5JerXvph2c3X7w+FMPwAYQOKOLL9m8Z/uW5uiy01qkw33Wpl1oXY/pYcot1DT gtRkKkZ3W+dDkneZZziujOtL4PnbbM69x35CfQ4NCxkDbt7JanRLaLPNkM+mx/jD jXEPPZZ2vRkOxkEFig== Received: from adranos.cab.de (adranos.cab.de [46.232.229.107]) by mx08-007fc201.pphosted.com (PPS) with ESMTPS id 47wvtyrd3c-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Wed, 16 Jul 2025 15:58:54 +0200 (MEST) Received: from Adranos.cab.de (10.10.1.54) by Adranos.cab.de (10.10.1.54) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.1748.10; Wed, 16 Jul 2025 15:59:10 +0200 Received: from Adranos.cab.de ([fe80::9298:8fc8:395c:3859]) by Adranos.cab.de ([fe80::9298:8fc8:395c:3859%7]) with mapi id 15.02.1748.010; Wed, 16 Jul 2025 15:59:10 +0200 To: "ptxdist@pengutronix.de" Thread-Topic: [ptxdist] [PATCH v3 2/2] libgpiod: version bump 2.1.3 -> 2.2.2 Thread-Index: AQHb9ZBCG4MrYTkfwUaBb80TEbJYEbQ0pt+A Date: Wed, 16 Jul 2025 13:59:09 +0000 Message-ID: References: <20250715135327.753766-1-lapeddk@gmail.com> <20250715135327.753766-2-lapeddk@gmail.com> In-Reply-To: <20250715135327.753766-2-lapeddk@gmail.com> Accept-Language: de-DE, en-US Content-Language: de-DE X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.10.3.180] Content-Type: text/plain; charset="us-ascii" Content-ID: Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-Authority-Analysis: v=2.4 cv=ZIXXmW7b c=1 sm=1 tr=0 ts=6877b01e cx=c_pps a=LmW7qmVeM6tFdl5svFU9Cg==:117 a=LmW7qmVeM6tFdl5svFU9Cg==:17 a=xqWC_Br6kY4A:10 a=kldc_9v1VKEA:10 a=kj9zAlcOel0A:10 a=Wb1JkmetP80A:10 a=VwQbUJbxAAAA:8 a=pGLkceISAAAA:8 a=DV9jAgoGzot4URnZfo8A:9 a=CjuIK1q_8ugA:10 X-Proofpoint-Spam-Details-Enc: AW1haW4tMjUwNzE1MDE3NSBTYWx0ZWRfX0uClvIvbhGjf PiUUdgnCGF4MYbXuLu6DcWaxo2EjkTZ2ojtv2dS0loWDDOOPb/yvg/XKyNKk7qSYd0JfyDhOl8d XRTeHHO/xDywSSpTbqPtPj2IYva+Hh9vuzSM4yAWUg7JZ371YwFTIE/fT+zXElYFLKxh8w5W+GT ZzEDw3wvp2EmsyQTZo3NB4+/v7onG6HYVsXYECooV+5XNGhk56CkClNV4rJfIaArTvFJ8RuF0zh Kz1MODiSQOnWAf2RjU7gUJzITLMIzqqMOWCVe0G2eE8nhKxSWuDTyKg6H5HdEoN8K66wdmjfrwT Ww3Bt17bML6MY7zOuL/3ynIPA11QaC1trNkuakRH7+hRg77UvG5J1PFp+JjudevUPe8jte5XvLR HhPb3LDQ X-Proofpoint-GUID: Vz0aJORJ9fWqlfxCwC4_SizNYFrxg9dn X-Proofpoint-ORIG-GUID: Vz0aJORJ9fWqlfxCwC4_SizNYFrxg9dn X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on metis.whiteo.stw.pengutronix.de X-Spam-Level: X-Spam-Status: No, score=-2.6 required=4.0 tests=AWL,BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_VALIDITY_RPBL_BLOCKED, RCVD_IN_VALIDITY_SAFE_BLOCKED,SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.2 Subject: Re: [ptxdist] [PATCH v3 2/2] libgpiod: version bump 2.1.3 -> 2.2.2 X-BeenThere: ptxdist@pengutronix.de X-Mailman-Version: 2.1.29 Precedence: list List-Id: PTXdist Development Mailing List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , From: Markus Heidelberg via ptxdist Reply-To: ptxdist@pengutronix.de Cc: Markus Heidelberg , Lars Alex Pedersen Sender: "ptxdist" X-SA-Exim-Connect-IP: 127.0.0.1 X-SA-Exim-Mail-From: ptxdist-bounces@pengutronix.de X-SA-Exim-Scanned: No (on metis.whiteo.stw.pengutronix.de); SAEximRunCond expanded to false On Tue, Jul 15, 2025 at 03:53:27PM +0200, Lars Alex Pedersen wrote: > * Add new options to install the dbus daemon with and without systemd. > * Add gpio in group > * Add gpio-manager user and group > * Python binding now requires host pybuild and pip for build and > install, together with a patch to get the right build options > * Removed ac_cv_prog_has_python_config line in rule, since it has been > fixed in this release > * Configure options updated and reordered to match configure helper > * New option --disable-introspection is removed by autogen.sh >=20 > https://git.kernel.org/pub/scm/libs/libgpiod/libgpiod.git/tree/NEWS?h=3Dv= 2.2.2 >=20 > Signed-off-by: Lars Alex Pedersen > --- > ...1-Add-ptxdist-python-compile-options.patch | 22 ++++++++ > patches/libgpiod-2.2.2/autogen.sh | 1 + > patches/libgpiod-2.2.2/series | 4 ++ Python bindings can be built now without local adaption. > projectroot/etc/group | 2 + > projectroot/etc/passwd | 1 + > rules/libgpiod.in | 27 ++++++++-- > rules/libgpiod.make | 53 ++++++++++++++----- > 7 files changed, 94 insertions(+), 16 deletions(-) > create mode 100644 patches/libgpiod-2.2.2/0001-Add-ptxdist-python-compil= e-options.patch > create mode 120000 patches/libgpiod-2.2.2/autogen.sh > create mode 100644 patches/libgpiod-2.2.2/series > diff --git a/rules/libgpiod.make b/rules/libgpiod.make > index 6272333b5..6110130f2 100644 > --- a/rules/libgpiod.make > +++ b/rules/libgpiod.make > @@ -28,9 +28,18 @@ LIBGPIOD_LICENSE_FILES :=3D \ > file://LICENSES/GPL-2.0-only.txt;md5=3Db234ee4d69f5fce4486a80fdaf4a4263= \ > file://LICENSES/Linux-syscall-note.txt;md5=3D6b0dff741019b948dfe290c05d= 6f361c > =20 > +ifdef PTXCONF_LIBGPIOD_DBUS_DAEMON > +LIBGPIOD_LICENSE +=3D AND GPL-2.0-or-later AND CC-BY-SA-4.0 AND CC0-1.0 > +LIBGPIOD_LICENSE_FILES +=3D \ > + file://LICENSES/GPL-2.0-or-later.txt;md5=3Db234ee4d69f5fce4486a80fdaf4a= 4263 \ > + file://LICENSES/CC-BY-SA-4.0.txt;md5=3Dfba3b94d88bfb9b81369b869a1e9a20f= \ > + file://LICENSES/CC0-1.0.txt;md5=3D65d3616852dbf7b1a6d4b53b00626032 CC0-1.0 is only used for gpio-manager.service and 90-gpio.rules, so it should be dependent on PTXCONF_LIBGPIOD_DBUS_DAEMON_SYSTEMD. However, if installing rules files independently of systemd (see below), license selection would be correct. > -LIBGPIOD_TOOLS-$(PTXCONF_LIBGPIOD_GPIODETECT) +=3D gpiodetect > -LIBGPIOD_TOOLS-$(PTXCONF_LIBGPIOD_GPIOINFO) +=3D gpioinfo > -LIBGPIOD_TOOLS-$(PTXCONF_LIBGPIOD_GPIOGET) +=3D gpioget > -LIBGPIOD_TOOLS-$(PTXCONF_LIBGPIOD_GPIOSET) +=3D gpioset > -LIBGPIOD_TOOLS-$(PTXCONF_LIBGPIOD_GPIOMON) +=3D gpiomon > -LIBGPIOD_TOOLS-$(PTXCONF_LIBGPIOD_GPIONOTIFY) +=3D gpionotify > +LIBGPIOD_BINS-$(PTXCONF_LIBGPIOD_GPIODETECT) +=3D gpiodetect > +LIBGPIOD_BINS-$(PTXCONF_LIBGPIOD_GPIOINFO) +=3D gpioinfo > +LIBGPIOD_BINS-$(PTXCONF_LIBGPIOD_GPIOGET) +=3D gpioget > +LIBGPIOD_BINS-$(PTXCONF_LIBGPIOD_GPIOSET) +=3D gpioset > +LIBGPIOD_BINS-$(PTXCONF_LIBGPIOD_GPIOMON) +=3D gpiomon > +LIBGPIOD_BINS-$(PTXCONF_LIBGPIOD_GPIONOTIFY) +=3D gpionotify > +LIBGPIOD_BINS-$(PTXCONF_LIBGPIOD_DBUS_DAEMON) +=3D gpio-manager > +LIBGPIOD_BINS-$(PTXCONF_LIBGPIOD_DBUS_DAEMON) +=3D gpiocli > =20 > # ----------------------------------------------------------------------= ------ > # Target-Install > @@ -79,11 +93,24 @@ $(STATEDIR)/libgpiod.targetinstall: > =20 > @$(call install_lib, libgpiod, 0, 0, 0644, libgpiod) > =20 > - @for tool in $(LIBGPIOD_TOOLS-y); do \ > + @for tool in $(LIBGPIOD_BINS-y); do \ > $(call install_copy, libgpiod, 0, 0, 0755, -, \ > /usr/bin/$$tool); \ > done Good idea to rename LIBGPIOD_TOOLS-* to avoid confusion with the slightly unrelated PTXCONF_LIBGPIOD_TOOLS. > +ifdef PTXCONF_LIBGPIOD_DBUS_DAEMON > + @$(call install_tree, libgpiod, 0, 0, -, /usr/share/dbus-1/interfaces) > + @$(call install_tree, libgpiod, 0, 0, -, /etc/dbus-1/system.d) > + @$(call install_lib, libgpiod, 0, 0, 0644, libgpiodbus) > + @$(call install_lib, libgpiod, 0, 0, 0644, libgpiod-glib) > +endif > +ifdef PTXCONF_LIBGPIOD_DBUS_DAEMON_SYSTEMD > + @$(call install_tree, libgpiod, 0, 0, -, /usr/lib/udev/rules.d) Shouldn't udev rules (90-gpio.rules in this case) be installed independently of systemd? This would also be how nearly all existing packages containing udev rules files behave. Sorry for this late feedback. I only had a deeper look into the dbus part after adaption of the license information. So maybe v4 :) > + @$(call install_alternative, libgpiod, 0, 0, 0644, \ > + /usr/lib/systemd/system/gpio-manager.service) > + @$(call install_link, libgpiod, ../gpio-manager.service, \ > + /usr/lib/systemd/system/multi-user.target.wants/gpio-manager.service) > +endif Markus=