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

Reply via email to