Re: create_archive_ia32.pl (ddf)

[Daniel Wagner]

Hi Etienne

Thanks for your patch.  As I see you have used not the cvs version.
Well, I guess that's my fault.  I have forgotten to announce that the
formaly known ddf, now fabrica, can be access through the savannah cvs
server.

> +# Maintain backward compatibility
> +use 5.004;

I don't think that's really needed.

> I've added the ability to retrieve OUTPUT_FORMAT|OUTPUT_ARCH from the
> linker, but as a side-effect, it might not work for every systems,
> depending on whether your GNU ld returns OUTPUT_FORMAT with some \"foo\"
> separators or not.

It does not work for my system (debian, unstable).  

> +my @tmp              = grep(/$ARCH/, `ld --verbose`);
> +$OUTPUT_FORMAT       = join('',$tmp[1],$tmp[2]);

address@hidden:~/src/hurd/fabrica/fabrica-archive$ ld --verbose  | grep i386
   elf_i386
   i386linux
OUTPUT_FORMAT("elf32-i386", "elf32-i386",
              "elf32-i386")
OUTPUT_ARCH(i386)
SEARCH_DIR("/usr/i386-linux/lib"); SEARCH_DIR("/usr/lib"); SEARCH_DIR("/usr/l   
l/lib"); SEARCH_DIR("/lib");

As you can see ld reports the interesting info on line 3 and 4 instead
of 2 and 3.  I think you should grep for OUTPUT_FORMAT and OUTPUT_ARCH 
to make it more robust. 

> Also, I've changed the output filename to return
> per-architecture ("ddf-archive-script-$ARCH") names instead of
> "ddf-archive-script".

That doesn't work with the Makefile.  See how create_archive_x86.pl is
invoked and then how ld is invoked.

I have applied the other improvements.  

Thanks,
daniel