[GitHub] [maven-deploy-plugin] vtitov commented on issue #6: [MDEPLOY-255] altDeploymentRepositoryURL w/a for altDeploymentRepository in mingw/c…

[GitBox <gi...@apache.org>]

vtitov commented on issue #6: [MDEPLOY-255] altDeploymentRepositoryURL w/a for altDeploymentRepository in mingw/c…
URL: https://github.com/apache/maven-deploy-plugin/pull/6#issuecomment-484439396
 
 
   @michael-o, @eolivelli,
   
   For me it is indeed w/a, which lets me modify, compile and deploy some stuff from github to internal repo to debug the project I'm working on.
   I consider it as a draft because I'm not sure it's me who have to decide if deploy plugin should get a couple (or three couples) of new parameters or there is another w/a.
   I suppose the problem might be related to org.apache.maven.shared.transfer.project.deploy.ProjectDeployerRequest from maven-artifact-transfer, but I considered as overkill for me to debug this project.
   he problem itself might be in incorrect storing/parsing colons and URL part of altDeploymentRepository parameter in mingw environment, because cygwin/mings might make attempts to convert unix/windows path separators: 
   - ':'<->';'
   - '/'<->'\\'
   When I pass repo id as text and url as URL plugin works fine.
   I'm not sure I'm ready for writing tests right now.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services