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

Chris Johns chrisj at rtems.org
Wed Sep 16 01:43:28 UTC 2020


On 16/9/20 12:48 am, Joel Sherrill wrote:
> On Mon, Sep 14, 2020 at 9:33 PM <chrisj at rtems.org <mailto:chrisj at rtems.org>> wrote:
> 
>     From: Chris Johns <chrisj at rtems.org <mailto:chrisj at rtems.org>>
> 
>     - Provide support for separate user and kernel include paths in
>       libbsd.py.
> 
>     - Update all added files with a suitable context to build them
>       with. Supported contexts are `kernel` and `user`.
> 
>     - Kernel source use the kernel, CPU, and build header paths in
>       this order.
> 
>     - User source use the user, kernel, CPU and build header paths
>       in this order. The FreeBSD /usr/include tree has some kernel
>       header files installed as well as user land header files. This
>       complicates the separation as some kernel header files are not
>       visible to user land code while other are. This is handled by
>       appeanding the kernel header paths to the user header paths so
> 
> 
> ^^^ appending

Thanks.

> Otherwise, I am just asking does Python code need to pass. pylint
> and yapf are mentioned in this thread but 
> https://books.agiliq.com/projects/essential-python-tools/en/latest/linters.html
> has a list of tools and claims that pycodelint matches pep8. I thought we 
> were trying to follow pep8. 

I thought yapf did pep8. I followed the 6.5.2 guide lines.

> It also mentions pyflake which sounds useful.
> 
> What is the recommended set of tools to pass and with what settings?
> 
> I know we aren't completely passing on all Python code but what's the
> objective from a tool checking viewpoint.

This is existing code and I did not think we needed to pass pylint and yapf to
make changes. I did pass the code through yapf as a separate patch to help the
process. See 1/4. Pylint is more work than I have time avalable to do on this code.

Chris


More information about the devel mailing list