SonarQube Plugins
  1. SonarQube Plugins
  2. SONARPLUGINS-2435

Activate by default the git blame "-w" option to ignore white-space changes

    Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: SCM-ACTIVITY-1.6
    • Component/s: SCM Activity
    • Labels:
      None
    • Environment:
      Jenkins on Linux with Maven3 build and sonar on Win7 with MYSQL
    • Number of attachments :
      0

      Description

      We moved our subversion repositories to Git where we changed all LineEndings from Windows-LE to Unix-LE. This introduced changes to all files from one person. SCM Blame doesn't make much sense anymore.

        Issue Links

          Activity

          Hide
          Jorge Costa added a comment -

          Hi,

          Can you check my last comment in http://jira.codehaus.org/browse/SCM-681#comment-323446. As it seems you will not be able to fix this, since it looks there is a bug in the implementation done in the Maven SCM plugin.

          Show
          Jorge Costa added a comment - Hi, Can you check my last comment in http://jira.codehaus.org/browse/SCM-681#comment-323446 . As it seems you will not be able to fix this, since it looks there is a bug in the implementation done in the Maven SCM plugin.
          Hide
          Stephan Leicht Vogt added a comment -

          Thanks for the info. Seems we have to wait for a fix in the maven-scm-provider-git.
          -
          Stephan

          Show
          Stephan Leicht Vogt added a comment - Thanks for the info. Seems we have to wait for a fix in the maven-scm-provider-git. - Stephan
          Hide
          Julien HENRY added a comment -

          We are already overriding MavenGitBlameCommand implementation for another reason so here is one more reason.
          I have forced ignoreWhitespace option as indeed current implementation looks wrong.

          Show
          Julien HENRY added a comment - We are already overriding MavenGitBlameCommand implementation for another reason so here is one more reason. I have forced ignoreWhitespace option as indeed current implementation looks wrong.
          Hide
          Fabrice Bellingard added a comment -

          Tested.

          Show
          Fabrice Bellingard added a comment - Tested.
          Hide
          Julien HENRY added a comment -

          No IT

          Show
          Julien HENRY added a comment - No IT

            People

            • Assignee:
              Julien HENRY
              Reporter:
              Stephan Leicht Vogt
            • Votes:
              1 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: