subversion-users mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Daniel Shahaf <...@daniel.shahaf.name>
Subject Re: Feature request: expand wildcards
Date Tue, 04 Jan 2011 22:26:30 GMT
Stefan Sperling wrote on Tue, Jan 04, 2011 at 22:44:49 +0100:
> On Tue, Jan 04, 2011 at 10:47:52PM +0200, Daniel Shahaf wrote:
> > Daniel Shahaf wrote on Tue, Jan 04, 2011 at 22:22:18 +0200:
> > > It's hacky, but:
> > > 
> > >   svn commit --changelist :glob:'*.foo'
> > > 
> > > where the "changelist" is then interpreted as a glob pattern
> > > (let's say apr_fnmatch()).
> > > 
> > > Thoughts?
> > > 
> > 
> > That wasn't hard:
> > 
> > [[[
> > % $svn st -q
> > M       subversion/libsvn_wc/adm_ops.c
> > M       COMMITTERS
> > % $svn st -q --cl ':glob:*.c'
> > M       subversion/libsvn_wc/adm_ops.c
> > %
> > ]]]
> > 
> > Daniel
> > (still want to hear if people think it's actually a useful addition)
> 
> This is quite similar to the --include-pattern and --exclude-pattern
> options we temporarily had on trunk (for svn patch; they have since been
> removed again).
> 
> Do we really need to overload --changelist?

It does the same thing this feature does (specify targets), and
overloading means we don't have to revv the 23 client APIs which use
changelists.

But I'm not sure I like the change myself.

Daniel
(btw, do we have a way to say "except files in $this changelist"?)

Mime
View raw message