I'll need to think about it and read the code to be sure, I think that using '/' in test run names may cause some strange behavior in the html pages because of assumptions. There's also the windows '\\' issue to think about.
But, either way you need to put makedirs in a try/except block
try:
os.makedirs(...)
except OSError:
pass
On Saturday, August 30, 2014 11:49:51 AM Jason Ekstrand wrote:
> This way test names can contain "/" characters. I to name tests after git
> branches and I tend to have git branches named wip/whatever. This prevents
> piglit from crashing.
> ---
> framework/summary.py | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/framework/summary.py b/framework/summary.py
> index 332aa40..1815dc7 100644
> --- a/framework/summary.py
> +++ b/framework/summary.py
> @@ -454,7 +454,7 @@ class Summary:
>
> # Iterate across the tests creating the various test specific files
> for each in self.results:
> - os.mkdir(path.join(destination, each.name))
> + os.makedirs(path.join(destination, each.name))
>
> if each.time_elapsed is not None:
> time = datetime.timedelta(0, each.time_elapsed)
> --
> 2.1.0
>
> _______________________________________________
> Piglit mailing list
> Piglit@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/piglit
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ Piglit mailing list Piglit@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/piglit
