All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] ncftp: fix host/target confusion
@ 2017-02-28 22:54 Thomas Petazzoni
  2017-03-01 20:13 ` Yann E. MORIN
  2017-03-01 20:48 ` Thomas Petazzoni
  0 siblings, 2 replies; 14+ messages in thread
From: Thomas Petazzoni @ 2017-02-28 22:54 UTC (permalink / raw)
  To: buildroot

The ncftp build process tries to build and run a small program called
ccdv to beautify the build process output. If it manages to build and
run it, then it uses it.

Unfortunately, this doesn't work well when the target architecture is
close to the host architecture, but not exactly the same. Because both
architectures are close to each other, the test run of ccdv succeeds,
but real use of ccdv during ncftp build process causes an Illegal
instruction issue.

This for example happens with the CodeSourcery AMD64 toolchain, on a
build machine running an i7-4600U, and has been detected in the
autobuilders since the CodeSourcery AMD64 toolchain was upgraded at
the end of January:

  http://autobuild.buildroot.net/?reason=ncftp-3.2.6

The issue was also reported by Christopher Arguin back in July 2016:

  http://lists.busybox.net/pipermail/buildroot/2016-July/168026.html

and at the time, we identified that simply disabling the ccdv tool, by
passing --disable-ccdv, was enough to solve the issue. But Christopher
never submitted the patch, so the problem remained unfixed.

Therefore, we pass --disable-ccdv to the configure script, which
fixes:

  http://autobuild.buildroot.net/results/6eadad0e879ca70bb07b13b4196d42c64b11699f/

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 package/ncftp/ncftp.mk | 1 +
 1 file changed, 1 insertion(+)

diff --git a/package/ncftp/ncftp.mk b/package/ncftp/ncftp.mk
index 5c88556..11bfcca 100644
--- a/package/ncftp/ncftp.mk
+++ b/package/ncftp/ncftp.mk
@@ -12,6 +12,7 @@ NCFTP_LICENSE = Clarified Artistic License
 NCFTP_LICENSE_FILES = doc/LICENSE.txt
 
 NCFTP_DEPENDENCIES = host-autoconf
+NCFTP_CONF_OPTS = --disable-ccdv
 
 # The bundled configure script is generated by autoconf 2.13 and doesn't
 # detect cross-compilation correctly. Therefore, we have to regenerate it.
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 14+ messages in thread

end of thread, other threads:[~2017-04-26  6:48 UTC | newest]

Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-02-28 22:54 [Buildroot] [PATCH] ncftp: fix host/target confusion Thomas Petazzoni
2017-03-01 20:13 ` Yann E. MORIN
2017-03-01 20:48 ` Thomas Petazzoni
2017-03-02 16:28   ` Peter Korsgaard
2017-03-02 19:07     ` Danomi Manchego
2017-03-02 20:43       ` Thomas Petazzoni
2017-03-03 20:50         ` Peter Korsgaard
2017-03-03 22:04           ` Arnout Vandecappelle
2017-03-03 22:08             ` Peter Korsgaard
2017-03-07 16:48           ` Peter Korsgaard
2017-03-07 20:10             ` Thomas Petazzoni
2017-03-07 21:10               ` Peter Korsgaard
2017-04-14 22:35                 ` Arnout Vandecappelle
2017-04-26  6:48                   ` Peter Korsgaard

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.