Re: [gentoo-dev] [PATCH] linux-info.eclass: avoid lexicographical compare on numbers, bug #705248
On Sat, 23 May 2020 14:56:06 -0400 Mike wrote: > On 5/22/20 2:57 PM, Sergei Trofimovich wrote: > > Originally found in bug #705240 as: > > > > ``` > > error=0 > > ... > > if [[ ${error} > 0 ]]; then > > ... > > ``` > > > > '>' are string comparisons. They are benign in this case, but let's > > be consistent and use integer comparison. > > > > CC: ker...@gentoo.org > > Closes: https://bugs.gentoo.org/705248 > > Signed-off-by: Sergei Trofimovich > > --- > > eclass/linux-info.eclass | 6 +++--- > > 1 file changed, 3 insertions(+), 3 deletions(-) > > > > diff --git a/eclass/linux-info.eclass b/eclass/linux-info.eclass > > index 44eebcf52a9..405ef5571e1 100644 > > --- a/eclass/linux-info.eclass > > +++ b/eclass/linux-info.eclass > > @@ -813,7 +813,7 @@ check_extra_config() { > > linux_chkconfig_present ${config} || error=1 > > fi > > > > - if [[ ${error} > 0 ]]; then > > + if [[ ${error} -gt 0 ]]; then > > local report_func="eerror" local_error > > local_error="ERROR_${config}" > > local_error="${!local_error}" > > @@ -848,14 +848,14 @@ check_extra_config() { > > fi > > done > > > > - if [[ ${hard_errors_count} > 0 ]]; then > > + if [[ ${hard_errors_count} -gt 0 ]]; then > > eerror "Please check to make sure these options are set > > correctly." > > eerror "Failure to do so may cause unexpected problems." > > eerror "Once you have satisfied these options, please try > > merging" > > eerror "this package again." > > export > > LINUX_CONFIG_EXISTS_DONE="${old_LINUX_CONFIG_EXISTS_DONE}" > > die "Incorrect kernel configuration options" > > - elif [[ ${soft_errors_count} > 0 ]]; then > > + elif [[ ${soft_errors_count} -gt 0 ]]; then > > ewarn "Please check to make sure these options are set > > correctly." > > ewarn "Failure to do so may cause unexpected problems." > > else > > > > Thanks. LGTM Pushed as: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=aa1d259decdfd73a49e0c49e88a8ec5675453266 -- Sergei
Re: [gentoo-dev] [PATCH] linux-info.eclass: avoid lexicographical compare on numbers, bug #705248
On 5/22/20 2:57 PM, Sergei Trofimovich wrote: > Originally found in bug #705240 as: > > ``` > error=0 > ... > if [[ ${error} > 0 ]]; then > ... > ``` > > '>' are string comparisons. They are benign in this case, but let's > be consistent and use integer comparison. > > CC: ker...@gentoo.org > Closes: https://bugs.gentoo.org/705248 > Signed-off-by: Sergei Trofimovich > --- > eclass/linux-info.eclass | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/eclass/linux-info.eclass b/eclass/linux-info.eclass > index 44eebcf52a9..405ef5571e1 100644 > --- a/eclass/linux-info.eclass > +++ b/eclass/linux-info.eclass > @@ -813,7 +813,7 @@ check_extra_config() { > linux_chkconfig_present ${config} || error=1 > fi > > - if [[ ${error} > 0 ]]; then > + if [[ ${error} -gt 0 ]]; then > local report_func="eerror" local_error > local_error="ERROR_${config}" > local_error="${!local_error}" > @@ -848,14 +848,14 @@ check_extra_config() { > fi > done > > - if [[ ${hard_errors_count} > 0 ]]; then > + if [[ ${hard_errors_count} -gt 0 ]]; then > eerror "Please check to make sure these options are set > correctly." > eerror "Failure to do so may cause unexpected problems." > eerror "Once you have satisfied these options, please try > merging" > eerror "this package again." > export > LINUX_CONFIG_EXISTS_DONE="${old_LINUX_CONFIG_EXISTS_DONE}" > die "Incorrect kernel configuration options" > - elif [[ ${soft_errors_count} > 0 ]]; then > + elif [[ ${soft_errors_count} -gt 0 ]]; then > ewarn "Please check to make sure these options are set > correctly." > ewarn "Failure to do so may cause unexpected problems." > else > Thanks. LGTM signature.asc Description: OpenPGP digital signature
[gentoo-dev] [PATCH] linux-info.eclass: avoid lexicographical compare on numbers, bug #705248
Originally found in bug #705240 as: ``` error=0 ... if [[ ${error} > 0 ]]; then ... ``` '>' are string comparisons. They are benign in this case, but let's be consistent and use integer comparison. CC: ker...@gentoo.org Closes: https://bugs.gentoo.org/705248 Signed-off-by: Sergei Trofimovich --- eclass/linux-info.eclass | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/eclass/linux-info.eclass b/eclass/linux-info.eclass index 44eebcf52a9..405ef5571e1 100644 --- a/eclass/linux-info.eclass +++ b/eclass/linux-info.eclass @@ -813,7 +813,7 @@ check_extra_config() { linux_chkconfig_present ${config} || error=1 fi - if [[ ${error} > 0 ]]; then + if [[ ${error} -gt 0 ]]; then local report_func="eerror" local_error local_error="ERROR_${config}" local_error="${!local_error}" @@ -848,14 +848,14 @@ check_extra_config() { fi done - if [[ ${hard_errors_count} > 0 ]]; then + if [[ ${hard_errors_count} -gt 0 ]]; then eerror "Please check to make sure these options are set correctly." eerror "Failure to do so may cause unexpected problems." eerror "Once you have satisfied these options, please try merging" eerror "this package again." export LINUX_CONFIG_EXISTS_DONE="${old_LINUX_CONFIG_EXISTS_DONE}" die "Incorrect kernel configuration options" - elif [[ ${soft_errors_count} > 0 ]]; then + elif [[ ${soft_errors_count} -gt 0 ]]; then ewarn "Please check to make sure these options are set correctly." ewarn "Failure to do so may cause unexpected problems." else -- 2.26.2