Re: [U-Boot] [PATCH 1/5] check-config: fix wrong comment about how to build whitelist
2016-10-06 1:50 GMT+09:00 Simon Glass: > BTW, what do you think about updating moveconfig.py to remove things > from the whitelist as well? Yeah, I was also thinking of this. -- Best Regards Masahiro Yamada ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 1/5] check-config: fix wrong comment about how to build whitelist
On 26 September 2016 at 19:13, Masahiro Yamadawrote: > Hi Simon, > > 2016-09-27 9:34 GMT+09:00 Simon Glass : >> Hi Masahiro, >> >> On 25 September 2016 at 22:04, Masahiro Yamada >> wrote: >>> The command suggested in this comment block is wrong; it would not >>> rip off CONFIG options that had already been converted to Kconfig. >>> >>> Instead, we should use the scripts/build-whitelist.sh tool. >>> >>> Signed-off-by: Masahiro Yamada >>> --- >>> >>> scripts/check-config.sh | 9 ++--- >>> 1 file changed, 2 insertions(+), 7 deletions(-) >>> >>> diff --git a/scripts/check-config.sh b/scripts/check-config.sh >>> index 28c8fe9..6618dfb 100755 >>> --- a/scripts/check-config.sh >>> +++ b/scripts/check-config.sh >>> @@ -5,13 +5,8 @@ >>> # Check that the u-boot.cfg file provided does not introduce any new >>> # ad-hoc CONFIG options >>> # >>> -# You can generate the list of current ad-hoc CONFIG options (those which >>> are >>> -# not in Kconfig) with this command: >>> -# >>> -# export LC_ALL=C LC_COLLATE=C >>> -# git grep CONFIG_ |tr ' \t' '\n\n' |sed -n >>> 's/^\(CONFIG_[A-Z0-9_]*\).*/\1/p' \ >>> -# |sort |uniq >scripts/config_whitelist.txt; >>> -# unset LC_ALL LC_COLLATE >>> +# Use scripts/build-whitelist.sh to generate the list of current ad-hoc >>> +# CONFIG options (those which are not in Kconfig). >> >> For me the LC setup is needed. Does it work correctly without it for >> you? I found that the sorting was wrong. > > > I am not quite sure about this, but it worked for me. > > > > I can see > > export LC_ALL=C > export LC_COLLATE=C > > in both check-config.sh and build-whitelist.sh > > > That's why? OK thanks. I must have fixed it and forgotten about it. Reviewed-by: Simon Glass BTW, what do you think about updating moveconfig.py to remove things from the whitelist as well? Regards, Simon ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 1/5] check-config: fix wrong comment about how to build whitelist
Hi Simon, 2016-09-27 9:34 GMT+09:00 Simon Glass: > Hi Masahiro, > > On 25 September 2016 at 22:04, Masahiro Yamada > wrote: >> The command suggested in this comment block is wrong; it would not >> rip off CONFIG options that had already been converted to Kconfig. >> >> Instead, we should use the scripts/build-whitelist.sh tool. >> >> Signed-off-by: Masahiro Yamada >> --- >> >> scripts/check-config.sh | 9 ++--- >> 1 file changed, 2 insertions(+), 7 deletions(-) >> >> diff --git a/scripts/check-config.sh b/scripts/check-config.sh >> index 28c8fe9..6618dfb 100755 >> --- a/scripts/check-config.sh >> +++ b/scripts/check-config.sh >> @@ -5,13 +5,8 @@ >> # Check that the u-boot.cfg file provided does not introduce any new >> # ad-hoc CONFIG options >> # >> -# You can generate the list of current ad-hoc CONFIG options (those which >> are >> -# not in Kconfig) with this command: >> -# >> -# export LC_ALL=C LC_COLLATE=C >> -# git grep CONFIG_ |tr ' \t' '\n\n' |sed -n >> 's/^\(CONFIG_[A-Z0-9_]*\).*/\1/p' \ >> -# |sort |uniq >scripts/config_whitelist.txt; >> -# unset LC_ALL LC_COLLATE >> +# Use scripts/build-whitelist.sh to generate the list of current ad-hoc >> +# CONFIG options (those which are not in Kconfig). > > For me the LC setup is needed. Does it work correctly without it for > you? I found that the sorting was wrong. I am not quite sure about this, but it worked for me. I can see export LC_ALL=C export LC_COLLATE=C in both check-config.sh and build-whitelist.sh That's why? -- Best Regards Masahiro Yamada ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 1/5] check-config: fix wrong comment about how to build whitelist
Hi Masahiro, On 25 September 2016 at 22:04, Masahiro Yamadawrote: > The command suggested in this comment block is wrong; it would not > rip off CONFIG options that had already been converted to Kconfig. > > Instead, we should use the scripts/build-whitelist.sh tool. > > Signed-off-by: Masahiro Yamada > --- > > scripts/check-config.sh | 9 ++--- > 1 file changed, 2 insertions(+), 7 deletions(-) > > diff --git a/scripts/check-config.sh b/scripts/check-config.sh > index 28c8fe9..6618dfb 100755 > --- a/scripts/check-config.sh > +++ b/scripts/check-config.sh > @@ -5,13 +5,8 @@ > # Check that the u-boot.cfg file provided does not introduce any new > # ad-hoc CONFIG options > # > -# You can generate the list of current ad-hoc CONFIG options (those which are > -# not in Kconfig) with this command: > -# > -# export LC_ALL=C LC_COLLATE=C > -# git grep CONFIG_ |tr ' \t' '\n\n' |sed -n > 's/^\(CONFIG_[A-Z0-9_]*\).*/\1/p' \ > -# |sort |uniq >scripts/config_whitelist.txt; > -# unset LC_ALL LC_COLLATE > +# Use scripts/build-whitelist.sh to generate the list of current ad-hoc > +# CONFIG options (those which are not in Kconfig). For me the LC setup is needed. Does it work correctly without it for you? I found that the sorting was wrong. > > # Usage > #check-config.shdir> > -- > 1.9.1 > Regards, Simon ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH 1/5] check-config: fix wrong comment about how to build whitelist
The command suggested in this comment block is wrong; it would not rip off CONFIG options that had already been converted to Kconfig. Instead, we should use the scripts/build-whitelist.sh tool. Signed-off-by: Masahiro Yamada--- scripts/check-config.sh | 9 ++--- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/scripts/check-config.sh b/scripts/check-config.sh index 28c8fe9..6618dfb 100755 --- a/scripts/check-config.sh +++ b/scripts/check-config.sh @@ -5,13 +5,8 @@ # Check that the u-boot.cfg file provided does not introduce any new # ad-hoc CONFIG options # -# You can generate the list of current ad-hoc CONFIG options (those which are -# not in Kconfig) with this command: -# -# export LC_ALL=C LC_COLLATE=C -# git grep CONFIG_ |tr ' \t' '\n\n' |sed -n 's/^\(CONFIG_[A-Z0-9_]*\).*/\1/p' \ -# |sort |uniq >scripts/config_whitelist.txt; -# unset LC_ALL LC_COLLATE +# Use scripts/build-whitelist.sh to generate the list of current ad-hoc +# CONFIG options (those which are not in Kconfig). # Usage #check-config.sh -- 1.9.1 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot