> -----Original Message----- > From: Bruce Ashfield <[email protected]> > Sent: den 25 augusti 2021 15:18 > To: Peter Kjellerstedt <[email protected]> > Cc: Andrey Zhizhikin <[email protected]>; OE Core mailing list > <[email protected]> > Subject: Re: [OE-core] [PATCH] lttng-modules: Make it build when > CONFIG_TRACEPOINTS is not enabled again > > On Wed, Aug 25, 2021 at 8:57 AM Peter Kjellerstedt > <[email protected]> wrote: > > > > > -----Original Message----- > > > From: Andrey Zhizhikin <[email protected]> > > > Sent: den 25 augusti 2021 12:20 > > > To: Peter Kjellerstedt <[email protected]> > > > Cc: Bruce Ashfield <[email protected]>; OE Core mailing list > > > <[email protected]> > > > Subject: Re: [OE-core] [PATCH] lttng-modules: Make it build when > > > CONFIG_TRACEPOINTS is not enabled again > > > > > > Hello Peter, > > > > > > On Wed, Aug 25, 2021 at 11:41 AM Peter Kjellerstedt > > > <[email protected]> wrote: > > > > > > > > Makefile-Do-not-fail-if-CONFIG_TRACEPOINTS-is-not-en.patch was > updated > > > > with the 2.13.0 update (commit 5dad15af), but unfortunately it no > longer > > > > did what it was intended to do. > > > > > > > > Signed-off-by: Peter Kjellerstedt <[email protected]> > > > > --- > > > > ...fail-if-CONFIG_TRACEPOINTS-is-not-en.patch | 42 +++++++++++----- > --- > > > > 1 file changed, 25 insertions(+), 17 deletions(-) > > > > > > > > diff --git a/meta/recipes-kernel/lttng/lttng-modules/Makefile-Do- > not- > > > fail-if-CONFIG_TRACEPOINTS-is-not-en.patch b/meta/recipes- > > > kernel/lttng/lttng-modules/Makefile-Do-not-fail-if-CONFIG_TRACEPOINTS- > is- > > > not-en.patch > > > > > > I guess the patch you've refreshed is not a correct one... > > > > > > There is another patch introduced by Bruce in the commit 18bf7b0 > > > ("lttng-modules: update devupstream to v2.13-rc "), namely > > > 0001-src-Kbuild-change-missing-CONFIG_TRACEPOINTS-to-warn.patch > > > > > > It make sense to refresh rather that one instead of the Makefile one, > > > and have it included in the SRC_URI (see discussion in > > > https://lists.openembedded.org/g/openembedded- > core/topic/84950303#154888). > > > > > > -- > > > Regards, > > > Andrey. > > > > Right, I had missed that there were two patches doing the same thing > > (though neither actually solved the problem after the update to 2.13.0). > > I will send an updated review unifying the patches and adding my > solution > > to disable building the module when CONFIG_TRACEPOINTS is not enabled. > > > > We really should stop dragging these patches along at all. > > I have a unified set of patches here as well, but continuing to allow > lttng modules to build in configurations like these, just doesn't make > sense (which is why I haven't sent them). > > We should have a distro feature, and simply drop it from package lists > when appropriate. > > Bruce > > > //Peter
I know nothing about lttng myself, but has passed on your concerns to our maintainer for the functionality. Unfortunately he is on vacation for a couple of weeks more, so I do not know when I can expect an answer from him. So for now I hope we can update the recipe per my patches so that it continues to work as before, and then we can take the discussion of whether it should be possible to build lttng-modules without CONFIG_TRACEPOINTS enabled separately. //Peter
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#155298): https://lists.openembedded.org/g/openembedded-core/message/155298 Mute This Topic: https://lists.openembedded.org/mt/85131925/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
