Re: [PATCH v2] kbuild: rust: auto generate rust helper exports

From: Boqun Feng
Date: Sun Aug 18 2024 - 20:29:19 EST


On Mon, Aug 19, 2024 at 02:17:08AM +0200, Miguel Ojeda wrote:
> On Mon, Aug 19, 2024 at 1:56 AM Gary Guo <gary@xxxxxxxxxxx> wrote:
> >
> > It's the mistake on my patch. I noticed the exact issue while I was
> > trying to rebase my helper-lto patch :)
> >
> > I believe that reason that it builds fine for me previously and for
> > Boqun was that we didn't delete the helpers.o generated prior to
> > applying the helper split patch, and we know that kbuild is not
> > hermetic.
>

Oh yes, that's the case, I have a .kunit/rust/helpers.o created at Aug 7
:-(

> Yeah, probably it was that -- as you say, it would require a stale
> `rust/helpers.o` from a build before Andreas' patch (and not just
> before this patch).
>
> > Your rebase and the fix is identical to the one I got locally, and I
> > can confirm that it compiles fine for a clean build.
>
> Thanks a lot for taking a look!
>

Let me rerun your fixed version.

Regards,
Boqun

> Cheers,
> Miguel