From: Garrett Cooper <yanegomi@gmail.com>
To: Stephen Smalley <sds@tycho.nsa.gov>
Cc: ltp-list <ltp-list@lists.sf.net>
Subject: Re: [LTP] selinux-testsuite has moved
Date: Mon, 25 Apr 2011 12:38:55 -0700 [thread overview]
Message-ID: <96AA5F80-E871-4368-BEB7-BC9FA24FFABD@gmail.com> (raw)
In-Reply-To: <1303738356.4244.67.camel@moss-pluto>
On Apr 25, 2011, at 6:32 AM, Stephen Smalley <sds@tycho.nsa.gov> wrote:
> On Sat, 2011-04-23 at 02:13 +0530, Shubham Goyal wrote:
>> On Friday 22 April 2011 06:38 PM, Stephen Smalley wrote:
>>> Hi,
>>>
>>> A recent posting on lkml noted that the selinux testsuite in ltp doesn't
>>> work anymore. As no one seems to be maintaining it in the ltp, I'd
>>> suggest just removing it entirely. You may want to leave a README.MOVED
>>> or similar file pointing to the new location, e.g.:
>>>
>>> The selinux testsuite has been moved to a standalone testsuite
>>> separate from the ltp. You can obtain it via:
>>> git clone git://git.kernel.org/pub/scm/tests/selinux-testsuite
>>>
>>> Thanks.
>>>
>>
>> Hi Stephan,
>>
>> Can you please tell what was the exact problem in running LTP's selinux
>> testsuite?
>> I tried running them and it worked for me with a minor modification. The
>> current
>> 'test_selinux.sh' sets the LTPROOT as the LTP source's 'testscripts'
>> directory but as
>> per current build process '/opt/ltp' is the install directory. After
>> making these changes
>> in 'test_selinux.sh' script, the test cases worked fine for me.
>
> I don't know the exact problem; I just know that it didn't work for the
> person who mentioned it in passing on lkml. And apparently it didn't
> work for you either until you changed test_selinux.sh. But I know that
> test_selinux.sh has worked in the past, even after the change
> to /opt/ltp, and that we haven't changed it recently.
>
> Actually, I just tried running it via cd /opt/ltp
> && ./testscripts/test_selinux.sh and it seemed to work fine. This was
> with the current git, building in-tree, installing to /opt/ltp.
>
>> I believe keeping it as a part of LTP tree is a better idea and would
>> have better
>> chances of getting it maintained along with other LTP test case and modules.
>
> This has been discussed a bit in the past, e.g. see this thread:
> http://marc.info/?t=127790181000003&r=1&w=2
>
> At present we see no benefit, only cost, to maintaining the selinux
> testsuite in the ltp as it is regularly broken by unrelated changes
> elsewhere in the ltp and as it is not truly integrated into the ltp (you
> have to perform separate steps to build and run it). In comparison, we
> don't have to worry about unrelated changes breaking the standalone
> selinux testsuite, and it is certainly no harder to do this:
>
> git clone git://git.kernel.org/pub/scm/tests/selinux-testsuite
> cd selinux-testsuite
> sudo make test
>
> than to do this:
>
> git clone git://ltp.git.sourceforge.net/gitroot/ltp/ltp
> cd ltp
> make autotools
> ./configure
> make all
> make install
> cd testcases/kernel/security/selinux-testsuite
> make all
> make install
> cd /opt/ltp
> ./testscripts/test_selinux.sh
>
> If the selinux testsuite were fully integrated into the ltp (e.g. built
> and run by default if SELinux is enabled on the system), then that might
> be more worthwhile to keep it as part of the ltp. But in the 6 years
> since it was added to the ltp, that hasn't happened.
Do what's easiest for you guys to maintain upstream; what you suggested makes sense: pull from git and run selinux as-is. The twist I'm proposing here is as follows:
1. The selinux test suite needs to be made modular and portable. I'll provide you with autoconf/other patches sometime after I get back from vacation on friday.
2. I'll add logic to pull from git in ltp, which means the following: releases will come with the selinux test suite; archives and other non-release mechanisms won't. Thus, if you want to run the selinux testcases, you will need git, and will have to specify --with-selinux=yes when running configure, etc.
Sound ok?
Thanks!
-Garrett
------------------------------------------------------------------------------
WhatsUp Gold - Download Free Network Management Software
The most intuitive, comprehensive, and cost-effective network
management toolset available today. Delivers lowest initial
acquisition cost and overall TCO of any competing solution.
http://p.sf.net/sfu/whatsupgold-sd
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list
next prev parent reply other threads:[~2011-04-25 19:39 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-04-22 13:08 [LTP] selinux-testsuite has moved Stephen Smalley
2011-04-22 20:43 ` Shubham Goyal
2011-04-25 13:32 ` Stephen Smalley
2011-04-25 19:20 ` Lucas Meneghel Rodrigues
2011-04-25 19:38 ` Garrett Cooper [this message]
2011-04-26 13:21 ` Stephen Smalley
2011-05-06 11:25 ` Cyril Hrubis
[not found] ` <1304686654.1821.14.camel@moss-pluto>
2011-05-11 12:39 ` Cyril Hrubis
2012-01-17 9:48 ` Jan Stancek
2012-01-18 13:16 ` Cyril Hrubis
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=96AA5F80-E871-4368-BEB7-BC9FA24FFABD@gmail.com \
--to=yanegomi@gmail.com \
--cc=ltp-list@lists.sf.net \
--cc=sds@tycho.nsa.gov \
/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