SonarQube Plugins

Scm Activity no longer works for ClearCase

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: SCM-ACTIVITY-1.2, SCM-ACTIVITY-1.3
  • Fix Version/s: SCM-ACTIVITY-1.4
  • Component/s: SCM Activity
  • Labels:
    None
  • 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. Text File
    patch-SONARPLUGINS-1094.txt
    03/May/11 4:12 PM
    0.5 kB
    Jérémie Lagarde
  2. Text File
    patch-SONARPLUGINS-1094-UsingCommitDate.txt
    18/May/12 3:21 AM
    3 kB
    Guillaume Fournols

Issue Links

depends upon

New Feature - A new feature of the product, which has yet to be developed. SCM-619 [patch] Add revision number in changelog clearcase command.

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Task - A task that needs to be done. SONARPLUGINS-1871 Stop using the changelog/revision to determine if the blame information should be updated on a source file

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.
is related to

Bug - A problem which impairs or prevents the functions of the product. SONARPLUGINS-1192 Revision number cannot be aggregated up to project level for some VCS

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Evgeny Mandrikov added a comment -

Fixed in r3671.

Show
Evgeny Mandrikov added a comment - Fixed in r3671 .
Hide
Permalink
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
Permalink
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
Vote (12)
Watch (6)

Dates

  • Created:
    Updated:
    Resolved: