Re: [PATCH] import: Add PyPI importer.

2014-09-30 Thread David Thompson
Alex Kost writes: > Hello, I don't have guile-json and "make" has failed for me because > (guix import pypi) wants json module. Is it required now? > > Perhaps it would be safe (?) to adjust “Makefile.am” like this: > Sorry about that. Thanks for fixing it! -- David Thompson Web Developer -

Re: [PATCH] import: Add PyPI importer.

2014-09-30 Thread Ludovic Courtès
Alex Kost skribis: > David Thompson (2014-09-30 03:31 +0400) wrote: > >> Ludovic Courtès writes: >> >>> David Thompson skribis: >>> Oh yeah, almost forgot about the Automake rule! My autotools skills are weak so I'm not sure how to write this. Could you point me in the right di

Re: [PATCH] import: Add PyPI importer.

2014-09-29 Thread Alex Kost
David Thompson (2014-09-30 03:31 +0400) wrote: > Ludovic Courtès writes: > >> David Thompson skribis: >> >>> Oh yeah, almost forgot about the Automake rule! My autotools skills are >>> weak so I'm not sure how to write this. Could you point me in the right >>> direction? >> >> In configure.ac,

Re: [PATCH] import: Add PyPI importer.

2014-09-29 Thread David Thompson
Ludovic Courtès writes: > David Thompson skribis: > >> Oh yeah, almost forgot about the Automake rule! My autotools skills are >> weak so I'm not sure how to write this. Could you point me in the right >> direction? > > In configure.ac, do something like: > > GUILE_MODULE_AVAILABLE([have_gui

Re: [PATCH] import: Add PyPI importer.

2014-09-29 Thread Ludovic Courtès
David Thompson skribis: > Ludovic Courtès writes: > >> David Thompson skribis: >> >>> I should mention that the PyPI import requires the guile-json library. >>> How should this be handled in our build scripts? >> >> I think it’s fine to install it whether or not guile-json is available. >> Now

Re: [PATCH] import: Add PyPI importer.

2014-09-29 Thread David Thompson
Ludovic Courtès writes: > David Thompson skribis: > >> I should mention that the PyPI import requires the guile-json library. >> How should this be handled in our build scripts? > > I think it’s fine to install it whether or not guile-json is available. > Now, when there are tests ;-), there’ll

Re: [PATCH] import: Add PyPI importer.

2014-09-29 Thread Ludovic Courtès
David Thompson skribis: > Ludovic Courtès writes: > >> I wonder if there may be shared options between all the importers >> (like an option for import & live build.) >> >> That can still be addressed by exporting an option list from (guix >> scripts import), like (guix scripts build) does, I thi

Re: [PATCH] import: Add PyPI importer.

2014-09-28 Thread David Thompson
d?). > 'factorize-uri' is better, so I'm using that now. I created a new module '(guix import utils)' for commonly used import procedures. New patch attached. >From 0696b70fd31630791b5125625a3a15ea0dfdaf1d Mon Sep 17 00:00:00 2001 From: David Thompson Date: Sat, 27 Sep

Re: [PATCH] import: Add PyPI importer.

2014-09-27 Thread Ludovic Courtès
looks good to me. Some comments below: > From b3ec259fd097034631cf311040af7aa12f7c5ebc Mon Sep 17 00:00:00 2001 > From: David Thompson > Date: Sat, 27 Sep 2014 10:16:23 -0400 > Subject: [PATCH] import: Add PyPI importer. > > * guix/snix.scm: Delete. > * guix/import/snix.scm: N

Re: [PATCH] import: Add PyPI importer.

2014-09-27 Thread Ludovic Courtès
David Thompson skribis: > I should mention that the PyPI import requires the guile-json library. > How should this be handled in our build scripts? I think it’s fine to install it whether or not guile-json is available. Now, when there are tests ;-), there’ll have to be an Automake conditional

Re: [PATCH] import: Add PyPI importer.

2014-09-27 Thread David Thompson
I should mention that the PyPI import requires the guile-json library. How should this be handled in our build scripts? -- David Thompson Web Developer - Free Software Foundation - http://fsf.org GPG Key: 0FF1D807 Support the FSF: https://fsf.org/donate

[PATCH] import: Add PyPI importer.

2014-09-27 Thread David Thompson
view. :) >From b3ec259fd097034631cf311040af7aa12f7c5ebc Mon Sep 17 00:00:00 2001 From: David Thompson Date: Sat, 27 Sep 2014 10:16:23 -0400 Subject: [PATCH] import: Add PyPI importer. * guix/snix.scm: Delete. * guix/import/snix.scm: New file. * guix/scripts/import/nix.scm: New file. * guix/import/pypi.scm: New file. * g