<div dir="ltr"><div>Hello,</div><div>Please can you help explain what you mean by <i><b>Adding a "check-style" target to the RTEMS build system</b></i>?</div><div>And how I could possibly go about this?</div><div><br></div><div>Cheers,</div><div>Ida<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Mar 29, 2021 at 9:45 PM Gedare Bloom <<a href="mailto:gedare@rtems.org">gedare@rtems.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Mon, Mar 29, 2021 at 1:28 PM Ida Delphine <<a href="mailto:idadelm@gmail.com" target="_blank">idadelm@gmail.com</a>> wrote:<br>
><br>
> Yes I have. But wondering how to run it with the given configuration I saw in this thread(<a href="https://lists.rtems.org/pipermail/devel/2020-October/062770.html" rel="noreferrer" target="_blank">https://lists.rtems.org/pipermail/devel/2020-October/062770.html</a>).<br>
><br>
<br>
If you download/copy the configuration into a cfg file, then you can<br>
use the examples from<br>
<a href="https://github.com/uncrustify/uncrustify#running-the-program" rel="noreferrer" target="_blank">https://github.com/uncrustify/uncrustify#running-the-program</a> and<br>
attempt to run it on some files within rtems.git/cpukit/score/src<br>
would be my suggestion.<br>
<br>
> On Mon, Mar 29, 2021 at 3:37 PM Gedare Bloom <<a href="mailto:gedare@rtems.org" target="_blank">gedare@rtems.org</a>> wrote:<br>
>><br>
>> Hi Ida,<br>
>><br>
>> On Mon, Mar 29, 2021 at 7:36 AM Ida Delphine <<a href="mailto:idadelm@gmail.com" target="_blank">idadelm@gmail.com</a>> wrote:<br>
>> ><br>
>> > Hello,<br>
>> > Please do you mind telling me how to run uncrustify with the given configuration with any sample file?<br>
>><br>
>> What have you tried? Any directions followed/attempted or notes that<br>
>> you have taken would be helpful.<br>
>><br>
>> I guess all the info that you should need is in Uncrustify's readme<br>
>> file. <a href="https://github.com/uncrustify/uncrustify" rel="noreferrer" target="_blank">https://github.com/uncrustify/uncrustify</a><br>
>><br>
>> Did you successfully compile uncrustify tool?<br>
>><br>
>> > I'm a bit stuck.<br>
>> ><br>
>> > Thanks,<br>
>> > Ida.<br>
>> ><br>
>> > On Wed, Mar 17, 2021 at 10:34 PM Gedare Bloom <<a href="mailto:gedare@rtems.org" target="_blank">gedare@rtems.org</a>> wrote:<br>
>> >><br>
>> >> On Wed, Mar 17, 2021 at 2:28 PM Ida Delphine <<a href="mailto:idadelm@gmail.com" target="_blank">idadelm@gmail.com</a>> wrote:<br>
>> >> ><br>
>> >> > Hello,<br>
>> >> > So I have gone through this configuration file and I think I'm getting it. However I'm a bit lost in the reading the messages in the thread. Do you mind explaining? Or we can start talking about a way forward.<br>
>> >> > Also can you help me with some steps on how to test this by myself if possible?<br>
>> >> ><br>
>> >><br>
>> >> It may be easier if you go "up" a level to see the full thread<br>
>> >> context: <a href="https://lists.rtems.org/pipermail/devel/2020-October/thread.html#62769" rel="noreferrer" target="_blank">https://lists.rtems.org/pipermail/devel/2020-October/thread.html#62769</a><br>
>> >> Then you can go through the messages non-linearly. Right now, the<br>
>> >> basic idea is to follow the steps outlined in the open project ticket.<br>
>> >> I think Christian has summarized it nicely in his recent email [1]: "I<br>
>> >> think the contributions from this project that would add value would<br>
>> >> be:<br>
>> >> 1. Finding a tool and a configuration that can do an RTEMS style or an<br>
>> >> acceptable close one.<br>
>> >> 2. Adding a "check-style" target to our build system.<br>
>> >> 3. Maybe add some kind of script similar to Linux "<a href="http://checkpatch.pl" rel="noreferrer" target="_blank">checkpatch.pl</a>" that<br>
>> >> could check whether patches would need changes _before_ they are<br>
>> >> applied.<br>
>> >> "<br>
>> >><br>
>> >> The proposal preparation phase should work through identifying the<br>
>> >> options and pros/cons for different tools while preparing a plan for<br>
>> >> how to integrate style checks in 2, 3 and thinking through the coding<br>
>> >> tasks for the summer.<br>
>> >><br>
>> >> Getting the style checking tool's configuration to match with the<br>
>> >> RTEMS style will be some effort, and testing it out and submitting<br>
>> >> some patches based on it could be a good proposal activity also to<br>
>> >> build some confidence about the tools that will be used.<br>
>> >><br>
>> >> We also have some Python style guidelines that might be worth<br>
>> >> addressing. Those are harder maybe, since the style refactoring might<br>
>> >> be challenging to review for correctness.<br>
>> >><br>
>> >> For getting started, I would recommend that you try running uncrustify<br>
>> >> with the given configuration on some files in RTEMS, see what it<br>
>> >> results in. Play around.<br>
>> >><br>
>> >> [1] <a href="https://lists.rtems.org/pipermail/devel/2021-March/065547.html" rel="noreferrer" target="_blank">https://lists.rtems.org/pipermail/devel/2021-March/065547.html</a><br>
>> >><br>
>> >> -Gedare<br>
>> >><br>
>> >> > Thanks,<br>
>> >> > Ida<br>
>> >> ><br>
>> >> > On Mon, Mar 15, 2021 at 9:39 PM Gedare Bloom <<a href="mailto:gedare@rtems.org" target="_blank">gedare@rtems.org</a>> wrote:<br>
>> >> >><br>
>> >> >> See the related thread, and we'll have to discuss how to move forward.<br>
>> >> >> The existing approach provides an uncrustify script:<br>
>> >> >> <a href="https://lists.rtems.org/pipermail/devel/2020-October/062769.html" rel="noreferrer" target="_blank">https://lists.rtems.org/pipermail/devel/2020-October/062769.html</a><br>
>> >> >><br>
>> >> >><br>
>> >> >> On Sun, Mar 14, 2021 at 9:47 PM Ida Delphine <<a href="mailto:idadelm@gmail.com" target="_blank">idadelm@gmail.com</a>> wrote:<br>
>> >> >> ><br>
>> >> >> > Hello everyone,<br>
>> >> >> > This ticket(<a href="https://devel.rtems.org/ticket/3860" rel="noreferrer" target="_blank">https://devel.rtems.org/ticket/3860</a>) was proposed to me and I'm interested in it for GSoC.<br>
>> >> >> > The first task there is to find a code checker or formater that can produce results that match the RTEMS coding conventions. It also made mention some tools have been discussed in the past. Please I will love suggestions on possible tools I could use to achieve this.<br>
>> >> >> ><br>
>> >> >> ><br>
>> >> >> > Cheers,<br>
>> >> >> > Ida.<br>
>> >> >> > _______________________________________________<br>
>> >> >> > devel mailing list<br>
>> >> >> > <a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
>> >> >> > <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div>