[PATCH 1/3] gnu: Add opusfile.

2015-10-19 Thread Paul van der Walt
* gnu/packages/xiph.scm (opusfile): New variable. --- gnu/packages/xiph.scm | 28 1 file changed, 28 insertions(+) diff --git a/gnu/packages/xiph.scm b/gnu/packages/xiph.scm index c4db640..78d4cfa 100644 --- a/gnu/packages/xiph.scm +++ b/gnu/packages/xiph.scm @@ -46,6

[PATCH] gnu: Add dosbox x86 emulator.

2015-10-19 Thread Paul van der Walt
/packages/games.scm @@ -12,6 +12,7 @@ ;;; Copyright © 2015 Christopher Allan Webber ;;; Copyright © 2015 Ricardo Wurmus ;;; Copyright © 2015 Alex Kost +;;; Copyright © 2015 Paul van der Walt ;;; ;;; This file is part of GNU Guix. ;;; @@ -34,7 +35,9 @@ #:use-module (guix packages) #:use

Re: Haskell Platform patchspam

2015-10-19 Thread Paul van der Walt
Greetings Ludo, On 2015-10-19 at 15:24, quoth Ludovic Courtès: > I think we are obviously reaching the limits of our review process > here. So what I would suggest is to leave, say, 4 more days for > people to review the branch and send any suggestions. Sounds fine. I'll see what happens. > On

Re: [PATCH 1/2] gnu: Add ghc-transformers

2015-10-19 Thread Paul van der Walt
Hi James, On 2015-10-15 at 19:21, quoth James David Trotter: > I was just about to submit a (somewhat overlapping) set of about > thirty patches in order to add Agda to guix. Hopefully, this should > become easier after your patches. I've pushed the branch wip-haskell-platform-7.10.2-a, perhaps i

Re: [PATCH 032/151] gnu: Add ghc-hspec-discover.

