[rtems commit] spec/librtemscpu: Fix installed headers

Chris Johns chrisj at rtems.org
Wed Aug 31 00:38:17 UTC 2022


On 31/8/2022 4:57 am, Sebastian Huber wrote:
> On 30/08/2022 19:55, Joel Sherrill wrote:
>> On Tue, Aug 30, 2022 at 8:12 AM Sebastian Huber
>> <sebastian.huber at embedded-brains.de
>> <mailto:sebastian.huber at embedded-brains.de>> wrote:
>>
>>     On 22/08/2022 01:30, Chris Johns wrote:
>>      > - keyimpl.h is not referenced and so not needed
>>
>>     We install all the *impl.h header files. Why should keyimpl.h be an
>>     exception?
>>
>>
>> I'd lean to installing all of them for consistency.
>>
>> I have written introspection code for applications which lived external to
>> rtems itself. it needed to peek inside.
> 
> Yes, I found it occasionally helpful to have all the header files installed. For
> example to write diagnostic and debug code for a particular issue.

I removed it as part of an audit of what failed to compile with C++. Because it
is "internal" it was not directly included so it was removed. I suspect it
failed when included in C++.

I am not sure what happens here. There are valid points for both cases.

Thanks
Chris


More information about the devel mailing list