From: Anand Jain <Anand.Jain@oracle.com>
To: Goffredo Baroncelli <kreijack@gmail.com>
Cc: linux-btrfs@vger.kernel.org
Subject: Re: [PATCH 0/4] Add show sub command to btrfs subvolume
Date: Mon, 15 Oct 2012 17:03:17 +0800 [thread overview]
Message-ID: <507BD155.7000609@oracle.com> (raw)
In-Reply-To: <5079CADA.4050604@gmail.com>
> Thanks for working on that; do you think that is it feasible to merge
> the works on "btrfs sub list" and "btrfs sub show" ?
> To date the output of "btrfs sub list" is quite cryptic: a lot of
> information on only on row, which is difficult to read. However it has
> the capability to traverse the filesystem to list the subvolumes, to
> sort the output, to filter it...
> Instead your work has not the capability to traverse/filter/sort but has
> a very nice output.
>
> The ideal is to extend the actual command (btrfs sub list) to have your
> output (as default) and the current one (as option) when used in script...
Thanks for the comments.
(just for note: this patch is on top the latest subvol
list framework).
generally multi-line output isn't suitable for the list
kind of output, right? since it makes scripting difficult.
So here the plan is to have both list and show. List is
to see all the items and show is know more about an item.
Show will be more exciting when we could add size info for
each subvol/snapshot.
-Anand
prev parent reply other threads:[~2012-10-15 8:59 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-10-12 5:20 [PATCH 0/4] Add show sub command to btrfs subvolume Anand jain
2012-10-12 5:20 ` [PATCH 1/4] Btrfs-progs: introduce btrfs_get_subvols function Anand jain
2012-10-12 5:20 ` [PATCH 2/4] Btrfs-progs: need struct root_info to be accesible Anand jain
2012-10-12 5:20 ` [PATCH 3/4] Btrfs-progs: method to fetch root info for subvol Anand jain
2012-10-12 5:20 ` [PATCH 4/4] Btrfs-progs: add btrfs subvol show cli Anand jain
2012-10-13 20:21 ` Goffredo Baroncelli
2012-10-13 20:11 ` [PATCH 0/4] Add show sub command to btrfs subvolume Goffredo Baroncelli
2012-10-15 9:03 ` Anand Jain [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=507BD155.7000609@oracle.com \
--to=anand.jain@oracle.com \
--cc=kreijack@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).