Re: [PATCH] Add libtermkey, vis and lua-lpeg packages

2016-12-02 Thread jmi2k
On 2016-12-02 23:45, jm...@openmailbox.org wrote: Hello, I made a patch to add vis and its dependencies: libtermkey and lua-lpeg; to Guix. Some things have to be polished, but the packages work fine. José Miguel Sánchez García Sorry, I made a mistake. Here is the correct version.From

[PATCH] Add libtermkey, vis and lua-lpeg packages

2016-12-02 Thread jmi2k
Hello, I made a patch to add vis and its dependencies: libtermkey and lua-lpeg; to Guix. Some things have to be polished, but the packages work fine. José Miguel Sánchez GarcíaFrom 28e6b5b08e58c3227d4b6a9af3fb63bfed6d3323 Mon Sep 17 00:00:00 2001 From:

Re: [PATCH] system: Remove spec->file-system.

2016-12-02 Thread David Craven
The reason I'm suggesting to remove this function, is because it seems like any case one would want to use it, one should not be using it. There is a separation of gnu/system and gnu/build and a spec->file-system function encourages people to try using the in the build part. I think that it's a

Re: [PATCH] gnu: cairomm: Update to 1.15.1.

2016-12-02 Thread Leo Famulari
On Fri, Dec 02, 2016 at 12:08:09PM +0100, Marius Bakke wrote: > The gnome mirrors appears to have stopped updating this package at > 1.12.0. This is the latest version from cairographics.org. > > From ed1c762e210eb9dfe6845d751b79b234edb6183c Mon Sep 17 00:00:00 2001 > From: Marius Bakke

[PATCH] system: Remove spec->file-system.

2016-12-02 Thread David Craven
* gnu/system/file-systems.scm (spec->file-system): Remove variable. * gnu/system/linux-container.scm (container-script): Refactor. --- gnu/system/file-systems.scm| 11 --- gnu/system/linux-container.scm | 6 ++ 2 files changed, 2 insertions(+), 15 deletions(-) diff --git

Re: [PATCH 1/1] gnu: tzdata: Update to 2016j.

2016-12-02 Thread Leo Famulari
On Fri, Dec 02, 2016 at 11:54:38AM +0100, Marius Bakke wrote: > Leo Famulari writes: > > > * gnu/packages/base.scm (tzdata): Update to 2016j. > > --- > > gnu/packages/base.scm | 6 +++--- > > 1 file changed, 3 insertions(+), 3 deletions(-) > > LGTM! Can probably be snuck in

[PATCH] gnu: python-pyqt: Fix build by explicitly setting the stubsdir.

2016-12-02 Thread Hartmut Goebel
Without this option the stub files are tried to be installed into the python package's site-package directory. * gnu/packages.qt.scm (python-pyqt, python2-pyqt)[arguments]: In phase 'configure' pass option --stubsdir. --- gnu/packages/qt.scm | 4 +++- 1 file changed, 3 insertions(+), 1

Re: [PATCH 0/5] Fixes for packages with new python build sytem

2016-12-02 Thread Hartmut Goebel
Am 02.12.2016 um 16:45 schrieb Marius Bakke: > > Thanks! These LGTM. Pushed as 447b947. Thansk for the quick review. -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thought are impossible |

Re: [PATCH] gnu: file-system-shepherd-service: Use mount-file-system.

2016-12-02 Thread David Craven
> Perhaps somebody who already has a copy of qemu in their store could check > that > for me? I tested it and pushed...

Re: [PATCH 2/2] system: Add btrfs file system support.

2016-12-02 Thread David Craven
> How to handle the return codes. This isn't a problem, the check-procedure can normalize the return codes. The confusing thing is this part: (else #~(system* (string-append "fsck." type) device) The default behavior should be to emit a warning to stderr and do nothing. I don't think this

[PATCH] gnu: Patch libxt's default search path.

2016-12-02 Thread John Darrington
* gnu/packages/patches/libxt-guix-search-paths.patch: New file. * gnu/packages/xorg.scm (libxt) [source]: Add patch. --- gnu/packages/patches/libxt-guix-search-paths.patch | 123 + gnu/packages/xorg.scm | 3 +- 2 files changed, 125 insertions(+),

