[JIRA] (JENKINS-13639) Use Token Macro capability to expand variables

Ales Nosek created JENKINS-13639:
------------------------------------

             Summary: Use Token Macro capability to expand variables
                 Key: JENKINS-13639
                 URL: https://issues.jenkins-ci.org/browse/JENKINS-13639
             Project: Jenkins
          Issue Type: Improvement
          Components: artifactdeployer
            Reporter: Ales Nosek
            Assignee: gbois


Jenkins provides a powerful macro expansion capability https://wiki.jenkins-ci.org/display/JENKINS/Token+Macro+Plugin

In file ArtifactDeployerBuilder.java I'd suggest to replace:
final String includes = build.getEnvironment(listener).expand(entry.getIncludes());
final String excludes = build.getEnvironment(listener).expand(entry.getExcludes());
final String outputPath = build.getEnvironment(listener).expand(entry.getRemote());

with something like:
final String includes = TokenMacro.expandAll(build, listener, entry.getIncludes());
final String excludes = TokenMacro.expandAll(build, listener, entry.getExcludes());
final String outputPath = TokenMacro.expandAll(build, listener, entry.getRemote());

Thank you for considering this and for the great ArtifactDeployer plugin.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jenkins-ci.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

       

    [ https://issues.jenkins-ci.org/browse/JENKINS-13639?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=162661#comment-162661 ]

Gregory Boissinot commented on JENKINS-13639:
---------------------------------------------

Thanks for using the artifactdeployer plugin.
Why not using this feature.
However, I don't want to introduce a dependency to the Token Macro plugin. It doesn't make sense.
Token Marco should extract its API to a dedicated library and then I could integrate this library.
               

> Use Token Macro capability to expand variables
> ----------------------------------------------
>
>                 Key: JENKINS-13639
>                 URL: https://issues.jenkins-ci.org/browse/JENKINS-13639
>             Project: Jenkins
>          Issue Type: Improvement
>          Components: artifactdeployer
>            Reporter: Ales Nosek
>            Assignee: Gregory Boissinot
>
> Jenkins provides a powerful macro expansion capability https://wiki.jenkins-ci.org/display/JENKINS/Token+Macro+Plugin
> In file ArtifactDeployerBuilder.java I'd suggest to replace:
> final String includes = build.getEnvironment(listener).expand(entry.getIncludes());
> final String excludes = build.getEnvironment(listener).expand(entry.getExcludes());
> final String outputPath = build.getEnvironment(listener).expand(entry.getRemote());
> with something like:
> final String includes = TokenMacro.expandAll(build, listener, entry.getIncludes());
> final String excludes = TokenMacro.expandAll(build, listener, entry.getExcludes());
> final String outputPath = TokenMacro.expandAll(build, listener, entry.getRemote());
> Thank you for considering this and for the great ArtifactDeployer plugin.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jenkins-ci.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

       

Gregory Boissinot updated JENKINS-13639

Use Token Macro capability to expand variables

Improvement for the token-macro plugin: Make a dedicated library for using it in another plugins without introduced hard dependency to the token-macro plugin.
Change By:	Gregory Boissinot (10/Jul/12 5:01 PM)
Component/s:	token-macro

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators.
For more information on JIRA, see: http://www.atlassian.com/software/jira