Uploaded image for project: 'Maven'
  1. Maven
  2. MNG-3402

MavenArtifactFilterManager needs to not filtering doxia-sink-api

    Details

      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
          siveton Siveton Vincent 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
          siveton Siveton Vincent 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
          brianf@infinity.nu Brian E. Fox (imported) added a comment -

          should we just shade it then?

          Show
          brianf@infinity.nu Brian E. Fox (imported) added a comment - should we just shade it then?
          Hide
          brettporter 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
          brettporter 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
          brianf@infinity.nu Brian E. Fox (imported) added a comment -

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

          Show
          brianf@infinity.nu Brian E. Fox (imported) added a comment - doxia-sink-api 1.0-beta-1 isn't released yet...
          Show
          brianf@infinity.nu Brian E. Fox (imported) added a comment - Doxia isn't ready yet. See here: http://www.nabble.com/DOXIA---is-the-next-release-ready--tt16064480s177.html#a16064480
          Hide
          jdcasey 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
          jdcasey 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
          jdcasey 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
          jdcasey 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
          brettporter Brett Porter added a comment -

          I believe we need to drop this from 2.1.0

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

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

          Show
          ltheussl Lukas Theussl added a comment - This has been done in r783525 , is the fix version correct?
          Hide
          olamy Olivier Lamy (*$^¨%`£) added a comment -

          update fix version

          Show
          olamy Olivier Lamy (*$^¨%`£) added a comment - update fix version
          Hide
          olamy 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
          olamy 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

            • Assignee:
              brettporter Brett Porter
              Reporter:
              siveton Siveton Vincent
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development