cmake: Restrict EGL usage to when PIGLIT_USE_WAFFLE is true.

EGL is only supported when waffle is used.

This prevents spurious dependencies on libEGL.so (which were causing
run-time failures here as my test slaves didn't have it installed.)

Trivial.
cubemap-getteximage-pbo: Tests reading cube faces into a PBO

Exercises an NVIDIA driver bug.
arb_texture_view-getteximage-srgb: test sRGB texture views

Exercises an NVIDIA driver bug related to glGetTexImage and sRGB
texture views.
update the HACKING file's Coding style section

I've tried to describe Piglit's coding style and conventions in more
detail.  Hopefully, new contributors will read this and it'll save
some some time and effort for the reviewers.

Please feel free to add/update this info.

Reviewed-by: Chad Versace <chad.versace@linux.intel.com>
Reviewed-by: Paul Berry <stereotype441@gmail.com>
arb_occlusion_query: remove unused variable
arb_uniform_buffer_object: silence asprintf() warning
arb_separate_shader_objects: silence string format warning
HACKING: Add note for python formatting

Signed-off-by: Dylan Baker <baker.dylan.c@gmail.com>
summary.py: use mako's TemplateLookup

This buys two things, a little less code duplication, and allows us to
load one mako template into another one.

Signed-off-by: Dylan Baker <baker.dylan.c@gmail.com>
templates: move templates into html subfolder

This just makes it easy to know what a template is for

Signed-off-by: Dylan Baker <baker.dylan.c@gmail.com>
index.mako: Split index.mako into three files

This is going to allow us to simplify the generation of the tables
considerably.

Signed-off-by: Dylan Baker <baker.dylan.c@gmail.com>
summary: Split code out of HTMLIndex

This patch moves the generation of the column headers out of the
HTMLIndex class and into a separate mako template.

This helps to simplify the HTMLIndex class, since this code included a
lot of special handling in that class.

Signed-off-by: Dylan Baker <baker.dylan.c@gmail.com>
templates/html/index_body.mako: remove unused subtestResult

This code was left over from an early iteration of the code, and isn't
used.

Signed-off-by: Dylan Baker <baker.dylan.c@gmail.com>
summary.py: Simplify the way HTMLIndex works

This patch replaces home-rolled code with built-in python functions,
this results in cleaner, simpler, and faster code.

Signed-off-by: Dylan Baker <baker.dylan.c@gmail.com>
Summary.py: Convert HTMLIndex class into a generator

This change does a couple of really nice things for us, first it cuts
down on the memory consumption by using a generator (which is inherently
lazy); the second is that because it is a part of the Summary class, it
doesn't have to have the summary class passed to it, since it inherently
has access to ``self''.

Signed-off-by: Dylan Baker <baker.dylan.c@gmail.com>