mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH V1 1/1] scripts/Kbuild: Add path of fixeddep script
@ 2022-04-22  9:05 Laxman Dewangan
  2022-04-22 17:55 ` Masahiro Yamada
  0 siblings, 1 reply; 2+ messages in thread
From: Laxman Dewangan @ 2022-04-22  9:05 UTC (permalink / raw)
  To: masahiroy, michal.lkml, ndesaulniers
  Cc: linux-kbuild, linux-kernel, Laxman Dewangan

Add path of the fixdep script from the object tree.
This will make the path of script absolute rather than
relative and execute the command from makefile from any
location.

Signed-off-by: Laxman Dewangan <ldewangan@nvidia.com>
---
 scripts/Kbuild.include | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/Kbuild.include b/scripts/Kbuild.include
index cdec22088423e..e07e9265829f8 100644
--- a/scripts/Kbuild.include
+++ b/scripts/Kbuild.include
@@ -147,7 +147,7 @@ if_changed_dep = $(if $(if-changed-cond),$(cmd_and_fixdep),@:)
 
 cmd_and_fixdep =                                                             \
 	$(cmd);                                                              \
-	scripts/basic/fixdep $(depfile) $@ '$(make-cmd)' > $(dot-target).cmd;\
+	$(objtree)/scripts/basic/fixdep $(depfile) $@ '$(make-cmd)' > $(dot-target).cmd;\
 	rm -f $(depfile)
 
 # Usage: $(call if_changed_rule,foo)
-- 
2.17.1


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH V1 1/1] scripts/Kbuild: Add path of fixeddep script
  2022-04-22  9:05 [PATCH V1 1/1] scripts/Kbuild: Add path of fixeddep script Laxman Dewangan
@ 2022-04-22 17:55 ` Masahiro Yamada
  0 siblings, 0 replies; 2+ messages in thread
From: Masahiro Yamada @ 2022-04-22 17:55 UTC (permalink / raw)
  To: Laxman Dewangan
  Cc: Michal Marek, Nick Desaulniers, Linux Kbuild mailing list,
	Linux Kernel Mailing List

On Fri, Apr 22, 2022 at 6:05 PM Laxman Dewangan <ldewangan@nvidia.com> wrote:
>
> Add path of the fixdep script from the object tree.
> This will make the path of script absolute rather than
> relative and execute the command from makefile from any
> location.



Why is this necessary?

Also, this patch does not make anything.
$(objtree)/ is still a relative path.
See the top Makefile.



>
> Signed-off-by: Laxman Dewangan <ldewangan@nvidia.com>
> ---
>  scripts/Kbuild.include | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/scripts/Kbuild.include b/scripts/Kbuild.include
> index cdec22088423e..e07e9265829f8 100644
> --- a/scripts/Kbuild.include
> +++ b/scripts/Kbuild.include
> @@ -147,7 +147,7 @@ if_changed_dep = $(if $(if-changed-cond),$(cmd_and_fixdep),@:)
>
>  cmd_and_fixdep =                                                             \
>         $(cmd);                                                              \
> -       scripts/basic/fixdep $(depfile) $@ '$(make-cmd)' > $(dot-target).cmd;\
> +       $(objtree)/scripts/basic/fixdep $(depfile) $@ '$(make-cmd)' > $(dot-target).cmd;\
>         rm -f $(depfile)
>
>  # Usage: $(call if_changed_rule,foo)
> --
> 2.17.1
>


-- 
Best Regards
Masahiro Yamada

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2022-04-22 18:00 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-04-22  9:05 [PATCH V1 1/1] scripts/Kbuild: Add path of fixeddep script Laxman Dewangan
2022-04-22 17:55 ` Masahiro Yamada

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®