On Sunday 03 January 2010 16:32:39, Danny Backx wrote: > On Sun, 2010-01-03 at 12:11 +0000, Pedro Alves wrote: > > On Saturday 02 January 2010 16:42:23, Danny Backx wrote: > > > On Fri, 2010-01-01 at 16:24 +0000, Pedro Alves wrote: > > > > FYI, I hadn't applied the ld patch myself because I was > > > > looking to confirm/hear if there's another cleaner way > > > > to get at the image base, but I can't find one. Anyway, I've > > > > now commited the mingw/pseudo-reloc patch. (A bit cleaned up from > > > > yesterday's, reducing divergence from upstream mingw). > > > > > > Looks like I committed only part, apologies. > > > > Sounds like the message came across with phase inverted. > > Oh well. I tried a SVN commit, bumped into a conflict (needed to do SVN > up), did that, fixed everything, and then must have forgotten to do > commit again.
:-) I meant, I was surprised that you had applied that patch yourself. There was a reason I hadn't applied the patches myself. I mean, why wouldn't I, if I had thought the patches were final? Example, notice that the final mingw patch I applied wasn't even the patch you tried to commit. I guess this is sounding overly pedantic by now... Oh well, x2. > The good thing about Windows Mobile is still that it's mobile. The bad > thing is it's still Windows :-( :-) > > I tried to look at the binutils patches that you've applied > > to svn, but they're all kind of mixed up. > > I will spend the near future trying to clear things up. Do you (or Kai, if you're still listening :-) ) still have Kai's original patch that corresponds to this? +2009-12-06 Danny Backx <dannyba...@users.sourceforge.net> + + * ld/emultempl/pe.em, ld/scripttempl/armcoff.sc, + ld/scripttempl/pep.sc, bfd/peXXigen.c: Work by Kai Tietz to fix + so-called version 2 pseudo relocation. + * ld/scripttempl/pe.sc: port Kai's work to the pe.sc file. + * ld/scripttempl/pe.sc: move the BSS stuff inside the .data section + instead of in its own .bss . This appears to be a Windows Mobile 6.1 + requirement. + * bfd/coff-arm.c, bfd/libpei.h: Avoid calling the + coff_arm_final_link_postscript, we need + bfd_coff_final_link_postscript. + * bfd/cofflink.c: Don't use bfd_coff_link_output_has_begun(), it forgets + to output the debug_info field of the first file linked. That's revision 1402 in our svn. I see this discussion <http://sourceware.org/ml/binutils/2009-01/msg00036.html> about it, but can't find the original of that patch. I'm suspecting that the .bss -> .data change, if required at all, is masking something else. But I'd like to confirm that it's really necessary in the end. Anyway, I had a bit of time to work on this today, but "today" is mostly gone by now. > Getting things to work was quite a journey. I needed to put this in > SVN while progressing so others - like you - had less trouble kicking > in when I needed it. That made the mess visible. Well, that's what svn branches are for. ;-) -- Pedro Alves ------------------------------------------------------------------------------ This SF.Net email is sponsored by the Verizon Developer Community Take advantage of Verizon's best-in-class app development support A streamlined, 14 day to market process makes app distribution fast and easy Join now and get one step closer to millions of Verizon customers http://p.sf.net/sfu/verizon-dev2dev _______________________________________________ Cegcc-devel mailing list Cegcc-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/cegcc-devel
