----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11114/#review21548 -----------------------------------------------------------
configure.ac <https://reviews.apache.org/r/11114/#comment44590> You're right that we should definitely be checking for Python.h. It seems python-config doesn't exist on CentOS 5.5 (at least), so this doesn't seem like a portable solution. Alternatively, you can construct a small C program to #include Python.h, and test if it compiles. You'll notice we've done this elsewhere in this file, for jni.h. - Ben Mahler On June 6, 2013, 2:12 a.m., Brenden Matthews wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/11114/ > ----------------------------------------------------------- > > (Updated June 6, 2013, 2:12 a.m.) > > > Review request for mesos. > > > Description > ------- > > Check for Python.h in configure script. > > Review: https://reviews.apache.org/r/11114 > > > Diffs > ----- > > configure.ac 8b6d74fd78613965340ecff71bb933c9c4d24e9e > > Diff: https://reviews.apache.org/r/11114/diff/ > > > Testing > ------- > > Used in production at airbnb. > > > Thanks, > > Brenden Matthews > >
