journalctl: do not ellipsize when using pager

If a pager is used, ellipsization is redundant — the pager does
that better by hiding the part that cannot be shown. Pager's advantage
is that the user can press → to view the hidden part of a message,
and then ← to return.
This commit is contained in:
Zbigniew Jędrzejewski-Szmek 2012-07-17 07:35:08 +02:00 committed by Lennart Poettering
parent 085d71209b
commit 92a1fd9e95
3 changed files with 9 additions and 4 deletions

View file

@ -312,6 +312,7 @@ int main(int argc, char *argv[]) {
bool need_seek = false; bool need_seek = false;
sd_id128_t previous_boot_id; sd_id128_t previous_boot_id;
bool previous_boot_id_valid = false; bool previous_boot_id_valid = false;
bool have_pager;
log_parse_environment(); log_parse_environment();
log_open(); log_open();
@ -397,7 +398,8 @@ int main(int argc, char *argv[]) {
goto finish; goto finish;
} }
if (!arg_no_pager && !arg_follow) { have_pager = !arg_no_pager && !arg_follow;
if (have_pager) {
columns(); columns();
pager_open(); pager_open();
} }
@ -410,6 +412,8 @@ int main(int argc, char *argv[]) {
for (;;) { for (;;) {
for (;;) { for (;;) {
sd_id128_t boot_id; sd_id128_t boot_id;
int flags = (arg_show_all*OUTPUT_SHOW_ALL |
have_pager*OUTPUT_FULL_WIDTH);
if (need_seek) { if (need_seek) {
r = sd_journal_next(j); r = sd_journal_next(j);
@ -434,8 +438,7 @@ int main(int argc, char *argv[]) {
line ++; line ++;
r = output_journal(j, arg_output, line, 0, r = output_journal(j, arg_output, line, 0, flags);
arg_show_all ? OUTPUT_SHOW_ALL : 0);
if (r < 0) if (r < 0)
goto finish; goto finish;

View file

@ -225,7 +225,8 @@ static int output_short(sd_journal *j, unsigned line, unsigned n_columns,
else if (!utf8_is_printable_n(message, message_len)) { else if (!utf8_is_printable_n(message, message_len)) {
char bytes[FORMAT_BYTES_MAX]; char bytes[FORMAT_BYTES_MAX];
printf(": [%s blob data]\n", format_bytes(bytes, sizeof(bytes), message_len)); printf(": [%s blob data]\n", format_bytes(bytes, sizeof(bytes), message_len));
} else if (message_len + n < n_columns) } else if ((flags & OUTPUT_FULL_WIDTH) ||
(message_len + n < n_columns))
printf(": %.*s\n", (int) message_len, message); printf(": %.*s\n", (int) message_len, message);
else if (n < n_columns) { else if (n < n_columns) {
char *e; char *e;

View file

@ -44,6 +44,7 @@ typedef enum OutputFlags {
OUTPUT_MONOTONIC_MODE = 1 << 1, OUTPUT_MONOTONIC_MODE = 1 << 1,
OUTPUT_FOLLOW = 1 << 2, OUTPUT_FOLLOW = 1 << 2,
OUTPUT_WARN_CUTOFF = 1 << 3, OUTPUT_WARN_CUTOFF = 1 << 3,
OUTPUT_FULL_WIDTH = 1 << 4,
} OutputFlags; } OutputFlags;
int output_journal(sd_journal *j, OutputMode mode, unsigned line, int output_journal(sd_journal *j, OutputMode mode, unsigned line,