Re: X server crashes during 'guix system reconfigure' on mips64el

[Leo Famulari]

On Mon, Aug 22, 2016 at 01:28:39AM -0400, Mark H Weaver wrote:
> Hi Ludovic,
> 
> l...@gnu.org (Ludovic Courtès) writes:
> 
> > civodul pushed a commit to branch master
> > in repository guix.
> >
> > commit 3cf319a3f8e23831960a0f1320122cc514188a37
> > Author: Ludovic Courtès 
> > Date:   Sun Aug 21 19:03:29 2016 +0200
> >
> > services: elogind: Provide '%elogind-file-systems' by extension.
> > 
> > * gnu/system/file-systems.scm (%base-file-systems): Remove
> > %ELOGIND-FILE-SYSTEMS.
> > * gnu/services/desktop.scm (elogind-service-type): Extend
> > FILE-SYSTEM-SERVICE-TYPE to provide %ELOGIND-FILE-SYSTEMS.
> 
> This commit causes a major problem on my Lemote YeeLoong (mips64el)
> machine running GuixSD.  I don't know whether it might affect others.
> 
> When I run 'guix system reconfigure' to update to a system with this
> commit, it unloads the older %elogind-file-system services, which causes
> my X server to crash and leave me in text mode with my keyboard in the
> wrong mode, so I have to hard reset the machine.
> 
> In addition, the 'guix system reconfigure' gets stuck.  (I can see this
> because I always run it in a text terminal; I do all my 'root' work
> there.)

I have the same experience on x86_64.

Re: [PATCH 03/13] gnu: kde-frameworks: Add kcompletion.

[Mark H Weaver]

David Craven  writes:

> * gnu/packages/kde-frameworks.scm (kcompletion): New variable.
>
> Co-authored-by: Hartmut Goebel 

This fails to build on hydra:

  https://hydra.gnu.org/eval/109095?filter=kcompletion

  Mark

Re: [PATCH] gnu: asciidoc: Use local docbook-xsl package.

[Tomáš Čech]

On Sun, Aug 21, 2016 at 05:44:12PM -0400, Leo Famulari wrote:

On Fri, Aug 19, 2016 at 09:08:43PM +0200, Tomáš Čech wrote:

* gnu/packages/documentation.scm (asciidoc)[inputs]: Add docbook-xsl.
[arguments]: Add 'make-local-docbook-xsl' phase.


Thanks! I added a comment above the phase and pushed as dd10ba6356.


Thanks!


And then I remembered that you could have pushed it yourself. Oops!


That doesn't matter :)

I think we can close bug#24145 now.

S_W


signature.asc
Description: Digital signature

X server crashes during 'guix system reconfigure' on mips64el

[Mark H Weaver]

Hi Ludovic,

l...@gnu.org (Ludovic Courtès) writes:

> civodul pushed a commit to branch master
> in repository guix.
>
> commit 3cf319a3f8e23831960a0f1320122cc514188a37
> Author: Ludovic Courtès 
> Date:   Sun Aug 21 19:03:29 2016 +0200
>
> services: elogind: Provide '%elogind-file-systems' by extension.
> 
> * gnu/system/file-systems.scm (%base-file-systems): Remove
> %ELOGIND-FILE-SYSTEMS.
> * gnu/services/desktop.scm (elogind-service-type): Extend
> FILE-SYSTEM-SERVICE-TYPE to provide %ELOGIND-FILE-SYSTEMS.

This commit causes a major problem on my Lemote YeeLoong (mips64el)
machine running GuixSD.  I don't know whether it might affect others.

When I run 'guix system reconfigure' to update to a system with this
commit, it unloads the older %elogind-file-system services, which causes
my X server to crash and leave me in text mode with my keyboard in the
wrong mode, so I have to hard reset the machine.

In addition, the 'guix system reconfigure' gets stuck.  (I can see this
because I always run it in a text terminal; I do all my 'root' work
there.)

I've tried it twice, and transcribed (by hand) the last few lines of the
output from 'guix system reconfigure':

First time:

  making '/gnu/store/f0z88r...-system' the current system...
  guix system: unloading service 'file-system-/sys/fs/cgroup/elogind'...
  wlp0s14f5u3: deauthenticating from xx:xx:xx:xx:xx:xx by local choice (Reason: 
3=DEAUTH_LEAVING)

Second time:

  making '/gnu/store/f0z88r...-system' the current system...
  guix system: unloading service 'file-system-/run/systemd'...

I'm not sure if it's relevant, but I don't have 'elogin-service' on my
YeeLoong.  I can't run it there because it depends on 'mozjs' which
fails to build on mips64el.

   Mark

Re: [PATCH] gnu: gzochi: Update to 0.10.

[Mark H Weaver]

Hi Julian,

Julian Graham  writes:
> Linted, built, inspected by hand. Let me know if I've missed anything.
>
> From 584a3d62fe7105b90df433e460d0b6232716b1fc Mon Sep 17 00:00:00 2001
> From: Julian Graham 
> Date: Sun, 21 Aug 2016 11:10:41 -0400
> Subject: [PATCH] gnu: gzochi: Update to 0.10.
>
> * gnu/packages/game-development.scm (gzochi): Update to 0.10.

This update seems to have broken gzochi on i686 and armhf.  Segfaults
and aborts occur during the test suite.

  https://hydra.gnu.org/eval/109095?filter=gzochi

  https://hydra.gnu.org/build/1444785  (i686)
  https://hydra.gnu.org/build/1444781  (armhf)

Build logs are available from those pages.  As I write this, our
continuous integration system has not yet attempted the build on
mips64el, but when it does the results will be available here:

  https://hydra.gnu.org/build/1444727  (mips64el)

Any idea what's going wrong here?

 Thanks,
   Mark

Security update needed for spice

[Mark H Weaver]

Hi David,

There are two high-severity security flaws in spice that are apparently
fixed in spice-0.12.8:  CVE-2016-0749 and CVE-2016-2150

  https://lwn.net/Articles/697698/
  https://bugzilla.redhat.com/show_bug.cgi?id=1343135
  https://bugzilla.redhat.com/show_bug.cgi?id=1343137

While investigating, I noticed that we're using a "development release"
of spice (0.13.x) instead of a "stable release" (0.12.x):

  http://www.spice-space.org/download.html#stable-release

We should probably be using the stable release.  What do you think?

Anyway, would you be willing to handle this security update, by
switching Guix to a version of spice that's not vulnerable?
The summary line could end with "[fixes CVE-2016-{0749,2150}]."

Thanks for your contributions.

Mark

Re: [PATCH] gnu: Add dlib.

[Ben Woodcroft]

On 21/08/16 16:17, Leo Famulari wrote:

On Fri, Aug 19, 2016 at 11:52:37AM +0100, Marius Bakke wrote:

Leo Famulari  writes:


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 :)

I noticed this fails to build on Hydra. What's worse is that the i686,
x86_64 and armhf targets fails at completely different things. armhf and
i686 exits cleanly after failing 2 and 5 tests respectively, while the
x86_64 target seems to get the segfault we saw with lapack in inputs.

What should we do? I'd prefer to keep the package so it can easily be
tested on various architectures, but can understand if it is reverted.
Perhaps we can disable substitutes or tests, to stop bothering Hydra?

I'll try reproducing it this weekend on various qemu configurations.

Let us know about the results of your tests. Then we can decide what to
do.
Can this be fixed simply by using #parallel-build #f ? Doing so makes it 
reproducible for me so arguably we should be doing that anyway, even if 
it takes longer to build.


ben

Re: GCC bootstrap failure on ARM

[David Craven]

> Until the bootstrapping problem on ARM is fixed, we can't do any of
> this.  At the moment, that's blocking us from even updating to 4.9.4,
> nevermind 5 or 6.  If you want to help us upgrade our default GCC,
> investigating that problem would be the best way.

I have some unfinished stuff that I want to check of my todo list before
starting something like this... But I guess everyone is in the same boat
in this regard =)

Re: [PATCH] gnu: gzochi: Update to 0.10.

[David Craven]

Pushed as c8edd7e65fec92f3656cfdc4332a76a5b78e46af

Re: [PATCH 11/13] gnu: vlc: Fix regression caused by update to qt 5.7.

[Leo Famulari]

On Mon, Aug 22, 2016 at 01:19:05AM +0200, David Craven wrote:
> > So, I do think it's a good idea to use that string as documentation but,
> > in this case, I think that using "qtbase" is inappropriate. What do you
> > think?
> 
> So what would be a better string? "needs qtbase"?

I would just use "qt", and mention that it should instead use qtbase in
a code comment.

Re: [PATCH 11/13] gnu: vlc: Fix regression caused by update to qt 5.7.

[David Craven]

I ended up doing it like this:
 ("qt" ,qt) ; FIXME: reenable modular qt after update - requires building
 ;("qtbase" ,qtbase) with -std=gnu++11.
 ;("qtx11extras" ,qtx11extras)

Re: [PATCH 11/13] gnu: vlc: Fix regression caused by update to qt 5.7.

[David Craven]

> So, I do think it's a good idea to use that string as documentation but,
> in this case, I think that using "qtbase" is inappropriate. What do you
> think?

So what would be a better string? "needs qtbase"?

Re: [PATCH 11/13] gnu: vlc: Fix regression caused by update to qt 5.7.

[Leo Famulari]

On Mon, Aug 22, 2016 at 01:06:39AM +0200, David Craven wrote:
> I left the qtbase string as documentation, as has been done in other
> places. Do you think it's a bad idea (to not use the same string as
> the package name)?

My understanding is that the 'qt' package is not the same thing as the
'qtbase' package. 'qt' is the all-in-one Qt, whereas 'qtbase' is the
base package of the modularized Qt that Efraim has been working on.

So, I do think it's a good idea to use that string as documentation but,
in this case, I think that using "qtbase" is inappropriate. What do you
think?

Re: [PATCH 11/13] gnu: vlc: Fix regression caused by update to qt 5.7.

[David Craven]

libxi is required for it to build, I don't know why it worked before.
libinput is not strictly required, but I saw libinput.h not found
falling back to ... and since libinput is already in qt's dependency
graph, we can add it. But I'll make it a separate commit.

I left the qtbase string as documentation, as has been done in other
places. Do you think it's a bad idea (to not use the same string as
the package name)?

Re: [PATCH 07/13] gnu: gpsbabel: Fix regression caused by update to qt 5.7.

[David Craven]

Here is a sample of the test failures, they all look similar. This is
what I'd call a rounding error, but maybe it's something else?

   
-95.195312
+95.195313
 2013-08-15T15:19:01Z
 522.50
 TP8440
@@ -50705,7 +50705,7 @@
 TP8445
   
   
-95.101562
+95.101563
 2013-08-15T15:19:01Z
 511.944458
 TP8446

Re: [PATCH 04/13] gnu: poppler: Update to 0.47.0.

[David Craven]

> This would entail about 3000 rebuilds.  If it's important, I'd prefer to
> build it out on another branch.  If it's not important, maybe save it
> for core-updates?

I didn't realize it was such an important package. No it's not important,
updating a package is just the first thing I do when something doesn't
work that's all. I'll add the gnu++11 to poppler-qt5 only, that should
require less rebuilds.

Re: [PATCH 03/13] gnu: pinentry-qt: Fix regression.

[David Craven]

I removed gettext and added

(arguments
  `(#:configure-flags '("CXXFLAGS=-std=gnu++11")))

instead. I don't know why adding gettext as a native-input fixed the
build failure.

On Sun, Aug 21, 2016 at 10:16 PM, Mark H Weaver  wrote:
> David Craven  writes:
>
>> * gnu/packages/gnupg.scm (pinentry-qt)[native-inputs]: Add gnu-gettext.
>
> The summary line should say "Add gnu-gettext to native-inputs" instead
> of "Fix regression".
>
>> ---
>>  gnu/packages/gnupg.scm | 3 +++
>>  1 file changed, 3 insertions(+)
>>
>> diff --git a/gnu/packages/gnupg.scm b/gnu/packages/gnupg.scm
>> index 7df96b4..fde6018 100644
>> --- a/gnu/packages/gnupg.scm
>> +++ b/gnu/packages/gnupg.scm
>> @@ -28,6 +28,7 @@
>>#:use-module (gnu packages)
>>#:use-module (gnu packages adns)
>>#:use-module (gnu packages curl)
>> +  #:use-module (gnu packages gettext)
>>#:use-module (gnu packages openldap)
>>#:use-module (gnu packages perl)
>>#:use-module (gnu packages pth)
>> @@ -642,6 +643,8 @@ passphrase when @code{gpg} or @code{gpg2} is run and 
>> needs it.")))
>>(package
>>  (inherit pinentry-tty)
>>  (name "pinentry-qt")
>> +(native-inputs
>> + `(("gettext" ,gnu-gettext)))
>
> It would be good to understand why this is needed, because it shouldn't
> be needed for a tarball release.  Can you add a comment explaining the
> reason, possibly with "FIXME" in it?
>
>  Thanks,
>Mark

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

[Leo Famulari]

On Sat, Aug 20, 2016 at 09:27:25AM +, ng0 wrote:
> > From 3aec89f0444bd1e75f8dfa945e586a79790d254d Mon Sep 17 00:00:00 2001
> > From: ng0 
> > Date: Sat, 20 Aug 2016 08:37:22 +
> > Subject: [PATCH] gnu: gnupg: Fix test failures.
> >
> > * gnu/packages/gnupg.scm (gnupg)[arguments]: Set $HOME before 'check.

Thank you for catching this! Pushed as dd582f4ae5.

Re: [PATCH] gnu: asciidoc: Use local docbook-xsl package.

[Leo Famulari]

On Fri, Aug 19, 2016 at 09:08:43PM +0200, Tomáš Čech wrote:
> * gnu/packages/documentation.scm (asciidoc)[inputs]: Add docbook-xsl.
> [arguments]: Add 'make-local-docbook-xsl' phase.

Thanks! I added a comment above the phase and pushed as dd10ba6356.
And then I remembered that you could have pushed it yourself. Oops!

Re: [PATCH] gnu: gzochi: Update to 0.10.

[Julian Graham]

Hi David,

On Sun, Aug 21, 2016 at 5:20 PM, David Craven  wrote:
> I hadn't realized that you are the author of gzochi.

Yep!


> What I meant was that gmp end up being referenced by gzochi even dough
> it's not explicitly in the inputs field. Are you saying that this
> shouldn't be the case then?

No, I think that's fine. I'm not an expert with Guix, but I assume the
inclusion of GMP happens transitively - gzochi depends on Guile, Guile
depends on GMP.

Re: [PATCH] gnu: gzochi: Update to 0.10.

[David Craven]

Hi Julian,

I hadn't realized that you are the author of gzochi.

What I meant was that gmp end up being referenced by gzochi even dough
it's not explicitly in the inputs field. Are you saying that this
shouldn't be the case then?

guix gc --references $(guix build gzochi)
/gnu/store/00qwdmjrzflh0q4g9n18wcx8nvhhxcj1-bdb-6.2.23
/gnu/store/1c7rrngfbrfspfvwj7vvasvmc1cjqjlh-libltdl-2.4.6
/gnu/store/1qv5i6rfxjc4d0rg7z6r9dapmf85kzmy-gnutls-3.5.2
/gnu/store/2z0lv1px7izrmzxsqi3555yavqa56mkq-libidn-1.32
/gnu/store/3fhnmvbdxlsh64pr3zg4y74x3hlx33qw-libtasn1-4.8
/gnu/store/5992iq1v7arqa14ym3di58n4la0893nv-zlib-1.2.8
/gnu/store/5n16jhjpw494jrq87va51wqsmgaqm0yb-sdl-1.2.15
/gnu/store/7zhj5adf7kq7pr6j46wj1gjdc8vi0mwk-nettle-3.2
/gnu/store/95x5fxvcggyaa8ymjjgv5q0dnzkrign9-guile-2.0.11
/gnu/store/9nifwk709wajpyfwa0jzaa3p6mf10vxs-gcc-4.9.3-lib
/gnu/store/b8i9qnxwd4lsxk0g09xnkgshfn4arpmz-libmicrohttpd-0.9.50
>> /gnu/store/cdkrfbl10kbyyjjw3yfk9hckfw8n1b7g-gmp-6.1.0
/gnu/store/i9yr2vg3ksp7iyvvinwv9js82rgv4l04-libunistring-0.9.6
/gnu/store/l1s4cw9g58hmcpd2qgbckfl228143qzx-glib-2.48.0
/gnu/store/lv1jqm6cd9l265jwnihd48a5n7qxzg4d-libgcrypt-1.7.0
/gnu/store/m9vxvhdj691bq1f85lpflvnhcvrdilih-glibc-2.23
/gnu/store/sncfamh3fjrrdgd950d79g6yml2s6a07-libffi-3.2.1
/gnu/store/wdsqcvk2wrr83w4lyr4acpgscn07w6hy-libgpg-error-1.22
/gnu/store/x09wbcsh8wjc5y9yhc9mwf2a9gjnc0hv-libgc-7.4.2
/gnu/store/xl19qrfzga52vrvp4ncccwjlnrjqwj95-ncurses-6.0
/gnu/store/z4d6bwpzr05ga4mih3nza8k3jal37b5a-gzochi-0.10
/gnu/store/zwc6ck9j0wv80kz5snw5acwb39ws88m1-pcre-8.38

Re: [PATCH 12/13] gnu: avidemux: Fix regression caused by update to qt 5.7.

[David Craven]

The problem isn't with qt, when adding -std=gnu++11 the build fails.
And I'm not really a C/C++ developer, so sometimes the error messages
look a little intimidating. But yes I'll make the changes...

Re: [PATCH] gnu: gzochi: Update to 0.10.

[Julian Graham]

Hi Andreas,

On Sun, Aug 21, 2016 at 12:21 PM, Andreas Enge  wrote:
> Could you comment on this? Is it not needed any more? Or does dropping it
> removes an optional feature?

It's not needed as a first-order dependency any more - although as
David Craven notes in a separate email, it's a dependency of several
other core parts of Guix (such as Guile).


Regards,
Julian

Re: GNOME Applications menu can cause crashes

[Kei Kebreau]

Kei Kebreau  writes:

> Hello all,
>
> I was wondering if any other GNOME users encountered a odd bug while
> using the Applications menu. Dragging an icon while in the Applications
> menu causes GNOME shell to crash for unknown reasons. I'd like to help
> with fixing this bug if I can, but I don't know where to start or even
> where to look.
>
> I've attached the relevant information I could find from my system in
> case it helps at all.

I just got a gdb backtrace on the crash, following the instructions at
https://wiki.gnome.org/Projects/GnomeShell/Debugging. It is attached to
this email.

GNU gdb (GDB) 7.11.1
Copyright (C) 2016 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later 
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.  Type "show copying"
and "show warranty" for details.
This GDB was configured as "x86_64-unknown-linux-gnu".
Type "show configuration" for configuration details.
For bug reporting instructions, please see:
.
Find the GDB manual and other documentation resources online at:
.
For help, type "help".
Type "apropos word" to search for commands related to "word".
Attaching to process 8786
[New LWP 8790]
[New LWP 8791]
[New LWP 8793]
[New LWP 8795]
[New LWP 8796]
[New LWP 8797]
[Thread debugging using libthread_db enabled]
Using host libthread_db library 
"/gnu/store/m9vxvhdj691bq1f85lpflvnhcvrdilih-glibc-2.23/lib/libthread_db.so.1".
0x7f00bd535cdd in poll () from 
/gnu/store/m9vxvhdj691bq1f85lpflvnhcvrdilih-glibc-2.23/lib/libc.so.6
(gdb) continue
Continuing.
[New Thread 0x7f008700 (LWP 8907)]
[Thread 0x7f008700 (LWP 8907) exited]
[New Thread 0x7f008700 (LWP 8926)]
[New Thread 0x7f00b3fff700 (LWP 8927)]
[New Thread 0x7f008f3ad700 (LWP 8928)]
[New Thread 0x7f008ebac700 (LWP 8929)]
[Thread 0x7f008f3ad700 (LWP 8928) exited]
[Thread 0x7f00b3fff700 (LWP 8927) exited]
[Thread 0x7f008ebac700 (LWP 8929) exited]

Thread 1 "..gnome-shell-r" received signal SIGTRAP, Trace/breakpoint trap.
0x7f00bf47aed0 in g_logv () from 
/gnu/store/l1s4cw9g58hmcpd2qgbckfl228143qzx-glib-2.48.0/lib/libglib-2.0.so.0
(gdb) t a a bt

Thread 9 (Thread 0x7f008700 (LWP 8926)):
#0  0x7f00bd53a8a9 in syscall () from 
/gnu/store/m9vxvhdj691bq1f85lpflvnhcvrdilih-glibc-2.23/lib/libc.so.6
#1  0x7f00bf4b7492 in g_cond_wait_until () from 
/gnu/store/l1s4cw9g58hmcpd2qgbckfl228143qzx-glib-2.48.0/lib/libglib-2.0.so.0
#2  0x7f00bf44a259 in g_async_queue_pop_intern_unlocked () from 
/gnu/store/l1s4cw9g58hmcpd2qgbckfl228143qzx-glib-2.48.0/lib/libglib-2.0.so.0
#3  0x7f00bf44a84b in g_async_queue_timeout_pop () from 
/gnu/store/l1s4cw9g58hmcpd2qgbckfl228143qzx-glib-2.48.0/lib/libglib-2.0.so.0
#4  0x7f00bf49a78c in g_thread_pool_thread_proxy () from 
/gnu/store/l1s4cw9g58hmcpd2qgbckfl228143qzx-glib-2.48.0/lib/libglib-2.0.so.0
#5  0x7f00bf499d45 in g_thread_proxy () from 
/gnu/store/l1s4cw9g58hmcpd2qgbckfl228143qzx-glib-2.48.0/lib/libglib-2.0.so.0
#6  0x7f00befa7434 in start_thread () from 
/gnu/store/m9vxvhdj691bq1f85lpflvnhcvrdilih-glibc-2.23/lib/libpthread.so.0
#7  0x7f00bd53ed8d in clone () from 
/gnu/store/m9vxvhdj691bq1f85lpflvnhcvrdilih-glibc-2.23/lib/libc.so.6

Thread 7 (Thread 0x7f00b1de9700 (LWP 8797)):
#0  0x7f00befad07f in pthread_cond_wait@@GLIBC_2.3.2 () from 
/gnu/store/m9vxvhdj691bq1f85lpflvnhcvrdilih-glibc-2.23/lib/libpthread.so.0
#1  0x7f00bcbbd094 in PR_WaitCondVar () from 
/gnu/store/vhib3yw12yk8dg5ns4mdkijp3f9pqb0f-nspr-4.12/lib/libnspr4.so
#2  0x7f00ccbbbd9e in js::SourceCompressorThread::compressorThread(void*) ()
   from 
/gnu/store/dvk7wp9k9c8p9px7aii1psmhqnpjpvg8-mozjs-24.2.0/lib/libmozjs-24.so
#3  0x7f00bcbc4b12 in _pt_root () from 
/gnu/store/vhib3yw12yk8dg5ns4mdkijp3f9pqb0f-nspr-4.12/lib/libnspr4.so
#4  0x7f00befa7434 in start_thread () from 
/gnu/store/m9vxvhdj691bq1f85lpflvnhcvrdilih-glibc-2.23/lib/libpthread.so.0
#5  0x7f00bd53ed8d in clone () from 
/gnu/store/m9vxvhdj691bq1f85lpflvnhcvrdilih-glibc-2.23/lib/libc.so.6

Thread 6 (Thread 0x7f00b25ea700 (LWP 8796)):
#0  0x7f00befad07f in pthread_cond_wait@@GLIBC_2.3.2 () from 
/gnu/store/m9vxvhdj691bq1f85lpflvnhcvrdilih-glibc-2.23/lib/libpthread.so.0
#1  0x7f00bcbbd094 in PR_WaitCondVar () from 
/gnu/store/vhib3yw12yk8dg5ns4mdkijp3f9pqb0f-nspr-4.12/lib/libnspr4.so
#2  0x7f00ccb49b0e in js::GCHelperThread::threadLoop() () from 
/gnu/store/dvk7wp9k9c8p9px7aii1psmhqnpjpvg8-mozjs-24.2.0/lib/libmozjs-24.so
#3  0x7f00bcbc4b12 in _pt_root () from 
/gnu/store/vhib3yw12yk8dg5ns4mdkijp3f9pqb0f-nspr-4.12/lib/libnspr4.so
#4  0x7f00befa7434 in start_thread () from 
/gnu/store/m9vxvhdj691bq1f85lpflvnhcvrdilih-glibc-2.23/lib/libpthread.so.0
#5  0x7f00bd53ed8d in clone () from 
/gnu/store/m9vxvhdj691bq1f85lpflvnhcvrdilih-glibc-2.23/lib/libc.so.6

Thread 5 (Thread

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

[Leo Famulari]

On Thu, Aug 18, 2016 at 01:09:42PM +0200, Ricardo Wurmus wrote:
> * gnu/packages/bioinformatics.scm (r-bamsignals): New variable.

LGTM

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

[Leo Famulari]

On Thu, Aug 18, 2016 at 01:09:41PM +0200, Ricardo Wurmus wrote:
> * gnu/packages/bioinformatics.scm (r-rhtslib): New variable.

LGTM

Re: GCC bootstrap failure on ARM

[Mark H Weaver]

David Craven  writes:

> I can help fixing package regressions if someone gets the ball
> rolling. Is there a reason to upgrade to gcc 5 or does it make sense
> to jump to gcc 6 directly? If I understand correctly most of the work
> required for an update to gcc 5 has already been done?

Until the bootstrapping problem on ARM is fixed, we can't do any of
this.  At the moment, that's blocking us from even updating to 4.9.4,
nevermind 5 or 6.  If you want to help us upgrade our default GCC,
investigating that problem would be the best way.

   Mark

Re: [PATCH 03/13] gnu: pinentry-qt: Fix regression.

[Mark H Weaver]

David Craven  writes:

> * gnu/packages/gnupg.scm (pinentry-qt)[native-inputs]: Add gnu-gettext.

The summary line should say "Add gnu-gettext to native-inputs" instead
of "Fix regression".

> ---
>  gnu/packages/gnupg.scm | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/gnu/packages/gnupg.scm b/gnu/packages/gnupg.scm
> index 7df96b4..fde6018 100644
> --- a/gnu/packages/gnupg.scm
> +++ b/gnu/packages/gnupg.scm
> @@ -28,6 +28,7 @@
>#:use-module (gnu packages)
>#:use-module (gnu packages adns)
>#:use-module (gnu packages curl)
> +  #:use-module (gnu packages gettext)
>#:use-module (gnu packages openldap)
>#:use-module (gnu packages perl)
>#:use-module (gnu packages pth)
> @@ -642,6 +643,8 @@ passphrase when @code{gpg} or @code{gpg2} is run and 
> needs it.")))
>(package
>  (inherit pinentry-tty)
>  (name "pinentry-qt")
> +(native-inputs
> + `(("gettext" ,gnu-gettext)))

It would be good to understand why this is needed, because it shouldn't
be needed for a tarball release.  Can you add a comment explaining the
reason, possibly with "FIXME" in it?

 Thanks,
   Mark

Re: [PATCH] gnu: Add dlib.

[Leo Famulari]

On Fri, Aug 19, 2016 at 11:52:37AM +0100, Marius Bakke wrote:
> Leo Famulari  writes:
> 
> > 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 :)
> 
> I noticed this fails to build on Hydra. What's worse is that the i686,
> x86_64 and armhf targets fails at completely different things. armhf and
> i686 exits cleanly after failing 2 and 5 tests respectively, while the
> x86_64 target seems to get the segfault we saw with lapack in inputs.
> 
> What should we do? I'd prefer to keep the package so it can easily be
> tested on various architectures, but can understand if it is reverted.
> Perhaps we can disable substitutes or tests, to stop bothering Hydra?
> 
> I'll try reproducing it this weekend on various qemu configurations.

Let us know about the results of your tests. Then we can decide what to
do.

Re: [PATCH 12/13] gnu: avidemux: Fix regression caused by update to qt 5.7.

[Mark H Weaver]

David Craven  writes:

> * gnu/packages/video.scm (avidemux)[inputs]: Use qt 5.6.1-1.

In the summary line and in the line above, how about writing
"Use monolithic Qt 5.6."?

> ---
>  gnu/packages/video.scm | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
> index 45f8a28..7c68be8 100644
> --- a/gnu/packages/video.scm
> +++ b/gnu/packages/video.scm
> @@ -1144,8 +1144,8 @@ for use with HTML5 video.")
> ("perl" ,perl)
> ("pulseaudio" ,pulseaudio)
> ("python" ,python-wrapper)
> -   ("qtbase" ,qtbase)
> -   ("qttools" ,qttools)
> +   ("qtbase" ,qt)
> +   ;("qttools" ,qttools)

Please change "qtbase" => "qt" and add a FIXME comment reminding us to
get this working with modular Qt 5.7+.

What do you think?

  Mark

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

[Leo Famulari]

On Wed, Aug 17, 2016 at 05:53:31PM +0100, Marius Bakke wrote:
> Subject: [PATCH] gnu: Add python-odfpy.
> 
> * gnu/packages/python.scm (python-odfpy, python2-odfpy): New variables.

Thanks for the patch!

> +(arguments
> + `(#:phases
> +   (modify-phases %standard-phases
> + (replace 'check
> +   (lambda _ (zero? (system* "make" "-C" "tests")))
> +(native-inputs
> + ;; The test runner wants to check both python versions at once.
> + ;; To avoid patching it, we add them here.
> + `(("python-2" ,python-2)
> +   ("python-3" ,python-3)))

I think we should take another approach. Doing it this way makes the
python-odfpy and python2-odfpy built outputs refer to both Python
versions:

$ guix gc --references $(./pre-inst-env guix build python2-odfpy)
/gnu/store/m4gc2wx4q9if1vrhgclpspdil7rqsn21-python-3.4.3
/gnu/store/s0ckr1yhrzgfisxpsfz5b8mybjmvqkf1-python2-odfpy-1.3.3
/gnu/store/vcx1n5nj4gr52xx5m6gvi7zrwngy06s3-python-2.7.11
/gnu/store/ykzwykkvr2c80rw4l1qh3mvfdkl7jibi-bash-4.3.42

This looks like the file to patch:
https://github.com/eea/odfpy/blob/master/tests/runtests

Re: [PATCH 11/13] gnu: vlc: Fix regression caused by update to qt 5.7.

[Mark H Weaver]

David Craven  writes:
> * gnu/packages/video.scm (vlc)[inputs]: Use qt 5.6.1-1. Add libinput and 
> libxi.

The summary line is a bit misleading, because it leads me to guess that
this commit gets vlc working with Qt 5.7.  Better to write "gnu: vlc:
Use Qt 5.6.1."

Is it necessary to add 'libinput' and 'libxi' to fix this regression, or
is that an unrelated change to add new features?  If the latter, please
add those in a separate commit.

> @@ -635,7 +638,7 @@ audio/video codec library.")
> ("perl" ,perl)
> ("pulseaudio" ,pulseaudio)
> ("python" ,python-wrapper)
> -   ("qtbase" ,qtbase)
> +   ("qtbase" ,qt)

Please change the string to "qt" as well.

  Thanks,
Mark

Re: [PATCH] gnu: Add python-requests-oauthlib

[Leo Famulari]

On Fri, Aug 12, 2016 at 06:15:43PM -0700, Dylan Jeffers wrote:
> * gnu/packages/python.scm (python-requests-oauthlib,
> python2-requests-oauthlib): New variables.

Thank you!

> +(define-public python2-requests-oauthlib
> +  (package-with-python2 python-requests-oauthlib))

This python-2 variant of the package did not build because it needs
Setuptools, so I used the python2-variant system to provide
python2-setuptools.

I'd like to push the attached patch.

But, can you confirm which email address you want to use for copyright
attribution in 'gnu/packages/python.scm'? Currently, it is
 [0], which is different from
what you used in this patch. Please clarify :)

[0]
http://git.savannah.gnu.org/cgit/guix.git/tree/gnu/packages/python.scm#n27
>From e7c419b98362f69926f00d6ced08c7ea6ab91336 Mon Sep 17 00:00:00 2001
From: Dylan Jeffers 
Date: Fri, 12 Aug 2016 18:10:16 -0700
Subject: [PATCH] gnu: Add python-requests-oauthlib.

* gnu/packages/python.scm (python-requests-oauthlib,
python2-requests-oauthlib): New variables.

Co-authored-by: Leo Famulari 
---
 gnu/packages/python.scm | 44 ++--
 1 file changed, 42 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
index 15c957e..55e2ddc 100644
--- a/gnu/packages/python.scm
+++ b/gnu/packages/python.scm
@@ -1,4 +1,3 @@
-;;; GNU Guix --- Functional package management for GNU
 ;;; Copyright © 2013 Nikita Karetnikov 
 ;;; Copyright © 2013, 2014, 2015, 2016 Ludovic Courtès 
 ;;; Copyright © 2013, 2014, 2015, 2016 Andreas Enge 
@@ -24,7 +23,7 @@
 ;;; Copyright © 2016 Sou Bunnbu 
 ;;; Copyright © 2016 Troy Sankey 
 ;;; Copyright © 2016 ng0 
-;;; Copyright © 2016 Dylan Jeffers 
+;;; Copyright © 2016 Dylan Jeffers 
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -73,6 +72,7 @@
   #:use-module (gnu packages multiprecision)
   #:use-module (gnu packages networking)
   #:use-module (gnu packages ncurses)
+  #:use-module (gnu packages openstack)
   #:use-module (gnu packages pcre)
   #:use-module (gnu packages perl)
   #:use-module (gnu packages pkg-config)
@@ -9983,3 +9983,43 @@ hardware-accelerated multitouch applications.")
 
 (define-public python2-kivy-next
   (package-with-python2 python-kivy-next))
+
+(define-public python-requests-oauthlib
+  (package
+(name "python-requests-oauthlib")
+(version "0.6.2")
+(source
+ (origin
+   (method url-fetch)
+   (uri (pypi-uri "requests-oauthlib" version))
+   (sha256
+(base32
+ "0ykff67sjcl227c23g0rxzfx34rr5bf21kwv0z3zmgk0lfmch7hn"
+(build-system python-build-system)
+(arguments
+ `(#:phases
+   (modify-phases %standard-phases
+ ;; removes tests that require network access
+ (add-before 'check 'pre-check
+   (lambda _ (delete-file "tests/test_core.py")
+   #t)
+(native-inputs
+ `(("python-requests-mock" ,python-requests-mock)
+   ("python-mock" ,python-mock)))
+(inputs
+ `(("python-oauthlib" ,python-oauthlib)
+   ("python-requests" ,python-requests)))
+(home-page "https://github.com/requests/requests-oauthlib";)
+(synopsis "OAuthlib authentication support for Requests")
+(description "This package provides first-class OAuth library support for
+python-requests.")
+(license isc)
+(properties `((python2-variant . ,(delay python2-requests-oauthlib))
+
+(define-public python2-requests-oauthlib
+  (let ((base (package-with-python2
+   (strip-python2-variant python-requests-oauthlib
+(package
+  (inherit base)
+  (native-inputs `(("python2-setuptools" ,python2-setuptools)
+   ,@(package-native-inputs base))
-- 
2.9.3

Re: [PATCH 10/13] gnu: v4l-utils: Fix regression caused by update to qt 5.7.

[Mark H Weaver]

David Craven  writes:

> * gnu/packages/video.scm (v4l-utils)[arguments]: Require gnu++11.
> ---
>  gnu/packages/video.scm | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
> index bc6415e..28e49b3 100644
> --- a/gnu/packages/video.scm
> +++ b/gnu/packages/video.scm
> @@ -1396,7 +1396,8 @@ tools, XML authoring components, and an extensible 
> plug-in based API.")
>   '(#:configure-flags
> (list (string-append "--with-udevdir="
>  (assoc-ref %outputs "out")
> -"/lib/udev"
> +"/lib/udev")
> +  "CXXFLAGS=-std=gnu++11")))

The " should be lined up under the "(", so that last line should be
moved one column to the left.

Also, although I neglected to mention it, in general when the change
made is simple enough to describe in the summary line, better to do
that, so instead of writing "Fix regression caused by update to qt 5.7",
please write "Require gnu++11.".

Otherwise looks good to me.

 Thanks,
   Mark
   

Re: [PATCH 09/13] gnu: scribus: Fix regression caused by update to qt 5.7.

[Mark H Weaver]

David Craven  writes:

> * gnu/packages/scribus.scm (scribus)[arguments]: Require gnu++11.
> ---
>  gnu/packages/scribus.scm | 5 -
>  1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/gnu/packages/scribus.scm b/gnu/packages/scribus.scm
> index 68e57cd..4f3f306 100644
> --- a/gnu/packages/scribus.scm
> +++ b/gnu/packages/scribus.scm
> @@ -49,7 +49,10 @@
> (base32
>  "0f2adwg58w37sdi3xrk8xqw486p3pcfjaypcsswjl76r2f3yd0hq"
>  (build-system cmake-build-system)
> -(arguments `(#:tests? #f)) ; no test target
> +(arguments
> + `(#:tests? #f ; no test target
> +   #:configure-flags
> +'("-DCMAKE_CXX_FLAGS=-std=gnu++11")))

The ' should be under the #, but otherwise okay.

 Thanks,
   Mark

Re: [PATCH 08/13] gnu: synthv1: Fix regression caused by update to qt 5.7.

[Mark H Weaver]

David Craven  writes:

> * gnu/packages/music.scm (synthv1)[arguments]: Require gnu++11.
> ---
>  gnu/packages/music.scm | 6 --
>  1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/gnu/packages/music.scm b/gnu/packages/music.scm
> index dfd3f77..52eb3e0 100644
> --- a/gnu/packages/music.scm
> +++ b/gnu/packages/music.scm
> @@ -702,8 +702,10 @@ Editor.  It is compatible with Power Tab Editor 1.7 and 
> Guitar Pro.")
> (base32
>  "0h5zja78phf9705i9g54zh61iczb24iv7rxhljyms30sjgajig1y"
>  (build-system gnu-build-system)
> -;; There are no tests.
> -(arguments `(#:tests? #f))
> +(arguments
> + `(#:tests? #f ; There are no tests.
> +   #:configure-flags
> +'("CXXFLAGS=-std=gnu++11")))

For future reference, the ' should be under the #.
Otherwise looks okay.

  Thanks,
Mark

Re: [PATCH 07/13] gnu: gpsbabel: Fix regression caused by update to qt 5.7.

[Mark H Weaver]

David Craven  writes:

> * gnu/packages/gps.scm (gpsbabel)[arguments]: Require gnu++11. Disable tests
> on all platforms due to rounding error.
> ---
>  gnu/packages/gps.scm | 7 +--
>  1 file changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/gnu/packages/gps.scm b/gnu/packages/gps.scm
> index ccd9636..8124990 100644
> --- a/gnu/packages/gps.scm
> +++ b/gnu/packages/gps.scm
> @@ -48,7 +48,9 @@
>  "0xf7wmy2m29g2lm8lqc74yf8rf7sxfl3cfwbk7dpf0yf42pb0b6w"
>  (build-system gnu-build-system)
>  (arguments
> - `(#:configure-flags '("--with-zlib=system")
> + `(#:configure-flags
> +'("--with-zlib=system"
> +  "CXXFLAGS=-std=gnu++11")

The ' should be under the #, so the two lines above should be moved one
column to the left.

> #:phases
> (modify-phases %standard-phases
>  (add-before 'configure 'pre-configure
> @@ -59,7 +61,8 @@
> ;; On i686, 'raymarine.test' fails because of a rounding error:
> ;; .  As a workaround, disable 
> tests
> ;; on these platforms.
> -   #:tests? ,(not (string-prefix? "i686" (%current-system)
> +   ;; On x86_64 with -std=gnu++11 tests also fail due to rounding error.
> +   #:tests? #f))

Please add a FIXME to this comment.  We should not be in the habit of
simply disabling test suites that fail and forgetting about them.

I don't see why there should be a rounding error on x86_64.  Unlike
i686, double-rounding doesn't happen on that platform, and it's what
developers are mostly testing on.

Anyway, okay for now with the FIXME comment.

   Mark

Re: [PATCH 06/13] gnu: qsynth: Fix regression caused by update to qt 5.7.

[Mark H Weaver]

David Craven  writes:

> * gnu/packages/audio.scm (qsynth)[arguments]: Require gnu++11.
> ---
>  gnu/packages/audio.scm | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/gnu/packages/audio.scm b/gnu/packages/audio.scm
> index 6c0f066..2d741d5 100644
> --- a/gnu/packages/audio.scm
> +++ b/gnu/packages/audio.scm
> @@ -2113,7 +2113,9 @@ interface.")
>  (base32 "034p6mbwrjnxd9b6h20cidxi4ilkk3cgpjp154j0jzjs1ipf7x2h"
>  (build-system gnu-build-system)
>  (arguments
> - `(#:tests? #f)) ; no "check" phase
> + `(#:tests? #f ; no "check" phase
> +   #:configure-flags
> +'("CXXFLAGS=-std=gnu++11")))

It's not worth redoing this patch, but for future reference, the '
should be lined up under the #, although in this case it might be better
to just have it on the same line.

Otherwise, it looks good to me.

  Thanks,
Mark

Re: [PATCH 05/13] gnu: alsa-modular-synth: Fix regression caused by update to qt 5.7.

[Mark H Weaver]

David Craven  writes:

> * gnu/packages/audio.scm (alsa-modular-synth)[arguments]: Require gnu++11.
> ---
>  gnu/packages/audio.scm | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/gnu/packages/audio.scm b/gnu/packages/audio.scm
> index a214ef3..6c0f066 100644
> --- a/gnu/packages/audio.scm
> +++ b/gnu/packages/audio.scm
> @@ -85,7 +85,9 @@
>  "1azbrhpfk4nnybr7kgmc7w6al6xnzppg853vas8gmkh185kk11l0"
>  (build-system gnu-build-system)
>  (arguments
> - `(#:configure-flags '("--enable-qt5")
> + `(#:configure-flags
> +   '("--enable-qt5"
> + "CXXFLAGS=-std=gnu++11")
> #:phases
> (modify-phases %standard-phases
>   ;; Insert an extra space between linker flags.

Looks good to me.

 Thanks,
   Mark

Re: [PATCH 04/13] gnu: poppler: Update to 0.47.0.

[Mark H Weaver]

David Craven  writes:
> * gnu/packages/pdf.scm (poppler): Update it.
> (poppler)[arguments]: Require gnu++11.

This would entail about 3000 rebuilds.  If it's important, I'd prefer to
build it out on another branch.  If it's not important, maybe save it
for core-updates?

  Mark

Re: [PATCH 02/13] gnu: kwidgetsaddons: Fix regression caused by update to qt 5.7.

[Mark H Weaver]

David Craven  writes:

> * gnu/packages/kde-frameworks.scm (kwidgetsaddons)[arguments]: Disable tests.

You did more than disable tests.  Please describe the other changes.

> ---
>  gnu/packages/kde-frameworks.scm | 5 -
>  1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/gnu/packages/kde-frameworks.scm b/gnu/packages/kde-frameworks.scm
> index ba26952..20ab37a 100644
> --- a/gnu/packages/kde-frameworks.scm
> +++ b/gnu/packages/kde-frameworks.scm
> @@ -816,10 +816,13 @@ represented by a QPoint or a QSize.")
>  (inputs
>   `(("qtbase" ,qtbase)))
>  (arguments
> - `(#:phases
> + `(#:tests? #f ; FIXME: Regression after update to qt 5.7
> +   #:phases
>  (modify-phases %standard-phases
>(add-before 'check 'check-setup
>  (lambda* _
> +  (setenv "QT_QPA_PLATFORM" "offscreen") ; a better solution to 
> Xvfb
> +  (setenv "CTEST_OUTPUT_ON_FAILURE" "1") ; enable debug info
>(setenv "DBUS_FATAL_WARNINGS" "0")))

lambda* => lambda, and there should be a #t added after the last setenv.
Otherwise, looks okay, although I hope the test suite can be re-enabled
soon.

  Mark

Re: [PATCH 01/13] gnu: kguiaddons: Fix regression caused by update to qt 5.7.

[Mark H Weaver]

David Craven  writes:

> * gnu/packages/kde-frameworks.scm (kguiaddons)[arguments]:
> Remove phase 'start-xorg-server. Add phase 'check-setup.
> ---
>  gnu/packages/kde-frameworks.scm | 9 +++--
>  1 file changed, 3 insertions(+), 6 deletions(-)
>
> diff --git a/gnu/packages/kde-frameworks.scm b/gnu/packages/kde-frameworks.scm
> index af81547..ba26952 100644
> --- a/gnu/packages/kde-frameworks.scm
> +++ b/gnu/packages/kde-frameworks.scm
> @@ -539,12 +539,9 @@ infrastructure.")
>  (arguments
>   `(#:phases
>  (modify-phases %standard-phases
> -  (add-before 'check 'start-xorg-server
> -(lambda* (#:key inputs #:allow-other-keys)
> -  ;; The test suite requires a running X server.
> -  (system (string-append (assoc-ref inputs "xorg-server")
> - "/bin/Xvfb :1 &"))
> -  (setenv "DISPLAY" ":1")
> +  (add-before 'check 'check-setup
> +(lambda* _
> +  (setenv "QT_QPA_PLATFORM" "offscreen")
>   #t)

The #t should line up with (setenv ...), i.e. it should be moved one
column to the right.  Otherwise, looks good to me!

 Thanks,
   Mark

Re: [PATCH] Add Emacs Org mode

[Ricardo Wurmus]

Nicolas Goaziou  writes:

> The following patch adds Org mode to the list of Emacs packages.

I pushed this to master as 59065bb3304d6281c29ad7394a90fe17e8fb2b3b
after making these two changes:

- reordered fields to match the common order
- reflowed the description

Thanks again for the patch and the explanations!

~~ Ricardo

Re: [PATCH] gnu: gimp: Fix python plugin.

[Leo Famulari]

On Sun, Aug 21, 2016 at 12:32:17AM +0800, 宋文武 wrote:
> * gnu/packages/gimp.scm (gimp)[arguments]: Add phase to
> install 'sitecustomize.py'.

On #guix, one of the GIMP developers (schumaml) suggest we instead set
PYTHONPATH in the file 'data/environ/default.env':
https://git.gnome.org/browse/gimp/tree/data/environ

The relevant conversation begins here:
https://gnunet.org/bot/log/guix/2016-08-21#T1110898

I don't understand the pros and cons of the two different solutions, so
I'm not suggesting we change how we solve this problem. But, I wanted to
pass on the suggestion from the GIMP developer.

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

[Leo Famulari]

On Sun, Aug 21, 2016 at 05:31:50PM +0300, Manolis Ragkousis wrote:
> Hello Leo,
> 
> > 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.
> 
> This package only works with python 2. Actually if you try to build
> python-cheetah, you will get:

Okay, I think we should only package the python-2 version of cheetah.

GNOME Applications menu can cause crashes

[Kei Kebreau]

Hello all,

I was wondering if any other GNOME users encountered a odd bug while
using the Applications menu. Dragging an icon while in the Applications
menu causes GNOME shell to crash for unknown reasons. I'd like to help
with fixing this bug if I can, but I don't know where to start or even
where to look.

I've attached the relevant information I could find from my system in
case it helps at all.

[  899.142390] traps: ..gnome-shell-r[1215] trap int3 ip:7fd81d212ed0 
sp:7fff21008100 error:0 in libglib-2.0.so.0.4800.0[7fd81d1c4000+10c000]
Aug 21 13:56:10 localhost vmunix: [  899.142390] traps: ..gnome-shell-r[1215] 
trap int3 ip:7fd81d212ed0 sp:7fff21008100 error:0 in 
libglib-2.0.so.0.4800.0[7fd81d1c4000+10c000]
Aug 21 13:56:10 localhost gnome-session-binary[408]: WARNING: Application 
'org.gnome.Shell.desktop' killed by signal 5 
Aug 21 13:56:12 localhost dbus[317]: [system] Activating service 
name='org.freedesktop.GeoClue2' (using servicehelper)
Aug 21 13:56:12 localhost dbus[317]: [system] Successfully activated service 
'org.freedesktop.GeoClue2'
Aug 21 13:56:13 localhost dbus[317]: [system] Rejected send message, 2 matched 
rules; type="method_call", sender=":1.47" (uid=1000 pid=1382 
comm="/gnu/store/vfnc8lv7a5h6sb98qwmk9qi2zbziqkh1-gnome-") 
interface="org.freedesktop.DBus.ObjectManager" member="GetManagedObjects" error 
name="(unset)" requested_reply="0" destination=":1.3" (uid=0 pid=326 
comm="/gnu/store/c21lmnh3sd79jwi9znm1pz4wibmx2yvh-bluez-")



signature.asc
Description: PGP signature

Re: Python 3.5 start of update

[Leo Famulari]

On Wed, Jun 22, 2016 at 08:15:02AM -0500, Christopher Allan Webber wrote:
> Ludovic Courtès writes:
> >> From bbc1f68204318d79ece3fa46ed71d7c21d6ec8a2 Mon Sep 17 00:00:00 2001
> >> From: Christopher Allan Webber 
> 
> >> diff --git a/gnu/packages/patches/python-fix-tests-py3.5.patch 
> >> b/gnu/packages/patches/python-fix-tests-py3.5.patch
> >> new file mode 100644
> >> index 000..47dded4
> >> --- /dev/null
> >> +++ b/gnu/packages/patches/python-fix-tests-py3.5.patch
> >> @@ -0,0 +1,35 @@
> >> +Additional test fixes which affect Python 3.5 (and presumably later) but 
> >> not
> >> +prior revisions of Python.
> >> +
> >> +--- Lib/test/test_pathlib.py 2014-03-01 03:02:36.088311000 +0100
> >>  Lib/test/test_pathlib.py 2014-03-01 04:56:37.768311000 +0100
> >> +@@ -1986,8 +1986,9 @@
> >> + expect = set() if not support.fs_is_case_insensitive(BASE) else 
> >> given
> >> + self.assertEqual(given, expect)
> >> + self.assertEqual(set(p.rglob("FILEd*")), set())
> >> + 
> >> ++@unittest.skipIf(True, "Guix builder home is '/' which causes 
> >> trouble for these tests")
> >
> > Could it be that adding a phase that does (setenv "HOME" (getcwd)) would
> > help?

IIUC, we already do that in python-2, and python-3 inherits from
python-2. So, it seems that it's not enough.

> I'll try to get the rest of this done this afternoon, but it might
> happen sometime later this week.

;)

I picked this up. I've attached two patches. They upgrade python-3.4 to
3.4.5, and then upgrade python-3 to 3.5.2 while preserving a python-3.4.
These are the latest releases in their respective series.

I had to split part of the python-fix-tests.patch into 3.4 and 3.5
specific patches, since those packages' codebases have diverged.

python@3.4.5 and python@3.5.2 both build on master and wip-python.
python-setuptools builds with python@3.5.2 on master.

I'd like to put these on core-updates. The wip-python branch will take a
while to get ready. I found this saga about what Nix had to do to
upgrade setuptools:
https://github.com/NixOS/nixpkgs/pull/12552
From 34f52581bed88368396490d6e42fd1cb50e0a3f1 Mon Sep 17 00:00:00 2001
From: Christopher Allan Webber 
Date: Mon, 20 Jun 2016 14:05:20 -0500
Subject: [PATCH 1/2] gnu: python-3.4: Update to 3.4.5.

* gnu/packages/patches/python-disable-ssl-test.patch: Delete file.
* gnu/local.mk (dist_patch_DATA): Remove it.
* gnu/packages/python.scm (python-3.4): Update to 3.4.5.
[source]: Remove patch.
[arguments]: Remove field.
(python-minimal)[inputs]: Add libffi.
---
 gnu/local.mk   |  1 -
 gnu/packages/patches/python-disable-ssl-test.patch | 12 
 gnu/packages/python.scm| 14 +-
 3 files changed, 5 insertions(+), 22 deletions(-)
 delete mode 100644 gnu/packages/patches/python-disable-ssl-test.patch

diff --git a/gnu/local.mk b/gnu/local.mk
index 3444c20..f201232 100644
--- a/gnu/local.mk
+++ b/gnu/local.mk
@@ -734,7 +734,6 @@ dist_patch_DATA =   
\
   %D%/packages/patches/python-3-deterministic-build-info.patch \
   %D%/packages/patches/python-3-search-paths.patch \
   %D%/packages/patches/python-dendropy-exclude-failing-tests.patch \
-  %D%/packages/patches/python-disable-ssl-test.patch   \
   %D%/packages/patches/python-fix-tests.patch  \
   %D%/packages/patches/python-ipython-inputhook-ctype.patch\
   %D%/packages/patches/python-rarfile-fix-tests.patch  \
diff --git a/gnu/packages/patches/python-disable-ssl-test.patch 
b/gnu/packages/patches/python-disable-ssl-test.patch
deleted file mode 100644
index e351c77..000
--- a/gnu/packages/patches/python-disable-ssl-test.patch
+++ /dev/null
@@ -1,12 +0,0 @@
-Disable a test that fails with openssl-1.0.2b.
-
 Lib/test/test_ssl.py.orig  2015-02-25 06:27:45.0 -0500
-+++ Lib/test/test_ssl.py   2015-06-12 03:14:09.395212502 -0400
-@@ -2718,6 +2718,7 @@
-chatty=True, connectionchatty=True)
- self.assertIs(stats['compression'], None)
- 
-+@unittest.skipIf(True, "openssl 1.0.2b complains: dh key too small")
- def test_dh_params(self):
- # Check we can get a connection with ephemeral Diffie-Hellman
- context = ssl.SSLContext(ssl.PROTOCOL_TLSv1)
diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
index a83e4ee..5d529b0 100644
--- a/gnu/packages/python.scm
+++ b/gnu/packages/python.scm
@@ -301,21 +301,19 @@ data types.")
 
 (define-public python-3.4
   (package (inherit python-2)
-(version "3.4.3")
+(version "3.4.5")
 (source (origin
   (method url-fetch)
   (uri (string-append "https://www.python.org/ftp/python/";
   version "/Python-" version ".tar.xz"))
   (patches (search-patches
 "python-fix-tests.patch"
-;; X

[PATCH] Update entr to 3.6

[Matthew Jordan]

Good Day,

This patch moves entr to the shellutils file.  And updates the version.

>From 4cae2720568bab62785bd48ae0536a49eafabb81 Mon Sep 17 00:00:00 2001
From: "Matthew O'N.S Jordan" 
Date: Sat, 20 Aug 2016 11:32:25 -0400
Subject: [PATCH] gnu: Modify entr

 * gnu/packages/shellutils.scm (entr): Updated package version.
 * gnu/packages/entr.scm (entr): Removed file.
---
 gnu/local.mk|  1 -
 gnu/packages/entr.scm   | 66 -
 gnu/packages/shellutils.scm | 43 +
 3 files changed, 43 insertions(+), 67 deletions(-)
 delete mode 100644 gnu/packages/entr.scm

diff --git a/gnu/local.mk b/gnu/local.mk
index b8c5378..fe60187 100644
--- a/gnu/local.mk
+++ b/gnu/local.mk
@@ -111,7 +111,6 @@ GNU_SYSTEM_MODULES =\
   %D%/packages/enchant.scm			\
   %D%/packages/engineering.scm			\
   %D%/packages/enlightenment.scm		\
-  %D%/packages/entr.scm\
   %D%/packages/erlang.scm			\
   %D%/packages/fcitx.scm			\
   %D%/packages/feh.scm  \
diff --git a/gnu/packages/entr.scm b/gnu/packages/entr.scm
deleted file mode 100644
index 0e0f875..000
--- a/gnu/packages/entr.scm
+++ /dev/null
@@ -1,66 +0,0 @@
-;;; GNU Guix --- Functional package management for GNU
-;;; Copyright © 2016 Matthew Jordan 
-;;; Copyright © 2016 Ludovic Courtès 
-;;;
-;;; 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 entr)
-  #:use-module (guix licenses)
-  #:use-module (guix packages)
-  #:use-module (guix download)
-  #:use-module (guix build-system gnu))
-
-(define-public entr
-  (package
-(name "entr")
-(version "3.5")
-(source (origin
-  (method url-fetch)
-  (uri (string-append "http://entrproject.org/code/entr-";
-  version ".tar.gz"))
-  (sha256
-   (base32
-"05k4jyjna0pr2dalwc1l1dhrcyk6pw7hbss7jl4ykwfadcs5br73"
-(build-system gnu-build-system)
-(arguments
- `(#:test-target "test"
-   #:phases (modify-phases %standard-phases
-  (replace 'configure
-(lambda* (#:key outputs #:allow-other-keys)
-  (let ((out (assoc-ref outputs "out")))
-(setenv "CONFIG_SHELL" (which "bash"))
-(setenv "CC" (which "gcc"))
-(setenv "DESTDIR" (string-append out "/"))
-(setenv "PREFIX" "")
-(setenv "MANPREFIX" "man")
-(zero? (system* "./configure")
-  (add-before 'build 'remove-fhs-file-names
-(lambda _
-  ;; Use the tools available in $PATH.
-  (substitute* "entr.c"
-(("/bin/cat") "cat")
-(("/usr/bin/clear") "clear")))
-(home-page "http://entrproject.org/";)
-(synopsis "Run arbitrary commands when files change")
-(description
- "entr is a zero-configuration tool with no external build- or run-time
-dependencies.  The interface to entr is not only minimal, it aims to be simple
-enough to create a new category of ad hoc automation.  These micro-tests
-reduce keystrokes, but more importantly they emphasize the utility of
-automated checks.")
-
-;; Per 'LICENSE', portability code under missing/ is under BSD-2.
-(license isc)))
diff --git a/gnu/packages/shellutils.scm b/gnu/packages/shellutils.scm
index 27f45ba..be9aaf0 100644
--- a/gnu/packages/shellutils.scm
+++ b/gnu/packages/shellutils.scm
@@ -1,4 +1,5 @@
 ;;; GNU Guix --- Functional package management for GNU
+;;; Copyright © 2016 Ludovic Courtès 
 ;;; Copyright © 2016 Matthew Jordan 
 ;;; Copyright © 2016 Alex Griffin 
 ;;;
@@ -88,3 +89,45 @@ FreeDesktop.org trash can used by GNOME, KDE, XFCE, and other common desktop
 environments.  It can move files to the trash, and remove or list files that
 are already there.")
 (license gpl2+)))
+
+(define-public entr
+  (package
+(name "entr")
+(version "3.6")
+(source (origin
+  (method url-fetch)
+  (uri (string-append "http://entrproject.org/code/";
+  name "-" version ".tar.gz"))
+  (sha256
+   (base32
+"1sy81

[PATCH 1/2] Feed2maildir: Add python-feedparser

[Matthew Jordan]

Good Day,

This patch is to add python-feedparser.
Left me know if need to make any adjustments.

>From 41357ed77b80d7d151b341eef509685b45266546 Mon Sep 17 00:00:00 2001
From: "Matthew O'N.S Jordan" 
Date: Sun, 21 Aug 2016 11:45:52 -0400
Subject: [PATCH 1/2] gnu: Add python-feedparser

* gnu/packages/python.scm (python-feedparser): New variable.
---
 gnu/packages/python.scm | 27 +++
 1 file changed, 27 insertions(+)

diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
index 15c957e..a8a9857 100644
--- a/gnu/packages/python.scm
+++ b/gnu/packages/python.scm
@@ -25,6 +25,7 @@
 ;;; Copyright © 2016 Troy Sankey 
 ;;; Copyright © 2016 ng0 
 ;;; Copyright © 2016 Dylan Jeffers 
+;;; Copyright © 2016 Matthew Jordan 
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -9983,3 +9984,29 @@ hardware-accelerated multitouch applications.")
 
 (define-public python2-kivy-next
   (package-with-python2 python-kivy-next))
+
+(define-public python-feedparser
+  (package
+(name "python-feedparser")
+(version "5.2.1")
+(source
+ (origin
+   (method url-fetch)
+   (uri (string-append
+ "https://pypi.python.org/packages/91/d8/7d37fec71ff7c9dbcdd80d2b48bcdd86d6af502156fc93846fb0102cb2c4/feedparser-";
+ version ".tar.bz2"))
+   (sha256
+(base32
+ "00hb4qg2am06g81mygfi1jsbx8830024jm45g6qp9g8fr6am91yf"
+(build-system python-build-system)
+(inputs
+ `(("python-setuptools" ,python-setuptools)))
+(home-page "https://github.com/kurtmckee/feedparser";)
+(synopsis
+ "Universal feed parser, handles RSS 0.9x, RSS 1.0, RSS 2.0, CDF,
+Atom 0.3, and Atom 1.0 feeds")
+(description
+ "Universal feed parser, handles RSS 0.9x, RSS 1.0, RSS 2.0, CDF,
+Atom 0.3, and Atom 1.0 feeds")
+(license
+ (non-copyleft "https://github.com/kurtmckee/feedparser/blob/develop/LICENSE";
-- 
2.9.0


Sincerely,

Matthew Jordan

[PATCH 2/2] Feed2maildir: Add feed2maildir

[Matthew Jordan]

Good Day,

This a patch to add feed2maildir.
Let me know if I need to change anything.

>From 2bc27ff76ae53f9e7556d55a3dc2b4ffde6f72f7 Mon Sep 17 00:00:00 2001
From: "Matthew O'N.S Jordan" 
Date: Sun, 21 Aug 2016 11:47:17 -0400
Subject: [PATCH 2/2] gnu: Add feed2maildir

* gnu/packages/mail.scm (python-feed2maildir): New variable.
---
 gnu/packages/mail.scm | 27 +++
 1 file changed, 27 insertions(+)

diff --git a/gnu/packages/mail.scm b/gnu/packages/mail.scm
index 49bf8f7..d37557c 100644
--- a/gnu/packages/mail.scm
+++ b/gnu/packages/mail.scm
@@ -16,6 +16,7 @@
 ;;; Copyright © 2016 Alex Kost 
 ;;; Copyright © 2016 Troy Sankey 
 ;;; Copyright © 2016 ng0 
+;;; Copyright © 2016 Matthew Jordan 
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -1305,3 +1306,29 @@ Khard can also be used from within the email client @command{mutt}.")
   (description "Mail::SPF is the Sender Policy Framework implemented
 in Perl.")
   (license bsd-3)))
+
+(define-public python-feed2maildir
+  (package
+(name "feed2maildir")
+(version "0.3.5.3")
+(source
+ (origin
+   (method url-fetch)
+   (uri (string-append
+ "https://pypi.python.org/packages/5f/e7/d6d7235f13aced7aaa08bbd0d5bb95c87f2945c29e290caa399a014eaeae/";
+ name "-" version ".tar.gz"))
+   (sha256
+(base32
+ "18g3v99s6703wgc90cgrdz6i3viqk857liy05ndvfjsidnikym44"
+(build-system python-build-system)
+(arguments
+ `(#:tests? #f))
+(inputs
+ `(("python-setuptools" ,python-setuptools)
+   ("python-feedparser" ,python-feedparser)
+   ("python-dateutil" ,python-dateutil-2)))
+(home-page "https://github.com/sulami/feed2maildir";)
+(synopsis "Convert feeds to maildirs")
+(description "Convert feeds to maildirs")
+(license
+ (non-copyleft "https://github.com/sulami/feed2maildir/blob/master/LICENSE";
-- 
2.9.0


Sincerely,

Matthew Jordan

Re: Build machine sysadmin support

[David Craven]

So guix doesn't work inside a vm - because /gnu/store is mounted
readonly. To be able to offload arm builds once guix system vm
--system=armhf-linux works, /gnu/store needs to be writeable.

Can multiple daemons concurrently use the same store?

or alternatively, can we move the

(unless (zero? (system* unionfs "-o"

"cow,allow_other,use_ino,suid,dev,max_files=65536"
"/rw-root=RW:/real-root=RO"
"/root"))
  (error "unionfs failed"))

to after the 9p filesystems have been mounted, so that the unionfs
allows writing to /gnu/store too?

Re: [PATCH] gnu: Add libtoxcore.

[ng0]

I have started a request with other people to let tox* publish release
tarballs instead of just git and github's "master.tar.gz".

>From 3df3eee3c5841f954aca35c1024e54ec09e28f24 Mon Sep 17 00:00:00 2001
From: ng0 
Date: Sat, 20 Aug 2016 22:14:17 +
Subject: [PATCH 2/5] gnu: Add libtoxcore.

* gnu/packages/messaging.scm (libtoxcore): New variable.
---
 gnu/packages/messaging.scm | 48 +-
 1 file changed, 47 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/messaging.scm b/gnu/packages/messaging.scm
index de9ac17..e3c0c0b 100644
--- a/gnu/packages/messaging.scm
+++ b/gnu/packages/messaging.scm
@@ -5,6 +5,7 @@
 ;;; Copyright © 2015 Andreas Enge 
 ;;; Copyright © 2015 Ricardo Wurmus 
 ;;; Copyright © 2015 Efraim Flashner 
+;;; Copyright © 2016 ng0 
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -28,6 +29,7 @@
   #:use-module (guix utils)
   #:use-module (guix packages)
   #:use-module (guix download)
+  #:use-module (guix git-download)
   #:use-module (guix build-system gnu)
   #:use-module (guix build-system glib-or-gtk)
   #:use-module (guix build-system python)
@@ -35,6 +37,7 @@
   #:use-module (gnu packages autotools)
   #:use-module (gnu packages avahi)
   #:use-module (gnu packages check)
+  #:use-module (gnu packages crypto)
   #:use-module (gnu packages cyrus-sasl)
   #:use-module (gnu packages databases)
   #:use-module (gnu packages documentation)
@@ -58,7 +61,9 @@
   #:use-module (gnu packages admin)
   #:use-module (gnu packages linux)
   #:use-module (gnu packages tls)
-  #:use-module (gnu packages icu4c))
+  #:use-module (gnu packages icu4c)
+  #:use-module (gnu packages video)
+  #:use-module (gnu packages xiph))
 
 (define-public libotr
   (package
@@ -492,4 +497,45 @@ transformation; audio and video conferences; file transfer; TLS, GPG and
 end-to-end encryption support; XML console.")
 (license gpl3+)))
 
+(define-public libtoxcore
+  (let ((revision "1")
+(commit "755f084e8720b349026c85afbad58954cb7ff1d4"))
+(package
+  (name "libtoxcore")
+  (version (string-append "0.0.0" "-"
+  revision (string-take commit 8)))
+  (source (origin
+(method git-fetch)
+(uri (git-reference
+  (url "https://github.com/irungentoo/toxcore.git";)
+  (commit commit)))
+(file-name (string-append name "-" version "-checkout"))
+(sha256
+ (base32
+  "0ap1gvlyihnfivv235dbrgsxsiiz70bhlmlr5gn1027w3h5kqz8w"
+  (build-system gnu-build-system)
+  (native-inputs
+   `(("autoconf" ,autoconf)
+ ("automake" ,automake)
+ ("libtool" ,libtool)
+ ;; TODO: Add when test suite is capable of passing.
+ ;; ("check" ,check)
+ ("pkg-config" ,pkg-config)))
+  (inputs
+   `(("libsodium" ,libsodium)
+ ("opus" ,opus)
+ ("libvpx" ,libvpx)))
+  (arguments
+   `(#:phases
+ (modify-phases %standard-phases
+   (add-after 'unpack 'autoconf
+ (lambda _
+   (zero? (system* "./autogen.sh")
+ #:tests? #f)) ; FIXME: Testsuite fails, reasons unspecific.
+  (synopsis "Library for the Tox encrypted messenger protocol")
+  (description
+   "C library implementation of the Tox encrypted messenger protocol.")
+  (license gpl3+)
+  (home-page "https://tox.chat";
+
 ;;; messaging.scm ends here
-- 
2.9.3


About the very short description of freealut: I have 0 clue what exactly
this is other than what the description says and that it is needed for
tox client "toxic".

>From b04d4bf50354dbe481d237380428b4e2de3c22fe Mon Sep 17 00:00:00 2001
From: ng0 
Date: Sun, 21 Aug 2016 02:17:27 +
Subject: [PATCH 3/5] gnu: Add freealut.

* gnu/packages/audio.scm (freealut): New variable.
---
 gnu/packages/audio.scm | 27 +++
 1 file changed, 27 insertions(+)

diff --git a/gnu/packages/audio.scm b/gnu/packages/audio.scm
index a214ef3..900fb4b 100644
--- a/gnu/packages/audio.scm
+++ b/gnu/packages/audio.scm
@@ -6,6 +6,7 @@
 ;;; Copyright © 2015, 2016 Mark H Weaver 
 ;;; Copyright © 2016 Efraim Flashner 
 ;;; Copyright © 2016 Alex Griffin 
+;;; Copyright © 2016 ng0 
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -1509,6 +1510,32 @@ buffers, and audio capture.")
 (home-page "http://kcat.strangesoft.net/openal.html";)
 (license license:lgpl2.0+)))
 
+(define-public freealut
+  (package
+(name "freealut")
+(version "1.1.0")
+(source (origin
+  (method url-fetch)
+  ;; Upstream url is unclear, many systems use Fedora, there is also
+  ;; https://github.com/vancegroup/freealut though the status of it
+  ;; (official? unofficial?) is not clear.
+  (uri (string-append
+"https://pkgs.fedoraproject.org/repo/pkgs/"; name "/" name "-"
+ver

Re: [PATCH] gnu: gzochi: Update to 0.10.

[David Craven]

Isn't gmp already an implicit input of the gnu-build-system? It still
ends up in the runpath, it's just not required to be an input.

Re: [PATCH] gnu: Add toxic (Need help with ncurses+libnotify not being found).

[ng0]

ng0  writes:

> Hi,
>
> thanks for the CPATH info, I will try that.
>
> Alex Kost  writes:
>
>> ng0 (2016-08-21 05:25 +0300) wrote:
>>
>>> There are serious cosmetic improvements I need to to make on these
>>> patches, I am aware of this, but what I am asking help for is:
>>>
>>> I've tried for hours many ways to make toxic find "libnotify" and
>>> "ncursesw" ... It does not
>>> happen... setenv. substitute. ldflags. user_ldflags. cflags. user_cflags. 
>>> nothing..
>>>
>>> Can someone shed some light on this?
>>
>> As I see 'toxic' has a hand-written Makefile, so it probably doesn't use
>> pkg-config properly to find "libnotify".  Did you try to add
>> "/include/libnotify" to CPATH (as it is
>> done in 'webkitgtk' package, for example)?
>
> I think so, but I will try again, thanks for the package example.
>
>> [...]
>>> +(define-public libtoxcore
>>> +  (let ((revision "1")
>>> +(commit "755f084e8720b349026c85afbad58954cb7ff1d4"))
>>> +(package
>>> +  (name "libtoxcore")
>>> +  (version (string-append "0.0.0" "-"
>>> +  revision (string-take commit 8)))
>>> +  (source (origin
>>> +(method git-fetch)
>>> +(uri (git-reference
>>> +  (url "https://github.com/irungentoo/toxcore.git";)
>>> +  (commit commit)))
>>> +(file-name (string-append name "-" version))
>>
>> I prefer:  (file-name (string-append name "-" version "-checkout"))
>> I think it is more clean.
>
> ^ see "I am aware of the fact that it needs cosmetic changes" or: wip is
> very wip at my side usually, I would've changed this before an official
> patch send.
>
>>> +(sha256
>>> + (base32
>>> +  
>>> "0ap1gvlyihnfivv235dbrgsxsiiz70bhlmlr5gn1027w3h5kqz8w"
>>> +  (build-system gnu-build-system)
>>> +  (native-inputs
>>> +   `(("autoconf" ,autoconf)
>>> + ("automake" ,automake)
>>> + ("libtool" ,libtool)
>>> + ;; TODO: Add when test suite is capable of passing.
>>> + ;; ("check" ,check)
>>> + ("pkg-config" ,pkg-config)))
>>> +  (inputs
>>> +   `(("libsodium" ,libsodium)
>>> + ("opus" ,opus)
>>> + ("libvpx" ,libvpx)))
>>> +  (arguments
>>> +   `(#:phases
>>> + (modify-phases %standard-phases
>>> +   (add-before 'configure 'autoconf
>>
>> It should be added after 'unpack' phase, because... I forgot the reason :-)
>> but it relates to other phases between 'unpack' and 'configure' on "arm"
>> or "mips" systems.
>
> This package is a version bumped + changed alist to add-after/before
> package based on the pacakage davexunit has. I will adjust later.
>
>> [...]
>>> +(define-public toxic
>>> +  (package
>>> +(name "toxic")
>>> +(version "0.7.0")
>>> +(source (origin
>>> +  (method url-fetch)
>>> +  (uri (string-append "https://github.com/JFreegman/";
>>> +  name "/archive/v" version ".tar.gz"))
>>> +  (file-name (string-append name "-" version))
>>
>> Don't forget to change it to:
>>
>>  (file-name (string-append name "-" version ".tar.gz"))

I don't want to fix this. This is upstreams (toxic) problem, so I wait
on the result of my bugtickets.
The utox OrangeGoblin packaged works, I version bumped it.. I will
create an updated patch of libtoxcore, so that OrangeGoblin can send in
utox based on that.

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

Re: [PATCH] gnu: gzochi: Update to 0.10.

[Andreas Enge]

Hello,

thanks for your contribution!

On Sun, Aug 21, 2016 at 11:20:54AM -0400, Julian Graham wrote:
> -  ("gmp" ,gmp)

Could you comment on this? Is it not needed any more? Or does dropping it
removes an optional feature?

Andreas

Re: [PATCH] gnu: gzochi: Update to 0.10.

[David Craven]

Should be:

-;;; Copyright © 2015 Julian Graham 
+;; Copyright © 2015,2016 Julian Graham 

Should be mentioned in commit message.

> -  ("gmp" ,gmp)

No need to resend the patch.

Thank you for the update!

[PATCH] gnu: gzochi: Update to 0.10.

[David Craven]

From: Julian Graham 

* gnu/packages/game-development.scm (gzochi): Update to 0.10.
---
 gnu/packages/game-development.scm | 7 +++
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/gnu/packages/game-development.scm 
b/gnu/packages/game-development.scm
index 0bfd0c5..ba97a64 100644
--- a/gnu/packages/game-development.scm
+++ b/gnu/packages/game-development.scm
@@ -1,13 +1,13 @@
 ;;; GNU Guix --- Functional package management for GNU
 ;;; Copyright © 2014 Tomáš Čech 
 ;;; Copyright © 2015 Mark H Weaver 
-;;; Copyright © 2015 Julian Graham 
 ;;; Copyright © 2015 Ludovic Courtès 
 ;;; Copyright © 2015 Alex Kost 
 ;;; Copyright © 2015, 2016 David Thompson 
 ;;; Copyright © 2016 Efraim Flashner 
 ;;; Copyright © 2016 Kei Kebreau 
 ;;; Copyright © 2016 Ricardo Wurmus 
+;;; Copyright © 2016 Julian Graham 
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -93,14 +93,14 @@ is used in some video games and movies.")
 (define-public gzochi
   (package
 (name "gzochi")
-(version "0.9")
+(version "0.10")
 (source (origin
   (method url-fetch)
   (uri (string-append "mirror://savannah/gzochi/gzochi-"
   version ".tar.gz"))
   (sha256
(base32
-"1nf8naqbc4hmhy99b8n70yswg9j71nh5mfpwwh6d8pdw5mp9b46a"
+"055m7ywgl48ljwxf0kjhl76ldck890y5afdwjhk5s3p65xyaxh0k"
 (build-system gnu-build-system)
 (arguments
  '(#:phases (modify-phases %standard-phases
@@ -116,7 +116,6 @@ is used in some video games and movies.")
 (native-inputs `(("pkgconfig" ,pkg-config)))
 (inputs `(("bdb" ,bdb)
   ("glib" ,glib)
-  ("gmp" ,gmp)
   ("guile" ,guile-2.0)
   ("libmicrohttpd" ,libmicrohttpd)
   ("ncurses" ,ncurses)
-- 
2.5.0

[PATCH] gnu: gzochi: Update to 0.10.

[Julian Graham]

Linted, built, inspected by hand. Let me know if I've missed anything.
From 584a3d62fe7105b90df433e460d0b6232716b1fc Mon Sep 17 00:00:00 2001
From: Julian Graham 
Date: Sun, 21 Aug 2016 11:10:41 -0400
Subject: [PATCH] gnu: gzochi: Update to 0.10.

* gnu/packages/game-development.scm (gzochi): Update to 0.10.
---
 gnu/packages/game-development.scm | 7 +++
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/gnu/packages/game-development.scm b/gnu/packages/game-development.scm
index 0bfd0c5..ba97a64 100644
--- a/gnu/packages/game-development.scm
+++ b/gnu/packages/game-development.scm
@@ -1,13 +1,13 @@
 ;;; GNU Guix --- Functional package management for GNU
 ;;; Copyright © 2014 Tomáš Čech 
 ;;; Copyright © 2015 Mark H Weaver 
-;;; Copyright © 2015 Julian Graham 
 ;;; Copyright © 2015 Ludovic Courtès 
 ;;; Copyright © 2015 Alex Kost 
 ;;; Copyright © 2015, 2016 David Thompson 
 ;;; Copyright © 2016 Efraim Flashner 
 ;;; Copyright © 2016 Kei Kebreau 
 ;;; Copyright © 2016 Ricardo Wurmus 
+;;; Copyright © 2016 Julian Graham 
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -93,14 +93,14 @@ is used in some video games and movies.")
 (define-public gzochi
   (package
 (name "gzochi")
-(version "0.9")
+(version "0.10")
 (source (origin
   (method url-fetch)
   (uri (string-append "mirror://savannah/gzochi/gzochi-"
   version ".tar.gz"))
   (sha256
(base32
-"1nf8naqbc4hmhy99b8n70yswg9j71nh5mfpwwh6d8pdw5mp9b46a"
+"055m7ywgl48ljwxf0kjhl76ldck890y5afdwjhk5s3p65xyaxh0k"
 (build-system gnu-build-system)
 (arguments
  '(#:phases (modify-phases %standard-phases
@@ -116,7 +116,6 @@ is used in some video games and movies.")
 (native-inputs `(("pkgconfig" ,pkg-config)))
 (inputs `(("bdb" ,bdb)
   ("glib" ,glib)
-  ("gmp" ,gmp)
   ("guile" ,guile-2.0)
   ("libmicrohttpd" ,libmicrohttpd)
   ("ncurses" ,ncurses)
-- 
2.5.0

Re: Build machine sysadmin support

[David Craven]

guix install probably needs support for handling local-file's. This is
what I added as a template to gnu/system/examples. Apart from easily
deploying build-machines, I think this is also useful for getting guix
build --system=armhf-linux working.

;; GuixSD configuration file for the build machines.
;; Copyright © 2016 Ludovic Courtès 
;; Released under the GNU GPLv3 or any later version.

(use-modules (gnu system modules build-machines)
 (gnu system modules people)
 (guix))

(define %sysadmins
  (list (sysadmin (name "Alice")
  (full-name "Bob's sister Alice")
  (lsh-public-key (local-file
   (string-append (getenv "HOME")
  "/.ssh/id_ed25519.pub"))

(define %authorized-guix-keys
  ;; List of authorized 'guix archive' keys.
  (list (local-file "../../../hydra.gnu.org.pub")
(local-file "/etc/guix/signing-key.pub")))

;; The actual machine.
(build-machine-os "build-machine" "Europe/Paris" %sysadmins
  #:authorized-guix-keys %authorized-guix-keys)

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

[Manolis Ragkousis]

Hello Leo,

> 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.

This package only works with python 2. Actually if you try to build
python-cheetah, you will get:

phase `patch-source-shebangs' succeeded after 0.0 seconds
starting phase `patch-generated-file-shebangs'
phase `patch-generated-file-shebangs' succeeded after 0.0 seconds
starting phase `build'
running "python setup.py" with command "build" and parameters ()
Traceback (most recent call last):
  File "setup.py", line 10, in 
import SetupTools
  File
"/tmp/guix-build-python-cheetah-2.4.4.drv-0/Cheetah-2.4.4/SetupTools.py",
line 50
except DistutilsPlatformError, x:
 ^
SyntaxError: invalid syntax

I followed the other recipes as an example to write this one. Only
python2-cheetah works as expected.

Thank you,
Manolis


6bjwal0vhnzldn4sw1wwn0affy0k4f-python-cheetah-2.4.4.drv.bz2
Description: application/bzip

[PATCH 11/13] gnu: vlc: Fix regression caused by update to qt 5.7.

[David Craven]

* gnu/packages/video.scm (vlc)[inputs]: Use qt 5.6.1-1. Add libinput and libxi.
---
 gnu/packages/video.scm | 5 -
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index 28e49b3..45f8a28 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -53,6 +53,7 @@
   #:use-module (gnu packages databases)
   #:use-module (gnu packages elf)
   #:use-module (gnu packages fontutils)
+  #:use-module (gnu packages freedesktop)
   #:use-module (gnu packages fribidi)
   #:use-module (gnu packages gettext)
   #:use-module (gnu packages ghostscript)
@@ -617,6 +618,7 @@ audio/video codec library.")
("gnutls" ,gnutls)
("libcddb" ,libcddb)
("libgcrypt" ,libgcrypt)
+   ("libinput" ,libinput-minimal)
("libkate" ,libkate)
("libmad" ,libmad)
("libogg" ,libogg)
@@ -626,6 +628,7 @@ audio/video codec library.")
("libvorbis" ,libvorbis)
("libtheora" ,libtheora)
("libxext" ,libxext)
+   ("libxi" ,libxi)
("libxinerama" ,libxinerama)
("libxml2" ,libxml2)
("libxpm" ,libxpm)
@@ -635,7 +638,7 @@ audio/video codec library.")
("perl" ,perl)
("pulseaudio" ,pulseaudio)
("python" ,python-wrapper)
-   ("qtbase" ,qtbase)
+   ("qtbase" ,qt)
;("qtx11extras" ,qtx11extras)
("sdl" ,sdl)
("sdl-image" ,sdl-image)
-- 
2.9.0

[PATCH 03/13] gnu: pinentry-qt: Fix regression.

[David Craven]

* gnu/packages/gnupg.scm (pinentry-qt)[native-inputs]: Add gnu-gettext.
---
 gnu/packages/gnupg.scm | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/gnu/packages/gnupg.scm b/gnu/packages/gnupg.scm
index 7df96b4..fde6018 100644
--- a/gnu/packages/gnupg.scm
+++ b/gnu/packages/gnupg.scm
@@ -28,6 +28,7 @@
   #:use-module (gnu packages)
   #:use-module (gnu packages adns)
   #:use-module (gnu packages curl)
+  #:use-module (gnu packages gettext)
   #:use-module (gnu packages openldap)
   #:use-module (gnu packages perl)
   #:use-module (gnu packages pth)
@@ -642,6 +643,8 @@ passphrase when @code{gpg} or @code{gpg2} is run and needs 
it.")))
   (package
 (inherit pinentry-tty)
 (name "pinentry-qt")
+(native-inputs
+ `(("gettext" ,gnu-gettext)))
 (inputs
  `(("qtbase" ,qtbase)
,@(package-inputs pinentry-tty)))
-- 
2.9.0

[PATCH 05/13] gnu: alsa-modular-synth: Fix regression caused by update to qt 5.7.

[David Craven]

* gnu/packages/audio.scm (alsa-modular-synth)[arguments]: Require gnu++11.
---
 gnu/packages/audio.scm | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/audio.scm b/gnu/packages/audio.scm
index a214ef3..6c0f066 100644
--- a/gnu/packages/audio.scm
+++ b/gnu/packages/audio.scm
@@ -85,7 +85,9 @@
 "1azbrhpfk4nnybr7kgmc7w6al6xnzppg853vas8gmkh185kk11l0"
 (build-system gnu-build-system)
 (arguments
- `(#:configure-flags '("--enable-qt5")
+ `(#:configure-flags
+   '("--enable-qt5"
+ "CXXFLAGS=-std=gnu++11")
#:phases
(modify-phases %standard-phases
  ;; Insert an extra space between linker flags.
-- 
2.9.0

[PATCH 12/13] gnu: avidemux: Fix regression caused by update to qt 5.7.

[David Craven]

* gnu/packages/video.scm (avidemux)[inputs]: Use qt 5.6.1-1.
---
 gnu/packages/video.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index 45f8a28..7c68be8 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -1144,8 +1144,8 @@ for use with HTML5 video.")
("perl" ,perl)
("pulseaudio" ,pulseaudio)
("python" ,python-wrapper)
-   ("qtbase" ,qtbase)
-   ("qttools" ,qttools)
+   ("qtbase" ,qt)
+   ;("qttools" ,qttools)
("sdl" ,sdl)
("sqlite" ,sqlite)
("yasm" ,yasm)
-- 
2.9.0

[PATCH 02/13] gnu: kwidgetsaddons: Fix regression caused by update to qt 5.7.

[David Craven]

* gnu/packages/kde-frameworks.scm (kwidgetsaddons)[arguments]: Disable tests.
---
 gnu/packages/kde-frameworks.scm | 5 -
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/kde-frameworks.scm b/gnu/packages/kde-frameworks.scm
index ba26952..20ab37a 100644
--- a/gnu/packages/kde-frameworks.scm
+++ b/gnu/packages/kde-frameworks.scm
@@ -816,10 +816,13 @@ represented by a QPoint or a QSize.")
 (inputs
  `(("qtbase" ,qtbase)))
 (arguments
- `(#:phases
+ `(#:tests? #f ; FIXME: Regression after update to qt 5.7
+   #:phases
 (modify-phases %standard-phases
   (add-before 'check 'check-setup
 (lambda* _
+  (setenv "QT_QPA_PLATFORM" "offscreen") ; a better solution to 
Xvfb
+  (setenv "CTEST_OUTPUT_ON_FAILURE" "1") ; enable debug info
   (setenv "DBUS_FATAL_WARNINGS" "0")))
   (add-before 'check 'start-xorg-server
 (lambda* (#:key inputs #:allow-other-keys)
-- 
2.9.0

[PATCH 06/13] gnu: qsynth: Fix regression caused by update to qt 5.7.

[David Craven]

* gnu/packages/audio.scm (qsynth)[arguments]: Require gnu++11.
---
 gnu/packages/audio.scm | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/audio.scm b/gnu/packages/audio.scm
index 6c0f066..2d741d5 100644
--- a/gnu/packages/audio.scm
+++ b/gnu/packages/audio.scm
@@ -2113,7 +2113,9 @@ interface.")
 (base32 "034p6mbwrjnxd9b6h20cidxi4ilkk3cgpjp154j0jzjs1ipf7x2h"
 (build-system gnu-build-system)
 (arguments
- `(#:tests? #f)) ; no "check" phase
+ `(#:tests? #f ; no "check" phase
+   #:configure-flags
+'("CXXFLAGS=-std=gnu++11")))
 (native-inputs
  `(("qttools" ,qttools)))
 (inputs
-- 
2.9.0

[PATCH 08/13] gnu: synthv1: Fix regression caused by update to qt 5.7.

[David Craven]

* gnu/packages/music.scm (synthv1)[arguments]: Require gnu++11.
---
 gnu/packages/music.scm | 6 --
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/music.scm b/gnu/packages/music.scm
index dfd3f77..52eb3e0 100644
--- a/gnu/packages/music.scm
+++ b/gnu/packages/music.scm
@@ -702,8 +702,10 @@ Editor.  It is compatible with Power Tab Editor 1.7 and 
Guitar Pro.")
(base32
 "0h5zja78phf9705i9g54zh61iczb24iv7rxhljyms30sjgajig1y"
 (build-system gnu-build-system)
-;; There are no tests.
-(arguments `(#:tests? #f))
+(arguments
+ `(#:tests? #f ; There are no tests.
+   #:configure-flags
+'("CXXFLAGS=-std=gnu++11")))
 (inputs
  `(("jack" ,jack-1)
("lv2" ,lv2)
-- 
2.9.0

[PATCH 10/13] gnu: v4l-utils: Fix regression caused by update to qt 5.7.

[David Craven]

* gnu/packages/video.scm (v4l-utils)[arguments]: Require gnu++11.
---
 gnu/packages/video.scm | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index bc6415e..28e49b3 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -1396,7 +1396,8 @@ tools, XML authoring components, and an extensible 
plug-in based API.")
  '(#:configure-flags
(list (string-append "--with-udevdir="
 (assoc-ref %outputs "out")
-"/lib/udev"
+"/lib/udev")
+  "CXXFLAGS=-std=gnu++11")))
 (native-inputs
  `(("pkg-config" ,pkg-config)))
 (inputs
-- 
2.9.0

[PATCH 07/13] gnu: gpsbabel: Fix regression caused by update to qt 5.7.

[David Craven]

* gnu/packages/gps.scm (gpsbabel)[arguments]: Require gnu++11. Disable tests
on all platforms due to rounding error.
---
 gnu/packages/gps.scm | 7 +--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/gps.scm b/gnu/packages/gps.scm
index ccd9636..8124990 100644
--- a/gnu/packages/gps.scm
+++ b/gnu/packages/gps.scm
@@ -48,7 +48,9 @@
 "0xf7wmy2m29g2lm8lqc74yf8rf7sxfl3cfwbk7dpf0yf42pb0b6w"
 (build-system gnu-build-system)
 (arguments
- `(#:configure-flags '("--with-zlib=system")
+ `(#:configure-flags
+'("--with-zlib=system"
+  "CXXFLAGS=-std=gnu++11")
#:phases
(modify-phases %standard-phases
 (add-before 'configure 'pre-configure
@@ -59,7 +61,8 @@
;; On i686, 'raymarine.test' fails because of a rounding error:
;; .  As a workaround, disable tests
;; on these platforms.
-   #:tests? ,(not (string-prefix? "i686" (%current-system)
+   ;; On x86_64 with -std=gnu++11 tests also fail due to rounding error.
+   #:tests? #f))
 (inputs
  `(("expat" ,expat)
("zlib" ,zlib)
-- 
2.9.0

[PATCH 09/13] gnu: scribus: Fix regression caused by update to qt 5.7.

[David Craven]

* gnu/packages/scribus.scm (scribus)[arguments]: Require gnu++11.
---
 gnu/packages/scribus.scm | 5 -
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/scribus.scm b/gnu/packages/scribus.scm
index 68e57cd..4f3f306 100644
--- a/gnu/packages/scribus.scm
+++ b/gnu/packages/scribus.scm
@@ -49,7 +49,10 @@
(base32
 "0f2adwg58w37sdi3xrk8xqw486p3pcfjaypcsswjl76r2f3yd0hq"
 (build-system cmake-build-system)
-(arguments `(#:tests? #f)) ; no test target
+(arguments
+ `(#:tests? #f ; no test target
+   #:configure-flags
+'("-DCMAKE_CXX_FLAGS=-std=gnu++11")))
 (inputs
  `(("cairo" ,cairo)
("cups" ,cups)
-- 
2.9.0

[PATCH 04/13] gnu: poppler: Update to 0.47.0.

[David Craven]

* gnu/packages/pdf.scm (poppler): Update it.
(poppler)[arguments]: Require gnu++11.
---
 gnu/packages/pdf.scm | 7 ---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/gnu/packages/pdf.scm b/gnu/packages/pdf.scm
index 4ddcedd..2b3512f 100644
--- a/gnu/packages/pdf.scm
+++ b/gnu/packages/pdf.scm
@@ -57,14 +57,14 @@
 (define-public poppler
   (package
(name "poppler")
-   (version "0.43.0")
+   (version "0.47.0")
(source (origin
 (method url-fetch)
 (uri (string-append "https://poppler.freedesktop.org/poppler-";
 version ".tar.xz"))
 (sha256
  (base32
-  "0mi4zf0pz3x3fx3ir7szz1n57nywgbpd4mp2r7mvf47f4rmf4867"
+  "0hnjkcqqk87dw3hlda4gh4l7brkslniax9a79g772jn3iwiffwmq"
(build-system gnu-build-system)
;; FIXME:
;;  use libcurl:no
@@ -94,7 +94,8 @@
 "--enable-zlib"
 
 ;; Saves 8 MiB of .a files.
-"--disable-static")
+"--disable-static"
+"CXXFLAGS=-std=gnu++11")
   #:phases
   (alist-cons-before
'configure 'setenv
-- 
2.9.0

[PATCH 01/13] gnu: kguiaddons: Fix regression caused by update to qt 5.7.

[David Craven]

* gnu/packages/kde-frameworks.scm (kguiaddons)[arguments]:
Remove phase 'start-xorg-server. Add phase 'check-setup.
---
 gnu/packages/kde-frameworks.scm | 9 +++--
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/gnu/packages/kde-frameworks.scm b/gnu/packages/kde-frameworks.scm
index af81547..ba26952 100644
--- a/gnu/packages/kde-frameworks.scm
+++ b/gnu/packages/kde-frameworks.scm
@@ -539,12 +539,9 @@ infrastructure.")
 (arguments
  `(#:phases
 (modify-phases %standard-phases
-  (add-before 'check 'start-xorg-server
-(lambda* (#:key inputs #:allow-other-keys)
-  ;; The test suite requires a running X server.
-  (system (string-append (assoc-ref inputs "xorg-server")
- "/bin/Xvfb :1 &"))
-  (setenv "DISPLAY" ":1")
+  (add-before 'check 'check-setup
+(lambda* _
+  (setenv "QT_QPA_PLATFORM" "offscreen")
  #t)
 (home-page "https://community.kde.org/Frameworks";)
 (synopsis "Utilities for graphical user interfaces")
-- 
2.9.0

Re: Build machine sysadmin support

[David Craven]

> Also, all the sysadmin work will happen in a bunch of Scheme files,
> which means that everyone can now contribute to sysadmin, without
> necessarily having shell access to the machines.  No excuses!  :-)

Is there a reason why this isn't in gnu/system/examples? I think that
this is a common use case and would be nice to have it there, and
enable it in gnu/system/install.scm as an out-of-the-box experience.

Re: GCC bootstrap failure on ARM

[Andreas Enge]

On Sun, Aug 21, 2016 at 02:17:10PM +0200, David Craven wrote:
> Why are we using different package versions for the arm bootstrap
> binaries than for other platforms?

Simply because they have been created later, and there was no reason
to change the existing ones.

Andreas

Re: GCC bootstrap failure on ARM

[David Craven]

Why are we using different package versions for the arm bootstrap
binaries than for other platforms?

Re: GCC bootstrap failure on ARM

[David Craven]

I can help fixing package regressions if someone gets the ball
rolling. Is there a reason to upgrade to gcc 5 or does it make sense
to jump to gcc 6 directly? If I understand correctly most of the work
required for an update to gcc 5 has already been done?

Re: [PATCH] gnu: Add toxic (Need help with ncurses+libnotify not being found).

[ng0]

Hi,

thanks for the CPATH info, I will try that.

Alex Kost  writes:

> ng0 (2016-08-21 05:25 +0300) wrote:
>
>> There are serious cosmetic improvements I need to to make on these
>> patches, I am aware of this, but what I am asking help for is:
>>
>> I've tried for hours many ways to make toxic find "libnotify" and
>> "ncursesw" ... It does not
>> happen... setenv. substitute. ldflags. user_ldflags. cflags. user_cflags. 
>> nothing..
>>
>> Can someone shed some light on this?
>
> As I see 'toxic' has a hand-written Makefile, so it probably doesn't use
> pkg-config properly to find "libnotify".  Did you try to add
> "/include/libnotify" to CPATH (as it is
> done in 'webkitgtk' package, for example)?

I think so, but I will try again, thanks for the package example.

> [...]
>> +(define-public libtoxcore
>> +  (let ((revision "1")
>> +(commit "755f084e8720b349026c85afbad58954cb7ff1d4"))
>> +(package
>> +  (name "libtoxcore")
>> +  (version (string-append "0.0.0" "-"
>> +  revision (string-take commit 8)))
>> +  (source (origin
>> +(method git-fetch)
>> +(uri (git-reference
>> +  (url "https://github.com/irungentoo/toxcore.git";)
>> +  (commit commit)))
>> +(file-name (string-append name "-" version))
>
> I prefer:  (file-name (string-append name "-" version "-checkout"))
> I think it is more clean.

^ see "I am aware of the fact that it needs cosmetic changes" or: wip is
very wip at my side usually, I would've changed this before an official
patch send.

>> +(sha256
>> + (base32
>> +  "0ap1gvlyihnfivv235dbrgsxsiiz70bhlmlr5gn1027w3h5kqz8w"
>> +  (build-system gnu-build-system)
>> +  (native-inputs
>> +   `(("autoconf" ,autoconf)
>> + ("automake" ,automake)
>> + ("libtool" ,libtool)
>> + ;; TODO: Add when test suite is capable of passing.
>> + ;; ("check" ,check)
>> + ("pkg-config" ,pkg-config)))
>> +  (inputs
>> +   `(("libsodium" ,libsodium)
>> + ("opus" ,opus)
>> + ("libvpx" ,libvpx)))
>> +  (arguments
>> +   `(#:phases
>> + (modify-phases %standard-phases
>> +   (add-before 'configure 'autoconf
>
> It should be added after 'unpack' phase, because... I forgot the reason :-)
> but it relates to other phases between 'unpack' and 'configure' on "arm"
> or "mips" systems.

This package is a version bumped + changed alist to add-after/before
package based on the pacakage davexunit has. I will adjust later.

> [...]
>> +(define-public toxic
>> +  (package
>> +(name "toxic")
>> +(version "0.7.0")
>> +(source (origin
>> +  (method url-fetch)
>> +  (uri (string-append "https://github.com/JFreegman/";
>> +  name "/archive/v" version ".tar.gz"))
>> +  (file-name (string-append name "-" version))
>
> Don't forget to change it to:
>
>  (file-name (string-append name "-" version ".tar.gz"))
>
> -- 
> Alex

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

Re: [PATCH v6] gnu: Add u-boot, device-tree-compiler.

[David Craven]

Is this necessary?

+   #:make-flags '("HOSTCC=gcc")

this line also seems weird to me, why does the device-tree-compiler
respect HOME instead of PREFIX?

+ `(#:make-flags `("CC=gcc" ,(string-append "HOME=" (assoc-ref
%outputs "out"))) ; Note: or patch out PREFIX

I thought we were phasing #:select for licenses out?

+  #:use-module ((guix licenses) #:select (gpl2))

I think that we should add shortcuts for uboot loaders that would work
with our supported platforms in qemu. For arm-system-qemu that would
be the versatile-express board that works out of the box, I don't know
what the equivalent is for x86_64 and mips. (The rationale being that
the goal is to get guix system vm --system=armhf-linux working)

+;(define-public u-boot-a20-olinuxino-lime2
+;  (make-u-boot-package "A20-OLinuXino-Lime2"))
+
+;(define-public u-boot-efi-x86
+;  (make-u-boot-package "efi-x86"))

Re: [PATCH] Add Emacs Org mode

[Nicolas Goaziou]

Hello,

Ricardo Wurmus  writes:

> For the current stable release isn’t this the URL:
> http://orgmode.org/org-8.3.5.tar.gz ?  Should we use this instead?

The difference is about how Org is packaged. With the link above, you
need gnu-build-system and modify-phases, much like haskell-mode does.

However, Org is also distributed as an ELPA package. If we use this one,
we can rely on emacs-build-system and the recipe is trivial.

Regards,

-- 
Nicolas Goaziou0x80A93738

Re: [PATCH v6] gnu: Add u-boot, device-tree-compiler.

[Danny Milosavljevic]

Any news on this? This is the version Debian uses and that I'm using myself and 
it's probably not going to change substantially. It would be great if someone 
could get this into the Guix git repo if there are no objections.

Re: [PATCH] gnu: Add toxic (Need help with ncurses+libnotify not being found).

[Alex Kost]

ng0 (2016-08-21 05:25 +0300) wrote:

> There are serious cosmetic improvements I need to to make on these
> patches, I am aware of this, but what I am asking help for is:
>
> I've tried for hours many ways to make toxic find "libnotify" and
> "ncursesw" ... It does not
> happen... setenv. substitute. ldflags. user_ldflags. cflags. user_cflags. 
> nothing..
>
> Can someone shed some light on this?

As I see 'toxic' has a hand-written Makefile, so it probably doesn't use
pkg-config properly to find "libnotify".  Did you try to add
"/include/libnotify" to CPATH (as it is
done in 'webkitgtk' package, for example)?

[...]
> +(define-public libtoxcore
> +  (let ((revision "1")
> +(commit "755f084e8720b349026c85afbad58954cb7ff1d4"))
> +(package
> +  (name "libtoxcore")
> +  (version (string-append "0.0.0" "-"
> +  revision (string-take commit 8)))
> +  (source (origin
> +(method git-fetch)
> +(uri (git-reference
> +  (url "https://github.com/irungentoo/toxcore.git";)
> +  (commit commit)))
> +(file-name (string-append name "-" version))

I prefer:  (file-name (string-append name "-" version "-checkout"))
I think it is more clean.

> +(sha256
> + (base32
> +  "0ap1gvlyihnfivv235dbrgsxsiiz70bhlmlr5gn1027w3h5kqz8w"
> +  (build-system gnu-build-system)
> +  (native-inputs
> +   `(("autoconf" ,autoconf)
> + ("automake" ,automake)
> + ("libtool" ,libtool)
> + ;; TODO: Add when test suite is capable of passing.
> + ;; ("check" ,check)
> + ("pkg-config" ,pkg-config)))
> +  (inputs
> +   `(("libsodium" ,libsodium)
> + ("opus" ,opus)
> + ("libvpx" ,libvpx)))
> +  (arguments
> +   `(#:phases
> + (modify-phases %standard-phases
> +   (add-before 'configure 'autoconf

It should be added after 'unpack' phase, because... I forgot the reason :-)
but it relates to other phases between 'unpack' and 'configure' on "arm"
or "mips" systems.

[...]
> +(define-public toxic
> +  (package
> +(name "toxic")
> +(version "0.7.0")
> +(source (origin
> +  (method url-fetch)
> +  (uri (string-append "https://github.com/JFreegman/";
> +  name "/archive/v" version ".tar.gz"))
> +  (file-name (string-append name "-" version))

Don't forget to change it to:

 (file-name (string-append name "-" version ".tar.gz"))

-- 
Alex

Re: [PATCH] Add Emacs Org mode

[Ricardo Wurmus]

Nicolas Goaziou  writes:

> Hello,
>
> Efraim Flashner  writes:
>
>> On Sat, Aug 20, 2016 at 10:53:48AM +0200, Ricardo Wurmus wrote:
>>
>>> Thank you for the patch.  Org mode is already part of Emacs; is a
>>> separate package for Org mode still needed in this case?
>>> 
>> I notice the release date is about 2 weeks ago, so development seems to
>> be continuing, not just as part of emacs. How recently was orgmode added
>> to emacs? If it was quite recently then could this still be useful to
>> people using Guix on a foreign distro?
>
> Org has been part of Emacs for quite a long time already. However, as
> you notice, Org shipped with Emacs (8.2.10) is lagging behind current
> stable release (8.3.5). As a data point, Org 8.3 was released more than
> one year ago, and 8.3.5 differs from 8.2.10 by 2700+ commit. More
> importantly, bugs reported on 8.2.10 are unlikely to be fixed.
>
> This patch adds Org current stable release. However, a new release is
> going to be shipped on Monday.

For the current stable release isn’t this the URL:
http://orgmode.org/org-8.3.5.tar.gz ?  Should we use this instead?

~~ Ricardo