On Sat, Oct 28, 2023 at 9:51 AM Richard Purdie
<richard.pur...@linuxfoundation.org> wrote:
>
> On Sat, 2023-10-28 at 08:35 -0400, Bruce Ashfield wrote:
> > On Fri, Oct 27, 2023 at 8:07 PM BELHADJ SALEM Talel <bhsta...@gmail.com> 
> > wrote:
> > >
> > > The module class provides what is needed to compile a Linux Kernel module
> > > but since the Makefile is always the same for simple modules that have
> > > their sources in one directory level, I thought of creating new class 
> > > wrapper
> > > to automatically prepare the Makefile.
> > > Also, using variable flag feature, a recipe can provide multiple modules
> > > and can be handled the same way as PACKAGECONFIG is used to 
> > > enabled/disable
> > > features.
> >
> > I find this method of preparing a kernel module much harder to follow
> > (and debug).
> >
> > So Richard is ok with the class, I have no objections, it may serve a class 
> > of
> > developers, but it isn't something I'd support as a first option.
> >
> > But please, create a separate example for it, not modifying hello-world, we 
> > want
> > that simple and explicit example to persist.
>
> Richard isn't convinced.
>
> In most cases a kernel module from a source repository would have some
> form of makefile included with it. yes, if you're creating one from
> scratch, this is helpful but in general you do create a makefile at
> that point and are unlikely to want to keep auto-generating one.
>
> As such I'm not convinced this class matches the siutation most people
> wanting to build software would be in.

Everyone probably knows that I'm not in favour of generating pretty much
anything that can be clearer and more explicit. :)

I re-read my first reply, and it wasn't clear. It should have read "if
Richard is ok",
... those joining words are important.  What I meant to convey is that
I wouldn't
block something like this, if you were ok with it!

Bruce

>
> Cheers,
>
> Richard



-- 
- Thou shalt not follow the NULL pointer, for chaos and madness await
thee at its end
- "Use the force Harry" - Gandalf, Star Trek II
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#189767): 
https://lists.openembedded.org/g/openembedded-core/message/189767
Mute This Topic: https://lists.openembedded.org/mt/102233042/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to