[Bug 1675] mcf5329 BSP post-link conversion broken

[bugzilla-daemon at rtems.org]

https://www.rtems.org/bugzilla/show_bug.cgi?id=1675

--- Comment #7 from Chris Johns <chrisj at rtems.org> 2010-08-17 01:24:03 CDT ---
(In reply to comment #6)
> (In reply to comment #5)
> > (In reply to comment #4)
> > > We should abandon all these make hooks in the long run, since they don't
> > > integrate well into Automake and other build systems.  
> > 
> > The hooks are fine with autoconf. I would like to see them stay. They could
> > call a script.
> Firstly, we are talking about automake, not autoconf.
> 

Yes you are correct, it is automake.

> Secondly, I hate these hooks.

I do not have a strong preference either way. I hate the '.exe' extension much
more than the hooks. :)

> 
> > > It would be better to
> > > provide scripts that can be reused more conveniently.
> > 
> > Moving the logic to a patch is fine if you can solve the problem of managing
> > lots of script for lots of BSPs, documenting them, enforcing some sort of
> > standard, and creating them for all existing BSPs.
> The point Sebastian has brought up (again), isn't new. It has been haunting
> RTEMS for a very long time and reappears every now and then, whenever somebody
> is facing the limitations of this approach.
> 
> I recall me having proposed to remove the hooks and to replace them with
> scripts, many years ago (probably >> 10 years) and me actually having worked on
> them, until something was introduced into RTEMS which broke this approach (I
> don't recall what it was).

Would it be possible for you to describe the basic approach ?

My concern with scripts is getting the specific information for specific script
related to the BSP. I am looking at the simulator scripts in rtems-testing.
They are a great resource but there are a lot of them and they all require
maintenance, testing, documentation etc.

> 
> May-be it's time to revisit this issue once again.

Sure lets do this. Maybe this PR should be renamed and I will commit a fix for
this BSP.

-- 
Configure bugmail: https://www.rtems.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.