Re: [PATCH] http: fix some printf format warnings on 32-bit builds

On 13/11/15 08:57, Eric Sunshine wrote:

On Fri, Nov 13, 2015 at 3:46 AM, Lars Schneider
[off-list ref] wrote:

quoted

On 11 Nov 2015, at 18:49, Ramsay Jones [off-list ref] wrote:

quoted

On 11/11/15 02:00, Stefan Beller wrote:

quoted

On Tue, Nov 10, 2015 at 5:22 PM, Eric Sunshine [off-list ref] wrote:

quoted

On Tue, Nov 10, 2015 at 7:23 PM, Ramsay Jones
[off-list ref] wrote:

quoted

Commit f8117f55 ("http: use off_t to store partial file size",
02-11-2015) changed the type of some variables from long to off_t.
The 32-bit build, which enables the large filesystem interface
(_FILE_OFFSET_BITS == 64), defines the off_t type as a 64-bit
integer, whereas long is a 32-bit integer. This results in a couple
of printf format warnings.

My machine is 64-bit, though, so perhaps it's misleading to
characterize this as a fix for 32-bit builds. In particular, off_t is
'long long' on this machine, so it complains about the "long" format
specifier.

I wonder if 32 bit compilation can be part of travis.

Did this warning show up on the OS X build?

Yes, I added CFLAGS="-Werror=format" to the my experimental TravisCI
build and it breaks the build on OS X.
See here (you need to scroll all the way down):
https://travis-ci.org/larsxschneider/git/jobs/90899656

BTW: I tried to set "-Werror" but then I got a bunch of macro redefined errors like this:
./git-compat-util.h:614:9: error: 'strlcpy' macro redefined [-Werror]

Is this a known issue? Is this an issue at all?

Odd. I don't experience anything like that on my Mac.

Hmm, from the output, it looks like the configure script is
not detecting that 'strlcpy' is available (so setting
NO_STRLCPY=YesPlease in the config.mak.autogen file).
However, it seems to be a 'macro redirect' set in the
/usr/include/secure/_string.h header file (presumably it
redirects between a more or less secure version ;-)

Unfortunately, I don't have access to a mac - so I can't
help you with the debugging. :(

ATB,
Ramsay Jones