guix-patches
[Top][All Lists]
Advanced

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

[bug#65479] [PATCH core-updates 14/61] gnu: libxslt: Set search-paths fo


From: Maxim Cournoyer
Subject: [bug#65479] [PATCH core-updates 14/61] gnu: libxslt: Set search-paths for XML and SGML catalogs.
Date: Thu, 05 Oct 2023 19:59:48 -0400
User-agent: Gnus/5.13 (Gnus v5.13)

Hi,

Bruno Victal <mirai@makinata.eu> writes:

> According to xsltproc manpage it supports both XML_CATALOG_FILES and
> SGML_CATALOG_FILES for catalog lookup.
>
> * gnu/packages/xml.scm (libxslt)[native-search-paths]: Add SGML_CATALOG_FILES
> and XML_CATALOG_FILES.
> [search-paths]: Likewise.
> * gnu/packages/perl.scm (perl-app-xml-docbook-builder)[native-search-paths]:
> Refer to libxslt native-search-paths.
> ---
>  gnu/packages/perl.scm | 2 +-
>  gnu/packages/xml.scm  | 3 +++
>  2 files changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/gnu/packages/perl.scm b/gnu/packages/perl.scm
> index 7f9b19b6ee..5995aa0346 100644
> --- a/gnu/packages/perl.scm
> +++ b/gnu/packages/perl.scm
> @@ -602,7 +602,7 @@ (define-public perl-app-xml-docbook-builder
>       (list perl-class-xsaccessor perl-test-trap))
>      (native-search-paths
>       ;; xsltproc's search paths, to avoid propagating libxslt.
> -     (list $XML_CATALOG_FILES))
> +     (package-native-search-paths libxslt))

Because search path fields are not delayed, this is susceptible to
introduce circular dependencies between modules (see: "info (guix)
Cyclic Modules Dependencies").  It' be better to either repeat it, or
define some %libxslt-search-paths variable among the (guix search-paths)
module to be reused elsewhere.

-- 
Thanks,
Maxim





reply via email to

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