Messages in this thread | | | Date | Sat, 15 Mar 2014 03:27:07 +0100 | From | Andi Kleen <> | Subject | Re: [PATCH 01/10] perf, tools: Add jsmn `jasmine' JSON parser |
| |
On Fri, Mar 14, 2014 at 04:41:31PM -0600, David Ahern wrote: > On 3/14/14, 3:31 PM, Andi Kleen wrote: > >@@ -374,6 +376,8 @@ LIB_OBJS += $(OUTPUT)util/stat.o > > LIB_OBJS += $(OUTPUT)util/record.o > > LIB_OBJS += $(OUTPUT)util/srcline.o > > LIB_OBJS += $(OUTPUT)util/data.o > >+LIB_OBJS += $(OUTPUT)util/jsmn.o > >+LIB_OBJS += $(OUTPUT)util/json.o > > > > LIB_OBJS += $(OUTPUT)ui/setup.o > > LIB_OBJS += $(OUTPUT)ui/helpline.o > > > > CONFIG driven? Allow a user to omit this.
Why? It has no external dependencies. AFAIK that's the only reason for configs.
(unless you count the commands used by the shell script, but the basic parser works fine even without the script)
-Andi
| |