On Thu, Oct 25, 2012 at 2:36 AM, Rainer Orth
<r...@cebitec.uni-bielefeld.de> wrote:
> Ian Lance Taylor <i...@google.com> writes:
>
>> There is a decent change that this will break something on non-x86
>> systems.  I will do what testing I am able to do after the commit.
>
> As expected, it did break the Solaris libgo build:
>
> * udpsock_posix.go lacked definitions of joinIPv4Group, joinIPv6Group,
>   setIPv6MulticastInterface, setIPv6MulticastLoopback.  It turned out
>   that sockoptip_solaris.go isn't needed any longer, but
>   sockoptip_posix.go can be used instead.
>
> * Solaris lacked some stat_atim*.go in go/archive/tar initially.  With
>   stat_atim.go used, it didn't build initially.  Since Stat_t.[AC]tim
>   are Timestruc, we need a corresponding Unix(), now provided in
>   syscall_solaris.go.
>
> With those changes, libgo builds again, and Solaris/x86 testsuite
> results are reasonable:

Thanks.  I committed your patch to mainline.

Ian

Reply via email to