[#MRELEASE-35] release plugin doesn't tag correctly with svn+ssh when remote and local username don't match - jira.codehaus.org

Maven 2.x Release Plugin

release plugin doesn't tag correctly with svn+ssh when remote and local username don't match

Created: 02/Dec/05 12:02 AM Updated: 21/Mar/07 03:49 PM Resolved: 21/Mar/07 03:49 PM

Component/s:

scm

Affects Version/s:

None

Fix Version/s:

2.0-beta-5

Time Tracking:

Original Estimate:

30 minutes

Remaining Estimate:

30 minutes

Time Spent:

Not Specified

Issue Links:

dependent

This issue depends upon:

~~SCM-217~~ workaround svn issue with svn+ssh scheme requiring to specify user@host and ignoring --username

Description

« Hide

svn is very stubborn about this. Basically you have to have the value in <developerConnection> match the url used to check it out.

Things that will fail:

if you pass in a different username to --username, it is not passed on to ssh
if you specify a username in the ssh command it works, as long as the URL is the same in both. svn+ssh://bporter@foo/... is considered a different repository to svn+ssh://foo/... even if I am bporter

I think we can take a couple of steps:

for svn+ssh, put the username in the URL instead of passing --username
possibly check the current checkout (svn info) and derive the tag location from that, ignoring the local checkout OR if they don't match, relocate the local checkout to that of developerConnection
use javasvn instead

Sort Order:

[ Permalink | « Hide ]

Brett Porter added a comment - 08/Dec/05 01:03 PM

additional issue to note from Greg: if the tagBase contains scm:svn or svn: instead of just the svn url, then that doesn't work without helpful error reporting

[ Permalink | « Hide ]

Emmanuel Venisse added a comment - 08/Dec/05 02:05 PM

for release plugin code in svn, tagBase isn't necessary if projects use subversion standards : trunk, tags, branches

[ Permalink | « Hide ]

Jerome Lacoste added a comment - 23/Jun/06 09:39 AM

I've just been beaten by that

Basically, --username is ignored for every command with a URLs based on svn+ssh form. Dixit darix on #svn. Will try to find a fix.

[ Permalink | « Hide ]

Jerome Lacoste added a comment - 23/Jun/06 10:08 AM

After some more tests, of all the commands currently implemented in the scm provider,

log, co, diff, list, remove all work with --username and no username in the svn+ssh://server.com...

so copy seems to be the only command that requires to be fixed. Taking issue...

[ Permalink | « Hide ]

Jerome Lacoste added a comment - 23/Jun/06 12:42 PM

I fixed the scm svn provider to add or override the user into the url for tag and checkout commands. See ~~SCM-217~~ for patch with unit tests.

Patched release plugin was successfully tested.

[ Permalink | « Hide ]

Jerome Lacoste added a comment - 23/Jun/06 12:45 PM

Just as a side note, the patched release plugin failed during site generation because of cobertura failing to find the CommandLineBuilder class. probably another issue, but wanted to add it here just in case.