Am Mittwoch, 3. Oktober 2012 schrieb Roman Mamedov: > On Wed, 3 Oct 2012 08:22:06 +0200 > > Goffredo Baroncelli <kreijack@xxxxxxxxx> wrote: > > On Wed, Oct 3, 2012 at 1:46 AM, Chris Mason > > <chris.mason@xxxxxxxxxxxx> wrote: [...] > > > > > I like it, thanks. Could you please update btrfs fi df to show > > > this instead of adding a new command though? > > > > Hi Chris, > > no problem to update the patches, however I have one suggestion: > > - leave "btrfs fi df" as is to no break any script (if any) which > > would uses it. Hide it from the help and deprecating it in the man > > page. This because the output is very different. > > - renaming "btrfs fi disk-usage" in "btrfs fi disk-free", which make > > sense. > > 1) btrfs is still labeled experimental, so no script author should have > seriously relied on utilities text output to stay unchanged long-term; > personally I don't see modifying it (especially not in some superficial > way, but as a part of major improvement) to be a problem; I am not fond of parsing command output unless using specific options that some commands provide to just provide one value. I.e. if the command provides options specifically for scripts parsing output. And if I do it nonetheless, I am not surprised when it breaks at a time, like with find - ls which now reports decimal part of file modification timestamps. I think having a library with a function to get those values via a defined API would be way to go. Or querying such stuff via udisks or whatever. I am very happy to see better size reporting. -- Martin 'Helios' Steigerwald - http://www.Lichtvoll.de GPG: 03B0 0D6C 0040 0710 4AFA B82F 991B EAAC A599 84C7 -- 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
