mirror of
https://github.com/systemd/systemd.git
synced 2025-02-28 05:57:33 +03:00
journalctl: flip to --full by default
We already shew lines in full when using a pager or not on a tty. The commit disables ellipsization in the sole remaining case, namely when --follow is used. This has been a popular request for a long time, and indeed, full output seems much more useful. Old behaviour can still be requested by using --no-full. Old options retain their behaviour for compatiblity, but aren't advertised as much. This change applies only to jornalctl, not to systemctl, when ellipsization is useful to keep the layout. https://bugzilla.redhat.com/show_bug.cgi?id=984758
This commit is contained in:
parent
c26547d612
commit
2b8f6883a1
@ -152,11 +152,20 @@
|
||||
</varlistentry>
|
||||
|
||||
<varlistentry>
|
||||
<term><option>-l</option></term>
|
||||
<term><option>--no-full</option></term>
|
||||
<term><option>--full</option></term>
|
||||
<term><option>-l</option></term>
|
||||
|
||||
<listitem><para>Show all (printable) fields in
|
||||
full.</para></listitem>
|
||||
<listitem><para>Ellipsize fields when
|
||||
they don't fit in available columns.
|
||||
The default is to show full fields,
|
||||
allowing them to wrap or be truncated
|
||||
by the pager if one is used.</para>
|
||||
|
||||
<para>Old options
|
||||
<option>-l</option>/<option>--full</option>
|
||||
not useful anymore, except to undo
|
||||
<option>--no-full</option>.</para></listitem>
|
||||
</varlistentry>
|
||||
|
||||
<varlistentry>
|
||||
|
@ -64,7 +64,7 @@
|
||||
static OutputMode arg_output = OUTPUT_SHORT;
|
||||
static bool arg_pager_end = false;
|
||||
static bool arg_follow = false;
|
||||
static bool arg_full = false;
|
||||
static bool arg_full = true;
|
||||
static bool arg_all = false;
|
||||
static bool arg_no_pager = false;
|
||||
static int arg_lines = -1;
|
||||
@ -138,7 +138,7 @@ static int help(void) {
|
||||
" short-precise, short-monotonic, verbose,\n"
|
||||
" export, json, json-pretty, json-sse, cat)\n"
|
||||
" -x --catalog Add message explanations where available\n"
|
||||
" -l --full Do not ellipsize fields\n"
|
||||
" --no-full Ellipsize fields\n"
|
||||
" -a --all Show all fields, including long and unprintable\n"
|
||||
" -q --quiet Don't show privilege warning\n"
|
||||
" --no-pager Do not pipe output into a pager\n"
|
||||
@ -175,6 +175,7 @@ static int parse_argv(int argc, char *argv[]) {
|
||||
enum {
|
||||
ARG_VERSION = 0x100,
|
||||
ARG_NO_PAGER,
|
||||
ARG_NO_FULL,
|
||||
ARG_NO_TAIL,
|
||||
ARG_NEW_ID128,
|
||||
ARG_USER,
|
||||
@ -208,6 +209,7 @@ static int parse_argv(int argc, char *argv[]) {
|
||||
{ "output", required_argument, NULL, 'o' },
|
||||
{ "all", no_argument, NULL, 'a' },
|
||||
{ "full", no_argument, NULL, 'l' },
|
||||
{ "no-full", no_argument, NULL, ARG_NO_FULL },
|
||||
{ "lines", optional_argument, NULL, 'n' },
|
||||
{ "no-tail", no_argument, NULL, ARG_NO_TAIL },
|
||||
{ "new-id128", no_argument, NULL, ARG_NEW_ID128 },
|
||||
@ -298,6 +300,10 @@ static int parse_argv(int argc, char *argv[]) {
|
||||
arg_full = true;
|
||||
break;
|
||||
|
||||
case ARG_NO_FULL:
|
||||
arg_full = false;
|
||||
break;
|
||||
|
||||
case 'a':
|
||||
arg_all = true;
|
||||
break;
|
||||
@ -1622,7 +1628,7 @@ int main(int argc, char *argv[]) {
|
||||
|
||||
flags =
|
||||
arg_all * OUTPUT_SHOW_ALL |
|
||||
(arg_full || !on_tty() || pager_have()) * OUTPUT_FULL_WIDTH |
|
||||
arg_full * OUTPUT_FULL_WIDTH |
|
||||
on_tty() * OUTPUT_COLOR |
|
||||
arg_catalog * OUTPUT_CATALOG;
|
||||
|
||||
|
Loading…
x
Reference in New Issue
Block a user