[
https://issues.apache.org/jira/browse/SCM-992?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17571092#comment-17571092
]
ASF GitHub Bot commented on SCM-992:
------------------------------------
kwin commented on code in PR #159:
URL: https://github.com/apache/maven-scm/pull/159#discussion_r929286409
##########
maven-scm-providers/maven-scm-providers-git/maven-scm-provider-gitexe/src/main/java/org/apache/maven/scm/provider/git/gitexe/command/GitCommandLineUtils.java:
##########
@@ -196,4 +209,27 @@ public static int execute( Commandline cl,
CommandLineUtils.StringStreamConsumer
return exitCode;
}
+ static Map<String, String> prepareEnvVariablesForRepository(
GitScmProviderRepository repository,
+ Map<String,
String> environmentVariables )
+ {
+ Map<String, String> effectiveEnvironmentVariables = new HashMap<>();
+ if ( environmentVariables != null )
+ {
+ effectiveEnvironmentVariables.putAll( environmentVariables );
+ }
+ if ( StringUtils.isNotBlank( repository.getPrivateKey() ) )
+ {
+ if ( effectiveEnvironmentVariables.putIfAbsent(
VARIABLE_GIT_SSH_COMMAND, "ssh -IdentitiesOnly=yes -i "
+ + FilenameUtils.separatorsToUnix(
repository.getPrivateKey() ) ) != null )
+ {
+ LOGGER.warn( "Ignore GitScmProviderRepository.privateKey as
environment variable {} is already set",
+ VARIABLE_GIT_SSH_COMMAND );
+ }
+ }
+ if ( StringUtils.isNotBlank( repository.getPassphrase() ) )
+ {
+ LOGGER.warn( "GitScmProviderRepository.passphrase currently not
supported by provider 'git'" );
Review Comment:
Not sure. For me "gitexe" is more an internal term. It isn't mentioned in
https://maven.apache.org/scm/git.html. I would stick to "git" as that is the
provider name used in the SCM URL.
> Support explicitly configured SSH private key for gitexe
> --------------------------------------------------------
>
> Key: SCM-992
> URL: https://issues.apache.org/jira/browse/SCM-992
> Project: Maven SCM
> Issue Type: Improvement
> Components: maven-scm-provider-gitexe
> Reporter: Konrad Windszus
> Assignee: Michael Osipov
> Priority: Major
> Fix For: 2.0.0-M2
>
>
> Currently the {{privateKey}} parameter set on
> {{ScmProviderRepositoryWithHost}} is only evaluated in provider "jgit". It
> should also be used for provider "git". Same for {{passphrase}} parameter.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
