Re: [PATCH 01/31] gnu: qtbase: Add native search paths.

On Sat, Aug 27, 2016 at 08:21:55AM +0200, Ricardo Wurmus wrote: > > David Craven writes: > > > * gnu/packages/qt.scm (qtbase)[native-search-paths]: > > Add QML2_IMPORT_PATH. > > Add QT_PLUGIN_PATH. > > Add XDG_DATA_DIRS. > > Add XDG_CONFIG_DIRS. > > This looks good to me.

[PATCH] gnu: Add libnfsidmap

* gnu/packages/linux.scm (libnfsidmap): New variable. --- gnu/packages/linux.scm | 25 + 1 file changed, 25 insertions(+) diff --git a/gnu/packages/linux.scm b/gnu/packages/linux.scm index 1fd792d..1173550 100644 --- a/gnu/packages/linux.scm +++ b/gnu/packages/linux.scm

Re: [PATCH] Fix compiling on CentOS 7.

Roel Janssen writes: > Dear Guix, > > Due to an old Automake version (1.13), running the `./configure' phase on > CentOS 7 fails with: > >> autoreconf: running: automake --add-missing --copy --force-missing >> configure.ac:21: warning: The 'AM_PROG_MKDIR_P' macro is deprecated,

Re: [PATCH 01/31] gnu: qtbase: Add native search paths.

David Craven writes: > * gnu/packages/qt.scm (qtbase)[native-search-paths]: > Add QML2_IMPORT_PATH. > Add QT_PLUGIN_PATH. > Add XDG_DATA_DIRS. > Add XDG_CONFIG_DIRS. This looks good to me. Thank you! Will this result in mass rebuilds? If so this should be pushed to

Re: [PATCH 04/31] gnu: Add baloo.

David Craven writes: > * gnu/packages/kde-frameworks.scm (baloo): New variable. […] > +(license (list license:gpl2+ license:lgpl2.1+ Please add a comment above that explains what this list means, e.g. to what file which of the two licenses apply. Other than that

Re: [PATCH] Fix compiling on CentOS 7.

On Fri, Aug 26, 2016 at 09:48:36PM +0200, Roel Janssen wrote: Dear Guix, Due to an old Automake version (1.13), running the `./configure' phase on CentOS 7 fails with: autoreconf: running: automake --add-missing --copy --force-missing configure.ac:21: warning: The 'AM_PROG_MKDIR_P' macro is

Re: [PATCH] gnu: octave: Update to 4.0.3.

Kei Kebreau writes: > I've removed curl and friends from the dependency list to avoid license > incompatibilities. Until we figure out what's up with curl and openssl, > we can keep network capabilities out of the octave definition to stay on > the safe side. What do you

Re: [PATCH 03/31] build: Add wrap-qt-program.

David Craven writes: > * guix/build/qt-utils.scm (wrap-qt-program): New file. Please also mention the change to MODULES in the Makefile.am. Other than that it looks good to me. ~~ Ricardo > --- > Makefile.am | 1 + > guix/build/qt-utils.scm | 40

Re: [PATCH 02/12] import: utils: Symbols from 'license->symbol' have a license: prefix.

David Craven writes: > * guix/import/utils.scm (define-module): guix licenses import uses a prefix. > (string->licenses): Add license: prefix to licenses. > (license->symbol): Add license: prefix to licenses. Return a symbol with the > prefix license:. Nitpick: please

Re: base64: Restore original license header (license compliance issue)

Christopher Allan Webber writes: > When incorporating a copyleft licensed program with a lax licensed program, > it is correct to preserve the original license notice. See also: > > https://www.softwarefreedom.org/resources/2007/gpl-non-gpl-collaboration.html > >

Re: [PATCH 02/31] gnu: qt: Import licenses using #:prefix license:.

David Craven writes: > * gnu/packages/qt.scm > (qt)[license]: Use license:. > (qt-4)[license]: Use license:. > (qtbase)[license]: Use license:. > (python-sip)[license]: Use license:. > (python-pyqt)[license]: Use license:. > (qtkeychain)[license]: Use license:. The patch looks

Re: [PATCH 06/31] gnu: Add kactivities-stats.

David Craven writes: > * gnu/packages/kde-frameworks.scm (kactivities-stats): New variable. > > Co-authored-by: Hartmut Goebel > --- > gnu/packages/kde-frameworks.scm | 34 ++ > 1 file changed, 34 insertions(+) > >

Re: [PATCH] gnu: Add linux-pam.

ren...@openmailbox.org writes: > This is a patch for linux-pam, at compile on the Hurd system searches > the file fsuid.h. The patch was taken from the Debian project. > > * This patch is prerequisite for lsh/openssh packages. > * The patch was build and installed on Linux and the Hurd

[PATCH] Sddm pam authentication failure.

So I got sddm to start an xfce session when I build sddm with -DENABLE_PAM=OFF. Any ideas what the problem might be? I tried all kinds of stuff to get PAM to give me some more information than authentication failure. The /etc/pam_debug file didn't work. Compiling PAM with --enable-debug didn't

[PATCH] services: Add sddm service.

* gnu/services/sddm.scm: New file. * gnu/local.mk (GNU_SYSTEM_MODULES): Add it. --- gnu/local.mk | 1 + gnu/services/sddm.scm | 312 ++ 2 files changed, 313 insertions(+) create mode 100644 gnu/services/sddm.scm diff --git

Re: [PATCH] Sddm pam authentication failure.

So it looks like elogind sets XDG_RUNTIME_DIR. Need to fix the pam issue and then the weston session will hopefully just work (just kidding, like that ever happens =P)

[PATCH] gnu: guile-gnunet: Use https URL.

>From f4bcc1e6aeee861ee9dc11c1803ca94146c5d86c Mon Sep 17 00:00:00 2001 From: ng0 Date: Sat, 27 Aug 2016 13:38:36 + Subject: [PATCH 2/4] gnu: guile-gnunet: Use https URL. * gnu/packages/gnunet.scm (guile-gnunet)[home-page]: Use https URL. ---

Re: pre-release [PATCH] git-service [v2]

Alex Kost writes: > ng0 (2016-08-25 12:55 +0300) wrote: > >> Hi, >> >> thanks for finding the errors. I applied your suggestions but the >> service still goes into respawn loop when the VM boots. Do you have any >> idea why this could happen? > > Is there any output in syslog?

Re: base64: Restore original license header (license compliance issue)

Ricardo Wurmus writes: > Christopher Allan Webber writes: > >> When incorporating a copyleft licensed program with a lax licensed program, >> it is correct to preserve the original license notice. See also: >> >>

[PATCH] gnu: Add sddm.

* gnu/packages/hawaii.scm (sddm): New variable. --- gnu/packages/hawaii.scm | 65 + 1 file changed, 65 insertions(+) diff --git a/gnu/packages/hawaii.scm b/gnu/packages/hawaii.scm index fbf642b..8c99e5f 100644 --- a/gnu/packages/hawaii.scm +++

[PATCH] gnu: grub: Use https URL.

>From a9367fd2dbe5bb77d67ee3ee38594ed7bddaf9a2 Mon Sep 17 00:00:00 2001 From: ng0 Date: Sat, 27 Aug 2016 13:47:19 + Subject: [PATCH 4/4] gnu: grub: Use https URL. * gnu/packages/grub.scm (grub)[home-page]: Use https URL. --- gnu/packages/grub.scm | 2 +- 1 file

[PATCH] gnu: guile-opengl: Use https URL.

>From 42845c318bbdfa915cc73df8fcdaf71bf8fcf270 Mon Sep 17 00:00:00 2001 From: ng0 Date: Sat, 27 Aug 2016 13:44:28 + Subject: [PATCH 3/4] gnu: guile-opengl: Use https URL. * gnu/packages/gl.scm (guile-opengl)[home-page]: Use https URL. --- gnu/packages/gl.scm | 3

