From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from ptx.hi.pengutronix.de ([2001:67c:670:100:1d::c0]) by metis.ext.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1jxVOI-0002YB-7E for ptxdist@pengutronix.de; Mon, 20 Jul 2020 14:59:34 +0200 Received: from mol by ptx.hi.pengutronix.de with local (Exim 4.92) (envelope-from ) id 1jxVOH-0000bj-U7 for ptxdist@pengutronix.de; Mon, 20 Jul 2020 14:59:33 +0200 Date: Mon, 20 Jul 2020 14:59:33 +0200 From: Michael Olbrich Message-ID: <20200720125933.GA7949@pengutronix.de> References: <20200720115117.21293-1-rhi@pengutronix.de> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20200720115117.21293-1-rhi@pengutronix.de> Subject: Re: [ptxdist] [PATCH 1/3] gpsd: merge GPSD_SYSTEMD and GPSD_SYSTEMD_UNIT options List-Id: PTXdist Development Mailing List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: ptxdist@pengutronix.de Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ptxdist-bounces@pengutronix.de Sender: "ptxdist" To: ptxdist@pengutronix.de On Mon, Jul 20, 2020 at 01:51:15PM +0200, Roland Hieber wrote: > The current systemd socket unit relies on compiled-in systemd support in > the gpsd binary, so we would need to enable GPSD_SYSTEMD with > GPSD_SYSTEMD_UNIT anyways. Vice versa, there is no use in having > compiled-in systemd socket activation support without the systemd socket > unit. Merge both options. It can be useful to disable systemd units but not the support. E.g. if the service should not be enabled by default. I'd prefer remove the prompt for GPSD_SYSTEMD instead and enable it with systemd (like rsyslog handles this, for example). Michael > Signed-off-by: Roland Hieber > --- > rules/gpsd.in | 9 ++------- > rules/gpsd.make | 2 +- > scripts/migrate/migrate_ptx | 8 ++++++++ > 3 files changed, 11 insertions(+), 8 deletions(-) > > diff --git a/rules/gpsd.in b/rules/gpsd.in > index 16b6dcb70a72..d01ce4291ac9 100644 > --- a/rules/gpsd.in > +++ b/rules/gpsd.in > @@ -63,11 +63,6 @@ config GPSD_USB > > comment "Other options ---" > > -config GPSD_SYSTEMD > - bool > - depends on SYSTEMD > - prompt "systemd" > - > config GPSD_RECONFIGURE > bool > prompt "enable reconfigure" > @@ -260,7 +255,7 @@ menu "install options" > queries with a format that is substantially easier to > parse than the NMEA 0183 emitted by most GPS receivers. > > - config GPSD_SYSTEMD_UNIT > + config GPSD_SYSTEMD > bool > default y > depends on INITMETHOD_SYSTEMD && GPSD_GPSD > @@ -268,7 +263,7 @@ menu "install options" > > config GPSD_GPSD_ARGS > string > - depends on GPSD_SYSTEMD_UNIT > + depends on GPSD_SYSTEMD > prompt "gpsd arguments" > > config GPSD_GPS2UDP > diff --git a/rules/gpsd.make b/rules/gpsd.make > index f75caa5b4bbd..16a407cb505e 100644 > --- a/rules/gpsd.make > +++ b/rules/gpsd.make > @@ -160,7 +160,7 @@ $(STATEDIR)/gpsd.targetinstall: > ifdef PTXCONF_GPSD_GPSD > @$(call install_copy, gpsd, 0, 0, 0755, -, /usr/sbin/gpsd) > endif > -ifdef PTXCONF_GPSD_SYSTEMD_UNIT > +ifdef PTXCONF_GPSD_SYSTEMD > @$(call install_alternative, gpsd, 0, 0, 644, \ > /usr/lib/systemd/system/gpsd.service) > @$(call install_replace, gpsd, \ > diff --git a/scripts/migrate/migrate_ptx b/scripts/migrate/migrate_ptx > index 34c9a5ccedf8..484715cfeffe 100755 > --- a/scripts/migrate/migrate_ptx > +++ b/scripts/migrate/migrate_ptx > @@ -402,3 +402,11 @@ s/^\(\(# \)\?PTXCONF_GST_PLUGINS_\)BAD1_STEREO\>/\1GOOD1_AUDIOFX/ > # reason : option was renamed > # > s/^\(\(# \)\?PTXCONF_SQLITE_LOAD_\)EXTENTION/\1EXTENSION/ > + > +# > +# from : ptxdist-2020.07.0 > +# to : ptxdist-2020.08.0 > +# symbol : GPSD_SYSTEMD_UNIT -> GPSD_SYSTEMD > +# reason : options were renamed / merged > +# > +s/^\(\(# \)\?PTXCONF_GPSD_SYSTEMD\)_UNIT\>/\1/ > -- > 2.27.0 > > > _______________________________________________ > ptxdist mailing list > ptxdist@pengutronix.de > To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ ptxdist mailing list ptxdist@pengutronix.de To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de