[SSSD] [PATCH] Make configure script compatible with older python versions

Sumit Bose sbose at redhat.com
Tue Sep 22 07:40:34 UTC 2009


On Tue, Sep 22, 2009 at 09:32:22AM +0200, Sumit Bose wrote:
> On Mon, Sep 21, 2009 at 04:47:14PM -0400, Stephen Gallagher wrote:
> > On 09/21/2009 04:35 PM, Stephen Gallagher wrote:
> > > Older python versions (such as that used in RHEL5) do not have a
> > > python-config executable to report CFLAGS and LIBS. In order to
> > > support such versions of python, we will duplicate the logic that
> > > python-config would have performed directly in our configure
> > > script
> > > 
> > > 
> > > The original patch that enabled the python support broke the RHEL5
> > > builds. This patch will fix it. Please review promptly.
> > > 
> > 
> > It would be easier to review promptly if I'd actually attached the patch...
> 
> yes, it was easier this way :-)
> 
> ACK
> 
> Btw. I have seen a couple for compiler warnings not related to this
> patch but to the python bindings in general.
> 

ah, sorry, I have forgotten a minor comment. Please consider adding a
comment to python.m4 why python-config is not used here. Because in a
couple of month someone who does not read the commit logs carefully
might be tempted to 'optimize' this section with the help of
python-config.

bye,
Sumit



More information about the sssd-devel mailing list