From: Shakur Shams Mullick <shakursmullick@gmail.com>
To: util-linux@vger.kernel.org
Cc: Shakur Shams Mullick <shakursmullick@gmail.com>
Subject: [PATCH v2 8/8] misc-utils/lsblk.c: adds sorting support for lsblk
Date: Sun, 11 May 2014 14:20:38 +0600 [thread overview]
Message-ID: <1399796438-5495-8-git-send-email-shakursmullick@gmail.com> (raw)
In-Reply-To: <1399796438-5495-1-git-send-email-shakursmullick@gmail.com>
lsblk --order-by <column>
---
misc-utils/lsblk.c | 55 ++++++++++++++++++++++++++++++++++++++++++------------
1 file changed, 43 insertions(+), 12 deletions(-)
diff --git a/misc-utils/lsblk.c b/misc-utils/lsblk.c
index 2efb2ec..9fa54e7 100644
--- a/misc-utils/lsblk.c
+++ b/misc-utils/lsblk.c
@@ -61,6 +61,7 @@
#include "closestream.h"
#include "mangle.h"
#include "optutils.h"
+#include "list.h"
/* column IDs */
enum {
@@ -110,18 +111,24 @@ enum {
/* basic table settings */
enum {
- LSBLK_ASCII = (1 << 0),
- LSBLK_RAW = (1 << 1),
- LSBLK_NOHEADINGS = (1 << 2),
- LSBLK_EXPORT = (1 << 3),
- LSBLK_TREE = (1 << 4),
+ LSBLK_ASCII = (1 << 0),
+ LSBLK_RAW = (1 << 1),
+ LSBLK_NOHEADINGS = (1 << 2),
+ LSBLK_EXPORT = (1 << 3),
+ LSBLK_TREE = (1 << 4),
+};
+
+enum {
+ sort_none = -1,
+ sort_name,
+ sort_size
};
/* column names */
struct colinfo {
const char *name; /* header */
double whint; /* width hint (N < 1 is in percent of termwidth) */
- int flags; /* SCOLS_FL_* */
+ int flags; /* SCOLS_FL_* */
const char *help;
};
@@ -1355,6 +1362,7 @@ static void __attribute__((__noreturn__)) help(FILE *out)
fputs(_(" -i, --ascii use ascii characters only\n"), out);
fputs(_(" -I, --include <list> show only devices with specified major numbers\n"), out);
fputs(_(" -l, --list use list format output\n"), out);
+ fputs(_(" -c, --order-by <column> show list sorted by name\n"), out);
fputs(_(" -m, --perms output info about permissions\n"), out);
fputs(_(" -n, --noheadings don't print headings\n"), out);
fputs(_(" -o, --output <list> output columns\n"), out);
@@ -1388,21 +1396,23 @@ static void check_sysdevblock(void)
int main(int argc, char *argv[])
{
struct lsblk _ls;
+ struct libscols_column *cl, *tmp;
int scols_flags = LSBLK_TREE;
+ int sort_flags = sort_none;
int i, c, status = EXIT_FAILURE;
char *outarg = NULL;
static const struct option longopts[] = {
- { "all", 0, 0, 'a' },
+ { "all", 0, 0, 'a' },
{ "bytes", 0, 0, 'b' },
{ "nodeps", 0, 0, 'd' },
{ "discard", 0, 0, 'D' },
- { "help", 0, 0, 'h' },
+ { "help", 0, 0, 'h' },
{ "output", 1, 0, 'o' },
{ "perms", 0, 0, 'm' },
{ "noheadings", 0, 0, 'n' },
{ "list", 0, 0, 'l' },
- { "ascii", 0, 0, 'i' },
+ { "ascii", 0, 0, 'i' },
{ "raw", 0, 0, 'r' },
{ "inverse", 0, 0, 's' },
{ "fs", 0, 0, 'f' },
@@ -1413,6 +1423,7 @@ int main(int argc, char *argv[])
{ "pairs", 0, 0, 'P' },
{ "scsi", 0, 0, 'S' },
{ "version", 0, 0, 'V' },
+ { "order-by", 1, 0, 'c' },
{ NULL, 0, 0, 0 },
};
@@ -1432,7 +1443,7 @@ int main(int argc, char *argv[])
memset(lsblk, 0, sizeof(*lsblk));
while((c = getopt_long(argc, argv,
- "abdDe:fhlnmo:pPiI:rstVS", longopts, NULL)) != -1) {
+ "abdDe:fhlnmo:pPiI:rstVc:S", longopts, NULL)) != -1) {
err_exclusive_options(c, longopts, excl, excl_st);
@@ -1462,6 +1473,13 @@ int main(int argc, char *argv[])
case 'l':
scols_flags &= ~LSBLK_TREE; /* disable the default */
break;
+ case 'c':
+ if(strncasecmp(optarg, "size", 4) == 0)
+ sort_flags = sort_size;
+ else if(strncasecmp(optarg, "name", 4) == 0)
+ sort_flags = sort_name;
+ scols_flags &= ~LSBLK_TREE; /* disable the default */
+ break;
case 'n':
scols_flags |= LSBLK_NOHEADINGS;
break;
@@ -1565,6 +1583,7 @@ int main(int argc, char *argv[])
scols_table_enable_ascii(lsblk->table, !!(scols_flags & LSBLK_ASCII));
scols_table_enable_noheadings(lsblk->table, !!(scols_flags & LSBLK_NOHEADINGS));
+ cl = tmp = NULL;
for (i = 0; i < ncolumns; i++) {
struct colinfo *ci = get_column_info(i);
int fl = ci->flags;
@@ -1572,17 +1591,29 @@ int main(int argc, char *argv[])
if (!(scols_flags & LSBLK_TREE) && get_column_id(i) == COL_NAME)
fl &= ~SCOLS_FL_TREE;
- if (!scols_table_new_column(lsblk->table, ci->name, ci->whint, fl)) {
+ if (!(tmp = scols_table_new_column(lsblk->table, ci->name, ci->whint, fl))) {
warn(_("failed to initialize output column"));
goto leave;
}
+ if(((sort_flags == sort_name) && get_column_id(i) == COL_NAME) ||
+ ((sort_flags == sort_size) && get_column_id(i) == COL_SIZE)){
+ cl = tmp;
+ }
}
if (optind == argc)
status = iterate_block_devices();
else while (optind < argc)
status = process_one_device(argv[optind++]);
-
+
+
+ if(sort_flags == sort_name)
+ scols_column_set_sortcmp(cl, scols_cmpstr_cells, NULL);
+
+ else if(sort_flags == sort_size)
+ scols_column_set_sortcmp(cl, scols_cmpnum_cells, NULL);
+
+ scols_table_set_key_column(lsblk->table, cl);
scols_print_table(lsblk->table);
leave:
--
1.8.3.2
prev parent reply other threads:[~2014-05-11 8:21 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-05-11 8:20 [PATCH v2 1/8] libsmartcols/src/libsmartcols.h.in: add library functions Shakur Shams Mullick
2014-05-11 8:20 ` [PATCH v2 2/8] libsmartcols/src/libsmartcols.sym: add symbols Shakur Shams Mullick
2014-05-11 8:20 ` [PATCH v2 3/8] libsmartcols/src/smartcolsP.h: add new field to struct libscols_table Shakur Shams Mullick
2014-05-11 8:20 ` [PATCH 4/8] libsmartcols/src/cell.c: add two functions to compare number and string cells Shakur Shams Mullick
2014-05-12 12:37 ` Karel Zak
2014-05-11 8:20 ` [PATCH 5/8] libsmartcols/src/column.c: add function to set comparison function scols_column_set_sortcmp() sets the cell comparison function to use Shakur Shams Mullick
2014-05-11 8:20 ` [PATCH v2 6/8] libsmartcols/src/table.c: set and get key column to use for sorting scols_table_set_key_column() sets the column to use scols_table_get_key_column() returns the column being used Shakur Shams Mullick
2014-05-11 8:20 ` [PATCH v2 7/8] libsmartcols/src/table_print.c: sort the list before printing Shakur Shams Mullick
2014-05-12 12:43 ` Karel Zak
2014-05-12 17:22 ` shams
2014-05-11 8:20 ` Shakur Shams Mullick [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=1399796438-5495-8-git-send-email-shakursmullick@gmail.com \
--to=shakursmullick@gmail.com \
--cc=util-linux@vger.kernel.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox