Maven 2.x and 3.x Deploy Plugin

deploy-file fails with the <files/> element containing spaces or line breaks

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Not A Bug
  • Affects Version/s: 2.7
  • Fix Version/s: None
  • Component/s: deploy:deploy-file
  • Labels:
    None
  • Environment:
    all
  • Patch Submitted:
    Yes
  • Number of attachments :
    1

Description

The deploy-file goal fails if we set a list of files with line breaks or spaces (for better readability since we have a long list of aside artifacts to deploy)

The following sample fails :

<configuration>
    <classifiers>
        classifier1, classifier2
        classifier3
    </classifiers>
    <files>
        ${basedir}/src/main/assets/artifact1.zip, ${basedir}/src/main/assets/artifact2.jar,
        ${basedir}/src/main/assets/artifact3.jar
    </files>
    <types>
        zip,
        jar, jar
    </types>
</configuration>

while the following works fine :

<configuration>
    <classifiers>
        classifier1, classifier2
        classifier3
    </classifiers>
    <files>
        ${basedir}/src/main/assets/artifact1.zip,${basedir}/src/main/assets/artifact2.jar,${basedir}/src/main/assets/artifact3.jar
    </files>
    <types>
        zip,
        jar, jar
    </types>
</configuration>

As visible in these configuration samples, the elements <classifiers/> end <types/> are not affected and work fine even with spaces or line breaks in them.

Activity

Hide
Permalink
Lucas Lampietti added a comment -

Forget this issue, cause filepath can contain some spaces, so trim is not a solution.
A simple workaround is to use the build-helper-maven-plugin:attach-artifact goal when necessary in POM and use the deploy-file goal only for command line

Show
Lucas Lampietti added a comment - Forget this issue, cause filepath can contain some spaces, so trim is not a solution. A simple workaround is to use the build-helper-maven-plugin:attach-artifact goal when necessary in POM and use the deploy-file goal only for command line

People

  • Assignee:
    Unassigned
    Reporter:
    Lucas Lampietti
Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: