From: "Miguel Ángel Moreno" <me@mianmoreno.com>
To: 62806@debbugs.gnu.org
Subject: [bug#62806] [PATCH] gnu: home: services: fontutils: Add support for SXML fragments.
Date: Thu, 18 May 2023 11:11:40 +0200 [thread overview]
Message-ID: <86o7mihulv.fsf@mianmoreno.com> (raw)
In-Reply-To: <87a5yajbe8.fsf@trop.in> (Andrew Tropin's message of "Fri, 12 May 2023 10:45:35 +0400")
On 2023-05-12 10:45, Andrew Tropin wrote:
> On 2023-05-11 14:34, Ludovic Courtès wrote:
>
>> Hi Andrew,
>>
>> Sorry for the delay and thanks for your feedback.
>>
>> Andrew Patterson <andrewpatt7@gmail.com> skribis:
>>
>>> It seems to me that the main options are:
>>> 1) Just use my patch, or
>>> 2) write a new patch with an actual configuration record type,
>>> based on conses and Taiju HAGASHI's patches, either with
>>> a) a single font family for the default font family settings,
>>> b) a list of font families for the default font families, or
>>> c) allowing either.
>>
>> I went with #1, but that doesn’t prevent us from considering #2 as an
>> improvement going forward. Conses, Andrews: you’re welcome to share
>> your thoughts!
>
> The implementation LGTM and probably it covers most of use cases
> mentioned on the related threads, however, I'm a little bit rusty on
> this topic and can't say it for sure :)
>
Yeah, I think it's not as intrusive as Taiju's or mine.
> I think Miguel (conses) have more to say on this, because AFAIR he
> recently did something related to fontconfig configuration.
>
Not really, since this is more of a general approach, we can add the
font records and default families as part of RDE for those who want a
more batteries-included approach.
>>
>>> If we don't want to just use my patch, I can work on a new patch
>>> with a configuration record. How do you print a deprecation
>>> warning?
>>
>> The (guix deprecation) module provides facilities for that. If that’s
>> too restrictive, use ‘warning’ from (guix diagnostics).
>>
>> Thanks!
>>
>> Ludo’.
--
Best regards,
Miguel Ángel Moreno
prev parent reply other threads:[~2023-05-18 13:33 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-04-13 3:40 [bug#62806] [PATCH] gnu: home: services: fontutils: Add support for SXML fragments Andrew Patterson
2023-04-24 20:41 ` Ludovic Courtès
2023-04-24 23:24 ` Andrew Patterson
2023-05-11 12:34 ` Ludovic Courtès
2023-05-12 6:45 ` Andrew Tropin
2023-05-18 9:11 ` Miguel Ángel Moreno [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=86o7mihulv.fsf@mianmoreno.com \
--to=me@mianmoreno.com \
--cc=62806@debbugs.gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/guix.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.