<div dir="ltr"><div>I submitted a couple of patches that still have not been applied to the piglit git repository so I figure It would be a good idea to bring up the fixes now...<br><br>==================================<br>
[Piglit] [PATCH 1/8] cmake: fix out of tree Installation and package generation.<br><a href="http://lists.freedesktop.org/archives/piglit/2013-September/007572.html">http://lists.freedesktop.org/archives/piglit/2013-September/007572.html</a><br>
<br>This patch is designed to fix the problem of the packages not including ANY generated tests when using out of tree build. This is a somewhat major problem because not everyone builds piglit in-source. Some people build piglit out of tree. Currently the installation expects you to be building in-tree and this is wrong. <br>
<br>The variable CMAKE_BINARY_DIR equals CMAKE_SOURCE_DIR when building in tree, but when you build the source out of tree the packaging script finds no tests because it only looks in the source tree. When you build out of tree the generated tests appear in the cmake_binary_dir folder.<br>
<br>==================================<br>[Piglit] [PATCH 2/8] cmake: add *.geom to the list of installed files.<br><a href="http://lists.freedesktop.org/archives/piglit/2013-September/007573.html">http://lists.freedesktop.org/archives/piglit/2013-September/007573.html</a><br>
[Piglit] [PATCH 3/8] cmake: add *.cl to the list of installed generated tests.<br><a href="http://lists.freedesktop.org/archives/piglit/2013-September/007574.html">http://lists.freedesktop.org/archives/piglit/2013-September/007574.html</a><br>
<br></div>These two patches simply add the missing files that are required for testing OpenCL and OpenGL Geometry shaders to the list of installed files.<br><div><br><br>==================================<br>[Piglit] [PATCH 5/8] cmake: Increase minimum version from 2.6 to 2.8.<br>
<a href="http://lists.freedesktop.org/archives/piglit/2013-September/007576.html">http://lists.freedesktop.org/archives/piglit/2013-September/007576.html</a><br><br>I know this change was contested, and this is a minor change that is aimed at politely reminding users on windows and OSX that CMake 2.6 is old and that they need to update their systems because the last release in this series was May 2009. For users on linux, This is already a build requirement because waffle requires cmake 2.8 to even compile.<br>
<br></div><div>==================================<br>[Piglit] [PATCH 6/8] <a href="http://config.h.in">config.h.in</a>: add define for HAVE_STRNDUP<br><a href="http://lists.freedesktop.org/archives/piglit/2013-October/007584.html">http://lists.freedesktop.org/archives/piglit/2013-October/007584.html</a><br>
<br></div><div>This fixes compile problems on numerous systems that already define and implement strndup.<br></div><div><br>==================================<br>[Piglit] [PATCH 7/8] piglit-util-gl-common: Add Function to clear extensions list.<br>
<a href="http://lists.freedesktop.org/archives/piglit/2013-September/007578.html">http://lists.freedesktop.org/archives/piglit/2013-September/007578.html</a><br>[Piglit] [PATCH 8/8] util/waffle: reinitialize extensions<br>
<a href="http://lists.freedesktop.org/archives/piglit/2013-September/007579.html">http://lists.freedesktop.org/archives/piglit/2013-September/007579.html</a><br><br>These two patches have already been reviewed previously  Chad [2] early last month and the suggested changes where applied.<br>
<br><br>[1] Kitware Wiki - CMake Useful Variables<br><a href="http://www.cmake.org/Wiki/CMake_Useful_Variables">http://www.cmake.org/Wiki/CMake_Useful_Variables</a><br><br>[2] [Piglit] [PATCH v2 5/5] util/waffle: reinitialize extensions<br>
<a href="http://lists.freedesktop.org/archives/piglit/2013-September/007166.html">http://lists.freedesktop.org/archives/piglit/2013-September/007166.html</a><br></div></div>