[#MNG-3402] MavenArtifactFilterManager needs to not filtering doxia-sink-api - jira.codehaus.org

Maven 2 & 3

MavenArtifactFilterManager needs to not filtering doxia-sink-api

Created: 15/Feb/08 07:41 AM Updated: 27/Jul/09 07:03 PM Resolved: 27/Jul/09 07:03 PM

Return to search

Component/s:

Artifacts and Repositories

Affects Version/s:

2.0.8

Fix Version/s:

3.0-alpha-3

Time Tracking:

Not Specified

File Attachments:

MavenArtifactFilterManager.diff (1 kB)

Issue Links:

Related

This issue is related to:

~~MNG-3602~~ Schedule and release Doxia-1.1

dependent

This issue is depended upon by:

~~MSITE-299~~ Add Doxia logging support

Description

« Hide

A plugin (like pdf-plugin) needs to use the most recent sink API and not the API embedded in maven.

Sort Order:

[ Permalink | « Hide ]

Vincent Siveton added a comment - 21/Feb/08 05:58 AM - edited

An other solution could be to bump doxia-sink-api:1.0-beta-1 (when release done) in maven-reporting-api

[ Permalink | « Hide ]

Brian Fox added a comment - 03/Mar/08 11:00 AM

should we just shade it then?

[ Permalink | « Hide ]

Brett Porter added a comment - 03/Mar/08 04:13 PM

I don't think shading will work as the Sink interface is passed between core and plugin.

In doing this, there will be an implicit contract that the sink API can not have anything removed from it over time - though that's probably fair enough.

[ Permalink | « Hide ]

Brian Fox added a comment - 14/Mar/08 10:43 PM

doxia-sink-api 1.0-beta-1 isn't released yet...

[ Permalink | « Hide ]

Brian Fox added a comment - 17/Mar/08 12:16 PM

Doxia isn't ready yet. See here: http://www.nabble.com/DOXIA---is-the-next-release-ready--tt16064480s177.html#a16064480

[ Permalink | « Hide ]

John Casey added a comment - 03/Jul/08 07:06 PM

Pushing to 2.0.11 so we can have a smaller set of high-value issues to target for the next release (2.0.10).

[ Permalink | « Hide ]

John Casey added a comment - 09/Feb/09 09:35 AM

Consolidating to 2.1.0-M1 so we can then rename to 2.1.0. We can weed out any issues we want to push to a later release from this set once we've done the consolidation.

[ Permalink | « Hide ]

Brett Porter added a comment - 19/Feb/09 10:38 AM

I believe we need to drop this from 2.1.0

[ Permalink | « Hide ]

Lukas Theussl added a comment - 26/Jun/09 02:09 AM

This has been done in r783525, is the fix version correct?

[ Permalink | « Hide ]

Olivier Lamy added a comment - 26/Jun/09 02:25 AM

update fix version

[ Permalink | « Hide ]

Olivier Lamy added a comment - 27/Jul/09 07:03 PM

I close it as it's done in trunk.
I will open a separate issue concerning site plugin with current trunk.