Jivin Mike Frysinger lays it down ...
> On Wednesday 08 July 2009 20:56:20 David McCullough wrote:
> > Jivin Mike Frysinger lays it down ...
> >
> > > On Tue, Jul 7, 2009 at 06:43, David McCullough wrote:
> > > > Jivin Mike Frysinger lays it down ...
> > > >
> > > >> Due to shell portability issues (which is to say shell scripts are not
> > > >> portable -- think Windows), convert elf2flt to C code.
> > > >>
> > > >> I've updated this code base to the latest elf2flt tree and actually
> > > >> done some basic tests -- building the three Blackfin tuples (ELF,
> > > >> FLAT, FDPIC) and running programs on my Blackfin boards.  This process
> > > >> found errors in the original implementation as well as some of the
> > > >> cleanups I did.
> > > >>
> > > >> Signed-off-by: Nathan Sidwell <nat...@codesourcery.com>
> > > >> Signed-off-by: Mike Frysinger <vap...@gentoo.org>
> > > >
> > > > Rather than rename ld-elf2flt.in I merged dropped patch 4 and made some
> > > > changes to patch 6 and fixed a logic error with the configure
> > > > use_ld_elf2flt_script logic so that it is not inverted.
> > > >
> > > > Try this patch against current CVS and see if it's ok for you,
> > >
> > > the reason i liked renaming ld-elf2flt script to .sh is that you could
> > > run configure by default and end up with both the binary and the shell
> > > script.  if something was wonky, this lets you quickly test either one
> > > without having to re-run configure to switch modes.  i'm not terribly
> > > set on keeping this functionality, just pointing out that it was done
> > > this way on purpose.
> >
> > And some of why I didn't rename it is just leaving the CVS history die with
> > the original file.  Of course the rename to ld-elf2flt.in probably ate that
> > already :-(
> 
> ah, that does suck.  time to move to git ! :)
> 
> > I just figured that this will be mostly temporary and that the script will
> > get dropped due to being defaulted to off and bit rotting and .....
> 
> hmm, i'd forgotten about an oft unused config.status feature ... how about 
> adding a dedicated rule that looks like:
> ld-elf2flt.sh: $(srcdir)/ld-elf2flt.in
>       ./config.status --file=$@:$<
> 
> this would allow people debugging things to manually run `make ld-elf2flt.sh` 
> and get the right thing

Even better :-)

        AC_OUTPUT(ld-elf2flt.sh:ld-elf2flt.in Makefile elf2flt.ld)

lets us keep the old name and generate both all the time, with other fixups
of course.  We can work out what I have missed once it's committed ;-)

Cheers,
Davidm


-- 
David McCullough,  david_mccullo...@securecomputing.com,  Ph:+61 734352815
McAfee - SnapGear  http://www.snapgear.com                http://www.uCdot.org
_______________________________________________
uClinux-dev mailing list
uClinux-dev@uclinux.org
http://mailman.uclinux.org/mailman/listinfo/uclinux-dev
This message was resent by uclinux-dev@uclinux.org
To unsubscribe see:
http://mailman.uclinux.org/mailman/options/uclinux-dev

Reply via email to