From: Roman Neuhauser <neuhauser@sigpipe.cz>
To: Daniel Shahaf <d.s@daniel.shahaf.name>
Cc: zsh-users@zsh.org
Subject: Re: globbing in conditional expressions
Date: Thu, 15 May 2014 12:05:47 +0200 [thread overview]
Message-ID: <20140515100547.GH1629@isis.sigpipe.cz> (raw)
In-Reply-To: <20140515091458.GB2174@tarsus.local2>
# d.s@daniel.shahaf.name / 2014-05-15 09:14:58 +0000:
> Roman Neuhauser wrote on Wed, May 14, 2014 at 23:09:23 +0200:
> > wonderful, thank you!
>
> You're welcome, but beware: the code doesn't work properly in all cases,
> e.g., as Bart suspected, [[ -m **/*attr.yo(/) ]] (in a built zsh source
> tree) false positives.
understood, still, thanks a lot for kickstarting it.
> > i'm trying to think of a situation where [[ -m $pat ]] && mangle $REPLY
> > would be useful for something other than foot-shooting... any ideas?
>
> Well, there is:
>
> while [[ -m $pat ]] ;
> # Suppose -m implicitly sets $REPLY to a matching filename ...
> () { mangle $1 && rm $1 } $REPLY
>
> Since the filename returned would be the first in readdir() order, this
> could result in starvation (some files never reaching mangle()) unless
> the underlying filesystem's readdir() provides more guarantees than the
> POSIX readdir() does.
that's the foot-shooting part. :) the above imo screams for change to
for f in $~pat; do
mangle $f && rm $f
done
globbing can provide for ordering (like if you needed the oldest or
largest file).
if you need a random file, shortcircuiting readdir() is not the answer.
--
roman
next prev parent reply other threads:[~2014-05-15 10:06 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-05-07 12:41 Roman Neuhauser
2014-05-07 14:44 ` Peter Stephenson
2014-05-08 10:55 ` Daniel Shahaf
2014-05-08 11:20 ` Peter Stephenson
2014-05-08 15:34 ` Bart Schaefer
2014-05-08 20:19 ` Roman Neuhauser
2014-05-13 15:41 ` Bart Schaefer
2014-05-14 4:19 ` Daniel Shahaf
2014-05-14 7:18 ` Bart Schaefer
2014-05-15 4:55 ` Bart Schaefer
2014-05-15 9:35 ` Daniel Shahaf
2014-05-15 14:33 ` Bart Schaefer
2014-05-15 9:29 ` Daniel Shahaf
2014-05-15 14:50 ` Bart Schaefer
2014-05-26 23:52 ` Daniel Shahaf
2014-05-30 3:59 ` Bart Schaefer
2014-05-30 8:47 ` Peter Stephenson
2014-05-30 15:55 ` Bart Schaefer
2014-05-31 4:43 ` Bart Schaefer
2014-05-31 17:34 ` Daniel Shahaf
2014-05-31 17:59 ` Bart Schaefer
2014-06-02 9:40 ` Peter Stephenson
2014-06-02 12:57 ` Daniel Shahaf
2014-06-02 14:15 ` Peter Stephenson
2014-05-30 14:45 ` Daniel Shahaf
2014-05-14 7:42 ` Christoph (Stucki) von Stuckrad
2014-05-14 21:09 ` Roman Neuhauser
2014-05-15 9:14 ` Daniel Shahaf
2014-05-15 10:05 ` Roman Neuhauser [this message]
2014-05-07 14:57 ` Mikael Magnusson
2014-05-07 15:03 ` Bart Schaefer
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=20140515100547.GH1629@isis.sigpipe.cz \
--to=neuhauser@sigpipe.cz \
--cc=d.s@daniel.shahaf.name \
--cc=zsh-users@zsh.org \
/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.
Code repositories for project(s) associated with this public inbox
https://git.vuxu.org/mirror/zsh/
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).