Maven 2 & 3

Maven Reporting API is binary incompatible in 2.0.8-SNAPSHOT by r579987

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Blocker Blocker
  • Resolution: Fixed
  • Affects Version/s: 2.0.8
  • Fix Version/s: 2.0.8
  • Component/s: Sites & Reporting
  • Labels:
    None
  • Number of attachments :
    3

Description

See example exception file attached.

I tracked the problem down to this change:

r579987 | vsiveton | 2007-09-27 22:31:07 +1000 (Thu, 27 Sep 2007) | 2 lines
o bumped to the release doxia-sink-api
o used org.apache.maven.doxia.sink.Sink instead of the deprecated org.codehaus.doxia.sink.Sink

Ill attempt a rollback locally of the change and report back.

  1. atlassian-user-latest.tar.gz
    18/Oct/07 8:28 PM
    340 kB
    jameswdumay
  2. maven-doxia-rollback.patch
    18/Oct/07 12:24 AM
    0.6 kB
    jameswdumay
  3. sample-exception-report.txt
    17/Oct/07 11:44 PM
    3 kB
    jameswdumay

Issue Links

Activity

Hide
jameswdumay added a comment -

Patch fixes binary compatibility issue.

Ive tested this against the latest site plugin and all seems well.

Show
jameswdumay added a comment - Patch fixes binary compatibility issue. Ive tested this against the latest site plugin and all seems well.
Hide
Vincent Siveton added a comment -

org.codehaus.doxia.sink.Sink will be removed in Doxia 1.0 so we will break the binary compatible in the futur.

Show
Vincent Siveton added a comment - org.codehaus.doxia.sink.Sink will be removed in Doxia 1.0 so we will break the binary compatible in the futur.
Hide
Vincent Siveton added a comment -

I cannot reproduce your error with both maven and checkstyle snapshot. If you use 2.0.8-SNAPSHOT, try with 2.0.7.

FYI, no change in the API had been reported in 2.0.5, 2.0.6 and 2.0.7.

Show
Vincent Siveton added a comment - I cannot reproduce your error with both maven and checkstyle snapshot. If you use 2.0.8-SNAPSHOT, try with 2.0.7. FYI, no change in the API had been reported in 2.0.5, 2.0.6 and 2.0.7.
Hide
Brett Porter added a comment -

Vincent, the problem is that it won't work with non-snapshot versions of everything (which is what most people are using . If you use maven 2.0.8-SNAPSHOT with a released version of the checkstyle plugin, can you reproduce the problem?

Show
Brett Porter added a comment - Vincent, the problem is that it won't work with non-snapshot versions of everything (which is what most people are using . If you use maven 2.0.8-SNAPSHOT with a released version of the checkstyle plugin, can you reproduce the problem?
Hide
Vincent Siveton added a comment -

Brett, I try these steps:

Clean my repo
Install maven 2.0.8-SNAPSHOT (r586191)
Call mvn checkstyle:checkstyle (so using maven-checkstyle-plugin:2.1)

no error here...

Show
Vincent Siveton added a comment - Brett, I try these steps: Clean my repo Install maven 2.0.8-SNAPSHOT (r586191) Call mvn checkstyle:checkstyle (so using maven-checkstyle-plugin:2.1) no error here...
Hide
jameswdumay added a comment -

Vincent, checkout a project that uses site docs.

then generate the site using:

mvn clean site

You should get the exception.

Ill upload a testcase for you in a moment.

Show
jameswdumay added a comment - Vincent, checkout a project that uses site docs. then generate the site using: mvn clean site You should get the exception. Ill upload a testcase for you in a moment.
Hide
jameswdumay added a comment -

Unpack and run with maven 2.0.8-SNAPSHOT from trunk:

mvn clean site

Show
jameswdumay added a comment - Unpack and run with maven 2.0.8-SNAPSHOT from trunk: mvn clean site
Hide
Vincent Siveton added a comment -

Thanks, I will revert.

Brett, any suggestions to solve DOXIA-146 and doesn't break builds?

Show
Vincent Siveton added a comment - Thanks, I will revert. Brett, any suggestions to solve DOXIA-146 and doesn't break builds?
Hide
Brett Porter added a comment -

I think we can only cycle it over time - but certainly not an urgent problem - I would leave it in for 1.0.

Show
Brett Porter added a comment - I think we can only cycle it over time - but certainly not an urgent problem - I would leave it in for 1.0.
Hide
Vincent Siveton added a comment -

done

Show
Vincent Siveton added a comment - done

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: