On Wed, Jan 30, 2008 at 06:57:34AM +0100, Tristan Gingold wrote: > On Tue, Jan 29, 2008 at 03:46:50PM +0100, Marco Gerards wrote: > > Robert suggested some changes. I also noticed in the discussion that > > you didn't follow common practise (like an existing grub-mkimage > > implementation). Personally, as maintainer, I am against maintaining > > two different approaches of the same problem. It costs us time to > > maintain this and both case have separate bugs. So I rather see the > > code shared, one way or the other. To safe time, it might save you > > some work if you initiate a discussion about this. > > The facts: > > util/i386/efi/grub-mkimage.c and elf2pe.c do almost the same thing: converting > an ELF file to a PE file. But: > > * the input format is slightly different (Elf32 vs Elf64) > * the output format is slightly different (PE vs PE+) > * the relocating work is different (x86 vs ia64, Rel vs Rela) > * The entry point is a descriptor on ia64 > * grub-mkimage.c can insert modules in a section.
Sounds like they're very different. Maybe it'd be fine to just change the name and user interface from elf2pe.c to util/ia64/efi/grub-mkimage.c ? What do you think? -- Robert Millan <GPLv2> I know my rights; I want my phone call! <DRM> What use is a phone call… if you are unable to speak? (as seen on /.) _______________________________________________ Grub-devel mailing list Grub-devel@gnu.org http://lists.gnu.org/mailman/listinfo/grub-devel