Maven Checkstyle Plugin
  1. Maven Checkstyle Plugin
  2. MCHECKSTYLE-42

checkstyle does not take into account proxy settings from settings.xml

    Details

    • Type: Bug Bug
    • Status: Open Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 2.0
    • Fix Version/s: None
    • Labels:
      None
    • Number of attachments :
      1

      Description

      I've been hesitating wether to report it as bug or as improvement, but at the moment I'd rate it as a bug.
      It took me quite some time to figure out why this was going wrong.
      In my settings.xml I've defined our company proxysettings. These settings are used by Maven when connecting to the remote repository.

      However when using the checkstyle plugin as part of the site generation I can not obtain our checkstyle.xml which is available via http.
      I found a solution by adding the following parameters on the command line when continuum launches maven:
      -Dhttp.proxyHost=myproxy -Dhttp.proxyPort=80

      Wouldn't it be possible for the maven checkstyle plugin to use the settings defined in settings.xml, so I've only to define those once?

      FYI the error generated:
      [INFO] Generate "Dependencies" report.
      [INFO] Generate "Issue Tracking" report.
      [INFO] Generate "Project License" report.
      [INFO] Generate "Mailing Lists" report.
      [INFO] Generate "Source Repository" report.
      [INFO] Generate "Project Team" report.
      [INFO] Generate "Maven Surefire Report" report.
      [INFO] Generate "Checkstyle" report.
      [INFO] ------------------------------------------------------------------------
      [ERROR] BUILD ERROR
      [INFO] ------------------------------------------------------------------------
      [INFO] Error during report generation

      Embedded error: Unable to find configuration file location.
      http://spirou.mycompany.be/javadev/install/checkstyle/mycompany-checkstyle-1.5.xml
      [INFO] ------------------------------------------------------------------------
      [INFO] Trace
      org.apache.maven.lifecycle.LifecycleExecutionException: Error during report generation
      at org.apache.maven.lifecycle.DefaultLifecycleExecutor.executeGoals(DefaultLifecycleExecutor.java:559)
      at org.apache.maven.lifecycle.DefaultLifecycleExecutor.executeGoalWithLifecycle(DefaultLifecycleExecutor.java:475)
      at org.apache.maven.lifecycle.DefaultLifecycleExecutor.executeGoal(DefaultLifecycleExecutor.java:454)
      at org.apache.maven.lifecycle.DefaultLifecycleExecutor.executeGoalAndHandleFailures(DefaultLifecycleExecutor.java:306)
      at org.apache.maven.lifecycle.DefaultLifecycleExecutor.executeTaskSegments(DefaultLifecycleExecutor.java:273)
      at org.apache.maven.lifecycle.DefaultLifecycleExecutor.execute(DefaultLifecycleExecutor.java:140)
      at org.apache.maven.DefaultMaven.doExecute(DefaultMaven.java:322)
      at org.apache.maven.DefaultMaven.execute(DefaultMaven.java:115)
      at org.apache.maven.cli.MavenCli.main(MavenCli.java:256)
      at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
      at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
      at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
      at java.lang.reflect.Method.invoke(Method.java:324)
      at org.codehaus.classworlds.Launcher.launchEnhanced(Launcher.java:315)
      at org.codehaus.classworlds.Launcher.launch(Launcher.java:255)
      at org.codehaus.classworlds.Launcher.mainWithExitCode(Launcher.java:430)
      at org.codehaus.classworlds.Launcher.main(Launcher.java:375)
      Caused by: org.apache.maven.plugin.MojoExecutionException: Error during report generation
      at org.apache.maven.plugins.site.SiteMojo.execute(SiteMojo.java:389)
      at org.apache.maven.plugin.DefaultPluginManager.executeMojo(DefaultPluginManager.java:412)
      at org.apache.maven.lifecycle.DefaultLifecycleExecutor.executeGoals(DefaultLifecycleExecutor.java:534)
      ... 16 more
      Caused by: org.apache.maven.reporting.MavenReportException: Unable to find configuration file location.
      at org.apache.maven.plugin.checkstyle.CheckstyleReport.getConfigFile(CheckstyleReport.java:879)
      at org.apache.maven.plugin.checkstyle.CheckstyleReport.executeReport(CheckstyleReport.java:466)
      at org.apache.maven.reporting.AbstractMavenReport.generate(AbstractMavenReport.java:98)
      at org.apache.maven.plugins.site.SiteMojo.generateReportsPages(SiteMojo.java:802)
      at org.apache.maven.plugins.site.SiteMojo.execute(SiteMojo.java:301)
      ... 18 more
      Caused by: java.io.FileNotFoundException: http://spirou.mycompany.be/javadev/install/checkstyle/mycompany-checkstyle-1.5.xml
      at sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLConnection.java:789)
      at java.net.URL.openStream(URL.java:913)
      at org.codehaus.plexus.util.FileUtils.copyURLToFile(FileUtils.java:876)
      at org.apache.maven.plugin.checkstyle.Locator.resolveLocation(Locator.java:99)
      at org.apache.maven.plugin.checkstyle.CheckstyleReport.getConfigFile(CheckstyleReport.java:870)
      ... 22 more
      [INFO] ------------------------------------------------------------------------
      [INFO] Total time: 48 seconds
      [INFO] Finished at: Thu May 18 17:54:23 CEST 2006
      [INFO] Final Memory: 20M/38M
      [INFO] ------------------------------------------------------------------------

        Issue Links

          Activity

          Joakim Erdfelt made changes -
          Field Original Value New Value
          Link This issue depends upon MCHECKSTYLE-61 [ MCHECKSTYLE-61 ]
          Jerome Lacoste made changes -
          Link This issue relates to MWEBSTART-39 [ MWEBSTART-39 ]
          Olivier Lamy made changes -
          Link This issue is duplicated by MCHECKSTYLE-160 [ MCHECKSTYLE-160 ]
          Arlo Louis O'Keeffe made changes -
          Attachment fix-proxy-usage.patch [ 63359 ]

            People

            • Assignee:
              Unassigned
              Reporter:
              Frederic
            • Votes:
              7 Vote for this issue
              Watchers:
              8 Start watching this issue

              Dates

              • Created:
                Updated: