All of lore.kernel.org
 help / color / mirror / Atom feed
From: Anand Jain <Anand.Jain@oracle.com>
To: Stefan Behrens <sbehrens@giantdisaster.de>
Cc: linux-btrfs@vger.kernel.org
Subject: Re: [PATCH] Btrfs-progs: remove btrfsctl, btrfs-show and btrfs-vol from default build
Date: Mon, 28 Jan 2013 11:03:44 +0800	[thread overview]
Message-ID: <5105EA90.6000507@oracle.com> (raw)
In-Reply-To: <1339178417-26051-1-git-send-email-sbehrens@giantdisaster.de>


  This patch is much needed. thanks.

  Further, IMO the summary may provide a table of mapping of
  the original to its predecessor command-and-option so that
  - it will ensure we have taken care of all the original
  features in the new cli, and it will help to document
  for the onlooker.

  eg:
  old-command        replaced-by
   btrfsctl -D        btrfs subvol del
   ::
   ::

  (if this is already there somewhere providing a ref/copying
  here will help)

Thanks, Anand

On 06/09/2012 02:00 AM, Stefan Behrens wrote:
> Remove btrfsctl, btrfs-show and btrfs-vol from "all" target of Makefile.
>
> Signed-off-by: Stefan Behrens <sbehrens@giantdisaster.de>
> ---
>   Makefile |    2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Makefile b/Makefile
> index aaf1381..79e7a56 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -18,7 +18,7 @@ bindir = $(prefix)/bin
>   LIBS=-luuid
>   RESTORE_LIBS=-lz
>
> -progs = btrfsctl mkfs.btrfs btrfs-debug-tree btrfs-show btrfs-vol btrfsck \
> +progs = mkfs.btrfs btrfs-debug-tree btrfsck \
>   	btrfs btrfs-map-logical btrfs-image btrfs-zero-log btrfs-convert \
>   	btrfs-find-root btrfs-restore btrfstune
>
>

      parent reply	other threads:[~2013-01-28  2:56 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-06-08 18:00 [PATCH] Btrfs-progs: remove btrfsctl, btrfs-show and btrfs-vol from default build Stefan Behrens
2013-01-25 11:57 ` Gene Czarcinski
2013-01-25 15:32   ` Eric Sandeen
2013-01-25 15:49     ` Stefan Behrens
2013-01-28  3: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=5105EA90.6000507@oracle.com \
    --to=anand.jain@oracle.com \
    --cc=linux-btrfs@vger.kernel.org \
    --cc=sbehrens@giantdisaster.de \
    /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.