[Piglit] [PATCH 0/3] Update piglit install locations

Dylan Baker baker.dylan.c at gmail.com
Fri May 2 00:50:17 PDT 2014


I think patches 1 and 2 look good. You have my r-b on those.

I have a branch here: https://github.com/dcbaker/piglit piglit-cmd
that IMO has a more elegant solution for a piglit executable.
The big advantages I see to my approach is that there is very little
hand rolled code, we don't need cmake to populate any data for us,
and it works in all three use cases (not built, not installed;
built, not installed; built, installed).

I have tested my piglit in all three applications and it works for me,
would you have a look?


On Wednesday, April 30, 2014 10:28:54 Jordan Justen wrote:
> Move piglit binaries and libraries out of the system paths
> Add a piglit wrapper command to easily run piglit commands
> in the new install location
> git://people.freedesktop.org/~jljusten/piglit piglit-cmd
> Jordan Justen (3):
>   cmake: Install under separate piglit directory
>   cmake: Add version suffix for install paths
>   piglit: Add piglit command
>  CMakeLists.txt          | 43 ++++++++++++++++++----
>  cmake/piglit_util.cmake |  4 +--
>  piglit.in               | 42 ++++++++++++++++++++++
>  piglit_cmd.py           | 96 +++++++++++++++++++++++++++++++++++++++++++++++++
>  4 files changed, 177 insertions(+), 8 deletions(-)
>  create mode 100755 piglit.in
>  create mode 100755 piglit_cmd.py

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20140502/bb9d8223/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 490 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20140502/bb9d8223/attachment.sig>

More information about the Piglit mailing list