Re: [PATCH 1/2 V2] btrfs, a new tool to manage a btrfs filesystem; source code

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,
It's a cool utility with tidy source code.
At my first glance, there's typo, a question and an impression.

 - There's no "<source>" in usage of utility.
 - will occupe -> will occupy
   (usage of utility : "btrfs resize")
 - Is specifying a target device of resizing isn't public
   even in your utility?
 - I felt somehow confusing with the short commands.
   It's not easy to associate it from its command.

Regards,
taruisi

(2010/02/18 5:02), Goffredo Baroncelli wrote:
> This is the source code of btrfs.c.
> The main changes from my previous patch are:
> - Rearranged the short command in order to avoid conflicts
> - Renamed the 'create' verb is 'subvolume'
> - Rearranged the parsing code, which now is integrated in the help in order to avoid mismatch
> 
> BR
> G.Baroncelli
> 
 - snip -

--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Filesystem Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux