Re: [PATCH 11/13] build: Delete emacs/guix-autoloads.el when 'make clean'.

l...@gnu.org (Ludovic Courtès) writes: > Mathieu Lirzin skribis: > >> * emacs.am (CLEANFILES) [HAVE_EMACS]: Add $(AUTOLOADS). > > Additionally, $(AUTOLOADS) must be moved from ‘dist_lisp_DATA’ to > ‘nodist_lisp_DATA’. The commit’s subject line should become “Remove > guix-autoloads.el from the d

gtk-vnc package

hi, can you help me to review this patch, gtk-vnc is a API for Vinagre program. Regards.From cfeac9d0ca6505f98ba206aa4dae5170431baf8e Mon Sep 17 00:00:00 2001 From: Rene Saavedra Date: Tue, 26 Jan 2016 21:47:36 +0100 Subject: [PATCH] gnu: Add gtk-vnc. --- gnu-system.am| 1 + gn

Re: [PATCH 07/13] build: Add 'DL' silent rule.

l...@gnu.org (Ludovic Courtès) writes: > Mathieu Lirzin skribis: >> >> /bin/mkdir -p `dirname >> "gnu/packages/bootstrap/mips64el-linux/guile-2.0.9.tar.xz"` >> GUILE_LOAD_COMPILED_PATH=".:$GUILE_LOAD_COMPILED_PATH" >> /usr/bin/guile --no-auto-compile -L "." -L "." >> "./build-aux/download.scm" >

Re: [v2 1/1] doc: Show how to boot result of 'vm-image'.

Leo Famulari skribis: > On Tue, Jan 26, 2016 at 11:08:09AM +0100, Ludovic Courtès wrote: [...] >> “Defaults to 128@tie{}MiB, which may be insufficient for some >> operations.” (It’s not the daemon specifically, it’s mostly if you want >> to run X + Xfce, for instance. The former ‘guix substit

Re: [PATCH 04/13] utils: Use '@' for separating package names and version numbers.

Mathieu Lirzin skribis: > l...@gnu.org (Ludovic Courtès) writes: > >> Mathieu Lirzin skribis: >> >>> +++ b/guix/build/utils.scm >>> >>> (define (package-name->name+version name) >>> - "Given NAME, a package name like \"foo-0.9.1b\", return two values: >>> -\"foo\" and \"0.9.1b\". When the ver

Re: [PATCH 07/13] build: Add 'DL' silent rule.

Mathieu Lirzin skribis: > l...@gnu.org (Ludovic Courtès) writes: > >> Mathieu Lirzin skribis: >> >>> * Makefile.am (AM_V_DL, AM_V_DL_, AM_V_DL_0): New variables. >>> * gnu-system.am (gnu/packages/bootstrap/x86_64-linux/guile-2.0.9.tar.xz) >>> (gnu/packages/bootstrap/i686-linux/guile-2.0.9.tar.xz

Re: [PATCH 04/13] utils: Use '@' for separating package names and version numbers.

l...@gnu.org (Ludovic Courtès) writes: > Mathieu Lirzin skribis: > >> +++ b/guix/build/utils.scm >> >> (define (package-name->name+version name) >> - "Given NAME, a package name like \"foo-0.9.1b\", return two values: >> -\"foo\" and \"0.9.1b\". When the version part is unavailable, NAME and >

Re: [PATCH 06/13] build: Generate man pages after compiling guile objects.

l...@gnu.org (Ludovic Courtès) writes: > Mathieu Lirzin skribis: > >> * configure.ac (BUILD_FROM_GIT): New Automake conditional. >> * doc.am (SUBCOMMANDS): Delete variable. >> (dist_man1_MANS): List all subcommands man pages. >> (doc/guix.1): Build only if BUILD_FROM_GIT. Depend on 'scripts/guix

Re: [v2 1/1] doc: Show how to boot result of 'vm-image'.

On Tue, Jan 26, 2016 at 11:08:09AM +0100, Ludovic Courtès wrote: > Leo Famulari skribis: > > > * doc/guix.texi (Running GuixSD in a VM): New node. > > (Invoking guix system): Add reference to 'Running GuixSD in a VM'. > > [...] > > > +@node Running GuixSD in a VM > > +@subsection Running GuixSD

Re: [PATCH 07/13] build: Add 'DL' silent rule.

Mathieu Lirzin writes: > We could use the full word “DOWNLOAD” instead of “DL” but on the other > hand this will visually blend with the “LOAD” for guile modules. This might be subjective but I think "DOWNLOAD" and "LOAD" are pretty significantly different, both in shape and meaning (specificall

Perl-tk

Hello, there is one more package on core-updates which I noticed to pose problems, perl-tk, which only one dependency, clusterssh. It fails on all four architectures. So it would be nice if someone with an interest in one or the other of the packages could have a look. http://hydra.gnu.org:3000

Re: [PATCH 04/13] utils: Use '@' for separating package names and version numbers.

Alex Kost writes: > Mathieu Lirzin (2016-01-26 00:41 +0300) wrote: > >> Finally, There is some code redundancy for the emacs UI but If I >> remember correctly there is a reason for that. (Alex?) > > If there is redundancy in the code, it should be removed. I didn't find > a procedure to make a n

Re: [PATCH 07/13] build: Add 'DL' silent rule.

l...@gnu.org (Ludovic Courtès) writes: > Mathieu Lirzin skribis: > >> * Makefile.am (AM_V_DL, AM_V_DL_, AM_V_DL_0): New variables. >> * gnu-system.am (gnu/packages/bootstrap/x86_64-linux/guile-2.0.9.tar.xz) >> (gnu/packages/bootstrap/i686-linux/guile-2.0.9.tar.xz) >> (gnu/packages/bootstrap/armhf

Re: [PATCH 06/13] build: Generate man pages after compiling guile objects.

Mathieu Lirzin skribis: > This improves compilation speed since scripts are not compiled twice. > When building with ‘make -j4‘ from a clean repository on an Intel > i5-2540M, the time of compilation gets from: > > real 5m7.220s > user 18m49.788s > sys 0m12.964s > > down to: > > real 2m20.82

Re: [SHEPHERD] [PATCH] service: 'conflicts-with' returns a list of services.

Alex Kost skribis: > From 9ce0f8b9ba720d21eca43fdb9b2ce749079a305b Mon Sep 17 00:00:00 2001 > From: Alex Kost > Date: Tue, 26 Jan 2016 12:12:41 +0300 > Subject: [PATCH] service: 'conflicts-with' returns a list of services. > > This fixes a regression introduced in commit > cb168150d4581ffbc357ab

Re: [PATCH 12/13] build: Use 'GEN' and 'at' silent rules.

Mathieu Lirzin skribis: > * daemon.am (nix/libstore/schema.sql.hh, etc/guix-daemon.service) > (etc/guix-daemon.conf): Use $(AM_V_GEN). > * doc.am (doc/os-config-%.texi, .png.eps): Likewise. > * Makefile.am (guix-binary.%.tar.xz, gen-ChangeLog, gen-AUTHORS): > Likewise. > (assert-binaries-availabl

Re: [PATCH 08/13] build: Add 'DOT' silent rule.

Mathieu Lirzin skribis: > * Makefile.am (AM_V_DOT, AM_V_DOT_, AM_V_DOT_0): New variables. > * doc.am (.dot.png, .dot.pdf, .dot.eps): Use $(AM_V_DOT). OK! Ludo'.

Re: [PATCH 13/13] Update .gitignore.

Mathieu Lirzin skribis: > * .gitignore: Sort lines. Use globbing for "*.{eps,pdf,png,tar.xz}" > files. OK.

Re: [PATCH 11/13] build: Delete emacs/guix-autoloads.el when 'make clean'.

Mathieu Lirzin skribis: > * emacs.am (CLEANFILES) [HAVE_EMACS]: Add $(AUTOLOADS). Additionally, $(AUTOLOADS) must be moved from ‘dist_lisp_DATA’ to ‘nodist_lisp_DATA’. The commit’s subject line should become “Remove guix-autoloads.el from the distribution.” $(AUTOLOADS) were wrongfully part of

Re: [PATCH 10/13] build: Add 'HELP2MAN' silent rule.

Mathieu Lirzin skribis: > * Makefile.am (AM_V_HELP2MAN, AM_V_HELP2MAN_, AM_V_HELP2MAN_0): New > variables. > * doc.am [BUILD_FROM_GIT] (gen_man): Use $(AM_V_EMACS). OK!

Re: [PATCH 09/13] build: Add 'EMACS' silent rule.

Mathieu Lirzin skribis: > * Makefile.am (AM_V_EMACS, AM_V_EMACS_, AM_V_EMACS_0): New variables. > * emacs.am [HAVE_EMACS] ($(AUTOLOADS)): Use $(AM_V_EMACS). OK!

Re: [PATCH 07/13] build: Add 'DL' silent rule.

Mathieu Lirzin skribis: > * Makefile.am (AM_V_DL, AM_V_DL_, AM_V_DL_0): New variables. > * gnu-system.am (gnu/packages/bootstrap/x86_64-linux/guile-2.0.9.tar.xz) > (gnu/packages/bootstrap/i686-linux/guile-2.0.9.tar.xz) > (gnu/packages/bootstrap/armhf-linux/guile-2.0.11.tar.xz) > (gnu/packages/boo

Re: [PATCH 05/13] packages: Use '@' in package record printers.

Mathieu Lirzin skribis: > * guix/packages.scm : Use '@' in record printer. > * guix/import/cabal.scm : Likewise > * guix/import/elpa.scm : Likewise. > * tests/packages.scm: Adapt to it. OK! Ludo'.

Re: [PATCH 04/13] utils: Use '@' for separating package names and version numbers.

Mathieu Lirzin skribis: > From 7fef43249b704db3c4d511b2f1b62428740cfa73 Mon Sep 17 00:00:00 2001 > From: Mathieu Lirzin > Date: Sun, 24 Jan 2016 01:54:44 +0100 > Subject: [PATCH] utils: Use '@' for separating package names and version > numbers. > > Fixes . > > * guix

Re: [PATCH 03/13] lint: Rewrite 'check-patch-file-names'.

Mathieu Lirzin skribis: > * guix/scripts/lint.scm (check-patch-file-names): Improve clarity by > reversing the logic. > --- > guix/scripts/lint.scm | 29 + > 1 file changed, 13 insertions(+), 16 deletions(-) OK.

Re: [PATCH 02/13] lint: Remove an unneeded clause in 'check-patch-file-names'.

Mathieu Lirzin skribis: > * guix/scripts/lint.scm (check-patch-file-names): Don't check if patches > start with the package full name since matching the package name (which > is a prefix of the full name) is sufficient. Indeed, OK.

Re: [PATCH 01/13] tests: Use 'dummy-origin' for lint tests when possible.

Mathieu Lirzin skribis: > * guix/tests/lint.scm ("patches: file names", "patches: not found"): Use > 'dummy-origin'. 'sha256' field was wrongly set to a string instead of a > bytevector. OK! (The issues was harmless though in this context.) Ludo’.

Re: [shepherd] 06/09: Rename default system configuration file.

Alex Kost skribis: > --- a/modules/shepherd/support.scm > +++ b/modules/shepherd/support.scm > @@ -280,7 +280,7 @@ TARGET should be a string representing a filepath + name." > global system configuration file when running as 'root'. As a side effect, > create a template configuration file if n

Re: [PATCH] import: pypi: read requirements from wheels.

Cyril Roelandt skribis: > On 01/24/2016 09:08 PM, Ludovic Courtès wrote: >> Cyril Roelandt skribis: >> >>> * guix/import/pypi.scm (latest-wheel-release): New function. >> >> s/function/procedure/ :-) >> >> Please also mention the changes in ‘guess-requirements’, >> ‘compute-inputs’, etc. >>

Re: [v2 1/1] doc: Show how to boot result of 'vm-image'.

Leo Famulari skribis: > * doc/guix.texi (Running GuixSD in a VM): New node. > (Invoking guix system): Add reference to 'Running GuixSD in a VM'. [...] > +@node Running GuixSD in a VM > +@subsection Running GuixSD in a VM s/VM/virtual machine/ on the second line. > +One way to run GuixSD in a

Re: [PATCH] segfaulting gnunet-gtk.

Ricardo Wurmus skribis: > the attached patches make for a buildable gnunet-gtk package, but it > doesn’t seem fully functional. I see a segfault when switching tabs > in “gnunet-setup” after first interacting with the URL dropdown. > > In “gnunet-gtk” I also don’t see anything more than the imag

Re: [PATCH 4/4] Rename 'dmd' to 'shepherd' in a generated user config.

Alex Kost skribis: > * modules/shepherd/support.scm (make-bare-init-file): Adjust comment > lines for the rename. Looks good, please push. If you think there are other patches that I forgot, let me know! Thanks, Ludo’.

[SHEPHERD] [PATCH] service: 'conflicts-with' returns a list of services.

As explained in the commit message, currently (after commit cb16815¹) 'conflicts-with' returns a list of names (symbols), so 'conflicts-with-running' fails as 'running?' can be applied only to a service object, but not to a symbol. ¹ http://git.savannah.gnu.org/cgit/shepherd.git/commit/?id=cb1681

Re: [SHEPHERD] [PATCH] service: Improve 'service-list'.

Ludovic Courtès (2016-01-26 00:49 +0300) wrote: > Alex Kost skribis: > >> Ludovic Courtès (2016-01-25 18:06 +0300) wrote: >> >>> Alex Kost skribis: >>> From f3d21e3ec8a100a966153d03264639ebe48e8872 Mon Sep 17 00:00:00 2001 From: Alex Kost Date: Mon, 25 Jan 2016 11:18:00 +0300 >>>

Re: [PATCH 7/8] utils: Rename 'dmd-gettext.scm' to 'shepherd-gettext.scm'.

Ludovic Courtès (2016-01-26 00:51 +0300) wrote: > Alex Kost skribis: > >> Ludovic Courtès (2016-01-25 18:12 +0300) wrote: >> >>> Alex Kost skribis: >>> * utils/dmd-gettext.scm: Rename to... * utils/shepherd-gettext.scm: ... this. * utils/Makefile.am (EXTRA_DIST): Use it. >>> >>> A