Add an interface public interface MavenSessionAware { public void setMavenSession(MavenSession session); public MavenSession getMavenSession(); }
and make the impl additionally implement the MavenSessionAware interface? 2009/8/4 Milos Kleint <mkle...@gmail.com> > unless the MavenSession can be somehow computed (how?) there's no other way > I can see. > > Milos > > On Tue, Aug 4, 2009 at 10:49 AM, Benjamin Bentmann < > benjamin.bentm...@udo.edu> wrote: > > > Hi, > > > > in the interface ToolchainManagerPrivate, i.e. the one used by the > > maven-toolchain-plugin to setup the toolchains for a build, we currently > > have the method > > > > ToolchainPrivate[] getToolchainsForType( String type ) > > > > The impl of this method processes the toolchains.xml. Given the above > > method signature, there is currently no way to configure the path to the > > toolchains.xml without using global state in the component. This is > > troublesome for multi-threaded Maven executions, say during embedded > usage. > > > > Hence I suggest to extend the method to > > > > ToolchainPrivate[] getToolchainsForType( String type, MavenSession > > session ) > > > > i.e. additionally passing in the session. This is one way to allow the > impl > > to query per-session config like the path to the toolchains.xml. > > > > The maven-toolchain-plugin is the only client of the > > ToolchainManagerPrivate interface and was not released yet. However, > > changing the interface means the next plugin version can only be used > with a > > new Maven version, say 2.3+. > > > > Any other ideas? > > > > > > Benjamin > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org > > For additional commands, e-mail: dev-h...@maven.apache.org > > > > >