Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] ltp-testsuite without native RPC (commit 8f1c4c49a2ab0fc9e6cf73370f4483e253e5ac2d)
@ 2015-10-15 14:06 Thomas De Schampheleire
  2015-10-15 18:17 ` Baruch Siach
  0 siblings, 1 reply; 7+ messages in thread
From: Thomas De Schampheleire @ 2015-10-15 14:06 UTC (permalink / raw)
  To: buildroot

Hi Baruch, all,

I only now saw your commit 8f1c4c49a2ab0fc9e6cf73370f4483e253e5ac2d
that disables again building ltp-testsuite without a native-RPC
toolchain. Using commit baedef979cda7783b399099b49cd23536e44b1c7 I had
added that support.

First of all, it looks to me that the patch is incomplete: the
Config.in file still allows selecting ltp-testsuite without RPC
support, and selects libtirpc in that case. While I haven't tried it
again, I think this will break compilation for these cases.

But more to the core of the problem: I wonder if disabling the entire
ltp-testsuite here is the right approach. The RPC support is only
needed for the small amount of network tests that use RPC. The ltp
testsuite as a whole is much broader than that. By disabling
ltp-testsuite entirely the users without native rpc toolchain are thus
'punished' unnecessarily hard.

What about patching ltp-testsuite instead, not enabling the RPC tests
in this case? At first sight this was added with following (ltp)
commit:
https://github.com/linux-test-project/ltp/commit/8e9a4dbb8eaa561e51d6d8ebe5a5342f3e498224

Thanks,
Thomas

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2015-10-22 17:00 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-10-15 14:06 [Buildroot] ltp-testsuite without native RPC (commit 8f1c4c49a2ab0fc9e6cf73370f4483e253e5ac2d) Thomas De Schampheleire
2015-10-15 18:17 ` Baruch Siach
2015-10-19 13:26   ` Thomas De Schampheleire
2015-10-19 17:35     ` Baruch Siach
2015-10-20  3:49       ` Baruch Siach
2015-10-22  8:24         ` Thomas De Schampheleire
2015-10-22 17:00           ` Baruch Siach

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox