Kenny,

Just looking at the libtool-related changes:

usr/src/cmd/libtool/Makefile.sfw:

   43 all64: $(VER64)/config.status
   44         (cd $(VER64); env - \
   45             CC=$(CC64) \
   46             CFLAGS="$(CFLAGS64)" \
   47             CPPFLAGS="-m64" \
   48             CXX="g++ -m64" \
   49             CXXFLAGS="-m64" \
   50             FFLAGS="-m64 -w" \
   51             LDFLAGS="-L$(ROOTSFWLIB)/$(MACH64) -L$(ROOTLIB64)" \
   52             LD_OPTIONS="-M $(SRC)/cmd/mapfile_noexstk" \
   53             PATH=$(SFW_PATH) \
   54             MAKE=$(CCSMAKE) \
   55             $(CCSMAKE))

Do you need to set the C++ and Fortran flags here?  If they're
not needed to 32-bit, I'd be surprised if they're needed for 64.


67         $(SH) ./install-sfw
68         VER64=$(VER64) MACH64=$(MACH64) $(SHELL) ./install-sfw-64

Both install scripts should use the same shell, either
SH or SHELL.


Otherwise, seems fine.


- Dermot






On 05/14/09 09:42, Jim Walker wrote:
> Dermot,
> 
> Can you look at the libtool changes?
> 
> Cheers,
> Jim
> 
> kang zhao wrote:
>> Hi all,
>>
>> Please review the  webrev of guile/autogen.
>>
>> Based on  comments, there is some code change accordingly.
>>
>> *add 64bit library to guile
>>
>> *include a 64bit library file to libltdl in this webrev
>> 64bit guile compiling need 64bit libltdl to compile 64bit  first.
>>
>> *change all METADATA file to new format
>>
>> *change all install-sfw file to standard format
>>
>>
>> http://cr.opensolaris.org/~kz196224/guile_autogen/
>>
>>
>> Thanks,
>> Kenny
>> _______________________________________________
>> sfwnv-discuss mailing list
>> sfwnv-discuss at opensolaris.org
>> http://mail.opensolaris.org/mailman/listinfo/sfwnv-discuss
> 

Reply via email to