SonarQube Plugins
  1. SonarQube Plugins
  2. SONARPLUGINS-1094

Scm Activity no longer works for ClearCase

    Details

    • Number of attachments :
      2

      Description

      The plugin uses the revision number to identify changes. ( SONARPLUGINS-854 )
      But the clearcase changelog command doesn't provide this information. ( SCM-619 )

      I hope this can be corrected by the next scm version.

      1. patch-SONARPLUGINS-1094.txt
        0.5 kB
        Jérémie Lagarde
      2. patch-SONARPLUGINS-1094-UsingCommitDate.txt
        3 kB
        Guillaume Fournols

        Issue Links

          Activity

          Hide
          Evgeny Mandrikov added a comment -

          Fixed in r3671.

          Show
          Evgeny Mandrikov added a comment - Fixed in r3671 .
          Hide
          Jérémie Lagarde added a comment -

          In ClearCase the revisions are specific to each file not by commit.
          The second analysis doesn't work, see : https://github.com/apache/maven-scm/blob/trunk/maven-scm-api/src/main/java/org/apache/maven/scm/command/changelog/AbstractChangeLogCommand.java#L48

          I propose a patch to disable the partial changelog for clearcase.

          Show
          Jérémie Lagarde added a comment - In ClearCase the revisions are specific to each file not by commit. The second analysis doesn't work, see : https://github.com/apache/maven-scm/blob/trunk/maven-scm-api/src/main/java/org/apache/maven/scm/command/changelog/AbstractChangeLogCommand.java#L48 I propose a patch to disable the partial changelog for clearcase.
          Hide
          Guillaume Fournols added a comment - - edited

          Hi,

          Not sure if it is a valid strategy, but my idea was to use the commit date as revision number for Clearcase files. Since revision number in CC is not global we cannot rely on it to base the partial changelog, but by replacing it by the commit date, the ProjectStatus will always have as revision the latest commit date.
          This way, we can implement in FixedClearCaseChangeLogCommand the method executeChangeLogCommand using the start version by decoding the start revision into a date and using this date as "-since" parameter of the lshistory command. This enables then the partial changelog for Clearcase.

          We tested this strategy successfully on our projects. I will attach our patch for this strategy.

          Regards,
          Guillaume

          Show
          Guillaume Fournols added a comment - - edited Hi, Not sure if it is a valid strategy, but my idea was to use the commit date as revision number for Clearcase files. Since revision number in CC is not global we cannot rely on it to base the partial changelog, but by replacing it by the commit date, the ProjectStatus will always have as revision the latest commit date. This way, we can implement in FixedClearCaseChangeLogCommand the method executeChangeLogCommand using the start version by decoding the start revision into a date and using this date as "-since" parameter of the lshistory command. This enables then the partial changelog for Clearcase. We tested this strategy successfully on our projects. I will attach our patch for this strategy. Regards, Guillaume

            People

            • Assignee:
              Unassigned
              Reporter:
              Jérémie Lagarde
            • Votes:
              12 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: