Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-11846

Plugin is not showing the path for the package grouping for all warnings that are not from Java or C# files

    XMLWordPrintable

Details

    • Bug
    • Status: Resolved (View Workflow)
    • Major
    • Resolution: Fixed
    • checkstyle-plugin
    • None
    • Linux 2.6.18-194.17.1.el5

    Description

      The feature we discussed together on GitHub is not working anymore.

      I don't know if it worked before because I used my custom plugin or because of the 3.20 revision of yours.

      Anyway, could we please investigate on why the plugin does not display the packages for a non-java files?
      I will try to make more efficients tests to help you.

      Regards

      Attachments

        Activity

          drulli Ulli Hafner added a comment -

          Hmm, I don't remember that I changed something in that part of the code.

          Are there packages shown currently? Or is there simply no such tab?

          drulli Ulli Hafner added a comment - Hmm, I don't remember that I changed something in that part of the code. Are there packages shown currently? Or is there simply no such tab?
          yciabaud yciabaud added a comment -

          There is no package tab, in fact I doubt that it worked previously.

          Maybe the results I looked at were generated with the custom plugin I forked from yours.

          If we can restart from the beginning maybe we will see what is wrong.

          yciabaud yciabaud added a comment - There is no package tab, in fact I doubt that it worked previously. Maybe the results I looked at were generated with the custom plugin I forked from yours. If we can restart from the beginning maybe we will see what is wrong.
          drulli Ulli Hafner added a comment -

          Ok, that makes sense. Can you attach one serialized warning in your checkstyle-warnings.xml file in your build folder. I need to see what text is written in the package attribute.

          drulli Ulli Hafner added a comment - Ok, that makes sense. Can you attach one serialized warning in your checkstyle-warnings.xml file in your build folder. I need to see what text is written in the package attribute.
          yciabaud yciabaud added a comment -

          One example of not packaged checkstyle-warnings.xml file.

          It contains a <pathName> tag but it is not well parsed by the plugin.

          yciabaud yciabaud added a comment - One example of not packaged checkstyle-warnings.xml file. It contains a <pathName> tag but it is not well parsed by the plugin.

          Code changed in jenkins
          User: Ulli Hafner
          Path:
          src/main/java/hudson/plugins/analysis/util/PackageDetectors.java
          src/main/java/hudson/plugins/analysis/util/model/AbstractAnnotation.java
          http://jenkins-ci.org/commit/analysis-core-plugin/603ad77bea6375f84af1377de3786d4ed0f95c64
          Log:
          JENKINS-11846 Fixed naming of undefined packages.

          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: src/main/java/hudson/plugins/analysis/util/PackageDetectors.java src/main/java/hudson/plugins/analysis/util/model/AbstractAnnotation.java http://jenkins-ci.org/commit/analysis-core-plugin/603ad77bea6375f84af1377de3786d4ed0f95c64 Log: JENKINS-11846 Fixed naming of undefined packages.
          drulli Ulli Hafner added a comment -

          I think I found the bug. I wrote 'undefined' into the package name instead of "-". Can you please re-test when our Jenkins-on-Jenkins CI finished the build?

          drulli Ulli Hafner added a comment - I think I found the bug. I wrote 'undefined' into the package name instead of "-". Can you please re-test when our Jenkins-on-Jenkins CI finished the build?
          yciabaud yciabaud added a comment -

          I am building the plugin from git and I tell you.

          yciabaud yciabaud added a comment - I am building the plugin from git and I tell you.
          yciabaud yciabaud added a comment - - edited

          OK it works now, thank you for this fix and excuse me for my bad testing previously.

          Thank you.
          – Yoann Ciabaud

          yciabaud yciabaud added a comment - - edited OK it works now, thank you for this fix and excuse me for my bad testing previously. Thank you. – Yoann Ciabaud
          drulli Ulli Hafner added a comment -

          Excellent!

          drulli Ulli Hafner added a comment - Excellent!
          dogfood dogfood added a comment -

          Integrated in plugins_analysis-core #3010
          JENKINS-11846 Fixed naming of undefined packages.

          Ulli Hafner :
          Files :

          • src/main/java/hudson/plugins/analysis/util/PackageDetectors.java
          • src/main/java/hudson/plugins/analysis/util/model/AbstractAnnotation.java
          dogfood dogfood added a comment - Integrated in plugins_analysis-core #3010 JENKINS-11846 Fixed naming of undefined packages. Ulli Hafner : Files : src/main/java/hudson/plugins/analysis/util/PackageDetectors.java src/main/java/hudson/plugins/analysis/util/model/AbstractAnnotation.java
          dogfood dogfood added a comment -

          Integrated in plugins_analysis-core #3061
          JENKINS-11846 Fixed naming of undefined packages.

          Ulli Hafner :
          Files :

          • src/main/java/hudson/plugins/analysis/util/PackageDetectors.java
          • src/main/java/hudson/plugins/analysis/util/model/AbstractAnnotation.java
          dogfood dogfood added a comment - Integrated in plugins_analysis-core #3061 JENKINS-11846 Fixed naming of undefined packages. Ulli Hafner : Files : src/main/java/hudson/plugins/analysis/util/PackageDetectors.java src/main/java/hudson/plugins/analysis/util/model/AbstractAnnotation.java
          dogfood dogfood added a comment -

          Integrated in plugins_analysis-core #3175
          JENKINS-11846 Fixed naming of undefined packages.

          Ulli Hafner :
          Files :

          • src/main/java/hudson/plugins/analysis/util/PackageDetectors.java
          • src/main/java/hudson/plugins/analysis/util/model/AbstractAnnotation.java
          dogfood dogfood added a comment - Integrated in plugins_analysis-core #3175 JENKINS-11846 Fixed naming of undefined packages. Ulli Hafner : Files : src/main/java/hudson/plugins/analysis/util/PackageDetectors.java src/main/java/hudson/plugins/analysis/util/model/AbstractAnnotation.java
          dogfood dogfood added a comment -

          Integrated in plugins_analysis-core #3178
          JENKINS-11846 Fixed naming of undefined packages.

          Ulli Hafner :
          Files :

          • src/main/java/hudson/plugins/analysis/util/model/AbstractAnnotation.java
          • src/main/java/hudson/plugins/analysis/util/PackageDetectors.java
          dogfood dogfood added a comment - Integrated in plugins_analysis-core #3178 JENKINS-11846 Fixed naming of undefined packages. Ulli Hafner : Files : src/main/java/hudson/plugins/analysis/util/model/AbstractAnnotation.java src/main/java/hudson/plugins/analysis/util/PackageDetectors.java
          dogfood dogfood added a comment -

          Integrated in plugins_analysis-core #4315
          JENKINS-11846 Fixed naming of undefined packages.

          Ulli Hafner :
          Files :

          • src/main/java/hudson/plugins/analysis/util/model/AbstractAnnotation.java
          • src/main/java/hudson/plugins/analysis/util/PackageDetectors.java
          dogfood dogfood added a comment - Integrated in plugins_analysis-core #4315 JENKINS-11846 Fixed naming of undefined packages. Ulli Hafner : Files : src/main/java/hudson/plugins/analysis/util/model/AbstractAnnotation.java src/main/java/hudson/plugins/analysis/util/PackageDetectors.java
          dogfood dogfood added a comment -

          Integrated in plugins_analysis-core #6379
          JENKINS-11846 Fixed naming of undefined packages.

          Ulli Hafner :
          Files :

          • src/main/java/hudson/plugins/analysis/util/PackageDetectors.java
          • src/main/java/hudson/plugins/analysis/util/model/AbstractAnnotation.java
          dogfood dogfood added a comment - Integrated in plugins_analysis-core #6379 JENKINS-11846 Fixed naming of undefined packages. Ulli Hafner : Files : src/main/java/hudson/plugins/analysis/util/PackageDetectors.java src/main/java/hudson/plugins/analysis/util/model/AbstractAnnotation.java
          dogfood dogfood added a comment -

          Integrated in plugins_analysis-core #6390
          JENKINS-11846 Fixed naming of undefined packages.

          Ulli Hafner :
          Files :

          • src/main/java/hudson/plugins/analysis/util/model/AbstractAnnotation.java
          • src/main/java/hudson/plugins/analysis/util/PackageDetectors.java
          dogfood dogfood added a comment - Integrated in plugins_analysis-core #6390 JENKINS-11846 Fixed naming of undefined packages. Ulli Hafner : Files : src/main/java/hudson/plugins/analysis/util/model/AbstractAnnotation.java src/main/java/hudson/plugins/analysis/util/PackageDetectors.java
          dogfood dogfood added a comment -

          Integrated in plugins_analysis-core #6417
          JENKINS-11846 Fixed naming of undefined packages.

          Ulli Hafner :
          Files :

          • src/main/java/hudson/plugins/analysis/util/model/AbstractAnnotation.java
          • src/main/java/hudson/plugins/analysis/util/PackageDetectors.java
          dogfood dogfood added a comment - Integrated in plugins_analysis-core #6417 JENKINS-11846 Fixed naming of undefined packages. Ulli Hafner : Files : src/main/java/hudson/plugins/analysis/util/model/AbstractAnnotation.java src/main/java/hudson/plugins/analysis/util/PackageDetectors.java
          dogfood dogfood added a comment -

          Integrated in plugins_analysis-core #6517
          JENKINS-11846 Fixed naming of undefined packages.

          Ulli Hafner :
          Files :

          • src/main/java/hudson/plugins/analysis/util/PackageDetectors.java
          • src/main/java/hudson/plugins/analysis/util/model/AbstractAnnotation.java
          dogfood dogfood added a comment - Integrated in plugins_analysis-core #6517 JENKINS-11846 Fixed naming of undefined packages. Ulli Hafner : Files : src/main/java/hudson/plugins/analysis/util/PackageDetectors.java src/main/java/hudson/plugins/analysis/util/model/AbstractAnnotation.java
          drulli Ulli Hafner added a comment -

          Integrated in Jenkins Analysis Plug-ins (Compile) #381
          JENKINS-11846 Fixed naming of undefined packages. (Revision 603ad77bea6375f84af1377de3786d4ed0f95c64)

          Result = SUCCESS

          drulli Ulli Hafner added a comment - Integrated in Jenkins Analysis Plug-ins (Compile) #381 JENKINS-11846 Fixed naming of undefined packages. (Revision 603ad77bea6375f84af1377de3786d4ed0f95c64) Result = SUCCESS
          dogfood dogfood added a comment -

          Integrated in plugins_analysis-core #10407
          JENKINS-11846 Fixed naming of undefined packages. (Revision 603ad77bea6375f84af1377de3786d4ed0f95c64)

          Result = SUCCESS
          Ulli Hafner :
          Files :

          • src/main/java/hudson/plugins/analysis/util/model/AbstractAnnotation.java
          • src/main/java/hudson/plugins/analysis/util/PackageDetectors.java
          dogfood dogfood added a comment - Integrated in plugins_analysis-core #10407 JENKINS-11846 Fixed naming of undefined packages. (Revision 603ad77bea6375f84af1377de3786d4ed0f95c64) Result = SUCCESS Ulli Hafner : Files : src/main/java/hudson/plugins/analysis/util/model/AbstractAnnotation.java src/main/java/hudson/plugins/analysis/util/PackageDetectors.java

          People

            drulli Ulli Hafner
            yciabaud yciabaud
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: