Re: [Qemu-devel] [PATCH] .travis.yml: basic compile and check recipes

["Alex Bennée" <alex.bennee@linaro.org>]

peter.maydell@linaro.org writes:

> On 25 September 2013 01:31,  <alex.bennee@linaro.org> wrote:
>> +    # Debug related options
>> +    - env: TARGETS=i386-softmmu,x86_64-softmmu
>> +           EXTRA_CONFIG="--enable-debug --enable-debug-tcg --enable-tcg-interpreter"
>
> --enable-debug implies --enable-debug-tcg, you don't need to specify
> it separately.
>
> I think we should have an --enable-debug config tested which is actually
> testing TCG, not the TCG interpreter (which is a very rarely used and
> not IMHO terribly useful config).

OK I'll split those two up.

>> +    # This disabled make check for the ftrace backend which needs more setting up
>> +    # Currently broken on 12.04 due to mis-packaged liburcu and changed API, will be pulled.
>> +    #- env: TARGETS=i386-softmmu,x86_64-softmmu
>> +    #       EXTRA_PKGS="liblttng-ust-dev liburcu-dev"
>> +    #       EXTRA_CONFIG="--enable-trace-backend=ust"
>
> Does our configure identify the busted library and refuse to configure with
> this config? It probably ought to.

It's a mess. It probably still works on some set-ups but in discussion
with Stefan on IRC it looks like it's regressed on most modern set-ups.
The fundamental issue is lttng's lack of stable API. I hunted around a
bit trying to get it working but realised the script needs fixing up as
well so gave up.

Really ust just needs to be ripped out for now unless someone else wants
to dig into to supporting multiple versions painlessly.

-- 
Alex Bennée