<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Apr 18, 2021, 5:29 PM Chris Johns <<a href="mailto:chrisj@rtems.org">chrisj@rtems.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 16/4/21 3:28 pm, Sebastian Huber wrote:<br>
> On 15/04/2021 22:58, Gedare Bloom wrote:<br>
>> I guess we should define something to add here? Is anything written<br>
>> somewhere we can adopt?<br>
> the spirit of the Python guide is to have rules which are enforced by tools.<br>
<br>
I see a guide as a means to provide robust and portable code in a common style<br>
where we document and learn from our experiences. Any forced line like needing a<br>
tool would create an artificial barrier where needed rules are not added because<br>
we do not have a tool. I doubt any of us would want that to happen.<br>
<br>
> If<br>
> a certain component should run on platform X, then I think some CI system should<br>
> be in place which runs the test suite on platform X.<br>
<br>
This is hard to do, costs money and is only as good as a tests. It is important<br>
but we should also use reviews as a way to catch issues.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Does the PEP cover this? If so, then some tool should exist to help. Otherwise, we need some heuristics to at least flag an inspection.</div><div dir="auto"><br></div><div dir="auto">I had a shell script years ago to check BSPs for some basic rules and fixed a lot.  It was about the time we tried to be more standardized on file names in BSPs and some methods would always be in a file of a particular name. Not the brightest and hardest rules but the script could flag what needs to be inspected.</div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
>> Was this considered at all under the pre-qual effort (which stimulated<br>
>> these guidelines in the first place)? Do we have a Windows blind spot<br>
>> in our Python code bases?<br>
> The Python modules in rtems-central should run on Windows, however, I never<br>
> tested this.<br>
<br>
A random pick of a file and a brief scan ...<br>
<br>
<a href="https://git.rtems.org/rtems-central/tree/rtemsspec/interface.py#n313" rel="noreferrer noreferrer" target="_blank">https://git.rtems.org/rtems-central/tree/rtemsspec/interface.py#n313</a><br>
<br>
For generating code it is important to make sure you use the platform's line<br>
separator or git will get upset.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Ok. And a grep would find these. Is this part of PEP?</div><div dir="auto"><br></div><div dir="auto">Are there files which should never have \n? All? Some? </div><div dir="auto"><br></div><div dir="auto">Is there other things that can be enforced now or we build some script which has a number of rules which we grow.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
:)<br>
<br>
Chris<br>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org" target="_blank" rel="noreferrer">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div></div></div>