guix-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug#65479] [PATCH core-updates v2 12/62] search-paths: Add $XML_CATALOG


From: Bruno Victal
Subject: [bug#65479] [PATCH core-updates v2 12/62] search-paths: Add $XML_CATALOG_FILES and $SGML_CATALOG_FILES.
Date: Mon, 25 Sep 2023 14:11:07 +0100
User-agent: Mozilla Thunderbird

Hi Simon,

On 2023-09-25 09:00, Simon Tournier wrote:

> And here that’s not the new example from the manual.  Somehow, the way
> using ’(search-path-specification (variable "XML_CATALOG_FILES") …)
> seems documented but not (list $XML_CATALOG_FILES).  And that is
> confusing me.

[…]

>> +(define $XML_CATALOG_FILES
>> +  ;; $XML_CATALOG_FILES lists 'catalog.xml' files found in under the 'xml'
>> +  ;; sub-directory of any given package.
>> +  (search-path-specification
>> +   (variable "XML_CATALOG_FILES")
>> +   (separator " ")
>> +   (files '("xml"))
>> +   (file-pattern "^catalog\\.xml$")
>> +   (file-type 'regular)))
> 
> Well, somehow the manual should provide an example using,
> 
>     (search-path-specification
>      (variable "SOMETHING_RELEVANT")
>      (separator " ")
>      (files '("foo"))
>      (file-pattern "^catalog\\.bar$")
>      (file-type 'regular))
> 
> instead of the one using XML_CATALOG_FILES, especially if this former is
> now replaced by the shorthand $XML_CATALOG_FILES.  Else, I am confused.

Right, I think I can see the potential for misunderstanding with the
updated doc section.

I'll address it in a v3 after giving some time for others to chime in
with the remainder of the series.

-- 
Furthermore, I consider that nonfree software must be eradicated.

Cheers,
Bruno.





reply via email to

[Prev in Thread] Current Thread [Next in Thread]