Messages in this thread Patch in this message | | | Date | Thu, 28 May 2015 13:57:16 -0700 | From | Sukadev Bhattiprolu <> | Subject | Re: [PATCH 3/3] perf, tools: Fix jevents dependencies |
| |
Andi Kleen [andi@firstfloor.org] wrote: | From: Andi Kleen <ak@linux.intel.com> | | The "find" based dependencies for the JSON files didn't work me. I | didn't get a rebuild when the JSON files changed.
Hmm, it works for me every time. I like the 'find' better because it is more compact and allows mulitple levels.
| Change it to | use wildcard. Also add a dependency for the jevents binary itself | and for the mapfile.
I was trying this to address Jiri's comment about being silent if some architectures don't have JSON files (and based on your comments above, added the depedency for mapfile and jevents)
diff --git a/tools/perf/pmu-events/Build b/tools/perf/pmu-events/Build index 7e0c85c..45b05a7 100644 --- a/tools/perf/pmu-events/Build +++ b/tools/perf/pmu-events/Build @@ -1,10 +1,12 @@ jevents-y += json.o jsmn.o jevents.o pmu-events-y += pmu-events.o -JSON = $(shell find pmu-events/arch/$(ARCH) -name '*.json') - +EVDIR = pmu-events/arch/$(ARCH) +JSON = $(shell [ -d $(EVDIR) ] && \ + find $(EVDIR) -name '*.json' -o -name 'mapfile*') +# # # Locate/process JSON files in pmu-events/arch/ # directory and create tables in pmu-events.c. # -$(OUTPUT)pmu-events/pmu-events.c: $(JSON) +$(OUTPUT)pmu-events/pmu-events.c: $(JSON) $(JEVENTS) $(Q)$(call echo-cmd,gen)$(JEVENTS) $(ARCH) pmu-events/arch $(OUTPUT)pmu-events/pmu-events.c
| |