Maven Site Plugin
  1. Maven Site Plugin
  2. MSITE-265

filename with dot are considered as duplicate

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Affects Version/s: 2.0-beta-6
    • Fix Version/s: 2.0-beta-6
    • Component/s: doxia integration
    • Labels:
      None
    • Number of attachments :
      0

      Description

      I have two files :
      src/site/xdoc/releases/release1.6.3.xml
      src/site/xdoc/releases/release1.6.xml

      The output is : Embedded error: Files 'releases\release1.6.xml' clashes with existing 'c:\projets\ServicesLayerServer\src\site\xdoc\releases\release1.6.3.xml'.
      As I see this comes from org.apache.maven.doxia.module.xhtml.decoration.render.RenderingContext.
      There is

       
      this.outputName = document.substring( 0, document.indexOf( "." ) ).replace( '\\', '/' ) + ".html";
      

      IMHO it should be :

       
      this.outputName = document.substring( 0, document.lastIndexOf( "." ) ).replace( '\\', '/' ) + ".html";
      

      I have to try to update dependency in maven-site-plugin to last doxia to test this fix but I have some compilation error due to some changes in doxia

      c:\maven-svn\plugins\maven-site-plugin\src\main\java\org\apache\maven\plugins\site\ReportDocumentRenderer.java:[139,26] generate(org.codehaus.doxia.sink.Sink,java.util.Locale) in org.apache.maven.reporting.MavenReport cannot be applied to (org.apache.maven.doxia.siterenderer.sink.SiteRendererSink,java.util.Locale)

      This works with 2.0-beta-5.

        Issue Links

          Activity

          Hide
          Olivier Lamy added a comment -

          This need only a new release of doxia-module-xhtml.
          Whats about a branch starting from 1.0-alpha-9 and release a doxia-module-xhtml-1.0-alpha-10.

          Thoughts ?

          Show
          Olivier Lamy added a comment - This need only a new release of doxia-module-xhtml. Whats about a branch starting from 1.0-alpha-9 and release a doxia-module-xhtml-1.0-alpha-10. Thoughts ?
          Hide
          Olivier Lamy added a comment -

          fix in rev 586801.
          upgrade doxia version to 1.0-alpha-10-SNAPSHOT

          Show
          Olivier Lamy added a comment - fix in rev 586801. upgrade doxia version to 1.0-alpha-10-SNAPSHOT
          Hide
          Olivier Lamy added a comment -

          As DOXIA-161 has been reopened, this must be reopened.
          The it must be completed with a test on file like download.apt.vm.

          Show
          Olivier Lamy added a comment - As DOXIA-161 has been reopened, this must be reopened. The it must be completed with a test on file like download.apt.vm.
          Hide
          Olivier Lamy added a comment -

          fix in doxia-branches and in doxia trunks.
          look linked issue.

          Show
          Olivier Lamy added a comment - fix in doxia-branches and in doxia trunks. look linked issue.

            People

            • Assignee:
              Olivier Lamy
              Reporter:
              Olivier Lamy
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: