Re: [PATCH 3/9] tools/lib/traceevent: Install trace-seq.h API header file

[Steven Rostedt]

On Tue, 4 Dec 2018 16:47:39 +0900
Namhyung Kim <namhyung@xxxxxxxxxx> wrote:


> > @@ -302,6 +302,7 @@ install_headers:
> >  	$(call QUIET_INSTALL, headers) \
> >  		$(call do_install,event-parse.h,$(prefix)/include/traceevent,644); \
> >  		$(call do_install,event-utils.h,$(prefix)/include/traceevent,644); \
> > +		$(call do_install,trace-seq.h,$(prefix)/include/traceevent,644); \
> >  		$(call do_install,kbuffer.h,$(prefix)/include/traceevent,644)  
> 
> Do you still wanna have 'traceevent' directory prefix?  I just
> sometimes feel a bit annoying to type it. ;-)

I'd still want the separate directory for it. I'll probably have a
ftrace.h file added to this for ftrace specific code in the future.

> 
> Or you can rename it something like 'tep' or 'libtep' - and hopefully
> having only single header file to include..
>

Hmm, I wonder if we should just call the directory "trace"?

-- Steve