diff options
| author | Pádraig Brady <P@draigBrady.com> | 2014-11-27 00:51:00 +0000 |
|---|---|---|
| committer | Pádraig Brady <P@draigBrady.com> | 2014-11-27 14:25:00 +0000 |
| commit | 7d1fe8868e29ae2644831685df4a29638721d6ba (patch) | |
| tree | dca250b9dd338b191879afd36e115eba244db1ae /src/split.c | |
| parent | a78d85386bf4a55d7ccbd7c03c0075615b3f61d2 (diff) | |
| download | coreutils-7d1fe8868e29ae2644831685df4a29638721d6ba.tar.gz | |
rm: fix prompted number of arguments to remove on some platforms
"zu" was output on solaris 8 for example rather than the number,
since coreutils-8.22.
* cfg.mk: Disallow %z, since we don't currently use the gnulib
fprintf module, so any usage with it is non portable. Also
our usage with error() currently works only through an ancillary
dependency on the vfprintf gnulib module.
* src/rm.c (main): Use %PRIuMAX rather than %zu for portability.
* src/dd.c (alloc_[io]buf): Likewise for consistency.
* src/od.c (main): Likewise.
* src/split.c (set_suffix_length): Likewise.
* NEWS: Mention the rm bug fix.
Reported in http://bugs.gnu.org/19184
Diffstat (limited to 'src/split.c')
| -rw-r--r-- | src/split.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/split.c b/src/split.c index ec0da7deb..0eec3ecc2 100644 --- a/src/split.c +++ b/src/split.c @@ -158,7 +158,7 @@ set_suffix_length (uintmax_t n_units, enum Split_type split_type) { #define DEFAULT_SUFFIX_LENGTH 2 - size_t suffix_needed = 0; + uintmax_t suffix_needed = 0; /* The suffix auto length feature is incompatible with a user specified start value as the generated suffixes @@ -183,7 +183,7 @@ set_suffix_length (uintmax_t n_units, enum Split_type split_type) if (suffix_length < suffix_needed) { error (EXIT_FAILURE, 0, - _("the suffix length needs to be at least %zu"), + _("the suffix length needs to be at least %"PRIuMAX), suffix_needed); } suffix_auto = false; |