[PATCH v1 0/7] Refactor perf python module build
Ian Rogers
irogers at google.com
Thu Jun 13 09:55:17 PDT 2024
On Thu, Jun 13, 2024 at 8:10 AM James Clark <james.clark at arm.com> wrote:
[...]
> >> Reviewed-by: James Clark <james.clark at arm.com>
> >>
> >> It does require a clean build to avoid some -fPIC errors presumably
> >> because not everything that requires it gets rebuilt, for anyone who
> >> gets stuck on that.
> >
> > We need to find a way to avoid requiring the 'make clean' :-/
> >
> > - Arnaldo
> >
>
> Do we need to make it so that if any of the Makefiles are touched it
> does a clean? I'm assuming that was the cause of the issue I experienced
> here and that the Makefile and/or Build files aren't mentioned as
> dependencies of any target.
Perhaps we can do something with the FEATURE_DUMP. It'd be nice to
detect build argument changes and rebuild when that occurs. I'm used
to doing lots of `make clean` due to sanitizer builds, something we
should probably capture through features and/or `perf version
--build-options`. Anyway, out-of-scope for these patches.
I need to cut a v2 due to failing to add the new `.a` files to the clean target.
Thanks for the review,
Ian
More information about the linux-riscv
mailing list