libdrm patch merge request
michel at daenzer.net
Fri Aug 9 14:13:11 UTC 2019
This broke the CI pipeline:
Looks like the problem is that the autotools build doesn't properly
disable the amdgpu tests when the json-c library is missing. I suggest
1. Add a HAVE_JSONC guard in tests/Makefile.am
2. Add libjson-c-dev to the packages installed by the oldest-autotools
job in .gitlab-ci.yml
Until libdrm uses GitLab merge requests to catch this kind of issue
before it hits master, please push changes to a branch in a forked
personal repository and make sure the CI pipeline comes back green
before asking for them to be pushed to master.
On 2019-08-09 3:25 a.m., Chen, Guchun wrote:
> Thanks, Alex.
> -----Original Message-----
> From: Alex Deucher <alexdeucher at gmail.com>
> Sent: Friday, August 9, 2019 1:24 AM
> To: Chen, Guchun <Guchun.Chen at amd.com>
> Cc: Deucher, Alexander <Alexander.Deucher at amd.com>; amd-gfx at lists.freedesktop.org; Zhou1, Tao <Tao.Zhou1 at amd.com>; Li, Candice <Candice.Li at amd.com>; Li, Dennis <Dennis.Li at amd.com>; Zhang, Hawking <Hawking.Zhang at amd.com>
> Subject: Re: libdrm patch merge request
> On Thu, Aug 8, 2019 at 5:18 AM Chen, Guchun <Guchun.Chen at amd.com> wrote:
>> Hi Alex,
>> Would you mind merging attached 3 patches to libdrm master branch?
>> These changes are implemented for gfx and umc ras inject unit test by amdgpu_test.
>> Thanks a lot.
>> amd-gfx mailing list
>> amd-gfx at lists.freedesktop.org
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
Earthling Michel Dänzer | https://www.amd.com
Libre software enthusiast | Mesa and X developer
More information about the amd-gfx