Maven 2 & 3

MavenArtifactFilterManager needs to not filtering doxia-sink-api

Details

  • Number of attachments :
    1

Description

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

Issue Links

Activity

Hide
Vincent Siveton added a comment - - edited

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

Show
Vincent Siveton added a comment - - edited An other solution could be to bump doxia-sink-api:1.0-beta-1 (when release done) in maven-reporting-api
Hide
Brian Fox added a comment -

should we just shade it then?

Show
Brian Fox added a comment - should we just shade it then?
Hide
Brett Porter added a comment -

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.

Show
Brett Porter added a comment - 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.
Hide
Brian Fox added a comment -

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

Show
Brian Fox added a comment - doxia-sink-api 1.0-beta-1 isn't released yet...
Hide
Brian Fox added a comment -
Show
Brian Fox added a comment - Doxia isn't ready yet. See here: http://www.nabble.com/DOXIA---is-the-next-release-ready--tt16064480s177.html#a16064480
Hide
John Casey added a comment -

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).

Show
John Casey added a comment - 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).
Hide
John Casey added a comment -

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.

Show
John Casey added a comment - 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.
Hide
Brett Porter added a comment -

I believe we need to drop this from 2.1.0

Show
Brett Porter added a comment - I believe we need to drop this from 2.1.0
Hide
Lukas Theussl added a comment -

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

Show
Lukas Theussl added a comment - This has been done in r783525, is the fix version correct?
Hide
Olivier Lamy added a comment -

update fix version

Show
Olivier Lamy added a comment - update fix version
Hide
Olivier Lamy added a comment -

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

Show
Olivier Lamy added a comment - I close it as it's done in trunk. I will open a separate issue concerning site plugin with current trunk.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: