[PATCH] Revert "wafsamba: Enable warnings for missing field initializer"

Ralph Böhme slow at samba.org
Tue Apr 30 04:21:32 UTC 2019


Am 30.04.2019 um 01:21 schrieb Christof Schmitt <cs at samba.org>:
> Yes. CI is running now at
> https://gitlab.com/samba-team/devel/samba/pipelines/59040252

Thanks!

>> Also, could you please look into adding CentOS 7 as a supported
>> platform via our bootstrap system so we don't regress here in the
>> future?  There is partial support already, but it wasn't finished
>> (mostly to avoid chasing two rabbits at once). 
> 
> That is a new area to me. I see that CentOS is listed under
> bootstrap/generated-dists/centos7/. Would the only missing piece be the
> centos7 entries in .gitlab-ci.yml, or am i missing something? I can give
> that a try tomorrow.

that would be much appreciated and I can help with that, but it's certainly not a requirement to get your patch for the missing-field-initializers in. I'll review later on.

-slow

-- 
Ralph Boehme, Samba Team                https://samba.org/
Samba Developer, SerNet GmbH   https://sernet.de/en/samba/
GPG-Fingerprint   FAE2C6088A24252051C559E4AA1E9B7126399E46




More information about the samba-technical mailing list