Re: [PATCH] RTEMS: Prefer int for int32_t

On 13/06/18 08:22, Joel Sherrill wrote: On Wed, Jun 13, 2018, 8:20 AM Sebastian Huber > wrote: I will check in this patch tomorrow (GCC 7, 8, and 9). Ok. Just a reminder that the next time newlib bumps versions, it will have a change  to correc

Re: error while building leon3 with gcov flags

On Wed, 13 Jun 2018, 11:51 Joel Sherrill, wrote: > > > On Sun, Jun 10, 2018, 6:02 PM Vijay Kumar Banerjee < > vijaykumar9...@gmail.com> wrote: > >> >> >> On Sun, 10 Jun 2018, 20:52 Joel Sherrill, wrote: >> >>> >>> >>> On Sat, Jun 9, 2018, 6:02 PM Vijay Kumar Banerjee < >>> vijaykumar9...@gmail.c

Re: [PATCH] RTEMS: Prefer int for int32_t

On Wed, Jun 13, 2018, 8:20 AM Sebastian Huber < sebastian.hu...@embedded-brains.de> wrote: > I will check in this patch tomorrow (GCC 7, 8, and 9). > Ok. Just a reminder that the next time newlib bumps versions, it will have a change to correct two pthread prototypes. RTEMS will.need a patch I

Re: error while building leon3 with gcov flags

On Sun, Jun 10, 2018, 6:02 PM Vijay Kumar Banerjee wrote: > > > On Sun, 10 Jun 2018, 20:52 Joel Sherrill, wrote: > >> >> >> On Sat, Jun 9, 2018, 6:02 PM Vijay Kumar Banerjee < >> vijaykumar9...@gmail.com> wrote: >> >>> On 10 June 2018 at 04:16, Joel Sherrill wrote: >>> I suggest this becau

Re: [PATCH] RTEMS: Prefer int for int32_t

I will check in this patch tomorrow (GCC 7, 8, and 9). -- Sebastian Huber, embedded brains GmbH Address : Dornierstr. 4, D-82178 Puchheim, Germany Phone : +49 89 189 47 41-16 Fax : +49 89 189 47 41-09 E-Mail : sebastian.hu...@embedded-brains.de PGP : Public key available on request.

Re: [PATCH v4 2/2] Adding trace documentation

On 13/06/2018 13:04, Vidushi Vashishth wrote: > > I have made the changes you suggested. Thanks. > I am still working on CTF support. > >>> +  $ ../samples >>What is this command? > > This is not a command. It is used to show which directory we are in. This > method > was used in the quicksta

Re: [PATCH v4 2/2] Adding trace documentation

Hi I have made the changes you suggested. I am still working on CTF support. >> + $ ../samples >What is this command? This is not a command. It is used to show which directory we are in. This method was used in the quickstart section. But I have removed this. >There is some doco that would be

Re: [PATCH v4 2/2] Adding trace documentation

Hi, This documentation looks good and I welcome this being done and added to the RTEMS project. I have some comments below I hope are easy to resolve. Thanks Chris On 13/06/2018 11:23, Vidushi Vashishth wrote: > --- > user/index.rst | 2 + > user/tracing/captureengine.rst | 1

Re: [PATCH v3 2/2] Adding trace documentation

I was using Sublime text. It appeared okay on the editor. I have changed my editor and sent another patch. Thanks. On Tue, Jun 12, 2018 at 9:46 PM, Gedare Bloom wrote: > On Tue, Jun 12, 2018 at 11:15 AM, Vidushi Vashishth > wrote: > > > > Adjusted line length to 79-80 > > --- > > user/index.r

[PATCH v4 2/2] Adding trace documentation

--- user/index.rst | 2 + user/tracing/captureengine.rst | 184 ++ user/tracing/examples.rst | 12 ++ user/tracing/index.rst | 29 + user/tracing/introduction.rst | 182 ++ user/tracing/tracelinker.rst | 287 ++

Re: [PATCH] Fix --show-commands.

