Hi..

Responses inline

Fabian Groffen wrote:
> On 17-01-2009 23:28:36 +0100, "C. Bergström" wrote:
>   
>> So.. it finally came up on my TODO list again and I got sandbox to 
>> compile on open Solaris..
>>     
>
> Cool!
>   
I concur :)
>   
>> These are not correct, but should give some idea of what I needed to do 
>> to get it to build..
>>
>> econf --disable-static 'LIBS=-lmagic' || die
>>
>> -----------
>> --- libsandbox/Makefile.in.old  2009-01-18 11:51:04.304407327 -0800
>> +++ libsandbox/Makefile.in      2009-01-18 11:38:09.852318831 -0800
>> @@ -59,7 +59,7 @@
>>  libsandbox_la_OBJECTS = $(am_libsandbox_la_OBJECTS)
>>  libsandbox_la_LINK = $(LIBTOOL) --tag=CC $(AM_LIBTOOLFLAGS) \
>>         $(LIBTOOLFLAGS) --mode=link $(CCLD) $(libsandbox_la_CFLAGS) \
>> -       $(CFLAGS) $(libsandbox_la_LDFLAGS) $(LDFLAGS) -o $@
>> +       $(CFLAGS) $(libsandbox_la_LDFLAGS) -M libsandbox.map $(LDFLAGS) 
>> -o $@
>>  DEFAULT_INCLUDES = -I.@am__isrc@ -I$(top_builddir)
>>  depcomp = $(SHELL) $(top_srcdir)/depcomp
>>  am__depfiles_maybe = depfiles
>> @@ -210,8 +210,8 @@
>>  # and destructor will not be executed ...
>>  libsandbox_la_LDFLAGS = \
>>         -no-undefined \
>> -       -nodefaultlibs \
>> -       -Wl,--version-script,libsandbox.map
>> +       -nodefaultlibs
>>     
>
> This looks like a Sun Studio fix/change, is that right?
>   
the -M is needed for the mapfile and it didn't seem to like 
"--version-script".. I wasn't sure how/if possible to put -Wl,-M 
libsandbox.map
>   
>>  libsandbox_la_SOURCES = \
>>         libsandbox.h \
>> ----------------
>>
>> --- configure.old       2009-01-18 09:03:01.286581347 -0800
>> +++ configure   2009-01-18 09:04:39.534604306 -0800
>> @@ -23460,6 +23460,7 @@
>>  LIBC_PATH=$(
>>         $AWK '/attempt to open/ { if (($4 ~ /\/libc\.so/) && ($5 == 
>> "succeeded")) LIBC = $4; }; END {print LIBC}' libctest.log
>>  )
>> +LIBC_PATH=/lib/amd64/libc.so
>>     
>
> I guess this needs some branching.  Betther use /lib/64/libc.so as well,
> that includes the sparcv9 one.
>   
Makes sense..

The problem with this is how to pass on the configure line.. I should 
try passing with ./configure directly.. that would solve the multilib 
problem as well.. otherwise would require sed..
>   
>>  rm -f libctest*
>>  if test x"$LIBC_PATH" = x || ! test -r "$LIBC_PATH" ; then
>>    { { $as_echo "$as_me:$LINENO: error: Unable to determine LIBC PATH 
>> ($LIBC_PATH)" >&5
>> @@ -23478,7 +23479,7 @@
>>  { $as_echo "$as_me:$LINENO: checking libc version" >&5
>>  $as_echo_n "checking libc version... " >&6; }
>>  echo "int main(void) { return 0; }" > libctest.c
>> -$CC $CFLAGS $CPPFLAGS $LDFLAGS -Wall -o libctest libctest.c
>> +$CC $CFLAGS $CPPFLAGS $LDFLAGS  -o libctest libctest.c
>>     
>
> Is it the compiler not liking -Wall, or you? ;)
>   
Compiler.. If you'd like the build to die rather quickly we could swap 
it for -v (which I think is sun cc equivalent)

If you think any of these changes could be fudged into place.. let me know..


./C