GNU bug report logs - #26676
[PATCH] gnu: Add xerces-c.

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 26676 in the body.
You can then email your comments to 26676 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Ricardo Wurmus <rekado <at> elephly.net>
To: guix-patches <at> gnu.org
Cc: Ricardo Wurmus <rekado <at> elephly.net>
Subject: [PATCH] gnu: Add xerces-c.
Date: Thu, 27 Apr 2017 08:52:36 +0200

* gnu/packages/xml.scm (xerces-c): New variable.
---
 gnu/packages/xml.scm | 27 ++++++++++++++++++++++++++-
 1 file changed, 26 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/xml.scm b/gnu/packages/xml.scm
index 247107917..38dd07259 100644
--- a/gnu/packages/xml.scm
+++ b/gnu/packages/xml.scm
@@ -3,7 +3,7 @@
 ;;; Copyright © 2013, 2015 Andreas Enge <andreas <at> enge.fr>
 ;;; Copyright © 2015 Eric Bavier <bavier <at> member.fsf.org>
 ;;; Copyright © 2015 Sou Bunnbu <iyzsong <at> gmail.com>
-;;; Copyright © 2015, 2016 Ricardo Wurmus <rekado <at> elephly.net>
+;;; Copyright © 2015, 2016, 2017 Ricardo Wurmus <rekado <at> elephly.net>
 ;;; Copyright © 2015, 2016 Mark H Weaver <mhw <at> netris.org>
 ;;; Copyright © 2015, 2016 Efraim Flashner <efraim <at> flashner.co.il>
 ;;; Copyright © 2015 Raimon Grau <raimonster <at> gmail.com>
@@ -1145,3 +1145,28 @@ spreadsheet.")
     (license (list license:gpl2+
                    license:lgpl2.1+
                    license:mpl1.1))))
+
+(define-public xerces-c
+  (package
+    (name "xerces-c")
+    (version "3.1.4")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append "mirror://apache/xerces/c/3/sources/"
+                                  "xerces-c-" version ".tar.xz"))
+              (sha256
+               (base32
+                "0hb29c0smqlpxj0zdm09s983z5jx37szlliccnvgh0qq91wwqwwr"))))
+    (build-system gnu-build-system)
+    (arguments
+     `(#:configure-flags (list "--disable-sse2")))
+    (native-inputs
+     `(("perl" ,perl)))
+    (home-page "http://xerces.apache.org/xerces-c/")
+    (synopsis "Validating XML parser library for C++")
+    (description "Xerces-C++ is a validating XML parser written in a portable
+subset of C++.  Xerces-C++ makes it easy to give your application the ability
+to read and write XML data.  A shared library is provided for parsing,
+generating, manipulating, and validating XML documents using the DOM, SAX, and
+SAX2 APIs.")
+    (license license:asl2.0)))
-- 
2.12.2

Message #8 received at 26676 <at> debbugs.gnu.org (full text, mbox):

From: Marius Bakke <mbakke <at> fastmail.com>
To: Ricardo Wurmus <rekado <at> elephly.net>, 26676 <at> debbugs.gnu.org
Cc: Ricardo Wurmus <rekado <at> elephly.net>
Subject: Re: bug#26676: [PATCH] gnu: Add xerces-c.
Date: Thu, 27 Apr 2017 15:29:07 +0200

[Message part 1 (text/plain, inline)]

Ricardo Wurmus <rekado <at> elephly.net> writes:

> * gnu/packages/xml.scm (xerces-c): New variable.

[...]

> +(define-public xerces-c
> +  (package
> +    (name "xerces-c")
> +    (version "3.1.4")
> +    (source (origin
> +              (method url-fetch)
> +              (uri (string-append "mirror://apache/xerces/c/3/sources/"
> +                                  "xerces-c-" version ".tar.xz"))
> +              (sha256
> +               (base32
> +                "0hb29c0smqlpxj0zdm09s983z5jx37szlliccnvgh0qq91wwqwwr"))))
> +    (build-system gnu-build-system)
> +    (arguments
> +     `(#:configure-flags (list "--disable-sse2")))

SSE2 is part of the AMD64 specification[0], so enabling it for x86_64
would be slightly nicer. Other than that looks good!

[0] https://en.wikipedia.org/wiki/SSE2#CPU_support

> +    (native-inputs
> +     `(("perl" ,perl)))
> +    (home-page "http://xerces.apache.org/xerces-c/")
> +    (synopsis "Validating XML parser library for C++")
> +    (description "Xerces-C++ is a validating XML parser written in a portable
> +subset of C++.  Xerces-C++ makes it easy to give your application the ability
> +to read and write XML data.  A shared library is provided for parsing,
> +generating, manipulating, and validating XML documents using the DOM, SAX, and
> +SAX2 APIs.")
> +    (license license:asl2.0)))
> -- 
> 2.12.2

[signature.asc (application/pgp-signature, inline)]

Message #13 received at 26676-done <at> debbugs.gnu.org (full text, mbox):

From: Ricardo Wurmus <rekado <at> elephly.net>
To: Marius Bakke <mbakke <at> fastmail.com>
Cc: 26676-done <at> debbugs.gnu.org
Subject: Re: bug#26676: [PATCH] gnu: Add xerces-c.
Date: Fri, 28 Apr 2017 17:48:15 +0200

Marius Bakke <mbakke <at> fastmail.com> writes:

> Ricardo Wurmus <rekado <at> elephly.net> writes:
>
>> * gnu/packages/xml.scm (xerces-c): New variable.
>
> [...]
>
>> +(define-public xerces-c
>> +  (package
>> +    (name "xerces-c")
>> +    (version "3.1.4")
>> +    (source (origin
>> +              (method url-fetch)
>> +              (uri (string-append "mirror://apache/xerces/c/3/sources/"
>> +                                  "xerces-c-" version ".tar.xz"))
>> +              (sha256
>> +               (base32
>> +                "0hb29c0smqlpxj0zdm09s983z5jx37szlliccnvgh0qq91wwqwwr"))))
>> +    (build-system gnu-build-system)
>> +    (arguments
>> +     `(#:configure-flags (list "--disable-sse2")))
>
> SSE2 is part of the AMD64 specification[0], so enabling it for x86_64
> would be slightly nicer. Other than that looks good!
>
> [0] https://en.wikipedia.org/wiki/SSE2#CPU_support

Ah, you’re right.  I pushed it with a fix with commit 162a4a032^.
Thanks for the review!

--
Ricardo

  GPG: BCA6 89B6 3655 3801 C3C6  2150 197A 5888 235F ACAC
  https://elephly.net

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.