On Tue, Sep 9, 2014 at 5:53 AM, Dominik Vogt <v...@linux.vnet.ibm.com> wrote:
>
> This patch fixes the compiler flags in libgo/mksysinfo.sh.  In one
> place, some compiler flags were missing that are consistently used
> elswhere, resulting in an error message.
>
> ChangeLog
> 2014-09-05  Dominik Vogt  <v...@linux.vnet.ibm.com>
>
>         * libgo/mksysinfo.sh (OUT):
>         Add the same compile flags that configure uses to detect whether 
> off64_t
>         is present or not when generating the go structures for the C types.
>         Otherwise the go type for off64_t may not be generated.

I don't understand why this patch is necessary.  The invocation of
mksysinfo in libgo/Makefile.am sets the CC environment variable to
include $(OSCFLAGS), and OSCFLAGS should include those three options.

Trying to list the flags separately in mksysinfo.sh is going to be
incomplete on some systems--see OSCFLAGS in libgo/configure.ac.

Can you explain further why this patch is required?  Thanks.

Ian

Reply via email to