Re: [PATCH] lscpu: add cpu flags entry to summary output

[Heiko Carstens <heiko.carstens@de.ibm.com>]

On Fri, May 15, 2015 at 12:54:02PM +0200, Karel Zak wrote:
> On Fri, May 15, 2015 at 09:37:48AM +0200, Heiko Carstens wrote:
> > On Wed, May 13, 2015 at 11:32:07AM +0200, Karel Zak wrote:
> > > On Wed, May 13, 2015 at 11:15:05AM +0800, WANG Chao wrote:
> > > >  sys-utils/lscpu.c | 3 +++
> > > >  1 file changed, 3 insertions(+)
> > > 
> > >  Applied, thanks.
> > 
> > Hello Karel,
> > 
> > just a general question, from my understanding the the summary output
> > is supposed to be human readable. I'm not sure how a 500+ character
> > line fits into this.
> 
> It's last line printed by lscpu, so terminal wraps it and from my
> point of view it's still pretty human readable.
> 
> > At least it doesn't fit to the current layout of the summary output,
> > but maybe that's just my impression.
> 
> Any suggestion how to improve it?
> 
> I think that provide incomplete info and force people to read also
> /proc/cpuinfo is bad idea.

Ok, I was just wondering what the general "guide line" for stuff like
this is.
When I applied the patch and saw the output it looked strange to me
and it didn't fit to the rest of the output. However that's probably
just my personal taste ;)

Anyway, no further objections.