maven-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Robert Scholte" <apa...@sourcegrounds.com>
Subject Re: SCM initRepo method
Date Fri, 30 Mar 2012 16:30:27 GMT
Hi,

Not every SCM is repository-based.
I see more and more differences between SCM's.
Should we split it into Client/Server versus Distributed model?

-Robert

Op Fri, 30 Mar 2012 09:46:51 +0200 schreef Chris Graham  
<chrisgwarp@gmail.com>:

> Heh. Done!
>
> -Chris
>
> On Fri, Mar 30, 2012 at 6:03 PM, Olivier Lamy <olamy@apache.org> wrote:
>
>> just do it in your coming patch :-)
>>
>> 2012/3/30 Chris Graham <chrisgwarp@gmail.com>:
>> > Any ideas of how to get to initialised loggers (from the plexus
>> container)
>> > in this method?
>> > My only option so far, is a new DefaultLog().
>> >
>> > I'd perfer to use the normal ConsoleLogger etc as set up by the
>> container,
>> > and available to any descendants of the AbstractCommand...
>> >
>> > It's just a nice to have.
>> >
>> > Also, anyone object to me adding a removeRepo method to
>> ScmTckTestCase.java?
>> >
>> > IE:
>> >
>> >    /**
>> >     * This method is available to those SCM clients that need to  
>> perform
>> >     * a cleanup at the end of the tests. It is needed when server side
>> >     * operations are performed, or the check out dirs are outside
>> >     * of the normal target directory.
>> >     */
>> >    public void removeRepo()
>> >        throws Exception
>> >    {
>> >    }
>> >
>> >    /**
>> >     * Provided to allow removeRepo() to be called.
>> >     * @see junit.framework.TestCase#tearDown()
>> >     */
>> >    @Override
>> >    protected void tearDown()
>> >        throws Exception
>> >    {
>> >        super.tearDown();
>> >        removeRepo();
>> >    }
>> >
>> >
>> > I'm thinking that the jazz provider (when scm provides a delete  
>> workspace
>> > command...) and possibly the other server based SCM's may want this,
>> > ClearCase, etc might benefit.
>> >
>> > It's just a nice to have. And in ScmTckTestCase is the right place for
>> it,
>> > as that is where initRepo() lives and is called by setUp().
>> >
>> > -Chris
>>
>>
>>
>> --
>> Olivier Lamy
>> Talend: http://coders.talend.com
>> http://twitter.com/olamy | http://linkedin.com/in/olamy
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
>> For additional commands, e-mail: dev-help@maven.apache.org
>>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


Mime
View raw message