Please set up hdyra-job for python-branch

2016-11-25 Thread Hartmut Goebel
Hi, the python-branch is ready to be build since some weeks now. Could anybody being in charge please set up a hydra-channel to get it build? Thanks! -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which

Preparing mass-packaging of 170 KDE application - hwo to do?

2016-11-25 Thread Hartmut Goebel
artmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thought are impossible |

is using eval good style in guile?(was: [PATCH] improve nginx-service)

2016-11-25 Thread Hartmut Goebel
ded bad programming style. Is this different in guile? -- Schönen Gruß Hartmut Goebel Dipl.-Informatiker (univ), CISSP, CSSLP, ISO 27001 Lead Implementer Information Security Management, Security Governance, Secure Software Development Goebel Consult, Landshut http://www.goebel-consult.de

Re: [PATCH] gnu: Add python-polib

2016-11-25 Thread Hartmut Goebel
hy are you not using package-with-python2 here? -- Schönen Gruß Hartmut Goebel Dipl.-Informatiker (univ), CISSP, CSSLP, ISO 27001 Lead Implementer Information Security Management, Security Governance, Secure Software Development Goebel Consult, Landshut http://www.goebel-consult.de Blog: http://www.go

Re: [PATCH] gc: Add option --keep-going.

2016-11-24 Thread Hartmut Goebel
sg(lvlError, format("deleting garbage...")); else printMsg(lvlError, format("determining live/dead paths...")); So, what do you think? -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thought are impossible |

Re: [herd] Have status display services as a bulleted list.

2016-11-24 Thread Hartmut Goebel
Am 23.11.2016 um 22:03 schrieb Christopher Allan Webber: > Whew! I can read that finally. And grep it too! +1 -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thought are impossible |

Re: [Patch v2 2/2] Add system start-up files for "guix publish".

2016-11-24 Thread Hartmut Goebel
Am 23.11.2016 um 22:25 schrieb Ludovic Courtès: > OK with these changes, thank you! Pushed with these changes. -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thought are impossible |

Re: [PATCH] improve nginx-service

2016-11-22 Thread Hartmut Goebel
(option (type 'server_name) ;; taken from "nginx" scope (value (list 'default))) -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thought are impossible |

Re: [Patch v2] daemon: Set ownership of kept build directories to the calling user.

2016-11-21 Thread Hartmut Goebel
' exactly this way (via Settings). Which means if passing clientUid/Gid via constructors, we would have two different ways of passing information used in the same small function - which would be inconsistent. Thus said, I'm in strong favour of using Settings. -- Regards Hartmut Goebel | Hartmut

Re: [PATCH 1/7] gnu: Add KDE frameworkintegration.

2016-11-21 Thread Hartmut Goebel
ist of licenses is taken from Debian, which has a detailed lists of all the licenses used in each package. And KDE often has a *fast* number of different licenses in one package. -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.

[Patch v2 1/2] doc: Symlink daemon start-up files.

2016-11-21 Thread Hartmut Goebel
This patch ensures that always the current profile's daemon will be used. The .service file contains the path to the guix-daemon within the store. Thus when copying the file, it will point to the very version of guix-daemon used at the time of copying – even after system upgrade or when this

Re: [Patch v2] daemon: Set ownership of kept build directories to the calling user.

2016-11-21 Thread Hartmut Goebel
ill already have owner and group from ‘getuid’ and ‘getgid’, so there is no need to change them. Or did I miss something? -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thought are impossible |

[Patch v2 2/2] Add system start-up files for "guix publish".

2016-11-21 Thread Hartmut Goebel
* .gitignore: add etc/guix-publish.conf and /etc/guix-publish.service. * etc/guix-publish.conf.in: New file. * etc/guix-publish.service.in: New file. * nix/local.mk (etc/guix-%.service, etc/guix-%.conf): Generalized former build-rules for by using patterns. (nodist_systemdservice_DATA): Add

[Patch v2 0/2] Symlink daemon start-up files and guix publish

2016-11-21 Thread Hartmut Goebel
to provide more public stores - which at least could be used for "guix challenge". *** BLURB HERE *** Hartmut Goebel (2): doc: Symlink daemon start-up files. Add system start-up files for "guix publish". .gitignore | 2 ++ doc/guix.texi

