[PATCH v1] uefi-sct/SctPkg: Changed compiler check from greater 5 to equal to 5

[PATCH v1] uefi-sct/SctPkg: Changed compiler check from greater 5 to equal to 5

[Joseph Hemann]

From: Joseph Hemann <Joseph.heman@arm.com>

gcc compilers versions 5.* also need to be converted from string to integer before testing

Cc: G Edhaya Chandran <Edhaya.Chandran@arm.com>
Cc: Eric Jin <eric.jin@intel.com>
Cc: Irene Park <ipark@nvidia.com>
Cc: Heinrich Schuchardt <xypron.glpk@gmx.de>
Cc: Samer El-Haj-Mahmoud <samer.el-haj-mahmoud@arm.com>
Cc: Stuart Yoder <Stuart.Yoder@arm.com>
Signed-off-by: Joseph Hemann <Joseph.heman@arm.com>
Change-Id: Ica60b6629f698d5f5c61b861ac4aefcc9cd24b7a
---
 uefi-sct/SctPkg/build.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/uefi-sct/SctPkg/build.sh b/uefi-sct/SctPkg/build.sh
index 37667711..ba9babe7 100755
--- a/uefi-sct/SctPkg/build.sh
+++ b/uefi-sct/SctPkg/build.sh
@@ -56,7 +56,7 @@ function get_gcc_version
 {
 	gcc_version=$($1 -dumpversion)
 
-	if [ ${gcc_version%%.*} -gt 5 ]; then
+	if [ ${gcc_version%%.*} -ge 5 ]; then
 		gcc_version=5
 	fi
 
-- 
2.17.1

Re: [PATCH v1] uefi-sct/SctPkg: Changed compiler check from greater 5 to equal to 5

[Heinrich Schuchardt]

On 14.12.20 15:35, Joseph Hemann wrote:
> From: Joseph Hemann <Joseph.heman@arm.com>
>
> gcc compilers versions 5.* also need to be converted from string to integer before testing
>
> Cc: G Edhaya Chandran <Edhaya.Chandran@arm.com>
> Cc: Eric Jin <eric.jin@intel.com>
> Cc: Irene Park <ipark@nvidia.com>
> Cc: Heinrich Schuchardt <xypron.glpk@gmx.de>
> Cc: Samer El-Haj-Mahmoud <samer.el-haj-mahmoud@arm.com>
> Cc: Stuart Yoder <Stuart.Yoder@arm.com>
> Signed-off-by: Joseph Hemann <Joseph.heman@arm.com>
> Change-Id: Ica60b6629f698d5f5c61b861ac4aefcc9cd24b7a

Reviewed-by: Heinrich Schuchardt <xypron.glpk@gmx.de>

> ---
>  uefi-sct/SctPkg/build.sh | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/uefi-sct/SctPkg/build.sh b/uefi-sct/SctPkg/build.sh
> index 37667711..ba9babe7 100755
> --- a/uefi-sct/SctPkg/build.sh
> +++ b/uefi-sct/SctPkg/build.sh
> @@ -56,7 +56,7 @@ function get_gcc_version
>  {
>  	gcc_version=$($1 -dumpversion)
>
> -	if [ ${gcc_version%%.*} -gt 5 ]; then
> +	if [ ${gcc_version%%.*} -ge 5 ]; then
>  		gcc_version=5
>  	fi
>
>

Re: [PATCH v1] uefi-sct/SctPkg: Changed compiler check from greater 5 to equal to 5

[G Edhaya Chandran]

Reviewed-By: G Edhaya Chandran <edhaya.chandran@arm.com>

> -----Original Message-----
> From: Heinrich Schuchardt <xypron.glpk@gmx.de>
> Sent: 14 December 2020 20:30
> To: Joseph Hemann <Joseph.Hemann@arm.com>; devel@edk2.groups.io
> Cc: Joseph Hemann <Joseph.heman@arm.com>; G Edhaya Chandran
> <Edhaya.Chandran@arm.com>; Eric Jin <eric.jin@intel.com>; Irene Park
> <ipark@nvidia.com>; Samer El-Haj-Mahmoud <Samer.El-Haj-
> Mahmoud@arm.com>; Stuart Yoder <Stuart.Yoder@arm.com>
> Subject: Re: [PATCH v1] uefi-sct/SctPkg: Changed compiler check from greater
> 5 to equal to 5
>
> On 14.12.20 15:35, Joseph Hemann wrote:
> > From: Joseph Hemann <Joseph.heman@arm.com>
> >
> > gcc compilers versions 5.* also need to be converted from string to
> > integer before testing
> >
> > Cc: G Edhaya Chandran <Edhaya.Chandran@arm.com>
> > Cc: Eric Jin <eric.jin@intel.com>
> > Cc: Irene Park <ipark@nvidia.com>
> > Cc: Heinrich Schuchardt <xypron.glpk@gmx.de>
> > Cc: Samer El-Haj-Mahmoud <samer.el-haj-mahmoud@arm.com>
> > Cc: Stuart Yoder <Stuart.Yoder@arm.com>
> > Signed-off-by: Joseph Hemann <Joseph.heman@arm.com>
> > Change-Id: Ica60b6629f698d5f5c61b861ac4aefcc9cd24b7a
>
> Reviewed-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
>
> > ---
> >  uefi-sct/SctPkg/build.sh | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/uefi-sct/SctPkg/build.sh b/uefi-sct/SctPkg/build.sh index
> > 37667711..ba9babe7 100755
> > --- a/uefi-sct/SctPkg/build.sh
> > +++ b/uefi-sct/SctPkg/build.sh
> > @@ -56,7 +56,7 @@ function get_gcc_version  {
> >  gcc_version=$($1 -dumpversion)
> >
> > -if [ ${gcc_version%%.*} -gt 5 ]; then
> > +if [ ${gcc_version%%.*} -ge 5 ]; then
> >  gcc_version=5
> >  fi
> >
> >

IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.