Re: [PATCH] gnu: Add mktorrent.

2016-12-02 Thread Tobias Geerinckx-Rice
Marius, On 02/12/16 17:09, Marius Bakke wrote: > LGTM, but the SHA1 implementation that's used in absence of OpenSSL is > public-domain. Good catch, thanks. Kind regards, T G-R signature.asc Description: OpenPGP digital signature

Re: [PATCH] gnu: Add mktorrent.

2016-12-02 Thread Marius Bakke
Tobias Geerinckx-Rice writes: > * gnu/packages/bittorrent.scm (mktorrent): New variable. > --- > > ‘guix lint’ — sorry: @code{guix lint} complained about quotes, so I > cheated a bit and went with @dfn tags instead. It's a sort of mutual > equality definition, after all. > > Kind

[PATCH] gnu: Add mktorrent.

2016-12-02 Thread Tobias Geerinckx-Rice
* gnu/packages/bittorrent.scm (mktorrent): New variable. --- ‘guix lint’ — sorry: @code{guix lint} complained about quotes, so I cheated a bit and went with @dfn tags instead. It's a sort of mutual equality definition, after all. Kind regards, T G-R gnu/packages/bittorrent.scm | 34

Re: [PATCH 0/5] Fixes for packages with new python build sytem

2016-12-02 Thread Marius Bakke
Hartmut Goebel writes: > This patches fixes some build issues caused by using the new python build > system. > > Hartmut Goebel (5): > gnu: calibre: Do not use python setuptools for building. > gnu: python-numpydoc: Correct inputs. > gnu: python2-ipython: Fix

[PATCH 0/5] Fixes for packages with new python build sytem

2016-12-02 Thread Hartmut Goebel
This patches fixes some build issues caused by using the new python build system. Hartmut Goebel (5): gnu: calibre: Do not use python setuptools for building. gnu: python-numpydoc: Correct inputs. gnu: python2-ipython: Fix inputs. gnu: python-numpy: Fix inputs. gnu: dblatex: Update

[PATCH 5/5] gnu: dblatex: Update comment.

2016-12-02 Thread Hartmut Goebel
* gnu/packages/docbook.scm (dblatex)[arguments]: Detail comment about why the use of setuptools needs to be disabled. --- gnu/packages/docbook.scm | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/gnu/packages/docbook.scm b/gnu/packages/docbook.scm index

[PATCH 2/5] gnu: python-numpydoc: Correct inputs.

2016-12-02 Thread Hartmut Goebel
This does what 213d1745c6adbfd274b1edc575529e026b0ab743 was supposed to do. * gnu/packages/python.scm (python-numpydoc, python-numpydoc) Change first of the duplicate [native-inputs] definitions to [propagated-inputs]. --- gnu/packages/python.scm | 2 +- 1 file changed, 1 insertion(+), 1

[PATCH 1/5] gnu: calibre: Do not use python setuptools for building.

2016-12-02 Thread Hartmut Goebel
* gnu/packages/ebook.scm (calibre)[arguments]: Set #:use-setuptools? to false. --- gnu/packages/ebook.scm | 3 +++ 1 file changed, 3 insertions(+) diff --git a/gnu/packages/ebook.scm b/gnu/packages/ebook.scm index ad18716..3843fb2 100644 --- a/gnu/packages/ebook.scm +++ b/gnu/packages/ebook.scm

Re: gtk-doc build failure

2016-12-02 Thread Hartmut Goebel
Am 01.12.2016 um 23:51 schrieb Marius Bakke: > I've committed a workaround in 3cc0342b45406f97d675266cc9ac7877be855122. > […] > -package_base = > r"gnu/store/k6211w9hlqyxvzhrsmh2a113rnzdqlkc-dblatex-0.3.5/share/dblatex" > +package_base = >

Re: [PATCH] gnu: mesa: Update to 13.0.1.

