On 30/09/17 19:17, Holger Hoffstätte wrote: > On 09/30/17 19:56, Holger Hoffstätte wrote: >> shell hackery as alternative. Anyway, I was sure that at the time the >> other letters sounded even worse/were taken, but that may just have been >> in my head. ;-) >> >> I just rechecked and -S is still available, so that's good. > > Except that it isn't really, since there is already an 'S' > case in cmds-subvolume.c as shortcut to --sort: That's a shame (and it is also a shame to waste a single letter option without documenting it!). I still would encourage you to avoid -P. I think there is user confusion by "parent" having more than one meaning even within btrfs. And I feel it also tends to perpetuate the mistaken belief that snapshots are somehow "special", and different from other subvolumes (rather than just a piece of information about how two subvolumes are related). It also allows -P to be used one day for the "search by parent UUID" feature. Given the constraints, I would suggest -n. It is mostly arbitrary but it is the second letter of snapshot and also the first of "not a snapshot". Thanks for considering. Graham -- 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
