Re: Configure fails on 32-bit platform

2018-02-11 Thread Jeffrey Walton
On Sun, Feb 11, 2018 at 4:40 PM, Niels Möller  wrote:
> Jeffrey Walton  writes:
>
>> But please see -m32 -march=native, which was provided to configure to
>> configure with.
>
> Setting CFLAGS simply specifies what flags to be passed to the C
> compiler, totally overriding configure's automatic selection of compiler
> flags. But CFLAGS is *not* interpreted by configure in any way.
>
> Please use the documented mechanism to get the configuration you want.

That's not true. In fact, Stallman specifically tells the project to
adds the flags it needs. Below is from Section 7.2.3 of
https://www.gnu.org/prep/standards/standards.html. This would fix the
ARM compile problem, too:



If there are C compiler options that must be used for proper
compilation of certain files, do not include them in CFLAGS. Users
expect to be able to specify CFLAGS freely themselves. Instead,
arrange to pass the necessary options to the C compiler independently
of CFLAGS, by writing them explicitly in the compilation commands or
by defining an implicit rule, like this:

CFLAGS = -g
ALL_CFLAGS = -I. $(CFLAGS)
.c.o:
$(CC) -c $(CPPFLAGS) $(ALL_CFLAGS) $<

Do include the ‘-g’ option in CFLAGS, because that is not required for
proper compilation. You can consider it a default that is only
recommended. If the package is set up so that it is compiled with GCC
by default, then you might as well include ‘-O’ in the default value
of CFLAGS as well.



Jeff
___
gmp-bugs mailing list
gmp-bugs@gmplib.org
https://gmplib.org/mailman/listinfo/gmp-bugs


Re: repl-vsnprintf.c:396:0: error: ISO C forbids an empty translation unit

2018-02-11 Thread Vincent Lefevre
On 2018-02-11 10:40:55 +0100, Torbjorn Granlund wrote:
> Dennis Clarke  writes:
> 
>   repl-vsnprintf.c:396:0: error: ISO C forbids an empty translation unit
> 
> Our nightly builds get a few of those too, as warnings.  I've decided I
> can live with those as no platform seems to actually dislike the
> resulting object files.

Or you can just add a dummy typedef to ensure that the translation unit
is not empty.

-- 
Vincent Lefèvre  - Web: 
100% accessible validated (X)HTML - Blog: 
Work: CR INRIA - computer arithmetic / AriC project (LIP, ENS-Lyon)
___
gmp-bugs mailing list
gmp-bugs@gmplib.org
https://gmplib.org/mailman/listinfo/gmp-bugs


Re: Configure fails on 32-bit platform

2018-02-11 Thread Niels Möller
Jeffrey Walton  writes:

> But please see -m32 -march=native, which was provided to configure to
> configure with.

Setting CFLAGS simply specifies what flags to be passed to the C
compiler, totally overriding configure's automatic selection of compiler
flags. But CFLAGS is *not* interpreted by configure in any way.

Please use the documented mechanism to get the configuration you want.

Regards,
/Niels

-- 
Niels Möller. PGP-encrypted email is preferred. Keyid 368C6677.
Internet email is subject to wholesale government surveillance.
___
gmp-bugs mailing list
gmp-bugs@gmplib.org
https://gmplib.org/mailman/listinfo/gmp-bugs


Re: repl-vsnprintf.c:396:0: error: ISO C forbids an empty translation unit

2018-02-11 Thread Torbjörn Granlund
Dennis Clarke  writes:

  repl-vsnprintf.c:396:0: error: ISO C forbids an empty translation unit

Our nightly builds get a few of those too, as warnings.  I've decided I
can live with those as no platform seems to actually dislike the
resulting object files.
  

-- 
Torbjörn
Please encrypt, key id 0xC8601622
___
gmp-bugs mailing list
gmp-bugs@gmplib.org
https://gmplib.org/mailman/listinfo/gmp-bugs