Messages in this thread | | | Date | Tue, 29 Mar 2011 17:40:25 -0300 | From | Arnaldo Carvalho de Melo <> | Subject | Re: [PATCH] Use the environment variable PYTHON if defined |
| |
Em Tue, Mar 29, 2011 at 02:15:30PM -0500, Michael Witten escreveu: > On Tue, Mar 29, 2011 at 13:15, Raghavendra D Prabhu <rprabhu@wnohang.net> wrote: > > the patch submitted by Michael seems to be taking care of far > > more cases than mine, so that is much better. > > One major issue with my current patch is that I opted to stop the > build with an error message even when using the default python command > names; is this undesirable? Is it better to fail silently as before > (via the somewhat cryptic Python.h message), or is it better to force > the user to specify that no python support should be built?
I think that the best course of action is to emit a warning and go, i.e. we don't have to make it harder for people that don't want $FOO support to make that clear.
Just not having python-dev{el} installed should be enough to disable the feature. For people that want the feature the warning is enough to tell them to install the missing package.
- Arnaldo
| |