Skip to contentSkip to conversation

dhcpcd: unavailable for bfin and fix another bug

PATCH New Gustavo Zacarias 2 participants 7 messages Jun 28, 2013 – Jun 29, 2013

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 dhcpcd: unavailable for bfin and fix another bug

dhcpcd needs a toolchain with getifaddrs support which the current blackfin toolchain lacks. Fixes:
http://autobuild.buildroot.net/results/f4dd9ca2774bc89e6f976d75a13190d8a1e457f3/

This is not detected by dhcpcd's configure because it's being called with default (non-cross) CC and other variables.
Thus pass TARGET_CONFIGURE_OPTS to configure to fix this. Fixes another build failure i've encountered where the host has very modern headers and dhcpcd tries to use the latest features which the cross toolchain lacks.

---
 package/dhcpcd/Config.in | 1 +
 package/dhcpcd/dhcpcd.mk | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)
package/dhcpcd/Config.in +1 -0
index a06a973..a032694 100644
@@ -1,5 +1,6 @@
 config BR2_PACKAGE_DHCPCD
 	depends on BR2_INET_IPV6
+	depends on !BR2_bfin
 	bool "dhcpcd"
 	help
 	  An RFC2131 compliant DHCP client
package/dhcpcd/dhcpcd.mk +1 -1
index 9d4a72b..28da6cd 100644
@@ -15,7 +15,7 @@ endif
 
 define DHCPCD_CONFIGURE_CMDS
 	(cd $(@D); \
-	./configure \
+	$(TARGET_CONFIGURE_OPTS) ./configure \
 		--target=$(BR2_GCC_TARGET_ARCH) \
 		--os=linux \
 		$(DHCPCD_CONFIG_OPT) )
Signature · 1 line
1.8.1.5
#1 lore raw

PATCH 1/1 dhcpcd: unavailable for bfin and fix another bug

Dear Gustavo Zacarias,

package/dhcpcd/Config.in line 3

Quoted 8 lines · Gustavo Zacarias
package/dhcpcd/Config.in +1 -0
index a06a973..a032694 100644
@@ -1,5 +1,6 @@
 config BR2_PACKAGE_DHCPCD
 	depends on BR2_INET_IPV6
+	depends on !BR2_bfin

I would prefer to see an explicit "depends on !" on the specific Blackfin external toolchains that are causing problems. If one day we get the internal backend to properly build a toolchain for Blackfin, then doing a "depends on !BR2_bfin" wouldn't make sense.

The Blackin external toolchain versions aren't changing that often, so it shouldn't cause too much hassle to update dhpcd dependencies when that happens.

Best regards,

Thomas

Signature · 4 lines
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
#2 lore raw

PATCH 1/1 dhcpcd: unavailable for bfin and fix another bug

Quoted 8 lines · Thomas Petazzoni

I would prefer to see an explicit "depends on !" on the specific Blackfin external toolchains that are causing problems. If one day we get the internal backend to properly build a toolchain for Blackfin, then doing a "depends on !BR2_bfin" wouldn't make sense.

The Blackin external toolchain versions aren't changing that often, so it shouldn't cause too much hassle to update dhpcd dependencies when that happens.

That would be all good and well if the internal toolchain worked, snip:

checking for F_SETLKW... yes
checking if mkdir takes one argument... no
This target does not support --with-cpu.
Valid --with options are:
make[1]: *** [configure-gcc] Error 1
make[1]: Leaving directory `/home/gustavoz/b/fin/output/toolchain/gcc-4.7.3-initial'
make: *** [/home/gustavoz/b/fin/output/toolchain/gcc-4.7.3-initial/.compiled] Error 2

(all defaults except +IPV6 for dhcpcd).
And 2012R1-RC2 fails to download properly (2012R2-RC2 too for that matter but i've got a local copy).
I kind of doubt that 2012R1 being older has getifaddrs enabled though. Regards.

#3 lore raw

PATCH 1/1 dhcpcd: unavailable for bfin and fix another bug

Dear Gustavo Zacarias,

Quoted 11 lines · Gustavo Zacarias
Quoted 8 lines

I would prefer to see an explicit "depends on !" on the specific Blackfin external toolchains that are causing problems. If one day we get the internal backend to properly build a toolchain for Blackfin, then doing a "depends on !BR2_bfin" wouldn't make sense.

The Blackin external toolchain versions aren't changing that often, so it shouldn't cause too much hassle to update dhpcd dependencies when that happens.

That would be all good and well if the internal toolchain worked, snip:

Quoted 9 lines

checking for F_SETLKW... yes
checking if mkdir takes one argument... no
This target does not support --with-cpu.
Valid --with options are:
make[1]: *** [configure-gcc] Error 1
make[1]: Leaving directory `/home/gustavoz/b/fin/output/toolchain/gcc-4.7.3-initial'
make: *** [/home/gustavoz/b/fin/output/toolchain/gcc-4.7.3-initial/.compiled] Error 2

The fact that the internal toolchain doesn't work today for Blackfin is a separate issue :)

Quoted 4 lines

(all defaults except +IPV6 for dhcpcd).
And 2012R1-RC2 fails to download properly (2012R2-RC2 too for that matter but i've got a local copy).
I kind of doubt that 2012R1 being older has getifaddrs enabled though.

Yes, one recent patch to the download infrastructure broke this. I should sent a different e-mail with this.

Best regards,

Thomas

Signature · 4 lines
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
#4 lore raw

PATCH 1/1 dhcpcd: unavailable for bfin and fix another bug

Quoted 7 lines · Thomas Petazzoni
Quoted 4 lines

(all defaults except +IPV6 for dhcpcd).
And 2012R1-RC2 fails to download properly (2012R2-RC2 too for that matter but i've got a local copy).
I kind of doubt that 2012R1 being older has getifaddrs enabled though.

Yes, one recent patch to the download infrastructure broke this. I should sent a different e-mail with this.

Downloaded manually and as i supposed, it lacks getifaddrs too. Regards.

#5 lore raw

PATCH 1/1 dhcpcd: unavailable for bfin and fix another bug

Dear Gustavo Zacarias,

Quoted 11 lines · Gustavo Zacarias
Quoted 7 lines · Thomas Petazzoni
Quoted 4 lines

(all defaults except +IPV6 for dhcpcd).
And 2012R1-RC2 fails to download properly (2012R2-RC2 too for that matter but i've got a local copy).
I kind of doubt that 2012R1 being older has getifaddrs enabled though.

Yes, one recent patch to the download infrastructure broke this. I should sent a different e-mail with this.

Downloaded manually and as i supposed, it lacks getifaddrs too.

Yes, of course it lacks getifaddrs, but I don't see how it changes the point I was making originally: it is not because /some/ external toolchains are affected that we should exclude an entire architecture from building a particular package.

Best regards,

Thomas

Signature · 4 lines
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
#6 lore raw

PATCH 1/1 dhcpcd: unavailable for bfin and fix another bug

Quoted 4 lines · Thomas Petazzoni

Yes, of course it lacks getifaddrs, but I don't see how it changes the point I was making originally: it is not because /some/ external toolchains are affected that we should exclude an entire architecture from building a particular package.

Oh come on!
The ct-ng sample doesn't have UCLIBC_SUPPORT_AI_ADDRCONFIG set (default=n in uClibc upstream).
None of Analog's toolchains have it either.
There's some old Ronetix blackfin toolchain that doesn't support it either. So that just leaves a customized ct-ng or hand-built toolchain as a canditate, which i take it as "very unlikely" to happen. Regards.