Re: octave license is incompatible with openssl

2016-08-09 Thread Alex Vong
Mike Miller writes: > On Wed, Aug 10, 2016 at 00:00:59 +0800, Alex Vong wrote: >> Hi octave devs, >> >> During a look of the octave package in guix (a functional package >> manager, part of gnu), we notice octave have an optional dependency on >> openssl. >> >> However, since the license of oct

Re: octave license is incompatible with openssl

2016-08-09 Thread Mark H Weaver
k...@openmailbox.org writes: > On 2016-08-09 14:30, Ricardo Wurmus wrote: >> Mike Miller writes: >> >>> On Wed, Aug 10, 2016 at 00:00:59 +0800, Alex Vong wrote: So, we drop the optional dependency to avoid the problem. >>> >>> Precisely what is the optional dependency that is dropped? >>> >>

Re: octave license is incompatible with openssl

2016-08-09 Thread Mark H Weaver
Mike Miller writes: > I would recommend that you use a libcurl that is built against GnuTLS > as we do on Debian. Our libcurl is already built against GnuTLS. Mark

Re: [PATCH] daemon: Break CHROOT_ENABLED into CHROOT_ENABLED and CLONE_ENABLED.

2016-08-09 Thread Mark H Weaver
Manolis Ragkousis writes: > I was looking at nix's git repo and Eelco's 8f67325 commit is a better > solution to the issue. I cherry picked it and modified it to apply to > our version of the daemon which I will send in another mail. > > For this reason forget this patch. Okay, thanks for lookin

Re: [PATCH] Fix artanis paths

2016-08-09 Thread Mark H Weaver
Ignacio Torres Masdeu writes: > The last patch[1] for artanis added the DESTDIR path to place files in > $DESTDIR/etc. > > [1] http://lists.gnu.org/archive/html/guix-devel/2016-03/msg00813.html > > This didn't account for the fact that $DESTDIR is used in the build > system install process as a p

[PATCH 1/3] guix: ruby-build-system: Build compiled gems reproducibly.

2016-08-09 Thread Ben Woodcroft
* guix/build/ruby-build-system.scm (log-file-deletion): New procedure. (install): Remove files containing non-reproducible elements. Print when each file is deleted. --- guix/build/ruby-build-system.scm | 38 -- 1 file changed, 32 insertions(+), 6 deletions(-)

[PATCH 3/3] gnu: ruby-concurrent: Adjust for 'replace-git-ls-files'.

2016-08-09 Thread Ben Woodcroft
* gnu/packages/ruby.scm (ruby-concurrent)[arguments]: Adjust for new build phase 'replace-git-ls-files'. --- gnu/packages/ruby.scm | 15 ++- 1 file changed, 6 insertions(+), 9 deletions(-) diff --git a/gnu/packages/ruby.scm b/gnu/packages/ruby.scm index 0ff7d56..88bb306 100644 --- a/g

[PATCH 2/3] guix: ruby-build-system: Add replace-git-ls-files.

2016-08-09 Thread Ben Woodcroft
* guix/build/ruby-build-system.scm (replace-git-ls-files): New variable. (%standard-phases): Add it. --- guix/build/ruby-build-system.scm | 19 +++ 1 file changed, 15 insertions(+), 4 deletions(-) diff --git a/guix/build/ruby-build-system.scm b/guix/build/ruby-build-system.scm ind

[PATCH 0/3]; ruby-build-system improvements.

2016-08-09 Thread Ben Woodcroft
Hi all, Attached is a patch to make the building gems with native extensions reproducible, and a another to introduce a phase to remove `git ls-files` in Rakefiles as suggested by Dave: http://lists.gnu.org/archive/html/guix-devel/2016-06/msg00262.html With these patches and some other small fixe

Re: [PATCH] gnu: Add stress-make.

2016-08-09 Thread Eric Bavier
On Sat, 6 Aug 2016 21:59:16 -0400 Leo Famulari wrote: > On Tue, Aug 02, 2016 at 11:53:33AM -0500, Eric Bavier wrote: > > Hello Guix, > > > > I'm mostly looking for a second-opinion on the license of this package. > > Other comments welcome too, of course. > > > > * gnu/packages/debug.scm (stres

