<div dir="ltr"><div><div><div><div>We Probably should also have two reminders for people that will be highly useful.<br><br></div>1) To please pay attention to the various FileSystem valid file names. This means to refer to the relevant wikipedia article ( <a href="http://en.wikipedia.org/wiki/Filename">http://en.wikipedia.org/wiki/Filename</a> ) over this and pay attention to what each platform can handle. As a good rule of Thumb for File name Limitations I would use "Windows VFAT", NTFS, Mac OS HFS+ and Posix as a list. This means that the following characters should not be generated filenames ( This is key for Python since glean is known to generate one or more of these).... :|\?*<":>+[]/<br>
<br></div>2) Overall file names need to be somewhat Limited. I understand that 70 characters in a single filename is fine, but a reminder that overall filesystem limitations still matter. Two tests exist that already exist that violate this principle and those are... same-attachment-glFramebufferRenderbuffer-GL_DEPTH_STENCIL_ATTACHMENT and same-attachment-glFramebufferTexture2D-GL_DEPTH_STENCIL_ATTACHMENT. With these tests, the cmake overhead winds up turning them into a mess that can easily exceed 256 characters in a path. because the longest file for each of these are as follows...<br>
<br>target_api/gl/tests/spec/arb_framebuffer_object/CMakeFiles/same-attachment-glFramebufferRenderbuffer-GL_DEPTH_STENCIL_ATTACHMENT.dir/same-attachment-glFramebufferRenderbuffer-GL_DEPTH_STENCIL_ATTACHMENT.c.o<br></div>and <br>
target_api/gl/tests/spec/arb_framebuffer_object/CMakeFiles/same-attachment-glFramebufferTexture2D-GL_DEPTH_STENCIL_ATTACHMENT.dir/same-attachment-glFramebufferTexture2D-GL_DEPTH_STENCIL_ATTACHMENT.c.o <br><br></div>With both of these, it is easy to see that the file name is really long and can easily get out of hand. On windows this is especially bad when you take into considering that a user will probably try to build the source from their desktop and that can easily add another 34 characters... Just as a note, nmake ( and visual studio ) can run into a problem if the full path exceeds 218 characters ( See: <a href="http://support.microsoft.com/kb/213983">http://support.microsoft.com/kb/213983</a> NOTE: This applies to excel, but does occur when using visual studio 2010 also )<br>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Nov 4, 2013 at 8:41 AM, Brian Paul <span dir="ltr"><<a href="mailto:brianp@vmware.com" target="_blank">brianp@vmware.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 11/02/2013 09:02 PM, Dylan Baker wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Could you add a note about python code?<br>
<br>
Just something like: "All python code should conform to python's PEP8<br>
standard, using only spaces and no tabs" would be great.<br>
</blockquote>
<br></div>
Go for it!<span class="HOEnZb"><font color="#888888"><br>
<br>
-Brian</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
<br>
______________________________<u></u>_________________<br>
Piglit mailing list<br>
<a href="mailto:Piglit@lists.freedesktop.org" target="_blank">Piglit@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/piglit" target="_blank">http://lists.freedesktop.org/<u></u>mailman/listinfo/piglit</a><br>
</div></div></blockquote></div><br></div>