On Mon, 1 Mar 2010 16:24:30 +0100, Sverre Rabbelier <srabbelier@xxxxxxxxx>
wrote:
> Heya,
>
> On Mon, Mar 1, 2010 at 16:15, Tay Ray Chuan <rctay89@xxxxxxxxx> wrote:
>> (adding Sverre to the Cc list, perhaps he has something to add)
>
> This is Johan's code, but IIRC setup.py is pretty much generic
setuptools
> code.
>
>> you need to install setuptools from python, or pass NO_PYTHON to
>> make/configure.
>
> Correct.
>
>>> I've checked the README and INSTALL file, but I don't see that there
>>> are new
>>> requirements? (That is not regarding doc.)
>>
>> I guess we should improve on this.
>
> Aye, at the very least we need to document this requirement, but it
> would probably be even better to test for the existence of setuptools
> and give a useful error message if it's not found?
Something like the following in the Makefile perhaps?
Python_distutils = $(shell sh -c "$(PYTHON_PATH) -c 'import distutils'
2>/dev/null || echo not")
ifeq ($(Python_distutils),not)
$(warning "Disabling Python: please install setuptools to
enable")
NO_PYTHON=SadlyYes
endif
--
Julian
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html