2015-10-20 Thread Paul van der Walt
Hi Eric, I'm starting to fix up your comments. When i'm done, should i git push -f to the wip-haskell-platform-7.10.2-a branch, or should i drop that one and create a new one? On 2015-10-19 at 16:56, quoth ericbav...@openmailbox.org: > +(define-public ghc-hspec-discover > + (package > +(nam

Re: [PATCH 19/43] gnu: Add ghc-stringbuilder.

2015-10-20 Thread Paul van der Walt
On 2015-10-19 at 16:20, quoth Eric Bavier: >> +(arguments `(#:tests? #f)) ; circular dependencies with tests enabled >> +(inputs >> + `(("ghc-quickcheck" ,ghc-quickcheck))) > > Is this input still necessary with tests disabled? It appears not. Removed.

Re: [PATCH 19/43] gnu: ghc-setenv: New variable.

2015-10-20 Thread Paul van der Walt
On 2015-10-19 at 16:14, quoth Eric Bavier: >> +(home-page >> + "http://hackage.haskell.org/package/setenv";) > > Please put the string on the same line as a field name. Done. >> +(synopsis >> + "A cross-platform library for setting environment variables") > > Ditto. (Note: A rea

Re: [PATCH 18/43] gnu: ghc-async: New variable.

2015-10-20 Thread Paul van der Walt
On 2015-10-19 at 16:08, quoth Eric Bavier: >> +(arguments `(#:tests? #f)) ; FIXME: testing libraries are missing. > > If this is the case, can the "ghc-hunit" input be left out? It seems so. It compiles without. Fixed. >> +(home-page "https://github.com/simonmar/async";) >> +(synop

Re: [PATCH 17/43] gnu: ghc-containers: Remove variable.

2015-10-20 Thread Paul van der Walt
On 2015-10-19 at 16:01, quoth Eric Bavier: > Based on our git log, it appears the commit subject line should be: > > "gnu: Remove ghc-containers" Fixed. >> * gnu/packages/haskell.scm (ghc-containers): Remove variable, package >> provided by GHC 7.10.2. > > The justifaction could be moved prior

Re: [PATCH 16/43] gnu: ghc-ansi-terminal: New variable.

2015-10-20 Thread Paul van der Walt
On 2015-10-19 at 15:53, quoth Eric Bavier: >> +(synopsis "Simple ANSI terminal support, with Windows compatibility") > > We don't really care about Winblows compatibility, and "simple" is > subjective. How about: " ANSI terminal support for Haskell" Done, and >> +(description "ANSI term

Re: [PATCH 14/43] gnu: ghc-http: Update to 4000.2.20.

2015-10-20 Thread Paul van der Walt
On 2015-10-19 at 15:49, quoth Eric Bavier: >> (propagated-inputs >> - `(("ghc-parsec" ,ghc-parsec) >> + `(("ghc-old-time" ,ghc-old-time) >> + ("ghc-parsec" ,ghc-parsec) >> ("ghc-mtl" ,ghc-mtl) >> ("ghc-network" ,ghc-network) >> ("ghc-network-uri" ,ghc-net

Re: [PATCH 13/43] gnu: ghc-old-time: Add variable.

2015-10-20 Thread Paul van der Walt
On 2015-10-19 at 15:46, quoth Eric Bavier: > First line of commit message should rather be: > > "gnu: Add ghc-old-time" Done. >> +(arguments >> + `(#:phases >> + (alist-cons-before >> +'configure 'fix-/bin/sh >> +(lambda _ >> + ;; Use `sh', not `/bin/sh'. >

Re: [PATCH 04/43] gnu: ghc-xmonad-contrib: Update to version 0.11.4.

2015-10-20 Thread Paul van der Walt
On 2015-10-19 at 15:37, quoth Eric Bavier: >> (propagated-inputs >> `(("ghc-mtl" ,ghc-mtl) >> + ("ghc-old-time" ,ghc-old-time) >> ("ghc-random" ,ghc-random) > > Please mention this new input in the commit message. Done.

Re: [PATCH 03/43] gnu: xmonad: Patch to work on new Haskell Platform

2015-10-20 Thread Paul van der Walt
On 2015-10-19 at 15:36, quoth Eric Bavier: >> + ("xmonad-cabal-patch" >> +,(origin >> + (method url-fetch) >> + (uri (string-append >> + "https://hackage.haskell.org/package/xmonad-"; version >> "/revision/1.cabal")) >> + (sha256 >> +

Re: [PATCH 21/43] gnu: ghc-transformers-compat: New variables.

2015-10-20 Thread Paul van der Walt
On 2015-10-19 at 16:37, quoth Eric Bavier: >> + (package >> +(name "ghc-exceptions") >> . >> +(inputs >> + `(("ghc-quickcheck" ,ghc-quickcheck))) > > Is this still required even with '#:tests? #f'? Turns out it's not. Fixed. >> +(description >> + "Extensible optionally-

Re: [PATCH 032/151] gnu: Add ghc-hspec-discover.

2015-10-20 Thread Paul van der Walt
Hello Eric, Ludo', On 2015-10-20 at 16:57, quoth Ludovic Courtès: > ‘push -f’ is fine. The convention is that ‘wip-’ branches may be > rebased/modified any time. Eric: Thanks for the comments, i've gone over the descriptions and tried to make them all better (no sentence fragments, etc.). Thank

[PATCH] gnu: Update list of GHC Core libraries for 7.10.2.

2015-10-21 Thread Paul van der Walt
Update ghc-standard-libraries to match the output of `ghc-pkg list` when using GHC 7.10.2. * guix/import/hackage.scm (ghc-standard-libraries): Sort and update list of core GHC libraries. --- guix/import/hackage.scm | 40 +++- 1 file changed, 19 insertions(+),

Re: 01/01: gnu: Hackage import script: Fix grammar.

2015-10-21 Thread Paul van der Walt
On 2015-10-21 at 17:52, quoth Ludovic Courtès: > A minor point, but the convention would be something like: > > “import: hackage: Fix grammar.” Okay, i'll modify it! > The “gnu:” prefix is for packages (although it looks weird nowadays.) Ah, i meant to ask about that some time, i had no idea.

Re: [PATCH 1/3] gnu: Add opusfile.

2015-10-21 Thread Paul van der Walt
On 2015-10-21 at 21:23, quoth Ludovic Courtès: >> +(inputs >> + `(("pkg-config" ,pkg-config) > > This one should be in ‘native-inputs’ (normally ‘guix lint’ warns about > it.) > > OK to push with this change. Right, fixed and pushed. Is it surprising that it worked as i had it, though?

Re: [PATCH 2/3] gnu: Add libcue.

2015-10-21 Thread Paul van der Walt
On 2015-10-21 at 21:29, quoth Ludovic Courtès: > What about: “Libcue is C library to parse so-called @dfn{cue sheets}, > which contain meta-data for CD/DVD tracks. It provides an API to > manipulate the data.” > > ... > > OK with a change along these lines, thanks! Thanks, done! p.

Re: [PATCH] gnu: Add dosbox x86 emulator.

2015-10-21 Thread Paul van der Walt
On 2015-10-21 at 21:41, quoth Ludovic Courtès: >> +(name "dosbox") >> +(version "0.74") > > Rather 0.74.svn3947 or similar. Right, fixed. >> + ;; Using SVN head, since the last release (2010) is >> incompatible >> + ;; with GCC5 (see >> + ;; >>

[PATCH] gnu: Hackage import script: Fix grammar.

2015-10-21 Thread Paul van der Walt
Minor clarification regarding the test-dependencies command line option. * guix/scripts/import/hackage.scm (show-help): Minor grammatical fix. --- guix/scripts/import/hackage.scm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/guix/scripts/import/hackage.scm b/guix/scripts/imp

Re: [PATCH] gnu: Add dosbox x86 emulator.

2015-10-22 Thread Paul van der Walt
Hi Mark, On 2015-10-22 at 00:40, quoth Mark H Weaver: >> +(arguments >> + `(#:phases (modify-phases %standard-phases >> + (add-before >> + 'configure 'autogen.sh >> + (lambda _ >> + (zero? (system* "sh" "autogen.sh"))

[PATCH 2/2] gnu: dosbox: Add file-name property.

2015-10-22 Thread Paul van der Walt
* gnu/packages/games.scm (dosbox): Add file-name property, since the package is built from an SVN checkout. --- gnu/packages/games.scm | 1 + 1 file changed, 1 insertion(+) diff --git a/gnu/packages/games.scm b/gnu/packages/games.scm index 9d2058a..9c44bbe 100644 --- a/gnu/packages/games.scm ++

Re: [PATCH 3/3] gnu: Add cmus.

2015-10-22 Thread Paul van der Walt
Hi Ricardo, Thanks for your comments! Last night i already fixed some of these issues when Ludo pointed them out. On 2015-10-22 at 08:41, quoth Ricardo Wurmus: > Please use ‘(file-name ...)’ here as the tarball is only named > ‘v2.7.1.tar.gz’. OK, good point. I admit that i wasn't sure about t

[PATCH 1/2] gnu: cmus: Add file-name property.

2015-10-22 Thread Paul van der Walt
* gnu/packages/music.scm (cmus): Add file-name, since the downloaded tar.gz only contains version number. --- gnu/packages/music.scm | 1 + 1 file changed, 1 insertion(+) diff --git a/gnu/packages/music.scm b/gnu/packages/music.scm index 7970d17..c693f0f 100644 --- a/gnu/packages/music.scm +++

Re: [PATCH 03/43] gnu: xmonad: Patch to work on new Haskell Platform

2015-10-22 Thread Paul van der Walt
Hi Eric, On 2015-10-22 at 00:08, quoth Eric Bavier: > I think passing '#:configure-flags '("--allow-newer=utf8-string")' in > arguments might do the trick. This argument doesn't seem to exist, unfortunately. > Otherwise, the "(substitute* ...)" approach, with a comment, seems > clearest to me.

Re: Reviews for wip-haskell-platform-7.10.2-a

2015-10-22 Thread Paul van der Walt
brary search path patch will influence things. > In the absense of comments on a patch, assume a "LGTM!" :) Okay, well, i'll deal with all the comments before merging anything. > Thank you, Paul for all the effort, Seems you went to a lot of effort too, thank you for your thorough reviews! On to a more Haskell-friendly Guix! :) Cheers, p.

Re: [PATCH] gnu: Add ghc-attoparsec.

2015-10-22 Thread Paul van der Walt
On 2015-10-22 at 05:10, quoth ericbav...@openmailbox.org: >> +(arguments `(#:tests? #f)) ; FIXME: ghc-test-framework unavailable >> +(propagated-inputs >> + `(("ghc-scientific" ,ghc-scientific))) >> +(inputs >> + `( >> + ("ghc-text" ,ghc-text) > > Move this onto the previ

Re: [PATCH] gnu: Add ghc-base64-bytestring.

2015-10-22 Thread Paul van der Walt
On 2015-10-22 at 05:10, quoth ericbav...@openmailbox.org: >> +(arguments `(#:tests? #f)) ; FIXME: testing libraries are >> missing. >> +(inputs >> + `(("ghc-quickcheck" ,ghc-quickcheck) >> + ("ghc-hunit" ,ghc-hunit))) > > These inputs could probably be left out if #:test

Re: [PATCH] gnu: Add ghc-blaze-markup.

2015-10-22 Thread Paul van der Walt
On 2015-10-22 at 05:18, quoth ericbav...@openmailbox.org: >> +(arguments `(#:tests? #f)) ; FIXME: testing libraries are missing. >> +(inputs >> + `(("ghc-blaze-builder" ,ghc-blaze-builder) >> + ("ghc-text" ,ghc-text) >> + ("ghc-hunit" ,ghc-hunit) >> + ("ghc-quickcheck

Re: [PATCH] gnu: Add ghc-bytestring-builder.

2015-10-22 Thread Paul van der Walt
On 2015-10-22 at 05:18, quoth ericbav...@openmailbox.org: >> +(arguments `(#:haddock? #f)) ; FIXME: testing framework unavailable > > Did you mean '#:tests? #f'? or is the comment incorrect? Indeed the package contains no documentation. Comment fixed. Also took the opportunity to improve th

Re: [PATCH] gnu: Add ghc-blaze-builder.

2015-10-22 Thread Paul van der Walt
On 2015-10-22 at 05:18, quoth ericbav...@openmailbox.org: >> +(arguments `(#:tests? #f)) ; FIXME: Missing test libraries. >> +(inputs >> + `(("ghc-text" ,ghc-text) >> + ("ghc-hunit" ,ghc-hunit) >> + ("ghc-quickcheck" ,ghc-quickcheck) >> + ("ghc-utf8-string" ,

Re: [PATCH] gnu: Add ghc-cpphs.

2015-10-22 Thread Paul van der Walt
Hi Eric, On 2015-10-22 at 05:25, quoth ericbav...@openmailbox.org: >> +(name "ghc-cpphs") > > We should probably name this package "cpphs". Indeed. I've also renamed ghc-hlint to hlint. >> +(license lgpl2.1))) > > It looks to me as if the library portions are under the lgpl2.1+ (the '+'

Re: [PATCH] gnu: Add ghc-cereal.

2015-10-22 Thread Paul van der Walt
On 2015-10-22 at 05:25, quoth ericbav...@openmailbox.org: >> +(home-page >> + "http://hackage.haskell.org/package/cereal";) >> +(synopsis "Binary serialization library") >> +(description >> + "This package provides a binary serialization library, similar to >> binary, > > It m

Re: [PATCH] gnu: Add ghc-fingertree.

2015-10-22 Thread Paul van der Walt
On 2015-10-22 at 05:32, quoth ericbav...@openmailbox.org: >> +(arguments `(#:tests? #f)) ; FIXME: testing libraries are missing. >> +(inputs >> + `(("ghc-hunit" ,ghc-hunit) >> + ("ghc-quickcheck" ,ghc-quickcheck))) > > Test-only inputs could be left out completely here. Done. >

Re: [PATCH] gnu: Add ghc-free.

2015-10-22 Thread Paul van der Walt
On 2015-10-22 at 05:32, quoth ericbav...@openmailbox.org: >> +(synopsis "Monads for free") > > How about: "Unrestricted monads for Haskell" Done.

Re: [PATCH] gnu: Add ghc-doctest.

2015-10-22 Thread Paul van der Walt
On 2015-10-22 at 05:32, quoth ericbav...@openmailbox.org: > Testing for and with doctest is tricky. I'm not sure your comment about > missing test frameworks is accurate; it looks, from what I can tell, like all > the necessary inputs are present. > > I believe the issue is that tests that call t

Re: [PATCH] gnu: Add ghc-haskell-src-exts.

2015-10-22 Thread Paul van der Walt
On 2015-10-22 at 10:47, quoth ericbav...@openmailbox.org: >> + "Manipulating Haskell source: abstract syntax, lexer, parser, and >> +pretty-printer") > > How about: "Manipulate Haskell source" Done!

Re: [PATCH] gnu: Add ghc-gluraw.

2015-10-22 Thread Paul van der Walt
On 2015-10-22 at 10:47, quoth ericbav...@openmailbox.org: >> +(synopsis >> + "Raw bindings for the OpenGL graphics system") > > How about: "Raw Haskell bindings for GLU" Done!

Re: [PATCH] gnu: Add ghc-clock.

2015-10-22 Thread Paul van der Walt
Hi Eric, On 2015-10-22 at 05:25, quoth ericbav...@openmailbox.org: >> +(define-public ghc-clock >> + (package >> +(name "ghc-clock") >> +(version "0.5.1") >> +(source >> + (origin >> + (method url-fetch) >> + (uri (string-append >> + "http://hackage.haskell

Re: [PATCH] gnu: Add ghc-zip-archive.

2015-10-22 Thread Paul van der Walt
On 2015-10-22 at 10:47, quoth ericbav...@openmailbox.org: >> +(synopsis "Zip archive library") > > Let's distinguish this package with a "for Haskell" in the synopsis. WDYT? Good point, done!

Re: [PATCH] gnu: Add Idris, the dependently-typed language.

2015-10-22 Thread Paul van der Walt
Hi Eric, On 2015-10-22 at 10:47, quoth ericbav...@openmailbox.org: > With our conventions, the subject line should be just "gnu: Add Idris" Awww, no embellishments allowed *sniff*? :) Fixed though. > Try adding "--allow-newer=vector" to #:configure-flags. I tried, this option does not exist.

Re: [PATCH] gnu: Add ghc-xml.

2015-10-22 Thread Paul van der Walt
On 2015-10-22 at 10:47, quoth ericbav...@openmailbox.org: >> +(define-public ghc-xml >> + (package > [...] >> +(inputs `(("ghc-text" ,ghc-text))) > > This input should probably be propagated. Changed.

Re: [PATCH] gnu: Add ghc-semigroups.

2015-10-22 Thread Paul van der Walt
On 2015-10-22 at 10:47, quoth ericbav...@openmailbox.org: >> +(inputs >> + `(("ghc-text" ,ghc-text) >> + ("ghc-hashable" ,ghc-hashable))) > > Do these not need to also be propagated? This is a comment that comes up frequently. To be honest, i don't know how to check. My packaging

Re: [PATCH] gnu: Add ghc-pcre-light.

2015-10-22 Thread Paul van der Walt
On 2015-10-22 at 10:47, quoth ericbav...@openmailbox.org: >> +(synopsis >> + "Efficient and portable Perl-compatible regex library") > > In order to disambiguate with other pcre libraries, how about: > > "Haskell library for Perl 5 compatible regular expressions" > >> +(description >>

Re: [PATCH] gnu: Add ghc-uniplate.

2015-10-22 Thread Paul van der Walt
On 2015-10-22 at 10:47, quoth ericbav...@openmailbox.org: >> +(synopsis >> + "Help writing simple, concise and fast generic operations") > > How about: "Simple, concise, and fast generic operations" Changed. >> +(description "Uniplate is library for writing simple and concise generic

Re: [PATCH] gnu: Add ghc-semigroups.

2015-10-22 Thread Paul van der Walt
On 2015-10-22 at 10:47, quoth ericbav...@openmailbox.org: >> +(define-public ghc-semigroups >> + (package > [...] >> +(inputs >> + `(("ghc-text" ,ghc-text) >> + ("ghc-hashable" ,ghc-hashable))) > > Do these not need to also be propagated? I don't know, but it works for me (includin

Re: [PATCH] gnu: Add ghc-semigroupoids.

2015-10-22 Thread Paul van der Walt
On 2015-10-22 at 10:47, quoth ericbav...@openmailbox.org: >> +(synopsis "Semigroupoids: Category sans id") > > How about: "Semigroupoid operations for Haskell" Fixed.

Re: [PATCH] gnu: Add ghc-regex-tdfa-rc.

2015-10-22 Thread Paul van der Walt
On 2015-10-22 at 10:47, quoth ericbav...@openmailbox.org: >> +(inputs >> + `(("ghc-mtl" ,ghc-mtl))) > > I would be surprised if this did not also need to be propagated, but I confess > I do not know of a reliable way to check. Works for me. Maybe we should look at this stuff after the na

Re: [PATCH] gnu: Add ghc-openglraw.

2015-10-22 Thread Paul van der Walt
On 2015-10-22 at 10:47, quoth ericbav...@openmailbox.org: >> +(synopsis >> + "Raw bindings for the OpenGL graphics system") > > "Raw bindings" -> "Raw Haskell bindings" Done.

Re: [PATCH] gnu: Add ghc-regex-posix.

2015-10-22 Thread Paul van der Walt
On 2015-10-22 at 10:47, quoth ericbav...@openmailbox.org: >> +(synopsis "Replaces/Enhances Text.Regex") > > How about: "POSIX regular expressions for Haskell" Fixed.

Re: [PATCH] gnu: Add ghc-regex-base.

2015-10-22 Thread Paul van der Walt
On 2015-10-22 at 10:47, quoth ericbav...@openmailbox.org: >> +(description >> + "Interface API for regex-posix, pcre, parsec, tdfa, dfa.") > > How about: > > Text.Regex.Base provides the interface API for regex-posix, regex-pcre, > regex-parsec, regex-tdfa, and regex-dfa. Fixed.

Re: [PATCH] gnu: Add ghc-hscolour.

2015-10-22 Thread Paul van der Walt
On 2015-10-22 at 10:47, quoth ericbav...@openmailbox.org: >> +(define-public ghc-hscolour >> + (package >> +(name "ghc-hscolour") > > I would suggest we name this package "hscolour" because it is primarily > program or tool. Indeed, i have renamed it. > It also has libraries; perhaps those

Re: [PATCH] gnu: Add ghc-haddock-api.

2015-10-22 Thread Paul van der Walt
On 2015-10-22 at 10:47, quoth ericbav...@openmailbox.org: >> +(synopsis >> + "Documentation-generation tool for Haskell libraries") >> +(description >> + "Haddock is a documentation-generation tool for Haskell libraries.") > > Could you adjust the synopsis and description to differ

Re: [PATCH] gnu: Add ghc-opengl.

2015-10-22 Thread Paul van der Walt
On 2015-10-22 at 10:47, quoth ericbav...@openmailbox.org: >> +(define-public ghc-opengl >> + (package > [...] >> +(synopsis >> + "Bindings for the OpenGL graphics system") > > "Haskell bindings for the OpenGL graphics system" > >> +(description >> + "A Haskell binding for the Open

Re: [PATCH] gnu: Add ghc-unix-time.

2015-10-22 Thread Paul van der Walt
On 2015-10-22 at 10:47, quoth ericbav...@openmailbox.org: >> + #:phases >> + (alist-cons-before >> +'configure 'fix-/bin/sh >> +(lambda _ >> + ;; Use `sh', not `/bin/sh'. >> + (substitute* (find-files "." "Makefile|configure") >> +(("/bin/s

Re: [PATCH] gnu: Add ghc-hlint.

2015-10-22 Thread Paul van der Walt
On 2015-10-22 at 10:47, quoth ericbav...@openmailbox.org: >> +(define-public ghc-hlint >> + (package >> +(name "ghc-hlint") > > I would suggest calling this package simple "hlint", as it is primarily a > tool. Indeed, i had the same feeling :). Done. >> +(synopsis "Source code suggest

Re: [PATCH] gnu: Add ghc-wai.

2015-10-22 Thread Paul van der Walt
On 2015-10-22 at 10:47, quoth ericbav...@openmailbox.org: >> +(synopsis "WAI: Web Application Interface") > > How about: "Web application interface library for Haskell." > >> +(description "Web Application Interface library for Haskell.") > > and: > > "This package provides a Web Applicati

Re: [PATCH] gnu: Add ghc-reflection.

2015-10-22 Thread Paul van der Walt
On 2015-10-22 at 10:47, quoth ericbav...@openmailbox.org: >> +(synopsis "Reifies arbitrary terms into types that can be reflected back >> +into terms") > > "Reifies" -> "Reify" > >> +(description "This package addresses the 'configuration problem' which >> is >> +propogating configuration

Re: [PATCH] gnu: Add ghc-tasty-golden.

2015-10-22 Thread Paul van der Walt
On 2015-10-22 at 10:47, quoth ericbav...@openmailbox.org: >> +(description >> + "This package provides support for «golden testing». A golden test is >> an > > It might be best to use standard quotes here, rather than the guillemets, and > let translators handle localization. WDYT? I s

Re: [PATCH] gnu: Add ghc-sdl-mixer.

2015-10-22 Thread Paul van der Walt
On 2015-10-22 at 10:47, quoth ericbav...@openmailbox.org: >> + `(#:configure-flags >> + (let* ((sdl-mixer (assoc-ref %build-inputs "sdl-mixer")) >> + (sdl-mixer-lib (string-append sdl-mixer "/lib")) >> + (sdl-mixer-include (string-append sdl-mixer "/include/SDL"

Re: [PATCH] gnu: ghc-split: Update dependencies.

2015-10-22 Thread Paul van der Walt
On 2015-10-22 at 10:47, quoth ericbav...@openmailbox.org: >> +(arguments >> + `(#:phases >> + (modify-phases %standard-phases >> + (add-before >> + 'configure 'update-cabal >> + ;; the cabal file on hackage is updated, but the tar.gz does not >> + ;

Re: [PATCH] gnu: Add ghc-unix-time.

2015-10-22 Thread Paul van der Walt
Hey Andreas, On 2015-10-22 at 18:22, quoth Andreas Enge: > On Thu, Oct 22, 2015 at 05:43:03PM +0200, Paul van der Walt wrote: >> Right, i see what you mean. I agree that it'd probably be the right >> thing to do. However, to expedite the merge of wip-haskell-… i propos

Re: [PATCH] gnu: Add ghc-hscolour.

2015-10-22 Thread Paul van der Walt
On 2015-10-22 at 18:48, quoth Eric Bavier: >> I will defer to your better judgement, but for now i've only changed >> the package name. > > Seems fair. We can revisit later if need be. Thanks for sticking out > the review. Haha, thank you too, it's probably equally tiring :p. p.

Re: [PATCH] gnu: Add Idris, the dependently-typed language.

2015-10-22 Thread Paul van der Walt
On 2015-10-22 at 19:02, quoth Eric Bavier: >>> Try adding "--allow-newer=vector" to #:configure-flags. >> >> I tried, this option does not exist. > > Interesting. Thanks for trying, I honestly haven't used the flag > before, but I saw it in the Cabal documentation, so thought I make the > sugg

Re: [PATCH] gnu: ghc-split: Update dependencies.

2015-10-22 Thread Paul van der Walt
On 2015-10-22 at 19:06, quoth Eric Bavier: > Great! I'll take one last look, then I think we can merge. WDYT? FYI, i've pushed the changes to wip-haskell-… now. Only idris, xmonad, ghc-split are affected compared to the push you were looking at. Thanks again, where do i send the bottle of wine

Re: Dealing with mass rebuilds

2015-10-22 Thread Paul van der Walt
On 2015-10-21 at 18:45, quoth Ludovic Courtès: > The TeX Live issue is orthogonal: We seriously need a ‘texlive-minimal’ > variant, and the full-blown ‘texlive’ must not be depended on by any > other package. Even more orthogonal: can packages be marked as leaves (or roots, depending on your poin

Re: [PATCH] gnu: Add xcompmgr

2015-10-23 Thread Paul van der Walt
Hello Cyrill, Small comment. > From dc8d0477c0644e9d54ebba6af955c3d4e29270fc Mon Sep 17 00:00:00 2001 > From: Cyrill Schenkel > Date: Fri, 23 Oct 2015 13:25:11 +0200 > Subject: [PATCH] gnu: Add xcompmgr > > * gnu/packages/xorg.scm (xcompmgr): New variable. > --- > gnu/packages/xorg.scm | 41 ++

Re: [PATCH] gnu: Add xcompmgr

2015-10-23 Thread Paul van der Walt
...and another thing while i'm at it, although i'm less sure about this one... On 2015-10-23 at 11:46, quoth Paul van der Walt: >> +(setenv "NOCONFIGURE" "t") Does that do what i think it does? If so, why not just use some

Re: [PATCH] gnu: Add xcompmgr

2015-10-23 Thread Paul van der Walt
Hey Cyrill, On 2015-10-23 at 16:45, quoth Cyrill Schenkel: >> On 2015-10-23 at 11:46, quoth Paul van der Walt: >>>> +(setenv "NOCONFIGURE" "t") >> >> Does that do what i think it does? If so, why not just use somet

[PATCH] build-system/haskell: CONFIG_SHELL env variable.

2015-10-23 Thread Paul van der Walt
-1,5 +1,6 @@ ;;; GNU Guix --- Functional package management for GNU ;;; Copyright © 2015 Federico Beffa +;;; Copyright © 2015 Paul van der Walt ;;; ;;; This file is part of GNU Guix. ;;; @@ -96,6 +97,14 @@ and parameters ~s~%" &#

[PATCH] gnu: haskell: Remove CONFIG_SHELL patches.

2015-10-23 Thread Paul van der Walt
A previous commit to the haskell-build-system made sure that if necessary, the CONFIG_SHELL environment variable is set. That obviates the individual patches applied to some packages. Remove them. * gnu/packages/haskell.scm (ghc-network): Remove CONFIG_SHELL patch. * gnu/packages/haskell.scm (gh

Re: [PATCH] build-system/haskell: CONFIG_SHELL env variable.

2015-10-26 Thread Paul van der Walt
Hey Ludo, On 2015-10-25 at 22:58, quoth Ludovic Courtès: >> * gnu/packages/haskell.scm (ghc-network): Remove CONFIG_SHELL patch. >> * gnu/packages/haskell.scm (ghc-old-time): Remove CONFIG_SHELL patch. >> * gnu/packages/haskell.scm (ghc-sdl-image): Remove CONFIG_SHELL patch. >> * gnu/packages/hask

Re: [PATCH 2/2] gnu: Add Mlucas.

2015-10-26 Thread Paul van der Walt
Hello Alex, On 2015-10-25 at 07:12, quoth Alex Vong: > Subject: [PATCH 2/2] gnu: Add Mlucas. > > ... > > * gnu/packages/mlucas.scm: New file. I'm not an expert, and i'm not 100% sure on this so it's an honest question, but does this application really deserve a new package file? Shouldn't mlucas

Re: [PATCH 2/2] gnu: Add Mlucas.

2015-10-26 Thread Paul van der Walt
On 2015-10-26 at 12:49, quoth Alex Vong: > I don't know there is maths.scm! Is there a general guideline on where > to put new package definition? For example, putting elisp packages in > emacs.scm is pretty obvious but how about other cases? I am far from the right person to be answering this qu

Re: State of dbus-update

2015-10-29 Thread Paul van der Walt
On 2015-10-28 at 14:15, quoth Andreas Enge: > On Wed, Oct 28, 2015 at 01:58:44PM +0100, Andreas Enge wrote: >> raincat on x86_64. > > Here a solution is to add freeglut as an input to raincat. > Alternatively, we could propagate the freeglut input of ghc-glut. > What do(es) the specialist(s) think

Re: State of dbus-update

2015-10-29 Thread Paul van der Walt
On 2015-10-29 at 10:59, quoth Paul van der Walt: > On 2015-10-28 at 14:15, quoth Andreas Enge: >> On Wed, Oct 28, 2015 at 01:58:44PM +0100, Andreas Enge wrote: >>> raincat on x86_64. >> >> Here a solution is to add freeglut as an input to raincat. >> Alternativ

Adding xmobar.

2015-11-04 Thread Paul van der Walt
Hello Guix, Long story short, i realised that we are providing xmonad without xmobar. While there is no deep relationship between the two, i like using them together and feel that xmonad without xmobar is weird. I'm sending some patches to add xmobar and dependencies. I hope that they'll be of

[PATCH 03/21] gnu: Add ghc-monads-tf.

2015-11-04 Thread Paul van der Walt
* gnu/packages/haskell.scm (ghc-monads-tf): New variable. --- gnu/packages/haskell.scm | 25 + 1 file changed, 25 insertions(+) diff --git a/gnu/packages/haskell.scm b/gnu/packages/haskell.scm index 5195425..765b669 100644 --- a/gnu/packages/haskell.scm +++ b/gnu/packages/

[PATCH 04/21] gnu: Add ghc-language-haskell-extract.

2015-11-04 Thread Paul van der Walt
* gnu/packages/haskell.scm (ghc-language-haskell-extract): New variable. --- gnu/packages/haskell.scm | 24 1 file changed, 24 insertions(+) diff --git a/gnu/packages/haskell.scm b/gnu/packages/haskell.scm index 765b669..bf2755b 100644 --- a/gnu/packages/haskell.scm +++ b

[PATCH 02/21] gnu: ghc-regex-compat: Propagate inputs.

2015-11-04 Thread Paul van der Walt
* gnu/packages/haskell.scm (ghc-regex-compat): Move ghc-regex-base and ghc-regex-posix to propagated inputs. --- gnu/packages/haskell.scm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gnu/packages/haskell.scm b/gnu/packages/haskell.scm index 0ecd7b1..5195425 100644 --- a/gn

[PATCH 01/21] gnu: ghc-regex-posix: Propagate inputs.

2015-11-04 Thread Paul van der Walt
* gnu/packages/haskell.scm (ghc-regex-posix): Move ghc-regex-base to propagated inputs. --- gnu/packages/haskell.scm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gnu/packages/haskell.scm b/gnu/packages/haskell.scm index b9387f8..0ecd7b1 100644 --- a/gnu/packages/haskell.sc

[PATCH 11/21] gnu: Add ghc-options.

2015-11-04 Thread Paul van der Walt
* gnu/packages/haskell.scm (ghc-options): New variable. --- gnu/packages/haskell.scm | 28 1 file changed, 28 insertions(+) diff --git a/gnu/packages/haskell.scm b/gnu/packages/haskell.scm index 03a8e37..42e3a74 100644 --- a/gnu/packages/haskell.scm +++ b/gnu/packages

[PATCH 10/21] gnu: Add ghc-lifted-async.

2015-11-04 Thread Paul van der Walt
* gnu/packages/haskell.scm (ghc-lifted-async): New variable. --- gnu/packages/haskell.scm | 34 ++ 1 file changed, 34 insertions(+) diff --git a/gnu/packages/haskell.scm b/gnu/packages/haskell.scm index 88a7fe5..03a8e37 100644 --- a/gnu/packages/haskell.scm +++ b/g

[PATCH 08/21] gnu: Add ghc-constraints.

2015-11-04 Thread Paul van der Walt
* gnu/packages/haskell.scm (ghc-constraints): New variable. --- gnu/packages/haskell.scm | 23 +++ 1 file changed, 23 insertions(+) diff --git a/gnu/packages/haskell.scm b/gnu/packages/haskell.scm index 44310de..af5a330 100644 --- a/gnu/packages/haskell.scm +++ b/gnu/packages/

[PATCH 12/21] gnu: Add ghc-chell.

2015-11-04 Thread Paul van der Walt
* gnu/packages/haskell.scm (ghc-chell): New variable. --- gnu/packages/haskell.scm | 29 + 1 file changed, 29 insertions(+) diff --git a/gnu/packages/haskell.scm b/gnu/packages/haskell.scm index 42e3a74..35e71cc 100644 --- a/gnu/packages/haskell.scm +++ b/gnu/packages/

[PATCH 14/21] gnu: Add ghc-system-filepath.

2015-11-04 Thread Paul van der Walt
* gnu/packages/haskell.scm (ghc-system-filepath): New variable. --- gnu/packages/haskell.scm | 27 +++ 1 file changed, 27 insertions(+) diff --git a/gnu/packages/haskell.scm b/gnu/packages/haskell.scm index ad94ca4..fd782c1 100644 --- a/gnu/packages/haskell.scm +++ b/gnu/p

[PATCH 16/21] gnu: Add ghc-enclosed-exceptions.

2015-11-04 Thread Paul van der Walt
* gnu/packages/haskell.scm (ghc-enclosed-exceptions): New variable. --- gnu/packages/haskell.scm | 29 + 1 file changed, 29 insertions(+) diff --git a/gnu/packages/haskell.scm b/gnu/packages/haskell.scm index 323e1a3..6f956d7 100644 --- a/gnu/packages/haskell.scm +++ b

[PATCH 21/21] gnu: Add xmobar.

2015-11-04 Thread Paul van der Walt
* gnu/packages/wm.scm (xmobar): New variable. --- gnu/packages/wm.scm | 44 1 file changed, 44 insertions(+) diff --git a/gnu/packages/wm.scm b/gnu/packages/wm.scm index db5fa1e..4b1d3b1 100644 --- a/gnu/packages/wm.scm +++ b/gnu/packages/wm.scm @@ -31

[PATCH 19/21] gnu: Add ghc-alsa-core.

2015-11-04 Thread Paul van der Walt
* gnu/packages/haskell.scm (ghc-alsa-core): New variable. --- gnu/packages/haskell.scm | 28 1 file changed, 28 insertions(+) diff --git a/gnu/packages/haskell.scm b/gnu/packages/haskell.scm index da8e932..90171e7 100644 --- a/gnu/packages/haskell.scm +++ b/gnu/packag

[PATCH 15/21] gnu: Add ghc-system-fileio.

2015-11-04 Thread Paul van der Walt
* gnu/packages/haskell.scm (ghc-system-fileio): New variable. --- gnu/packages/haskell.scm | 27 +++ 1 file changed, 27 insertions(+) diff --git a/gnu/packages/haskell.scm b/gnu/packages/haskell.scm index fd782c1..323e1a3 100644 --- a/gnu/packages/haskell.scm +++ b/gnu/pac

[PATCH 05/21] gnu: Add ghc-tasty-th.

2015-11-04 Thread Paul van der Walt
* gnu/packages/haskell.scm (ghc-tasty-th): New variable. --- gnu/packages/haskell.scm | 28 1 file changed, 28 insertions(+) diff --git a/gnu/packages/haskell.scm b/gnu/packages/haskell.scm index bf2755b..f66baf0 100644 --- a/gnu/packages/haskell.scm +++ b/gnu/package

[PATCH 20/21] gnu: Add ghc-alsa-mixer.

2015-11-04 Thread Paul van der Walt
* gnu/packages/haskell.scm (ghc-alsa-mixer): New variable. --- gnu/packages/haskell.scm | 24 1 file changed, 24 insertions(+) diff --git a/gnu/packages/haskell.scm b/gnu/packages/haskell.scm index 90171e7..889cd55 100644 --- a/gnu/packages/haskell.scm +++ b/gnu/packages/

[PATCH 17/21] gnu: Add ghc-shelly.

2015-11-04 Thread Paul van der Walt
* gnu/packages/haskell.scm (ghc-shelly): New variable. --- gnu/packages/haskell.scm | 39 +++ 1 file changed, 39 insertions(+) diff --git a/gnu/packages/haskell.scm b/gnu/packages/haskell.scm index 6f956d7..76ff287 100644 --- a/gnu/packages/haskell.scm +++ b/gn

[PATCH 13/21] gnu: Add ghc-chell-quickcheck.

2015-11-04 Thread Paul van der Walt
* gnu/packages/haskell.scm (ghc-chell-quickcheck): New variable. --- gnu/packages/haskell.scm | 25 + 1 file changed, 25 insertions(+) diff --git a/gnu/packages/haskell.scm b/gnu/packages/haskell.scm index 35e71cc..ad94ca4 100644 --- a/gnu/packages/haskell.scm +++ b/gnu/pa

[PATCH 18/21] gnu: Add c2hs.

2015-11-04 Thread Paul van der Walt
* gnu/packages/haskell.scm (c2hs): New variable. --- gnu/packages/haskell.scm | 35 ++- 1 file changed, 34 insertions(+), 1 deletion(-) diff --git a/gnu/packages/haskell.scm b/gnu/packages/haskell.scm index 76ff287..da8e932 100644 --- a/gnu/packages/haskell.scm +++

[PATCH 06/21] gnu: Add ghc-newtype.

2015-11-04 Thread Paul van der Walt
* gnu/packages/haskell.scm (ghc-newtype): New variable. --- gnu/packages/haskell.scm | 23 +++ 1 file changed, 23 insertions(+) diff --git a/gnu/packages/haskell.scm b/gnu/packages/haskell.scm index f66baf0..90e0a16 100644 --- a/gnu/packages/haskell.scm +++ b/gnu/packages/hask

[PATCH 07/21] gnu: Add ghc-patience.

2015-11-04 Thread Paul van der Walt
* gnu/packages/haskell.scm (ghc-patience): New variable. --- gnu/packages/haskell.scm | 25 + 1 file changed, 25 insertions(+) diff --git a/gnu/packages/haskell.scm b/gnu/packages/haskell.scm index 90e0a16..44310de 100644 --- a/gnu/packages/haskell.scm +++ b/gnu/packages/h

[PATCH 09/21] gnu: Add ghc-language-c.

2015-11-04 Thread Paul van der Walt
* gnu/packages/haskell.scm (ghc-language-c): New variable. --- gnu/packages/haskell.scm | 28 1 file changed, 28 insertions(+) diff --git a/gnu/packages/haskell.scm b/gnu/packages/haskell.scm index af5a330..88a7fe5 100644 --- a/gnu/packages/haskell.scm +++ b/gnu/packa

<    1   2   3   4   5   6   >