2017-11-05 18:44 GMT+09:00 Bjørn Forsman <bjorn.forsman@xxxxxxxxx>: > Most places use pwd and rely on $PATH lookup. Moving the remaining > absolute path /bin/pwd users over for consistency. > > Also, a reason for doing /bin/pwd -> pwd instead of the other way around > is because I believe build systems should make little assumptions on > host filesystem layout. Case in point, we do this kind of patching > already in NixOS. > > Ref. commit 028568d84da3cfca49f5f846eeeef01441d70451 > ("kbuild: revert $(realpath ...) to $(shell cd ... && /bin/pwd)"). > > Signed-off-by: Bjørn Forsman <bjorn.forsman@xxxxxxxxx> > --- > Documentation/ia64/xen.txt | 2 +- > Makefile | 2 +- > tools/power/cpupower/Makefile | 2 +- > tools/scripts/Makefile.include | 2 +- > 4 files changed, 4 insertions(+), 4 deletions(-) > > diff --git a/Documentation/ia64/xen.txt b/Documentation/ia64/xen.txt > index c61a99f7c8bb..a12c74ce2773 100644 > --- a/Documentation/ia64/xen.txt > +++ b/Documentation/ia64/xen.txt > @@ -41,7 +41,7 @@ Getting and Building Xen and Dom0 > > 5. make initrd for Dom0/DomU > # make -C linux-2.6.18-xen.hg ARCH=ia64 modules_install \ > - O=$(/bin/pwd)/build-linux-2.6.18-xen_ia64 > + O=$(pwd)/build-linux-2.6.18-xen_ia64 > # mkinitrd -f /boot/efi/efi/redhat/initrd-2.6.18.8-xen.img \ > 2.6.18.8-xen --builtin mptspi --builtin mptbase \ > --builtin mptscsih --builtin uhci-hcd --builtin ohci-hcd \ > diff --git a/Makefile b/Makefile > index 3a8868ee967e..a3eed17080b0 100644 > --- a/Makefile > +++ b/Makefile > @@ -132,7 +132,7 @@ ifneq ($(KBUILD_OUTPUT),) > # check that the output directory actually exists > saved-output := $(KBUILD_OUTPUT) > KBUILD_OUTPUT := $(shell mkdir -p $(KBUILD_OUTPUT) && cd $(KBUILD_OUTPUT) \ > - && /bin/pwd) > + && pwd) > $(if $(KBUILD_OUTPUT),, \ > $(error failed to create output directory "$(saved-output)")) > > diff --git a/tools/power/cpupower/Makefile b/tools/power/cpupower/Makefile > index d6e1c02ddcfe..c76b971d0423 100644 > --- a/tools/power/cpupower/Makefile > +++ b/tools/power/cpupower/Makefile > @@ -26,7 +26,7 @@ endif > > ifneq ($(OUTPUT),) > # check that the output directory actually exists > -OUTDIR := $(shell cd $(OUTPUT) && /bin/pwd) > +OUTDIR := $(shell cd $(OUTPUT) && pwd) > $(if $(OUTDIR),, $(error output directory "$(OUTPUT)" does not exist)) > endif > > diff --git a/tools/scripts/Makefile.include b/tools/scripts/Makefile.include > index 654efd9768fd..3fab179b1aba 100644 > --- a/tools/scripts/Makefile.include > +++ b/tools/scripts/Makefile.include > @@ -13,7 +13,7 @@ endif > > # check that the output directory actually exists > ifneq ($(OUTPUT),) > -OUTDIR := $(shell cd $(OUTPUT) && /bin/pwd) > +OUTDIR := $(shell cd $(OUTPUT) && pwd) > $(if $(OUTDIR),, $(error output directory "$(OUTPUT)" does not exist)) > endif Applied to linux-kbuild/kbuild. Thanks! I will try my best to push this in the current MW. -- Best Regards Masahiro Yamada -- To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html