2016-12-02 Thread Marius Bakke
Ludovic Courtès writes: >>> The parts about apply the MIPS patch is good to me. Perhaps you can >>> already push it as a separate patch to ‘staging’. >> >> I can submit an update for 12.0.4 if the major version bump is too risky >> right now, or bump to 13.0.2 which was released

Re: RFH: Add prosody service

2016-12-02 Thread Clément Lassieur
Hi Ludovic, >> Here are my problems. >> >> A. A virtualhost-configuration has a lot in common with >>prosody-configuration, and I don't want to repeat stuff. > > What about a record that would have > ‘global-1’…‘global-N’ fields, plus a ‘virtual-host-settings’ that would > aggregate a ?

Re: [PATCH 2/2] system: Add btrfs file system support.

2016-12-02 Thread Chris Marusich
David Craven writes: > Ludo: Is there any way we can speed up the installer system tests? :) I've also wondered about this. I'm not sure how long each test takes to run, so an analysis of the tests to find their slow areas might help guide us. I suspect that we could probably

[PATCH] gnu: cairomm: Update to 1.15.1.

2016-12-02 Thread Marius Bakke
The gnome mirrors appears to have stopped updating this package at 1.12.0. This is the latest version from cairographics.org. >From ed1c762e210eb9dfe6845d751b79b234edb6183c Mon Sep 17 00:00:00 2001 From: Marius Bakke Date: Wed, 30 Nov 2016 19:08:38 +0100 Subject: [PATCH]

Re: [PATCH 0/2] Move image viewers to (gnu packages image-viewers).

2016-12-02 Thread Marius Bakke
Alex Kost writes: > As discussed around here: > , > these patches move image viewers to their module. Also exiv2 is moved > to (gnu packages image). > > [PATCH 1/2] gnu: exiv2: Move to (gnu packages image). >

Re: [PATCH 1/1] gnu: tzdata: Update to 2016j.

2016-12-02 Thread Marius Bakke
Leo Famulari writes: > * gnu/packages/base.scm (tzdata): Update to 2016j. > --- > gnu/packages/base.scm | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) LGTM! Can probably be snuck in on staging before freeze. Could you switch source and home-page to HTTPS as

Re: [PATCH 2/2] system: Add btrfs file system support.

2016-12-02 Thread David Craven
I looked at your improvement suggestion. One issue I'm thinking about: How to handle the return codes, since the fsck.vfat returns slightly different error codes than fsck.ext. For example fsck.vfat error code 2 is a usage error, which would cause an infinite reboot cycle if the passed device is

[PATCH 2/2] gnu: Move image viewers to (gnu packages image-viewers).

2016-12-02 Thread Alex Kost
* gnu/packages/feh.scm (feh): Remove file. Move package to... * gnu/packages/geeqie.scm (geeqie): Likewise... * gnu/packages/sxiv.scm (sxiv): Likewise... * gnu/packages/image-viewers.scm: ... here. * gnu/local.mk (GNU_SYSTEM_MODULES): Remove files. --- gnu/local.mk | 3 -

[PATCH 0/2] Move image viewers to (gnu packages image-viewers).

2016-12-02 Thread Alex Kost
As discussed around here: , these patches move image viewers to their module. Also exiv2 is moved to (gnu packages image). [PATCH 1/2] gnu: exiv2: Move to (gnu packages image). [PATCH 2/2] gnu: Move image viewers to (gnu

[PATCH 1/2] gnu: exiv2: Move to (gnu packages image).

2016-12-02 Thread Alex Kost
* gnu/packages/geeqie.scm (exiv2): Move to... * gnu/packages/image.scm: ... here. * gnu/packages/gnome.scm: Adjust the used modules. * gnu/packages/gnunet.scm: Likewise. * gnu/packages/gps.scm: Likewise. --- gnu/packages/geeqie.scm| 33 -

[PATCH 1/1] gnu: tzdata: Update to 2016j.

2016-12-02 Thread Leo Famulari
* gnu/packages/base.scm (tzdata): Update to 2016j. --- gnu/packages/base.scm | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/gnu/packages/base.scm b/gnu/packages/base.scm index 1c01874bb..545da2729 100644 --- a/gnu/packages/base.scm +++ b/gnu/packages/base.scm @@ -939,7