From: Yang Xu <xuyang2018.jy@cn.fujitsu.com>
To: ltp@lists.linux.it
Subject: [LTP] [PATCH] m4: remove useless ltp-securebits.m4
Date: Mon, 22 Jul 2019 17:59:24 +0800 [thread overview]
Message-ID: <5D3588FC.6030607@cn.fujitsu.com> (raw)
In-Reply-To: <1b8b7f73-fd31-58d2-5162-3648fa9729a5@163.com>
Hi Xiao
I have sent a v2 patch about removing more useless m4.
> Hi Xu,
>
> LGTM. :-)
>
> Reviewed-by: Xiao Yang <ice_yangxiao@163.com>
>
> Best Regards,
>
> Xiao Yang
>
> On 07/19/2019 06:48 PM, Yang Xu wrote:
>> Since commit fe9d9218c, we have detected linux/securebits.h in
>> confiure.ac. one place used the HAVE_SECUREBITS(ltp-securebits.m4
>> defined), it only controls compile. kernel/security/securebits/
>> check_keepcaps.c has HAVE_LINUX_SECUREBITS_H check in internal,
>> so removing it is safe.
>>
>> Signed-off-by: Yang Xu <xuyang2018.jy@cn.fujitsu.com>
>> ---
>> configure.ac | 1 -
>> include/mk/features.mk.default | 3 ---
>> include/mk/features.mk.in | 3 ---
>> m4/ltp-securebits.m4 | 10 ----------
>> testcases/kernel/security/Makefile | 4 ----
>> 5 files changed, 21 deletions(-)
>> delete mode 100644 m4/ltp-securebits.m4
>>
>> diff --git a/configure.ac b/configure.ac
>> index 3dcf282e8..f899d8ebc 100644
>> --- a/configure.ac
>> +++ b/configure.ac
>> @@ -226,7 +226,6 @@ LTP_CHECK_NOMMU_LINUX
>> LTP_CHECK_PERF_EVENT
>> LTP_CHECK_PRCTL_SUPPORT
>> LTP_CHECK_RLIMIT64
>> -LTP_CHECK_SECUREBITS
>> LTP_CHECK_SELINUX
>> LTP_CHECK_SIGNAL
>> LTP_CHECK_STATX
>> diff --git a/include/mk/features.mk.default
>> b/include/mk/features.mk.default
>> index 94b8b0c54..3a6cc5176 100644
>> --- a/include/mk/features.mk.default
>> +++ b/include/mk/features.mk.default
>> @@ -29,9 +29,6 @@ WITH_PYTHON := no
>> # Features knobs
>> -# Is securebits[.h], et all support available?
>> -HAVE_SECUREBITS := no
>> -
>> # Test suite knobs
>> # Enable testcases/kernel/power_management's compile and install?
>> diff --git a/include/mk/features.mk.in b/include/mk/features.mk.in
>> index 7536b6f20..8e561b738 100644
>> --- a/include/mk/features.mk.in
>> +++ b/include/mk/features.mk.in
>> @@ -29,9 +29,6 @@ WITH_PYTHON := @WITH_PYTHON@
>> # Features knobs
>> -# Is securebits[.h], et all support available?
>> -HAVE_SECUREBITS := @HAVE_SECUREBITS@
>> -
>> # Test suite knobs
>> # Enable testcases/kernel/power_management's compile and install?
>> diff --git a/m4/ltp-securebits.m4 b/m4/ltp-securebits.m4
>> deleted file mode 100644
>> index 7888e6335..000000000
>> --- a/m4/ltp-securebits.m4
>> +++ /dev/null
>> @@ -1,10 +0,0 @@
>> -dnl SPDX-License-Identifier: GPL-2.0-or-later
>> -dnl Copyright (c) Serge Hallyn (2010)
>> -
>> -AC_DEFUN([LTP_CHECK_SECUREBITS],[
>> - AC_CHECK_HEADERS(linux/securebits.h,[have_securebits=yes])
>> -if test "x$have_securebits" != xyes; then
>> - have_securebits=no
>> -fi
>> -AC_SUBST(HAVE_SECUREBITS,$have_securebits)
>> -])
>> diff --git a/testcases/kernel/security/Makefile
>> b/testcases/kernel/security/Makefile
>> index 0b4b98b83..eea794aa9 100644
>> --- a/testcases/kernel/security/Makefile
>> +++ b/testcases/kernel/security/Makefile
>> @@ -24,10 +24,6 @@ top_srcdir ?= ../../..
>> include $(top_srcdir)/include/mk/env_pre.mk
>> -ifneq ($(HAVE_SECUREBITS),yes)
>> -FILTER_OUT_DIRS += securebits
>> -endif
>> -
>> OPT_CFLAGS += -O
>> DEBUG_LDFLAGS += -s
>
>
>
>
> .
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.linux.it/pipermail/ltp/attachments/20190722/e9d70efb/attachment-0001.htm>
prev parent reply other threads:[~2019-07-22 9:59 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-07-19 10:48 [LTP] [PATCH] m4: remove useless ltp-securebits.m4 Yang Xu
2019-07-19 10:59 ` Xiao Yang
2019-07-22 9:52 ` [LTP] [PATCH v2 1/2] m4: remove some useless m4 Yang Xu
2019-07-22 9:52 ` [LTP] [PATCH v2 2/2] ltp-cap.m4: uncouple licap and cap_compare Yang Xu
2019-07-29 12:26 ` [LTP] [PATCH v2 1/2] m4: remove some useless m4 Petr Vorel
2019-07-29 13:16 ` Cyril Hrubis
2019-07-29 14:42 ` Petr Vorel
2019-07-30 5:18 ` Yang Xu
2019-07-30 5:45 ` [LTP] [PATCH v3 1/5] m4: remove useless ltp-securebits.m4 Yang Xu
2019-07-30 5:45 ` [LTP] [PATCH v3 2/5] m4: remove useless ltp-time.m4 Yang Xu
2019-07-30 5:45 ` [LTP] [PATCH v3 3/5] m4:remove useless ltp-fs_ioc_flags.m4 Yang Xu
2019-07-30 5:45 ` [LTP] [PATCH v3 4/5] m4: remove useless ltp-cgroupstats.m4 Yang Xu
2019-07-30 5:45 ` [LTP] [PATCH v3 5/5] m4: clean up ltp-signalfd.m4 Yang Xu
2019-08-02 8:57 ` [LTP] [PATCH v3 1/5] m4: remove useless ltp-securebits.m4 Petr Vorel
2019-07-22 9:59 ` Yang Xu [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=5D3588FC.6030607@cn.fujitsu.com \
--to=xuyang2018.jy@cn.fujitsu.com \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox