all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [PATCH 1/6] gnu: docbook-xsl update to 1.78.1
@ 2014-07-15 18:31 John Darrington
  2014-07-15 18:31 ` [PATCH 2/6] gnu: Add docbook-xml version 4.3 and 4.4 John Darrington
                   ` (5 more replies)
  0 siblings, 6 replies; 21+ messages in thread
From: John Darrington @ 2014-07-15 18:31 UTC (permalink / raw)
  To: guix-devel; +Cc: John Darrington

* gnu/packages/docbook.scm (docbook-xsl): Upgrade to version 1.78.1
---
 gnu/packages/docbook.scm |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/docbook.scm b/gnu/packages/docbook.scm
index 29fa409..6d380ed 100644
--- a/gnu/packages/docbook.scm
+++ b/gnu/packages/docbook.scm
@@ -64,14 +64,14 @@ by no means limited to these applications.)  This package provides XML DTDs.")
 (define-public docbook-xsl
   (package
     (name "docbook-xsl")
-    (version "1.72.0")
+    (version "1.78.1")
     (source (origin
               (method url-fetch)
               (uri (string-append "mirror://sourceforge/docbook/docbook-xsl-"
                                   version ".tar.bz2"))
               (sha256
                (base32
-                "1cnrfgqz8pc9wnlgqjch2338ad7jki6d4h6b2fhaxn1a2201df5k"))))
+                "0rxl013ncmz1n6ymk2idvx3hix9pdabk8xn01cpcv32wmfb753y9"))))
     (build-system trivial-build-system)
     (arguments
      `(#:builder (begin
-- 
1.7.10.4

^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 2/6] gnu: Add docbook-xml version 4.3 and 4.4
  2014-07-15 18:31 [PATCH 1/6] gnu: docbook-xsl update to 1.78.1 John Darrington
@ 2014-07-15 18:31 ` John Darrington
  2014-07-15 20:49   ` Ludovic Courtès
  2014-07-15 18:31 ` [PATCH 3/6] gnu: docbook-xml: Substite remote xml_catalog uris with local ones John Darrington
                   ` (4 subsequent siblings)
  5 siblings, 1 reply; 21+ messages in thread
From: John Darrington @ 2014-07-15 18:31 UTC (permalink / raw)
  To: guix-devel; +Cc: John Darrington

* gnu/packages/docbook.scm (docbook-xml-4.3, docbook-xml-4.4): New variables.
---
 gnu/packages/docbook.scm |   22 ++++++++++++++++++++++
 1 file changed, 22 insertions(+)

diff --git a/gnu/packages/docbook.scm b/gnu/packages/docbook.scm
index 6d380ed..1a48d97 100644
--- a/gnu/packages/docbook.scm
+++ b/gnu/packages/docbook.scm
@@ -61,6 +61,28 @@ suited to books and papers about computer hardware and software (though it is
 by no means limited to these applications.)  This package provides XML DTDs.")
     (license (x11-style "" "See file headers."))))
 
+(define-public docbook-xml-4.4
+  (package (inherit docbook-xml)
+   (version "4.4")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append "http://www.docbook.org/xml/" version
+                                  "/docbook-xml-" version ".zip"))
+              (sha256
+               (base32
+                "141h4zsyc71sfi2zzd89v4bb4qqq9ca1ri9ix2als9f4i3mmkw82"))))))
+
+(define-public docbook-xml-4.3
+  (package (inherit docbook-xml)
+   (version "4.3")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append "http://www.docbook.org/xml/" version
+                                  "/docbook-xml-" version ".zip"))
+              (sha256
+               (base32
+                "0r1l2if1z4wm2v664sqdizm4gak6db1kx9y50jq89m3gxaa8l1i3"))))))
+
 (define-public docbook-xsl
   (package
     (name "docbook-xsl")
-- 
1.7.10.4

^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 3/6] gnu: docbook-xml: Substite remote xml_catalog uris with local ones.
  2014-07-15 18:31 [PATCH 1/6] gnu: docbook-xsl update to 1.78.1 John Darrington
  2014-07-15 18:31 ` [PATCH 2/6] gnu: Add docbook-xml version 4.3 and 4.4 John Darrington
@ 2014-07-15 18:31 ` John Darrington
  2014-07-15 20:51   ` Ludovic Courtès
  2014-07-15 18:31 ` [PATCH 4/6] gnu: docbook-xsl write absolute file:// uris in catalogs instead of ./ relative ones John Darrington
                   ` (3 subsequent siblings)
  5 siblings, 1 reply; 21+ messages in thread
From: John Darrington @ 2014-07-15 18:31 UTC (permalink / raw)
  To: guix-devel; +Cc: John Darrington

* gnu/packages/docbook.scm: substitute remote uri with local ones.
---
 gnu/packages/docbook.scm |    8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/docbook.scm b/gnu/packages/docbook.scm
index 1a48d97..431da7f 100644
--- a/gnu/packages/docbook.scm
+++ b/gnu/packages/docbook.scm
@@ -50,8 +50,12 @@
                           (dtd    (string-append out "/xml/dtd/docbook")))
                      (mkdir-p dtd)
                      (with-directory-excursion dtd
-                       (system* unzip source))))
-       #:modules ((guix build utils))))
+                       (system* unzip source))
+                     (substitute* (string-append out "/xml/dtd/docbook/catalog.xml")
+                       (("uri=\"") 
+                        (string-append 
+                         "uri=\"file://" dtd "/")))))
+                 #:modules ((guix build utils))))
     (native-inputs `(("unzip" ,unzip)))
     (home-page "http://docbook.org")
     (synopsis "DocBook XML DTDs for document authoring")
-- 
1.7.10.4

^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 4/6] gnu: docbook-xsl write absolute file:// uris in catalogs instead of ./ relative ones
  2014-07-15 18:31 [PATCH 1/6] gnu: docbook-xsl update to 1.78.1 John Darrington
  2014-07-15 18:31 ` [PATCH 2/6] gnu: Add docbook-xml version 4.3 and 4.4 John Darrington
  2014-07-15 18:31 ` [PATCH 3/6] gnu: docbook-xml: Substite remote xml_catalog uris with local ones John Darrington
@ 2014-07-15 18:31 ` John Darrington
  2014-07-15 20:51   ` Ludovic Courtès
  2014-07-15 18:31 ` [PATCH 5/6] gnu: Enable gnome-doc-utils tests John Darrington
                   ` (2 subsequent siblings)
  5 siblings, 1 reply; 21+ messages in thread
From: John Darrington @ 2014-07-15 18:31 UTC (permalink / raw)
  To: guix-devel; +Cc: John Darrington

* gnu/packages/docbook.scm (docbook-xsl):  Replace ./ paths with file:// uris
---
 gnu/packages/docbook.scm |   13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/gnu/packages/docbook.scm b/gnu/packages/docbook.scm
index 431da7f..c689c9c 100644
--- a/gnu/packages/docbook.scm
+++ b/gnu/packages/docbook.scm
@@ -100,7 +100,7 @@ by no means limited to these applications.)  This package provides XML DTDs.")
                 "0rxl013ncmz1n6ymk2idvx3hix9pdabk8xn01cpcv32wmfb753y9"))))
     (build-system trivial-build-system)
     (arguments
-     `(#:builder (begin
+     `(#:builder (let ((name-version (string-append ,name "-" ,version)))
                    (use-modules (guix build utils))
 
                    (let* ((bzip2  (assoc-ref %build-inputs "bzip2"))
@@ -112,10 +112,13 @@ by no means limited to these applications.)  This package provides XML DTDs.")
                      (system* (string-append tar "/bin/tar") "xvf" source)
 
                      (mkdir-p xsl)
-                     (copy-recursively (string-append ,name "-" ,version)
-                                       (string-append xsl "/" ,name
-                                                      "-" ,version))))
-       #:modules ((guix build utils))))
+                     (copy-recursively name-version
+                                       (string-append xsl "/" name-version))
+
+                     (substitute* (string-append xsl "/" name-version "/catalog.xml")
+                       (("rewritePrefix=\"./") 
+                        (string-append "rewritePrefix=\"file://" xsl "/" name-version "/")))))
+                 #:modules ((guix build utils))))
     (native-inputs `(("bzip2" ,bzip2)
                      ("tar" ,tar)))
     (home-page "http://docbook.org")
-- 
1.7.10.4

^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 5/6] gnu: Enable gnome-doc-utils tests
  2014-07-15 18:31 [PATCH 1/6] gnu: docbook-xsl update to 1.78.1 John Darrington
                   ` (2 preceding siblings ...)
  2014-07-15 18:31 ` [PATCH 4/6] gnu: docbook-xsl write absolute file:// uris in catalogs instead of ./ relative ones John Darrington
