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: [email protected]
> Closes: https://bugs.gentoo.org/705248
> Signed-off-by: Sergei Trofimovich <[email protected]>
> ---
> 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
