Patchwork wpa_supplicant: compilation error

login
register
mail settings
Submitter Narayanraddi
Date Feb. 10, 2014, 11:56 a.m.
Message ID <1392033376-7335-1-git-send-email-narayanraddi.m@globaledgesoft.com>
Download mbox | patch
Permalink /patch/318767/
State New
Headers show

Comments

Narayanraddi - Feb. 10, 2014, 11:56 a.m.
Fix compilation error in wpa_supplicant

Signed-hostap: Narayanraddi <narayanraddi.m@globaledgesoft.com>
---
 wpa_supplicant/Makefile |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Jouni Malinen - Feb. 13, 2014, 1:36 p.m.
On Mon, Feb 10, 2014 at 05:26:16PM +0530, Narayanraddi wrote:
> Fix compilation error in wpa_supplicant

> diff --git a/wpa_supplicant/Makefile b/wpa_supplicant/Makefile
> @@ -132,7 +132,7 @@ endif
>  ifdef CONFIG_EAPOL_TEST
> -CFLAGS += -Werror -DEAPOL_TEST
> +CFLAGS += -DEAPOL_TEST
>  endif

This does not look like a compilation error fix.. It looks more like
hiding a compilation warning. wpa_supplicant is expected to compile
without showing compilation warnings. What is the warning that this is
trying to work around?

Patch

diff --git a/wpa_supplicant/Makefile b/wpa_supplicant/Makefile
index d1e11a3..799a1c2 100644
--- a/wpa_supplicant/Makefile
+++ b/wpa_supplicant/Makefile
@@ -132,7 +132,7 @@  endif
 
 
 ifdef CONFIG_EAPOL_TEST
-CFLAGS += -Werror -DEAPOL_TEST
+CFLAGS += -DEAPOL_TEST
 endif
 
 ifdef CONFIG_CODE_COVERAGE