[Buildroot] Analysis of build failures

[Thomas Petazzoni <thomas.petazzoni@free-electrons.com>]

Dear Thomas De Schampheleire,

On Wed, 28 Aug 2013 13:31:38 +0200, Thomas De Schampheleire wrote:

> > We don't have that many build failures today, so let's have a look
> > together at each of them and see what we can do about them.
> 
> Good idea! Ideally we can get them to 0, so that from that point
> onwards any autobuild failure becomes a clear focus point.

Agreed. Though we have to keep in mind that the release is in 3 days,
and after that 'next' will be merged in master, and we'll have plenty
of new autobuilder failures to look at.

> >>    powerpc | media-ctl-ac40b79f002a2315f... | NOK | http://autobuild.buildroot.net/results/7bcab9b94a81b89789dc1cabe9f6940ed4f5ea51/
> >
> > Too old kernel headers in toolchain. Options:
> >
> >  1/ Add an exception in the autobuilder script.
> >
> >  2/ Modify media-ctl to integrate the necessary kernel headers and use
> >     them when not available from the toolchain.
> >
> > I'm inclined to favor 1/.
> 
> I once took a look at this. The media-ctl package already has some
> copies of kernel headers to deal with older kernels. There was a check
> in configure to detect this, IIRC. The stupid thing was that the
> headers in the media-ctl sources would be applied unconditionally.
> My intent was to patch media-ctl to provide also this missing header,
> and cleanup the stupidity described. Unfortunately I didn't continue
> this yet.
> 
> What exactly do you mean with approach 1/? Forcing a recent kernel
> when building media-ctl?
> That doesn't solve the problem for other users that try to enable
> media-ctl on older kernels, right?

By 1/ I meant to not fix the problem, and only avoid it in the
autobuilders by making sure that when a non-suitable toolchain (too old
kernel headers) is used to build media-ctl, then we exclude media-ctl
from the build. I already have quite a few of such exclusions. What I
find not very nice is that the set of exclusions is not documented
anywhere, while they should be documented as "Known issues", or
something like that.

For reference, here is the current set of tweaks that I apply to the
randpackageconfig before starting the build. Note that when "return
False" is used, it means that the configuration is rejected and not
built, so a completely different random configuration will be tried.

    # Make sure Qt license is approved
    if "BR2_PACKAGE_QT=y\n" in config_items:
        if "# BR2_PACKAGE_QT_LICENSE_APPROVED is not set\n" in config_items:
            config_items.remove("# BR2_PACKAGE_QT_LICENSE_APPROVED is not set\n")
            config_items.append("BR2_PACKAGE_QT_LICENSE_APPROVED=y\n")
    if "BR2_PACKAGE_QT5BASE=y\n" in config_items:
        if "# BR2_PACKAGE_QT5BASE_LICENSE_APPROVED is not set\n" in config_items:
            config_items.remove("# BR2_PACKAGE_QT5BASE_LICENSE_APPROVED is not set\n")
            config_items.append("BR2_PACKAGE_QT5BASE_LICENSE_APPROVED=y\n")
    # Make sure LTP is not enabled when we have an uClibc toolchain
    if "BR2_PACKAGE_LTP_TESTSUITE=y\n" in config_items and \
            toolchain_is_uclibc(config_items):
        config_items.remove("BR2_PACKAGE_LTP_TESTSUITE=y\n")
    # Make sure xfsprogs is not enabled when we have an uClibc toolchain
    if "BR2_PACKAGE_XFSPROGS=y\n" in config_items and \
            toolchain_is_uclibc(config_items):
        config_items.remove("BR2_PACKAGE_XFSPROGS=y\n")
    # Make sure mrouted is not enabled when we have an uClibc toolchain
    if "BR2_PACKAGE_MROUTED=y\n" in config_items and \
            toolchain_is_uclibc(config_items):
        config_items.remove("BR2_PACKAGE_MROUTED=y\n")
    if 'BR2_TOOLCHAIN_EXTERNAL_URL="http://autobuild.buildroot.org/toolchains/tarballs/ctng-mips64-eglibc.tar.bz2"\n' in config_items and \
            "BR2_PACKAGE_SQUID=y\n" in config_items:
        # Reject this configuration
        return False
    if 'BR2_TOOLCHAIN_EXTERNAL_URL="http://autobuild.buildroot.org/toolchains/tarballs/br-mipsel-o32-full.tar.bz2"\n' in config_items and \
            "BR2_PACKAGE_SQUID=y\n" in config_items:
        # Reject this configuration
        return False
    if 'BR2_TOOLCHAIN_EXTERNAL_CODESOURCERY_MIPS44=y\n' in config_items and \
            "BR2_PACKAGE_SQUID=y\n" in config_items:
        # Reject this configuration
        return False
    if 'BR2_PACKAGE_CLASSPATH=y\n' in config_items:
        # Reject this configuration
        return False
    if 'BR2_sh2a=y\n' in config_items and  'BR2_PACKAGE_LIBFFI=y\n' in config_items:
        return False
    if 'BR2_arc=y\n' in config_items and  'BR2_PACKAGE_LIBFFI=y\n' in config_items:
        return False
    if 'BR2_PACKAGE_SUNXI_BOARDS=y\n' in config_items:
        config_items.remove("# BR2_PACKAGE_SUNXI_BOARDS_FEX_FILE is not set\n")
        config_items.append('BR2_PACKAGE_SUNXI_BOARDS_FEX_FILE="a10/hackberry.fex"\n')

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com