[bug#56859] [PATCH] gnu: xdg-desktop-portal-wlr: Change elogind to basu for sd-bus.

[bug#56859] [PATCH] gnu: xdg-desktop-portal-wlr: Change elogind to basu for sd-bus.

[muradm]

* gnu/packages/freedesktop.scm (xdg-desktop-portal-wlr):
[inputs]: Change elogind to basu.
[arguments]: Specify "-Dsd-bus-provider=basu" explicitly.
---
 gnu/packages/freedesktop.scm | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/gnu/packages/freedesktop.scm b/gnu/packages/freedesktop.scm
index cde7156383..1a30091da0 100644
--- a/gnu/packages/freedesktop.scm
+++ b/gnu/packages/freedesktop.scm
@@ -2408,7 +2408,7 @@ (define-public xdg-desktop-portal-wlr
     (arguments
      `(#:configure-flags
        '("-Dsystemd=disabled"
-         "-Dsd-bus-provider=libelogind")
+         "-Dsd-bus-provider=basu")
        #:phases
        (modify-phases %standard-phases
          (add-after 'unpack 'hardcode-binaries
@@ -2426,8 +2426,8 @@ (define-public xdg-desktop-portal-wlr
                   (string-append "execvp(\"" sh)))))))))
     (native-inputs
      (list cmake pkg-config))
-    (inputs (list elogind
-                  bash-minimal
+    (inputs (list bash-minimal
+                  basu
                   grim
                   iniparser
                   libinih
-- 
2.37.1

[bug#56859] [PATCH] gnu: xdg-desktop-portal-wlr: Change elogind to basu for sd-bus.

[Maxime Devos]

[-- Attachment #1.1.1: Type: text/plain, Size: 259 bytes --]


On 31-07-2022 17:50, muradm wrote:
> * gnu/packages/freedesktop.scm (xdg-desktop-portal-wlr):
> [inputs]: Change elogind to basu.
> [arguments]: Specify "-Dsd-bus-provider=basu" explicitly.

What's the reason for the switch?

Greetings,
Maxime



[-- Attachment #1.1.2: OpenPGP public key --]
[-- Type: application/pgp-keys, Size: 929 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 236 bytes --]

[bug#56859] [PATCH] gnu: xdg-desktop-portal-wlr: Change elogind to basu for sd-bus.

[( via Guix-patches via]

On Sun Jul 31, 2022 at 6:38 PM BST, Maxime Devos wrote:
> What's the reason for the switch?
Fewer dependencies (we include only the sd-bus library without the whole
of elogind), and it means eventually desktop users won't ever need elogind
to even exist in their store. (I think this is the reason. I'm not certain,
and there may be something more important, like some incompatibility with
seatd.)

    -- (

[bug#56859] [PATCH] gnu: xdg-desktop-portal-wlr: Change elogind to basu for sd-bus.

[Maxime Devos]

[-- Attachment #1.1.1: Type: text/plain, Size: 826 bytes --]


On 31-07-2022 23:45, ( wrote:
> On Sun Jul 31, 2022 at 6:38 PM BST, Maxime Devos wrote:
>> What's the reason for the switch?
> Fewer dependencies (we include only the sd-bus library without the whole
> of elogind), and it means eventually desktop users won't ever need elogind
> to even exist in their store. (I think this is the reason. I'm not certain,
> and there may be something more important, like some incompatibility with
> seatd.)
>
>      -- (

That's interesting but sounds very non-specific to 
xdg-desktop-portal-wlr; I think this would be better to propose on 
guix-devel@ to switch elogind->basu for _all_ dependents, not only 
xdg-desktop-portal-wlr -- the latter would increate the closure, not 
reduce it, because other installed packages would still use elogind.

Greetings,
Maxime.


[-- Attachment #1.1.2: OpenPGP public key --]
[-- Type: application/pgp-keys, Size: 929 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 236 bytes --]

[bug#56859] [PATCH] gnu: xdg-desktop-portal-wlr: Change elogind to basu for sd-bus.

[Maxime Devos]

[-- Attachment #1.1.1: Type: text/plain, Size: 990 bytes --]


On 01-08-2022 01:13, Maxime Devos wrote:
>
> On 31-07-2022 23:45, ( wrote:
>> On Sun Jul 31, 2022 at 6:38 PM BST, Maxime Devos wrote:
>>> What's the reason for the switch?
>> Fewer dependencies (we include only the sd-bus library without the whole
>> of elogind), and it means eventually desktop users won't ever need 
>> elogind
>> to even exist in their store. (I think this is the reason. I'm not 
>> certain,
>> and there may be something more important, like some incompatibility 
>> with
>> seatd.)
>>
>>      -- (
>
> That's interesting but sounds very non-specific to 
> xdg-desktop-portal-wlr; I think this would be better to propose on 
> guix-devel@ to switch elogind->basu for _all_ dependents, not only 
> xdg-desktop-portal-wlr -- the latter would increate the closure, not 
> reduce it, because other installed packages would still use elogind.

Additionally, the reason should have been mentioned in the commit message.

Greetings,
Maxime.


[-- Attachment #1.1.2: OpenPGP public key --]
[-- Type: application/pgp-keys, Size: 929 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 236 bytes --]

[bug#56859] [PATCH] gnu: xdg-desktop-portal-wlr: Change elogind to basu for sd-bus.

[muradm]

[-- Attachment #1: Type: text/plain, Size: 1765 bytes --]


Hi,

Regarding switching, I don't think it is very straight forward
to switch elogind with basu. Issue is that elogind "also" includes
sd-bus. Dependents may require either sd-bus and/or other parts
of elogind. Those who require sd-bus only to interface with bus
could be switched to basu gradually. Most likely this will be
suitable for "minimalist" applications.

I.e. basu is a drop-in replacement for those require sd-bus only,
but not whole elogind.

Thanks in advance,
muradm

Maxime Devos <maximedevos@telenet.be> writes:

> [[PGP Signed Part:Undecided]]
>
> On 01-08-2022 01:13, Maxime Devos wrote:
>>
>> On 31-07-2022 23:45, ( wrote:
>>> On Sun Jul 31, 2022 at 6:38 PM BST, Maxime Devos wrote:
>>>> What's the reason for the switch?
>>> Fewer dependencies (we include only the sd-bus library without 
>>> the whole
>>> of elogind), and it means eventually desktop users won't ever 
>>> need
>>> elogind
>>> to even exist in their store. (I think this is the reason. I'm 
>>> not
>>> certain,
>>> and there may be something more important, like some
>>> incompatibility with
>>> seatd.)
>>>
>>>      -- (
>>
>> That's interesting but sounds very non-specific to
>> xdg-desktop-portal-wlr; I think this would be better to propose 
>> on
>> guix-devel@ to switch elogind->basu for _all_ dependents, not 
>> only
>> xdg-desktop-portal-wlr -- the latter would increate the 
>> closure, not
>> reduce it, because other installed packages would still use 
>> elogind.
>
> Additionally, the reason should have been mentioned in the 
> commit message.
>
> Greetings,
> Maxime.
>
> [2. OpenPGP public key --- application/pgp-keys; 
> OpenPGP_0x49E3EE22191725EE.asc]...
>
> [[End of PGP Signed Part]]


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

[bug#56859] [PATCH v2] gnu: xdg-desktop-portal-wlr: Change elogind to basu for sd-bus.

[muradm]

* gnu/packages/freedesktop.scm (xdg-desktop-portal-wlr):
[inputs]: Change elogind to basu.
[arguments]: Specify "-Dsd-bus-provider=basu" explicitly.

xdg-desktop-portal-wlr requires only sd-bus which is now
available via basu as standalone.
---
 gnu/packages/freedesktop.scm | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/gnu/packages/freedesktop.scm b/gnu/packages/freedesktop.scm
index cde7156383..1a30091da0 100644
--- a/gnu/packages/freedesktop.scm
+++ b/gnu/packages/freedesktop.scm
@@ -2408,7 +2408,7 @@ (define-public xdg-desktop-portal-wlr
     (arguments
      `(#:configure-flags
        '("-Dsystemd=disabled"
-         "-Dsd-bus-provider=libelogind")
+         "-Dsd-bus-provider=basu")
        #:phases
        (modify-phases %standard-phases
          (add-after 'unpack 'hardcode-binaries
@@ -2426,8 +2426,8 @@ (define-public xdg-desktop-portal-wlr
                   (string-append "execvp(\"" sh)))))))))
     (native-inputs
      (list cmake pkg-config))
-    (inputs (list elogind
-                  bash-minimal
+    (inputs (list bash-minimal
+                  basu
                   grim
                   iniparser
                   libinih
-- 
2.37.1