<p dir="ltr">Hi Brian, </p>
<p dir="ltr">Let me begin by thanking you  for the feedback Brian. The intention of the patch (and other subsequent ones ) is to move the tests in the bugs/ directory to appropriate locations in the spec/ directory - an item in the TODO file. I appreciate your feedback and will rework this patch based on your recommendations. </p>
<p dir="ltr">Regards. <br>
Guy-Daniel Tiku. </p>
<div class="gmail_quote">On Apr 1, 2015 3:13 PM, "Brian Paul" <<a href="mailto:brianp@vmware.com">brianp@vmware.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 04/01/2015 07:58 AM, Guy-Daniel Tiku wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
From: guy-D <<a href="mailto:guydany37@gmail.com" target="_blank">guydany37@gmail.com</a>><br>
<br>
---<br>
  tests/bugs/CMakeLists.gl.txt                       |  1 -<br>
  tests/bugs/crash-cubemap-<u></u>order.c                   | 51 ----------------------<br>
  tests/spec/CMakeLists.txt                          |  1 +<br>
  .../arb_texture_cube_map_<u></u>order/CMakeLists.gl.txt   | 14 ++++++<br>
  .../spec/arb_texture_cube_map_<u></u>order/CMakeLists.txt |  1 +<br>
  .../crash-cubemap-order.c                          | 51 ++++++++++++++++++++++<br>
  6 files changed, 67 insertions(+), 52 deletions(-)<br>
  delete mode 100644 tests/bugs/crash-cubemap-<u></u>order.c<br>
  create mode 100644 tests/spec/arb_texture_cube_<u></u>map_order/CMakeLists.gl.txt<br>
  create mode 100644 tests/spec/arb_texture_cube_<u></u>map_order/CMakeLists.txt<br>
  create mode 100644 tests/spec/arb_texture_cube_<u></u>map_order/crash-cubemap-order.<u></u>c<br>
</blockquote>
<br>
spec/arb_texture_cube_map_<u></u>order/ is not the correct place for this.<br>
<br>
The directories under spec/ are named after extensions or GL/GLSL spec revisions.  There is no GL_ARB_texture_cube_map_order extension.<br>
<br>
The right directory would be spec/arb_texture_cube_map/<br>
<br>
Is it your intention to move other tests?  The tests/general/, tests/texturing/, tests/bugs/, etc. directories are kind of a legacy dumping ground for tests and ideally many of those tests would be organized under spec/, but it would be a huge job.  We should probably have a plan for people to comment on before going too far.<br>
<br>
-Brian<br>
<br>
<br>
</blockquote></div>