[PATCH] eng: Add build system chapter
Chris Johns
chrisj at rtems.org
Fri Nov 8 22:48:05 UTC 2019
On 9/11/19 12:56 am, Sebastian Huber wrote:
> Update #3818.
> ---
>
> This is the first draft of the documentation of the new build system from the
> RTEMS maintainer perspective.
Well done. This this effort is looking nice.
> I work the documentation for the user manual next week. The next step is to
> convert all tests to the new build system (except Ada tests). The final step is
> to convert all BSPs.
>
> I removed the binary files from the patch. For the generated document see:
>
> https://ftp.rtems.org/pub/rtems/people/sebh/eng.pdf
Looks great. I have a few minor points.
1. Is maintaining the existing installed layout and file names a requirement? I
think it is. I suggest adding this to the list at the start.
2. Fig 1 and 2 could be zoomed a little more for the PDF output.
3. Is the ${} expansion something you have implemented? It is a nice feature.
4. Are `links:` and `review:` a base64 format or something else? (I am happy to
see the more compact base64 format being used) Should the items controlled by
Doorstop be listed and lablled as "do not touch"?
Chris
More information about the devel
mailing list