diff options
author | Masahiro Yamada <masahiroy@kernel.org> | 2020-06-19 17:09:55 +0200 |
---|---|---|
committer | Masahiro Yamada <masahiroy@kernel.org> | 2020-06-21 17:19:14 +0200 |
commit | 2c6d9636ad92083d79a060edc84259526c153670 (patch) | |
tree | 781050e27f127022f60bf2327a43360d641bd882 /samples/Kconfig | |
parent | scripts: Fix typo in headers_install.sh (diff) | |
download | linux-2c6d9636ad92083d79a060edc84259526c153670.tar.xz linux-2c6d9636ad92083d79a060edc84259526c153670.zip |
Revert "Makefile: install modules.builtin even if CONFIG_MODULES=n"
This reverts commit e0b250b57dcf403529081e5898a9de717f96b76b,
which broke build systems that need to install files to a certain
path, but do not set INSTALL_MOD_PATH when invoking 'make install'.
$ make INSTALL_PATH=/tmp/destdir install
mkdir: cannot create directory ‘/lib/modules/5.8.0-rc1+/’: Permission denied
Makefile:1342: recipe for target '_builtin_inst_' failed
make: *** [_builtin_inst_] Error 1
While modules.builtin is useful also for CONFIG_MODULES=n, this change
in the behavior is quite unexpected. Maybe "make modules_install"
can install modules.builtin irrespective of CONFIG_MODULES as Jonas
originally suggested.
Anyway, that commit should be reverted ASAP.
Reported-by: Douglas Anderson <dianders@chromium.org>
Reported-by: Guenter Roeck <linux@roeck-us.net>
Cc: Jonas Karlman <jonas@kwiboo.se>
Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
Tested-by: Guenter Roeck <linux@roeck-us.net>
Diffstat (limited to 'samples/Kconfig')
0 files changed, 0 insertions, 0 deletions