trunk/branches/tags not always the first entry in the relative URL

TSYS-Merchant / posh-svn

A PowerShell environment for Subversion

12 stars 7 forks source link

trunk/branches/tags not always the first entry in the relative URL #16

Closed cdonnellytx closed 6 years ago

cdonnellytx commented 7 years ago

Fixed problem where it assumed trunk/branches/tags was always the first entry in the relative URL. That is not always true - a repo can have multiple directories each with their own trunk/branches/tags.

withinfocus commented 7 years ago

We're still fine with landing this if you are -- go ahead and merge.

cdonnellytx commented 7 years ago

Oops. Forgot I had a PR open on master. Let me clean out things I have going to my personal libraries and then see where it’s at. I think I also reformatted something too so I’ll undo that for the PR.

withinfocus commented 7 years ago

Formatting changes are honestly fine if they're a standard. We inherited this library and there are likely inconsistencies.

cdonnellytx commented 6 years ago

OK, I'm finished with my changes, and they should work without additional modules again.

Please let me know if you'd rather I resubmit the PR or break it up into smaller chunks.

cdonnellytx commented 6 years ago

All right, the only things remaining are:

Did I get the right authors? If not, what should they be?
Is there a preferred minimum SVN version? I picked 1.6.0 arbitrarily, I can go higher.
Is PowerShell 2.0 support still desired? The script should now work again in 2.0 but it does make the .psm1 more complex.