From: Greg Banks <gnb@melbourne.sgi.com>
To: Andreas Gruenbacher <agruen@suse.de>
Cc: tes@sgi.com, brandon@ifup.org, Brandon Philips <bphilips@suse.de>,
xfs@oss.sgi.com
Subject: Re: [patch 2/5] [PATCH] acl: various improvements for test/run
Date: Mon, 09 Feb 2009 15:29:28 +1100 [thread overview]
Message-ID: <498FB128.4060008@melbourne.sgi.com> (raw)
In-Reply-To: <200902090229.47550.agruen@suse.de>
Andreas Gruenbacher wrote:
> Brandon,
>
> the version of the run script in the attr and acl packages is somewhat dated;
> I have made a few improvements since. I'll move this out of {acl,attr}/test
> and put the latest version into the libmisc/ directory.
>
> Note that I recently rewrote the run script from scratch. Unlike the old
> version, the new one runs the whole script inside the same shell. There are
> still a few problems to be solved; most importantly, user switching doesn't
> work, yet.
I hope you get that feature done soon, and look forward to reviewing
it. Let me just be Captain Obvious and say that user switching is a
critical feature for code used to drive ACL tests ;-)
> Eventually I want to replace the old version, though.
>
> http://savannah.nongnu.org/projects/shrun
>
>
>
>
>> Add a | test line that is similar to > but is interpreted as a regular
>> expression.
>>
>
> I have been using ">~" instead of "|" for that. I'm still not sure that
> including regular expression matching was a good idea though, so shrun
> doesn't have any of that yet. We'll se how things develop.
>
>
>
Regexp matching is the kind of thing that sounds like it could be quite
useful, but it's a pain to implement. When I wrote a Python interpreter
for the shrun language for use in Newpynfs, I didn't bother implementing
the regexp feature.
--
Greg Banks, P.Engineer, SGI Australian Software Group.
the brightly coloured sporks of revolution.
I don't speak for SGI.
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
next prev parent reply other threads:[~2009-02-09 4:32 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-01-08 1:53 [patch 0/5] acl: test/ improvements and integrate with make brandon
2009-01-08 1:53 ` [patch 1/5] [PATCH] acl: add make test target and use make to run tests brandon
2009-02-09 1:34 ` Andreas Gruenbacher
2009-01-08 1:53 ` [patch 2/5] [PATCH] acl: various improvements for test/run brandon
2009-02-09 1:29 ` Andreas Gruenbacher
2009-02-09 4:29 ` Greg Banks [this message]
2009-01-08 1:53 ` [patch 3/5] [PATCH] acl: move root tests to their own folder brandon
2009-02-09 1:52 ` Andreas Gruenbacher
2009-01-08 1:53 ` [patch 4/5] [PATCH] acl: move nfs " brandon
2009-02-09 1:51 ` Andreas Gruenbacher
2009-01-08 1:54 ` [patch 5/5] [PATCH] acl: minor fix to cp.test brandon
2009-02-09 1:40 ` Andreas Gruenbacher
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=498FB128.4060008@melbourne.sgi.com \
--to=gnb@melbourne.sgi.com \
--cc=agruen@suse.de \
--cc=bphilips@suse.de \
--cc=brandon@ifup.org \
--cc=tes@sgi.com \
--cc=xfs@oss.sgi.com \
/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