[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] import: Add PyPI importer.
From: |
Ludovic Courtès |
Subject: |
Re: [PATCH] import: Add PyPI importer. |
Date: |
Mon, 29 Sep 2014 14:24:57 +0200 |
User-agent: |
Gnus/5.130011 (Ma Gnus v0.11) Emacs/24.3 (gnu/linux) |
David Thompson <address@hidden> skribis:
> Ludovic Courtès <address@hidden> writes:
>
>> David Thompson <address@hidden> 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 to decide whether or not to run those tests that require
>> guile-json.
>
> 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_guile_json], [(json)])
AM_CONDITIONAL([HAVE_GUILE_JSON], [text "x$have_guile_json" = "xyes"])
and in Makefile.am:
if HAVE_GUILE_JSON
TESTS += tests/pypi.scm
endif
Ludo’.
- [PATCH] import: Add PyPI importer., David Thompson, 2014/09/27
- Re: [PATCH] import: Add PyPI importer., David Thompson, 2014/09/27
- Re: [PATCH] import: Add PyPI importer., Ludovic Courtès, 2014/09/27
- Re: [PATCH] import: Add PyPI importer., David Thompson, 2014/09/29
- Re: [PATCH] import: Add PyPI importer.,
Ludovic Courtès <=
- Re: [PATCH] import: Add PyPI importer., David Thompson, 2014/09/29
- Re: [PATCH] import: Add PyPI importer., Alex Kost, 2014/09/30
- Re: [PATCH] import: Add PyPI importer., Ludovic Courtès, 2014/09/30
- Re: [PATCH] import: Add PyPI importer., David Thompson, 2014/09/30
Re: [PATCH] import: Add PyPI importer., Ludovic Courtès, 2014/09/27