Re: [PATCH] Add bwm-ng.

> For a first(?) package this is very good. > Other than these changes, which the person commiting your patch can > apply on your behalf, it looks good to me! Yes, this is my first package. Thank you! :-) signature.asc Description: PGP signature

[PATCH 2/4] gnu: node: Add search path specification for 'NODE_PATH'.

* gnu/packages/node.scm (node)[native-search-paths]: New field. --- gnu/packages/node.scm | 4 1 file changed, 4 insertions(+) diff --git a/gnu/packages/node.scm b/gnu/packages/node.scm index 545..d1c5e1b 100644 --- a/gnu/packages/node.scm +++ b/gnu/packages/node.scm @@ -151,6 +151,10

[PATCH 3/4] gnu: node: Do not use bundled dependencies.

The Node build system was previously building its own copies of C-ares and http-parser. * gnu/packages/node.scm (node)[inputs]: Add c-ares and http-parser. [arguments]: Add configure flags for using system libraries. --- gnu/packages/node.scm | 7 ++- 1 file changed, 6 insertions(+), 1

[PATCH 4/4] gnu: node: Use compression: prefix.

* gnu/packages/node.scm (define-module): Import gnu packages compression with a prefix (node): Likewise. --- gnu/packages/node.scm | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/gnu/packages/node.scm b/gnu/packages/node.scm index 7c020e6..351e988 100644 ---

[PATCH 1/4] gnu: node: Add http-parser.

* gnu/packages/node.scm (http-parser): New variable. * gnu/packages/node.scm (define-module): Import gnu packages tls with tls: prefix --- gnu/packages/node.scm | 39 +-- 1 file changed, 37 insertions(+), 2 deletions(-) diff --git a/gnu/packages/node.scm

Re: [PATCH] Fix compiling on CentOS 7.

Hi Roel Janssen writes: > Alex Kost writes: > >> Tomáš Čech (2016-08-27 09:57 +0300) wrote: >> >>> On Fri, Aug 26, 2016 at 09:48:36PM +0200, Roel Janssen wrote: Dear Guix, Due to an old Automake version (1.13), running the `./configure' phase on CentOS 7 fails

Re: [PATCH] Fix compiling on CentOS 7.

Alex Kost writes: > Tomáš Čech (2016-08-27 09:57 +0300) wrote: > >> On Fri, Aug 26, 2016 at 09:48:36PM +0200, Roel Janssen wrote: >>>Dear Guix, >>> >>>Due to an old Automake version (1.13), running the `./configure' phase on >>>CentOS 7 fails with: >>> autoreconf: running: automake

Re: Build machine sysadmin support

On Sun, Aug 21, 2016 at 7:38 PM, David Craven wrote: > So guix doesn't work inside a vm - because /gnu/store is mounted > readonly. To be able to offload arm builds once guix system vm > --system=armhf-linux works, /gnu/store needs to be writeable. > > Can multiple daemons

Re: [PATCH] gnu: Add p7zip.

Kei Kebreau writes: > Ricardo Wurmus writes: > >> Kei Kebreau writes: >> >>> Here's the corresponding patch. Maybe you or someone else can double (or >>> triple?) >>> check and make sure there are no proprietary files the source

Re: [PATCH] gnunet-svn, gnunet-gtk-svn

Hi, Catonano writes: > 2016-08-20 14:04 GMT+02:00 ng0 > > > >> Reported at https://gnunet.org/bugs/view.php?id=4619 >> >> > Christian Grothoff replied that he would like you to extract a stacktrace > from a core dump. > > Did you notice ? > > I

Re: [PATCH] Fix compiling on CentOS 7.

