[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