[ http://jira.codehaus.org/browse/MNG-1744?page=all ]
Vincent Siveton updated MNG-1744:
---------------------------------
Attachment: maven-site-plugin_doxia.diff
maven-reporting_doxia.diff
> Doxia move from org.codehaus to org.apache breaks Reporting API Contract.
> -------------------------------------------------------------------------
>
> Key: MNG-1744
> URL: http://jira.codehaus.org/browse/MNG-1744
> Project: Maven 2
> Type: Bug
> Components: design
> Versions: 2.0.1
> Reporter: Joakim Erdfelt
> Fix For: 2.0.2
> Attachments: maven-reporting_doxia.diff, maven-site-plugin_doxia.diff
>
>
> I encountered a bug with the XdocSink and XhtmlSink, and worked to fix them.
> In the process I discovered that testing this fix would require changes to
> the AbstractMavenReporting api in order for it to use the new signature for
> the moved Classes.
> This signature change will break exising Reporting plugins if it should occur.
> Some mechanism needs to be in place for this transition.
> --- (snip #maven) ---
> joakime: jvanzyl: uhm. the relocation of doxia breaks the reporting api.
> wouldn't that kind of change be considered a big deal?
> jvanzyl: not really, nothing we can't fix
> jvanzyl: how does the relocation break the reporting api?
> joakime: sure, but the AbstractMavenReport now has a different signature.
> joakime: that will break older report plugins that used the
> AbstractMavenReport object with the Sink from codehaus. Switching to apache
> would require a new AbstractMavenReport wouldn't it?.
> jvanzyl: yup, i guess it would but changes like this don't happen very often
> jvanzyl: we could make an adapter though
> jvanzyl: so not that hard to fix
> joakime: true.
> joakime: ok, this simple fix is turning into a bigger deal, with the
> reporting api needing to change and such.
> joakime: i'll hold off on that.
> jvanzyl: if that signature is exposed everywhere then that's no good
> joakime: it's part of the AbstractMavenReport.generate(Sink sink, Locale
> locale) method.
> joakime: and AbstractMavenReport.getSink()
> jvanzyl: crap
> joakime: *nod*
> joakime: want me to file a jira?
> jvanzyl: sure
> jvanzyl: thx
> joakime: np.
> joakime: would this be a DOXIA or MNG bug?
> jvanzyl: it affects the site plugin really
> jvanzyl: it's the client code that's going to be affected
> jvanzyl: it's not really a problem with doxia
> jvanzyl: we hosed our clients
> joakime: hmm.
> --- (/snip #maven) ----
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
