Uploaded image for project: 'Maven SCM'
  1. Maven SCM
  2. SCM-777

scm:validate ignores scmCheckWorkingDirectoryUrl configuration in favor of system property

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

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 1.9.1
    • 1.11.2
    • maven-plugin
    • None
    • Java 7 x64 on Windows 7

    Description

      org.apache.maven.scm.manager.AbstractScmManager.checkWorkingDirectoryUrl() uses...

       Boolean.getBoolean( CHECK_WORKING_DIRECTORY_URL ) 

      ...in order to check if it should check the repository on scm:validate. This will only react to the system property, and not to the maven configuration.

      Result: no maven config will enable the check working directory option, only passing it in as a jvm argument.

      Expected: this should work:

      <plugin>
          <groupId>org.apache.maven.plugins</groupId>
          <artifactId>maven-scm-plugin</artifactId>
          <configuration>
              <scmCheckWorkingDirectoryUrl>true</scmCheckWorkingDirectoryUrl>  <!-- this configuration is ignored, don't know why -->
          </configuration>
          <executions>
              <execution>
                <phase>validate</phase>
                <configuration>
                  <scmCheckWorkingDirectoryUrl>true</scmCheckWorkingDirectoryUrl> <!-- neither does this -->
                </configuration>
                <goals>
                  <goal>validate</goal>
                </goals>
              </execution>
          </executions>
      </plugin>
      
      

      Workaround: Use <systemProperty> section. Tried <systemPropertyVariables> and for some reason that didn't appear to work.

      Attachments

        Issue Links

        Activity

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

          People

            michael-o Michael Osipov
            maikhorma Mark Herman
            Votes:
            2 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment