[PATCH 01/31] shell: Use #include "..." for local header files
Sebastian Huber
sebastian.huber at embedded-brains.de
Tue Oct 9 13:23:39 UTC 2018
On 09/10/2018 14:50, Chris Johns wrote:
> On 8/10/18 10:35 pm, Sebastian Huber wrote:
>> On 09/10/2018 03:41, Chris Johns wrote:
>>> On 8/10/18 3:43 am, Sebastian Huber wrote:
>>>> Update #3375.
>>> Why?
>> To keep the count of command line arguments small. These three header files are
>> used only in the shell directory. They are not installed.
> Thanks, I see the issue now. This is a limitation of building this way. Waf
> allows options to added based on grouping of source.
>
>> The <err.h> and <fts.h> are candidates for Newlib. It is a bit of work to make
>> them available via Newlib since they are already provided by Cygwin. This patch
>> set is a preparation for this:
>>
>> https://sourceware.org/ml/newlib/2018/msg01004.html
> Nice.
>
>>> Some of these files are imported ....
>>>
>>>> ---
>>>> cpukit/libmisc/Makefile.am | 1 -
>>>> cpukit/libmisc/shell/cmp-ls.c | 2 +-
>>> ... for this example this file.
>> I don't think the changed #include statements will cause major problems if
>> someone decides to merge upstream patches.
> Yes I agree. The change is fine.
Thanks for the review, can I check in the complete patch set?
--
Sebastian Huber, embedded brains GmbH
Address : Dornierstr. 4, D-82178 Puchheim, Germany
Phone : +49 89 189 47 41-16
Fax : +49 89 189 47 41-09
E-Mail : sebastian.huber at embedded-brains.de
PGP : Public key available on request.
Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.
More information about the devel
mailing list