Compilation error with GCC about flock HOT 6 OPEN

This never got fixed, apparently. The proper fix, in my opinion, is to cast the result to either long or unsigned long, and use either %1ld or (respectively) %1lu. This would be portable, safe and non-hacky — there's no need to identify platforms.

from flock.

A brief search shows me this is at least correct for Linux. Need to confirm this is correct for illumos and bsd system types.
@mohammad-akhlaghi want to draft the changeset for this, or shall I?

from flock.

Please go ahead @josephholsten 😉...

from flock.

With the help of Zahra Sharbaf, we recently discovered that simply replacing %1u with %1lu is not a generic solution and will cause a crash on macOS (using LLVM compiler)! The more generic solution is to replace it with %ld (as the error message in my first comment also reports, the actual type is: long int). I have tested this fix with GCC and it fixes the problem.

from flock.

There was a mistake in my previous message: on macOS, the format is int, not long int! So this is what our building script looks like now (modifying the source of flock after unpacking it to build safely on GNU/Linux and macOS, I can now confirm that it works in both cases):

case $on_mac_os in
    yes) sed -e's/\%1u/\%d/'  src/flock.c > src/flock-new.c;;
    no)  sed -e's/\%1u/\%ld/' src/flock.c > src/flock-new.c;;
    *)   echo "pre-make-build.sh: '$on_mac_os' unrecognized value for on_mac_os";;
esac

It would be great if this can be taken into the source of flock for future versions so we can remove this extra manual correction 😉.

from flock.

Well, I don’t love putting this into the build script, but fixed in an ugly way is better than broken! I’m sorry I’m not really able to draft this in the near future. I’m giving you a commit bit, please push us a PR!

from flock.