mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Armand Grillet <agril...@mesosphere.io>
Subject Review Request 67322: Added Python 2 check for Python bindings when using `configure`.
Date Fri, 25 May 2018 13:01:02 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/67322/
-----------------------------------------------------------

Review request for mesos and Benjamin Bannier.


Bugs: MESOS-8955
    https://issues.apache.org/jira/browse/MESOS-8955


Repository: mesos


Description
-------

The Python bindings work with Python 2. If Python 3 is set as
the default `python` on the system, they should not be built.

This check in `configure.ac` ensures that the Python version is
not more than 2.7. This is similar to the current check we have
for the Python CLI.

CMake does not offer an option to build the Python bindings thus
we only need to add the check for autoconf users.


Diffs
-----

  configure.ac 03d333d65bcab2e46cff0b1329e5ad7bb26da697 


Diff: https://reviews.apache.org/r/67322/diff/1/


Testing
-------


Thanks,

Armand Grillet


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message