linux-kernel - Re: [PATCH 3/3] perf, tool: Add new event group management

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 21 Mar 2012 12:54:12 +0100
From:	Peter Zijlstra <a.p.zijlstra@...llo.nl>
To:	Ingo Molnar <mingo@...nel.org>
Cc:	Jiri Olsa <jolsa@...hat.com>, acme@...hat.com, mingo@...e.hu,
	paulus@...ba.org, cjashfor@...ux.vnet.ibm.com, fweisbec@...il.com,
	linux-kernel@...r.kernel.org, Thomas Gleixner <tglx@...utronix.de>
Subject: Re: [PATCH 3/3] perf, tool: Add new event group management

On Wed, 2012-03-21 at 10:52 +0100, Ingo Molnar wrote:
> * Peter Zijlstra <a.p.zijlstra@...llo.nl> wrote:
> 
> > # echo {}
> > {}
> > # echo {en,dis}able
> > enable disable
> > 
> > 
> > It somehow special cases {}, which is horrible.
> 
> Oh, indeed: brace expansion and sequence expressions both use 
> curly braces:
> 
>  $ echo foo-{a,b,c}-bar
>  foo-a-bar foo-b-bar foo-c-bar
> 
>  $ echo {1..10}
>  1 2 3 4 5 6 7 8 9 10
> 
> Too bad, it would be rather intuitive. All the brace characters 
> are taken by Bash.
> 
> Maybe something like:
> 
>  $ echo /minor-faults,major-faults/
>  /minor-faults,major-faults/
> 
> although it looks a bit weird.
> 
> So ... how about using another grouping operator, such as '+'?
> 
> Something like:
> 
>  -e minor-faults+major-faults
> 
> While when comma separated they are not grouped, or so.

I would much prefer a syntax that's more natural but requires quoting
than one that's quirky and tailor made to avoid whatever current bash
does. For one, there's other shells out there that might have different
quoting needs and bash is of course free to extend its syntax.


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/