Re: [PATCH 2/2] Add system start-up files for "guix publish".

2016-11-21 Thread Hartmut Goebel
to add a cross-reference to “Invoking > guix archive” where we mention that a key needs to be generated. Invoking guix publish is the perfect place: There already is a paragraph about enabling the service on GuixSD and also a reference to guix archive. Thanks for pointing there. -- Rega

Re: [PATCH 1/2] doc: Symlink daemon start-up files.

2016-11-21 Thread Hartmut Goebel
Am 18.11.2016 um 21:31 schrieb Leo Famulari: > Does systemd allow symlinked unit files (aka service definitions)? I > read this old discussion that says "No" [0] but it could have changed by > now [1]. Yes it does. -- Regards Hartmut Goebel | Hartmut Goebel

Re: [PATCH] gnu: shared-mime-info: Update to 1.7

2016-11-21 Thread Hartmut Goebel
Am 21.11.2016 um 15:02 schrieb Ludovic Courtès: > >> * gnu/packages/gnome.scm (shared-mime-info): Update to 1.7. > OK for ‘staging’! Pushed to staging as 856f30b0009dbb65446de7d62f0497e31eedd88c -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-c

[PATCH 5/7] gnu: Add kjsembed.

2016-11-21 Thread Hartmut Goebel
* gnu/packages/kde-frameworks.scm (kjsembed): New variable. --- gnu/packages/kde-frameworks.scm | 29 + 1 file changed, 29 insertions(+) diff --git a/gnu/packages/kde-frameworks.scm b/gnu/packages/kde-frameworks.scm index 928b7ec..109d3f2 100644 ---

[PATCH 1/7] gnu: Add KDE frameworkintegration.

2016-11-21 Thread Hartmut Goebel
* gnu/packages/kde-frameworks.scm (frameworkintegration): New variable. --- gnu/packages/kde-frameworks.scm | 51 + 1 file changed, 51 insertions(+) diff --git a/gnu/packages/kde-frameworks.scm b/gnu/packages/kde-frameworks.scm index 9df37ac..0e6b8fb

[PATCH 2/7] gnu: Add kdelibs4support.

2016-11-21 Thread Hartmut Goebel
* gnu/packages/kde-frameworks.scm (kdelibs4support): New variable. --- gnu/packages/kde-frameworks.scm | 88 + 1 file changed, 88 insertions(+) diff --git a/gnu/packages/kde-frameworks.scm b/gnu/packages/kde-frameworks.scm index 0e6b8fb..a537aa4 100644 ---

[PATCH 6/7] gnu: Add kmediaplayer.

2016-11-21 Thread Hartmut Goebel
* gnu/packages/kde-frameworks.scm (kmediaplayer): New variable. --- gnu/packages/kde-frameworks.scm | 46 + 1 file changed, 46 insertions(+) diff --git a/gnu/packages/kde-frameworks.scm b/gnu/packages/kde-frameworks.scm index 109d3f2..c86d82c 100644 ---

[PATCH 7/7] gnu: Add kross.

2016-11-21 Thread Hartmut Goebel
* gnu/packages/kde-frameworks.scm (kross): New variable. --- gnu/packages/kde-frameworks.scm | 43 + 1 file changed, 43 insertions(+) diff --git a/gnu/packages/kde-frameworks.scm b/gnu/packages/kde-frameworks.scm index c86d82c..88824e2 100644 ---

[PATCH 3/7] gnu: Add khtml.

2016-11-21 Thread Hartmut Goebel
* gnu/packages/kde-frameworks.scm (khtml): New variable. --- gnu/packages/kde-frameworks.scm | 57 + 1 file changed, 57 insertions(+) diff --git a/gnu/packages/kde-frameworks.scm b/gnu/packages/kde-frameworks.scm index a537aa4..07c6716 100644 ---

[PATCH 0/7] KDE Framework: Tier 4 and porting aids

2016-11-21 Thread Hartmut Goebel
This patch series complets the KDE Frameworks 5: Tier 4 and porting aids. See https://api.kde.org/frameworks/ for a list of all framework components. Hartmut Goebel (7): gnu: Add KDE frameworkintegration. gnu: Add kdelibs4support. gnu: Add khtml. gnu: Add kjs. gnu: Add kjsembed. gnu

[PATCH 4/7] gnu: Add kjs.

2016-11-21 Thread Hartmut Goebel
* gnu/packages/kde-frameworks.scm (kjs): New variable. --- gnu/packages/kde-frameworks.scm | 32 1 file changed, 32 insertions(+) diff --git a/gnu/packages/kde-frameworks.scm b/gnu/packages/kde-frameworks.scm index 07c6716..928b7ec 100644 ---

Re: [Patch v2] daemon: Set ownership of kept build directories to the calling user.

2016-11-21 Thread Hartmut Goebel
is why I've put these here, too. But if you prefer passing them around, I'll try to implement this. -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thought are impossible |

[PATCH] gnu: shared-mime-info: Update to 1.7

2016-11-20 Thread Hartmut Goebel
* gnu/packages/gnome.scm (shared-mime-info): Update to 1.7. --- gnu/packages/gnome.scm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/gnu/packages/gnome.scm b/gnu/packages/gnome.scm index 3aa0f56..8b94105 100644 --- a/gnu/packages/gnome.scm +++ b/gnu/packages/gnome.scm @@

[PATCH 1/2] doc: Symlink daemon start-up files.

2016-11-18 Thread Hartmut Goebel
This patch ensures that always the current profile's daemon will be used. The .service file contains the path to the guix-daemon within the store. Thus when copying the file, it will point to the very version of guix-daemon used at the time of copying – even after system upgrade or when this

[PATCH 2/2] Add system start-up files for "guix publish".

2016-11-18 Thread Hartmut Goebel
* .gitignore: add etc/guix-publish.conf and /etc/guix-publish.service. * etc/guix-publish.conf.in: New file. * etc/guix-publish.service.in: New file. * nix/local.mk (etc/guix-%.service, etc/guix-%.conf): Generalized former build-rules for by using patterns. (nodist_systemdservice_DATA): Add

[PATCH 0/2]

2016-11-18 Thread Hartmut Goebel
" service, just for convenience for those who want it. Primary this is meant for use in the local network. Hopefully this will encourage people to provide more public stores - which at least could be used for "guix Hartmut Goebel (2): doc: Symlink daemon start-up files.

Re: [PATCH 0/3] Add a generalized `wrap-language-programs`

2016-11-18 Thread Hartmut Goebel
What about consolidating the efforts and avoid rebuilding the world three times? (How long will it take? The Lord took seven days for one world :-) -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you t

Re: [PATCH 0/3] Add a generalized `wrap-language-programs`

2016-11-17 Thread Hartmut Goebel
Am 17.11.2016 um 18:09 schrieb Alex Sassmannshausen: > In both cases, the programs work fine with the generalized patch. Fine :-) -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thou

[Patch v2] daemon: Set ownership of kept build directories to the calling user.

2016-11-17 Thread Hartmut Goebel
Fixes . * nix/libstore/globals.hh (Settings) Add clientUid and clientGid. * nix/nix-daemon/nix-daemon.cc (pdaemonLoop] Store UID and GID of the caller in settings. * nix/libstore/build.cc (_chown): New function. (DerivationGoal::deleteTmpDir): Use it, change

Re: [PATCH] Add php

2016-11-17 Thread Hartmut Goebel
adding "Packaging guideline" for "PHP Modules" section to the manual? * Do yo think we need some php build system? -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thought are impossible |

[PATCH] gc: Add option --keep-going.

2016-11-16 Thread Hartmut Goebel
* guix/scripts/gc.scm (show-help, %options): Add option -k/--keep-going. (guix-gc): Pass value off option --keep-going on to delete-paths. * guix/store.scm (%protocol-version): Increment to 17. (delete-paths) New key-word parameter `keep-going?`, pass it on to run-rc. (run-gc): New

[PATCH] daemon: Set ownership of kept build directories to the calling user.

2016-11-16 Thread Hartmut Goebel
/store.scm @@ -1,5 +1,6 @@ ;;; GNU Guix --- Functional package management for GNU ;;; Copyright © 2012, 2013, 2014, 2015, 2016 Ludovic Courtès <l...@gnu.org> +;;; Copyright © 2016 Hartmut Goebel <h.goe...@crazy-compilers.com> ;;; ;;; This file is part of GNU Guix. ;;; @@ -

Re: [PATCH] guxi: cmake-build-system: Enable output for failing test-cases.

2016-11-15 Thread Hartmut Goebel
Am 15.11.2016 um 22:13 schrieb Leo Famulari: > Hartmut, can you push this to the recently created "staging" branch? Done. -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thought are impossible |

Re: Please review: documentation for python build system

2016-11-15 Thread Hartmut Goebel
Am 15.11.2016 um 09:24 schrieb Chris Marusich: > My comments below are primarily grammatical or stylistic. I hope you > find my feedback helpful. Thank you very much. That's exactly the kind of feedback I also wanted. -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...

Please review: documentation for python build system

2016-11-13 Thread Hartmut Goebel
info" and "install_scripts". These sub-commands are executed even if "--single-version-externally-managed" is set, thus the .egg-info directory and the scripts defined in entry-points will always be created. Note: Even if the "easy-install.pth" is not longer created, we kept this phase. There still may be packages creating an "easy-install.pth" manually for some good reason. -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thought are impossible |

Re: [PATCH] build-system/perl: Add wrap phase.

2016-11-13 Thread Hartmut Goebel
t rebase on top of the new master. Wip-* > branches indicate that history can be broken, in my opinion. > > It's up to you. I'll rebase on top of master later this week. -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.c

Re: [PATCH] build-system/perl: Add wrap phase.

2016-11-13 Thread Hartmut Goebel
egards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thought are impossible |

Re: [PATCH 1/4] gnu: Separate util-linux into three packages.

2016-11-11 Thread Hartmut Goebel
her two. Sorry for not being precise enough. I meant why are us using "inherit" here? I thought (source (package-source util-linux/base)) would suffice and result in the same? -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thought are impossible |

Re: [PATCH 1/4] gnu: Separate util-linux into three packages.

2016-11-11 Thread Hartmut Goebel
> + (package > +(inherit util-linux/base) > +(name "util-linux") > +(source (origin (inherit (package-source util-linux/base I'm curious: Why do you need to inherit the package-source here? -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@craz

Re: [PATCH 1/3] gnu: connman: Use localstatedir outside of store.

2016-11-11 Thread Hartmut Goebel
Am 10.11.2016 um 08:55 schrieb Efraim Flashner: > > + ;; location for daemon state files and internal configuration > > + ;; needs to be writeable I wonder if this shouldn't be the default for gnu-build-system? -- Regards Hartmut Goebel | Ha

Re: [Patch v2 3/3] gnu: Add pdfposter.

2016-11-10 Thread Hartmut Goebel
Am 10.11.2016 um 11:48 schrieb Ludovic Courtès: > LGTM with these changes. I pushed the patch series with all your requests handled. Thanks for reviewing and expeling my errors :-) -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.cr

Re: [Patch v2 1/3] gnu: Add python-pypdf2.

2016-11-10 Thread Hartmut Goebel
Am 10.11.2016 um 13:35 schrieb Ludovic Courtès: > Hmm, OK. That doesn’t make it sounds like it’s really safe to use with > Python 3. You challenged me :-) I fixed the test-suite using a simple change which opens a file in binary mode. -- Regards Hartmut Goebel | Hartmut

Re: [Patch v2 1/3] gnu: Add python-pypdf2.

2016-11-10 Thread Hartmut Goebel
e old pyPdf does not. So we need Python 3. The test suite does not fail in the sense that it shows bugs, but in a way that shows the test suite was not adopted to work with Python 3. The test-suite is negligible anyway. I'll update the comment. -- Regards Hartmut Goebel | Hartmut Goebel

[Patch v2 3/3] gnu: Add pdfposter.

2016-11-09 Thread Hartmut Goebel
* gnu/packages/pdf.scm(pdfposter): New variable. --- gnu/packages/pdf.scm | 32 1 file changed, 32 insertions(+) diff --git a/gnu/packages/pdf.scm b/gnu/packages/pdf.scm index e0279c6..06cfab9 100644 --- a/gnu/packages/pdf.scm +++ b/gnu/packages/pdf.scm @@ -851,3

[Patch v2 2/3] gnu: Add python-pypdf.

2016-11-09 Thread Hartmut Goebel
* gnu/packages/pdf.scm(python2-pypdf): New variable. --- gnu/packages/pdf.scm | 22 ++ 1 file changed, 22 insertions(+) diff --git a/gnu/packages/pdf.scm b/gnu/packages/pdf.scm index e243006..e0279c6 100644 --- a/gnu/packages/pdf.scm +++ b/gnu/packages/pdf.scm @@ -829,3

[Patch v2 1/3] gnu: Add python-pypdf2.

2016-11-09 Thread Hartmut Goebel
* gnu/packages/pdf.scm(python-pypdf2, python2-pypdf2): New variables. --- gnu/packages/pdf.scm | 45 + 1 file changed, 45 insertions(+) diff --git a/gnu/packages/pdf.scm b/gnu/packages/pdf.scm index ee33e3c..e243006 100644 --- a/gnu/packages/pdf.scm

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

2016-11-09 Thread Hartmut Goebel
/pdfposter/tree/develop/test These are only for visual control, not a test-suite. -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thought are impossible |

Re: [PATCH 2/3] gnu: Add python-pypdf.

2016-11-09 Thread Hartmut Goebel
Am 09.11.2016 um 08:19 schrieb Efraim Flashner: > One of the things I've picked up is that this will work with the pypi > importer if you feed it "pyPdf" Fixed. I tries some combinations but obviously missed the obvious one :-) -- Regards Hartmut Goebel | Hartmut Goebel

Re: [PATCH 2/3] gnu: Add python-pypdf.

2016-11-09 Thread Hartmut Goebel
Am 09.11.2016 um 00:57 schrieb Leo Famulari: >> > +(home-page "http://mstamy2.github.com/PyPDF2;) > Also, does it have its own home-page? Well spotted :-) -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.cr

Re: [PATCH 1/3] gnu: Add python-pypdf2.

2016-11-09 Thread Hartmut Goebel
hyphen in "Pure-Python" with a space. > We have to use enumerate instead of itemize due to a limitation in Done. I'll send updates patches since again it's late at night :-) -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.cra

[PATCH 3/3] gnu: Add pdfposter.

2016-11-08 Thread Hartmut Goebel
* gnu/packages/pdf.scm(pdfposter): New variable. --- gnu/packages/pdf.scm | 35 +++ 1 file changed, 35 insertions(+) diff --git a/gnu/packages/pdf.scm b/gnu/packages/pdf.scm index ca3f830..eed37d6 100644 --- a/gnu/packages/pdf.scm +++ b/gnu/packages/pdf.scm @@

[PATCH 2/3] gnu: Add python-pypdf.

2016-11-08 Thread Hartmut Goebel
* gnu/packages/pdf.scm(python2-pypdf): New variable. --- gnu/packages/pdf.scm | 34 ++ 1 file changed, 34 insertions(+) diff --git a/gnu/packages/pdf.scm b/gnu/packages/pdf.scm index 558ea89..ca3f830 100644 --- a/gnu/packages/pdf.scm +++ b/gnu/packages/pdf.scm @@

Re: [PATCH]: gnu: python-jsonschema: Correct inputs.

2016-11-08 Thread Hartmut Goebel
puts]: Move >> > python2-functools32 to ... >> > [propagated-inputs]: ... here. >> > [native-inputs]: Add python2-setuptools. > It works for me, thanks! > > Hartmut, what do you think? LGTM -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compi

Re: [PATCH v2] gnu: sqlalchemy-utils: Update description to include optional dependencies.

2016-11-06 Thread Hartmut Goebel
@item @code{python-psycopg2} > +@item @code{python-furl} > +@item @code{python-flask-babel} > +@end enumerate > +") > (properties `((python2-variant . ,(delay python2-sqlalchemy-utils > (license license:bsd-3))) > > LGTM. -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thought are impossible |

Re: Phase `validate-runpath' fails: lib not in RUNPATH

2016-11-06 Thread Hartmut Goebel
Am 06.11.2016 um 20:28 schrieb Ricardo Wurmus: > Hartmut Goebel <h.goe...@crazy-compilers.com> writes: > >> >> /gnu/store/…-thunderbird-45.4.0/lib/firefox-45.4.1/browser/components/libbrowsercomps.so: >> error: depends on 'libplds4.so', which cannot be found

Phase `validate-runpath' fails: lib not in RUNPATH

2016-11-06 Thread Hartmut Goebel
in RUNPATH /gnu/store/…-thunderbird-45.4.0/lib/firefox-devel-45.4.1/sdk/lib/libxul.so: error: depends on 'libnspr4.so', which cannot be found in RUNPATH Any hint what this can be caused by? -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com

Re: Building many packages in order

2016-11-06 Thread Hartmut Goebel
e?” as a predicate, but you > could use something else in its place (e.g. a procedure testing the > package name). Thanks for sharing this. Since I'm not a guile hacker, I'll stay with my bash-script, even if it is not that performant :-) -- Regards Hartmut Goebel | Hartmut G

Re: Shutdown of googlecode end of the year - we have affected packages!

2016-11-06 Thread Hartmut Goebel
Hi, I've done this one: > > gnu/packages/java.scm:1183: (uri (string-append > > "https://hamcrest.googlecode.com/files/; Update source uri and checksum. [snippet]: Remove directory "lib" completely. -- Regards Hartmut Goebel | Hartmut Goebe

Re: [PATCH 1/6] gnu: Add fatfsck-static.

2016-11-05 Thread Hartmut Goebel
Am 05.11.2016 um 13:55 schrieb Marius Bakke: > +(define-public fatfsck/static > + (package > +(name "fatfsck-static") What is the reason for using a different name in "define"? -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.co

Re: [PATCH] improve nginx-service

2016-11-04 Thread Hartmut Goebel
ginx-vhost-configuration (root taler-landing-page)) > What is your configuration exactly, and what is taler-landing-page? > The configuration is like in your gtk-doc-vhost, except that I'm not using gtk+2 and omitting the "string-append" for the path. -- Regards Hartmut Goebel | Ha

Question regarding record types

2016-11-04 Thread Hartmut Goebel
would remove this redundancy. -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thought are impossible |

Re: [PATCH] improve nginx-service

2016-11-04 Thread Hartmut Goebel
rver_tokens " #$(if (nginx-vhost-configuration-server-tokens? > vhost) + "on" "off") ";\n" Could you please (maybe in another patch) add a way to include additional config lines? Both for the main server and each vhost.I'm gioing to need this for adding locations, backends and such. -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thought are impossible |

Re: [PATCH] mps-youtube, revision 3

2016-11-04 Thread Hartmut Goebel
Am 04.11.2016 um 20:23 schrieb ng0: > [PATCH 1/2] gnu: Add python-pafy. > > - made youtube-dl propagated-input as suggested by harmut > > [PATCH 2/2] gnu: Add mps-youtube. > > - no changes > Both LGTM. Adding the comment about youtube_dl was a good idea :-) -- Regards H

Re: [PATCH 1/2] gnu: Add python-pafy.

2016-11-04 Thread Hartmut Goebel
Am 04.11.2016 um 19:18 schrieb ng0: > Hartmut Goebel <h.goe...@crazy-compilers.com> writes: > >> Am 02.11.2016 um 11:40 schrieb ng0: >>> +(inputs >>> + `(("youtube-dl" ,youtube-dl))) >> This needs to be a propagated input since python-paf

Re: [PATCH 14/16] gnu: Add python-jupyter-console.

2016-11-04 Thread Hartmut Goebel
le-based interaction with > non-Python Jupyter kernels such as IJulia and IRKernel." LGTM -- Schönen Gruß Hartmut Goebel Dipl.-Informatiker (univ), CISSP, CSSLP, ISO 27001 Lead Implementer Information Security Management, Security Governance, Secure Software Development Goebel Consult

Re: [PATCH] improve nginx-service

2016-11-03 Thread Hartmut Goebel
.scm:702:27: In procedure map: ice-9/boot-9.scm:702:27: In procedure string-append: Wrong type (expecting string): # I tried with current master branch. -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thought are impossible |

Re: [PATCH 6/6] gnu: Enable passlib, babel, cryptography, pyt, psycopg2, furl, flask-babel and pytest in python-sqlalchemy-utils

2016-11-03 Thread Hartmut Goebel
t; feature add a single package. No complex stuff here. But alone babel adds 21 MB, cryptography another ca. 70 MB. I'm in favor of *not* adding any of these additional inputs. -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-com

Re: [PATCH 5/6] gnu: Add python-flask-babel.

2016-11-03 Thread Hartmut Goebel
ocalization" in the description since the abreviations are already used in the synopsis. -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thought are impossible |

Re: [PATCH 4/6] gnu: Add python-furl.

2016-11-03 Thread Hartmut Goebel
ns, but using these functions to perform common URL manipulations proves tedious. Furl makes manipulating URLs easy. -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thought are impossible |

Re: [PATCH 3/6] gnu: Add python-orderedmultidict.

2016-11-03 Thread Hartmut Goebel
An ordered multivalue dictionary is a multivalue dictionary that retains the order of insertions and deletions. -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thought are impossible |

Re: [PATCH 2/6] gnu: Add python-pycodestyle.

2016-11-03 Thread Hartmut Goebel
quot;pycodestyle (formerly pep8) is a tool to check your Python code against some of the style conventions in PEP 8 <http://www.python.org/dev/peps/pep-0008/>." Otherwise LGTM -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-c

Re: [PATCH 1/6] gnu: Enable python-psycopg2 support in python-sqlalchemy.

2016-11-03 Thread Hartmut Goebel
e for installing the required package. If the system is not using postgresql, this is just useless. And if OTOH we'd include all possible backends we'd end up with a huge package. Or did I miss something? -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com

Re: [PATCH 0/2] Add new pytest for special cases

2016-11-02 Thread Hartmut Goebel
me that I wanted to add some more comments and explanasions to python-build-system.scm) -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thought are impossible |

Re: [PATCH 1/2] gnu: Add python-pafy.

2016-11-02 Thread Hartmut Goebel
Am 02.11.2016 um 11:40 schrieb ng0: > +(inputs > + `(("youtube-dl" ,youtube-dl))) This needs to be a propagated input since python-pafy is a package itself. -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.cra

Re: [PATCH 05/16] gnu: Add python-testpath.

2016-11-02 Thread Hartmut Goebel
n build system exports a function "site-packages" which does exactly this. -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thought are impossible |

Re: [PATCH 09/16] gnu: Add python-entrypoints.

2016-11-02 Thread Hartmut Goebel
ot; ,version))) > + (install-file "entrypoints.py" target) > + #t)) Instread of this I recommend adding a minimal setup.py and let the build-system do the job. This would be more future proof. -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thought are impossible |

Re: [PATCH] improve nginx-service

2016-11-02 Thread Hartmut Goebel
e into a package. How can I use that package's directory as "root"? I tried these, but none worked: (root (package-file demo-website)) (root #$(file-append demo-website "/")) (root (assoc-ref demo-website "out")) (root #$demo-website) -- Regards Har

Re: [PATCH 07/16] gnu: Add python-nbformat.

2016-11-02 Thread Hartmut Goebel
("python-setuptools" ,python-setuptools) Why is setuptools required at run-time? This question also belongs to the other packages where setuptools are a propagated-input. -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thought are impossible |

Re: [PATCH v2 3/3] gnu: Add python2-wxpython.

2016-10-30 Thread Hartmut Goebel
which I assume to be *not* used at run-time and thus needs to be a native input. -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thought are impossible |

Re: [PATCH] gnu: Add python2-rpython.

2016-10-29 Thread Hartmut Goebel
sion > + ".tar.gz")) Please try to use "pypi-url" here like. > +(propagated-inputs > + `(("python2-setuptools" ,python2-setuptools))) Please add a comment why does this have to be propagated. -- Regards Hartmut Goebel | Hartmut Goebel

Re: [PATCH] gnu: Add python2-pathlib2.

2016-10-27 Thread Hartmut Goebel
Am 26.10.2016 um 14:46 schrieb Efraim Flashner: > This looks good to me. I'd add a comment near the arguments like the one > at the top as to why its python2- only. Thanks for the suggestion. I added it. -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compile

Re: Note for Python packages: packages do not have "inputs" (most of the time)

2016-10-25 Thread Hartmut Goebel
3.4.egg $ ~/.guix-profile/bin/python3 -c "import werkzeug ; print(werkzeug.__file__)" /gnu/store/0r2wdiciw1…-python-werkzeug-0.11.5/lib/python3.4/site-packages/Werkzeug-0.11.5-py3.4.egg/werkzeug/__init__.py [1] https://docs.python.org/3/library/site.html -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thought are impossible |

Re: [PATCH 2/3] gnu: Add python2-wxpython.

2016-10-25 Thread Hartmut Goebel
uot;build"))) Please set argument #:make-flags instread of calling setup.yp yourself. This will break with the upcoming change of the python build system. -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thought are impossible |

Re: [PATCH 1/1] gnu: twolame: Fix test failure with Perl 5.24.

2016-10-24 Thread Hartmut Goebel
;)) > + #t I'm curious: Are we really removing the tests from the sources "just" because they are failing? -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thought are impossible |

Re: [PATCH 0/4] Add some Flask modules

2016-10-23 Thread Hartmut Goebel
Am 20.10.2016 um 23:45 schrieb Danny Milosavljevic: > Danny Milosavljevic (4): > gnu: Add python-aniso8601. > gnu: Add python-flask-restful. > gnu: Add python-flask-basicauth. > gnu: Add python-flask-sqlalchemy. Applied with minor corrections. Thanks. -- Schönen Gruß Hart

Re: [PATCH] import: pypi: All inputs are propagated-inputs by default.

2016-10-23 Thread Hartmut Goebel
Am 17.10.2016 um 11:00 schrieb Hartmut Goebel: > * guix/import/pypi.scm (maybe-inputs): Return inputs as > "propagated-inputs". Applied as b5c347ad3d83ee580c111bd14c80b469b0dcb294 -- Schönen Gruß Hartmut Goebel Dipl.-Informatiker (univ), CISSP, CSSLP, ISO 27001 Lead Impleme

Note for Python packages: packages do not have "inputs" (most of the time)

2016-10-23 Thread Hartmut Goebel
cleaning up all these wrong inputs :-(( -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thought are impossible |

Re: Building many packages in order

2016-10-23 Thread Hartmut Goebel
ep that log-file after the build for failed build. -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thought are impossible |

Re: Building many packages in order

2016-10-21 Thread Hartmut Goebel
" and my code for selecting the packages is much more complicated. (I select them by name.) I really have to get more common with these guix subcommands. [1] http://unix.stackexchange.com/questions/120642/what-defines-the-maximum-size-for-a-command-single-argument -- Regards Hartmut Goebel |

Re: Building many packages in order

2016-10-21 Thread Hartmut Goebel
since I would need to know the correct variable name. -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thought are impossible |

Building many packages in order

2016-10-21 Thread Hartmut Goebel
to the number of arguments xargs passes to the command is restricted). Thus only partitions of the packages are build in order. So I need some way for passing *all* packages to be build to `guix build`. Any hints? -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com

Re: [PATCH 1/2] gnu: Add python-setproctitle and python2-setproctitle.

2016-10-20 Thread Hartmut Goebel
not the point. https://www.gnu.org/software/guile/manual/html_node/and-or.html#and-or says: "The value of the last evaluated expression is returned." If it wouldn't, the code would not have worked here :-) Thanks for the review, nevertheless:-) -- Regards Hartmut Goebel |

Re: [PATCH] gnu: Add chromaprint.

2016-10-20 Thread Hartmut Goebel
on many compiler options. Debian based distributions don't even ship a compiled package of gtest anymore. IMHO gtest should get onto the list of "software to note use". [1]https://bitbucket.org/acoustid/chromaprint/issues/19/build-failure-googletest-sources-are -- Regard

Re: [PATCH 2/2] gnu: Add python-axolotl.

2016-10-19 Thread Hartmut Goebel
port or just the "libaxolotl-android"? In any case, persons should not be glamorized in descriptions. We also do not write "PGP originally developed by Phill Zimmerman". This only leads to personality cult - and esp. this person is very debatable. -- Regards Hartmut Goebel | Ha

Re: [PATCH] guxi: cmake-build-system: Enable output for failing test-cases.

2016-10-18 Thread Hartmut Goebel
not want to tough it. But we could already push some changes to core-updates next to get them off our todo-lists. But maybe this is just what you call "staging" branch :-) -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.craz

Re: [PATCH 1/1] gnu: Add python-betamax.

2016-10-17 Thread Hartmut Goebel
me recherché. -- Regards Hartmut Goebel | Hartmut Goebel | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thought are impossible | 0xBF773B65.asc Description: application/pgp-keys

<    2   3   4   5   6   7   8   9   10   11   >