Skip to contentSkip to conversation

binutils: disable warning flags to prevent build failures

PATCH New Vlad Zakharov 2 participants 3 messages Aug 19, 2016 – Oct 3, 2016

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 binutils: disable warning flags to prevent build failures

binutils-arc-2016.09-eng008 build fails sometimes.

This happens due to the following issue:

binutils configure script has CC variable that stores compiler for target and CC_FOR_BUILD variable that stores compiler for host.

WARN_FLAGS are verified by the script to be compatible only with CC compiler but not with CC_FOR_BUILD compiler. Despite this WARN_FLAGS are passed to both CC and CC_FOR_BUILD compilers. This leads to unevident errors when cross-compiling. More information you can find in the binutils mailing list:
https://sourceware.org/ml/binutils/2016-08/msg00117.html

We are going to write a patch for bunutils that fixes the initial issue. Current patch contains temporary workaround. It should be deleted after real fix.

Fixes:
http://autobuild.buildroot.net/results/004/00455b6a8324920ad843d90ce0413451522691a8//

---
 package/binutils/binutils.mk | 1 +
 1 file changed, 1 insertion(+)
package/binutils/binutils.mk +1 -0
index aeabf75..2936eb5 100644
@@ -58,6 +58,7 @@ BINUTILS_CONF_OPTS = \
 	--host=$(GNU_TARGET_NAME) \
 	--target=$(GNU_TARGET_NAME) \
 	--enable-install-libiberty \
+	--enable-build-warnings=no \
 	$(BINUTILS_DISABLE_GDB_CONF_OPTS) \
 	$(BINUTILS_EXTRA_CONFIG_OPTIONS)
 
Signature · 1 line
2.6.3
#1 lore raw

PATCH 1/1 binutils: disable warning flags to prevent build failures

Hello,

Quoted 18 lines · Vlad Zakharov

binutils-arc-2016.09-eng008 build fails sometimes.

This happens due to the following issue:

binutils configure script has CC variable that stores compiler for target and CC_FOR_BUILD variable that stores compiler for host.

WARN_FLAGS are verified by the script to be compatible only with CC compiler but not with CC_FOR_BUILD compiler. Despite this WARN_FLAGS are passed to both CC and CC_FOR_BUILD compilers. This leads to unevident errors when cross-compiling. More information you can find in the binutils mailing list:
https://sourceware.org/ml/binutils/2016-08/msg00117.html

We are going to write a patch for bunutils that fixes the initial issue. Current patch contains temporary workaround. It should be deleted after real fix.

Quoted 7 lines

Fixes:
http://autobuild.buildroot.net/results/004/00455b6a8324920ad843d90ce0413451522691a8//

---
 package/binutils/binutils.mk | 1 +
 1 file changed, 1 insertion(+)

Applied to master, thanks.

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
#2 lore raw

PATCH 1/1 binutils: disable warning flags to prevent build failures

Hi all,

I submitted a patch to binutils that fixes the issue:
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;a=commit;h=c5da193232ea0452f714dffee0d417d965590476

So the workaround, proposed in the patch above, should be reverted after update to the upcoming binutils version.

Thanks.

Quoted 33 lines · Thomas Petazzoni

Hello,

Quoted 19 lines · Vlad Zakharov

binutils-arc-2016.09-eng008 build fails sometimes.

This happens due to the following issue:

binutils configure script has CC variable that stores compiler for target and CC_FOR_BUILD variable that stores compiler for host.

WARN_FLAGS are verified by the script to be compatible only with CC compiler but not with CC_FOR_BUILD compiler. Despite this WARN_FLAGS are passed to both CC and CC_FOR_BUILD compilers. This leads to unevident errors when cross-compiling. More information you can find in the binutils mailing list:
https://sourceware.org/ml/binutils/2016-08/msg00117.html

We are going to write a patch for bunutils that fixes the initial issue. Current patch contains temporary workaround. It should be deleted after real fix.

Quoted 7 lines

Fixes:
http://autobuild.buildroot.net/results/004/00455b6a8324920ad843d90ce0413451522691a8//

---
?package/binutils/binutils.mk | 1 +
?1 file changed, 1 insertion(+)

Applied to master, thanks.

Thomas

Signature · 2 lines
Best regards,
Vlad Zakharov <[email protected]>