diff mbox series

qt5base: remove use of QT5BASE_EXTRA_CFLAGS

Message ID 1522881016-6423-1-git-send-email-joshua.henderson@microchip.com
State Accepted
Headers show
Series qt5base: remove use of QT5BASE_EXTRA_CFLAGS | expand

Commit Message

Joshua Henderson April 4, 2018, 10:30 p.m. UTC
This variable was added as part of an EGL i.MX6 workaround in commit
(7e9f459adf), but appears to currently be abandonded and unused.  Remove the use
of the variable.

Signed-off-by: Joshua Henderson <joshua.henderson@microchip.com>
---
 package/qt5/qt5base/qt5base.mk | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Yann E. MORIN April 5, 2018, 6:49 p.m. UTC | #1
Joshua, All,

On 2018-04-04 15:30 -0700, Joshua Henderson spake thusly:
> This variable was added as part of an EGL i.MX6 workaround in commit
> (7e9f459adf), but appears to currently be abandonded and unused.  Remove the use
> of the variable.
> 
> Signed-off-by: Joshua Henderson <joshua.henderson@microchip.com>

Acked-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

Regards,
Yann E. MORIN.

> ---
>  package/qt5/qt5base/qt5base.mk | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/package/qt5/qt5base/qt5base.mk b/package/qt5/qt5base/qt5base.mk
> index a7eb0ec..750830a 100644
> --- a/package/qt5/qt5base/qt5base.mk
> +++ b/package/qt5/qt5base/qt5base.mk
> @@ -296,8 +296,8 @@ define QT5BASE_CONFIGURE_CMDS
>  		-nomake tests \
>  		-device buildroot \
>  		-device-option CROSS_COMPILE="$(TARGET_CROSS)" \
> -		-device-option BR_COMPILER_CFLAGS="$(TARGET_CFLAGS) $(QT5BASE_EXTRA_CFLAGS)" \
> -		-device-option BR_COMPILER_CXXFLAGS="$(TARGET_CXXFLAGS) $(QT5BASE_EXTRA_CFLAGS)" \
> +		-device-option BR_COMPILER_CFLAGS="$(TARGET_CFLAGS)" \
> +		-device-option BR_COMPILER_CXXFLAGS="$(TARGET_CXXFLAGS)" \
>  		$(QT5BASE_CONFIGURE_OPTS) \
>  	)
>  endef
> -- 
> 2.7.4
> 
> _______________________________________________
> buildroot mailing list
> buildroot@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
Thomas Petazzoni April 9, 2018, 9:26 a.m. UTC | #2
Hello,

On Wed, 4 Apr 2018 15:30:16 -0700, Joshua Henderson wrote:
> This variable was added as part of an EGL i.MX6 workaround in commit
> (7e9f459adf), but appears to currently be abandonded and unused.  Remove the use
> of the variable.
> 
> Signed-off-by: Joshua Henderson <joshua.henderson@microchip.com>
> ---
>  package/qt5/qt5base/qt5base.mk | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

Applied to master, thanks.

Thomas
diff mbox series

Patch

diff --git a/package/qt5/qt5base/qt5base.mk b/package/qt5/qt5base/qt5base.mk
index a7eb0ec..750830a 100644
--- a/package/qt5/qt5base/qt5base.mk
+++ b/package/qt5/qt5base/qt5base.mk
@@ -296,8 +296,8 @@  define QT5BASE_CONFIGURE_CMDS
 		-nomake tests \
 		-device buildroot \
 		-device-option CROSS_COMPILE="$(TARGET_CROSS)" \
-		-device-option BR_COMPILER_CFLAGS="$(TARGET_CFLAGS) $(QT5BASE_EXTRA_CFLAGS)" \
-		-device-option BR_COMPILER_CXXFLAGS="$(TARGET_CXXFLAGS) $(QT5BASE_EXTRA_CFLAGS)" \
+		-device-option BR_COMPILER_CFLAGS="$(TARGET_CFLAGS)" \
+		-device-option BR_COMPILER_CXXFLAGS="$(TARGET_CXXFLAGS)" \
 		$(QT5BASE_CONFIGURE_OPTS) \
 	)
 endef