On 12/06/2018 17:45, Christian Mauderer wrote: > Am 12.06.2018 um 09:29 schrieb Chris Johns: >> On 12/06/2018 17:18, Christian Mauderer wrote: >>> Am 12.06.2018 um 07:08 schrieb Chris Johns: On 09/06/2018 02:45, Christian Mauderer wrote: > > Did you also note Gedares suggestion to move

Re: [PATCH v3 2/2] Adding trace documentation

On Tue, Jun 12, 2018 at 11:15 AM, Vidushi Vashishth wrote: > > Adjusted line length to 79-80 > --- > user/index.rst | 2 + > user/tracing/captureengine.rst | 188 ++ > user/tracing/examples.rst | 12 ++ > user/tracing/index.rst | 33 + >

[PATCH v3 2/2] Adding trace documentation

Adjusted line length to 79-80 --- user/index.rst | 2 + user/tracing/captureengine.rst | 188 ++ user/tracing/examples.rst | 12 ++ user/tracing/index.rst | 33 + user/tracing/introduction.rst | 184 ++ user/tra

Re: [GSoC - Tracing] Status

Hi! >in case you receive comments for your patches the highest priority should be to fix the problems with the patches and send the next revision of the patch to the list, e.g. fixing trivial things like the line ?>length should be done in a couple of minutes. If you think a comment is invalid or

Re: [GSoC - Tracing] Status

Hello Vidushi, in case you receive comments for your patches the highest priority should be to fix the problems with the patches and send the next revision of the patch to the list, e.g. fixing trivial things like the line length should be done in a couple of minutes. If you think a comment i

Re: [PATCH] Fix --show-commands.

Am 12.06.2018 um 09:29 schrieb Chris Johns: > On 12/06/2018 17:18, Christian Mauderer wrote: >> Am 12.06.2018 um 07:08 schrieb Chris Johns: >>> On 09/06/2018 02:45, Christian Mauderer wrote: Did you also note Gedares suggestion to move the rtems_waf.git to the public repos? Maybe tha

Re: [PATCH] Fix --show-commands.

On 12/06/2018 17:18, Christian Mauderer wrote: > Am 12.06.2018 um 07:08 schrieb Chris Johns: >> On 09/06/2018 02:45, Christian Mauderer wrote: >>> >>> Did you also note Gedares suggestion to move the rtems_waf.git to the >>> public repos? Maybe that would be a good idea. >>> >> >> Yes I did and it

Re: [PATCH] Fix --show-commands.

Am 12.06.2018 um 09:18 schrieb Christian Mauderer: > The first one has a longer backward compatibility (relative URLs are > there since 2007: https://github.com/git/git/commit/f31a522a2d which > should be somewhere between git 1.5 and git 1.6). Just noted: That feature has been merged in 1.7.12. S

Re: [PATCH] coverage : Add support to run coverage in supported bsp without extra options

Looks fine to me. Chris will have the final say though. On Tue, 12 Jun 2018, 08:00 Vijay Kumar Banerjee, wrote: > > > On Tue, 12 Jun 2018, 12:27 Joel Sherrill, wrote: > >> Chris needs to speak up since it is Python. >> >> I see changes to HTML output which I doubt are related to the core change

Re: [PATCH] Fix --show-commands.

Am 12.06.2018 um 07:08 schrieb Chris Johns: > On 09/06/2018 02:45, Christian Mauderer wrote: >> >> Did you also note Gedares suggestion to move the rtems_waf.git to the >> public repos? Maybe that would be a good idea. >> > > Yes I did and it is a good idea. I have move the repo to the top level a

Re: [PATCH] coverage : Add support to run coverage in supported bsp without extra options

On Tue, 12 Jun 2018, 12:27 Joel Sherrill, wrote: > Chris needs to speak up since it is Python. > > I see changes to HTML output which I doubt are related to the core change. > There are no real changes to HTML, I have just added line breaks in the code because it was too long. > > On Tue, Jun 12