[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