[PATCH] CONTRIBUTING: Warn about white space changes
Joel Sherrill
joel at rtems.org
Wed Jun 8 13:40:54 UTC 2022
This looks good. I would have said "not reformat code" but that's ok.
--joel
On Wed, Jun 8, 2022 at 7:57 AM Sebastian Huber <
sebastian.huber at embedded-brains.de> wrote:
> ---
> CONTRIBUTING.rst | 12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/CONTRIBUTING.rst b/CONTRIBUTING.rst
> index 79a42db1..36cd91de 100644
> --- a/CONTRIBUTING.rst
> +++ b/CONTRIBUTING.rst
> @@ -279,6 +279,12 @@ original FreeBSD code.
> Rules for Modifying FreeBSD Source
> ==================================
>
> +Do not format original FreeBSD code. Do not perform white space changes
> even
> +if you get git commit warnings. **Check your editor settings so that it
> +does not perform white space changes automatically**, for example adding a
> +newline to the end of the file. White space changes may result in
> conflicts
> +during updates, especially changes at the end of a file.
> +
> Changes in FreeBSD files must be done using ``__rtems__`` C pre-processor
> guards.
> This makes synchronization with the FreeBSD upstream easier and is very
> important. Patches which do not follow these rules will be rejected.
> Only add
> @@ -380,12 +386,6 @@ In general, provide empty header files and do not
> guard includes.
> For new code use
> `STYLE(9) <
> http://www.freebsd.org/cgi/man.cgi?query=style&apropos=0&sektion=9>`_.
>
> -Do not format original FreeBSD code. Do not perform white space changes
> even
> -if you get git commit warnings. Check your editor settings so that it
> doesn't
> -perform white space changes automatically, for example adding a newline
> to the
> -end of the file. White space changes may result in conflicts during
> updates,
> -especially changes at the end of a file.
> -
> Automatically Generated FreeBSD Files
> =====================================
>
> --
> 2.35.3
>
> _______________________________________________
> devel mailing list
> devel at rtems.org
> http://lists.rtems.org/mailman/listinfo/devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/devel/attachments/20220608/22b3227d/attachment.htm>
More information about the devel
mailing list