Subject: bug#36896: [PATCH] added gsettings-desktop-schema to progragated inputs

From: Martin Becze <[email-masked]>
Date: Mon, 05 Aug 2019 12:17:29 -0700

On 2019-08-05 07:40, Ricardo Wurmus wrote:
> Hi,
> 
> null <[email-masked]> writes:
> 
>> ---
>>  gnu/packages/gnome.scm | 2 ++
>>  1 file changed, 2 insertions(+)
>>
>> diff --git a/gnu/packages/gnome.scm b/gnu/packages/gnome.scm
>> index 3f7bbac214..c33653f361 100644
>> --- a/gnu/packages/gnome.scm
>> +++ b/gnu/packages/gnome.scm
>> @@ -8074,6 +8074,8 @@ generic enough to work for everyone.")
>>         ("openldap" ,openldap)
>>         ("webkitgtk" ,webkitgtk)
>>         ("ytnef" ,ytnef)))
>> +    (propagated-inputs
>> +       `(("gsettings-desktop-schemas" ,gsettings-desktop-schemas)))
>>      (home-page "https://gitlab.gnome.org/GNOME/evolution")
>>      (synopsis "Manage your email, contacts and schedule")
>>      (description "Evolution is a personal information management application
> 
> Thank you for the patch.
> 
> I’m not sure this is the right thing to do.  I don’t think we ever
> propagate this package.  Instead we often wrap executables so that
> relevant environment variables are set.  Would this work in the case of
> Evolution as well?

The terminator packagage propagates gsetting-desktop-schema as well
https://github.com/guix-mirror/guix/blob/8126142baf2cadd7b0052ba7b1d0cd248c8cbc40/gnu/packages/gnome.scm#L8159-L8161

but maybe its also doing the wrong thing? I'll look in to how to wrap
the executable and see if that will work.