Subject: Re: [bug#39306] [PATCH] gnu: Add xsettingsd.

From: Marius Bakke <[email-masked]>
Date: Thu, 30 Jan 2020 13:14:59 +0100

David Wilson <[email-masked]> writes:

> * gnu/packages/wm.scm (xsettingsd): New variable.

I wonder if xdisorg.scm is better suited for this package, as it seems
mostly unrelated to window management.  WDYT?

[...]

> +       #:tests? #f ;; Tests require Google's gtest framework

gtest is provided by the 'googletest' package.  Can you try adding it?

Also, for margin comments, we only use one ';', without capitalizing the
comment.

Other than that LGTM.