flashrom package needs update

[ng0]

Someone should update flashrom to 0.9.9. 
(https://www.flashrom.org/Flashrom/0.9.9)
I tried and do not understand enough of flashrom to update the patch we need.
-- 
ng0
For non-prism friendly talk find me on http://www.psyced.org

Re: [patch] python-numpy, python-matplotlib and openblas update, python-cycler added

[Ricardo Wurmus]

Pjotr Prins  writes:

> Hi Ricardo,
>
> I ran into the eigen issue of numpy+openblas.
>
> See https://github.com/xianyi/OpenBLAS/issues/703
>
> The good news is that openblas now appears to have fixed eigen
> support. It was broken on numpy and the upgrades fixed the segfaults.

Thanks for the heads-up.  OpenBLAS has been updated in core-updates
already.

I cannot take care of the diff you sent any time soon (busy at work and
with other patches), so if someone else would like to turn this into a
proper commit please do so.

~~ Ricardo

Re: [PATCH][core-updates] gnu: gnupg, libgpg-error updates rebased

[Leo Famulari]

On Thu, Aug 18, 2016 at 07:27:14PM +, ng0 wrote:
> Leo Famulari  writes:
> 
> > On Thu, Aug 18, 2016 at 06:34:24PM +, ng0 wrote:
> >> GnuPG 2.1.15 has just been released, but please apply this patch, I'll
> >> have 2.1.15 ready later and send a separate patch of it based on this one.
> >
> > I applied your update patch for libgpg-error, but I rewrote the gnupg
> > update patch from scratch.
> >
> > So, no need for a "gnupg: Update to 2.1.15" patch. It's already done.
> 
> I'm not so sure. My build of it failed when the tests happened, which is
> why my patch included a fix for that:
> https://lists.gnu.org/archive/html/guix-devel/2016-08/msg01276.html

Ah, you're right. Can you send a patch for core-updates?

Re: [PATCH 24/24] gnu: qt: Update qt to 5.7.0.

[Mark H Weaver]

Hi David,

David Craven  writes:

> * gnu/packages/qt.scm
>   (qtbase): Update it.
>   (qtsvg): Update it.
>   (qtimageformats): Update it.
>   (qtx11extras): Update it.
>   (qtxmlpatterns): Update it.
>   (qtdeclarative): Update it.
>   (qtconnectivity): Update it.
>   (qtwebsockets): Update it.
>   (qtsensors): Update it.
>   (qtmultimedia): Update it.
>   (qtwayland): Update it.
>   (qtserialport): Update it.
>   (qtwebchannel): Update it.
>   (qtlocation): Update it.
>   (qttools): Update it.
>   (qtscript): Update it.
>   (qtquickcontrols): Update it.
>   (qtquickcontrols2): Update it.
>   (qtgraphicaleffects): Update it.

Thank you for all of this!  Hydra indicates that there are some
regressions.  Can you take a look?

  https://hydra.gnu.org/eval/109084#tabs-now-fail
  https://hydra.gnu.org/eval/109084#tabs-new

  Mark

Re: [PATCH] gnu: Add interrobang.

[Leo Famulari]

On Wed, Aug 17, 2016 at 08:24:37PM +, ng0 wrote:
> This patch adds interrobang. It should function, but I don't have zsh
> configured here in GuixSD, so my old config does not work here, and the
> system default is up to testing by other people.

> * gnu/packages/admin.scm (interrobang): New variable.

Thanks, pushed as c08533b2c with the following changes!

> +  (version (string-append "0.0.0." revision "." (string-take commit 8)))
   ^   ^
Characters replaced by -   7

> +  (synopsis "Tiny launcher menu packing a big bang syntax")

"Scriptable launcher menu"

> +  (license license:gpl3

gpl3+

Re: [PATCH] gnu: Add python-cheetah.

[Leo Famulari]

On Tue, Aug 16, 2016 at 02:15:25PM +0300, Manolis Ragkousis wrote:
> This patch adds python2-cheetah.

Does this python2-cheetah work as expected? Needing to make changes to
the Python 2 version of a package is the reason we introduced the
python2-variant system.

> +(define-public python2-cheetah
> +  (let ((base (package-with-python2 python-cheetah)))
> +(package
> +  (inherit base)
> +  (name "python2-cheetah")
> +  (native-inputs
> +   `(("python2-setuptools" ,python2-setuptools)
> + ("python2-markdown" ,python2-markdown)
> + ,@(package-native-inputs base))
> -- 
> 2.9.2
> 

Re: [PATCH] gnu: bs1770gain: Update to 0.4.11.

[Leo Famulari]

On Wed, Aug 17, 2016 at 11:07:53AM -0500, Alex Griffin wrote:
> This patch just updates bs1770gain to 0.4.11, which includes a number of
> bug fixes.
> -- 
> Alex Griffin

> From c08d5616bdb579780d910429bd81ec7ec36491f3 Mon Sep 17 00:00:00 2001
> From: Alex Griffin 
> Date: Tue, 16 Aug 2016 13:58:07 -0500
> Subject: [PATCH 1/2] gnu: bs1770gain: Update to 0.4.11.
> 
> * gnu/packages/audio.scm (bs1770gain): Update to 0.4.11.

Thanks, pushed as 1166c283b7!

Re: [PATCH] gnu: Add re2.

[Leo Famulari]

On Wed, Aug 17, 2016 at 04:18:48PM +0100, Marius Bakke wrote:
> Marius Bakke  writes:
> 
> > Leo Famulari  writes:
> >
> >> On Mon, Aug 15, 2016 at 02:05:16PM +0100, Marius Bakke wrote:
> >>> 
> >>> I wasn't sure where to put this, so went with its own file. It does not
> >>> fully implement PCRE so pcre.scm seems inappropriate. Perhaps that could
> >>> be renamed to regex.scm or similar.
> >>
> >> The patch looks good, but please put it in regex.scm! But, I don't want
> >> to move all the regex packages into this new regex module. Perhaps tre,
> >> since it appears to have no users in our tree.
> >
> > Moved to regex.scm. Thanks!
> 
> ..and here is a patch that moves tre.scm over as well. The code is
> unchanged apart from license: prefix.

When moving packages around, all users of the package need to have their
module imports updated. Copyright attribution must be carefully handled.
And merging the various *-updates branches into master and vice versa
becomes more complicated and prone to error.

I *think* this patch does it right. And it will really grate on my sense
of aesthetics to have both regex.scm and tre.scm. But, in general, do we
want to make this change? What does everyone think?

Re: [PATCH] gnu: Add re2.

[Leo Famulari]

On Wed, Aug 17, 2016 at 11:34:14AM +0100, Marius Bakke wrote:
> Leo Famulari  writes:
> 
> > On Mon, Aug 15, 2016 at 02:05:16PM +0100, Marius Bakke wrote:
> >> 
> >> I wasn't sure where to put this, so went with its own file. It does not
> >> fully implement PCRE so pcre.scm seems inappropriate. Perhaps that could
> >> be renamed to regex.scm or similar.
> >
> > The patch looks good, but please put it in regex.scm! But, I don't want
> > to move all the regex packages into this new regex module. Perhaps tre,
> > since it appears to have no users in our tree.
> 
> Moved to regex.scm. Thanks!

Thanks, pushed as dc253bf0a5!

Re: [PATCH] gnu: Add dlib.

[Leo Famulari]

On Wed, Aug 17, 2016 at 03:48:24PM +0100, Marius Bakke wrote:
> I've attached a patch with a #t in the disable-asserts phase, and also
> deleting the 6MB static library.
> 
> Since `guix build --rounds=2` passes, is there any reason to delay this
> patch? I've built this on GuixSD and a foreign distro and naively
> verified that they are the same (the .so and all headers have the same
> checksum; some of the recorded cmake input paths are different though).

I pushed the patch as 5f0ff6a9e. Hopefully dlib is still useful without
lapack. We should really figure out what the issue is and fix it :)

Re: [PATCH] gnu: Add dlib.

[Leo Famulari]

On Wed, Aug 17, 2016 at 01:24:46PM +1000, Ben Woodcroft wrote:
> On 17/08/16 09:45, Leo Famulari wrote:
> > How did it appear non-deterministic to you?
> Just based on guix build --check:
> 
> guix build: error: build failed: derivation
> `/gnu/store/sxybcxw64q1ajzq6dysal75ffgq6238i-dlib-19.1.drv' may not be
> deterministic: output
> `/gnu/store/il57dcii4pzii11zlixjjxxxw699bg5x-dlib-19.1' differs
> 
> I'm actually not sure, why does it say "may not be deterministic"? If it
> builds twice and the second version is different, doesn't that mean it is
> definitely not deterministic by counter-example, unless there has been some
> leakage into the build container?

Ah yes, that checks if the built output is bit-for-bit identical. I
thought that you meant there was some non-determinism in the build
system, like a race condition caused by parallel building. Which could
also be true.

Re: 19/36: gnu: Add yaml-cpp.

[Leo Famulari]

On Thu, Aug 18, 2016 at 03:58:29PM +, David Craven wrote:
> dvc pushed a commit to branch master
> in repository guix.
> 
> commit f50f4ae454ab8e32c3b43541c493ba652ec03a05
> Author: David Craven 
> Date:   Wed Aug 17 12:22:03 2016 +0200
> 
> gnu: Add yaml-cpp.
> 
> * gnu/packages/serialization.scm (yaml-cpp): New variable.

> +(native-inputs
> + `(("python" ,python)))

I imported the python module in f47dc8dd10.

Re: 13/36: gnu: Add gpgmepp.

[Leo Famulari]

On Thu, Aug 18, 2016 at 03:58:29PM +, David Craven wrote:
> dvc pushed a commit to branch master
> in repository guix.
> 
> commit 30ffab50063c2cc8a4523d2eb110be9ebe388f51
> Author: David Craven 
> Date:   Mon Aug 8 17:49:43 2016 +0200
> 
> gnu: Add gpgmepp.
> 
> * gnu/packages/kde-frameworks.scm (gpgmepp): New variable.

> +(propagated-inputs
> + `(("boost" ,boost)
> +   ("gpgme" ,gpgme)))

I imported the boost and gnupg modules in a34858554. Please remember to
check that new packages build before pushing to Savannah :)

Re: [PATCH][core-updates] gnu: gnupg, libgpg-error updates rebased

[ng0]

Leo Famulari  writes:

> On Thu, Aug 18, 2016 at 06:34:24PM +, ng0 wrote:
>> GnuPG 2.1.15 has just been released, but please apply this patch, I'll
>> have 2.1.15 ready later and send a separate patch of it based on this one.
>
> I applied your update patch for libgpg-error, but I rewrote the gnupg
> update patch from scratch.
>
> So, no need for a "gnupg: Update to 2.1.15" patch. It's already done.

I'm not so sure. My build of it failed when the tests happened, which is
why my patch included a fix for that:
https://lists.gnu.org/archive/html/guix-devel/2016-08/msg01276.html

It sets $HOME, so that the tests of gnupg-2.1.15 can succeed.
-- 
ng0
For non-prism friendly talk find me on http://www.psyced.org

[PATCH][core-updates] gnu: gnupg: Update to 2.1.15.

[ng0]

Requires previous 2.1.14 update patch.

>From 0044af341b0f01d2a325404141ff3168ab11da69 Mon Sep 17 00:00:00 2001
From: ng0 
Date: Thu, 18 Aug 2016 18:55:44 +
Subject: [PATCH] gnu: gnupg: Update to 2.1.15.

* gnu/packages/gnupg.scm (gnupg): Update to 2.1.15.
---
 gnu/packages/gnupg.scm | 12 +---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/gnu/packages/gnupg.scm b/gnu/packages/gnupg.scm
index 4edc282..0c20bab 100644
--- a/gnu/packages/gnupg.scm
+++ b/gnu/packages/gnupg.scm
@@ -209,14 +209,14 @@ compatible to GNU Pth.")
 (define-public gnupg
   (package
 (name "gnupg")
-(version "2.1.14")
+(version "2.1.15")
 (source (origin
   (method url-fetch)
   (uri (string-append "mirror://gnupg/gnupg/gnupg-" version
   ".tar.bz2"))
   (sha256
(base32
-"0hmsiscpdpdqd8kcjpzkz2gzcc3cnrvswk9p1jzi4sivd7lxwl4l"
+"1pgz02gd84ab94w4xdg67p9z8kvkyr9d523bvcxxd2hviwh1m362"
 (build-system gnu-build-system)
 (native-inputs
  `(("pkg-config" ,pkg-config)))
@@ -247,7 +247,13 @@ compatible to GNU Pth.")
   (lambda _
 (substitute* (find-files "tests" ".\\.scm$")
   (("/usr/bin/env gpgscm")
-   (string-append (getcwd) "/tests/gpgscm/gpgscm"
+   (string-append (getcwd) "/tests/gpgscm/gpgscm")
+(add-before 'check 'set-HOME
+  ;; Some tests require write access to $HOME, otherwise leading to
+  ;; "failed to create directory /homeless-shelter/.asy" error.
+  (lambda _
+(setenv "HOME" "/tmp")
+#t)
 (home-page "https://gnupg.org/";)
 (synopsis "GNU Privacy Guard")
 (description
-- 
2.9.3


-- 
ng0
For non-prism friendly talk find me on http://www.psyced.org

Re: [PATCH][core-updates] gnu: gnupg, libgpg-error updates rebased

[Leo Famulari]

On Thu, Aug 18, 2016 at 06:34:24PM +, ng0 wrote:
> GnuPG 2.1.15 has just been released, but please apply this patch, I'll
> have 2.1.15 ready later and send a separate patch of it based on this one.

I applied your update patch for libgpg-error, but I rewrote the gnupg
update patch from scratch.

So, no need for a "gnupg: Update to 2.1.15" patch. It's already done.

Re: [PATCH][core-updates] gnu: gnupg, libgpg-error updates rebased

[ng0]

GnuPG 2.1.15 has just been released, but please apply this patch, I'll
have 2.1.15 ready later and send a separate patch of it based on this one.

ng0  writes:

> These patches are applied directly on core-updates.
>
> From b63701cacb87653f6b128aac0046fa3000f8baed Mon Sep 17 00:00:00 2001
> From: ng0 
> Date: Thu, 18 Aug 2016 13:38:20 +
> Subject: [PATCH 1/2] gnu: libgpg-error: Update to 1.24.
>
> * gnu/packages/gnupg.scm (libgpg-error): Update to 1.24.
> ---
>  gnu/packages/gnupg.scm | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/gnu/packages/gnupg.scm b/gnu/packages/gnupg.scm
> index 69adf97..989496f 100644
> --- a/gnu/packages/gnupg.scm
> +++ b/gnu/packages/gnupg.scm
> @@ -6,7 +6,7 @@
>  ;;; Copyright © 2015 Paul van der Walt 
>  ;;; Copyright © 2015, 2016 Efraim Flashner 
>  ;;; Copyright © 2016 Christopher Allan Webber 
> -;;; Copyright © 2016 Nils Gillmann 
> +;;; Copyright © 2016 ng0 
>  ;;;
>  ;;; This file is part of GNU Guix.
>  ;;;
> @@ -50,7 +50,7 @@
>  (define-public libgpg-error
>(package
>  (name "libgpg-error")
> -(version "1.22")
> +(version "1.24")
>  (source
>   (origin
>(method url-fetch)
> @@ -58,7 +58,7 @@
>version ".tar.bz2"))
>(sha256
> (base32
> -"0ywxwswizmkyciy480kzczxn6nhbgzf3z8my4nk43nvv67k4x87j"
> +"0h75sf1ngr750c3fjfn4583q7wz40qm63jhg8vjfdrbx936f2s4j"
>  (build-system gnu-build-system)
>  (home-page "https://gnupg.org";)
>  (synopsis "Library of error values for GnuPG components")
> -- 
> 2.9.3
>
> From a929ccb1ad245c791d0ddd18b77650d0967d615b Mon Sep 17 00:00:00 2001
> From: ng0 
> Date: Thu, 18 Aug 2016 15:25:07 +
> Subject: [PATCH 2/2] gnu: gnupg: Update to 2.1.14.
>
> * gnu/packages/gnupg.scm (gnupg): Update to 2.1.14.
>
> Co-authored by: Leo Famulari 
> ---
>  gnu/packages/gnupg.scm | 11 ---
>  1 file changed, 8 insertions(+), 3 deletions(-)
>
> diff --git a/gnu/packages/gnupg.scm b/gnu/packages/gnupg.scm
> index 989496f..4edc282 100644
> --- a/gnu/packages/gnupg.scm
> +++ b/gnu/packages/gnupg.scm
> @@ -209,14 +209,14 @@ compatible to GNU Pth.")
>  (define-public gnupg
>(package
>  (name "gnupg")
> -(version "2.1.13")
> +(version "2.1.14")
>  (source (origin
>(method url-fetch)
>(uri (string-append "mirror://gnupg/gnupg/gnupg-" version
>".tar.bz2"))
>(sha256
> (base32
> -"0xcn46vcb5x5qx0bc803vpzhzhnn6wfhp7x71w9n1ahx4ak877ag"
> +"0hmsiscpdpdqd8kcjpzkz2gzcc3cnrvswk9p1jzi4sivd7lxwl4l"
>  (build-system gnu-build-system)
>  (native-inputs
>   `(("pkg-config" ,pkg-config)))
> @@ -242,7 +242,12 @@ compatible to GNU Pth.")
>(lambda _
>  (substitute* "tests/openpgp/defs.inc"
>(("/bin/pwd") (which "pwd")))
> -#t)
> +#t))
> +(add-after 'build 'patch-scheme-tests
> +  (lambda _
> +(substitute* (find-files "tests" ".\\.scm$")
> +  (("/usr/bin/env gpgscm")
> +   (string-append (getcwd) "/tests/gpgscm/gpgscm"
>  (home-page "https://gnupg.org/";)
>  (synopsis "GNU Privacy Guard")
>  (description
> -- 
> 2.9.3
>
>
>
> -- 
> ng0
> For non-prism friendly talk find me on http://www.psyced.org

-- 
ng0
For non-prism friendly talk find me on http://www.psyced.org

Re: How to "backport" a fix?

[Leo Famulari]

On Thu, Aug 18, 2016 at 05:24:30PM +0200, Hartmut Goebel wrote:
> Hi,
> 
> thanks to Marius Bakke's tip django test run fine now -- except of two.
> These are errors in the test-case which have been fixed just after the
> release.
> 
> What is the proper way to handle this? Include this fix as a patch? As a
> snippet to the source? Or should I disable the two tests?

If there are upstream patches that fix broken tests, then I think we
should try taking those patches from upstream and applying them in the
(source) field of the package.

The patch files we use should include a brief comment explaining what
they do (the upstream commit message might suffice) and a link to their
source.

[PATCH][core-updates] gnu: gnupg, libgpg-error updates rebased

[ng0]

These patches are applied directly on core-updates.

>From b63701cacb87653f6b128aac0046fa3000f8baed Mon Sep 17 00:00:00 2001
From: ng0 
Date: Thu, 18 Aug 2016 13:38:20 +
Subject: [PATCH 1/2] gnu: libgpg-error: Update to 1.24.

* gnu/packages/gnupg.scm (libgpg-error): Update to 1.24.
---
 gnu/packages/gnupg.scm | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/gnu/packages/gnupg.scm b/gnu/packages/gnupg.scm
index 69adf97..989496f 100644
--- a/gnu/packages/gnupg.scm
+++ b/gnu/packages/gnupg.scm
@@ -6,7 +6,7 @@
 ;;; Copyright © 2015 Paul van der Walt 
 ;;; Copyright © 2015, 2016 Efraim Flashner 
 ;;; Copyright © 2016 Christopher Allan Webber 
-;;; Copyright © 2016 Nils Gillmann 
+;;; Copyright © 2016 ng0 
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -50,7 +50,7 @@
 (define-public libgpg-error
   (package
 (name "libgpg-error")
-(version "1.22")
+(version "1.24")
 (source
  (origin
   (method url-fetch)
@@ -58,7 +58,7 @@
   version ".tar.bz2"))
   (sha256
(base32
-"0ywxwswizmkyciy480kzczxn6nhbgzf3z8my4nk43nvv67k4x87j"
+"0h75sf1ngr750c3fjfn4583q7wz40qm63jhg8vjfdrbx936f2s4j"
 (build-system gnu-build-system)
 (home-page "https://gnupg.org";)
 (synopsis "Library of error values for GnuPG components")
-- 
2.9.3

>From a929ccb1ad245c791d0ddd18b77650d0967d615b Mon Sep 17 00:00:00 2001
From: ng0 
Date: Thu, 18 Aug 2016 15:25:07 +
Subject: [PATCH 2/2] gnu: gnupg: Update to 2.1.14.

* gnu/packages/gnupg.scm (gnupg): Update to 2.1.14.

Co-authored by: Leo Famulari 
---
 gnu/packages/gnupg.scm | 11 ---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/gnu/packages/gnupg.scm b/gnu/packages/gnupg.scm
index 989496f..4edc282 100644
--- a/gnu/packages/gnupg.scm
+++ b/gnu/packages/gnupg.scm
@@ -209,14 +209,14 @@ compatible to GNU Pth.")
 (define-public gnupg
   (package
 (name "gnupg")
-(version "2.1.13")
+(version "2.1.14")
 (source (origin
   (method url-fetch)
   (uri (string-append "mirror://gnupg/gnupg/gnupg-" version
   ".tar.bz2"))
   (sha256
(base32
-"0xcn46vcb5x5qx0bc803vpzhzhnn6wfhp7x71w9n1ahx4ak877ag"
+"0hmsiscpdpdqd8kcjpzkz2gzcc3cnrvswk9p1jzi4sivd7lxwl4l"
 (build-system gnu-build-system)
 (native-inputs
  `(("pkg-config" ,pkg-config)))
@@ -242,7 +242,12 @@ compatible to GNU Pth.")
   (lambda _
 (substitute* "tests/openpgp/defs.inc"
   (("/bin/pwd") (which "pwd")))
-#t)
+#t))
+(add-after 'build 'patch-scheme-tests
+  (lambda _
+(substitute* (find-files "tests" ".\\.scm$")
+  (("/usr/bin/env gpgscm")
+   (string-append (getcwd) "/tests/gpgscm/gpgscm"
 (home-page "https://gnupg.org/";)
 (synopsis "GNU Privacy Guard")
 (description
-- 
2.9.3



-- 
ng0
For non-prism friendly talk find me on http://www.psyced.org

How to "backport" a fix?

[Hartmut Goebel]

Hi,

thanks to Marius Bakke's tip django test run fine now -- except of two.
These are errors in the test-case which have been fixed just after the
release.

What is the proper way to handle this? Include this fix as a patch? As a
snippet to the source? Or should I disable the two tests?

-- 
Regards
Hartmut Goebel

| Hartmut Goebel  | h.goe...@crazy-compilers.com   |
| www.crazy-compilers.com | compilers which you thought are impossible |

[PATCH v2 3/5] gnu: Add yosys.

[Danny Milosavljevic]

gnu: Add yosys.

* gnu/packages/fpga.scm (yosys): New variable.
---
 gnu/packages/fpga.scm | 81 +++
 1 file changed, 81 insertions(+)

diff --git a/gnu/packages/fpga.scm b/gnu/packages/fpga.scm
index 1d1c981..763d746 100644
--- a/gnu/packages/fpga.scm
+++ b/gnu/packages/fpga.scm
@@ -108,3 +108,84 @@ For synthesis, the compiler generates netlists in the desired format.")
 ;; Otherwise would be GPL2+.
 ;; You have to accept both GPL2 and LGPL2.1+.
 (license (list license:gpl2 license:lgpl2.1+
+
+(define-public yosys
+  (package
+(name "yosys")
+(version "0.6")
+(source (origin
+  (method url-fetch)
+  (uri
+   (string-append "https://github.com/cliffordwolf/yosys/archive/";
+  name "-" version ".tar.gz"))
+  (sha256
+(base32
+   "02j0c0m9dfyjccynalf0aggj6gy20k7iphpkg5cn6sdirlkv8gmx"))
+  (file-name (string-append name "-" version "-checkout.tar.gz"))
+  (modules '((guix build utils)))
+  (snippet
+'(substitute* "Makefile"
+   (("ABCREV = .*") "ABCREV = default\n")
+(build-system gnu-build-system)
+(arguments
+ `(#:test-target "test"
+   #:make-flags (list "CC=gcc"
+  "CXX=g++"
+  (string-append "PREFIX=" %output))
+   #:phases
+   (modify-phases %standard-phases
+ (replace 'configure
+   (lambda* (#:key inputs (make-flags '()) #:allow-other-keys)
+ (zero? (apply system* "make" "config-gcc" make-flags
+ (add-after 'configure 'prepare-abc
+   (lambda* (#:key inputs #:allow-other-keys)
+ (let* ((sourceabc (assoc-ref inputs "abc"))
+(sourcebin (string-append sourceabc "/bin"))
+(source (string-append sourcebin "/abc")))
+   (mkdir-p "abc")
+   (call-with-output-file "abc/Makefile"
+ (lambda (port)
+   (format port ".PHONY: all\nall:\n\tcp -f abc abc-default\n")))
+   (copy-file source "abc/abc")
+   (zero? (system* "chmod" "+w" "abc/abc")
+  (add-before 'check 'fix-iverilog-references
+ (lambda* (#:key inputs native-inputs #:allow-other-keys)
+   (let* ((xinputs (or native-inputs inputs))
+  (xdirname (assoc-ref xinputs "iverilog"))
+  (iverilog (string-append xdirname "/bin/iverilog")))
+ (substitute* '("./manual/CHAPTER_StateOfTheArt/synth.sh"
+"./manual/CHAPTER_StateOfTheArt/validate_tb.sh"
+"./techlibs/ice40/tests/test_bram.sh"
+"./techlibs/ice40/tests/test_ffs.sh"
+"./techlibs/xilinx/tests/bram1.sh"
+"./techlibs/xilinx/tests/bram2.sh"
+"./tests/bram/run-single.sh"
+"./tests/realmath/run-test.sh"
+"./tests/simple/run-test.sh"
+"./tests/techmap/mem_simple_4x1_runtest.sh"
+"./tests/tools/autotest.sh"
+"./tests/vloghtb/common.sh")
+(("if ! which iverilog") "if ! true")
+(("iverilog ") (string-append iverilog " "))
+(("iverilog_bin=\".*\"") (string-append "iverilog_bin=\""
+iverilog
+"\"")
+;; TODO add xdot [patch the path to it here] as soon as I find out where it is.
+(native-inputs
+ `(("pkg-config" ,pkg-config)
+   ("python" ,python)
+   ("bison" ,bison)
+   ("flex" ,flex)
+   ("gawk" , gawk) ; for the tests and "make" progress pretty-printing
+   ("tcl" ,tcl) ; tclsh for the tests
+   ("iverilog" ,iverilog) ; for the tests
+))
+(inputs
+ `(("tcl" ,tcl)
+   ("readline" ,readline)
+   ("libffi" ,libffi)
+   ("abc" ,abc)))
+(home-page "http://www.clifford.at/yosys/";)
+(synopsis "FPGA Verilog RTL synthesizer")
+(description "Yosys synthesizes Verilog-2005.")
+(license license:isc)))

[PATCH v2 2/5] gnu: Add iverilog.

[Danny Milosavljevic]

gnu: Add iverilog.

* gnu/packages/fpga.scm (iverilog): New variable.
---
 gnu/packages/fpga.scm | 35 +++
 1 file changed, 35 insertions(+)

diff --git a/gnu/packages/fpga.scm b/gnu/packages/fpga.scm
index f757a28..1d1c981 100644
--- a/gnu/packages/fpga.scm
+++ b/gnu/packages/fpga.scm
@@ -73,3 +73,38 @@
 formal verification.")
 (license
   (license:non-copyleft "https://fedoraproject.org/wiki/Licensing:MIT#Modern_Variants";)
+
+(define-public iverilog
+  (package
+(name "iverilog")
+(version "10.1.1")
+(source (origin
+  (method url-fetch)
+  (uri
+   (string-append "ftp://ftp.icarus.com/pub/eda/verilog/v10/";
+  "verilog-" version ".tar.gz"))
+  (sha256
+(base32
+   "1nnassxvq30rnn0r2p85rkb2zwxk97p109y13x3vr365wzgpbapx"
+(build-system gnu-build-system)
+(native-inputs
+ `(("flex" ,flex)
+   ("bison" ,bison)
+   ("ghostscript" ,ghostscript))) ; ps2pdf
+(home-page "http://iverilog.icarus.com/";)
+(synopsis "FPGA Verilog simulation and synthesis tool")
+(description "Icarus Verilog is a Verilog simulation and synthesis tool.
+It operates as a compiler, compiling source code written in Verilog
+(IEEE-1364) into some target format.
+For batch simulation, the compiler can generate an intermediate form
+called vvp assembly.
+This intermediate form is executed by the ``vvp'' command.
+For synthesis, the compiler generates netlists in the desired format.")
+;; GPL2 only because of:
+;; - ./driver/iverilog.man.in
+;; - ./iverilog-vpi.man.in
+;; - ./tgt-fpga/iverilog-fpga.man
+;; - ./vvp/vvp.man.in
+;; Otherwise would be GPL2+.
+;; You have to accept both GPL2 and LGPL2.1+.
+(license (list license:gpl2 license:lgpl2.1+

[PATCH v2 0/5] Add FPGA Tools

[Danny Milosavljevic]

This patch adds FPGA tools.

Danny Milosavljevic (5):
  gnu: Add abc.
  gnu: Add iverilog.
  gnu: Add yosys.
  gnu: Add icestorm.
  gnu: Add arachne-pnr.

 gnu/local.mk  |   1 +
 gnu/packages/fpga.scm | 273 ++
 2 files changed, 274 insertions(+)
 create mode 100644 gnu/packages/fpga.scm

[PATCH v2 1/5] gnu: Add abc.

[Danny Milosavljevic]

gnu: Add abc.
* gnu/packages/fpga.scm: New file.
* gnu/packages/fpga.scm (abc): New variable.
---
 gnu/local.mk  |  1 +
 gnu/packages/fpga.scm | 75 +++
 2 files changed, 76 insertions(+)
 create mode 100644 gnu/packages/fpga.scm

diff --git a/gnu/local.mk b/gnu/local.mk
index 53621d8..84aca08 100644
--- a/gnu/local.mk
+++ b/gnu/local.mk
@@ -124,6 +124,7 @@ GNU_SYSTEM_MODULES =\
   %D%/packages/fltk.scm\
   %D%/packages/fonts.scm			\
   %D%/packages/fontutils.scm			\
+  %D%/packages/fpga.scm\
   %D%/packages/freedesktop.scm			\
   %D%/packages/freeipmi.scm			\
   %D%/packages/ftp.scm\
diff --git a/gnu/packages/fpga.scm b/gnu/packages/fpga.scm
new file mode 100644
index 000..f757a28
--- /dev/null
+++ b/gnu/packages/fpga.scm
@@ -0,0 +1,75 @@
+;;; GNU Guix --- Functional package management for GNU
+;;; Copyright © 2016 Danny Milosavljevic 
+;;;
+;;; This file is part of GNU Guix.
+;;;
+;;; GNU Guix is free software; you can redistribute it and/or modify it
+;;; under the terms of the GNU General Public License as published by
+;;; the Free Software Foundation; either version 3 of the License, or (at
+;;; your option) any later version.
+;;;
+;;; GNU Guix is distributed in the hope that it will be useful, but
+;;; WITHOUT ANY WARRANTY; without even the implied warranty of
+;;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+;;; GNU General Public License for more details.
+;;;
+;;; You should have received a copy of the GNU General Public License
+;;; along with GNU Guix.  If not, see .
+
+(define-module (gnu packages fpga)
+  #:use-module ((guix licenses) #:prefix license:)
+  #:use-module (guix packages)
+  #:use-module (guix download)
+  #:use-module (guix git-download)
+  #:use-module (guix build-system gnu)
+  #:use-module (guix build-system cmake)
+  #:use-module (gnu packages)
+  #:use-module (gnu packages pkg-config)
+  #:use-module (gnu packages tcl)
+  #:use-module (gnu packages readline)
+  #:use-module (gnu packages python)
+  #:use-module (gnu packages bison)
+  #:use-module (gnu packages flex)
+  #:use-module (gnu packages libffi)
+  #:use-module (gnu packages zip)
+  #:use-module (gnu packages perl)
+  #:use-module (gnu packages ghostscript)
+  #:use-module (gnu packages gawk)
+  #:use-module (gnu packages version-control)
+  #:use-module (gnu packages libftdi))
+
+(define-public abc
+ (let ((commit "5ae4b975c49c")
+   (revision "1"))
+  (package
+(name "abc")
+(version (string-append "0.0-" revision "-" (string-take commit 9)))
+(source (origin
+  (method url-fetch)
+  (uri
+   (string-append "https://bitbucket.org/alanmi/abc/get/"; commit ".zip"))
+  (file-name (string-append name "-" version "-checkout.zip"))
+  (sha256
+(base32
+   "1syygi1x40rdryih3galr4q8yg1w5bvdzl75hd27v1xq0l5bz3d0"
+(build-system gnu-build-system)
+(native-inputs
+ `(("unzip" ,unzip)))
+(inputs
+ `(("readline" ,readline)))
+(arguments
+ `(#:tests? #f ; no check target
+   #:phases
+   (modify-phases %standard-phases
+ (delete 'configure)
+ (replace 'install
+   (lambda* (#:key outputs #:allow-other-keys)
+ (let* ((out (assoc-ref outputs "out"))
+(out-bin (string-append out "/bin")))
+   (install-file "abc" out-bin)))
+(home-page "http://people.eecs.berkeley.edu/~alanmi/abc/";)
+(synopsis "Sequential logic synthesis and formal verification")
+(description "ABC is a program for sequential logic synthesis and
+formal verification.")
+(license
+  (license:non-copyleft "https://fedoraproject.org/wiki/Licensing:MIT#Modern_Variants";)

[PATCH v2 5/5] gnu: Add arachne-pnr.

[Danny Milosavljevic]

gnu: Add arachne-pnr.

* gnu/packages/fpga.scm (arachne-pnr): New variable.
---
 gnu/packages/fpga.scm | 44 
 1 file changed, 44 insertions(+)

diff --git a/gnu/packages/fpga.scm b/gnu/packages/fpga.scm
index 8c41cd3..2d0d617 100644
--- a/gnu/packages/fpga.scm
+++ b/gnu/packages/fpga.scm
@@ -227,3 +227,47 @@ For synthesis, the compiler generates netlists in the desired format.")
 (description "Project IceStorm - Lattice iCE40 FPGAs Bitstream Tools. 
 Includes the actual FTDI connector.")
 (license license:isc
+
+(define-public arachne-pnr
+  (let ((commit "52e69ed207342710080d85c7c639480e74a021d7")
+(revision "1"))
+   (package
+(name "arachne-pnr")
+(version (string-append "0.0-" revision "-" (string-take commit 9)))
+(source (origin
+  (method git-fetch)
+  (uri (git-reference
+ (url "https://github.com/cseed/arachne-pnr.git";)
+ (commit commit)))
+  (file-name (string-append name "-" version "-checkout"))
+  (sha256
+(base32
+   "15bdw5yxj76lxrwksp6liwmr6l1x77isf4bs50ys9rsnmiwh8c3w"
+(build-system gnu-build-system)
+(arguments
+ `(#:test-target "test"
+   #:phases (modify-phases %standard-phases
+   (replace 'configure
+ (lambda* (#:key outputs inputs #:allow-other-keys)
+   (substitute* '("Makefile")
+ (("DESTDIR = .*") (string-append "DESTDIR = "
+ (assoc-ref outputs "out")
+ "\n"))
+ (("ICEBOX = .*") (string-append "ICEBOX = "
+ (assoc-ref inputs "icestorm")
+ "/share/icebox\n")))
+   (substitute* '("./tests/fsm/generate.py"
+  "./tests/combinatorial/generate.py")
+ (("#!/usr/bin/python") "#!/usr/bin/python2"))
+   #t)
+(inputs
+ `(("icestorm" ,icestorm)))
+(native-inputs
+ `(("git" ,git)  ; for determining its own version string
+   ("yosys" ,yosys) ; for tests
+   ("perl" ,perl) ; for shasum
+   ("python-2" ,python-2))) ; for tests
+(home-page "https://github.com/cseed/arachne-pnr";)
+(synopsis "Place-and-Route tool for FPGAs")
+(description "Arachne-PNR is a Place-and-Route Tool For FPGAs.")
+(license license:gpl2

[PATCH v2 4/5] gnu: Add icestorm.

[Danny Milosavljevic]

gnu: Add icestorm.

* gnu/packages/fpga.scm (icestorm): New variable.
---
 gnu/packages/fpga.scm | 38 ++
 1 file changed, 38 insertions(+)

diff --git a/gnu/packages/fpga.scm b/gnu/packages/fpga.scm
index 763d746..8c41cd3 100644
--- a/gnu/packages/fpga.scm
+++ b/gnu/packages/fpga.scm
@@ -189,3 +189,41 @@ For synthesis, the compiler generates netlists in the desired format.")
 (synopsis "FPGA Verilog RTL synthesizer")
 (description "Yosys synthesizes Verilog-2005.")
 (license license:isc)))
+
+(define-public icestorm
+  (let ((commit "12b2295c9087d94b75e374bb205ae4d76cf17e2f")
+(revision "1"))
+   (package
+(name "icestorm")
+(version (string-append "0.0-" revision "-" (string-take commit 9)))
+(source (origin
+  (method git-fetch)
+  (uri (git-reference
+ (url "https://github.com/cliffordwolf/icestorm.git";)
+ (commit commit)))
+  (file-name (string-append name "-" version "-checkout"))
+  (sha256
+(base32
+   "1mmzlqvap6w8n4qzv3idvy51arkgn03692ssplwncy3akjrbsd2b"
+(build-system gnu-build-system)
+(arguments
+ `(#:tests? #f ; no unit tests that don't need an FPGA exist.
+   #:make-flags (list "CC=gcc" "CXX=g++"
+  (string-append "PREFIX=" (assoc-ref %outputs "out")))
+   #:phases
+(modify-phases %standard-phases
+  (add-after 'unpack 'remove-usr-local
+(lambda _
+  (substitute* "iceprog/Makefile"
+(("-L/usr/local/lib") ""
+  (delete 'configure
+(inputs
+  `(("libftdi" ,libftdi)))
+(native-inputs
+  `(("python-3" ,python)
+("pkg-config" ,pkg-config)))
+(home-page "http://www.clifford.at/icestorm/";)
+(synopsis "Project IceStorm - Lattice iCE40 FPGAs bitstream tools")
+(description "Project IceStorm - Lattice iCE40 FPGAs Bitstream Tools. 
+Includes the actual FTDI connector.")
+(license license:isc

Re: Need help fixing testcases with time-difference

[Hartmut Goebel]

Hi,

> I think you need to tell it where to find timezone data as well. Try
> adding a phase like this:
>
>   (add-before 'check 'set-tzdir
> (lambda* (#:key inputs #:allow-other-keys)
>   (setenv "TZDIR"
>   (string-append (assoc-ref inputs "tzdata")
>  "/share/zoneinfo"))
>   #t))

This was what I've been searching for and this did the trick! Thanks a
lot! You are great!

-- 
Regards
Hartmut Goebel

| Hartmut Goebel  | h.goe...@crazy-compilers.com   |
| www.crazy-compilers.com | compilers which you thought are impossible |

Re: [PATCH v2 2/6] gnu: fpga: Add abc.

[Danny Milosavljevic]

> > Thanks again!  Would you like me to take care of the changes or would
> > you prefer to send an updated patch?  
> 
> Yes please do.

Whoops. I just saw that I forgot to send yosys in the patch series. Therefore 
I'll send a new series with it and the fixes.

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

[ng0]

Catonano  writes:

> 2016-08-07 10:21 GMT+02:00 ng0 :
>
>> ng0  writes:
>>
>> > The next 2.5 days will be unpleasant for me and probably difficult to
>> > focus, but I know how to fix the 3 failing tests for us.
>> > I'll have this fixed in thex next days.
>> >
>> > Appended is the failing tests output (for me as a backup) and the
>> > combined gnunet-svn + gnunet-gtk-svn patch. As we discussed this will
>> > not be in guix at the moment and is intended as a test package to get
>> > the next release of gnunet to build on guix when it is released.
>> >
>> > There are last things I want to transition from my Gentoo package to the
>> > guix packages.
>> >
>> > I'd like to have a gnunet package with --with-experimental enabled
>> > (features: conversation etc, at least last time I read the Makefile), so
>> > this will be available out of tree when all the features are too much
>> > for master tree (or --with-experimental is considered too unstable to
>> > ship).
>> >
>> > The log attachment was too big, log can be replicated with tests
>> > enabled.
>> >
>> > Have a nice weekend.
>>
>> For whatever reason the appended patches broke, here is another try:
>>
>>
>>
>>
>>
>
> I'd like to put down a few notes gnunet-gtk-svn
>
> First: there are these 3 warnings in the terminal
>
> ago 17 15:02:04-825169 util-28635 WARNING `access' failed on file
> `/gnu/store/icrh1m4dcjpxm0j9d986sdhwnc5nj92b-gnunet-gtk-svn-
> 0.10.1-1.svn37273/gnunet/libexec/gnunet-service-resolver' at
> os_installation.c:867 with error: File o directory non esistente
>
> ago 17 15:02:04-825285 util-28635 WARNING `access' failed on file
> `/gnu/store/icrh1m4dcjpxm0j9d986sdhwnc5nj92b-gnunet-gtk-svn-
> 0.10.1-1.svn37273/gnunet/libexec/gnunet-service-namestore' at
> os_installation.c:867 with error: File o directory non esistente
>
> ago 17 15:02:04-825303 util-28635 WARNING `access' failed on file
> `/gnu/store/icrh1m4dcjpxm0j9d986sdhwnc5nj92b-gnunet-gtk-svn-
> 0.10.1-1.svn37273/gnunet/libexec/gnunet-service-identity' at
> os_installation.c:867 with error: File o directory non esistente
>
> an "ls" in that directory shows
>
> ls /gnu/store/icrh1m4dcjpxm0j9d986sdhwnc5nj92b-gnunet-gtk-svn-
> 0.10.1-1.svn37273/
> bin  include  lib  share
>
> There's no gnunet/libexec
>
> But wait there's more: a ls in the gnunet-svn package (that's a dependency
> of gnunet-gtk-svn) shows
>
> ls /gnu/store/15r69amj11df3v9ka2kzsw62ad866ii7-gnunet-svn-0.10.1-1.svn37273/
> bin  include  lib  share
>
> There's no libexec here, either
>
>
>
>
>
> Next issue: in the setup window, there's a tab containing some ceckboxes
> for the transport plugins that I should check to activate a tranport plugin.
>
> But when I open the tab these errors show up in the terminal:
>
> ago 17 15:02:08-145220 gnunet-setup-28635 ERROR Configuration fails to
> specify option `PLUGINS' in section `transport'!
>
> ago 17 15:02:08-147950 gnunet-setup-28635 ERROR Configuration fails to
> specify option `PLUGINS' in section `transport'!
>
> ago 17 15:02:08-148828 gnunet-setup-28635 ERROR Configuration fails to
> specify option `PLUGINS' in section `transport'!
>
> ago 17 15:02:08-149459 gnunet-setup-28635 ERROR Configuration fails to
> specify option `PLUGINS' in section `transport'!
>
> ago 17 15:02:08-150018 gnunet-setup-28635 ERROR Configuration fails to
> specify option `PLUGINS' in section `transport'!
>
>
> and the checkboxes don't react to my clicks  so I can't activate any
> transport layer.

Sorry to have such a short answer, but please refer to upstream for bug
solving with this version (https://gnunet.org/bugs/). Depending on which
svn revision you use, the bugs may already be gone. That's 37273, HEAD
is currently at 37733.
-- 
ng0
For non-prism friendly talk find me on http://www.psyced.org

Re: [PATCH v2 2/6] gnu: fpga: Add abc.

[Danny Milosavljevic]

> What does this mean for us?  Should we offer two packages?  Or is this
> only needed for developer users of this package?  I’m inclined to just
> drop this comment.  What do you think?

I only install the binary and no library so it doesn't really matter. We can 
drop the comment - if we should need the library later we can find the comment 
in the mailing list archives.

> We prefer to have the full commit here and abbreviate it in the version
> string.  We usually also add a “revision” or “guix-revision” variable
> (starting at 0 or 1).  That’s easier to update than having to modify the
> version string directly each time the commit changes.

Yeah, back then I didn't take care of it. Sorry. Please add.

> “'check” (with the leading quote indicating that it is a Scheme symbol)
> is a phase name in Guix.  It would be less confusing if the comment just
> said
> 
>“no check target”

Yeah.

> “install-file” does not need to be given a target file name, just a
> target directory.

Ok!

> > +(home-page "http://people.eecs.berkeley.edu/~alanmi/abc/";)
> > +(synopsis "Sequential Logic Synthesis and Formal Verification")  
> 
> Let’s put this in lower case (except for the first word).
> 
> Thanks again!  Would you like me to take care of the changes or would
> you prefer to send an updated patch?

Yes please do.

Note that Leo also proposed to write

(license (license:non-copyleft 
"https://fedoraproject.org/wiki/Licensing:MIT#Modern_Variants";)

as license.

Re: Need help fixing testcases with time-difference

[Marius Bakke]

Hartmut Goebel  writes:

> Hi,
>
> I'm currently working on django, the web application framework.
> Unfortunalty some tests fail. These are all testing time- and timezone
> calculations.
>
> Failures are like this:
>
> AssertionError: datetime.timedelta(0, 3600, 16) not less than
> datetime.timedelta(0, 2)
>
> which means the returned time difference is ca. 1 hour, but allowed are
> only 2 minutes. The testcase os this one
> https://github.com/django/django/blob/master/tests/file_storage/tests.py#L239>
>
> I already added tzdata to native-inputs, but this does not solve the issue.

I think you need to tell it where to find timezone data as well. Try
adding a phase like this:

  (add-before 'check 'set-tzdir
(lambda* (#:key inputs #:allow-other-keys)
  (setenv "TZDIR"
  (string-append (assoc-ref inputs "tzdata")
 "/share/zoneinfo"))
  #t))

Cheers,
Marius

Re: Need help fixing testcases with time-difference

[Vincent Legoll]

Probably a long shot, but... Maybe you can try the following:

change line 158 of tests/file_storage/tests.py:

now_in_algiers = timezone.make_aware(datetime.now())

into

now_in_algiers = timezone.make_aware(datetime.now(), is_dst=False)

and see if this changes the outcome...

-- 
Vincent Legoll

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

[Catonano]

2016-08-07 10:21 GMT+02:00 ng0 :

> ng0  writes:
>
> > The next 2.5 days will be unpleasant for me and probably difficult to
> > focus, but I know how to fix the 3 failing tests for us.
> > I'll have this fixed in thex next days.
> >
> > Appended is the failing tests output (for me as a backup) and the
> > combined gnunet-svn + gnunet-gtk-svn patch. As we discussed this will
> > not be in guix at the moment and is intended as a test package to get
> > the next release of gnunet to build on guix when it is released.
> >
> > There are last things I want to transition from my Gentoo package to the
> > guix packages.
> >
> > I'd like to have a gnunet package with --with-experimental enabled
> > (features: conversation etc, at least last time I read the Makefile), so
> > this will be available out of tree when all the features are too much
> > for master tree (or --with-experimental is considered too unstable to
> > ship).
> >
> > The log attachment was too big, log can be replicated with tests
> > enabled.
> >
> > Have a nice weekend.
>
> For whatever reason the appended patches broke, here is another try:
>
>
>
>
>

I'd like to put down a few notes gnunet-gtk-svn

First: there are these 3 warnings in the terminal

ago 17 15:02:04-825169 util-28635 WARNING `access' failed on file
`/gnu/store/icrh1m4dcjpxm0j9d986sdhwnc5nj92b-gnunet-gtk-svn-
0.10.1-1.svn37273/gnunet/libexec/gnunet-service-resolver' at
os_installation.c:867 with error: File o directory non esistente

ago 17 15:02:04-825285 util-28635 WARNING `access' failed on file
`/gnu/store/icrh1m4dcjpxm0j9d986sdhwnc5nj92b-gnunet-gtk-svn-
0.10.1-1.svn37273/gnunet/libexec/gnunet-service-namestore' at
os_installation.c:867 with error: File o directory non esistente

ago 17 15:02:04-825303 util-28635 WARNING `access' failed on file
`/gnu/store/icrh1m4dcjpxm0j9d986sdhwnc5nj92b-gnunet-gtk-svn-
0.10.1-1.svn37273/gnunet/libexec/gnunet-service-identity' at
os_installation.c:867 with error: File o directory non esistente

an "ls" in that directory shows

ls /gnu/store/icrh1m4dcjpxm0j9d986sdhwnc5nj92b-gnunet-gtk-svn-
0.10.1-1.svn37273/
bin  include  lib  share

There's no gnunet/libexec

But wait there's more: a ls in the gnunet-svn package (that's a dependency
of gnunet-gtk-svn) shows

ls /gnu/store/15r69amj11df3v9ka2kzsw62ad866ii7-gnunet-svn-0.10.1-1.svn37273/
bin  include  lib  share

There's no libexec here, either





Next issue: in the setup window, there's a tab containing some ceckboxes
for the transport plugins that I should check to activate a tranport plugin.

But when I open the tab these errors show up in the terminal:

ago 17 15:02:08-145220 gnunet-setup-28635 ERROR Configuration fails to
specify option `PLUGINS' in section `transport'!

ago 17 15:02:08-147950 gnunet-setup-28635 ERROR Configuration fails to
specify option `PLUGINS' in section `transport'!

ago 17 15:02:08-148828 gnunet-setup-28635 ERROR Configuration fails to
specify option `PLUGINS' in section `transport'!

ago 17 15:02:08-149459 gnunet-setup-28635 ERROR Configuration fails to
specify option `PLUGINS' in section `transport'!

ago 17 15:02:08-150018 gnunet-setup-28635 ERROR Configuration fails to
specify option `PLUGINS' in section `transport'!


and the checkboxes don't react to my clicks  so I can't activate any
transport layer.

Re: Need help fixing testcases with time-difference

[Vincent Legoll]

>> I've not looked too hard, but that test seems to use the storage to check 
>> access
>> time on a file, could it be that your FS use the "noatime" option or
>> does not produce correctly updated file access timestamps ?
>
> Good spot! But my filesystems are mounted with "relatime", so this
> should not matter. https://blog.confirm.ch/mount-options-atime-vs-relatime/>

Yep the relatime bugs should have been ironed out a few years ago.

D'oh ! That would have been an easy way out...

Are you in a DST-enabled timezone or not ?

-- 
Vincent Legoll

Re: [PATCH 07/24] gnu: compression: Add snappy.

[David Craven]

> I wonder, are there any drawbacks of making the linter check that the
> URL provides the data named by the hash?

That may be useful. I think if guix refresh would accept a version
flag, and possibly a force flag, that would also be useful.

The version flag would be used as follows:

guix refresh qtbase -u --version=5.7.0

If the version is already set to 5.7.0 a -f flag would download and
update the hash anyway. This is useful when the download url has
changed, but the version hasn't as in the snappy example. I think
these two features could make guix refresh more generally applicable,
to packages that don't have an automatic updater.

Re: Need help fixing testcases with time-difference

[Hartmut Goebel]

Am 18.08.2016 um 13:10 schrieb Vincent Legoll:
> I've not looked too hard, but that test seems to use the storage to check 
> access
> time on a file, could it be that your FS use the "noatime" option or
> does not produce
> correctly updated file access timestamps ?

Good spot! But my filesystems are mounted with "relatime", so this
should not matter. https://blog.confirm.ch/mount-options-atime-vs-relatime/>

-- 
Regards
Hartmut Goebel

| Hartmut Goebel  | h.goe...@crazy-compilers.com   |
| www.crazy-compilers.com | compilers which you thought are impossible |

[PATCH 2/2] gnu: rhythmbox: Update to 3.4.

[Tobias Geerinckx-Rice]

* gnu/packages/gnome.scm (rhythmbox): Update to 3.4.
[arguments]: Add PYTHONPATH environment variable to 'wrap-program'.
---

WebKit ambitions seem to have been shelved by upstream due to OpenBSD
W^X concerns. Also remove them from our TODO list.

 gnu/packages/gnome.scm | 11 ++-
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/gnu/packages/gnome.scm b/gnu/packages/gnome.scm
index 66b461c..bac7f11 100644
--- a/gnu/packages/gnome.scm
+++ b/gnu/packages/gnome.scm
@@ -3263,7 +3263,7 @@ which can read a large number of file formats.")
 (define-public rhythmbox
  (package
(name "rhythmbox")
-   (version "3.2.1")
+   (version "3.4")
(source (origin
 (method url-fetch)
 (uri (string-append "mirror://gnome/sources/" name "/"
@@ -3271,7 +3271,7 @@ which can read a large number of file formats.")
 name "-" version ".tar.xz"))
 (sha256
  (base32
-  "0f3radhlji7rxl760yl2vm49fvfslympxrpm8497acbmbd7wlhxz"
+  "1347747m90aiz47wny1f8rdk5195qf2ph0554c6y91711sm951gg"
(build-system glib-or-gtk-build-system)
(arguments
 `(#:configure-flags
@@ -3289,11 +3289,13 @@ which can read a large number of file formats.")
(let ((out   (assoc-ref outputs "out"))
  (gi-typelib-path   (getenv "GI_TYPELIB_PATH"))
  (gst-plugin-path   (getenv "GST_PLUGIN_SYSTEM_PATH"))
- (grl-plugin-path   (getenv "GRL_PLUGIN_PATH")))
+ (grl-plugin-path   (getenv "GRL_PLUGIN_PATH"))
+ (python-path   (getenv "PYTHONPATH")))
  (wrap-program (string-append out "/bin/rhythmbox")
`("GI_TYPELIB_PATH"":" prefix (,gi-typelib-path))
`("GST_PLUGIN_SYSTEM_PATH" ":" prefix (,gst-plugin-path))
-   `("GRL_PLUGIN_PATH"":" prefix (,grl-plugin-path
+   `("GRL_PLUGIN_PATH"":" prefix (,grl-plugin-path))
+   `("PYTHONPATH" ":" prefix (,python-path
#t)
(propagated-inputs
 `(("dconf" ,dconf)))
@@ -3339,7 +3341,6 @@ which can read a large number of file formats.")
   ;; TODO:
   ;;  * libgpod
   ;;  * mx
-  ;;  * webkit
   ("brasero" ,brasero)))
(home-page "https://wiki.gnome.org/Apps/Rhythmbox";)
(synopsis "Music player for GNOME")
-- 
2.7.4

[PATCH 1/2] gnu: shotwell: Update to 0.23.5.

[Tobias Geerinckx-Rice]

* gnu/packages/gnome.scm (shotwell): Update to 0.23.5.
[propagated-inputs]: Add dconf.
[native-inputs]: Remove m4, desktop-file-utils, which, gnome-doc-utils,
python2 and python2-libxml2. Add itstool.
[inputs]: Remove gst-plugins-good, rest and gtk+. Add glib:bin.
---

The *inputs changed significantly due to a reported change in
build system, so I just started more or less with nothing and kept
what's needed to build. Inputs now don't include things like gtk+,
though, which might be a mistake.

Other removed inputs like rest and gst-plugins-good were obsolete or
unreferenced, respectively.

 gnu/packages/gnome.scm | 30 +-
 1 file changed, 9 insertions(+), 21 deletions(-)

diff --git a/gnu/packages/gnome.scm b/gnu/packages/gnome.scm
index e133fcc..66b461c 100644
--- a/gnu/packages/gnome.scm
+++ b/gnu/packages/gnome.scm
@@ -21,6 +21,7 @@
 ;;; Copyright © 2016 Leo Famulari 
 ;;; Copyright © 2016 Alex Griffin 
 ;;; Copyright © 2016 ng0 
+;;; Copyright © 2016 Tobias Geerinckx-Rice 
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -85,7 +86,6 @@
   #:use-module (gnu packages libusb)
   #:use-module (gnu packages lirc)
   #:use-module (gnu packages lua)
-  #:use-module (gnu packages m4)
   #:use-module (gnu packages image)
   #:use-module (gnu packages networking)
   #:use-module (gnu packages password-utils)
@@ -3841,7 +3841,7 @@ metadata in photo and video files of various formats.")
 (define-public shotwell
   (package
 (name "shotwell")
-(version "0.23.1")
+(version "0.23.5")
 (source (origin
   (method url-fetch)
   (uri (string-append "mirror://gnome/sources/" name "/"
@@ -3849,39 +3849,27 @@ metadata in photo and video files of various formats.")
   name "-" version ".tar.xz"))
   (sha256
(base32
-"12imip32mav0zqg1fh4xm6zk4qsgg2435xsyb6ljz47i37zk6kg2"
+"0fgs1rgvkmy79bmpxrsvm5w8rvqml4l1vnwma0xqx5zzm02p8a07"
 (build-system glib-or-gtk-build-system)
-(arguments
- `(#:tests? #f ;no "check" target
-   #:make-flags '("CC=gcc")
-   #:configure-flags '("--disable-gsettings-convert-install")
-   #:out-of-source? #f))
+(propagated-inputs
+ `(("dconf" ,dconf)))
 (native-inputs
  `(("pkg-config" ,pkg-config)
("gettext" ,gnu-gettext)
-   ("m4" ,m4)
-   ("desktop-file-utils" ,desktop-file-utils)
-   ("vala" ,vala)
-   ("which" ,which)
-   ("gnome-doc-utils" ,gnome-doc-utils)
-   ;; FIXME: I only added python2-libxml2 because xml2po needs it at
-   ;; runtime.  It should be propagated.
-   ("python2-libxml2" ,python2-libxml2)
-   ("python2" ,python-2)))
+   ("itstool" ,itstool)
+   ("vala" ,vala)))
 (inputs
- `(("gstreamer" ,gstreamer)
+ `(("glib:bin" ,glib "bin")
+   ("gstreamer" ,gstreamer)
("gst-plugins-base" ,gst-plugins-base)
-   ("gst-plugins-good" ,gst-plugins-good)
("libgee" ,libgee)
("gexiv2" ,gexiv2)
("libraw" ,libraw)
("json-glib" ,json-glib)
-   ("rest" ,rest)
("webkitgtk" ,webkitgtk)
("sqlite" ,sqlite)
("libsoup" ,libsoup)
("libxml2" ,libxml2)
-   ("gtk+" ,gtk+)
("libgudev" ,libgudev)
("libgphoto2" ,libgphoto2)))
 (home-page "https://wiki.gnome.org/Apps/Shotwell";)
-- 
2.7.4

Re: Feedback, ideas, discussion: tracking patches, discussions, bugs.

[Hartmut Goebel]

Am 16.08.2016 um 16:06 schrieb Alex Vong:
> Hartmut Goebel  writes:
>
>> The gogs.io website is using a lot of external content, namely from
>> google and jquery. The FAQ is hosted a another external side, discus.
>> Gogs is not even self-hosting gogs but hosted at github. gogs.io does
>> not even state which license it has ("Open Source It all at guithub"),
>>
> I am a bit confused here, are we going to (1) host the code on our own
> server, or (2) using code-hosting service provided by other
> organization?
>
> [...]
> What I suggested is doing (1) with gogs. Do I understand the situation
> correctly? WDYT?

Sorry for the confusion. Of course I also suggest doing (1). My
objection is just about gogs website: They are using content from other
sites. If we host it ourself, we'd need to take care about this - and
need to re-check this on every update. Otherwise we would "report
visitors to other organizations", which would prevent to get "B" grade
in then GNU Ethical Repository Criteria Evaluations.

-- 
Schönen Gruß
Hartmut Goebel
Dipl.-Informatiker (univ), CISSP, CSSLP, ISO 27001 Lead Implementer
Information Security Management, Security Governance, Secure Software
Development

Goebel Consult, Landshut
http://www.goebel-consult.de

Blog:
http://www.goebel-consult.de/blog/kleiner-erfahrungsbericht-mit-online-ocr-diensten

Kolumne:
http://www.cissp-gefluester.de/2012-01-in-die-cloud-in-die-cloud-aber-wo-soll-die-sein




smime.p7s
Description: S/MIME Cryptographic Signature

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

[Ricardo Wurmus]

* gnu/packages/bioinformatics.scm (r-bamsignals): New variable.
---
 gnu/packages/bioinformatics.scm | 30 ++
 1 file changed, 30 insertions(+)

diff --git a/gnu/packages/bioinformatics.scm b/gnu/packages/bioinformatics.scm
index ade6e37..2bd71c8 100644
--- a/gnu/packages/bioinformatics.scm
+++ b/gnu/packages/bioinformatics.scm
@@ -6350,6 +6350,36 @@ nucleotide sequence analysis.  The package is primarily 
useful to developers
 of other R packages who wish to make use of HTSlib.")
 (license license:lgpl2.0+)))
 
+(define-public r-bamsignals
+  (package
+(name "r-bamsignals")
+(version "1.4.3")
+(source
+ (origin
+   (method url-fetch)
+   (uri (bioconductor-uri "bamsignals" version))
+   (sha256
+(base32
+ "1xqiqvg52p6fcvhr4146djbz79r3j1kmh75mq7rndwglmiybpwmy"
+(build-system r-build-system)
+(propagated-inputs
+ `(("r-biocgenerics" ,r-biocgenerics)
+   ("r-genomicranges" ,r-genomicranges)
+   ("r-iranges" ,r-iranges)
+   ("r-rcpp" ,r-rcpp)
+   ("r-rhtslib" ,r-rhtslib)
+   ("r-zlibbioc" ,r-zlibbioc)))
+(inputs
+ `(("zlib" ,zlib)))
+(home-page "http://bioconductor.org/packages/bamsignals";)
+(synopsis "Extract read count signals from bam files")
+(description
+ "This package allows to efficiently obtain count vectors from indexed bam
+files.  It counts the number of nucleotide sequence reads in given genomic
+ranges and it computes reads profiles and coverage profiles.  It also handles
+paired-end data.")
+(license license:gpl2+)))
+
 (define-public emboss
   (package
 (name "emboss")
-- 
2.9.0

Re: [PATCH] Patchseries: perl-www-opensearch and dependencies.

[ng0]

Hi,

Eric Bavier  writes:

> On Sun, 14 Aug 2016 17:19:10 +
> ng0  wrote:
>
>> This patch series adds perl-www-opensearch and the 13 patches leading to
>> it (the dependencies).
>
> Thanks for all the patches!  Comments dispersed throughtout:
>
>> 
[...]
>
> Otherwise LGTM!
>
> `~Eric

Thanks for reviewing. All the problems are addressed, and some
additional changes to move comments when no test can be run are moved to
save lines.

>From 5f3065a36e27ddafdfd0c56b3b29838e0bf15228 Mon Sep 17 00:00:00 2001
From: ng0 
Date: Sun, 14 Aug 2016 12:42:12 +
Subject: [PATCH 01/14] gnu: Add perl-uri-template.

* gnu/packages/web.scm (perl-uri-template): New variable.
---
 gnu/packages/web.scm | 24 
 1 file changed, 24 insertions(+)

diff --git a/gnu/packages/web.scm b/gnu/packages/web.scm
index 29055c3..986bfcc 100644
--- a/gnu/packages/web.scm
+++ b/gnu/packages/web.scm
@@ -2964,6 +2964,30 @@ URI::Find::Schemeless.  For a command-line interface, urifind is provided.")
 methods for WebSocket URIs as it does for HTTP URIs.")
 (license (package-license perl
 
+(define-public perl-uri-template
+  (package
+(name "perl-uri-template")
+(version "0.22")
+(source (origin
+  (method url-fetch)
+  (uri (string-append "mirror://cpan/authors/id/B/BR/BRICAS/URI-Template-"
+  version ".tar.gz"))
+  (sha256
+   (base32
+"08kjjb4c0v9gqfrfnj1wkivylxl05finn11ra64dj136fhmnyrbg"
+(build-system perl-build-system)
+(inputs
+ `(("perl-uri" ,perl-uri)))
+(native-inputs
+ `(("perl-test-pod-coverage" ,perl-test-pod-coverage)
+   ("perl-test-pod" ,perl-test-pod)
+   ("perl-json" ,perl-json)))
+(home-page "http://search.cpan.org/dist/URI-Template";)
+(synopsis "Object for handling URI templates")
+(description "This perl module provides a wrapper around URI templates as described in
+RFC 6570.")
+(license (package-license perl
+
 (define-public perl-www-curl
   (package
 (name "perl-www-curl")
-- 
2.9.3

>From 97dd97c46ec0d77cab6df66595b60cacab5e4234 Mon Sep 17 00:00:00 2001
From: ng0 
Date: Sun, 14 Aug 2016 13:02:00 +
Subject: [PATCH 02/14] gnu: Add perl-class-errorhandler.

* gnu/packages/perl.scm (perl-class-errorhandler): New variable.
---
 gnu/packages/perl.scm | 21 +
 1 file changed, 21 insertions(+)

diff --git a/gnu/packages/perl.scm b/gnu/packages/perl.scm
index aef92f4..4e3d1fa 100644
--- a/gnu/packages/perl.scm
+++ b/gnu/packages/perl.scm
@@ -700,6 +700,27 @@ subclasses and can be overridden.")
 type for perl.")
 (license (package-license perl
 
+(define-public perl-class-errorhandler
+  (package
+(name "perl-class-errorhandler")
+(version "0.04")
+(source (origin
+  (method url-fetch)
+  (uri (string-append "mirror://cpan/authors/id/T/TO/TOKUHIROM/"
+  "Class-ErrorHandler-" version ".tar.gz"))
+  (sha256
+   (base32
+"00j5f0z4riyq7i95jww291dpmbn0hmmvkcbrh7p0p8lpqz7jsb9l"
+(build-system perl-build-system)
+(home-page "http://search.cpan.org/dist/Class-ErrorHandler";)
+(synopsis "Base class for error handling")
+(description
+ "@code{Class::ErrorHandler} provides an error-handling mechanism that is generic
+enough to be used as the base class for a variety of OO classes.  Subclasses inherit
+its two error-handling methods, error and errstr, to communicate error messages back
+to the calling program.")
+(license (package-license perl
+
 (define-public perl-class-factory-util
   (package
 (name "perl-class-factory-util")
-- 
2.9.3

>From c32d61f5ee7be18c9c25bc02f17d1b7d404e4928 Mon Sep 17 00:00:00 2001
From: ng0 
Date: Sun, 14 Aug 2016 13:13:18 +
Subject: [PATCH 03/14] gnu: Add perl-datetime-format-mail.

* gnu/packages/perl.scm (perl-datetime-format-mail): New variable.
---
 gnu/packages/perl.scm | 21 +
 1 file changed, 21 insertions(+)

diff --git a/gnu/packages/perl.scm b/gnu/packages/perl.scm
index 4e3d1fa..db941fb 100644
--- a/gnu/packages/perl.scm
+++ b/gnu/packages/perl.scm
@@ -1792,6 +1792,27 @@ to create DateTime objects, with some custom tweaks to smooth out the rough
 edges (mainly concerning timezone detection and selection).")
 (license (package-license perl
 
+(define-public perl-datetime-format-mail
+  (package
+(name "perl-datetime-format-mail")
+(version "0.403")
+(source (origin
+  (method url-fetch)
+  (uri (string-append "mirror://cpan/authors/id/B/BO/BOOK/"
+  "DateTime-Format-Mail-" version ".tar.gz"))
+  (sha256
+   (base32
+"1c7wapbi9g9p2za52l3skhh31vg4da5kx2yfqzsqyf3p8iff7y4d"
+(build-system perl-build-system)
+(inputs
+ `(("perl-datetime" ,perl-datetime)
+   ("perl-params-validate" 

Re: Need help fixing testcases with time-difference

[Vincent Legoll]

>> That looks like a Daylight Saving Time mismatch, could that be possible ?
>
> That's been my first thought, too. But the test is using timezone

I've not looked too hard, but that test seems to use the storage to check access
time on a file, could it be that your FS use the "noatime" option or
does not produce
correctly updated file access timestamps ?

-- 
Vincent Legoll

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

[Ricardo Wurmus]

* gnu/packages/bioinformatics.scm (r-rhtslib): New variable.
---
 gnu/packages/bioinformatics.scm | 25 +
 1 file changed, 25 insertions(+)

diff --git a/gnu/packages/bioinformatics.scm b/gnu/packages/bioinformatics.scm
index 233b7a8..ade6e37 100644
--- a/gnu/packages/bioinformatics.scm
+++ b/gnu/packages/bioinformatics.scm
@@ -6325,6 +6325,31 @@ two-dimensional genome scans.")
 libraries for systems that do not have these available via other means.")
 (license license:artistic2.0)))
 
+(define-public r-rhtslib
+  (package
+(name "r-rhtslib")
+(version "1.4.3")
+(source
+ (origin
+   (method url-fetch)
+   (uri (bioconductor-uri "Rhtslib" version))
+   (sha256
+(base32
+ "1wgpn9x8abjj7fc087pdavqc3fz0pl5xdh231mgjila18irwlhb3"
+(properties `((upstream-name . "Rhtslib")))
+(build-system r-build-system)
+(propagated-inputs
+ `(("r-zlibbioc" ,r-zlibbioc)))
+(inputs
+ `(("zlib" ,zlib)))
+(home-page "https://github.com/nhayden/Rhtslib";)
+(synopsis "High-throughput sequencing library as an R package")
+(description
+ "This package provides the HTSlib C library for high-throughput
+nucleotide sequence analysis.  The package is primarily useful to developers
+of other R packages who wish to make use of HTSlib.")
+(license license:lgpl2.0+)))
+
 (define-public emboss
   (package
 (name "emboss")
-- 
2.9.0

Re: [PATCH v2 2/6] gnu: fpga: Add abc.

[Ricardo Wurmus]

Hi Danny,

thanks for the patch!

Although there are a couple of minor issues I think we can take it as is
and make a few changes before pushing to master.   Or you could send a
new version of the patch if you prefer that.

> * gnu/packages/fpga.scm (abc): New variable.
> ---
>  gnu/packages/fpga.scm | 43 +++
>  1 file changed, 43 insertions(+)


> diff --git a/gnu/packages/fpga.scm b/gnu/packages/fpga.scm
> index 112d53b..7571f87 100644
> --- a/gnu/packages/fpga.scm
> +++ b/gnu/packages/fpga.scm
> @@ -38,3 +38,46 @@
>#:use-module (gnu packages version-control)
>#:use-module (gnu packages libftdi))
 
> +;; To compile as C code (default):
> +;;   make sure that CC=gcc and ABC_NAMESPACE is not defined.
> +;; To compile as C++ code with namespaces:
> +;;   make sure that CC=g++ and ABC_NAMESPACE is set to the namespace.
> +;;   For example, add -DABC_NAMESPACE=xxx to OPTFLAGS.

What does this mean for us?  Should we offer two packages?  Or is this
only needed for developer users of this package?  I’m inclined to just
drop this comment.  What do you think?

> +(define-public abc
> + (let ((commit "5ae4b975c49c"))

Others have already mentioned the cosmetic indentation issues, so I
won’t do it here :) Before applying the patch we just need to make sure
to run it through Emacs once more to be sure the indentation is
consistent.

We prefer to have the full commit here and abbreviate it in the version
string.  We usually also add a “revision” or “guix-revision” variable
(starting at 0 or 1).  That’s easier to update than having to modify the
version string directly each time the commit changes.

> +  (package
> +(name "abc")
> +(version (string-append "0.0-" (string-take commit 7)))

Here we normally take 9 characters of the hash.  The guix-internal
revision should be prefixed here.

> +(source (origin
> +  (method url-fetch)
> +  (uri
> +   (string-append "https://bitbucket.org/alanmi/abc/get/";
> +  commit ".zip"))
> +  (file-name (string-append name "-" version
> "-checkout.zip"))

I don’t think we need the “-checkout” part.

> +  (sha256
> +(base32
> +   "1syygi1x40rdryih3galr4q8yg1w5bvdzl75hd27v1xq0l5bz3d0"
> +(build-system gnu-build-system)
> +(native-inputs
> + `(("unzip" ,unzip)))
> +(inputs
> + `(("readline" ,readline)))
> +(arguments
> + `(#:tests? #f ; 'check target does not exist.

“'check” (with the leading quote indicating that it is a Scheme symbol)
is a phase name in Guix.  It would be less confusing if the comment just
said

   “no check target”

because this is about a Makefile target, not about a Guix build phase.

> +   #:phases
> +(modify-phases %standard-phases
> +  (delete 'configure)
> +  (replace 'install
> +(lambda* (#:key outputs #:allow-other-keys)
> +  (let* ((out (assoc-ref outputs "out"))
> + (outbin (string-append out "/bin"))
> + (target (string-append outbin "/abc")))
> +(mkdir-p outbin)
> +(copy-file "abc" target)))

Please end the phase with #t as “copy-file” doesn’t have a specified
return value.  Instead of “copy-file” you could use this:

(mkdir-p outbin)
(install-file "abc" outbin)

“install-file” does not need to be given a target file name, just a
target directory.

> +(home-page "http://people.eecs.berkeley.edu/~alanmi/abc/";)
> +(synopsis "Sequential Logic Synthesis and Formal Verification")

Let’s put this in lower case (except for the first word).

Thanks again!  Would you like me to take care of the changes or would
you prefer to send an updated patch?

~~ Ricardo

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

[Ricardo Wurmus]

Ben Woodcroft  writes:

> * guix/build/ruby-build-system.scm (log-file-deletion): New procedure.
> (install): Remove files containing non-reproducible elements.  Print when each
> file is deleted.

Thank you, this looks good to me.  Good job on making even more Ruby
gems build reproducibly!

~~ Ricardo

Re: [PATCH v2 3/6] gnu: fpga: Add iverilog.

[Danny Milosavljevic]

On Tue, 16 Aug 2016 14:30:18 -0400
"Thompson, David"  wrote:

> Is it really GPL2 only?

Most of the files are GPL2+, but these are GPL2 only:

- ./driver/iverilog.man.in
- ./iverilog-vpi.man.in
- ./tgt-fpga/iverilog-fpga.man
- ./vvp/vvp.man.in

In addition these files are LGPL2+:

./iverilog-vpi.sh
./tgt-vlog95/logic_lpm.c
./vpi/mt19937int.c
all the Makefile.in

And these are LGPL2.1+:

./tgt-vlog95/logic_lpm.c

Re: Need help fixing testcases with time-difference

[Hartmut Goebel]

Am 18.08.2016 um 10:59 schrieb Vincent Legoll:
> That looks like a Daylight Saving Time mismatch, could that be possible ?

That's been my first thought, too. But the test is using timezone
"Africa/Algiers" (see
)
which AFAIK does not have DST.

Maybe the timezone data is not available when running the test?

-- 
Regards
Hartmut Goebel

| Hartmut Goebel  | h.goe...@crazy-compilers.com   |
| www.crazy-compilers.com | compilers which you thought are impossible |

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

[Jan Nieuwenhuizen]

Alex Kost writes:

>> + (add-before
>> + 'configure 'setenv
>
> it's more good-looking if this line is moved to the previous one :-)
>
> (add-before 'configure 'setenv

Done.

Greetings,
Jan

>From fe679f99f0cd75051015c4513a77780d0ec36109 Mon Sep 17 00:00:00 2001
From: Jan Nieuwenhuizen 
Date: Thu, 14 Apr 2016 07:35:40 +0200
Subject: [PATCH 02/12] gnu: Add mingw-w64.

* 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: New files.
* gnu/local.mk (dist_patch_DATA): Add them.
---
 gnu/local.mk   |   3 +
 gnu/packages/mingw.scm |  81 
 .../patches/gcc-4.9.3-mingw-gthr-default.patch |  11 ++
 .../patches/mingw-w64-5.0rc2-gcc-4.9.3.patch   | 218 +
 4 files changed, 313 insertions(+)
 create mode 100644 gnu/packages/mingw.scm
 create mode 100644 gnu/packages/patches/gcc-4.9.3-mingw-gthr-default.patch
 create mode 100644 gnu/packages/patches/mingw-w64-5.0rc2-gcc-4.9.3.patch

diff --git a/gnu/local.mk b/gnu/local.mk
index 53621d8..e5aed2b 100644
--- a/gnu/local.mk
+++ b/gnu/local.mk
@@ -235,6 +235,7 @@ GNU_SYSTEM_MODULES =\
   %D%/packages/mc.scm\
   %D%/packages/mcrypt.scm			\
   %D%/packages/messaging.scm			\
+  %D%/packages/mingw.scm			\
   %D%/packages/mg.scm\
   %D%/packages/mit-krb5.scm			\
   %D%/packages/moe.scm\
@@ -516,6 +517,7 @@ dist_patch_DATA =		\
   %D%/packages/patches/gcc-arm-link-spec-fix.patch		\
   %D%/packages/patches/gcc-cross-environment-variables.patch	\
   %D%/packages/patches/gcc-libvtv-runpath.patch			\
+  %D%/packages/patches/gcc-4.9.3-mingw-gthr-default.patch	\
   %D%/packages/patches/gcc-5.0-libvtv-runpath.patch		\
   %D%/packages/patches/gd-fix-gd2-read-test.patch		\
   %D%/packages/patches/gd-fix-tests-on-i686.patch		\
@@ -664,6 +666,7 @@ dist_patch_DATA =		\
   %D%/packages/patches/mcrypt-CVE-2012-4426.patch			\
   %D%/packages/patches/mcrypt-CVE-2012-4527.patch			\
   %D%/packages/patches/mhash-keygen-test-segfault.patch		\
+  %D%/packages/patches/mingw-w64-5.0rc2-gcc-4.9.3.patch		\
   %D%/packages/patches/mpc123-initialize-ao.patch		\
   %D%/packages/patches/mplayer2-theora-fix.patch		\
   %D%/packages/patches/module-init-tools-moduledir.patch	\
diff --git a/gnu/packages/mingw.scm b/gnu/packages/mingw.scm
new file mode 100644
index 000..4eb2c8f
--- /dev/null
+++ b/gnu/packages/mingw.scm
@@ -0,0 +1,81 @@
+;;; GNU Guix --- Functional package management for GNU
+;;; Copyright © 2016 Jan Nieuwenhuizen 
+;;;
+;;; This file is part of GNU Guix.
+;;;
+;;; GNU Guix is free software; you can redistribute it and/or modify it
+;;; under the terms of the GNU General Public License as published by
+;;; the Free Software Foundation; either version 3 of the License, or (at
+;;; your option) any later version.
+;;;
+;;; GNU Guix is distributed in the hope that it will be useful, but
+;;; WITHOUT ANY WARRANTY; without even the implied warranty of
+;;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+;;; GNU General Public License for more details.
+;;;
+;;; You should have received a copy of the GNU General Public License
+;;; along with GNU Guix.  If not, see .
+
+(define-module (gnu packages mingw)
+  #:use-module ((guix licenses) #:prefix license:)
+  #:use-module (gnu packages)
+  #:use-module (gnu packages base)
+  #:use-module (gnu packages cross-base)
+  #:use-module (gnu packages gcc)
+  #:use-module (gnu packages compression)
+  #:use-module (gnu packages multiprecision)
+  #:use-module (guix build-system gnu)
+  #:use-module (guix packages)
+  #:use-module (guix download)
+  #:use-module (guix utils)
+  #:use-module (ice-9 match))
+
+(define-public mingw-w64
+  (package
+(name "mingw-w64")
+(version "5.0-rc2")
+(source (origin
+  (method url-fetch)
+  (uri (string-append
+"https://sourceforge.net/projects/mingw-w64/files/mingw-w64/";
+"mingw-w64-release/mingw-w64-v" version ".tar.bz2"))
+  (sha256
+   (base32 "0imdary8j07if8ih73pfgxiclpf2ax8h3mz8mxln07i8sbbd30c9"))
+  (patches (search-patches "mingw-w64-5.0rc2-gcc-4.9.3.patch"
+(native-inputs `(("xgcc-core" ,xgcc-sans-libc-i686-w64-mingw32)
+ ("xbinutils" ,xbinutils-i686-w64-mingw32)))
+(build-system gnu-build-system)
+(search-paths
+ (list (search-path-specification
+(variable "CROSS_C_INCLUDE_PATH")
+(files '("include" "i686-w64-mingw32/include")))
+   (search-path-specification
+(variable "CROSS_LIBRARY_PATH")
+(files
+ '("lib" "lib64" "i686-w64-mingw32/lib" "i686-w64-mingw32/lib64")
+(arguments
+ `(#:configure-flags '("--host=i686-w64-mingw32")
+   #:phases
+   (modify-phases %standard-phases
+ (add-before

[PATCH v2] gnu: Move libusb from inputs to propagated-inputs

[Danny Milosavljevic]

gnu: Move libusb from inputs to propagated-inputs
---
 gnu/packages/libftdi.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/libftdi.scm b/gnu/packages/libftdi.scm
index 5c2671a..208e135 100644
--- a/gnu/packages/libftdi.scm
+++ b/gnu/packages/libftdi.scm
@@ -37,8 +37,8 @@
(base32
 "088yh8pxd6q53ssqndydcw1dkq51cjqyahc03lm6iip22cdazcf0"
 (build-system cmake-build-system)
-(inputs
- `(("libusb" ,libusb)))
+(propagated-inputs
+ `(("libusb" ,libusb))) ; required by libftdi1.pc
 (home-page "http://www.intra2net.com/en/developer/libftdi/";)
 (synopsis "FTDI USB driver with bitbang mode")
 (description

Re: [PATCH 17/24] gnu: kwidgetsaddons: Fix test failure.

[Danny Milosavljevic]

> Again since the tests are run in a container, I'd expect that there
> isn't a xserver
> running unless I explicitly start it.
> 
> Are my assumptions wrong?

> Do you think it's worth tracking down the test failure when my solution works?

No, if that's the case it's fine to use Xvfb too. It's just that 
QT_QPA_PLATFORM would be a lot less compute-intensive.

Re: Need help fixing testcases with time-difference

[Vincent Legoll]

Hello,

On Thu, Aug 18, 2016 at 10:05 AM, Hartmut Goebel
 wrote:
> Hi,
>
> I'm currently working on django, the web application framework.
> Unfortunalty some tests fail. These are all testing time- and timezone
> calculations.
>
> Failures are like this:
>
> AssertionError: datetime.timedelta(0, 3600, 16) not less than
> datetime.timedelta(0, 2)
>
> which means the returned time difference is ca. 1 hour, but allowed are
> only 2 minutes. The testcase os this one
> https://github.com/django/django/blob/master/tests/file_storage/tests.py#L239>
>
> I already added tzdata to native-inputs, but this does not solve the issue.
>
> Any hints?

That looks like a Daylight Saving Time mismatch, could that be possible ?

DST is 1h delta, which would be datetime.timedelta(0, 3600), thus if we remove
that, the test assertion becomes true...

>>> assert datetime.timedelta(0, 3600, 16) < datetime.timedelta(0, 2)
Traceback (most recent call last):
  File "", line 1, in 
AssertionError
>>> assert (datetime.timedelta(0, 3600, 16) - datetime.timedelta(0, 3600)) < 
>>> datetime.timedelta(0, 2)

WDYT?

-- 
Vincent Legoll

Need help fixing testcases with time-difference

[Hartmut Goebel]

Hi,

I'm currently working on django, the web application framework.
Unfortunalty some tests fail. These are all testing time- and timezone
calculations.

Failures are like this:

AssertionError: datetime.timedelta(0, 3600, 16) not less than
datetime.timedelta(0, 2)

which means the returned time difference is ca. 1 hour, but allowed are
only 2 minutes. The testcase os this one
https://github.com/django/django/blob/master/tests/file_storage/tests.py#L239>

I already added tzdata to native-inputs, but this does not solve the issue.

Any hints?

Below please find the stripped down package definition.

(define-module (gnu packages django)
  #:use-module ((guix licenses) #:prefix license:)
  #:use-module (guix packages)
  #:use-module (guix download)
  #:use-module (guix build-system python)
  #:use-module (gnu packages base)
  #:use-module (gnu packages python))

(define-public python-django
  (package
(name "python-django")
(version "1.10")
(source (origin
  (method url-fetch)
  (uri (pypi-uri "Django" version))
  (sha256
   (base32
"01bh5yra6zyxcpqacahbwfbn0y4ivw07j2jsw3crvmjzivb6if26"
(build-system python-build-system)
(arguments
 '(#:phases
   (modify-phases %standard-phases
 (replace
 'check
   (lambda* _
   (let* ((old-path (getenv "PYTHONPATH")))
 (chdir "tests")
 (setenv "PYTHONPATH"
 (string-append ".." (if old-path
 (string-append ":" old-path
 (zero? (system* "python" "runtests.py"))
   ))
(inputs
 ; Django uses pkg_resources (which is part of setuptools) to
 ; locate templates at run-time.
 `(("python-setuptools" ,python-setuptools)))
(native-inputs
 `(("tzdata", tzdata)
   ("python-docutils" ,python-docutils)
   ("python-jinja2" ,python-jinja2) ; >= 2.7
   ("python-numpy" ,python-numpy)
   ("python-pillow" ,python-pillow)
   ("python-pyyaml" ,python-pyyaml)
   ("python-pytz" ,python-pytz)
   ("python-sqlparse" ,python-sqlparse)
   ("python-tblib" ,python-tblib)
   ;; for Python 2: enum34 and mock
   ("python-enum34" ,python-enum34)
   ("python-mock" ,python-mock)
 ))
(home-page "http://www.djangoproject.com/";)
(synopsis "High-level Python Web framework")
(description "")
(license license:bsd-3)))


-- 
Regards
Hartmut Goebel

| Hartmut Goebel  | h.goe...@crazy-compilers.com   |
| www.crazy-compilers.com | compilers which you thought are impossible |

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

[Alex Kost]

Jan Nieuwenhuizen (2016-08-18 09:08 +0300) wrote:

[...]
> +(arguments
> + `(#:configure-flags '("--host=i686-w64-mingw32")
> +   #:phases
> +   (modify-phases %standard-phases
> + (add-before
> + 'configure 'setenv

it's more good-looking if this line is moved to the previous one :-)

(add-before 'configure 'setenv

> +   (lambda* (#:key inputs #:allow-other-keys)
> + (let ((xgcc-core (assoc-ref inputs "xgcc-core"))
> +   (mingw-headers (string-append (getcwd) 
> "/mingw-w64-headers")))
> +   (setenv "CPP"
> +   (string-append xgcc-core "/bin/i686-w64-mingw32-cpp"))
> +   (setenv "CROSS_C_INCLUDE_PATH"
> +   (string-append
> +mingw-headers
> +":" mingw-headers "/include"
> +":" mingw-headers "/crt"
> +":" mingw-headers "/defaults/include"
> +":" mingw-headers "/direct-x/include"))

-- 
Alex

Re: [PATCH 10/24] gnu: networkmanager-qt: Propagate network-manager.

[Alex Kost]

David Craven (2016-08-17 21:14 +0300) wrote:

>> Is it possible to avoid propagating?
>
> Quoting the manual on the section on propagated-inputs:
>
>> For example this is necessary when a C/C++ library needs headers of another 
>> library to compile, ...
>
> So the fact that the headers of networkmanager-qt include headers from
> network-manager, means that a library that uses networkmanager-qt also
> needs network-manager's header files. After compilation if there
> aren't any references to network-manager it won't be included in the
> RUNPATH.

Ah, OK, I didn't know that, thanks for pointing!

-- 
Alex

Re: [PATCH 07/24] gnu: compression: Add snappy.

[Alex Kost]

David Craven (2016-08-17 22:27 +0300) wrote:

>>> I don't believe you :-)
>>
>> Oh, that's embarrassing. I must have forgotten to change the hash and
>> guix cached the tarball. Is that possible?
>
> Sorry about that. I make many stupid mistakes, but I'm not a liar... ;-)

Oh, I didn't mean that!  I just wanted to say that it works as expected,
so you probably made some mistake.  As for the hash, yes it's possible
as Leo explained.

-- 
Alex