Re: [PATCH v3 2/2] kconfig: rename silentoldconfig to syncconfig
2018-03-01 15:34 GMT+09:00 Masahiro Yamada: > As commit cedd55d49dee ("kconfig: Remove silentoldconfig from help > and docs; fix kconfig/conf's help") mentioned, 'silentoldconfig' is a > historical misnomer. That commit removed it from help and docs since > it is an internal interface. If so, it should be allowed to rename > it to something more intuitive. 'syncconfig' is the one I came up > with because it updates the .config if necessary, then synchronize > include/generated/autoconf.h and include/config/* with it. > > You should not manually invoke 'silentoldcofig'. Display warning if > used in case existing scripts are doing wrong. > > Signed-off-by: Masahiro Yamada > --- > > Changes in v3: > - Fix Documentation/networking/i40e.txt > - Display warning if silentoldconfig is used > > Changes in v2: > - newly added > Both applied to linux-kbuild/kconfig. -- Best Regards Masahiro Yamada
Re: [PATCH v3 2/2] kconfig: rename silentoldconfig to syncconfig
2018-03-01 15:34 GMT+09:00 Masahiro Yamada : > As commit cedd55d49dee ("kconfig: Remove silentoldconfig from help > and docs; fix kconfig/conf's help") mentioned, 'silentoldconfig' is a > historical misnomer. That commit removed it from help and docs since > it is an internal interface. If so, it should be allowed to rename > it to something more intuitive. 'syncconfig' is the one I came up > with because it updates the .config if necessary, then synchronize > include/generated/autoconf.h and include/config/* with it. > > You should not manually invoke 'silentoldcofig'. Display warning if > used in case existing scripts are doing wrong. > > Signed-off-by: Masahiro Yamada > --- > > Changes in v3: > - Fix Documentation/networking/i40e.txt > - Display warning if silentoldconfig is used > > Changes in v2: > - newly added > Both applied to linux-kbuild/kconfig. -- Best Regards Masahiro Yamada
Re: [PATCH v3 2/2] kconfig: rename silentoldconfig to syncconfig
On Thu, Mar 1, 2018 at 7:34 AM, Masahiro Yamadawrote: > As commit cedd55d49dee ("kconfig: Remove silentoldconfig from help > and docs; fix kconfig/conf's help") mentioned, 'silentoldconfig' is a > historical misnomer. That commit removed it from help and docs since > it is an internal interface. If so, it should be allowed to rename > it to something more intuitive. 'syncconfig' is the one I came up > with because it updates the .config if necessary, then synchronize > include/generated/autoconf.h and include/config/* with it. > > You should not manually invoke 'silentoldcofig'. Display warning if > used in case existing scripts are doing wrong. > > Signed-off-by: Masahiro Yamada > --- > > Changes in v3: > - Fix Documentation/networking/i40e.txt > - Display warning if silentoldconfig is used > > Changes in v2: > - newly added > > Documentation/kbuild/kconfig.txt | 2 +- > Documentation/networking/i40e.txt | 2 +- > Makefile | 2 +- > scripts/kconfig/Makefile | 13 ++--- > scripts/kconfig/conf.c| 20 ++-- > 5 files changed, 23 insertions(+), 16 deletions(-) > > diff --git a/Documentation/kbuild/kconfig.txt > b/Documentation/kbuild/kconfig.txt > index bbc99c0..7233118 100644 > --- a/Documentation/kbuild/kconfig.txt > +++ b/Documentation/kbuild/kconfig.txt > @@ -119,7 +119,7 @@ Examples: > 15% of tristates will be set to 'y', 15% to 'm', 70% to 'n' > > __ > -Environment variables for 'silentoldconfig' > +Environment variables for 'syncconfig' > > KCONFIG_NOSILENTUPDATE > -- > diff --git a/Documentation/networking/i40e.txt > b/Documentation/networking/i40e.txt > index 57e616e..c2d6e18 100644 > --- a/Documentation/networking/i40e.txt > +++ b/Documentation/networking/i40e.txt > @@ -32,7 +32,7 @@ Enabling the driver > The driver is enabled via the standard kernel configuration system, > using the make command: > > - Make oldconfig/silentoldconfig/menuconfig/etc. > + make config/oldconfig/menuconfig/etc. > > The driver is located in the menu structure at: > > diff --git a/Makefile b/Makefile > index 7f0aba6..4fb97e9 100644 > --- a/Makefile > +++ b/Makefile > @@ -597,7 +597,7 @@ $(KCONFIG_CONFIG) include/config/auto.conf.cmd: ; > # include/generated/ and include/config/. Update them if .config is newer > than > # include/config/auto.conf (which mirrors .config). > include/config/%.conf: $(KCONFIG_CONFIG) include/config/auto.conf.cmd > - $(Q)$(MAKE) -f $(srctree)/Makefile silentoldconfig > + $(Q)$(MAKE) -f $(srctree)/Makefile syncconfig > else > # external modules needs include/generated/autoconf.h and > include/config/auto.conf > # but do not care if they are up-to-date. Use auto.conf to trigger the test > diff --git a/scripts/kconfig/Makefile b/scripts/kconfig/Makefile > index 41e2a9f..753a6de4 100644 > --- a/scripts/kconfig/Makefile > +++ b/scripts/kconfig/Makefile > @@ -3,7 +3,7 @@ > # Kernel configuration targets > # These targets are used from top-level makefile > > -PHONY += xconfig gconfig menuconfig config silentoldconfig update-po-config \ > +PHONY += xconfig gconfig menuconfig config syncconfig update-po-config \ > localmodconfig localyesconfig > > ifdef KBUILD_KCONFIG > @@ -36,7 +36,7 @@ nconfig: $(obj)/nconf > > # This has become an internal implementation detail and is now deprecated > # for external use. > -silentoldconfig: $(obj)/conf > +syncconfig: $(obj)/conf > $(Q)mkdir -p include/config include/generated > $(Q)test -e include/generated/autoksyms.h || \ > touch include/generated/autoksyms.h > @@ -88,7 +88,7 @@ PHONY += $(simple-targets) > $(simple-targets): $(obj)/conf > $< $(silent) --$@ $(Kconfig) > > -PHONY += oldnoconfig savedefconfig defconfig > +PHONY += oldnoconfig silentoldconfig savedefconfig defconfig > > # oldnoconfig is an alias of olddefconfig, because people already are > dependent > # on its behavior (sets new symbols to their default value but not 'n') with > the > @@ -97,6 +97,13 @@ oldnoconfig: olddefconfig > @echo " WARNING: \"oldnoconfig\" target will be removed after Linux > 4.19" > @echo "Please use \"olddefconfig\" instead, which is an > alias." > > +# We do not expect manual invokcation of "silentoldcofig" (or "syncconfig"). > +silentoldconfig: syncconfig > + @echo " WARNING: \"silentoldconfig\" has been renamed to > \"syncconfig\"" > + @echo "and is now an internal implementation detail." > + @echo "What you want is probably \"oldconfig\"." > + @echo "\"silentoldconfig\" will be removed after Linux > 4.19" > + > savedefconfig: $(obj)/conf > $< $(silent) --$@=defconfig $(Kconfig) > > diff --git
Re: [PATCH v3 2/2] kconfig: rename silentoldconfig to syncconfig
On Thu, Mar 1, 2018 at 7:34 AM, Masahiro Yamada wrote: > As commit cedd55d49dee ("kconfig: Remove silentoldconfig from help > and docs; fix kconfig/conf's help") mentioned, 'silentoldconfig' is a > historical misnomer. That commit removed it from help and docs since > it is an internal interface. If so, it should be allowed to rename > it to something more intuitive. 'syncconfig' is the one I came up > with because it updates the .config if necessary, then synchronize > include/generated/autoconf.h and include/config/* with it. > > You should not manually invoke 'silentoldcofig'. Display warning if > used in case existing scripts are doing wrong. > > Signed-off-by: Masahiro Yamada > --- > > Changes in v3: > - Fix Documentation/networking/i40e.txt > - Display warning if silentoldconfig is used > > Changes in v2: > - newly added > > Documentation/kbuild/kconfig.txt | 2 +- > Documentation/networking/i40e.txt | 2 +- > Makefile | 2 +- > scripts/kconfig/Makefile | 13 ++--- > scripts/kconfig/conf.c| 20 ++-- > 5 files changed, 23 insertions(+), 16 deletions(-) > > diff --git a/Documentation/kbuild/kconfig.txt > b/Documentation/kbuild/kconfig.txt > index bbc99c0..7233118 100644 > --- a/Documentation/kbuild/kconfig.txt > +++ b/Documentation/kbuild/kconfig.txt > @@ -119,7 +119,7 @@ Examples: > 15% of tristates will be set to 'y', 15% to 'm', 70% to 'n' > > __ > -Environment variables for 'silentoldconfig' > +Environment variables for 'syncconfig' > > KCONFIG_NOSILENTUPDATE > -- > diff --git a/Documentation/networking/i40e.txt > b/Documentation/networking/i40e.txt > index 57e616e..c2d6e18 100644 > --- a/Documentation/networking/i40e.txt > +++ b/Documentation/networking/i40e.txt > @@ -32,7 +32,7 @@ Enabling the driver > The driver is enabled via the standard kernel configuration system, > using the make command: > > - Make oldconfig/silentoldconfig/menuconfig/etc. > + make config/oldconfig/menuconfig/etc. > > The driver is located in the menu structure at: > > diff --git a/Makefile b/Makefile > index 7f0aba6..4fb97e9 100644 > --- a/Makefile > +++ b/Makefile > @@ -597,7 +597,7 @@ $(KCONFIG_CONFIG) include/config/auto.conf.cmd: ; > # include/generated/ and include/config/. Update them if .config is newer > than > # include/config/auto.conf (which mirrors .config). > include/config/%.conf: $(KCONFIG_CONFIG) include/config/auto.conf.cmd > - $(Q)$(MAKE) -f $(srctree)/Makefile silentoldconfig > + $(Q)$(MAKE) -f $(srctree)/Makefile syncconfig > else > # external modules needs include/generated/autoconf.h and > include/config/auto.conf > # but do not care if they are up-to-date. Use auto.conf to trigger the test > diff --git a/scripts/kconfig/Makefile b/scripts/kconfig/Makefile > index 41e2a9f..753a6de4 100644 > --- a/scripts/kconfig/Makefile > +++ b/scripts/kconfig/Makefile > @@ -3,7 +3,7 @@ > # Kernel configuration targets > # These targets are used from top-level makefile > > -PHONY += xconfig gconfig menuconfig config silentoldconfig update-po-config \ > +PHONY += xconfig gconfig menuconfig config syncconfig update-po-config \ > localmodconfig localyesconfig > > ifdef KBUILD_KCONFIG > @@ -36,7 +36,7 @@ nconfig: $(obj)/nconf > > # This has become an internal implementation detail and is now deprecated > # for external use. > -silentoldconfig: $(obj)/conf > +syncconfig: $(obj)/conf > $(Q)mkdir -p include/config include/generated > $(Q)test -e include/generated/autoksyms.h || \ > touch include/generated/autoksyms.h > @@ -88,7 +88,7 @@ PHONY += $(simple-targets) > $(simple-targets): $(obj)/conf > $< $(silent) --$@ $(Kconfig) > > -PHONY += oldnoconfig savedefconfig defconfig > +PHONY += oldnoconfig silentoldconfig savedefconfig defconfig > > # oldnoconfig is an alias of olddefconfig, because people already are > dependent > # on its behavior (sets new symbols to their default value but not 'n') with > the > @@ -97,6 +97,13 @@ oldnoconfig: olddefconfig > @echo " WARNING: \"oldnoconfig\" target will be removed after Linux > 4.19" > @echo "Please use \"olddefconfig\" instead, which is an > alias." > > +# We do not expect manual invokcation of "silentoldcofig" (or "syncconfig"). > +silentoldconfig: syncconfig > + @echo " WARNING: \"silentoldconfig\" has been renamed to > \"syncconfig\"" > + @echo "and is now an internal implementation detail." > + @echo "What you want is probably \"oldconfig\"." > + @echo "\"silentoldconfig\" will be removed after Linux > 4.19" > + > savedefconfig: $(obj)/conf > $< $(silent) --$@=defconfig $(Kconfig) > > diff --git a/scripts/kconfig/conf.c b/scripts/kconfig/conf.c > index
[PATCH v3 2/2] kconfig: rename silentoldconfig to syncconfig
As commit cedd55d49dee ("kconfig: Remove silentoldconfig from help and docs; fix kconfig/conf's help") mentioned, 'silentoldconfig' is a historical misnomer. That commit removed it from help and docs since it is an internal interface. If so, it should be allowed to rename it to something more intuitive. 'syncconfig' is the one I came up with because it updates the .config if necessary, then synchronize include/generated/autoconf.h and include/config/* with it. You should not manually invoke 'silentoldcofig'. Display warning if used in case existing scripts are doing wrong. Signed-off-by: Masahiro Yamada--- Changes in v3: - Fix Documentation/networking/i40e.txt - Display warning if silentoldconfig is used Changes in v2: - newly added Documentation/kbuild/kconfig.txt | 2 +- Documentation/networking/i40e.txt | 2 +- Makefile | 2 +- scripts/kconfig/Makefile | 13 ++--- scripts/kconfig/conf.c| 20 ++-- 5 files changed, 23 insertions(+), 16 deletions(-) diff --git a/Documentation/kbuild/kconfig.txt b/Documentation/kbuild/kconfig.txt index bbc99c0..7233118 100644 --- a/Documentation/kbuild/kconfig.txt +++ b/Documentation/kbuild/kconfig.txt @@ -119,7 +119,7 @@ Examples: 15% of tristates will be set to 'y', 15% to 'm', 70% to 'n' __ -Environment variables for 'silentoldconfig' +Environment variables for 'syncconfig' KCONFIG_NOSILENTUPDATE -- diff --git a/Documentation/networking/i40e.txt b/Documentation/networking/i40e.txt index 57e616e..c2d6e18 100644 --- a/Documentation/networking/i40e.txt +++ b/Documentation/networking/i40e.txt @@ -32,7 +32,7 @@ Enabling the driver The driver is enabled via the standard kernel configuration system, using the make command: - Make oldconfig/silentoldconfig/menuconfig/etc. + make config/oldconfig/menuconfig/etc. The driver is located in the menu structure at: diff --git a/Makefile b/Makefile index 7f0aba6..4fb97e9 100644 --- a/Makefile +++ b/Makefile @@ -597,7 +597,7 @@ $(KCONFIG_CONFIG) include/config/auto.conf.cmd: ; # include/generated/ and include/config/. Update them if .config is newer than # include/config/auto.conf (which mirrors .config). include/config/%.conf: $(KCONFIG_CONFIG) include/config/auto.conf.cmd - $(Q)$(MAKE) -f $(srctree)/Makefile silentoldconfig + $(Q)$(MAKE) -f $(srctree)/Makefile syncconfig else # external modules needs include/generated/autoconf.h and include/config/auto.conf # but do not care if they are up-to-date. Use auto.conf to trigger the test diff --git a/scripts/kconfig/Makefile b/scripts/kconfig/Makefile index 41e2a9f..753a6de4 100644 --- a/scripts/kconfig/Makefile +++ b/scripts/kconfig/Makefile @@ -3,7 +3,7 @@ # Kernel configuration targets # These targets are used from top-level makefile -PHONY += xconfig gconfig menuconfig config silentoldconfig update-po-config \ +PHONY += xconfig gconfig menuconfig config syncconfig update-po-config \ localmodconfig localyesconfig ifdef KBUILD_KCONFIG @@ -36,7 +36,7 @@ nconfig: $(obj)/nconf # This has become an internal implementation detail and is now deprecated # for external use. -silentoldconfig: $(obj)/conf +syncconfig: $(obj)/conf $(Q)mkdir -p include/config include/generated $(Q)test -e include/generated/autoksyms.h || \ touch include/generated/autoksyms.h @@ -88,7 +88,7 @@ PHONY += $(simple-targets) $(simple-targets): $(obj)/conf $< $(silent) --$@ $(Kconfig) -PHONY += oldnoconfig savedefconfig defconfig +PHONY += oldnoconfig silentoldconfig savedefconfig defconfig # oldnoconfig is an alias of olddefconfig, because people already are dependent # on its behavior (sets new symbols to their default value but not 'n') with the @@ -97,6 +97,13 @@ oldnoconfig: olddefconfig @echo " WARNING: \"oldnoconfig\" target will be removed after Linux 4.19" @echo "Please use \"olddefconfig\" instead, which is an alias." +# We do not expect manual invokcation of "silentoldcofig" (or "syncconfig"). +silentoldconfig: syncconfig + @echo " WARNING: \"silentoldconfig\" has been renamed to \"syncconfig\"" + @echo "and is now an internal implementation detail." + @echo "What you want is probably \"oldconfig\"." + @echo "\"silentoldconfig\" will be removed after Linux 4.19" + savedefconfig: $(obj)/conf $< $(silent) --$@=defconfig $(Kconfig) diff --git a/scripts/kconfig/conf.c b/scripts/kconfig/conf.c index 11a4e45..4e08121 100644 --- a/scripts/kconfig/conf.c +++ b/scripts/kconfig/conf.c @@ -23,7 +23,7 @@ static void check_conf(struct menu *menu); enum input_mode { oldaskconfig, - silentoldconfig, + syncconfig, oldconfig,
[PATCH v3 2/2] kconfig: rename silentoldconfig to syncconfig
As commit cedd55d49dee ("kconfig: Remove silentoldconfig from help and docs; fix kconfig/conf's help") mentioned, 'silentoldconfig' is a historical misnomer. That commit removed it from help and docs since it is an internal interface. If so, it should be allowed to rename it to something more intuitive. 'syncconfig' is the one I came up with because it updates the .config if necessary, then synchronize include/generated/autoconf.h and include/config/* with it. You should not manually invoke 'silentoldcofig'. Display warning if used in case existing scripts are doing wrong. Signed-off-by: Masahiro Yamada --- Changes in v3: - Fix Documentation/networking/i40e.txt - Display warning if silentoldconfig is used Changes in v2: - newly added Documentation/kbuild/kconfig.txt | 2 +- Documentation/networking/i40e.txt | 2 +- Makefile | 2 +- scripts/kconfig/Makefile | 13 ++--- scripts/kconfig/conf.c| 20 ++-- 5 files changed, 23 insertions(+), 16 deletions(-) diff --git a/Documentation/kbuild/kconfig.txt b/Documentation/kbuild/kconfig.txt index bbc99c0..7233118 100644 --- a/Documentation/kbuild/kconfig.txt +++ b/Documentation/kbuild/kconfig.txt @@ -119,7 +119,7 @@ Examples: 15% of tristates will be set to 'y', 15% to 'm', 70% to 'n' __ -Environment variables for 'silentoldconfig' +Environment variables for 'syncconfig' KCONFIG_NOSILENTUPDATE -- diff --git a/Documentation/networking/i40e.txt b/Documentation/networking/i40e.txt index 57e616e..c2d6e18 100644 --- a/Documentation/networking/i40e.txt +++ b/Documentation/networking/i40e.txt @@ -32,7 +32,7 @@ Enabling the driver The driver is enabled via the standard kernel configuration system, using the make command: - Make oldconfig/silentoldconfig/menuconfig/etc. + make config/oldconfig/menuconfig/etc. The driver is located in the menu structure at: diff --git a/Makefile b/Makefile index 7f0aba6..4fb97e9 100644 --- a/Makefile +++ b/Makefile @@ -597,7 +597,7 @@ $(KCONFIG_CONFIG) include/config/auto.conf.cmd: ; # include/generated/ and include/config/. Update them if .config is newer than # include/config/auto.conf (which mirrors .config). include/config/%.conf: $(KCONFIG_CONFIG) include/config/auto.conf.cmd - $(Q)$(MAKE) -f $(srctree)/Makefile silentoldconfig + $(Q)$(MAKE) -f $(srctree)/Makefile syncconfig else # external modules needs include/generated/autoconf.h and include/config/auto.conf # but do not care if they are up-to-date. Use auto.conf to trigger the test diff --git a/scripts/kconfig/Makefile b/scripts/kconfig/Makefile index 41e2a9f..753a6de4 100644 --- a/scripts/kconfig/Makefile +++ b/scripts/kconfig/Makefile @@ -3,7 +3,7 @@ # Kernel configuration targets # These targets are used from top-level makefile -PHONY += xconfig gconfig menuconfig config silentoldconfig update-po-config \ +PHONY += xconfig gconfig menuconfig config syncconfig update-po-config \ localmodconfig localyesconfig ifdef KBUILD_KCONFIG @@ -36,7 +36,7 @@ nconfig: $(obj)/nconf # This has become an internal implementation detail and is now deprecated # for external use. -silentoldconfig: $(obj)/conf +syncconfig: $(obj)/conf $(Q)mkdir -p include/config include/generated $(Q)test -e include/generated/autoksyms.h || \ touch include/generated/autoksyms.h @@ -88,7 +88,7 @@ PHONY += $(simple-targets) $(simple-targets): $(obj)/conf $< $(silent) --$@ $(Kconfig) -PHONY += oldnoconfig savedefconfig defconfig +PHONY += oldnoconfig silentoldconfig savedefconfig defconfig # oldnoconfig is an alias of olddefconfig, because people already are dependent # on its behavior (sets new symbols to their default value but not 'n') with the @@ -97,6 +97,13 @@ oldnoconfig: olddefconfig @echo " WARNING: \"oldnoconfig\" target will be removed after Linux 4.19" @echo "Please use \"olddefconfig\" instead, which is an alias." +# We do not expect manual invokcation of "silentoldcofig" (or "syncconfig"). +silentoldconfig: syncconfig + @echo " WARNING: \"silentoldconfig\" has been renamed to \"syncconfig\"" + @echo "and is now an internal implementation detail." + @echo "What you want is probably \"oldconfig\"." + @echo "\"silentoldconfig\" will be removed after Linux 4.19" + savedefconfig: $(obj)/conf $< $(silent) --$@=defconfig $(Kconfig) diff --git a/scripts/kconfig/conf.c b/scripts/kconfig/conf.c index 11a4e45..4e08121 100644 --- a/scripts/kconfig/conf.c +++ b/scripts/kconfig/conf.c @@ -23,7 +23,7 @@ static void check_conf(struct menu *menu); enum input_mode { oldaskconfig, - silentoldconfig, + syncconfig, oldconfig, allnoconfig, allyesconfig,