module build:replace BUILD_LOADABLE with MODULES

[xuxin930]

Summary

this patch is relate to: apache/nuttx#13452

Impact

Testing

[cederom]

@patacongo please advise on nomenclature change :-)

[patacongo]

@patacongo please advise on nomenclature change :-)

I am not so much involved these days but in the past, configuration names began with CONFIG_ (of course) then what was being modified ( for example, NET for anyting affecting the network, and then the remainder describing what is being configured.

So by those old rules, CONFIG_MODULES would be wrong. Since it affects the build system, it should be CONFIG_BUILD_MODULES.

What is configured? BUILD system
What is controlled in the build system? MODULE support.

But since no one has been monitoring this I am sure that that naming system has gone to hell.

This is not documented to my knowledge, but there are other documented naming conventions here: https://cwiki.apache.org/confluence/display/NUTTX/Wiki

[cederom]

Lets finish the nomenclature change discussion resolution first on the nuttx PR: apache/nuttx#13452.

[masayuki2009]

@xuxin930 @xiaoxiang781216
I noticed that the PR does not work with kconfig-frontends but works with kconfiglib.
Do you mean that kconfig-frontends is no longer supported?

[xuxin930]

@xuxin930 @xiaoxiang781216 I noticed that the PR does not work with kconfig-frontends but works with kconfiglib. Do you mean that kconfig-frontends is no longer supported?

Hi @masayuki2009 , sorry for the compatibility issue. I'll try to modify it to add instead of replace to ensure compatibility

[xuxin930]

@xuxin930 @xiaoxiang781216 I noticed that the PR does not work with kconfig-frontends but works with kconfiglib. Do you mean that kconfig-frontends is no longer supported?

hi @masayuki2009 i add both MODULE config and modules option in this patch apache/nuttx#13532.
which both work in two Kconfig implementation library.