On Tue, Feb 28, 2017 at 12:08 PM, Rainer Orth
<r...@cebitec.uni-bielefeld.de> wrote:
> Hi Ian,
>
>> I goofed on the shell quoting in mksigtab.sh, causing various
>> system-specific signals to be entered incorrectly in the runtime
>> signal table.  This patch fixes the problem.  The test for this is
>> misc/cgo/testcarchive in the master repo, which
>> we don't yet run for gccgo.  Bootstrapped and ran Go testsuite on
>> x86_64-pc-linux-gnu.  Committed to mainline.
>
> this broke Solaris bootstrap:
>
> sigtab.go:38:2: error: duplicate value for index 22
>   _SIGPOLL:    {_SigNotify, "SIGPOLL: pollable event occurred"},
>   ^
> /vol/gcc/src/hg/trunk/local/libgo/go/runtime/signal_unix.go:395:3: error: 
> range clause must have array, slice, string, map, or channel type
>    for i := range sigtable {
>    ^
> /vol/gcc/src/hg/trunk/local/libgo/go/runtime/signal_unix.go:591:2: error: 
> range clause must have array, slice, string, map, or channel type
>   for i := range sigtable {
>   ^
> /vol/gcc/src/hg/trunk/local/libgo/go/runtime/signal_unix.go:395:7: error: 
> invalid type for range clause
>    for i := range sigtable {
>        ^
> /vol/gcc/src/hg/trunk/local/libgo/go/runtime/signal_unix.go:591:6: error: 
> invalid type for range clause
>   for i := range sigtable {
>       ^
> Makefile:3333: recipe for target 'runtime.lo' failed
> make[4]: *** [runtime.lo] Error 1
>
> <sys/iso/signal_iso.h> has
>
> #define SIGPOLL 22      /* pollable event occurred */
> #define SIGIO   SIGPOLL /* socket I/O possible (SIGPOLL alias) */
>
> this leads to
>
> const _SIGIO = _SIGPOLL
>
> and similarly
>
> const _SIGLWP = _SIGAIOCANCEL
>
> in gen-sysinfo.go.
>
> Fixed like this, which allows runtime.lo to compile.

Thanks.

Committed to mainline.

Ian

Reply via email to