All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ian Campbell <Ian.Campbell@citrix.com>
To: Ian Jackson <Ian.Jackson@eu.citrix.com>
Cc: Dario Faggioli <dario.faggioli@citrix.com>,
	Wei Liu <wei.liu2@citrix.com>,
	xen-devel@lists.xen.org
Subject: Re: [OSSTEST PATCH] make-flight: reorganize scheduling related test jobs
Date: Thu, 15 Jan 2015 11:15:25 +0000	[thread overview]
Message-ID: <1421320525.19103.339.camel@citrix.com> (raw)
In-Reply-To: <21687.41020.703240.722857@mariner.uk.xensource.com>

On Thu, 2015-01-15 at 11:10 +0000, Ian Jackson wrote:
> Ian Campbell writes ("Re: [OSSTEST PATCH] make-flight: reorganize scheduling related test jobs"):
> > On Wed, 2015-01-14 at 16:56 +0000, Ian Jackson wrote:
> > > That's what I meant, exactly.  But looking at it shows that it would
> > > benefit from being the output of
> > >   ./mg-show-flight-runvars standalone | sort
> > 
> > FWIW I've often thought that mg-show-flight-runvars sorts on the wrong
> > column, at least for the majority of my use cases.
> 
> It's more useful for it to sort on a non-initial column.  If you
> wanted it sorted on the initial column, you can use sort(1).  Whereas
> after your patch, if I want to see all the values of a particular
> variable in all the jobs, I have to do something complicated.

Like grep ;-)

> So if you're going to change this please add an option to make it sort
> the other way.  When I last considered this I thought it more effort
> than it was worth...

Yeah, I think I just need to learn to remember to sort. My finger macros
will figure it out eventually.

Ian.

  reply	other threads:[~2015-01-15 11:15 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-01-08 17:47 [OSSTEST PATCH] make-flight: reorganize scheduling related test jobs Dario Faggioli
2015-01-12 16:52 ` Ian Jackson
2015-01-13 14:15   ` Dario Faggioli
2015-01-14 16:56     ` Ian Jackson
2015-01-15  9:31       ` Ian Campbell
2015-01-15 11:10         ` Ian Jackson
2015-01-15 11:15           ` Ian Campbell [this message]
2015-01-20 15:39       ` Dario Faggioli
2015-01-20 15:54         ` Ian Jackson

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=1421320525.19103.339.camel@citrix.com \
    --to=ian.campbell@citrix.com \
    --cc=Ian.Jackson@eu.citrix.com \
    --cc=dario.faggioli@citrix.com \
    --cc=wei.liu2@citrix.com \
    --cc=xen-devel@lists.xen.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.