unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Leo Famulari <leo@famulari.name>
To: Jan Nieuwenhuizen <janneke@gnu.org>
Cc: guix-devel@gnu.org
Subject: Re: [PATCH] gnu: Add tclxml.
Date: Fri, 12 Feb 2016 19:22:24 -0500	[thread overview]
Message-ID: <20160213002224.GA10770@jasmine> (raw)
In-Reply-To: <87ziv5a2ww.fsf@gnu.org>

On Sat, Feb 13, 2016 at 01:07:27AM +0100, Jan Nieuwenhuizen wrote:
> Jan Nieuwenhuizen writes:
> 
> Hi,
> 
> Rebased against master, cleanups.
> 
> Greetings,
> Jan
> 

> From bf4e4975f7470d0f82dd2451d45de99e57193efd Mon Sep 17 00:00:00 2001
> From: Jan Nieuwenhuizen <janneke@gnu.org>
> Date: Sat, 13 Feb 2016 00:27:17 +0100
> Subject: [PATCH] gnu: Add tclxml.
> 
> * gnu/packages/patches/tclxml-3.2-install.patch: New file.
> * gnu-system.am (dist_patch_DATA): Add it.
> * gnu/packages/tcl.scm (tclxml): New variable.

Thanks for the patch! Pushed as 511539ae811

> ---
>  gnu-system.am                                 |  1 +
>  gnu/packages/patches/tclxml-3.2-install.patch | 17 ++++++++++
>  gnu/packages/tcl.scm                          | 46 +++++++++++++++++++++++++++
>  3 files changed, 64 insertions(+)
>  create mode 100644 gnu/packages/patches/tclxml-3.2-install.patch
> 
> diff --git a/gnu-system.am b/gnu-system.am
> index 7f3dfb8..19f1244 100644
> --- a/gnu-system.am
> +++ b/gnu-system.am
> @@ -710,6 +710,7 @@ dist_patch_DATA =						\
>    gnu/packages/patches/tar-d_ino_in_dirent-fix.patch		\
>    gnu/packages/patches/tar-skip-unreliable-tests.patch		\
>    gnu/packages/patches/tcl-mkindex-deterministic.patch		\
> +  gnu/packages/patches/tclxml-3.2-install.patch			\
>    gnu/packages/patches/tcsh-fix-autotest.patch			\
>    gnu/packages/patches/texi2html-document-encoding.patch	\
>    gnu/packages/patches/texi2html-i18n.patch			\
> diff --git a/gnu/packages/patches/tclxml-3.2-install.patch b/gnu/packages/patches/tclxml-3.2-install.patch
> new file mode 100644
> index 0000000..09d5992
> --- /dev/null
> +++ b/gnu/packages/patches/tclxml-3.2-install.patch
> @@ -0,0 +1,17 @@
> +Install everything in PREFIX, set EXTRA_PATH to find it there.
> +
> +--- ./Makefile.in~	2016-02-06 01:29:16.595090365 +0100
> ++++ ./Makefile.in	2016-02-06 09:29:04.286008931 +0100
> +@@ -133,9 +133,9 @@
> + # require for testing here (like TCLX_LIBRARY).
> + #========================================================================
> +
> +-EXTRA_PATH	= $(top_builddir):$(TCL_BIN_DIR)
> ++EXTRA_PATH	= $(top_builddir):$(TCL_BIN_DIR):$(libdir)
> + #EXTRA_PATH     = $(top_builddir):$(TCL_BIN_DIR):$(TK_BIN_DIR)
> +-TCLLIBPATH      = $(top_builddir)
> ++TCLLIBPATH      = $(pkglibdir)
> + TCLSH_ENV	= TCL_LIBRARY=`@CYGPATH@ $(TCL_SRC_DIR)/library` \
> + 		  @LD_LIBRARY_PATH_VAR@="$(EXTRA_PATH):$(@LD_LIBRARY_PATH_VAR@)" \
> + 		  PATH="$(EXTRA_PATH):$(PATH)" \
> + 
> \ No newline at end of file
> diff --git a/gnu/packages/tcl.scm b/gnu/packages/tcl.scm
> index d98d2d6..1f30145 100644
> --- a/gnu/packages/tcl.scm
> +++ b/gnu/packages/tcl.scm
> @@ -30,6 +30,7 @@
>    #:use-module (gnu packages fontutils)
>    #:use-module (gnu packages perl)
>    #:use-module (gnu packages pkg-config)
> +  #:use-module (gnu packages xml)
>    #:use-module (gnu packages xorg)
>    #:use-module (guix licenses))
>  
> @@ -244,3 +245,48 @@ interfaces (GUIs) in the Tcl language.")
>      (description "Tcllib, the standard Tcl library, is a collection of common
>  utility functions and modules all written in high-level Tcl.")
>      (license (package-license tcl))))
> +
> +(define-public tclxml
> +  (package
> +    (name "tclxml")
> +    (version "3.2")
> +    (source (origin
> +              (method url-fetch)
> +              (uri (string-append "mirror://sourceforge/" name "/"
> +                                  name "-" version ".tar.gz"))
> +              (sha256
> +               (base32
> +                "0ffb4aw63inig3aql33g4pk0kjk14dv238anp1scwjdjh1k6n4gl"))
> +              (patches (list (search-patch "tclxml-3.2-install.patch")))))
> +    (build-system gnu-build-system)
> +    (native-inputs
> +     `(("tcl" ,tcl)
> +       ("tcllib" ,tcllib)
> +       ("libxml2" ,libxml2)
> +       ("libxslt" ,libxslt)))
> +    (native-search-paths
> +     (list (search-path-specification
> +            (variable "TCLLIBPATH")
> +            (separator " ")
> +            (files (list (string-append "lib/Tclxml" version))))))
> +    (arguments
> +     `(#:configure-flags
> +       (list (string-append "--exec-prefix=" (assoc-ref %outputs "out"))
> +             (string-append "--with-tclconfig="
> +                            (assoc-ref %build-inputs "tcl") "/lib")
> +             (string-append "--with-xml2-config="
> +                            (assoc-ref %build-inputs "libxml2")
> +                            "/bin/xml2-config")
> +             (string-append "--with-xslt-config="
> +                            (assoc-ref %build-inputs "libxslt")
> +                            "/bin/xslt-config"))
> +       #:test-target "test"))
> +    (home-page "http://tclxml.sourceforge.net/")
> +    (synopsis "Tcl library for XML parsing")
> +    (description "TclXML provides event-based parsing of XML documents.  The
> +application may register callback scripts for certain document features, and
> +when the parser encounters those features while parsing the document the
> +callback is evaluated.")
> +    (license (non-copyleft
> +              "file://LICENCE"
> +              "See LICENCE in the distribution."))))
> -- 
> 2.1.4
> 

> 
> -- 
> Jan Nieuwenhuizen <janneke@gnu.org> | GNU LilyPond http://lilypond.org
> Freelance IT http://JoyofSource.com | Avatar®  http://AvatarAcademy.nl  

  reply	other threads:[~2016-02-13  0:22 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-02-09 20:20 [PATCH] gnu: Add tclxml Jan Nieuwenhuizen
2016-02-11 20:10 ` Efraim Flashner
2016-02-11 22:11   ` Jan Nieuwenhuizen
2016-02-13  0:07     ` Jan Nieuwenhuizen
2016-02-13  0:22       ` Leo Famulari [this message]
2016-02-13  8:09         ` Jan Nieuwenhuizen

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

  List information: https://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20160213002224.GA10770@jasmine \
    --to=leo@famulari.name \
    --cc=guix-devel@gnu.org \
    --cc=janneke@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 public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).