Re: octave license is incompatible with openssl

2016-08-09 Thread Mike Miller
On Wed, Aug 10, 2016 at 00:00:59 +0800, Alex Vong wrote: > Hi octave devs, > > During a look of the octave package in guix (a functional package > manager, part of gnu), we notice octave have an optional dependency on > openssl. > > However, since the license of octave (gpl3+) is incompatible > w

Re: octave license is incompatible with openssl

2016-08-09 Thread kei
On 2016-08-09 14:30, Ricardo Wurmus wrote: Mike Miller writes: On Wed, Aug 10, 2016 at 00:00:59 +0800, Alex Vong wrote: So, we drop the optional dependency to avoid the problem. Precisely what is the optional dependency that is dropped? Octave does not directly link with OpenSSL nor use an

Re: [PATCH] gnu: Add p7zip.

2016-08-09 Thread Ricardo Wurmus
Kei Kebreau writes: > Ricardo Wurmus writes: > >> k...@openmailbox.org writes: >> >>> On 2016-08-09 04:05, Ricardo Wurmus wrote: Leo Famulari writes: >> > I see that you took care of removing non-free parts in a patch and a >> > build phase. Since “guix build -S p7zip” does

Re: [PATCH] gnu: Add p7zip.

2016-08-09 Thread Kei Kebreau
Ricardo Wurmus writes: > k...@openmailbox.org writes: > >> On 2016-08-09 04:05, Ricardo Wurmus wrote: >>> Leo Famulari writes: >>> > > I see that you took care of removing non-free parts in a patch and a > > build phase. Since “guix build -S p7zip” doesn’t run the build phase > > u

Re: [PATCH 2/5] gnu: Add avr-gcc.

2016-08-09 Thread Ricardo Wurmus
Danny Milosavljevic writes: > Hi, > > hmm, does avr-gcc work for anyone on Guix? > > I tried to compile a simple Arduino project and I get: > > /home/dannym/.guix-profile/bin/avr-g++ -x c++ -include Arduino.h -MMD -c > -mmcu=atmega32u4 -DF_CPU=1600L -DARDUINO=100 -DARDUINO_ARCH_AVR > -D__P

Re: [PATCH] gnu: Add ircii.

