list: sort should accept multiple attributes #714
Comments
odeke-em
added a commit
that referenced
this issue
Aug 28, 2016
Fixes #714. Sort attributes passed in from the commandline are csv values e.g ```shell $ drive list -sort name_r,modtime ``` The bug previously was that these values were actually never getting split by comma values, in cmd/drive/main. This CL ensures that we do the necessary splitting and whitespace trimming before we make the sorters.
odeke-em
added a commit
that referenced
this issue
Aug 28, 2016
Fixes #714. Sort attributes passed in from the commandline are csv values e.g ```shell $ drive list -sort name_r,modtime ``` The bug previously was that these values were actually never getting split by comma values, in cmd/drive/main. This CL ensures that we do the necessary splitting and whitespace trimming before we make the sorters. It also updates the CLI doc on sort: * Before ```shell $ drive list -sort -sort string sort items in the order * md5. * name. * size. * type. * version ``` * After ```shell $ drive list -sort -sort string sort items by a combination of attributes * modtime. * md5. * name. * size. * type. * version ```
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I just noticed offline that trying to apply a sort over list only works on the first attribute
It should apply a sort on all the passed in attributes, separated by commas.
The text was updated successfully, but these errors were encountered: