Re: [Buildroot] [PATCH] package/sway: make systemd optional

From: Stefan Ott via buildroot <buildroot@buildroot.org>
To: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
Cc: Raphael Pavlidis <raphael.pavlidis@gmail.com>,
	Stefan Ott via buildroot <buildroot@buildroot.org>
Subject: Re: [Buildroot] [PATCH] package/sway: make systemd optional
Date: Thu, 18 Jan 2024 14:43:01 +0100	[thread overview]
Message-ID: <20240118134301.GA81291@mort> (raw)
In-Reply-To: <20240118135820.6fcfebb5@windsurf>

[-- Attachment #1.1: Type: text/plain, Size: 998 bytes --]

Hi!

On Thu, Jan 18, 2024 at 01:58:20PM +0100, Thomas Petazzoni wrote:
> Thanks for your patch! I am wondering if adding an option for
> this is needed. Can't we just enable systemd support when
> BR2_PACKAGE_SYSTEMD=y?

That would indeed seem to be a more reasonable approach. The
reason I chose to turn it into an option was to make the user
aware of the fact that there is such an option at all and that
they might get additional features when using systemd.

Whether this is relevant or not, I cannot say. But if you think
it would be acceptable to do it automatically, I'll update the
patch accordingly.

> Is there another sd-bus-provider option value that should be
> set when systemd is not available?

To be quite honest, I can't tell you what benefits you get from
this in combination with systemd or any of the other options. All
I know is that in my case sway does work with the default value
(which is 'auto').

cheers
-- 
Stefan Ott
https://www.ott.net/

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

[-- Attachment #2: Type: text/plain, Size: 150 bytes --]

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot