Julian Foad <julian.f...@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.
Thank you! I will start working on these one by one. > > 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". > > * "svn copy a ^/b c" doesn't detect the mixed source types in cl, only > in lib; should reject them at CLI level? > > * "svn info ^/b" -> "Not a valid URL"; should be "path '/b' not found > in revision REV"? > > * "svn mkdir ^/ a" -> "Illegal repository URL 'a'"; should say "can't > mix URL and local targets"? > > * "svn mkdir a ^/" -> "Can't create directory > 'https:/svn.apache.org/repos/asf'"; should not print the URL as if it's > a local path. > > * "svn mv ^/ ^/" -> "Cannot move path > 'https:/svn.apache.org/repos/asf' into itself"; should not print the URL > as if it's a local path. > > * "svn update ^/a" -> "Skipped > 'https://svn.apache.org/repos/asf/a' ..."; should fail: "'^/a' is not a > WC path"? > svn help update says that the argument should be a PATH. I think it is right to force the user to enter local PATH. Thanks and Regards Noorul