@ 2014-07-15 18:31 ` John Darrington
  2014-07-15 20:59   ` Ludovic Courtès
  2014-07-15 18:31 ` [PATCH 6/6] gnu: Tinyproxy: Cleanup xml dependencies John Darrington
  2014-07-15 20:49 ` [PATCH 1/6] gnu: docbook-xsl update to 1.78.1 Ludovic Courtès
  5 siblings, 1 reply; 21+ messages in thread
From: John Darrington @ 2014-07-15 18:31 UTC (permalink / raw)
  To: guix-devel; +Cc: John Darrington

* gnu/packages/gnome (gnome-doc-utils): Set the XML_CATALOG_FILES variable and
  enable the tests.
---
 gnu/packages/gnome.scm |   15 +++++++++++++--
 1 file changed, 13 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/gnome.scm b/gnu/packages/gnome.scm
index 6992116..3811fa6 100644
--- a/gnu/packages/gnome.scm
+++ b/gnu/packages/gnome.scm
@@ -26,6 +26,7 @@
   #:use-module (gnu packages)
   #:use-module (gnu packages bison)
   #:use-module (gnu packages flex)  
+  #:use-module (gnu packages docbook)
   #:use-module (gnu packages glib)
   #:use-module (gnu packages gnupg)
   #:use-module (gnu packages gstreamer)
@@ -137,14 +138,24 @@ The gnome-about program helps find which version of GNOME is installed.")
        (base32
         "19n4x25ndzngaciiyd8dd6s2mf9gv6nv3wv27ggns2smm7zkj1nb"))))
     (build-system gnu-build-system)
+    (arguments
+     `(#:phases
+       (alist-cons-before
+        'check 'pre-check
+        (lambda* (#:key inputs #:allow-other-keys #:rest args)
+          ;; This is needed, because without it, xmlint etc tries
+          ;; to download docbookx.dtd from the net
+          (setenv "XML_CATALOG_FILES" 
+                  (string-append (assoc-ref inputs "docbook-xml") 
+                                 "/xml/dtd/docbook/catalog.xml")))
+        %standard-phases)))
     (native-inputs
      `(("intltool" ,intltool)
+       ("docbook-xml" ,docbook-xml-4.4)
        ("libxml2" ,libxml2)
        ("libxslt" ,libxslt)
        ("pkg-config" ,pkg-config)
        ("python-2" ,python-2)))
-    (arguments
-     `(#:tests? #f)) ; tries to load http://www.oasis-open.org/docbook/xml/4.4/docbookx.dtd
     (home-page "https://wiki.gnome.org/GnomeDocUtils")
     (synopsis
      "Documentation utilities for the Gnome project")
-- 
1.7.10.4

^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 6/6] gnu: Tinyproxy: Cleanup xml dependencies
  2014-07-15 18:31 [PATCH 1/6] gnu: docbook-xsl update to 1.78.1 John Darrington
                   ` (3 preceding siblings ...)
  2014-07-15 18:31 ` [PATCH 5/6] gnu: Enable gnome-doc-utils tests John Darrington
@ 2014-07-15 18:31 ` John Darrington
  2014-07-15 21:00   ` Ludovic Courtès
  2014-07-15 20:49 ` [PATCH 1/6] gnu: docbook-xsl update to 1.78.1 Ludovic Courtès
  5 siblings, 1 reply; 21+ messages in thread
From: John Darrington @ 2014-07-15 18:31 UTC (permalink / raw)
  To: guix-devel; +Cc: John Darrington

* gnu/packages/web.scm (tinyproxy): Use the xml_catalog from docbook instead of
  generating a temporary one.
---
 gnu/packages/web.scm |   28 +++++++++-------------------
 1 file changed, 9 insertions(+), 19 deletions(-)

diff --git a/gnu/packages/web.scm b/gnu/packages/web.scm
index 6e94953..24886f9 100644
--- a/gnu/packages/web.scm
+++ b/gnu/packages/web.scm
@@ -528,25 +528,15 @@ help you implement simple HTTP servers.")
        (alist-cons-before
         'build 'pre-build
         (lambda* (#:key inputs #:allow-other-keys #:rest args)
-          ;; This stuff is needed, because without it, xmlint etc tries
-          ;; to download docbookx.dtd and docbook.xsl from the net
-          (let ((build (assoc-ref %standard-phases 'build))
-                (docbook-xml (assoc-ref inputs "docbook-xml"))
-                (docbook-xsl (assoc-ref inputs "docbook-xsl"))
-                (our-catalog "/tmp/docbook-xml.xml"))
-            (setenv "XML_CATALOG_FILES" our-catalog)
-            (with-output-to-file our-catalog
-              (lambda ()
-                (display (string-append
-                          "<?xml version=\"1.0\"?>
-<!DOCTYPE catalog PUBLIC \"-//OASIS//DTD XML Catalogs V1.0//EN\"
-\"file:///usr/share/xml/schema/xml-core/catalog.dtd\">
-<catalog xmlns=\"urn:oasis:names:tc:entity:xmlns:xml:catalog\">
-<system systemId=\"http://www.oasis-open.org/docbook/xml/4.5/docbookx.dtd\"
-uri=\"file://" docbook-xml  "/xml/dtd/docbook/docbookx.dtd\"/>
-<system systemId=\"http://docbook.sourceforge.net/release/xsl/current/manpages/docbook.xsl\"
-uri=\"file://" docbook-xsl  "/xml/xsl/docbook-xsl-1.72.0/manpages/docbook.xsl\"/>
-</catalog>\n"))))))
+          ;; Uncommenting the next two lines may assist in debugging
+          ;; (substitute* "docs/man5/Makefile" (("a2x") "a2x -v"))
+          ;; (setenv "XML_DEBUG_CATALOG" "1")
+
+          (setenv "XML_CATALOG_FILES" 
+                  (string-append
+                   (assoc-ref inputs "docbook-xsl") "/xml/xsl/docbook-xsl-1.78.1/catalog.xml"
+                   " "  ; This must be a space, not a : (contrary to the documentation)
+                   (assoc-ref inputs "docbook-xml") "/xml/dtd/docbook/catalog.xml")))
         %standard-phases)))
     ;; All of the below are used to generate the documentation
     ;; (Should they be propagated inputs of asciidoc ??)
-- 
1.7.10.4

^ permalink raw reply related	[flat|nested] 21+ messages in thread

* Re: [PATCH 1/6] gnu: docbook-xsl update to 1.78.1
  2014-07-15 18:31 [PATCH 1/6] gnu: docbook-xsl update to 1.78.1 John Darrington
                   ` (4 preceding siblings ...)
  2014-07-15 18:31 ` [PATCH 6/6] gnu: Tinyproxy: Cleanup xml dependencies John Darrington
@ 2014-07-15 20:49 ` Ludovic Courtès
  5 siblings, 0 replies; 21+ messages in thread
From: Ludovic Courtès @ 2014-07-15 20:49 UTC (permalink / raw)
  To: John Darrington; +Cc: guix-devel

John Darrington <jmd@gnu.org> skribis:

> * gnu/packages/docbook.scm (docbook-xsl): Upgrade to version 1.78.1

OK to push.

Ludo’.

^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [PATCH 2/6] gnu: Add docbook-xml version 4.3 and 4.4
  2014-07-15 18:31 ` [PATCH 2/6] gnu: Add docbook-xml version 4.3 and 4.4 John Darrington
@ 2014-07-15 20:49   ` Ludovic Courtès
  0 siblings, 0 replies; 21+ messages in thread
From: Ludovic Courtès @ 2014-07-15 20:49 UTC (permalink / raw)
  To: John Darrington; +Cc: guix-devel

John Darrington <jmd@gnu.org> skribis:

> * gnu/packages/docbook.scm (docbook-xml-4.3, docbook-xml-4.4): New variables.

OK!

Ludo’.

^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [PATCH 3/6] gnu: docbook-xml: Substite remote xml_catalog uris with local ones.
  2014-07-15 18:31 ` [PATCH 3/6] gnu: docbook-xml: Substite remote xml_catalog uris with local ones John Darrington
@ 2014-07-15 20:51   ` Ludovic Courtès
  0 siblings, 0 replies; 21+ messages in thread
From: Ludovic Courtès @ 2014-07-15 20:51 UTC (permalink / raw)
  To: John Darrington; +Cc: guix-devel

John Darrington <jmd@gnu.org> skribis:

> * gnu/packages/docbook.scm: substitute remote uri with local ones.

Typo in subject line, and please add the variable name in the log.
OK to push with that, thanks!

Ludo’.

^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [PATCH 4/6] gnu: docbook-xsl write absolute file:// uris in catalogs instead of ./ relative ones
  2014-07-15 18:31 ` [PATCH 4/6] gnu: docbook-xsl write absolute file:// uris in catalogs instead of ./ relative ones John Darrington
@ 2014-07-15 20:51   ` Ludovic Courtès
  0 siblings, 0 replies; 21+ messages in thread
From: Ludovic Courtès @ 2014-07-15 20:51 UTC (permalink / raw)
  To: John Darrington; +Cc: guix-devel

John Darrington <jmd@gnu.org> skribis:

> * gnu/packages/docbook.scm (docbook-xsl):  Replace ./ paths with file:// uris

OK.

^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [PATCH 5/6] gnu: Enable gnome-doc-utils tests
  2014-07-15 18:31 ` [PATCH 5/6] gnu: Enable gnome-doc-utils tests John Darrington
@ 2014-07-15 20:59   ` Ludovic Courtès
  2014-07-15 21:43     ` John Darrington
  0 siblings, 1 reply; 21+ messages in thread
From: Ludovic Courtès @ 2014-07-15 20:59 UTC (permalink / raw)
  To: John Darrington; +Cc: guix-devel

John Darrington <jmd@gnu.org> skribis:

> * gnu/packages/gnome (gnome-doc-utils): Set the XML_CATALOG_FILES variable and
>   enable the tests.

OK to push for now, but:

> +    (arguments
> +     `(#:phases
> +       (alist-cons-before
> +        'check 'pre-check
> +        (lambda* (#:key inputs #:allow-other-keys #:rest args)
> +          ;; This is needed, because without it, xmlint etc tries
> +          ;; to download docbookx.dtd from the net
> +          (setenv "XML_CATALOG_FILES" 
> +                  (string-append (assoc-ref inputs "docbook-xml") 
> +                                 "/xml/dtd/docbook/catalog.xml")))
> +        %standard-phases)))
>      (native-inputs
>       `(("intltool" ,intltool)
> +       ("docbook-xml" ,docbook-xml-4.4)

We can’t copy that around forever.

The problem is that ‘search-path-specification’ is meant for $PATH-like
variables that list directories, not files.

So I see two solutions:

  1. Patch libxml2 so that it honors a new variable, say
     $XML_CATALOG_DIRECTORIES, which would allow us to use
     ‘search-path-specification’.

  2. Augment support for search paths to allow file-based search paths.

(2) may be best in the long run, but it has ramifications in different
places.

Thoughts?

Ludo’.

^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [PATCH 6/6] gnu: Tinyproxy: Cleanup xml dependencies
  2014-07-15 18:31 ` [PATCH 6/6] gnu: Tinyproxy: Cleanup xml dependencies John Darrington
@ 2014-07-15 21:00   ` Ludovic Courtès
  2014-07-15 21:11     ` John Darrington
  0 siblings, 1 reply; 21+ messages in thread
From: Ludovic Courtès @ 2014-07-15 21:00 UTC (permalink / raw)
  To: John Darrington; +Cc: guix-devel

John Darrington <jmd@gnu.org> skribis:

> * gnu/packages/web.scm (tinyproxy): Use the xml_catalog from docbook instead of
>   generating a temporary one.

Limit lines to 80 chars please.

Also, “Use the XML catalog” no?

> +          (setenv "XML_CATALOG_FILES" 
> +                  (string-append
> +                   (assoc-ref inputs "docbook-xsl") "/xml/xsl/docbook-xsl-1.78.1/catalog.xml"
> +                   " "  ; This must be a space, not a : (contrary to the documentation)
> +                   (assoc-ref inputs "docbook-xml") "/xml/dtd/docbook/catalog.xml")))

Line length.

OK to push with this fixed.

Ludo’.

^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [PATCH 6/6] gnu: Tinyproxy: Cleanup xml dependencies
  2014-07-15 21:00   ` Ludovic Courtès
@ 2014-07-15 21:11     ` John Darrington
  0 siblings, 0 replies; 21+ messages in thread
From: John Darrington @ 2014-07-15 21:11 UTC (permalink / raw)
  To: Ludovic Court??s; +Cc: guix-devel, John Darrington

[-- Attachment #1: Type: text/plain, Size: 1053 bytes --]

On Tue, Jul 15, 2014 at 11:00:31PM +0200, Ludovic Court??s wrote:
     John Darrington <jmd@gnu.org> skribis:
     
     > * gnu/packages/web.scm (tinyproxy): Use the xml_catalog from docbook instead of
     >   generating a temporary one.
     
     Limit lines to 80 chars please.

By my count, that is exactly 78 characters.
     
     Also, ???Use the XML catalog??? no?
     
     > +          (setenv "XML_CATALOG_FILES" 
     > +                  (string-append
     > +                   (assoc-ref inputs "docbook-xsl") "/xml/xsl/docbook-xsl-1.78.1/catalog.xml"
     > +                   " "  ; This must be a space, not a : (contrary to the documentation)
     > +                   (assoc-ref inputs "docbook-xml") "/xml/dtd/docbook/catalog.xml")))
     
     Line length.
     
     OK to push with this fixed.
     
     Ludo???.
     

-- 
PGP Public key ID: 1024D/2DE827B3 
fingerprint = 8797 A26D 0854 2EAB 0285  A290 8A67 719C 2DE8 27B3
See http://sks-keyservers.net or any PGP keyserver for public key.


[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [PATCH 5/6] gnu: Enable gnome-doc-utils tests
  2014-07-15 20:59   ` Ludovic Courtès
@ 2014-07-15 21:43     ` John Darrington
  2014-07-16  9:23       ` Ludovic Courtès
  0 siblings, 1 reply; 21+ messages in thread
From: John Darrington @ 2014-07-15 21:43 UTC (permalink / raw)
  To: Ludovic Court??s; +Cc: guix-devel, John Darrington

[-- Attachment #1: Type: text/plain, Size: 1718 bytes --]

On Tue, Jul 15, 2014 at 10:59:17PM +0200, Ludovic Court??s wrote:
     > +    (arguments
     > +     `(#:phases
     > +       (alist-cons-before
     > +        'check 'pre-check
     > +        (lambda* (#:key inputs #:allow-other-keys #:rest args)
     > +          ;; This is needed, because without it, xmlint etc tries
     > +          ;; to download docbookx.dtd from the net
     > +          (setenv "XML_CATALOG_FILES" 
     > +                  (string-append (assoc-ref inputs "docbook-xml") 
     > +                                 "/xml/dtd/docbook/catalog.xml")))
     > +        %standard-phases)))
     >      (native-inputs
     >       `(("intltool" ,intltool)
     > +       ("docbook-xml" ,docbook-xml-4.4)
     
     We can???t copy that around forever.
     
     The problem is that ???search-path-specification??? is meant for $PATH-like
     variables that list directories, not files.

That occurred to me too.  But what problems does it actually cause?
     
     So I see two solutions:
     
       1. Patch libxml2 so that it honors a new variable, say
          $XML_CATALOG_DIRECTORIES, which would allow us to use
          ???search-path-specification???.
     
       2. Augment support for search paths to allow file-based search paths.
     
     (2) may be best in the long run, but it has ramifications in different
     places.
     

(1) seems like a good idea only if upstream can be persuaded to adopt it.

What are the ramifications of (2) ?


     

-- 
PGP Public key ID: 1024D/2DE827B3 
fingerprint = 8797 A26D 0854 2EAB 0285  A290 8A67 719C 2DE8 27B3
See http://sks-keyservers.net or any PGP keyserver for public key.


[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [PATCH 5/6] gnu: Enable gnome-doc-utils tests
  2014-07-15 21:43     ` John Darrington
@ 2014-07-16  9:23       ` Ludovic Courtès
  2014-07-16 10:08         ` John Darrington
  0 siblings, 1 reply; 21+ messages in thread
From: Ludovic Courtès @ 2014-07-16  9:23 UTC (permalink / raw)
  To: John Darrington; +Cc: guix-devel, John Darrington

John Darrington <john@darrington.wattle.id.au> skribis:

> On Tue, Jul 15, 2014 at 10:59:17PM +0200, Ludovic Court??s wrote:

[...]

>      The problem is that ???search-path-specification??? is meant for $PATH-like
>      variables that list directories, not files.
>
> That occurred to me too.  But what problems does it actually cause?

That we can’t use it for $XML_CATALOG_FILES.

>      So I see two solutions:
>      
>        1. Patch libxml2 so that it honors a new variable, say
>           $XML_CATALOG_DIRECTORIES, which would allow us to use
>           ???search-path-specification???.
>      
>        2. Augment support for search paths to allow file-based search paths.
>      
>      (2) may be best in the long run, but it has ramifications in different
>      places.
>      
>
> (1) seems like a good idea only if upstream can be persuaded to adopt it.

Which is unlikely, given that it’s redundant with $XML_CATALOG_FILES.

> What are the ramifications of (2) ?

There are changes in the build tools, for instance ‘search-path-as-list’
(used by ‘set-path-environment-variable’, used in gnu-build-system.scm)
expects directories, not files.  And all this calls things
‘directories’.

This is a change we could schedule for the next core-updates.

WDYT?

Ludo’.

^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [PATCH 5/6] gnu: Enable gnome-doc-utils tests
  2014-07-16  9:23       ` Ludovic Courtès
@ 2014-07-16 10:08         ` John Darrington
  2014-07-16 14:02           ` bug#18033: Add support for 'search-path-specifications' referring to files Ludovic Courtès
  0 siblings, 1 reply; 21+ messages in thread
From: John Darrington @ 2014-07-16 10:08 UTC (permalink / raw)
  To: Ludovic Court??s; +Cc: guix-devel, John Darrington

On Wed, Jul 16, 2014 at 11:23:12AM +0200, Ludovic Court??s wrote:
     John Darrington <john@darrington.wattle.id.au> skribis:
     
     > On Tue, Jul 15, 2014 at 10:59:17PM +0200, Ludovic Court??s wrote:
     
     [...]
     
     >      The problem is that ???search-path-specification??? is meant for $PATH-like
     >      variables that list directories, not files.
     >
     > That occurred to me too.  But what problems does it actually cause?
     
     That we can???t use it for $XML_CATALOG_FILES.
     
     >      So I see two solutions:
     >      
     >        1. Patch libxml2 so that it honors a new variable, say
     >           $XML_CATALOG_DIRECTORIES, which would allow us to use
     >           ???search-path-specification???.
     >      
     >        2. Augment support for search paths to allow file-based search paths.
     >      
     >      (2) may be best in the long run, but it has ramifications in different
     >      places.
     >      
     >
     > (1) seems like a good idea only if upstream can be persuaded to adopt it.
     
     Which is unlikely, given that it???s redundant with $XML_CATALOG_FILES.
     
     > What are the ramifications of (2) ?
     
     There are changes in the build tools, for instance ???search-path-as-list???
     (used by ???set-path-environment-variable???, used in gnu-build-system.scm)
     expects directories, not files.  And all this calls things
     ???directories???.
     
     This is a change we could schedule for the next core-updates.

This sounds like it is the most sensible solution. 

J'
-- 
PGP Public key ID: 1024D/2DE827B3 
fingerprint = 8797 A26D 0854 2EAB 0285  A290 8A67 719C 2DE8 27B3
See http://sks-keyservers.net or any PGP keyserver for public key.

^ permalink raw reply	[flat|nested] 21+ messages in thread

* bug#18033: Add support for 'search-path-specifications' referring to files
  2014-07-16 10:08         ` John Darrington
@ 2014-07-16 14:02           ` Ludovic Courtès
  2014-12-27 11:26             ` Ludovic Courtès
  0 siblings, 1 reply; 21+ messages in thread
From: Ludovic Courtès @ 2014-07-16 14:02 UTC (permalink / raw)
  To: 18033; +Cc: John Darrington

See use case below.

John Darrington <john@darrington.wattle.id.au> skribis:

> On Wed, Jul 16, 2014 at 11:23:12AM +0200, Ludovic Court??s wrote:
>      John Darrington <john@darrington.wattle.id.au> skribis:
>      
>      > On Tue, Jul 15, 2014 at 10:59:17PM +0200, Ludovic Court??s wrote:
>      
>      [...]
>      
>      >      The problem is that ???search-path-specification??? is meant for $PATH-like
>      >      variables that list directories, not files.
>      >
>      > That occurred to me too.  But what problems does it actually cause?
>      
>      That we can???t use it for $XML_CATALOG_FILES.
>      
>      >      So I see two solutions:
>      >      
>      >        1. Patch libxml2 so that it honors a new variable, say
>      >           $XML_CATALOG_DIRECTORIES, which would allow us to use
>      >           ???search-path-specification???.
>      >      
>      >        2. Augment support for search paths to allow file-based search paths.
>      >      
>      >      (2) may be best in the long run, but it has ramifications in different
>      >      places.
>      >      
>      >
>      > (1) seems like a good idea only if upstream can be persuaded to adopt it.
>      
>      Which is unlikely, given that it???s redundant with $XML_CATALOG_FILES.
>      
>      > What are the ramifications of (2) ?
>      
>      There are changes in the build tools, for instance ???search-path-as-list???
>      (used by ???set-path-environment-variable???, used in gnu-build-system.scm)
>      expects directories, not files.  And all this calls things
>      ???directories???.
>      
>      This is a change we could schedule for the next core-updates.
>
> This sounds like it is the most sensible solution. 
>
> J'

^ permalink raw reply	[flat|nested] 21+ messages in thread

* bug#18033: Add support for 'search-path-specifications' referring to files
  2014-07-16 14:02           ` bug#18033: Add support for 'search-path-specifications' referring to files Ludovic Courtès
@ 2014-12-27 11:26             ` Ludovic Courtès
  2014-12-27 22:57               ` Ludovic Courtès
  0 siblings, 1 reply; 21+ messages in thread
From: Ludovic Courtès @ 2014-12-27 11:26 UTC (permalink / raw)
  To: 18033; +Cc: John Darrington

Commit 6aa47e3 implements the build-side part for that.

Ludo’.

^ permalink raw reply	[flat|nested] 21+ messages in thread

* bug#18033: Add support for 'search-path-specifications' referring to files
  2014-12-27 11:26             ` Ludovic Courtès
@ 2014-12-27 22:57               ` Ludovic Courtès
  2015-01-02 23:30                 ` Mark H Weaver
  0 siblings, 1 reply; 21+ messages in thread
From: Ludovic Courtès @ 2014-12-27 22:57 UTC (permalink / raw)
  To: 18033-done; +Cc: John Darrington

Commit af07095 extends ‘search-path-specification’ with a ‘file-type’
field, which can be 'regular.

Then, commit 7b21fe5 adds a ‘file-pattern’ field, to look for files
matching a pattern recursively.

Lastly, commit 2a8d440 uses these features for libxml2 and
$XML_CATALOG_FILES.

Comments welcome!

Ludo’.

^ permalink raw reply	[flat|nested] 21+ messages in thread

* bug#18033: Add support for 'search-path-specifications' referring to files
  2014-12-27 22:57               ` Ludovic Courtès
@ 2015-01-02 23:30                 ` Mark H Weaver
  2015-01-03 18:48                   ` Ludovic Courtès
  0 siblings, 1 reply; 21+ messages in thread
From: Mark H Weaver @ 2015-01-02 23:30 UTC (permalink / raw)
  To: 18033

Hi Ludovic,

ludo@gnu.org (Ludovic Courtès) writes:

> Commit af07095 extends ‘search-path-specification’ with a ‘file-type’
> field, which can be 'regular.
>
> Then, commit 7b21fe5 adds a ‘file-pattern’ field, to look for files
> matching a pattern recursively.
>
> Lastly, commit 2a8d440 uses these features for libxml2 and
> $XML_CATALOG_FILES.
>
> Comments welcome!

Something seems to have gone wrong here.  Trying to build 'emacs' on
current 'core-updates' (bf0baaf) on i686, I get this:

--8<---------------cut here---------------start------------->8---
mhw:~/guix-core-updates$ git describe
v0.8-499-gbf0baaf
mhw:~/guix-core-updates$ ./pre-inst-env guile
GNU Guile 2.0.11
Copyright (C) 1995-2014 Free Software Foundation, Inc.

Guile comes with ABSOLUTELY NO WARRANTY; for details type `,show w'.
This program is free software, and you are welcome to redistribute it
under certain conditions; type `,show c' for details.

Enter `,help' for help.
scheme@(guile-user)> ,use (guix scripts build)
scheme@(guile-user)> (guix-build "emacs")
ERROR: In procedure concatenate:
ERROR: In procedure append: Wrong type argument in position 23 (expecting empty list): #<<search-path-specification> variable: "XML_CATALOG_FILES" files: ("xml") separator: " " file-type: regular file-pattern: "^catalog\\.xml$">

Entering a new prompt.  Type `,bt' for a backtrace or `,q' to continue.
scheme@(guile-user) [1]> ,bt
In ice-9/boot-9.scm:
   157:17 33 (catch srfi-34 #<procedure 92450a0 at guix/ui.scm:218:2 ()> #<procedure 927b2c0 at guix/ui.scm:218:2 (key c)> #)
   157:17 32 (catch system-error #<procedure 92450b0 at guix/scripts/build.scm:416:2 ()> #<procedure 927b300 at guix/ui.s…> …)
In guix/scripts/build.scm:
    420:6 31 (#<procedure 92450b0 at guix/scripts/build.scm:416:2 ()>)
    318:6 30 (options->derivations #<build-daemon 256.14 9211ae0> ((argument . "emacs") (system . "i686-linux") (# . #t) …))
In srfi/srfi-1.scm:
   664:12 29 (filter-map #<procedure 9e01a60 at guix/scripts/build.scm:318:18 (expr)> ((argument . #<package emacs-24.…>) …))
In guix/packages.scm:
   620:14 28 (cache #<package emacs-24.4 gnu/packages/emacs.scm:57 98f2060> ("i686-linux" . #t) #<procedure thunk ()>)
   869:10 27 (thunk)
    801:6 26 (bag->derivation #<build-daemon 256.14 9211ae0> #<<bag> name: "emacs-24.4" system: "i686-linux" target: #f b…> …)
In srfi/srfi-1.scm:
   578:27 25 (map #<procedure 9a5c498 at guix/packages.scm:803:30 (t-8656)> (("glib:bin" #<package glib-2.42.1 gnu/pa…>) …))
In guix/packages.scm:
   662:16 24 (expand-input #<build-daemon 256.14 9211ae0> #<package emacs-24.4 gnu/packages/emacs.scm:57 98f2060> ("gl…" …) …)
   620:14 23 (cache #<package glib-2.42.1 gnu/packages/glib.scm:120 99234e0> ("i686-linux" . #f) #<procedure thunk ()>)
   869:10 22 (thunk)
    801:6 21 (bag->derivation #<build-daemon 256.14 9211ae0> #<<bag> name: "glib-2.42.1" system: "i686-linux" target: #f …> …)
In srfi/srfi-1.scm:
   578:27 20 (map #<procedure 9a5c408 at guix/packages.scm:803:30 (t-8656)> (("source" #<origin "mirror://gnome/sourc…>) …))
In guix/packages.scm:
   662:16 19 (expand-input #<build-daemon 256.14 9211ae0> #<package glib-2.42.1 gnu/packages/glib.scm:120 99234e0> ("d…" …) …)
   620:14 18 (cache #<package dbus-1.8.12 gnu/packages/glib.scm:58 9923540> ("i686-linux" . #f) #<procedure thunk ()>)
   869:10 17 (thunk)
    801:6 16 (bag->derivation #<build-daemon 256.14 9211ae0> #<<bag> name: "dbus-1.8.12" system: "i686-linux" target: #f …> …)
In srfi/srfi-1.scm:
   578:27 15 (map #<procedure a00e390 at guix/packages.scm:803:30 (t-8656)> (("source" #<origin "http://dbus.freedesk…>) …))
In guix/packages.scm:
   662:16 14 (expand-input #<build-daemon 256.14 9211ae0> #<package dbus-1.8.12 gnu/packages/glib.scm:58 9923540> ("li…" …) …)
   620:14 13 (cache #<package libx11-1.6.2 gnu/packages/xorg.scm:4501 999ba20> ("i686-linux" . #f) #<procedure thunk ()>)
   869:10 12 (thunk)
    801:6 11 (bag->derivation #<build-daemon 256.14 9211ae0> #<<bag> name: "libx11-1.6.2" system: "i686-linux" target: #f…> …)
In srfi/srfi-1.scm:
   576:29 10 (map #<procedure a1ec390 at guix/packages.scm:803:30 (t-8656)> (("source" #<origin "mirror://xorg/indivi…>) …))
In guix/packages.scm:
   662:16  9 (expand-input #<build-daemon 256.14 9211ae0> #<package libx11-1.6.2 gnu/packages/xorg.scm:4501 999ba20> (# #) …)
   620:14  8 (cache #<package libxcb-1.11 gnu/packages/xorg.scm:4370 999bae0> ("i686-linux" . #f) #<procedure thunk ()>)
   869:10  7 (thunk)
    801:6  6 (bag->derivation #<build-daemon 256.14 9211ae0> #<<bag> name: "libxcb-1.11" system: "i686-linux" target: #f …> …)
In srfi/srfi-1.scm:
   578:27  5 (map #<procedure a388c00 at guix/packages.scm:803:30 (t-8656)> (("source" #<origin "mirror://xorg/indivi…>) …))
In guix/packages.scm:
   662:16  4 (expand-input #<build-daemon 256.14 9211ae0> #<package libxcb-1.11 gnu/packages/xorg.scm:4370 999bae0> ("…" …) …)
   620:14  3 (cache #<package libxslt-1.1.28 gnu/packages/xml.scm:106 9963c00> ("i686-linux" . #f) #<procedure thunk ()>)
   869:10  2 (thunk)
   806:26  1 (bag->derivation #<build-daemon 256.14 9211ae0> #<<bag> name: "libxslt-1.1.28" system: "i686-linux" target: …> …)
In unknown file:
           0 (concatenate (() () () () () () () () () () () () () () () () () (#<<search-path-specification> variabl…> …) …))
scheme@(guile-user) [1]>
--8<---------------cut here---------------end--------------->8---

      Mark

^ permalink raw reply	[flat|nested] 21+ messages in thread

* bug#18033: Add support for 'search-path-specifications' referring to files
  2015-01-02 23:30                 ` Mark H Weaver
@ 2015-01-03 18:48                   ` Ludovic Courtès
  0 siblings, 0 replies; 21+ messages in thread
From: Ludovic Courtès @ 2015-01-03 18:48 UTC (permalink / raw)
  To: Mark H Weaver; +Cc: 18033

Mark H Weaver <mhw@netris.org> skribis:

> Something seems to have gone wrong here.  Trying to build 'emacs' on
> current 'core-updates' (bf0baaf) on i686, I get this:
>
> mhw:~/guix-core-updates$ git describe
> v0.8-499-gbf0baaf
> mhw:~/guix-core-updates$ ./pre-inst-env guile
> GNU Guile 2.0.11
> Copyright (C) 1995-2014 Free Software Foundation, Inc.
>
> Guile comes with ABSOLUTELY NO WARRANTY; for details type `,show w'.
> This program is free software, and you are welcome to redistribute it
> under certain conditions; type `,show c' for details.
>
> Enter `,help' for help.
> scheme@(guile-user)> ,use (guix scripts build)
> scheme@(guile-user)> (guix-build "emacs")
> ERROR: In procedure concatenate:
> ERROR: In procedure append: Wrong type argument in position 23 (expecting empty list): #<<search-path-specification> variable: "XML_CATALOG_FILES" files: ("xml") separator: " " file-type: regular file-pattern: "^catalog\\.xml$">

A typo fixed in commit 7452806.  Thanks for the report!

Ludo’.

^ permalink raw reply	[flat|nested] 21+ messages in thread

end of thread, other threads:[~2015-01-03 18:49 UTC | newest]

Thread overview: 21+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-07-15 18:31 [PATCH 1/6] gnu: docbook-xsl update to 1.78.1 John Darrington
2014-07-15 18:31 ` [PATCH 2/6] gnu: Add docbook-xml version 4.3 and 4.4 John Darrington
2014-07-15 20:49   ` Ludovic Courtès
2014-07-15 18:31 ` [PATCH 3/6] gnu: docbook-xml: Substite remote xml_catalog uris with local ones John Darrington
2014-07-15 20:51   ` Ludovic Courtès
2014-07-15 18:31 ` [PATCH 4/6] gnu: docbook-xsl write absolute file:// uris in catalogs instead of ./ relative ones John Darrington
2014-07-15 20:51   ` Ludovic Courtès
2014-07-15 18:31 ` [PATCH 5/6] gnu: Enable gnome-doc-utils tests John Darrington
2014-07-15 20:59   ` Ludovic Courtès
2014-07-15 21:43     ` John Darrington
2014-07-16  9:23       ` Ludovic Courtès
2014-07-16 10:08         ` John Darrington
2014-07-16 14:02           ` bug#18033: Add support for 'search-path-specifications' referring to files Ludovic Courtès
2014-12-27 11:26             ` Ludovic Courtès
2014-12-27 22:57               ` Ludovic Courtès
2015-01-02 23:30                 ` Mark H Weaver
2015-01-03 18:48                   ` Ludovic Courtès
2014-07-15 18:31 ` [PATCH 6/6] gnu: Tinyproxy: Cleanup xml dependencies John Darrington
2014-07-15 21:00   ` Ludovic Courtès
2014-07-15 21:11     ` John Darrington
2014-07-15 20:49 ` [PATCH 1/6] gnu: docbook-xsl update to 1.78.1 Ludovic Courtès

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.