Fixing broken pacakges

2016-12-13 Thread Pjotr Prins
I just wrote a section on how you can fix problems with a build by using -K and git. The example is CMake here, but the strategy is generic: https://github.com/pjotrp/guix-notes/blob/master/HACKING.org#fixing-problems Git can help create a patch which can be used in Guix to patch the source.

[PATCH 1/2] gnu: Add r-bit.

2016-12-13 Thread Ben Woodcroft
* gnu/packages/statistics.scm (r-bit): New variable. --- gnu/packages/statistics.scm | 23 +++ 1 file changed, 23 insertions(+) diff --git a/gnu/packages/statistics.scm b/gnu/packages/statistics.scm index 36f1889ef..ea9ef0e39 100644 --- a/gnu/packages/statistics.scm +++ b/gnu/

[PATCH 2/2] gnu: Add r-bit64.

2016-12-13 Thread Ben Woodcroft
* gnu/packages/statistics.scm (r-bit64): New variable. --- gnu/packages/statistics.scm | 26 ++ 1 file changed, 26 insertions(+) diff --git a/gnu/packages/statistics.scm b/gnu/packages/statistics.scm index ea9ef0e39..8af94e6af 100644 --- a/gnu/packages/statistics.scm +++ b

Re: [PATCH] build coqide

2016-12-13 Thread julien lepiller
Le 2016-12-13 03:19, Eric Bavier a écrit : On Mon, 12 Dec 2016 23:57:35 +0100 Julien Lepiller wrote: hi, here are a few ocaml-related patches. The first one adds a native-search-path so ocaml-findlib can find the libraries it installs. Next, lablgtk is updated and installed using the recommen

Re: Fixing broken pacakges

2016-12-13 Thread Chris Marusich
On Tue, Dec 13, 2016, 01:10 Pjotr Prins wrote: > I just wrote a section on how you can fix problems with a build by > using -K and git. The example is CMake here, but the strategy is > generic: > > > https://github.com/pjotrp/guix-notes/blob/master/HACKING.org#fixing-problems > > Git can help cre

Re: NetworkManager doesn't "Connect automatically"

2016-12-13 Thread Thomas Haller
On Mon, 2016-12-12 at 22:51 -0800, Chris Marusich wrote: Hi, > * If I check the "Make available to other users" check-box, then >   NetworkManager DOES automatically connect to my wireless network. >   However, my understanding is that this is only a workaround, and > that >   in fact NetworkMa

Re: NetworkManager doesn't "Connect automatically"

2016-12-13 Thread rennes
Hello, On 2016-12-13 08:37, Thomas Haller wrote: This restricts the connection to a certain user, determined by the connection.permissions property (see `nmcli connection show $NAME` and `man nm-settings`). This makes the connection only available, when a session for that user exists. Is that u

Re: [PATCH 2/2] gnu: Add r-bit64.

2016-12-13 Thread Ricardo Wurmus
Hi Ben, > * gnu/packages/statistics.scm (r-bit64): New variable. > --- > gnu/packages/statistics.scm | 26 ++ > 1 file changed, 26 insertions(+) > > diff --git a/gnu/packages/statistics.scm b/gnu/packages/statistics.scm > index ea9ef0e39..8af94e6af 100644 > --- a/gnu/pack

Re: [PATCH 1/2] gnu: Add r-bit.

2016-12-13 Thread Ricardo Wurmus
Hi Ben, thanks for the patch! > * gnu/packages/statistics.scm (r-bit): New variable. > --- > gnu/packages/statistics.scm | 23 +++ > 1 file changed, 23 insertions(+) > > diff --git a/gnu/packages/statistics.scm b/gnu/packages/statistics.scm > index 36f1889ef..ea9ef0e39 10064

Berlin Guix gathering: notes!

2016-12-13 Thread Ricardo Wurmus
Hi Guix! Yesterday a couple of Guix hackers got together in Berlin to discuss a wide range of Guix-related topics. Attached are the notes I took during the meeting. If anything is unclear about these notes please just ask (while the meeting is still fresh in our minds). Hope you’ll find this in

Re: [PATCH 1/7] build-system: Add cargo build system.

2016-12-13 Thread Ludovic Courtès
Hi David! David Craven skribis: >> Given the work that has gone into these Rust patches, I think it would >> be nice to apply them and possibly document any shortcoming or future >> work items. > > I went over all your previous emails and fixed your previous comments. > Those aren't part of the

Re: [PATCH 01/12] upstream: Use a the first url from urls when find2 returns #f.

2016-12-13 Thread Ludovic Courtès
David Craven skribis: > * guix/upstream.scm (package-update): Use a url from the list when the > find2 procedure doesn't find a url sig-url pair. > --- > guix/upstream.scm | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/guix/upstream.scm b/guix/upstream.scm > index 868

Re: [PATCH 02/12] build-system: Add cargo build system.

