Re: [PATCH] gnu: rottlog: rotate messages daily.

2016-09-08 Thread Jan Nieuwenhuizen
Jan Nieuwenhuizen writes: Hi! >> ‘rottlog’ was looking for config files in OUT/etc, which made it >> impossible for people to configure it. So in commit >> 268ad34e0eadf8a015798b5c5587aad65b9f3a61 I changed it to look for >> configuration files in /etc/rottlog. >> >> Consequently, running

Re: [PATCH 1/2] gnu: Add edi.

2016-09-08 Thread Efraim Flashner
On Thu, Sep 08, 2016 at 11:41:27PM -0500, Eric Bavier wrote: > On Thu, 8 Sep 2016 13:44:38 +0300 > Efraim Flashner wrote: > > > * gnu/packages/enlightenment.scm (edi): New variable. > > --- > > gnu/packages/enlightenment.scm | 26 ++ > > 1 file

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

2016-09-08 Thread Eric Bavier
On Thu, 8 Sep 2016 13:44:39 +0300 Efraim Flashner wrote: > * gnu/packages/calendar.scm (libhdate): New variable. > --- > gnu/packages/calendar.scm | 21 + > 1 file changed, 21 insertions(+) > > diff --git a/gnu/packages/calendar.scm

Re: [PATCH 1/2] gnu: Add edi.

2016-09-08 Thread Eric Bavier
On Thu, 8 Sep 2016 13:44:38 +0300 Efraim Flashner wrote: > * gnu/packages/enlightenment.scm (edi): New variable. > --- > gnu/packages/enlightenment.scm | 26 ++ > 1 file changed, 26 insertions(+) > > diff --git a/gnu/packages/enlightenment.scm

Re: [PATCH]: opensmtpd, opensmtpd-extras, libasr.

2016-09-08 Thread Eric Bavier
On Thu, 08 Sep 2016 11:15:37 + ng0 wrote: > From 43b89c0b548e1bca2ff59271ac15b1fbe6e7ca6e Mon Sep 17 00:00:00 2001 > From: ng0 > Date: Tue, 6 Sep 2016 18:23:15 + > Subject: [PATCH 2/3] gnu: Add opensmtpd. > > * gnu/packages/mail.scm (opensmtpd):

Re: [PATCH] gnu: Add threadingbuildingblocks.

2016-09-08 Thread Eric Bavier
On Thu, 8 Sep 2016 23:07:06 + ng0 wrote: > * gnu/packages/parallel.scm (threadingbuildingblocks): New variable. > --- > gnu/packages/parallel.scm | 33 + > 1 file changed, 33 insertions(+) > > diff --git

Re: 02/02: gnu: Add edit warning to generated grub.cfg files

2016-09-08 Thread John Darrington
As I recall it, in the discusion I was asked to do this. Perhaps the discussion took place on irc - I don't recall. J' On Thu, Sep 08, 2016 at 09:44:19PM -0400, Leo Famulari wrote: On Thu, Sep 08, 2016 at 04:08:48PM +, John Darrington wrote: > jmd pushed a commit to branch master

Re: [PATCH] services: nginx: Actually check if configuration is valid.

2016-09-08 Thread Tobias Geerinckx-Rice
Hi Dave, On 09/09/16 04:29, Thompson, David wrote: > No, it doesn't. I wrote the nginx service. The config file used is > part of the g-expression and is absolutely not stateful. Wait. nckx@v5 ~$ sudo herd restart nginx Service nginx has been stopped. Service nginx has been started. nckx@v5

Re: [PATCH] services: nginx: Actually check if configuration is valid.

2016-09-08 Thread Thompson, David
On Thu, Sep 8, 2016 at 10:21 PM, Tobias Geerinckx-Rice wrote: > Dave, > > On 09/09/16 04:09, Thompson, David wrote: >> Maybe this could be done at build-time instead? > > The nginx service points to a stateful configuration file, like > /etc/nginx.conf, that isn't built or handled

Re: [PATCH] services: nginx: Actually check if configuration is valid.

