All of lore.kernel.org
 help / color / mirror / Atom feed
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

  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 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.