On Mon, Mar 04, 2024 at 10:50:49AM +0100, Nicolas Schier wrote:
> On Fri, Mar 01, 2024 at 12:51:01PM -0800, Elliott Mitchell wrote:
> > As $(abs_objtree) and $(abs_srctree) already exist, $(abspath )
> > shouldn't be used with $(objtree) or $(srctree).
> >
> > Fixes: 0e1aa629f1ce ("kbuild: Do not clean resolve_btfids if the output does not exist")
> > Signed-off-by: Elliott Mitchell <ehem+linux@xxxxxxx>
> > ---
> > I believe this is pretty much all fixes, but 0e1aa629f1ce was the most
> > recent one.
>
> If you add a 'Fixes' trailer, please just include changes that fix that
> commit. You're patch does much more than just modifying the changes
> from commit 0e1aa629f1ce.
Since all of these should have been used $(abs_srctree) or $(abs_objtree)
when committed, all of these are fixes. Issue is most of them moved at
least once, so tracking down the list is annoying.
You consider ignoring this is fixing around 5-13 commits reasonable?
On Mon, Mar 04, 2024 at 11:01:24AM +0100, Nicolas Schier wrote:
> oh and please do not re-use the same commit subject line for different
> changes. This will be quite confusing when reading the git history.
Several of these were originally a single commit. Many of them are
doing exactly the same job, just to a distinct filename pattern. Issue
is #5 is already so large squashing things together would likely make
reviewing harder.
--
(\___(\___(\______ --=> 8-) EHM <=-- ______/)___/)___/)
\BS ( | ehem+sigmsg@xxxxxxx PGP 87145445 | ) /
\_CS\ | _____ -O #include <stddisclaimer.h> O- _____ | / _/
8A19\___\_|_/58D2 7E3D DDF4 7BA6 <-PGP-> 41D1 B375 37D0 8714\_|_/___/5445
