<div dir="ltr">OK, the patchdir cleanup won't be needed as you've already applied that .patch file in the repo.<div><br></div><div>But I have forgot to update LIC_FILES_CHKSUM, which shouldn't have the "git/" anymore as it's relative to ${S}.</div><div><br></div><div>LIC_FILES_CHKSUM = "file://tools/kgit;beginline=5;endline=9;md5=9c30e971d435e249624278c3e343e501"<br></div><div><br></div><div>Please adjust in your consolidated pull requests.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Feb 5, 2020 at 3:54 AM Bruce Ashfield <<a href="mailto:bruce.ashfield@gmail.com">bruce.ashfield@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Tue, Feb 4, 2020 at 9:42 PM Martin Jansa <<a href="mailto:martin.jansa@gmail.com" target="_blank">martin.jansa@gmail.com</a>> wrote:<br>
><br>
> Signed-off-by: Martin Jansa <<a href="mailto:Martin.Jansa@gmail.com" target="_blank">Martin.Jansa@gmail.com</a>><br>
> ---<br>
> .../kern-tools/<a href="http://kern-tools-native_git.bb" rel="noreferrer" target="_blank">kern-tools-native_git.bb</a> | 11 +++++++----<br>
> 1 file changed, 7 insertions(+), 4 deletions(-)<br>
><br>
> diff --git a/meta/recipes-kernel/kern-tools/<a href="http://kern-tools-native_git.bb" rel="noreferrer" target="_blank">kern-tools-native_git.bb</a> b/meta/recipes-kernel/kern-tools/<a href="http://kern-tools-native_git.bb" rel="noreferrer" target="_blank">kern-tools-native_git.bb</a><br>
> index 8053483355..28080e510f 100644<br>
> --- a/meta/recipes-kernel/kern-tools/<a href="http://kern-tools-native_git.bb" rel="noreferrer" target="_blank">kern-tools-native_git.bb</a><br>
> +++ b/meta/recipes-kernel/kern-tools/<a href="http://kern-tools-native_git.bb" rel="noreferrer" target="_blank">kern-tools-native_git.bb</a><br>
> @@ -11,16 +11,19 @@ PV = "0.2+git${SRCPV}"<br>
> inherit native<br>
><br>
> SRC_URI = "git://<a href="http://git.yoctoproject.org/yocto-kernel-tools.git" rel="noreferrer" target="_blank">git.yoctoproject.org/yocto-kernel-tools.git</a>"<br>
> -S = "${WORKDIR}"<br>
> +S = "${WORKDIR}/git"<br>
<br>
While not common, this was on purpose. This isn't something that is<br>
typically patched, it is something I queue and bump for release.<br>
<br>
So I'm ok with the cleanup, but I'll drop the patch tweaking part when<br>
I queue for test, since I'll have the patch already in the repo.<br>
<br>
Bruce<br>
<br>
> UPSTREAM_CHECK_COMMITS = "1"<br>
><br>
> +do_configure() {<br>
> + :<br>
> +}<br>
> +<br>
> do_compile() {<br>
> :<br>
> }<br>
><br>
> do_install() {<br>
> - cd ${S}/git<br>
> - make DESTDIR=${D}${bindir} install<br>
> + oe_runmake DESTDIR=${D}${bindir} install<br>
> }<br>
><br>
> -SRC_URI += "file://0001-merge_config.sh-pass-LD-variable-from-shell-environm.patch;patchdir=git"<br>
> +SRC_URI += "file://0001-merge_config.sh-pass-LD-variable-from-shell-environm.patch"<br>
> --<br>
> 2.20.1<br>
><br>
<br>
<br>
-- <br>
- Thou shalt not follow the NULL pointer, for chaos and madness await<br>
thee at its end<br>
- "Use the force Harry" - Gandalf, Star Trek II<br>
</blockquote></div>