<div dir="auto">Sorry about forgetting the ticket. My main point was do we want to do this before branching or not?<div dir="auto"><br></div><div dir="auto">There are a lot of tickets outstanding and we need to start working to an end.</div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Mar 28, 2018, 3:45 PM Chris Johns <<a href="mailto:chrisj@rtems.org">chrisj@rtems.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 29/03/2018 00:26, Gedare Bloom wrote:<br>
> On Wed, Mar 28, 2018 at 1:16 AM, Sebastian Huber<br>
> <<a href="mailto:sebastian.huber@embedded-brains.de" target="_blank" rel="noreferrer">sebastian.huber@embedded-brains.de</a>> wrote:<br>
>> On 28/03/18 02:11, Chris Johns wrote:<br>
>>><br>
>>> On 28/03/2018 07:09, Joel Sherrill wrote:<br>
>>>><br>
>>>> On Tue, Mar 27, 2018 at 2:56 PM, Cudmore, Alan P. (GSFC-5820)<br>
>>>> <<a href="mailto:alan.p.cudmore@nasa.gov" target="_blank" rel="noreferrer">alan.p.cudmore@nasa.gov</a> <mailto:<a href="mailto:alan.p.cudmore@nasa.gov" target="_blank" rel="noreferrer">alan.p.cudmore@nasa.gov</a>>> wrote:<br>
>>>><br>
>>>>      I like the idea, but to me it would be confusing to have bsp and<br>
>>>> bspkit at<br>
>>>>      the top level, each having nearly the same directories. (but<br>
>>>> certainly less<br>
>>>>      confusing than before)____<br>
>>>><br>
>>>>      At the risk of introducing more work, would it make sense to have<br>
>>>> bspkit<br>
>>>>      with inc and src subdirectories?<br>
>>>><br>
>>> Alan, this is a good point and highlights a weakness in the current<br>
>>> positioning<br>
>>> of the bsp includes. In this context it would seem to be a mistake to have<br>
>>> the<br>
>>> includes under 'bsps' because it leads to a duplicated name of some type<br>
>>> at the<br>
>>> top level. A 'bsps/include' directory would make the BSPs consistent with<br>
>>> 'cpukit/include' plus it is a sensible path.<br>
>><br>
>><br>
>> I thought the structure is clear now. Everything BSP related should go to<br>
>> "bsps" with public headers in "include" subdirectories. This is the same<br>
>> structure we have in "cpukit". Where do you see duplicated names?<br>
>><br>
<br>
Yes, I missed the include directory when I looked. Sorry for the noise.<br>
<br>
>>><br>
>>>> Yeah. I forgot to mention that we would have to address that. Having a<br>
>>>> directory<br>
>>>> that is "pure installed .h" files is desirable and we would have to<br>
>>>> address having<br>
>>>> two similarly named directories.<br>
>>><br>
>>> I think a tree plan needs to be made before we move any more code about to<br>
>>> avoid<br>
>>> moving twice.<br>
>><br>
>><br>
>> I thought we already have a plan.<br>
>><br>
>> <a href="https://devel.rtems.org/ticket/3285" rel="noreferrer noreferrer" target="_blank">https://devel.rtems.org/ticket/3285</a><br>
>><br>
<br>
Sorry, I had forgotten we did.<br>
<br>
>>> As a starting point for a discussion here is a list:<br>
>>><br>
>>> - Removing 'c' moving all contents to other parts of the tree<br>
>>>    - bsps/. => bsps/include<br>
>><br>
>><br>
>> We already  have a bsps/include?<br>
>><br>
>>>    - c/src/lib/libbsp/. => bsps/<br>
>><br>
>><br>
>> Yes, but please with a common structure for all BSPs as defined by<br>
>><br>
>> <a href="https://devel.rtems.org/ticket/3285" rel="noreferrer noreferrer" target="_blank">https://devel.rtems.org/ticket/3285</a><br>
>><br>
>>>    - c/src/ada => ada<br>
>><br>
>><br>
>> There is no c/src/ada.<br>
>><br>
<br>
I should clean my tree of removed directories.<br>
<br>
>>>    - c/src/support/verison.c => cpukit/sapi/version-string.c<br>
>><br>
>><br>
>> Is the RTEMS_BSP define available in cpukit? If yes, then this is a bug.<br>
>><br>
Maybe not, I did not look into the code to see what it depended on.<br>
<br>
Where to ... bsps/shared/ ?<br>
<br>
>>>    - c/src/<a href="http://bsp.pc.in" rel="noreferrer noreferrer" target="_blank">bsp.pc.in</a> => bsps/<br>
>>>    - c/src/libchip => devices (not sure about this)<br>
>><br>
>><br>
>> This should move to bsps/shared/dev/*<br>
<br>
Sure.<br>
<br>
>><br>
>>> - Rename cpukit because *kit means nothing<br>
>>>    - cpukit => kernel<br>
>><br>
>><br>
>> No, please lets not do this. This makes searching the commit history more<br>
>> difficult just for the sake of some cosmetic naming change. Removing the<br>
>> preinstall step or getting rid of three directory levels is a different<br>
>> story.<br>
>><br>
> +6 (In concurrence with Sebastian.)<br>
><br>
<br>
Agreed, I had forgotten what happens with moves in git. Shame cause the name is<br>
not relevant any more.<br>
<br>
Chris<br>
</blockquote></div>