Age | Commit message (Collapse) | Author |
|
Some cmake options and features macros had form USE_${feature}, others
BUILD_${category}_TESTS. This patch consistently prefixes all such options
and macros with PIGLIT.
Piglit, be a good code citizen. Namespace your variables.
This patch was created with
find . -type f | xargs sed -i \
-e 's/\(^\|[^_]\)\(USE_\(GLX\|GLUT\|WAFFLE\|OPENGL\|OPENGL_ES1\|OPENGL_ES2\)\)\($\|[^_]\)/\1PIGLIT_\2\4/g' \
-e 's/\(^\|[^_]\)\(BUILD_\(CL\|GLX\|GLES1\|GLES2\)_TESTS\)\($\|[^_]\)/\1PIGLIT_\2\4/g' \
Reviewed-and-tested-by: Jordan Justen <jordan.l.justen@intel.com>
Acked-by: Kenneth Graunke <kenneth@whitecape.org>
Tested-by: Paul Berry <stereotype441@gmail.com>
Acked-by: Eric Anholt <eric@anholt.net>
Signed-off-by: Chad Versace <chad.versace@linux.intel.com>
|
|
Move piglit-util.* to piglit-util-gl-common.*
Signed-off-by: Blaž Tomažič <blaz.tomazic@gmail.com>
Reviewed-by: Chad Versace <chad.versace@linux.intel.com>
|
|
Nearly all CMakeLists.gl.txt contained the following:
include_directories(${GLUT_INCLUDE_DIR})
link_libraries(${GLUT_glut_LIBRARY})
This patch relocates them to cmake/target_api/gl/CMakeLists.txt.
Reviewed-by: Pauli Nieminen <pauli.nieminen@intel.com>
Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
Reviewed-by: Anuj Phogat <anuj.phogat@gmail.com>
Signed-off-by: Chad Versace <chad.versace@linux.intel.com>
|
|
Commit ebf571b7959ab7b15b4588474a8613e110fab5a5 causes the piglitglxutil
library to be after the glut library on the link command line, which causes
the glutInit function to not be found.
Reviewed-by: Chad Versace <chad.versace@linux.intel.com>
|
|
Reviewed-by: Chad Versace <chad.versace@linux.intel.com>
|
|
Reviewed-by: Chad Versace <chad.versace@linux.intel.com>
|
|
Replace
add_executable -> piglit_add_executable
add_library -> piglit_add_library
For now, each wrapper function trivially wraps the function it replaces,
and so this patch changes no behavior.
This commit was created using the command:
sed -i -e "s/add_executable/piglit_add_executable/" -e "s/add_library/piglit_add_library/" $(find src/ tests/ -name "*CMakeLists*")
Signed-off-by: Chad Versace <chad.versace@linux.intel.com>
|
|
And error out when they're not found, rather than failing to
compile later when GL/glxproto.h can't be found.
|
|
It was impossible to have the source directory read-only.
Also add the include_directories() directive for tests/util
in just one place, under tests/, so that all subdirectories
inherit it. A bunch of CMakeLists.txt files duplicate it,
so delete those redundant include flags:
sed -i -e "/^\t\${piglit_SOURCE_DIR}\/tests\/util/ d" \
`grep piglit_SOURCE_DIR -rl tests/ | grep "CMakeLists\.gl"`
|
|
Move the strstr-based code from glx_ext_import_context/common.c to a
new utility function called piglit_is_extension_in_string. Use this
function to implement both piglit_is_extension_supported and
piglit_is_glx_extension_supported.
Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Brian Paul <brianp@vmware.com>
Reviewed-by: Chad Versace <chad.versace@linux.intel.com>
|
|
This only tests the version of the function from
GLX_EXT_import_context and its trivial interaction with
GLX_SGIX_fbconfig. There are enough differences with GLX 1.3 that the
GLX 1.3 functionality should be tested separately.
Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Brian Paul <brianp@vmware.com>
|
|
Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Brian Paul <brianp@vmware.com>
|
|
The GLXContextID of the imported context should be the same as the
original.
Both NVIDIA's closed-source driver and AMD's closed-source driver pass
this test.
Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Brian Paul <brianp@vmware.com>
|
|
This test fails on NVIDIA's closed-source driver.
This test fails on AMD's closed-source driver because it hits an
assertion failure in XCB:
xcb_io.c:515: _XReply: Assertion `!dpy->xcb->reply_data' failed.
Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Brian Paul <brianp@vmware.com>
|
|
This test passes on NVIDIA's closed-source driver.
This test fails on AMD's closed-source driver because it hits an
assertion failure in XCB:
xcb_io.c:515: _XReply: Assertion `!dpy->xcb->reply_data' failed.
Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Brian Paul <brianp@vmware.com>
|
|
This test fails on NVIDIA's closed-source driver because they do not
generate a GLXBadContext error the invalid context ID 0 is imported.
They do, however, correctly generate the error when 0xDEADBEEF is
imported.
This test fails on AMD's closed-source driver because it hits an
assertion failure in XCB:
xcb_io.c:515: _XReply: Assertion `!dpy->xcb->reply_data' failed.
Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Brian Paul <brianp@vmware.com>
|
|
This test fails on NVIDIA's closed-source driver because they do not
generate a GLXBadContext error the invalid context ID 0 is imported.
They do, however, correctly generate the error when 0xDEADBEEF is
imported.
This test fails on AMD's closed-source driver because it hits an
assertion failure in XCB:
xcb_io.c:515: _XReply: Assertion `!dpy->xcb->reply_data' failed.
Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Brian Paul <brianp@vmware.com>
|
|
This test fails on NVIDIA's closed-source driver for a couple reasons.
First, inspite of what the spec says, they appear to release the
server-side context when glXFreeContextEXT is called. Second, they do
not generate a GLXBadContext error when a NULL context is freed.
This test fails on AMD's closed-source driver because it hits an
assertion failure in XCB:
xcb_io.c:515: _XReply: Assertion `!dpy->xcb->reply_data' failed.
Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Brian Paul <brianp@vmware.com>
|
|
This test fails on NVIDIA's closed-source driver because it segfaults
while trying to glXGetContextIDEXT on a NULL context.
This test fails on AMD's closed-source driver because it hits an
assertion failure in XCB:
xcb_io.c:515: _XReply: Assertion `!dpy->xcb->reply_data' failed.
Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Brian Paul <brianp@vmware.com>
|
|
v2: Fix mixed tabs and spaces issues noticed by Brian Paul.
Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Brian Paul <brianp@vmware.com>
|