[PATCH 6-freebsd-12 3/4] build: Separate the kernel and user land include paths

Sebastian Huber sebastian.huber at embedded-brains.de
Tue Sep 15 11:03:45 UTC 2020


On 15/09/2020 13:01, Chris Johns wrote:

>> On 15 Sep 2020, at 5:51 pm, Sebastian Huber<sebastian.huber at embedded-brains.de>  wrote:
>>
>> On 15/09/2020 04:33,chrisj at rtems.org  wrote:
>>
>>> -    def get_includes(self):
>>> +    def getIncludes(self):
>> I am not sure which coding style the file uses in general, but pylint complains about camelCase function names.
> I did not run pylint, I only ran yapf. I followed the convention in the file. Is this wrong?
Before I used pylint, I used CamalCase for Python code, but then learned 
that pylint is not happy about this.
>
> This code does need pylint run on it but I do not have the time.

Yes, no problem. It is just something I noticed. The patch is fine as it is.



More information about the devel mailing list