diff options
| author | Karel Zak | 2018-03-27 10:40:13 +0200 |
|---|---|---|
| committer | Karel Zak | 2018-03-27 10:55:27 +0200 |
| commit | 651c5d428c2ef103ee8c5b1a310d6f29f0304744 (patch) | |
| tree | 9d007566ef621fdf6e833537ec12ad49da5723f5 /tests/ts | |
| parent | docs: add to column(1) TODO (diff) | |
| download | kernel-qcow2-util-linux-651c5d428c2ef103ee8c5b1a310d6f29f0304744.tar.gz kernel-qcow2-util-linux-651c5d428c2ef103ee8c5b1a310d6f29f0304744.tar.xz kernel-qcow2-util-linux-651c5d428c2ef103ee8c5b1a310d6f29f0304744.zip | |
column: fix leading space characters bug
The bug has been introduced during column(1) rewrite. The function
read_input() need to skip leading space only temporary to detect empty
lines, but the rest of the code has to use the original buffer (line).
I've tried to fix one of the symptoms by 5c7b67fbbf41c973ca8d49b1e8bdba22dbb917aa
(alter), but this solution is unnecessary and too complex.
Changes:
* don't ignore leading space
* remove unnecessary stuff introduced by 5c7b67fbbf41c973ca8d49b1e8bdba22dbb917aa
* fix regression test with incorrect separator
Addresses: https://github.com/karelzak/util-linux/issues/575
Addresses: https://bugzilla.redhat.com/show_bug.cgi?id=1560283
Signed-off-by: Karel Zak <kzak@redhat.com>
Diffstat (limited to 'tests/ts')
| -rwxr-xr-x | tests/ts/column/table | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/ts/column/table b/tests/ts/column/table index 27b52e7c8..5c89d5eaf 100755 --- a/tests/ts/column/table +++ b/tests/ts/column/table @@ -37,7 +37,7 @@ $TS_CMD_COLUMN --separator ',' --table $TS_SELF/files/table-sep >> $TS_OUTPUT 2> ts_finalize_subtest ts_init_subtest "input-separator-space" -$TS_CMD_COLUMN --separator ',' --table $TS_SELF/files/table-sep-space >> $TS_OUTPUT 2>&1 +$TS_CMD_COLUMN --separator "$(echo -e '\t')" --table $TS_SELF/files/table-sep-space >> $TS_OUTPUT 2>&1 ts_finalize_subtest ts_init_subtest "long" |