<div dir="ltr"><br><br><br><div class="gmail_quote"><div dir="ltr">On Wed, 7 Nov 2018 at 10:47, Chris Johns <<a href="mailto:chrisj@rtems.org">chrisj@rtems.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 05/11/2018 05:14, Vijay Kumar Banerjee wrote:<br>
> On Sun, 28 Oct 2018 at 21:54, Vijay Kumar Banerjee <<a href="mailto:vijaykumar9597@gmail.com" target="_blank">vijaykumar9597@gmail.com</a><br>
> <mailto:<a href="mailto:vijaykumar9597@gmail.com" target="_blank">vijaykumar9597@gmail.com</a>>> wrote:<br>
> <br>
> I could run with copy_tree using this workaround <br>
> <br>
> =======================================<br>
> diff --git a/rtemstoolkit/path.py b/rtemstoolkit/path.py<br>
> index 760f4bd..df553b3 100644<br>
> --- a/rtemstoolkit/path.py<br>
> +++ b/rtemstoolkit/path.py<br>
> @@ -191,10 +191,10 @@ def copy_tree(src, dst):<br>
> hsrc = host(src)<br>
> hdst = host(dst)<br>
> <br>
> - if os.path.exists(src):<br>
> + if os.path.exists(src) and os.path.isdir(src):<br>
> names = os.listdir(src)<br>
> else:<br>
> - name = []<br>
> + names = []<br>
> <br>
> if not os.path.isdir(dst):<br>
> os.makedirs(dst)<br>
> <br>
> =====================================<br>
> <br>
> But there's still an issue with the file permission as it uses copystat which <br>
> copies the permission bits which is not required in this case. <br>
> It requires a manual `rm` with root permission before rerunning the coverage.<br>
> <br>
> ping :) <br>
> <br>
<br>
Sorry about the delay. This looks find, can you please provide a patch.<br>
<br></blockquote><div>Hi</div><div>I just tried it and it doesn't work unfortunately, I have added another line of modification here</div><div>and now it's working, I'll send two separate patches each for path and coverage. Please review. </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Thanks<br>
Chris<br>
</blockquote></div></div>