[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: create_archive_ia32.pl (ddf)

From: Daniel Wagner
Subject: Re: create_archive_ia32.pl (ddf)
Date: Sun, 20 Jun 2004 17:17:20 +0200
User-agent: Wanderlust/2.10.1 (Watching The Wheels) SEMI/1.14.6 (Maruoka) FLIM/1.14.6 (Marutamachi) APEL/10.6 Emacs/21.3 (i386-pc-linux-gnu) MULE/5.0 (SAKAKI)

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

> +# 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
OUTPUT_FORMAT("elf32-i386", "elf32-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.  


reply via email to

[Prev in Thread] Current Thread [Next in Thread]