Re: [PATCH] http: fix some printf format warnings on 32-bit builds
- From
Stefan Beller <sbeller@google.com>
- Date
- Nov 11, 2015, 17:38 UTC
- 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:
Show 8 quoted lines
> > 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?
Show 5 quoted lines
> > - Lars > > [1] http://docs.travis-ci.com/user/ci-environment/ > [2] https://github.com/travis-ci/travis-ci/issues/986