From: Petr Vorel <pvorel@suse.cz>
To: ltp@lists.linux.it
Subject: [LTP] [PATCH 0/4] TI-RPC: enhancements (pkg-config, libntirpc, travis)
Date: Wed, 22 Jan 2020 17:41:34 +0100 [thread overview]
Message-ID: <20200122164134.GB20996@dell5510> (raw)
In-Reply-To: <20200122144917.GB29412@rei>
Hi Cyril,
> Hi!
> I've looked over the patchset and it looks fine to me, acked.
Thanks for your review and ack.
Unfortunately there are errors, so I'm working on v2.
Missing AC_SUBST in first patch for TI-RPC CFLAGS and LIBS hide them,
when fixed it shows that TI-RPC is really just for libtirpc - not for glibc
nor for libntirpc due missing xdr_{u_,}char dependency (although I got some
reply on ticket for libntirpc [1]: use xdr_get{u,}int8 and xdr_put{u,}int8 in
libntirpc, so it might be fixed). This is dependency for both TI-RPC tests and
basic RPC tests (testcases/network/rpc/basic_tests). libntirpc might have also some
missing dependency for TI-RPC (pmap_unset, which is in glibc).
I still don't know whether current goal for libntirpc is to be API
compatible with libtirpc (similar to kerberos, which override libtirpc functions
with it's gssrpc_* variants via gssrpc/rename.h [2]).
On the other hand svc_unreg() is in both libtirpc and libntirpc, but missing in
glibc.
But I'm going to merge commit travis: Merge debian.cross-compile.*.sh
which is correct.
BTW I also get some "funny" errors on s390 on travis [3]:
install: error writing '/root/ltp-install/testcases/bin/mkdir05': No space left on device
(I never experienced lack of space on travis on intel, I guess non-intel archs are a bit experimental.)
Kind regards,
Petr
[1] https://github.com/nfs-ganesha/ntirpc/issues/200
[2] https://github.com/krb5/krb5/blob/master/src/include/gssrpc/rename.h
[3] https://api.travis-ci.org/v3/job/640230579/log.txt
prev parent reply other threads:[~2020-01-22 16:41 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-01-08 10:34 [LTP] [PATCH 0/4] TI-RPC: enhancements (pkg-config, libntirpc, travis) Petr Vorel
2020-01-08 10:34 ` [LTP] [PATCH 1/4] rpc-tirpc: Detect libtirpc with pkg-config Petr Vorel
2020-01-08 10:34 ` [LTP] [PATCH 2/4] rpc-tirpc: Add libntirpc support Petr Vorel
2020-01-08 10:34 ` [LTP] [PATCH 3/4] travis: Merge debian.cross-compile.*.sh Petr Vorel
2020-01-08 11:48 ` Petr Vorel
2020-01-08 10:34 ` [LTP] [PATCH 4/4] travis: Test also libntirpc Petr Vorel
2020-01-22 14:49 ` [LTP] [PATCH 0/4] TI-RPC: enhancements (pkg-config, libntirpc, travis) Cyril Hrubis
2020-01-22 16:41 ` Petr Vorel [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200122164134.GB20996@dell5510 \
--to=pvorel@suse.cz \
--cc=ltp@lists.linux.it \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.