diff mbox series

[bug#58484,1/1] gnu: home: home-dbus-service-type: Fix make-forkexec-constructor call.

Message ID 20221013064825.6731-1-paren@disroot.org
State New
Headers show
Series gnu: home: home-dbus-service-type: Fix make-forkexec-constructor call. | expand

Checks

Context Check Description
cbaines/applying patch fail
cbaines/issue success View issue
cbaines/applying patch fail
cbaines/issue success View issue
cbaines/applying patch fail
cbaines/issue success View issue
cbaines/applying patch fail View Laminar job
cbaines/issue success View issue

Commit Message

\( Oct. 13, 2022, 6:48 a.m. UTC
* gnu/home/services/desktop.scm (home-dbus-shepherd-services)[start]
  <#:environment-variables>: Use a quote instead of a gexp.
---
 gnu/home/services/desktop.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Andrew Tropin Oct. 14, 2022, 7:11 a.m. UTC | #1
On 2022-10-13 07:48, guix-patches@gnu.org wrote:

> * gnu/home/services/desktop.scm (home-dbus-shepherd-services)[start]
>   <#:environment-variables>: Use a quote instead of a gexp.
> ---

It's possible to type arbitrary text after ---, which won't be added to
commit message or affect a patch.  Just in case you would like to send
one email instead of two, this trick can be used.

>  gnu/home/services/desktop.scm | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/gnu/home/services/desktop.scm b/gnu/home/services/desktop.scm
> index 6549efd8df..e94e666d93 100644
> --- a/gnu/home/services/desktop.scm
> +++ b/gnu/home/services/desktop.scm
> @@ -201,7 +201,7 @@ (define (home-dbus-shepherd-services config)
>                                       (format #f "/run/user/~a"
>                                               (getuid)))))
>                     #:environment-variables
> -                   #~(list "DBUS_VERBOSE=1")
> +                   '(list "DBUS_VERBOSE=1")
>                     #:log-file
>                     (format #f "~a/dbus.log"
>                             (or (getenv "XDG_LOG_HOME")

Applied, pushed, thank you :)
\( Oct. 14, 2022, 7:12 a.m. UTC | #2
On Fri Oct 14, 2022 at 8:11 AM BST, Andrew Tropin wrote:
> Applied, pushed, thank you :)

Thanks! :)

    -- (
\( Oct. 14, 2022, 7:18 a.m. UTC | #3
On Fri Oct 14, 2022 at 8:11 AM BST, Andrew Tropin wrote:
> It's possible to type arbitrary text after ---, which won't be added to
> commit message or affect a patch.  Just in case you would like to send
> one email instead of two, this trick can be used.

I know, but I prefer to use the --cover-letter feature to generate the leading email :)

    -- (
diff mbox series

Patch

diff --git a/gnu/home/services/desktop.scm b/gnu/home/services/desktop.scm
index 6549efd8df..e94e666d93 100644
--- a/gnu/home/services/desktop.scm
+++ b/gnu/home/services/desktop.scm
@@ -201,7 +201,7 @@  (define (home-dbus-shepherd-services config)
                                      (format #f "/run/user/~a"
                                              (getuid)))))
                    #:environment-variables
-                   #~(list "DBUS_VERBOSE=1")
+                   '(list "DBUS_VERBOSE=1")
                    #:log-file
                    (format #f "~a/dbus.log"
                            (or (getenv "XDG_LOG_HOME")