1) Yes. :-) 2) I also quite surprisingly noticed this as well while converting a plugin to use BuildContext and making it m2e compatible. IMO it would be great if DefailtBuildContext could be improved so that will detect changes. However, there might be some good reason this hasn't been implemented yet...
3) Don't follow. How is this m2e related? m2e has a dependency to BuildContext, but this doesn't make Maven dependent on m2e. /Anders On Sat, Sep 1, 2012 at 12:21 PM, Mark Struberg <strub...@yahoo.de> wrote: > Hi! > > > While doing further research on the incremental build I came accross the > org.sonatype.plexus.build.incremental.BuildContext. > > There are a few things around this class which caught my interrest. > > 1.) This class has a package org.sonatype, the artifact name groupId > org.codehaus.plexus and the project in git is sisu-build-api. > Quite confusing... > > 2.) This replaced our internal scanning. What wonders me a bit is that the > DefaultBuildContext always returns true for hasDelta(). Not sure about the > original impl, but thiw might need to get changed if we like to support > incremental builds. The problematic code can be found in > DefaultMavenResourcesFiltering. Not sure if this got used somewhere else as > well. > > 3.) Having the BuildContext API in a m2eclipse related package is not exactly > wise imo. This makes m2eclipse depending on maven and maven depending on > m2eclipse. > > > Wdyt? How to proceed? > > > > LieGrue, > strub > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org > For additional commands, e-mail: dev-h...@maven.apache.org > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
