Maven 2.x and 3.x Site Plugin

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

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: