xiaoxiang781216 merged PR #2859:
URL: https://github.com/apache/nuttx-apps/pull/2859
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...
Gary-Hobson commented on PR #2859:
URL: https://github.com/apache/nuttx-apps/pull/2859#issuecomment-2491451482
> Hmm.. is it possible to make it work also on Sim? That sounds like a
perfect candidate for experimentation because no real hardware is involved it
can be build on any availab
Gary-Hobson commented on code in PR #2859:
URL: https://github.com/apache/nuttx-apps/pull/2859#discussion_r1852272927
##
system/gcov/gcov.c:
##
@@ -69,6 +69,72 @@ static void gcov_dump(FAR const char * path, FAR const char
*strip)
__gcov_dump();
}
+/**
W-M-R commented on PR #2859:
URL: https://github.com/apache/nuttx-apps/pull/2859#issuecomment-2490693227
> > > @cederom: Build errors come from dependency on [apache/nuttx#14838
(review)](https://github.com/apache/nuttx/pull/14838#pullrequestreview-2443216552)
right?
> >
> >
> >
xiaoxiang781216 commented on code in PR #2859:
URL: https://github.com/apache/nuttx-apps/pull/2859#discussion_r1851555349
##
system/gcov/gcov.c:
##
@@ -69,6 +69,72 @@ static void gcov_dump(FAR const char * path, FAR const char
*strip)
__gcov_dump();
}
+/**
cederom commented on PR #2859:
URL: https://github.com/apache/nuttx-apps/pull/2859#issuecomment-2490209562
> > @cederom: Build errors come from dependency on [apache/nuttx#14838
(review)](https://github.com/apache/nuttx/pull/14838#pullrequestreview-2443216552)
right?
>
> @Gary-Hobson
Gary-Hobson commented on PR #2859:
URL: https://github.com/apache/nuttx-apps/pull/2859#issuecomment-2490188064
> Build errors come from dependency on [apache/nuttx#14838
(review)](https://github.com/apache/nuttx/pull/14838#pullrequestreview-2443216552)
right?
This is because the sim
Gary-Hobson commented on code in PR #2859:
URL: https://github.com/apache/nuttx-apps/pull/2859#discussion_r1851438065
##
system/gcov/gcov.c:
##
@@ -69,6 +69,72 @@ static void gcov_dump(FAR const char * path, FAR const char
*strip)
__gcov_dump();
}
+/**
xiaoxiang781216 commented on code in PR #2859:
URL: https://github.com/apache/nuttx-apps/pull/2859#discussion_r1847800697
##
system/gcov/gcov.c:
##
@@ -69,6 +69,72 @@ static void gcov_dump(FAR const char * path, FAR const char
*strip)
__gcov_dump();
}
+/**
xiaoxiang781216 commented on code in PR #2859:
URL: https://github.com/apache/nuttx-apps/pull/2859#discussion_r1846963527
##
system/gcov/Kconfig:
##
@@ -5,7 +5,7 @@
config SYSTEM_GCOV
tristate "gcov tool"
- depends on SCHED_GCOV
+ depends on !COVERAGE_NONE
cederom commented on PR #2859:
URL: https://github.com/apache/nuttx-apps/pull/2859#issuecomment-2483614175
Build errors come from dependency on
https://github.com/apache/nuttx/pull/14838#pullrequestreview-2443216552 right?
--
This is an automated message from the Apache Git Service.
To re
nuttxpr commented on PR #2859:
URL: https://github.com/apache/nuttx-apps/pull/2859#issuecomment-2483255044
[**\[Experimental Bot, please feedback
here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues)
No, this PR does not fully meet the NuttX requiremen
Gary-Hobson opened a new pull request, #2859:
URL: https://github.com/apache/nuttx-apps/pull/2859
## Summary
system: support gcov output to stdout
For detailed description, please refer to
https://github.com/apache/nuttx/pull/14838
https://github.com/apache/nuttx/pull/14838
13 matches
Mail list logo