2016-12-13 Thread Ludovic Courtès
David Craven skribis: > * guix/build-system/cargo.scm: New file. > * guix/build/cargo-build-system.scm: New file. Along with gnu/local.mk please. OK with this change! Ludo’.

Re: [PATCH 03/12] import: utils: Add some utilities.

2016-12-13 Thread Ludovic Courtès
David Craven skribis: > * gnu/import/utils.scm (json-fetch, maybe-inputs, maybe-native-inputs, ^^^ guix/ > package->definition): New variables. IIRC there was an argument against adding ‘json-fetch’ in this module earlier this year, because that would make Guile-JSON mandatory. Could If

Re: [PATCH 04/12] import: Add importer for rust crates.

2016-12-13 Thread Ludovic Courtès
David Craven skribis: > * guix/import/crate.scm: New file. > * guix/scripts/import/crate.scm: New file. > * guix/scripts/import.scm (importers): Add crate importer. > * tests/crate.scm: New file. > * doc/guix.texi: Add crate importer to table. Please add the new files in Makefile.am, local.mk, e

Re: [PATCH 05/12] import: Add updater for rust crates.

2016-12-13 Thread Ludovic Courtès
David Craven skribis: > * guix/import/crate.scm (crate-package?, latest-release, > %crate-updater): New variables. > * guix/scripts/refresh.scm (%updaters): Add crate updater. Please add a line under “Invoking guix refresh” in guix.texi, like we did for the other updaters. OK with this change

Re: [PATCH 1/7] build-system: Add cargo build system.

2016-12-13 Thread David Craven
> Hmm why is there a second one? ‘gnu-build-system’ already provides gcc > and gcc:lib as implicit inputs, so maybe it’s just a matter of removing > ‘gcc’ from Rust’s ‘inputs’? So we do have an implicit gcc:lib package as an input. But I have to patchelf the rustc binaries and libraries so I need

Re: [PATCH 03/12] import: utils: Add some utilities.

2016-12-13 Thread David Craven
> IIRC there was an argument against adding ‘json-fetch’ in this module > earlier this year, because that would make Guile-JSON mandatory. Yep. This is a mistake. I thought that you suggested using http-fetch to avoid pulling in guile-json, but that was because of the temporary file thing. I think

Re: Berlin Guix gathering: notes!

2016-12-13 Thread Christopher Allan Webber
Ricardo Wurmus writes: > Hi Guix! > > Yesterday a couple of Guix hackers got together in Berlin to discuss a > wide range of Guix-related topics. Attached are the notes I took during > the meeting. If anything is unclear about these notes please just ask > (while the meeting is still fresh in ou

Re: [PATCH 01/12] upstream: Use a the first url from urls when find2 returns #f.

2016-12-13 Thread David Craven
> Thanks, and sorry for taking long for just one line! No problem. That's what code reviews are for... I removed my change and inserted (display urls) (display signature-urls) (display url) (display signature-url) here are the results: urls: (https://crates.io/api/v1

[PATCH] make nginx service extensible

2016-12-13 Thread Julien Lepiller
Hi, here are three patches for the nginx service. I finally worked on them. 1. s/vhost/server/ in the web.scm and guix.texi 2. fix a mistake with the list of names and index. 3. make the service extensible, change the default options and document it. I hope that's correct now :)>From 52f220a2

Help acme-client find its certificate store

2016-12-13 Thread Leo Famulari
I'm successfully using this package with this change. I don't know if this is the "right way" to solve this. Your thoughts? From a732b5ffaae2b80496d9c7fc5f957c69919072e5 Mon Sep 17 00:00:00 2001 From: Leo Famulari Date: Mon, 12 Dec 2016 21:04:43 -0500 Subject: [PATCH] gnu: acme-client: Provide pa

Re: [PATCH 11/11] gnu: Add pagure.

2016-12-13 Thread Ricardo Wurmus
Hi ng0, thanks for the patch set! > * gnu/packages/version-control.scm (pagure): New variable. > --- > gnu/packages/version-control.scm | 81 > +++- > 1 file changed, 80 insertions(+), 1 deletion(-) > > diff --git a/gnu/packages/version-control.scm > b/gnu/

Re: [PATCH 1/7] build-system: Add cargo build system.

2016-12-13 Thread Ludovic Courtès
David Craven skribis: >> Hmm why is there a second one? ‘gnu-build-system’ already provides gcc >> and gcc:lib as implicit inputs, so maybe it’s just a matter of removing >> ‘gcc’ from Rust’s ‘inputs’? > > So we do have an implicit gcc:lib package as an input. But I have to patchelf > the rustc

Re: [PATCH 01/12] upstream: Use a the first url from urls when find2 returns #f.

2016-12-13 Thread Ludovic Courtès
David Craven skribis: >> Thanks, and sorry for taking long for just one line! > > No problem. That's what code reviews are for... > > I removed my change and inserted >(display urls) >(display signature-urls) >(display url) >(display signature-url) > > here are the

Re: NetworkManager doesn't "Connect automatically"

2016-12-13 Thread Ludovic Courtès
Hello network managers! :-) ren...@openmailbox.org skribis: > Accordig to what Thomas comments, I have two ideas: > > * I wil try to compile 'network-manager' with > '--with-session-tracking=consolekit'. On GuixSD we use “elogind”, which is like “systemd-logind”. However, the build log says:

Re: [PATCH 07/12] gnu: Add rust bootstrap binaries.

2016-12-13 Thread Ludovic Courtès
David Craven skribis: > * gnu/packages/rust.scm: New file. > * gnu/local.mk (GNU_SYSTEM_MODULES): Add rust.scm. There are a couple of comments from that remain valid, I think. Could you double-check? > +(define (system->rust

Removing the python-updates branch [was Re: bug#25177: Test failures don't cause some Python packages to fail]

2016-12-13 Thread Leo Famulari
On Tue, Dec 13, 2016 at 10:34:03AM +0100, Marius Bakke wrote: > Sounds like we're going down the same road. I've started a branch with > the earlier patch and a few other fixes. Is it ok to overwrite the > existing 'python-updates' branch on Savannah? I'm going to delete that old branch tomorrow i

Re: [PATCH 09/12] gnu: Add rust-libc.

2016-12-13 Thread Ludovic Courtès
David Craven skribis: > * gnu/packages/rust.scm (rust-libc): New variable. [...] > +(license (list license:expat license:asl2.0 Please clarify what this means. Otherwise LGTM, thanks! Ludo’.

Re: [PATCH 08/12] gnu: Add rustc.

2016-12-13 Thread Ludovic Courtès
David Craven skribis: > * gnu/packages/rust.scm (rustc): New variable. > > Co-authored-by: Eric Le Bihan [...] > +(synopsis "Compiler for the Rust progamming language") > +(description "Rust is a systems programming language that runs blazingly > +fast, prevents segfaults, and guarante

Re: [PATCH 06/12] gnu: llvm: Enable install utils.

2016-12-13 Thread Ludovic Courtès
David Craven skribis: > * gnu/packages/llvm.scm (llvm)[arguments]: Add -DLLVM_INStALL_UTILS=ON > to configure-flags. > --- > gnu/packages/llvm.scm | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/gnu/packages/llvm.scm b/gnu/packages/llvm.scm > index 96679bf97..c425108

Re: [PATCH 12/12] gnu: Add cargo.

2016-12-13 Thread Ludovic Courtès
David Craven skribis: > * gnu/packages/rust.scm (cargo): New variable. LGTM, thanks! Ludo’.

Re: [PATCH 11/12] gnu: Add rust-rand.

2016-12-13 Thread Ludovic Courtès
David Craven skribis: > * gnu/packages/rust.scm (rust-rand): New variable. [...] > +(synopsis "Random number generators") > +(description "Random number generators and other randomness > functionality.") Full sentence please! > +(license (list license:expat license:asl2.0 Sa

Re: [PATCH] services: mysql: Add port to configuration

2016-12-13 Thread Ludovic Courtès
Christopher Baines skribis: > * gnu/services/databases.scm (): Add port field. > (mysql-configuration-file): Use the port field when creating the > configuration file. I added a guix.texi item and committed. Thanks! Ludo’.

Re: [PATCH 10/12] RECURSIVE IMPORTER wip

2016-12-13 Thread Ludovic Courtès
I’m punting on this one because this has to be synchronized with other people doing importer stuff. Ludo’.

Re: [PATCH] gnu: Add system test for the rpcbind-daemon service.

2016-12-13 Thread Ludovic Courtès
John Darrington skribis: > * gnu/tests/nfs.scm: New file. > * gnu/local.mk: Add it. Awesome! :-) > +(define %base-os > + (operating-system > +(host-name "olitupmok") > +(timezone "Europe/Berlin") > +(locale "en_US.UTF-8") > + > +(bootloader (grub-configuration (device "/dev/sd

Re: [PATCH 2/2] services: postgresql: Add locale to configuration

2016-12-13 Thread Ludovic Courtès
Christopher Baines skribis: > On 11/12/16 23:14, Ludovic Courtès wrote: >> Hi! >> >> Christopher Baines skribis: >> >>> * gnu/services/postgresql.scm (): Add locale >>> field. >>> (postgresql-shepherd-service): Pass locale to initdb. >>> (postgresql-service): Add locale default. >> >> [...

[PATCH] Elpa importer improvements

2016-12-13 Thread Carlo Zancanaro
I was trying to import some elpa packages recently and found some issues, so here are some patches to fix them. 1. call-with-downloaded-file had behaviour different to how it was documented to behave. This was primarily a problem when trying to import packages with no description (eg. color-th