Re: Adding support for VxWorks target

Stan Shebs
2010-05-04T15:43:12Z

Joel Brobecker wrote:
> Hello,
>
>
>> The following series of patches adds support for cross debugging
>> VxWorks targets (version 5.x, 6.x and limited support for VxWorks 653).
>> It's a lot of code overall, and many patches, but it is almost
>> completely contained in dedicated files, and we've split the changes
>> in modules/patches, hoping to make it easier for anyone wanting to
>> look at them.
>>
>
> Just wondering what the general sentiment is towards these patches?
> I just did a resync at AdaCore, and noticed a break due to the various
> attribute changes. It's not big deal to fix those, but now I have to
> do keep two sets of patches up to date...
>

I skimmed, and didn't see anything problematic. Organizationwise, my
personal preference would to be have fewer files that are "everything
wtx", which in turns reduces the amount of headers / header files. It
also forestalls some grumbles about long filenames from DOS-land. :-)
But don't feel compelled to glom them together, if you prefer multiple
files.

Stan