2016-09-08 Thread Tobias Geerinckx-Rice
Dave, On 09/09/16 04:09, Thompson, David wrote: > Maybe this could be done at build-time instead? The nginx service points to a stateful configuration file, like /etc/nginx.conf, that isn't built or handled by Guix. I'd like that to change, but that will have to wait for now. > Would be nice

Re: [PATCH] services: nginx: Actually check if configuration is valid.

2016-09-08 Thread Thompson, David
On Thu, Sep 8, 2016 at 10:05 PM, Tobias Geerinckx-Rice wrote: > * gnu/services/web.scm (nginx-activation): Fix path to nginx binary. > --- > > Hullo again! > > I suspect not many people run nginx (or dovecot, but that's a different > matter) on Guix. The nginx activation script

[PATCH] services: nginx: Actually check if configuration is valid.

2016-09-08 Thread Tobias Geerinckx-Rice
* gnu/services/web.scm (nginx-activation): Fix path to nginx binary. --- Hullo again! I suspect not many people run nginx (or dovecot, but that's a different matter) on Guix. The nginx activation script will now correctly print a non-fatal error if the configuration smells off. Why I'm mailing:

Re: 02/02: gnu: Add edit warning to generated grub.cfg files

2016-09-08 Thread Leo Famulari
On Thu, Sep 08, 2016 at 04:08:48PM +, John Darrington wrote: > jmd pushed a commit to branch master > in repository guix. > > commit fdf14c64f1dc7526e84b7e0ce41bf99e5b31c3f8 > Author: John Darrington > Date: Thu Sep 8 14:56:14 2016 +0200 > > gnu: Add edit warning to

Re: Linux kernel

2016-09-08 Thread Leo Famulari
On Fri, Sep 09, 2016 at 12:45:13AM +0200, David Craven wrote: > I reworked the patches for the linux-libre package and made some other > improvements that make customizing the linux-libre package easier. I > tested that everything builds and that there wasn't any breakage on > x86_64, but testing

[PATCH v2 2/3] gnu: Add filteraudio.

2016-09-08 Thread Erik Edrosa
* gnu/packages/audio.scm (filteraudio): New variable. --- gnu/packages/audio.scm | 33 + 1 file changed, 33 insertions(+) diff --git a/gnu/packages/audio.scm b/gnu/packages/audio.scm index 40cc029..cc51d88 100644 --- a/gnu/packages/audio.scm +++

Re: NPM and trusted binaries

2016-09-08 Thread Mike Gerwitz
On Thu, Sep 08, 2016 at 21:54:36 +0200, Jan Nieuwenhuizen wrote: > The question I'm trying to answer is: how does `a user' who builds a > package from the repository install the needed dependencies. Sorry, I misinterpreted. `npm install ' will by default install all devDependencies; the

Re: [PATCH] gnu: Add threadingbuildingblocks.

2016-09-08 Thread Marius Bakke
ng0 writes: > +(arguments > + `(;; XXX: make 'check' is run with 'make all', however the 'all' phase > takes > + ;; 30+ minutes on a AMD FX-4300 Quad-Core Processor buildmachine. > + #:tests? #f 30 minutes isn't bad, we have packages that takes

Re: wip v2 u-boot support

2016-09-08 Thread Danny Milosavljevic
New wip patch attached - applies to git guix... diff --git a/gnu.scm b/gnu.scm index 932e4cd..9207e38 100644 --- a/gnu.scm +++ b/gnu.scm @@ -35,6 +35,7 @@ (gnu system mapped-devices) (gnu system file-systems) (gnu system grub) ;

Re: wip v2 u-boot support

2016-09-08 Thread Danny Milosavljevic
Hi David, Hi Ludo, On Fri, 9 Sep 2016 00:09:57 +0200 David Craven wrote: > I'm testing your new u-boot code. I fixed a few bugs, but there is > still some work to do... ;-) > > guix system vm u-boot-test.scm --no-grub > > This should work without giving any errors and boot

Re: [PATCH] gnu: Add threadingbuildingblocks.

2016-09-08 Thread ng0
Jeff Mickey writes: > Yes, people will search for this as 'tbb'. Debian packages it as libtbb, > arch does intel-tbb. > > // codemac Thanks, that's what I had at first. What about the product name, what is it described as? Should I refer to debians description? I remember

Re: [PATCH]: opensmtpd, opensmtpd-extras, libasr.

2016-09-08 Thread ng0
Hi, thanks for reviewing. I'll apply and send changed later. Efraim Flashner writes: > [ Unknown signature status ] > On Thu, Sep 08, 2016 at 11:15:37AM +, ng0 wrote: >> This adds opensmtpd. I'll write a service as this is obviously >> not very functional without a

Re: [PATCH]: opensmtpd, opensmtpd-extras, libasr.

2016-09-08 Thread ng0
Marius Bakke writes: > ng0 writes: > >> From 30c8260a039d4c8d7cddbfd92341afa65577d680 Mon Sep 17 00:00:00 2001 >> From: ng0 >> Date: Tue, 6 Sep 2016 17:10:18 + >> Subject: [PATCH 1/3] gnu: Add libasr. >> >> * gnu/packages/web.scm

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

2016-09-08 Thread ng0
Hi, can someone else comment on this thread? I've listed part of my reasons, not all, my position should be clear. ng0 writes: > ng0 writes: > >> Alex Kost writes: >> >>> ng0 (2016-08-27 17:20 +0300) wrote: >>>

Re: [PATCH 07/43] gnu: Add ghc-haskeline.

2016-09-08 Thread ng0
ng0 writes: > Eric Bavier writes: > >> On Sun, 4 Sep 2016 16:17:06 + >> ng0 wrote: >> >>> * gnu/packages/haskell.scm (ghc-haskeline): New variable. >>> --- >>> gnu/packages/haskell.scm | 25

Re: [PATCH] gnu: Add threadingbuildingblocks.

2016-09-08 Thread Jeff Mickey
Yes, people will search for this as 'tbb'. Debian packages it as libtbb, arch does intel-tbb. // codemac * ng0 [2016-09-08 16:11]: > I am unclear about the name. What do people expect to find this as? Do I > include Intel as it intel® tbb? I just add this as a

Linux kernel

2016-09-08 Thread David Craven
I reworked the patches for the linux-libre package and made some other improvements that make customizing the linux-libre package easier. I tested that everything builds and that there wasn't any breakage on x86_64, but testing the arm kernel on real hardware is a wip. Is it easier if I push a

Re: [PATCH] gnu: Add threadingbuildingblocks.

2016-09-08 Thread ng0
I am unclear about the name. What do people expect to find this as? Do I include Intel as it intel® tbb? I just add this as a dependency Pixar USD needs. ng0 writes: > * gnu/packages/parallel.scm (threadingbuildingblocks): New variable. > --- >

[PATCH] gnu: Add threadingbuildingblocks.

2016-09-08 Thread ng0
* gnu/packages/parallel.scm (threadingbuildingblocks): New variable. --- gnu/packages/parallel.scm | 33 + 1 file changed, 33 insertions(+) diff --git a/gnu/packages/parallel.scm b/gnu/packages/parallel.scm index 3bb078e..223b318 100644 ---

Re: [PATCH] services: Add kmscon service.

2016-09-08 Thread David Craven
Nice! Thanks!

Re: [PATCH] services: Add kmscon service.

2016-09-08 Thread Ludovic Courtès
I noticed that despite the dependency of kmscon on elogind, elogind support was missing. I added it in commit 5acf00f99072cd4da8d3ba064f19cc819288d68b. Ludo’.

Re: wip v2 u-boot support

2016-09-08 Thread David Craven
Hi Danny, I'm testing your new u-boot code. I fixed a few bugs, but there is still some work to do... ;-) guix system vm u-boot-test.scm --no-grub This should work without giving any errors and boot (it doesn't matter that u-boot is built for arm). And after booting there should be the

Re: [PATCH] gnu: Add mash.

2016-09-08 Thread Marius Bakke
Leo Famulari writes: > On Thu, Sep 01, 2016 at 11:00:39AM +0100, Marius Bakke wrote: >> I had these in inputs initially and was surprised to see no references. >> Both seems to be compiled into the final program[0]: when running "mash >> info" on an invalid file (the provided

Re: [PATCH] services: Add kmscon service.

2016-09-08 Thread Ludovic Courtès
David Craven skribis: > + (list (shepherd-service > +(documentation "kmscon virtual terminal") > +(requirement '(user-processes udev)) Looking at the doc, I’m under the impression that this should also depend on elogind no? (I think it needs to talk to elogind to tell

Re: Four thousand!

2016-09-08 Thread Ludovic Courtès
Christopher Allan Webber skribis: > Ludovic Courtès writes: > >> Hello Guix! >> >> I wonder which one was the 4,000th, but we made it: >> >> --8<---cut here---start->8--- >> $ GUIX_PACKAGE_PATH= ./pre-inst-env guix package -A |wc -l >>

Re: [PATCH] services: Add login-service.

2016-09-08 Thread Ludovic Courtès
David Craven skribis: > * gnu/services/base.scm (%default-motd, , > login-pam-service, login-serivce-type, login-service): New variables. > (, mingetty-shepherd-service, > mingetty-serivce-type): Remove motd. Remove allow-empty-passwords?. > Remove mingetty-pam-service.

Re: [PATCH] services: Add kmscon service.

2016-09-08 Thread Ludovic Courtès
Hello! David Craven skribis: > * gnu/services/base.scm (, > kmscon-shepherd-service, kmscon-service-type, kmscon-service): New > variables. I didn’t know kmscon, that looks pretty interesting. > +(define-record-type* > + kmscon-configuration make-kmscon-configuration

[PATCH] gnu: glew: Update to 2.0.0.

2016-09-08 Thread ng0
* gnu/packages/gl.scm (glew): Update to 2.0.0. --- gnu/packages/gl.scm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/gnu/packages/gl.scm b/gnu/packages/gl.scm index 1473b44..1aa1201 100644 --- a/gnu/packages/gl.scm +++ b/gnu/packages/gl.scm @@ -357,14 +357,14 @@

Re: [PATCH] gnu: Simplify dtc package definition.

2016-09-08 Thread David Craven
Hi Danny, So when there's an attachment I git am the attachment. Don't know if that's how it's supposed to be done. The attachment starts at --2.9.1, so it doesn't include the commit information. Thanks for looking into this, hopefully we can move to gitlab soon =P David

Re: [PATCH] gnu: Add git-review.

2016-09-08 Thread Efraim Flashner
On Thu, Sep 08, 2016 at 10:28:37PM +1000, Ben Woodcroft wrote: > Hi there, thanks for the patch. I do not have any experience with Gerrit but > some comments below: > > > On 08/09/16 17:06, Clément Lassieur wrote: > > [..] > > +(source > > + (origin > > + (method url-fetch) > > +

Re: NPM and trusted binaries

2016-09-08 Thread Jan Nieuwenhuizen
Mike Gerwitz writes: > On Thu, Sep 08, 2016 at 10:45:57 +0200, Jan Nieuwenhuizen wrote: >> If a user builds an npm package from its source repository, I assume >> that they install the devDependencies needed for that using npm? > > Unfortunately that depends on the project. Some projects use >

Re: [PATCH]: opensmtpd, opensmtpd-extras, libasr.

2016-09-08 Thread Efraim Flashner
On Thu, Sep 08, 2016 at 11:15:37AM +, ng0 wrote: > This adds opensmtpd. I'll write a service as this is obviously > not very functional without a service on Guix. > > > OT: So far I hope I can just change the email address I am subscribed to > the list with back, I'm waiting for the hosting

Re: [PATCH] gnu: Simplify dtc package definition.

2016-09-08 Thread David Craven
Hi Danny, I don't have any special settings configured, just the default. Your patches are missing a header like this one: >From fc0e6e2ae96b0586035de96c65cfd08186ceb28a Mon Sep 17 00:00:00 2001 From: Efraim Flashner Date: Wed, 7 Sep 2016 21:02:28 +0300 Subject: [PATCH]

Re: NPM and trusted binaries

2016-09-08 Thread Mike Gerwitz
On Thu, Sep 08, 2016 at 10:45:57 +0200, Jan Nieuwenhuizen wrote: > If a user builds an npm package from its source repository, I assume > that they install the devDependencies needed for that using npm? Unfortunately that depends on the project. Some projects use devDependencies only for things

Re: [PATCH] gnu: Add git-review.

2016-09-08 Thread Efraim Flashner
On Thu, Sep 08, 2016 at 10:28:37PM +1000, Ben Woodcroft wrote: > Hi there, thanks for the patch. I do not have any experience with Gerrit but > some comments below: > > > On 08/09/16 17:06, Clément Lassieur wrote: > > [..] > > +(source > > + (origin > > + (method url-fetch) > > +

Re: [PATCH] gnu: Simplify dtc package definition.

2016-09-08 Thread Danny Milosavljevic
Hi, I think that's because I set it up to use attachments (Luda asked me to). The E-Mail, raw as git send-email sent it, looks like this: From 7e7c30119aebab0ee48c4e55d8c7471f0fde487c Mon Sep 17 00:00:00 2001 From: Danny Milosavljevic Date: Wed, 7 Sep 2016 09:44:55

Re: [PATCH] gnu: Add python-git-review.

2016-09-08 Thread Efraim Flashner
Patch pushed! -- Efraim Flashner אפרים פלשנר GPG key = A28B F40C 3E55 1372 662D 14F7 41AA E7DC CA3D 8351 Confidentiality cannot be guaranteed on emails sent or received unencrypted signature.asc Description: PGP signature

Re: [PATCH 1/1] gnu: curl: Update replacement to 7.50.2 [fixes CVE-2016-7141].

2016-09-08 Thread Leo Famulari
On Thu, Sep 08, 2016 at 08:40:12AM +0300, Efraim Flashner wrote: > On Wed, Sep 07, 2016 at 06:04:23PM -0400, Leo Famulari wrote: > > * gnu/packages/curl.scm (curl)[replacement]: Update to 7.50.2 > > (curl-7.50.1): Replace with ... > > (curl-7.50.2): ... this. > Looks good! Thanks, pushed!

Re: R

2016-09-08 Thread Pjotr Prins
OpenBLAS works fine for me - hit the same issue with Python before. Pj. On Thu, Sep 08, 2016 at 09:41:59PM +1000, Ben Woodcroft wrote: > Hi, > > I thought I'd respond to this old thread now that the openblas eigen > error seems to be fixed. I'm interested in using openblas so that > the (as yet

Re: [PATCH]: opensmtpd, opensmtpd-extras, libasr.

2016-09-08 Thread Marius Bakke
ng0 writes: > From 30c8260a039d4c8d7cddbfd92341afa65577d680 Mon Sep 17 00:00:00 2001 > From: ng0 > Date: Tue, 6 Sep 2016 17:10:18 + > Subject: [PATCH 1/3] gnu: Add libasr. > > * gnu/packages/web.scm (libasr): New variable. Nice! > --- >

Re: [PATCH] gnu: Simplify dtc package definition.

2016-09-08 Thread Danny Milosavljevic
Hi David, On Thu, 8 Sep 2016 17:09:12 +0200 David Craven wrote: > Why aren't you using git format-patch? :) > > git format-patch -1 > git send-email *.patch --to guix-devel@gnu.org > > is what I do... That's exactly what I do too - the exact same commands. I don't know what

Re: [PATCH] gnu: Add git-review.

2016-09-08 Thread Ben Woodcroft
Hi there, thanks for the patch. I do not have any experience with Gerrit but some comments below: On 08/09/16 17:06, Clément Lassieur wrote: [..] +(source + (origin + (method url-fetch) + (uri (string-append +

Re: [PATCH 1/2] gnu: add python-django.

2016-09-08 Thread Efraim Flashner
On Mon, Aug 22, 2016 at 01:31:36PM +0200, Hartmut Goebel wrote: > * gnu/packages/django.scm: New file. > * gnu/local.mk (GNU_SYSTEM_MODULES): Add it. > * gnu/packages/patches/django-fix-testcase.patch: New file. > --- > gnu/local.mk | 1 + >

Default shared libs for cmake-build-system experiment

2016-09-08 Thread Eric Bavier
Hello Guix, I recently explored the idea of adding "-DBUILD_SHARED_LIBS=TRUE" to cmake-build-system's default configure-flags. This is just a summary of the experience for future reference. At the time of testing, there were 124 packages that use cmake-build-system. Of those, 7 already

Re: [PATCH] gnu: Use a directory owned by ntpd user for drift file.

2016-09-08 Thread John Darrington
On Thu, Sep 08, 2016 at 08:57:57AM +0200, Vincent Legoll wrote: Hello, > +(define (ntp-service-activation config) > + "Return the activation gexp for config" > + #~(begin > + (use-modules (guix build utils)) > + > + (define %user > +

Re: [PATCH] gnu: Simplify dtc package definition.

2016-09-08 Thread David Craven
Why aren't you using git format-patch? :) git format-patch -1 git send-email *.patch --to guix-devel@gnu.org is what I do... >From what I can tell you are doing git diff > file.patch?

Re: R

2016-09-08 Thread Ricardo Wurmus
Ben Woodcroft writes: > Hi, > > I thought I'd respond to this old thread now that the openblas eigen > error seems to be fixed. I'm interested in using openblas so that the > (as yet not in master) WGCNA package works faster, among other things: > >

Re: [PATCH] gnu: Add surfraw.

2016-09-08 Thread Efraim Flashner
On Tue, Sep 06, 2016 at 12:38:49PM +, ng0 wrote: > ng0 writes: > > > Problem(?): > > > > ng0@shadowwalker > > /gnu/store/fvx31h4z9pf910f3ycdqlpv5z6ii7lx5-surfraw-2.2.9/bin$ ./surfraw > > surfraw ERROR: couldn't find global config in > >

[PATCH] gnu: icedtea: Patch dynamically loaded libraries with absolute paths.

2016-09-08 Thread 宋文武
I think this will fix the fontconfig issue (eg: #24327). Not tested (build) any jdk version yet... >From 3dbe71f7cd7b99851bd616f1ab6e1269f1fc6e2e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E5=AE=8B=E6=96=87=E6=AD=A6?= Date: Thu, 8 Sep 2016 23:25:23 +0800 Subject: [PATCH] gnu:

[PATCH] gnu: Add python-git-review.

2016-09-08 Thread Clément Lassieur
* gnu/packages/openstack.scm (python-git-review, python2-git-review): New variables. --- gnu/packages/openstack.scm | 34 ++ 1 file changed, 34 insertions(+) diff --git a/gnu/packages/openstack.scm b/gnu/packages/openstack.scm index 8474904..4cb38a9 100644 ---

Re: [PATCH] gnu: Use a directory owned by ntpd user for drift file.

2016-09-08 Thread Vincent Legoll
> I think you need to look a little harder, because the answer to both your > (very valid) questions is : I copied it from other, existing examples. Ah, couldn't have seen it, I only read the patch, not the entire file... You copied it from : tor-hidden-service-activation ? So, can an

Should java .jar-filenames include the version?

2016-09-08 Thread Hartmut Goebel
Hi, as I'm going to release patches for some java packages, I'd like to get consent on one point: Should java .jar-filenames include the version? This only effects those .jar for which there is no build.xml (or equivalent) is present and thus #:jar-file is to be specified. The jar-files

[PATCH]: opensmtpd, opensmtpd-extras, libasr.

2016-09-08 Thread ng0
This adds opensmtpd. I'll write a service as this is obviously not very functional without a service on Guix. OT: So far I hope I can just change the email address I am subscribed to the list with back, I'm waiting for the hosting problem to be solved. Might even be that I stick with this one,

[PATCH 1/2] gnu: Add edi.

2016-09-08 Thread Efraim Flashner
* gnu/packages/enlightenment.scm (edi): New variable. --- gnu/packages/enlightenment.scm | 26 ++ 1 file changed, 26 insertions(+) diff --git a/gnu/packages/enlightenment.scm b/gnu/packages/enlightenment.scm index 7cd11b5..a0c5a83 100644 ---

[PATCH 1/2] gnu: r-htmltools: Ensure temporary files can be deleted.

2016-09-08 Thread Ricardo Wurmus
* gnu/packages/web.scm (r-htmltools)[arguments]: Add phase "copy-files-without-mode". --- gnu/packages/web.scm | 14 ++ 1 file changed, 14 insertions(+) diff --git a/gnu/packages/web.scm b/gnu/packages/web.scm index 321a250..a682e6d 100644 --- a/gnu/packages/web.scm +++

Re: [PATCH] Update entr to 3.6

2016-09-08 Thread Efraim Flashner
On Thu, Sep 08, 2016 at 05:28:06AM -0400, Matthew Jordan wrote: > Good Day, > > Sorry for late reply been a bit busy with work. About this package, do > I still need to send seperate patches for the move and the update or has > this patch already been applied? > > Thanks, > Matthew. > > The

[PATCH 2/2] gnu: r-rmarkdown: Ensure temporary files can be deleted.

2016-09-08 Thread Ricardo Wurmus
* gnu/packages/statistics.scm (r-rmarkdown)[arguments]: Add phase "copy-files-without-mode". --- gnu/packages/statistics.scm | 14 ++ 1 file changed, 14 insertions(+) diff --git a/gnu/packages/statistics.scm b/gnu/packages/statistics.scm index a9aa83b..da64f66 100644 ---

Re: R

2016-09-08 Thread Ben Woodcroft
Hi, I thought I'd respond to this old thread now that the openblas eigen error seems to be fixed. I'm interested in using openblas so that the (as yet not in master) WGCNA package works faster, among other things:

[PATCH 2/2] gnu: Add libhdate.

2016-09-08 Thread Efraim Flashner
* gnu/packages/calendar.scm (libhdate): New variable. --- gnu/packages/calendar.scm | 21 + 1 file changed, 21 insertions(+) diff --git a/gnu/packages/calendar.scm b/gnu/packages/calendar.scm index cf85899..0c37caa 100644 --- a/gnu/packages/calendar.scm +++

[PATCH 0/2] two more packages

2016-09-08 Thread Efraim Flashner
Going through my GUIX_PACKAGE_PATH, I've pulled out two packages that should be ready to send upstream. More to follow, hopefully very soon. Efraim Flashner (2): gnu: Add edi. gnu: Add libhdate. gnu/packages/calendar.scm | 21 + gnu/packages/enlightenment.scm | 26

Re: [PATCH] Update entr to 3.6

2016-09-08 Thread Matthew Jordan
Good Day, Sorry for late reply been a bit busy with work. About this package, do I still need to send seperate patches for the move and the update or has this patch already been applied? Thanks, Matthew. Efraim Flashner writes: > On Sun, Aug 21, 2016 at 12:18:15PM -0400, Matthew Jordan wrote:

Re: [PATCH] gnu: Add git-review.

2016-09-08 Thread Efraim Flashner
On Thu, Sep 08, 2016 at 09:06:30AM +0200, Clément Lassieur wrote: > * gnu/packages/openstack.scm (git-review): New variable. > --- > gnu/packages/openstack.scm | 29 + > 1 file changed, 29 insertions(+) > > diff --git a/gnu/packages/openstack.scm

Re: [PATCH] gnu: Add android-tools.

2016-09-08 Thread Hartmut Goebel
Hi, some remarks: +(source (origin + (method url-fetch) + (uri (string-append +"https://launchpad.net/ubuntu/+archive/primary/+files/; Is this the original source? I'd expact this to come from Google or AOSP? +name "_"

Re: [PATCH 1/3] gnu: openal: Update to 1.17.2.

2016-09-08 Thread Efraim Flashner
On Wed, Sep 07, 2016 at 09:00:00PM -0400, Erik Edrosa wrote: > * gnu/packages/audio.scm (openal): Update to 1.17.2. > --- > gnu/packages/audio.scm | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/gnu/packages/audio.scm b/gnu/packages/audio.scm > index 9f03368..40cc029

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

2016-09-08 Thread Efraim Flashner
On Wed, Sep 07, 2016 at 09:00:02PM -0400, Erik Edrosa wrote: > * gnu/packages/messaging.scm (utox): New variable. > --- > gnu/packages/messaging.scm | 44 +++- > 1 file changed, 43 insertions(+), 1 deletion(-) > > diff --git a/gnu/packages/messaging.scm

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

2016-09-08 Thread Efraim Flashner
On Wed, Sep 07, 2016 at 09:00:01PM -0400, Erik Edrosa wrote: > * gnu/packages/audio.scm (filteraudio): New variable. > --- > gnu/packages/audio.scm | 31 +++ > 1 file changed, 31 insertions(+) > > diff --git a/gnu/packages/audio.scm b/gnu/packages/audio.scm > index

Re: NPM and trusted binaries

2016-09-08 Thread Jan Nieuwenhuizen
Mike Gerwitz writes: > If a user is able to build from source That's a question that I like to explore. If a user builds an npm package from its source repository, I assume that they install the devDependencies needed for that using npm? The transitive closure of installing all devDependencies

Re: "move-phase-after"?

2016-09-08 Thread Hartmut Goebel
Am 07.09.2016 um 14:20 schrieb Ludovic Courtès: > Yeah, that’s what I would do. I don’t think we need to extend > ‘modify-phases’ just for this. Okay. So I'll hack it into the build instructions. -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com

Re: NPM and trusted binaries

2016-09-08 Thread Jelle Licht
Just a quick note from me; AFAIK, the http module is a built-in of node, so you can probably save yourselves the efforts of packaging it ;-). Furthermore, lots of development dependencies are not strictly necessary; e.g. a minifier/uglifier is not required for most functionality of a package,

Re: [PATCH] gnu: Add git-review.

2016-09-08 Thread Vincent Legoll
Hello, On Thu, Sep 8, 2016 at 9:06 AM, Clément Lassieur wrote: > * gnu/packages/openstack.scm (git-review): New variable. Yay ! Cannot comment on the input / native-input thing, but otherwise LGTM... BTW, do any high-level Guix contributor have experience with

[PATCH] gnu: Add git-review.

2016-09-08 Thread Clément Lassieur
* gnu/packages/openstack.scm (git-review): New variable. --- gnu/packages/openstack.scm | 29 + 1 file changed, 29 insertions(+) diff --git a/gnu/packages/openstack.scm b/gnu/packages/openstack.scm index 8474904..2a54a20 100644 --- a/gnu/packages/openstack.scm +++

Re: [PATCH] gnu: Use a directory owned by ntpd user for drift file.

2016-09-08 Thread Vincent Legoll
Hello, > +(define (ntp-service-activation config) > + "Return the activation gexp for config" > + #~(begin > + (use-modules (guix build utils)) > + > + (define %user > +(getpw "ntpd")) > + > + (let ((directory "/var/run/ntpd")) > + (mkdir-p directory) > +

Re: NPM and trusted binaries

2016-09-08 Thread Pjotr Prins
On Wed, Sep 07, 2016 at 07:51:46PM +0200, Jan Nieuwenhuizen wrote: > Ludovic Courtès writes: > > >> Still, I think Guix would benefit from a somewhat more relaxed stance > >> in this. > > > > It’s part of Guix’s mission to build from source whenever that is > > possible, which is the case here,