Re: [PATCH] import: Add PyPI importer.

[ludo@gnu.org (Ludovic Courtès)]

David Thompson <dthompson2@worcester.edu> skribis:

> Ludovic Courtès <ludo@gnu.org> writes:
>
>> David Thompson <dthompson2@worcester.edu> 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’.