[PATCH] Coverage: add the style and js files to the symbol-set report directory

Vijay Kumar Banerjee vijaykumar9597 at gmail.com
Wed Nov 7 07:34:02 UTC 2018


On Wed, 7 Nov 2018 at 10:47, Chris Johns <chrisj at rtems.org> wrote:

> On 05/11/2018 05:14, Vijay Kumar Banerjee wrote:
> > On Sun, 28 Oct 2018 at 21:54, Vijay Kumar Banerjee <
> vijaykumar9597 at gmail.com
> > <mailto:vijaykumar9597 at gmail.com>> wrote:
> >
> >     I could run with copy_tree using this workaround
> >
> >     =======================================
> >     diff --git a/rtemstoolkit/path.py b/rtemstoolkit/path.py
> >     index 760f4bd..df553b3 100644
> >     --- a/rtemstoolkit/path.py
> >     +++ b/rtemstoolkit/path.py
> >     @@ -191,10 +191,10 @@ def copy_tree(src, dst):
> >          hsrc = host(src)
> >          hdst = host(dst)
> >
> >     -    if os.path.exists(src):
> >     +    if os.path.exists(src) and os.path.isdir(src):
> >              names = os.listdir(src)
> >          else:
> >     -        name = []
> >     +        names = []
> >
> >          if not os.path.isdir(dst):
> >              os.makedirs(dst)
> >
> >     =====================================
> >
> >     But there's still an issue with the file permission as it uses
> copystat which
> >     copies the permission bits which is not required in this case.
> >     It requires a manual `rm` with root permission before rerunning the
> coverage.
> >
> >  ping :)
> >
>
> Sorry about the delay. This looks find, can you please provide a patch.
>
> Hi
I just tried it and it doesn't work unfortunately, I have added another
line of modification here
and now it's working, I'll send two separate patches each for path and
coverage. Please review.

> Thanks
> Chris
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/devel/attachments/20181107/3728ca21/attachment-0002.html>


More information about the devel mailing list