From: Stefan Beller <sbeller@google.com>
Date: Wed, 11 Nov 2015 17:38:22 GMT
Subject: Re: [PATCH] http: fix some printf format warnings on 32-bit builds
Message-ID: <CAGZ79kbOQk4S3T7b9Ay=NnXnVBhoNLxCKeE6wefhh5qC19+_yg@mail.gmail.com>
In-Reply-To: <5C9FC321-5B5E-47E6-9BCE-6CDAA41D6E54@gmail.com>

On Wed, Nov 11, 2015 at 12:02 AM, Lars Schneider
<larsxschneider@gmail.com> wrote:
>
> Unfortunately no. All their machines are 64-bit [1] and they have "no immediate plans to add this feature" [2].
> However, we could the following build configuration on a 64-bit machine:
>
> export CFLAGS="-m32"
> export LDFLAGS="-m32"
>
> I think then we could catch these kind of warnings. Do you see a downside with this approach?

I think that should do it.

Assuming their 64 bit means x86-64 architecture, we may want to prefer
-mx32, such that we can also test it?

>
> - Lars
>
> [1] http://docs.travis-ci.com/user/ci-environment/
> [2] https://github.com/travis-ci/travis-ci/issues/986