<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small">Hi Cyril, Ziyao,</div></div><div><br></div><div><div class="gmail_default" style="font-size:small">Thanks for catching the failure, I have posted a new patch fixing that.</div><div class="gmail_default" style="font-size:small">And sorry for my negligence in code pushing.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jul 8, 2021 at 8:24 PM Cyril Hrubis <<a href="mailto:chrubis@suse.cz">chrubis@suse.cz</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi!<br>
> > I made some refine and pushed, thanks!<br>
> Thanks for your review. Besides, I noticed the CI fails to be compiled <br>
> on Apine. The difference appears to be caused by the underlying libc <br>
> implementation: glibc and Alpine's musl-libc.<br>
> <br>
> Is there anything that needs to be modified for this patch? Or simply <br>
> ignore it in the CI script.<br>
<br>
This has to be fixed, the whole point of the CI is to catch build<br>
failures on different libc/compiler combinations.<br>
<br>
Looking at musl the sethostid() is not implemented at all, so I guess<br>
that the correct solution here would be adding a configure check for<br>
sethostid() and skip the whole test if it's not available.<br>
<br>
-- <br>
Cyril Hrubis<br>
<a href="mailto:chrubis@suse.cz" target="_blank">chrubis@suse.cz</a><br>
<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>