2016-08-09 Thread Leo Famulari
On Tue, Aug 09, 2016 at 09:38:55PM +0200, Ricardo Wurmus wrote: I actually applied a tweaked version of this patch with c744f5c0ad. But, the improvements you suggest should be applied. > > + (lambda* (#:key inputs #:allow-other-keys) > > + (substitute* "bsdinstall" > > +

Re: [PATCH] gnu: Add ircii.

2016-08-09 Thread Ricardo Wurmus
ng0 writes: > From 1438e5654cfd5f4eb4141a42580bd73c716e7134 Mon Sep 17 00:00:00 2001 > From: ng0 > Date: Mon, 1 Aug 2016 19:31:23 + > Subject: [PATCH] gnu: Add ircii. > * gnu/packages/irc.scm (ircii): New variable. > --- > gnu/packages/irc.scm | 46

Re: [PATCH 2/5] gnu: Add avr-gcc.

2016-08-09 Thread Danny Milosavljevic
Hi, hmm, does avr-gcc work for anyone on Guix? I tried to compile a simple Arduino project and I get: /home/dannym/.guix-profile/bin/avr-g++ -x c++ -include Arduino.h -MMD -c -mmcu=atmega32u4 -DF_CPU=1600L -DARDUINO=100 -DARDUINO_ARCH_AVR -D__PROG_TYPES_COMPAT__ -Iarduino-1.6.10/hardware/a

Re: [PATCH] gnu: Add p7zip.

2016-08-09 Thread Ricardo Wurmus
k...@openmailbox.org writes: > On 2016-08-09 04:05, Ricardo Wurmus wrote: >> Leo Famulari writes: >> > I see that you took care of removing non-free parts in a patch and a > build phase. Since “guix build -S p7zip” doesn’t run the build phase > users would still end up with a so

Re: QtHaveModule

2016-08-09 Thread David Craven
Hi Andreas, > This makes me think that "qtHaveModule" does not work; and I wonder whether > in our other Qt modules that have Qt modules as input, these are actually > detected correctly. Well, I tried qtsensors, and did not notice this problem. Trying to package the missing qt modules I came to

Re: [PATCH v2] gnu: Add cbatticon.

2016-08-09 Thread Ricardo Wurmus
ng0 writes: > From ac578d27529cc2a5f39f66054b5991e44e65f0b9 Mon Sep 17 00:00:00 2001 > From: ng0 > Date: Tue, 9 Aug 2016 16:47:37 + > Subject: [PATCH] gnu: Add cbatticon. > * gnu/packages/admin.scm (cbatticon): New variable. > --- > gnu/packages/admin.scm | 46

[PATCH] gnu: Add sic.

2016-08-09 Thread ng0
This adds sic, another suckless irc client. I'm not sure about the description. Otherwise, it just works. From 1456c1d176bbbd0173a88959d8d9b5a61f014d0b Mon Sep 17 00:00:00 2001 From: ng0 Date: Tue, 9 Aug 2016 18:40:26 + Subject: [PATCH] gnu: Add sic. * gnu/packages/irc.scm (sic): New variab

Re: [PACKAGE] sinit

2016-08-09 Thread Ricardo Wurmus
Vincent Legoll writes: > It (#:make-flags?) was already there in the scm I submitted, but that > wasn't working when I tested it first time, maybe because I did not > know about guix environment at that time and tried the build it outside > of one... > > I'll clean that up and resubmit properly.

Re: octave license is incompatible with openssl

2016-08-09 Thread Ricardo Wurmus
Mike Miller writes: > On Wed, Aug 10, 2016 at 00:00:59 +0800, Alex Vong wrote: >> So, we drop the optional dependency to avoid the problem. > > Precisely what is the optional dependency that is dropped? > > Octave does not directly link with OpenSSL nor use any OpenSSL > functions. The Octave pa

Re: [PATCH] daemon: Break CHROOT_ENABLED into CHROOT_ENABLED and CLONE_ENABLED.

2016-08-09 Thread Manolis Ragkousis
Hello again, I was looking at nix's git repo and Eelco's 8f67325 commit is a better solution to the issue. I cherry picked it and modified it to apply to our version of the daemon which I will send in another mail. For this reason forget this patch. Thank you, Manolis On 08/08/16 15:25, Manolis

[PATCH] gnu: Add cbatticon.

2016-08-09 Thread ng0
This adds cbatticon, a small lightweight battery icon for the system tray. From bb91f4adbb3a9e0ac032c7287f35e84b325901dd Mon Sep 17 00:00:00 2001 From: ng0 Date: Tue, 9 Aug 2016 16:02:46 + Subject: [PATCH] gnu: Add cbatticon. * gnu/packages/admin.scm (cbatticon): New variable. --- gnu/packa

Re: [PATCH] gnu: Add p7zip.

2016-08-09 Thread kei
On 2016-08-09 04:05, Ricardo Wurmus wrote: Leo Famulari writes: > I see that you took care of removing non-free parts in a patch and a > build phase. Since “guix build -S p7zip” doesn’t run the build phase > users would still end up with a source archive containing the non-free > parts. The

Re: octave license is incompatible with openssl

2016-08-09 Thread Alex Vong
Hi octave devs, During a look of the octave package in guix (a functional package manager, part of gnu), we notice octave have an optional dependency on openssl. However, since the license of octave (gpl3+) is incompatible with that of openssl (https://people.gnome.org/~markmc/openssl-and-the-gpl

Re: [PATCH 05/11] gnu: libunistring: support mingw: propagate libiconv if needed.

2016-08-09 Thread Jan Nieuwenhuizen
Alex Kost writes: >> - (propagated-inputs '()) ; FIXME: add libiconv when >> !glibc >> + (propagated-inputs `(,@(libiconv-if-needed))) > > Or simply: > > (propagated-inputs (libiconv-if-needed)) Ok, thanks. Greetings, Jan -- Jan Nieuwenhuizen | GNU LilyPond http://

Re: [PATCH 07/11] gnu: ncurses: support mingw.

2016-08-09 Thread Jan Nieuwenhuizen
Alex Kost writes: >> + (mingw-target? (lambda* (#:optional (target target)) >> + (and target >> +(string-suffix? "-mingw32" >> target > > IIUC you don't use the same 'mingw-target?' procedure you added

[PATCH v2] gnu: Add cbatticon.

2016-08-09 Thread ng0
Last patch had mistakes. Fixed this. From ac578d27529cc2a5f39f66054b5991e44e65f0b9 Mon Sep 17 00:00:00 2001 From: ng0 Date: Tue, 9 Aug 2016 16:47:37 + Subject: [PATCH] gnu: Add cbatticon. * gnu/packages/admin.scm (cbatticon): New variable. --- gnu/packages/admin.scm | 46 +++

Re: [PATCH 08/11] gnu: cross-base: Add cross-libtool.

2016-08-09 Thread Jan Nieuwenhuizen
Alex Kost writes: >> + (add-before 'configure 'setenv >> +(lambda* (#:key inputs native-inputs #:allow-other-keys) > > It looks like 'native-inputs' is not needed, as it is not used further. Indeed, dropped. >> + (let ((xgcc (assoc-ref in

Re: [PATCH 02/11] gnu: Add mingw-w64.

2016-08-09 Thread Jan Nieuwenhuizen
Alex Kost writes: > Hello, I understand nothing in all this mingw stuff, I just have a couple > of insignificant comments. Thanks for looking, they all make sense and I have addressed most. With MinGW, you can cross build Windows binaries and run them with Wine, like so ./pre-inst-env guix

Re: mingw guile.exe cross build patch series v9

2016-08-09 Thread Jan Nieuwenhuizen
Andy Wingo writes: >> Now that core-updates has been merged the mingw cross patches build again. > > All looks fine to me, though Alex's nits are good of course I'd be happy > to have this in Guix and then we work on it some more in place. I have addressed all but one of Alex's comments, but not

[PATCH] Fix artanis paths

2016-08-09 Thread Ignacio Torres Masdeu
The last patch[1] for artanis added the DESTDIR path to place files in $DESTDIR/etc. [1] http://lists.gnu.org/archive/html/guix-devel/2016-03/msg00813.html This didn't account for the fact that $DESTDIR is used in the build system install process as a prefix for other files and the effect was tha

Re: [PATCH v2] gnu: node: Update to 6.3.1.

2016-08-09 Thread Thompson, David
On Tue, Aug 9, 2016 at 9:13 AM, Jelle Licht wrote: > > Attached you will find the updated version of this patch. Please let me > know what you think. I could point out a few nits about the commit message, but I think someone with commit access should just clean it up and push. The code itself lo

Re: [PATCH v2] gnu: node: Update to 6.3.1.

2016-08-09 Thread Jelle Licht
Leo Famulari writes: > On Sun, Aug 07, 2016 at 02:45:20PM +0200, Jelle Licht wrote: >> Leo Famulari writes: >> >> - (replace 'patch-shebangs >> >> - (lambda* (#:key outputs #:allow-other-keys #:rest all) >> >> - ;; Work around . >> >> -

Re: [PATCH] Add Elixir

2016-08-09 Thread Alex Sassmannshausen
Pjotr Prins writes: > GNU Guix is mentioned on the Elixir installation page: > > http://elixir-lang.org/install.html > > Pj. Neat! Congrats to both of your hard work on this! Alex

Re: [PATCH 05/26] gnu: kde-frameworks: Add bluez-qt.

2016-08-09 Thread 宋文武
David Craven writes: > * gnu/packages/kde-frameworks.scm (bluez-qt): New variable. > > Co-authored-by: Hartmut Goebel > --- > gnu/packages/kde-frameworks.scm | 35 +++ > 1 file changed, 35 insertions(+) > > diff --git a/gnu/packages/kde-frameworks.scm b/gnu/packa

Re: [PATCH] Add Elixir

2016-08-09 Thread Pjotr Prins
GNU Guix is mentioned on the Elixir installation page: http://elixir-lang.org/install.html Pj.

Re: [PATCHv2 2/2] gnu: bash-completion: Update to 2.3. Update the patch.

2016-08-09 Thread Tomáš Čech
On Mon, Aug 08, 2016 at 05:14:41PM -0400, Leo Famulari wrote: On Sun, Aug 07, 2016 at 12:07:15PM +0200, Tomáš Čech wrote: * gnu/packages/bash.scm(bash-completion): Update to 2.3. Same nit-pick about leaving a space before the variable name :) * gnu/packages/patches/bash-completion-directorie

Re: mingw guile.exe cross build patch series v9

2016-08-09 Thread Andy Wingo
On Tue 09 Aug 2016 08:41, Jan Nieuwenhuizen writes: > Finally I have the next iteration ready. After adressing all comments on v8 > and rebasing, the patches only built on core-updates. I spent a some time to > find the problem with master and cherry-picking from core-updates before > giving up

Re: gnunet-fs integration

2016-08-09 Thread ng0
n...@we.make.ritual.n0.is writes: > Hi, > > Ludovic Courtès writes: > >> Hello, >> >> n...@we.make.ritual.n0.is skribis: >> >>> While I'm still new to guile and guix, one of my main motivations >>> since I started contributing to guix was an item on the end of >>> the roadmap, using a decentralize

Re: [PATCH 08/11] gnu: cross-base: Add cross-libtool.

2016-08-09 Thread Alex Kost
Jan Nieuwenhuizen (2016-08-09 09:41 +0300) wrote: > * gnu/packages/cross-base.scm (cross-libtool): New function. > --- > gnu/packages/cross-base.scm | 52 > + > 1 file changed, 52 insertions(+) > > > diff --git a/gnu/packages/cross-base.scm b/gnu/packa

Re: [PATCH] gnu: Add p7zip.

2016-08-09 Thread Ricardo Wurmus
Leo Famulari writes: >> > I see that you took care of removing non-free parts in a patch and a >> > build phase. Since “guix build -S p7zip” doesn’t run the build phase >> > users would still end up with a source archive containing the non-free >> > parts. The best way is thus to remove things

Re: [PATCH 07/11] gnu: ncurses: support mingw.

2016-08-09 Thread Alex Kost
Jan Nieuwenhuizen (2016-08-09 09:41 +0300) wrote: > * gnu/packages/patches/ncurses-mingw.patch: New file. > * gnu-system.am (dist_patch_DATA): Add it. > * gnu/packages/ncurses.scm (ncurses): Support mingw. [...] > (define-public ncurses > @@ -36,6 +39,7 @@ > #:allow-other-key

Re: [PATCH 05/11] gnu: libunistring: support mingw: propagate libiconv if needed.

2016-08-09 Thread Alex Kost
Jan Nieuwenhuizen (2016-08-09 09:41 +0300) wrote: [...] > (define-public libunistring >(package > @@ -35,7 +37,7 @@ > (sha256 > (base32 >"0ixxmgpgh2v8ifm6hbwsjxl023myk3dfnj7wnvmqjivza31fw9cn" > - (propagated-inputs '()) ; FIXME

Re: [PATCH 02/11] gnu: Add mingw-w64.

2016-08-09 Thread Alex Kost
Hello, I understand nothing in all this mingw stuff, I just have a couple of insignificant comments. Jan Nieuwenhuizen (2016-08-09 09:41 +0300) wrote: > * gnu/packages/patches/gcc-4.9.3-mingw-gthr-default.patch, > gnu/packages/patches/mingw-w64-5.0rc2-gcc-4.9.3.patch, > gnu/packages/mingw.scm: Ne

Re: gnunet-fs integration

2016-08-09 Thread ng0
n...@we.make.ritual.n0.is writes: > Hi, > > While I'm still new to guile and guix, one of my main motivations > since I started contributing to guix was an item on the end of > the roadmap, using a decentralized system to distribute updates. > > I have started to read through the scripts and under