Uploaded image for project: 'Maven WAR Plugin'
  1. Maven WAR Plugin
  2. MWAR-376

Remove param properties that doesn't make sense for CLI usage

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 2.6
    • 3.0.0
    • None
    • None
    • n/a

    Description

      Currently the plugin supports configuring several of the parameters via CLI properties. This doesn't make sense in many cases and encourages bad Maven usage (builds should be reproducible). Remove the properties for those parameters. The v3.0.0 release is a good time to do this.

      First of all we need to identify candidates for removal.

      This is the list of parameters/properties maven-war-plugin contains at the moment:

      @Parameter( property = "archiveClasses", defaultValue = "false" )
       @Parameter( property = "resourceEncoding", defaultValue = "${project.build.sourceEncoding}" )
       @Parameter( property = "maven.war.webxml" )
       @Parameter( property = "maven.war.containerConfigXML" )
       @Parameter( property = "useCache", defaultValue = "false" )
       @Parameter( property = "maven.war.filteringDeploymentDescriptors", defaultValue = "false" )
       @Parameter( property = "maven.war.escapedBackslashesInFilePath", defaultValue = "false" )
       @Parameter( property = "maven.war.escapeString" )
       @Parameter( property = "maven.war.supportMultiLineFiltering", defaultValue = "false" )
       @Parameter( property = "maven.war.useJvmChmod", defaultValue = "true" )
       @Parameter( property = "primaryArtifact", defaultValue = "true" )
       @Parameter( property = "failOnMissingWebXml", defaultValue = "true" )
       @Parameter( property = "maven.war.skip", defaultValue = "false" )
      

      Attachments

        Issue Links

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            khmarbaise Karl Heinz Marbaise
            afloom Anders Hammar
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment