On Tue, 2016-05-24 at 19:38 +0300, Kirill Kapranov wrote:
> Hello.
> 
> In case of compilation the igb driver on non-RH systems, messages
> like this:
> [: -ge: unary operator expected
> appear. The following patch is intended to suppress them.
> Tested on igb-5.3.4.4
> Signed-off-by: Kirill Kapranov <kapran...@inbox.ru>
> ---

An alternative would be to add quotes around the variables in the shell
code.

Thanks,
Jake

> diff -ur org//igb-5.3.4.4/src/Makefile my//igb-5.3.4.4/src/Makefile
> --- org//igb-5.3.4.4/src/Makefile 2016-02-26 03:48:39.000000000 +0300
> +++ my//igb-5.3.4.4/src/Makefile 2016-05-24 18:19:36.000000000 +0300
> @@ -205,6 +205,10 @@
> RHEL_CODE := $(shell $(CC) $(EXTRA_CFLAGS) -E -dM $(VSP) 2>/dev/null
> |\
> grep -m 1 RHEL_RELEASE_CODE | awk '{ print $$3 }')
> +ifeq (${RHEL_CODE},)
> +# In order to avoid error message during evaluation the expression [
> $(RHEL_CODE) -ge 1540 ] in the line 229
> + RHEL_CODE = 0
> +endif
> # abort the build on kernels older than 2.4.21
> ifneq (1,$(shell [ $(KVER_CODE) -ge 132117 ] && echo 1 || echo 0))
> -------------------------------------------------------------------
> -----------
> Mobile security can be enabling, not merely restricting. Employees
> who
> bring their own devices (BYOD) to work are irked by the imposition of
> MDM
> restrictions. Mobile Device Manager Plus allows you to control only
> the
> apps on BYO-devices by containerizing them, leaving personal data
> untouched!
> https://ad.doubleclick.net/ddm/clk/304595813;131938128;j
> _______________________________________________
> E1000-devel mailing list
> E1000-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/e1000-devel
> To learn more about Intel&#174; Ethernet, visit http://communities.in
> tel.com/community/wired
------------------------------------------------------------------------------
Mobile security can be enabling, not merely restricting. Employees who
bring their own devices (BYOD) to work are irked by the imposition of MDM
restrictions. Mobile Device Manager Plus allows you to control only the
apps on BYO-devices by containerizing them, leaving personal data untouched!
https://ad.doubleclick.net/ddm/clk/304595813;131938128;j
_______________________________________________
E1000-devel mailing list
E1000-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/e1000-devel
To learn more about Intel&#174; Ethernet, visit 
http://communities.intel.com/community/wired

Reply via email to