GNU bug report logs - #78188
[PATCH] Add KWallet service

Previous Next

Package: guix-patches;

Reported by: Sergio Pastor Pérez <sergio.pastorperez <at> gmail.com>

Date: Thu, 1 May 2025 13:52:02 UTC

Severity: normal

Tags: patch

Done: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>

Bug is archived. No further changes may be made.

Full log

Message #23 received at 78188 <at> debbugs.gnu.org (full text, mbox):

From: Sergio Pastor Pérez <sergio.pastorperez <at> gmail.com>
To: Z572 <z572 <at> z572.online>
Cc: 78188 <at> debbugs.gnu.org, Sughosha <sughosha <at> disroot.org>
Subject: Re: [bug#78188] [PATCH] gnu: kwallet-pam: Fix socat not in
 'libexec/pam_kwallet_init' wrapper path.
Date: Thu, 01 May 2025 19:56:01 +0200

Z572 <z572 <at> z572.online> writes:
> Sergio Pastor Pérez <sergio.pastorperez <at> gmail.com> writes:
>
>> * gnu/packages/kde-plasma.scm (kwallet-pam)[phase]: Wrapper phase not setting
>> PATH variable to access 'socat' input.
>>
>> Change-Id: I6e7a421af3dbbc844e319b7852246c89d979609c
>> ---
>>  gnu/packages/kde-plasma.scm | 14 +++++++++++---
>>  1 file changed, 11 insertions(+), 3 deletions(-)
>>
>> diff --git a/gnu/packages/kde-plasma.scm b/gnu/packages/kde-plasma.scm
>> index 679a68cef6..a15f7ad8a2 100644
>> --- a/gnu/packages/kde-plasma.scm
>> +++ b/gnu/packages/kde-plasma.scm
>> @@ -1303,9 +1303,17 @@ (define-public kwallet-pam
>>                  "0dif5y7qbayb2yfgl7940978ayyir948kpjavczvgkr70czb293k"))))
>>      (build-system qt-build-system)
>>      (arguments
>> -     (list
>> -      #:qtbase qtbase
>> -      #:tests? #f)) ;no tests
>> +     `(#:qtbase ,qtbase
>> +       #:tests? #f ;no tests
>> +       #:phases
>> +       (modify-phases %standard-phases
>
> please use gexp.
>
>> +         (add-after 'install 'wrap-program
>> +           (lambda* (#:key inputs outputs #:allow-other-keys)
>> +             (let ((socat (search-input-file inputs "bin/socat")))
>> +               (wrap-program (search-input-file outputs
>> +                              "libexec/pam_kwallet_init")
>> +                 `("PATH" ":" prefix
>> +                   ,(list (dirname socat))))))))))
>
> This is just a sh script, I prefer to replace socat in the
> pam_kwallet_init file after unpack

Done!
This bug report was last modified 70 days ago.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.