[PATCH 4/6] bsps: Move version.c and use bspopts.h

Chris Johns chrisj at rtems.org
Wed Apr 4 08:10:58 UTC 2018


On 04/04/2018 18:03, Sebastian Huber wrote:
> On 04/04/18 09:55, Chris Johns wrote:
>>> I think we need only three Makefile.am and two configure.ac:
>>>
>>> configure.ac
>>> Makefile.am (builds also cpukit, uses AM_CONDITIONAL for cpukit/score/cpu/*)
>>> testsuites/Makefile.am (uses AM_CONDITIONAL for each test)
>> 500+ tests in the one file? Really? I hope not.
> 
> Is this better or worse than 794 Makefile.am in testsuites?
> 

Worse, it becomes a conflict hot spot. One per functional group of tests may be
manageable.

>>
>> You cannot have more than one test in a Makefile.am currently. See $(make-exe)
>> and LINK_OBJS and LINK_LIBS.
> 
> Yes, but if we remove the post link step,

That is gone for me.

> then we can probably use the standard Automake program stuff.

Hmmm sure, I will have play and let you know. I am in a position to do this.

> If not, then I will not touch the testsuites build at all and leave it as is.

It depends on lots above and around the place.

Chris



More information about the devel mailing list