<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Dec 9, 2014 at 4:06 AM, Iago Toral Quiroga <span dir="ltr"><<a href="mailto:itoral@igalia.com" target="_blank">itoral@igalia.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">From: Samuel Iglesias Gonsalvez <<a href="mailto:siglesias@igalia.com">siglesias@igalia.com</a>><br>
<br>
It is now a hard dependency because of the autogeneration of<br>
format pack and unpack functions.<br>
<br>
Update the documentation to reflect this change.<br>
<br>
v2:<br>
- Inline python script in m4 file and use PYTHON2<br>
<br>
Signed-off-by: Samuel Iglesias Gonsalvez <<a href="mailto:siglesias@igalia.com">siglesias@igalia.com</a>><br>
---<br>
 <a href="http://configure.ac" target="_blank">configure.ac</a>                      |  2 +<br>
 docs/install.html                 |  6 ++-<br>
 m4/ax_check_python_mako_module.m4 | 77 +++++++++++++++++++++++++++++++++++++++<br>
 3 files changed, 84 insertions(+), 1 deletion(-)<br>
 create mode 100644 m4/ax_check_python_mako_module.m4<br>
<br>
diff --git a/<a href="http://configure.ac" target="_blank">configure.ac</a> b/<a href="http://configure.ac" target="_blank">configure.ac</a><br>
index 1d9d015..f38acfd 100644<br>
--- a/<a href="http://configure.ac" target="_blank">configure.ac</a><br>
+++ b/<a href="http://configure.ac" target="_blank">configure.ac</a><br>
@@ -76,6 +76,8 @@ if test "x$INDENT" != "xcat"; then<br>
     AC_SUBST(INDENT_FLAGS, '-i4 -nut -br -brs -npcs -ce -TGLubyte -TGLbyte -TBool')<br>
 fi<br>
<br>
+AC_CHECK_PYTHON_MAKO_MODULE(0.7.3)<br>
+<br>
 AC_PROG_INSTALL<br>
<br>
 dnl We need a POSIX shell for parts of the build. Assume we have one<br>
diff --git a/docs/install.html b/docs/install.html<br>
index f12425f..b12e1cb 100644<br>
--- a/docs/install.html<br>
+++ b/docs/install.html<br>
@@ -38,6 +38,10 @@<br>
 Version 2.6.4 or later should work.<br>
 </li><br>
 <br><br>
+<li><a href="<a href="http://www.makotemplates.org/" target="_blank">http://www.makotemplates.org/</a>">Python Mako module</a> -<br>
+Python Mako module is required. Version 0.7.3 or later should work.<br>
+</li><br>
+</br><br>
 <li><a href="<a href="http://www.scons.org/" target="_blank">http://www.scons.org/</a>">SCons</a> is required for building on<br>
 Windows and optional for Linux (it's an alternative to autoconf/automake.)<br>
 </li><br>
@@ -78,7 +82,7 @@ the needed dependencies:<br>
 <pre><br>
   sudo yum install flex bison imake libtool xorg-x11-proto-devel libdrm-devel \<br>
   gcc-c++ xorg-x11-server-devel libXi-devel libXmu-devel libXdamage-devel git \<br>
-  expat-devel llvm-devel<br>
+  expat-devel llvm-devel python-mako<br>
 </pre><br>
<br>
<br>
diff --git a/m4/ax_check_python_mako_module.m4 b/m4/ax_check_python_mako_module.m4<br>
new file mode 100644<br>
index 0000000..f289f26<br>
--- /dev/null<br>
+++ b/m4/ax_check_python_mako_module.m4<br>
@@ -0,0 +1,77 @@<br>
+# ===========================================================================<br>
+#<br>
+# SYNOPSIS<br>
+#<br>
+#   AX_CHECK_PYTHON_MAKO_MODULE(MIN_VERSION_NUMBER)<br>
+#<br>
+# DESCRIPTION<br>
+#<br>
+#   Check whether Python mako module is installed and its version higher than<br>
+#   minimum requested.<br>
+#<br>
+#   Example of its use:<br>
+#<br>
+#   For example, the minimum mako version would be 0.7.3. Then <a href="http://configure.ac" target="_blank">configure.ac</a><br>
+#   would contain:<br>
+#<br>
+#   AC_CHECK_PYTHON_MAKO_MODULE(0.7.3)<br>
+#<br>
+# LICENSE<br>
+#<br>
+#   Copyright (c) 2014 Intel Corporation.<br>
+#<br>
+#   This program is free software; you can redistribute it and/or modify it<br>
+#   under the terms of the GNU General Public License as published by the<br>
+#   Free Software Foundation; either version 2 of the License, or (at your<br>
+#   option) any later version.<br>
+#<br>
+#   This program is distributed in the hope that it will be useful, but<br>
+#   WITHOUT ANY WARRANTY; without even the implied warranty of<br>
+#   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU General<br>
+#   Public License for more details.<br>
+#<br>
+#   You should have received a copy of the GNU General Public License along<br>
+#   with this program. If not, see <<a href="http://www.gnu.org/licenses/" target="_blank">http://www.gnu.org/licenses/</a>>.<br>
+#<br>
+#   As a special exception, the respective Autoconf Macro's copyright owner<br>
+#   gives unlimited permission to copy, distribute and modify the configure<br>
+#   scripts that are the output of Autoconf when processing the Macro. You<br>
+#   need not follow the terms of the GNU General Public License when using<br>
+#   or distributing such scripts, even though portions of the text of the<br>
+#   Macro appear in them. The GNU General Public License (GPL) does govern<br>
+#   all other use of the material that constitutes the Autoconf Macro.<br>
+#<br>
+#   This special exception to the GPL applies to versions of the Autoconf<br>
+#   Macro released by the Autoconf Archive. When you make and distribute a<br>
+#   modified version of the Autoconf Macro, you may extend this special<br>
+#   exception to the GPL to apply to your modified version as well.<br>
+<br>
+dnl macro that checks for mako module in python<br>
+AC_DEFUN([AC_CHECK_PYTHON_MAKO_MODULE],<br>
+[AC_MSG_CHECKING(if module mako in python is installed)<br>
+    echo "<br>
+try:<br>
+    import sys<br>
+    import mako<br>
+except ImportError as err:<br>
+    sys.exit(err)<br>
+else:<br>
+    ver_min_req_str = str('$1');<br>
+    ver_min_req = ver_min_req_str.split('.');<br></blockquote><div><br></div><div>Why are there semicolons?<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+    ver_str = mako.__version__<br>
+    ver = ver_str.split('.')<br>
+<br>
+    for i in range(len(ver)):<br>
+        ver_int = int(ver[[i]])<br>
+        ver_min_req_int = int(ver_min_req[[i]]);<br>
+        if ver_int < ver_min_req_int:<br>
+            sys.exit(1);<br>
+        if ver_int > ver_min_req_int:<br>
+            sys.exit(0);<br>
+    sys.exit(0);" | $PYTHON2 -<br></blockquote><div><br></div><div>It would be nice to put the " and the |$PYTHON2 stuff on its own line so we can see where the inline python ends.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+    if test $? -ne 0 ; then<br>
+       AC_MSG_ERROR(mako $1 or later is required.)<br>
+    else<br>
+        AC_MSG_RESULT(yes)<br>
+    fi<br>
+])<br>
<span class="HOEnZb"><font color="#888888">--<br>
1.9.1<br>
<br>
_______________________________________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/mesa-dev" target="_blank">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</font></span></blockquote></div><br></div></div>