On Thursday 02 July 2009 20:46:59 John Williams wrote: > On Fri, Jul 3, 2009 at 3:51 AM, Mike Frysinger <vap...@gentoo.org> wrote: > > Due to shell portability issues (which is to say shell scripts are not > > portable -- think Windows), convert elf2flt to C code. > > Honestly my gut reaction to this is "yuck". Not your implementation, just > the idea in general. C is great for lots of things, but string-heavy > control flow driving external utilities surely isn't one of them.
i agree, but facing the lack of any saner alternatives, here we are. i'm certainly open to different ideas. i have tried to unify/abstract as much of the code as possible so that the real meat of it (processing the linker script and customizing options) remains easy to extend. e.g. the only functions people should have to touch in general are parse_args() and do_final_link(). the rest is code that should largely remain unchanged. > Maybe I'm biased because I also have a pretty lukewarm feeling about doing > embedded Linux development in a Windows environment. Seems like a great > way to ensure a mediocre output from a product team, but anyway... while i agree, my opinion doesnt matter terribly much. the customers are paying the bills and the customers get what they're going to pay for. they arent going to pay for "let's ditch windows completely". i ever mention the time i worked with a customer who maintained the linux kernel source in Visual Source Safe and used Visual C++ for their IDE ? and were willing to pay significant dime to continue with said setup ? ;) > If I was to vote for (or had time for) serious labour investment in elf2flt > it would be a cleanup and refactor of the horrible "do_relocs" function in > elf2flt itself - that thing is torture. since i added -Wall to configure, i noticed some not too friendly warnings coming from that code. like "variable used uninitialized in this function". -mike
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ 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
