[PATCH 18/18] config: Add <rtems/confdefs/obsolete.h>

Joel Sherrill joel at rtems.org
Sat Feb 15 14:16:13 UTC 2020


On Sat, Feb 15, 2020, 2:38 AM Sebastian Huber <
sebastian.huber at embedded-brains.de> wrote:

> On 15/02/2020 05:22, Gedare Bloom wrote:
>
> > This makes sense. Is there any reason for the ordering?
> It should be alphabetically sorted.
> >
> > If possible, I think grouping by obsoleted version or alphabetical
> > ordering would be a good idea.
>
> Maybe I should add a comment like this to the top of the list:
>
> /*
>   * Please keep the list of obsolete configuration options
> alphabetically sorted.
>   *
>   * Use #warning for renamed options and define the new option accordingly.
>   *
>   * Use #warning for obsolete options which are now superfluous, e.g.
> because
>   * the objects are now self-contained.
>   *
>   * Use #error for options which require now a different configuration
> approach,
>   * e.g. options for an own configuration table.
>   *
>

And note when perhaps with a version when an option was deemed obsolete.
You might be able to get that from git blame in the future but for now we
just destroyed all that info for blame by creating a new file.

>
> _______________________________________________
> 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/20200215/a6d346d5/attachment.html>


More information about the devel mailing list