<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
  </head>
  <body>
    <p><br>
    </p>
    <div class="moz-cite-prefix">
      <pre>on 2019/11/21 16:21, Petr Vorel wrote:</pre>
    </div>
    <blockquote type="cite" cite="mid:20191121082118.GA14920@dell5510">
      <pre class="moz-quote-pre" wrap="">Hi Xu,

</pre>
      <blockquote type="cite">
        <pre class="moz-quote-pre" wrap="">Yes. I check <sys/quota.h> and <sys/prctl.h> on musl libc[1] and they don't include linux header files.
So I think checking both kernel and libc headers on other libc(musl,bionic) is meaningful.
</pre>
      </blockquote>
      <pre class="moz-quote-pre" wrap="">But in single C file we decide only on one of these two.
It's a similar problem as testing raw syscall or libc wrapper (which we already
solved with .test_variants).
NOTE: Bionic has generate it's headers from linux headers.</pre>
    </blockquote>
    <pre>I see.  
</pre>
    <pre>Thanks again for reminding Bionic like glibc uses linux headers.
</pre>
    <blockquote type="cite" cite="mid:20191121082118.GA14920@dell5510">
      <pre class="moz-quote-pre" wrap="">

</pre>
      <blockquote type="cite">
        <pre class="moz-quote-pre" wrap="">ps: If our travis-ci has a target with musl, I think it will be better. I don't know whether possible.
</pre>
      </blockquote>
      <pre class="moz-quote-pre" wrap="">It's here. But it requires some more fixes (in a meantime files are deleted),
so I'll post it once it's everything fixed
<a class="moz-txt-link-freetext" href="https://github.com/pevik/ltp/commits/travis/musl">https://github.com/pevik/ltp/commits/travis/musl</a>
<a class="moz-txt-link-freetext" href="https://travis-ci.org/pevik/ltp/builds/614575796">https://travis-ci.org/pevik/ltp/builds/614575796</a>

There is also CI on LTP on all glibc/uclibc-ng/musl in Buildroot
<a class="moz-txt-link-freetext" href="http://autobuild.buildroot.net/index.php?reason=ltp-testsuite-20190930">http://autobuild.buildroot.net/index.php?reason=ltp-testsuite-20190930</a>

+ There are other CI based on yocto/openembedded on glibc/uclibc-ng/musl</pre>
    </blockquote>
    <pre>It is so cool.  And I think I  can move your musl patches to my ltp fork, so that I can also test them on musl 
when I contribute my patches to ltp.

ps: I also think we should not block this patchset.
</pre>
    <blockquote type="cite" cite="mid:20191121082118.GA14920@dell5510">
      <pre class="moz-quote-pre" wrap="">

</pre>
      <blockquote type="cite">
        <pre class="moz-quote-pre" wrap="">[1]<a class="moz-txt-link-freetext" href="http://git.musl-libc.org/cgit/musl/tree/include/sys/quota.h">http://git.musl-libc.org/cgit/musl/tree/include/sys/quota.h</a>
</pre>
      </blockquote>
      <pre class="moz-quote-pre" wrap="">
Kind regards,
Petr


</pre>
    </blockquote>
  </body>
</html>