Re: Input validation observations
On Thu, 2010-12-02 at 14:05 +0530, Noorul Islam K M wrote:
> Julian Foad <julian.foad_at_wandisco.com> writes:
> > I tried some potentially invalid inputs to "svn" a week or two ago and
> > made notes on what I found. Just posting here in case anyone wants to
> > do something about one or more of them.
> > Noorul, I'm including you in the "To" addresses because you said you
> > were looking for more small tasks to do, so feel free to pick one of
> > these if you want to.
> > Where I end with a question mark, it means I'm not sure if we want this
> > change, it's just a suggestion.
> > * "svn checkout ^/ ^/y" -> "A asf/cxf, A asf/cxf/utils, ...". (Don't
> > try this without being ready on the Ctrl-C or Ctrl-\!) It seems to
> > ignore "^/y" and create ./(basename(^/)); should fail: "'^/y' is not a
> > WC path".
> > * "svn checkout ^/subversion/trunk/build ^/y" -> "Checked out revision
> > 1040465. URL 'https://svn.apache.org/repos/asf/y' doesn't exist".
> > Bleach - that's just crazy. Should fail: "'^/y' is not a WC path".
> In this case code assumes that the user is trying to checkout from both
> source into current directory. i.e, If all source targets are URLs then
> the checkout target becomes "."
Oh yes, you're right - there's no problem there. I didn't notice that
"svn checkout URL URL" was a supported syntax. Thanks.
Received on 2010-12-02 13:22:25 CET
This is an archived mail posted to the Subversion Dev