<div dir="ltr">On 13 October 2015 at 08:54, Matt Turner <span dir="ltr"><<a href="mailto:mattst88@gmail.com" target="_blank">mattst88@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Sat, Oct 10, 2015 at 10:30 PM, Rhys Kidd <<a href="mailto:rhyskidd@gmail.com">rhyskidd@gmail.com</a>> wrote:<br>
> Signed-off-by: Rhys Kidd <<a href="mailto:rhyskidd@gmail.com">rhyskidd@gmail.com</a>><br>
> ---<br>
>  README | 5 +++++<br>
>  1 file changed, 5 insertions(+)<br>
><br>
> diff --git a/README b/README<br>
> index 6ed3244..03be4e7 100644<br>
> --- a/README<br>
> +++ b/README<br>
> @@ -60,6 +60,11 @@ Build with:<br>
>  make<br>
>  ```<br>
><br>
> +run.py relies on a symbolic link to a built piglit bin directory, as follows:<br>
> +```<br>
> +ln -s <location_of_piglit>/bin "$PWD"/bin<br>
> +```<br>
<br>
</span>run.py prints an appropriate message, doesn't it? If we want to do<br>
anything at all, maybe we should just print a more verbose message<br>
from run.py?<br>
</blockquote></div><br></div><div class="gmail_extra">Hello Matt,<br><br>There is a relevant distinction though here, between a user-reported error (in the general sense) and a user-reported error due to a missing dependency.<br><br></div><div class="gmail_extra">Having a local built copy of piglit is a dependency of the run.py script, hence why I'm recommending including that fact prominently within README's dependency section per this patch.<br><br></div><div class="gmail_extra">That said, I'm not wedded to this approach -- so will shortly send out a re-roll of the two patches from this series that aren't upstream to the mailing list for further comment.<br><br></div><div class="gmail_extra">Regards,<br></div><div class="gmail_extra">Rhys<br></div></div>