[LLVMdev] Build issues on Solaris

Kenneth Boyd zaimoni at zaimoni.com
Wed Aug 19 08:02:35 PDT 2009


Duncan Sands wrote:
> Hi Nathan,
>
>   
>> The latter being what GenLibDeps.pl expects to see. All else being  
>> equal though, it might be better to change GenLibDeps.pl to recognize  
>> the first version:
>> --- GenLibDeps.pl       (revision 78653)
>> +++ GenLibDeps.pl       (working copy)
>> @@ -100,7 +100,7 @@
>>       print "  <dt><b>$lib</b</dt><dd><ul>\n";
>>     }
>>     open UNDEFS,
>> -    "$nmPath -u $Directory/$lib | sed -e 's/^[ 0]* U //' | sort |  
>> uniq |";
>> +    "$nmPath -u $Directory/$lib | sed -e 's/^[ 0]* U //' -e 's/^ *//'  
>> | sort | uniq |";
>>     my %DepLibs;
>>     while (<UNDEFS>) {
>>       chomp;
>>     
>
> starting at line 123 there seems to be a by-hand version of the
> sed command.  I guess this needs to be adjusted too.  Probably
> sed can be used nowadays.
>   
I'd adjust it to be able to run in the first place (attaching patch).

It's intended to run exactly when nm fails to find dependencies, so 
die'ing out is pointless before it runs.  If allowed to run, this 
supports Perls whose native C system command shell is cmd.exe: 
Strawberry Perl, ActivePerl, and any Perl built for Windows *other* than 
the MingW32 pre-built binaries.  (cmd.exe chokes on the pipe into sed, 
causing the whole construct to fail.)

If the first nm finds any dependencies at all, the by-hand version won't 
run.  It's currently optimized for MingW32 nm.

Kenneth
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: GenLibDeps.pl.patch
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20090819/28dca16b/attachment.ksh>


More information about the llvm-dev mailing list