<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small">Hi Petr,</div><div class="gmail_default" style="font-size:small"><br></div></div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Nov 11, 2020 at 7:48 PM Petr Vorel <<a href="mailto:pvorel@suse.cz">pvorel@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 Li,<br>
<br>
> On Wed, Nov 11, 2020 at 5:56 PM Petr Vorel <<a href="mailto:pvorel@suse.cz" target="_blank">pvorel@suse.cz</a>> wrote:<br>
<br>
> > Hi,<br>
<br>
> > > On Wed, Nov 11, 2020 at 5:38 PM Cyril Hrubis <<a href="mailto:chrubis@suse.cz" target="_blank">chrubis@suse.cz</a>> wrote:<br>
<br>
> > > > Hi!<br>
> > > > Looks good to me, let's apply this unless anybody complains.<br>
<br>
<br>
> > > Hold on, please.<br>
<br>
> > > Seems we have to replace TIRPC_LIBS/TIRPC_CFLAGS with LIBTIRPC_* in all<br>
> > > respective Makefiles? Otherwise, it will fail like the Travis CI job I<br>
> > > guess.<br>
<br>
> > Correct, currently would fail without it.<br>
> > <a href="https://travis-ci.org/github/pevik/ltp/builds/742685737" rel="noreferrer" target="_blank">https://travis-ci.org/github/pevik/ltp/builds/742685737</a><br>
<br>
> > I'll send v2 where I implement this.<br>
<br>
<br>
> Thanks, I modified base on your patch, test result FYI<br>
> <a href="https://github.com/wangli5665/ltp/commit/dad10021771101e7fdd416af894be7484fcb7a52" rel="noreferrer" target="_blank">https://github.com/wangli5665/ltp/commit/dad10021771101e7fdd416af894be7484fcb7a52</a><br>
> <a href="https://travis-ci.org/github/wangli5665/ltp/builds/742909686" rel="noreferrer" target="_blank">https://travis-ci.org/github/wangli5665/ltp/builds/742909686</a><br>
You implemented it, thank you. Please push it after Travis finishes. </blockquote><div><br></div><div class="gmail_default" style="font-size:small">Pushed, and thanks for the explanation.</div><div class="gmail_default" style="font-size:small"></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Just an explanation, why I introduced TIRPC_{CFLAGS,LIBS}: there was Sun-RPC<br>
implementation in glibc, which I wanted to test. Thus variable name without<br>
"LIB" to say, that it could be also glibc.<br>
<br>
But Sun-RPC is broken, was deprecated and my patchset which removes it from<br>
glibc was accepted and released in glibc 2.32. Thus I'm going to send patch<br>
which reverts the commit 60b20c428060b20c428 ("rpc: Enable and fix build basic<br>
RPC tests with glibc SunRPC").<br>
<br>
My long term goal: fix and cleanup RPC tests (some are failing also with<br>
libtirpc), move these tests to libtirpc sources (they needs to be cleaned. And<br>
also I'd prefer to not put there whole LTP library, but just needed subset).<br>
<br>
Kind regards,<br>
Petr<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>