Tomáš Čech (2016-08-27 09:57 +0300) wrote: > On Fri, Aug 26, 2016 at 09:48:36PM +0200, Roel Janssen wrote: >>Dear Guix, >> >>Due to an old Automake version (1.13), running the `./configure' phase on >>CentOS 7 fails with: >> >>> autoreconf: running: automake --add-missing --copy --force-missing

Re: [PATCH 04/11] gnu: Add python-nltk.

> +(description "Natural Language Toolkit") I'd elaborate on the description a bit. Otherwise LGTM. From the site, buzzwords removed: "It provides interfaces to over 50 corpora and lexical resources such as WordNet, along with a suite of text processing libraries for classification,

Re: [PATCH 06/31] gnu: Add kactivities-stats.

> What does this mean? Is the tier classification an upstream thing? Yes it is [0]. IMO the kactivities-stats package is missclassified. [0] https://api.kde.org/frameworks/index.html

Re: [PATCH] Fix compiling on CentOS 7.

On 08/27/2016 10:51 AM, Roel Janssen wrote: > I don't see a way to apply this patch "on the CentOS side" except for > waiting (another two or three years) until they upgrade Automake. When packaging guix for CentOS, just add a patch to the source package definition.. Regards, Flo P.S.: Why the

Re: GSoC NPM

Hi Ricardo Wurmus writes: > Hi > >> I also took both Ludovic', as well as Catonano's detailed feedback on the >> initial draft of the recursive importer into account when rewriting it. It >> should now only visit each node in the dependency graph once, and be a >>

Re: [PATCH 02/12] import: utils: Symbols from 'license->symbol' have a license: prefix.

> Is the purpose of this patch to make importers return licenses with this > prefix or is there another use case? Yes that's the intention. I was only thinking about the python importer, but this will affect all importers.

[PATCH 0/4] Unbundle node dependencies patch series

These patches allow us to make use of the existing c-ares package, as well as an unbundled version of http parser. Jelle Licht (4): gnu: node: Add http-parser. gnu: node: Add search path specification for 'NODE_PATH'. gnu: node: Do not use bundled dependencies. gnu: node: Use compression:

Re: [PATCH] Add bwm-ng.

Hi Arun, thanks for your contribution. Arun writes: > [ Unknown signature status ] > > Please find attached patch to add new package 'bwm-ng' to (gnu packages > admin). > > Regards, > Arun Isaac > > From 2a88508183f9b8005c2a0694992f1b3ae4208049 Mon Sep 17 00:00:00

Re: Package: workrave

Hi, I noticed your contribution has not been merged yet. Do you require some additional help or input on the workflow, or do you want one of us to create patches from your original contributions? Leo Famulari writes: > On Sat, Jun 11, 2016 at 07:32:32AM -0500, Jovany

Re: [PATCH] Add bwm-ng.

Hi Arun, > Please find attached patch to add new package 'bwm-ng' to (gnu packages > admin). thanks for the patch. I’ve made a couple of cosmetic changes and removed the list of supported operating systems from the description. I also moved it from admin.scm to networking.scm where we already

Re: [PATCH] gnu: Add linux-pam.

Hello Rene, First of all thank you for helping with the port :-). Now on the patch. > Subject: [PATCH] gnu: Add linux-pam. Maybe we should change the name of the patch to "[PATCH] gnu: Make linux-pam build on non Linux systems." Other than that looks good to me. As Ricardo said check the

Re: pre-release [PATCH] git-service [v2]

Alex Kost writes: > ng0 (2016-08-27 18:39 +0300) wrote: > > [...] >> (eval-when (expand load eval) (set! %load-path (cons >> "/gnu/store/8s6z5k9nfp3fyfj39vq6js8sfhxkn0y6-module-import" %load-path)) >> (set! %load-compiled-path (cons >>

Re: [GSoC] Porting GuixSD to GNU/Hurd Update

Hello Manolis, Manolis Ragkousis skribis: > On Hurd though, because of the lack of mount(), the above could not > work. But with the help of my libhurdutil library, which I wrote at the > beginning of this project, I created 2 wrappers inside > nix/libutil/call.(hh.cc) for

Re: Flex security update: RCE in generated code (CVE-2016-6354)

Hello! Leo Famulari skribis: > On Fri, Aug 26, 2016 at 06:14:26PM -0400, Leo Famulari wrote: >> Subject: [PATCH] gnu: flex: Fix CVE-2016-6354. >> >> * gnu/packages/flex.scm (flex)[replacement]: New field. >> (flex/fixed): New variable. >> *

Re: [PATCH 1/4] gnu: node: Add http-parser.

Jelle Licht (2016-08-27 14:23 +0300) wrote: > * gnu/packages/node.scm (http-parser): New variable. > * gnu/packages/node.scm (define-module): Import gnu packages tls with > tls: prefix These are 2 independent things, so I think it should be 2 separate patches, but actually why is it needed to

[PATCH] gnu: libmicrohttpd: Update to 0.9.51.

>From 4ea53551b4b6847f6f7f64e82e0343fe5cc3d89a Mon Sep 17 00:00:00 2001 From: ng0 Date: Sat, 27 Aug 2016 20:16:37 + Subject: [PATCH] gnu: libmicrohttpd: Update to 0.9.51. * gnu/packages/gnunet.scm (libmicrohttpd): Update to 0.9.51. --- gnu/packages/gnunet.scm | 4

Re: [PATCH 3/4] gnu: node: Do not use bundled dependencies.

Jelle Licht (2016-08-27 14:23 +0300) wrote: > The Node build system was previously building its own copies of > C-ares and http-parser. > > * gnu/packages/node.scm (node)[inputs]: Add c-ares and http-parser. > [arguments]: Add configure flags for using system libraries. > --- >

Re: [PATCH 4/4] gnu: node: Use compression: prefix.

Jelle Licht (2016-08-27 14:23 +0300) wrote: > * gnu/packages/node.scm (define-module): Import gnu packages compression > with a prefix > (node): Likewise. > --- > gnu/packages/node.scm | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > > diff --git a/gnu/packages/node.scm

Re: [PATCH] gnu: emacs: Use https for elpa.gnu.org

Alex Kost writes: > ng0 (2016-08-27 17:20 +0300) wrote: > >> From d2dfd0fcc34f5cdcb9d181093cffd5af16be6641 Mon Sep 17 00:00:00 2001 >> From: ng0 >> Date: Sat, 27 Aug 2016 13:33:31 + >> Subject: [PATCH 1/4] gnu: emacs: Use https for elpa.gnu.org.

Re: GSoC NPM

Hi, Jelle! Jelle Licht skribis: > To start of with something that did not work out as well as I had hoped, > getting > a popular build system (e.g. Gulp, Grunt, Broccoli and others) packaged. As > mentioned in my earlier mails, the list of transitive dependencies of any of >

[PATCH 1/4] gnu: yelp: Update to 3.21.3

From: John Darrington * gnu/packages/gnome.scm (yelp): Update to 3.21.3 --- gnu/packages/gnome.scm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/gnu/packages/gnome.scm b/gnu/packages/gnome.scm index f4488c9..5ea9714 100644 --- a/gnu/packages/gnome.scm +++

[PATCH 4/4] gnu: libtirpc: Remove the --disable-gssapi configure flag.

From: John Darrington * gnu/packages/onc-rpc.scm (libtirpc)[arguments]: Remove the --disable-gssapi configure flag. [inputs]: Add mit-krb5 --- gnu/packages/onc-rpc.scm | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/gnu/packages/onc-rpc.scm

[PATCH 2/4] CoC: Correct punctuation

From: John Darrington * CODE-OF-CONDUCT: "others" is plural. Hence the apostrophe follows the "s". --- CODE-OF-CONDUCT | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CODE-OF-CONDUCT b/CODE-OF-CONDUCT index 0458fd7..22a0446 100644 --- a/CODE-OF-CONDUCT +++

[PATCH 3/4] gnu: Add German Aspell dictionary.

From: John Darrington * gnu/packages/aspell.scm (aspell-dict-de): New variable. --- gnu/packages/aspell.scm | 7 +++ 1 file changed, 7 insertions(+) diff --git a/gnu/packages/aspell.scm b/gnu/packages/aspell.scm index 117cd33..f836819 100644 --- a/gnu/packages/aspell.scm +++

[PATCHv2 1/2] gnu: Add wlc.

* gnu/packages/freedesktop.scm(wlc): New variable. * gnu/packages/patches/wlc-implement-wlc_view_get_pid.patch: Add it * gnu/local.mk: Add entry for patch above. --- gnu/packages/freedesktop.scm | 41 - 1 file changed, 40 insertions(+), 1 deletion(-) diff

[PATCHv2 2/2] gnu: Add sway.

* gnu/packages/freedesktop.scm(sway): New variable. --- gnu/packages/freedesktop.scm | 44 1 file changed, 44 insertions(+) diff --git a/gnu/packages/freedesktop.scm b/gnu/packages/freedesktop.scm index 0bb87e7..fec7cf5 100644 ---

Re: [PATCH] gnu: Remove trademark acknowledgements.

Hi! Eric Bavier skribis: > From 46a8f3322392fcd6a7641062ff6b3d23685f394b Mon Sep 17 00:00:00 2001 > From: Eric Bavier > Date: Tue, 23 Aug 2016 02:08:02 -0500 > Subject: [PATCH] guix: lint: Check descriptions for trademark signs. > > *

Re: pre-release [PATCH] git-service [v2]

ng0 writes: > Alex Kost writes: > >> ng0 (2016-08-27 18:39 +0300) wrote: >> >> [...] >>> (eval-when (expand load eval) (set! %load-path (cons >>> "/gnu/store/8s6z5k9nfp3fyfj39vq6js8sfhxkn0y6-module-import" %load-path)) >>> (set! %load-compiled-path

Re: [PATCH] Add bwm-ng.

> thanks for the patch. I’ve made a couple of cosmetic changes and > removed the list of supported operating systems from the description. I > also moved it from admin.scm to networking.scm where we already have a > couple of network monitoring packages. Sure. > I’d like to add a copyright

Display diffs between generations.

>From 1ea5eaae4b492c82358c7394c22cd46497388449 Mon Sep 17 00:00:00 2001 From: Roel Janssen Date: Sun, 28 Aug 2016 00:54:06 +0200 Subject: [PATCH] guix package: Display generation diffs. --- guix/scripts/package.scm | 2 +- guix/ui.scm | 34

‘guix lint’ URL checks

Leo Famulari skribis: > On Wed, Aug 17, 2016 at 09:20:42PM +0200, David Craven wrote: >> Oh, that's embarrassing. I must have forgotten to change the hash and >> guix cached the tarball. Is that possible? > > If the hash is matched in the store, the URL is not used, so you

Re: Odd behavior with --dry-run and --upgrade

Roel Janssen writes: > Ludovic Courtès writes: > >> Roel Janssen skribis: >> >>> Ludovic Courtès writes: >>> Hi! Alex Kost skribis: > Roel Janssen (2016-07-23 18:11 +0300) wrote: > >> Dear Guix, >> >> For some time now,

Re: Flex security update: RCE in generated code (CVE-2016-6354)

On Sat, Aug 27, 2016 at 11:48:10PM +0200, Ludovic Courtès wrote: > Hello! > > Leo Famulari skribis: > > > On Fri, Aug 26, 2016 at 06:14:26PM -0400, Leo Famulari wrote: > >> Subject: [PATCH] gnu: flex: Fix CVE-2016-6354. > >> > >> * gnu/packages/flex.scm (flex)[replacement]:

Re: [PATCH] Fix compiling on CentOS 7.

Florian Paul Schmidt writes: > On 08/27/2016 10:51 AM, Roel Janssen wrote: > >> I don't see a way to apply this patch "on the CentOS side" except for >> waiting (another two or three years) until they upgrade Automake. > > When packaging guix for CentOS, just add a patch to the source package >