From: "Holger Hoffstätte" <holger.hoffstaette@googlemail.com>
To: linux-btrfs@vger.kernel.org
Subject: Re: [RFC][BTRFS-PROGS] Improve output of mkfs.btrfs command
Date: Wed, 17 Dec 2014 15:08:38 +0000 (UTC) [thread overview]
Message-ID: <pan.2014.12.17.15.08.38@googlemail.com> (raw)
In-Reply-To: 1418673780-22000-1-git-send-email-kreijack@inwind.it
On Mon, 15 Dec 2014 21:02:53 +0100, Goffredo Baroncelli wrote:
> enclosed a patches set to improve the output of mkfs.btrfs command.
This is very nice! I do have a comment about the device list:
> ..
> UUID ID SIZE PATH
> ------------------------------------ -- --------- -----------
> 62121322-5666-4ecf-bc8e-c9b3d9f60db9 1 50.00GiB /dev/vdb
> 6158cb13-3ae8-42b6-8603-660f1e5c8a7a 2 50.00GiB /dev/vdc
> b49516db-ddf5-4f54-8831-a4babc79e901 3 50.00GiB /dev/vdd
> 00b03d81-7d29-4894-8050-9dd205f97c41 4 50.00GiB /dev/vde
> f119a2ec-5ef0-436c-805e-c1b0612b05ca 5 50.00GiB /dev/vdf
> adee4f58-e094-4bd4-8c56-941527524f8d 6 50.00GiB /dev/vdg
> a8299171-2024-4057-ba56-1f83bf6d7e2e 7 50.00GiB /dev/vdh
> b694e275-e454-4dbd-beb0-e33c388cffa2 8 2.00GiB /dev/vdi
> 7cbe04b5-36cd-4ea7-be82-206d5487914e 9 2.00GiB /dev/vdj
> 7c320654-675e-456b-ac23-cfb148b8ea57 10 2.00GiB /dev/vdk
>
> Total disks size: 356.01GiB
I'm not sure how others feel about this, but the first column of UUIDs
contains almost no human-usable information (who remembers UUIDs?) but
takes up a huge chunk of the screen real estate. I'd suggest to reorder
the columns to be something like (ID, Size, Path, UUID). This puts more
relevant information in reading-order.
I hope this isn't too much to ask :)
thanks
Holger
prev parent reply other threads:[~2014-12-17 15:09 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-12-15 20:02 [RFC][BTRFS-PROGS] Improve output of mkfs.btrfs command Goffredo Baroncelli
2014-12-15 20:02 ` [PATCH 1/7] Add -v -q switches to mkfs.btrfs Goffredo Baroncelli
2014-12-16 4:56 ` Satoru Takeuchi
2014-12-15 20:02 ` [PATCH 2/7] Move group_profile_str() in utils.c Goffredo Baroncelli
2014-12-15 20:02 ` [PATCH 3/7] Add verbose option to btrfs_add_to_fsid() Goffredo Baroncelli
2014-12-15 20:02 ` [PATCH 4/7] Add strdup in btrfs_add_to_fsid() to track the device path Goffredo Baroncelli
2014-12-15 20:02 ` [PATCH 5/7] Return the fsid from make_btrfs() Goffredo Baroncelli
2014-12-15 20:02 ` [PATCH 6/7] Print the summary Goffredo Baroncelli
2014-12-16 1:58 ` Duncan
2014-12-16 3:47 ` Robert White
2014-12-16 9:05 ` Hugo Mills
2014-12-16 21:40 ` Duncan
2014-12-18 5:44 ` Robert White
2014-12-18 8:41 ` Terminology (was Re: [PATCH 6/7] Print the summary) Hugo Mills
2014-12-22 18:38 ` David Sterba
2014-12-17 18:59 ` [PATCH 6/7] Print the summary Goffredo Baroncelli
2014-12-16 3:27 ` Satoru Takeuchi
2014-12-15 20:03 ` [PATCH 7/7] Add -v and -o switches Goffredo Baroncelli
2014-12-16 3:23 ` [RFC][BTRFS-PROGS] Improve output of mkfs.btrfs command Satoru Takeuchi
2014-12-17 19:49 ` Goffredo Baroncelli
2014-12-17 14:29 ` David Sterba
2014-12-17 15:08 ` Holger Hoffstätte [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=pan.2014.12.17.15.08.38@googlemail.com \
--to=holger.hoffstaette@googlemail.com \
--cc=linux-btrfs@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.