> On April 17, 2013, 5:51 p.m., Ben Mahler wrote: > > > > Ben Mahler wrote: > Do you see CMake making these sorts of things easier?
I don't think it would make a difference really. I can simplify the check and just look for Python.h explicitly, but I think using python-config is the 'preferred' way. > On April 17, 2013, 5:51 p.m., Ben Mahler wrote: > > configure.ac, line 367 > > <https://reviews.apache.org/r/10562/diff/1/?file=281695#file281695line367> > > > > What is python-config? > > Brenden Matthews wrote: > It's part of the Python distribution: > > https://github.com/python-git/python/blob/master/Misc/python-config.in > > It's just a script that can be used to test the current python > installation. > > Ben Mahler wrote: > Ah I see, I have a linux machine here that doesn't know where it is: > > $ python-config > -bash: python-config: command not found > > With the default python being 2.4.3, was python-config not introduced at > that point? > > It has the following on its path: > $ python<TAB><TAB> > python python2 python2.4 python26 > python2.6 python2.6-config python2.7 I see. It's named python2.6-config on your system. You probably have the dev package installed for 2.6, but not the others. Why your OS doesn't link python-config to python2.6-config I don't know. Here's how it looks on my (debian sid) system: brenden@airbob ~ $ which python-config /usr/bin/python-config brenden@airbob ~ $ ll /usr/bin/python-config lrwxrwxrwx 1 root root 16 Feb 8 13:29 /usr/bin/python-config -> python2.7-config brenden@airbob ~ $ - Brenden ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10562/#review19335 ----------------------------------------------------------- On April 16, 2013, 10:07 p.m., Brenden Matthews wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10562/ > ----------------------------------------------------------- > > (Updated April 16, 2013, 10:07 p.m.) > > > Review request for mesos. > > > Description > ------- > > Check for Python.h in configure script. > > > Diffs > ----- > > configure.ac c3b5a05 > > Diff: https://reviews.apache.org/r/10562/diff/ > > > Testing > ------- > > Used in production at Airbnb. > > > Thanks, > > Brenden Matthews > >
