[PATCH 0/5] Implement support for drm properties "GAMMA_LUT" and "CTM"

[Emil Velikov]

On 25 June 2018 at 19:48, Matheus Santana <embs at cin.ufpe.br> wrote:
> Hi Harsha,
>
>> Weston needs to be rebuilt. But, no new library versions dependencies.
>
>
> All right.
>
> On Mon, Jun 25, 2018 at 10:21 AM, Harsha Manjula Mallikarjun (RBEI/ECF3)
> <Harsha.ManjulaMallikarjun at in.bosch.com> wrote:
>>
>> Hi Matheus,
>>
>> >From: Matheus Santana [mailto:embs at cin.ufpe.br]
>> >Sent: Friday, June 22, 2018 8:33 PM
>> >To: Harsha Manjula Mallikarjun (RBEI/ECF3)
>> > <Harsha.ManjulaMallikarjun at in.bosch.com>
>> >Cc: wayland-devel at lists.freedesktop.org
>> >Subject: Re: [PATCH 0/5] Implement support for drm properties "GAMMA_LUT"
>> > and "CTM"
>> >
>> >Hi Harsha,
>> >I am unable to run the tests you introduced. Tests execution gets stuck
>> > after reporting success for surface-global-test.
>>
>> Surface-global-test terminates the compositor. May be this is the reason.
>> I do not see a stuck test rather a terminated compositor.
>>
>> For testing I modified Weston.ini as follows:
>> modules=ivi-controller.so,
>> surface-global-test.so,gamma-test.so,ctm-test.so
>>
>> Are you running tests in an another way?
>
>
> I'm running them by just issuing `make check` after building weston. I
> basically execute three commands (within weston's root dir):
>
>     $ ./autogen.sh --prefix=$WLD
>     $ make
>     $ make check
>
> The output for `autogen.sh` (which is the same regardless patches
> application) is attached.
> All tests in suite run fine (24 passed, 1 skipped) when running these
> commands without the paches.
>
> I'm applying the patches by downloading the mbox and `git am patches.mbox`.
>
> Are you able to run all tests in suite by just `make check` after building
> the project?
>
Matheus your assumption is correct. make check should pass, without
any local changes.

Quick look points the following:
 - the drm (compositor-drm) backend supports the new functionality
 - the default backend (see tests/weston-tests-env) is headless
(compositor-headless)
 - xwayland is used - not sure if the xwayland codebase (which lives
in the Xserver repo) needs updates

HTH
Emil