You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm-dev@maven.apache.org by Wim Deblauwe <wi...@gmail.com> on 2005/11/18 16:41:45 UTC
release plugin update for clearcase
Hi guys,
I've implemented add, checkin, edit, remove, status and tag for clearcase.
Is somebody willing to update the release plugin so I can test? Do I need to
create a patch with all my current changes for someone to update the release
plugin?
regards,
Wim
Re: release plugin update for clearcase
Posted by Wim Deblauwe <wi...@gmail.com>.
I know, but I'll settle for release:prepare for now. I would be already very
happy with that.
regards,
Wim
2005/11/18, Emmanuel Venisse <em...@venisse.net>:
>
> you need checkout for release:perform
>
> Wim Deblauwe a écrit :
> > Hi guys,
> >
> > I've implemented add, checkin, edit, remove, status and tag for
> > clearcase. Is somebody willing to update the release plugin so I can
> > test? Do I need to create a patch with all my current changes for
> > someone to update the release plugin?
> >
> > regards,
> >
> > Wim
>
>
Re: release plugin update for clearcase
Posted by Emmanuel Venisse <em...@venisse.net>.
you need checkout for release:perform
Wim Deblauwe a écrit :
> Hi guys,
>
> I've implemented add, checkin, edit, remove, status and tag for
> clearcase. Is somebody willing to update the release plugin so I can
> test? Do I need to create a patch with all my current changes for
> someone to update the release plugin?
>
> regards,
>
> Wim
Re: release plugin update for clearcase
Posted by Emmanuel Venisse <em...@venisse.net>.
Can you send me a patch with added apache header? I have no time for the moment.
Emmanuel
Wim Deblauwe a écrit :
> ok, thank you for fixing that. What I noticed is that you have not put
> the apache header on top of the source files. Shouldn't we do that do be
> correct?
>
> regards,
>
> Wim
>
> 2005/11/21, Emmanuel Venisse <emmanuel@venisse.net
> <ma...@venisse.net>>:
>
> I applied your new patch that contains some tabs and failure in add
> command test. I fixed them, can
> you check if my change in clearcase add command is ok?
>
> Emmanuel
>
>
Re: release plugin update for clearcase
Posted by Wim Deblauwe <wi...@gmail.com>.
ok, thank you for fixing that. What I noticed is that you have not put the
apache header on top of the source files. Shouldn't we do that do be
correct?
regards,
Wim
2005/11/21, Emmanuel Venisse <em...@venisse.net>:
>
> I applied your new patch that contains some tabs and failure in add
> command test. I fixed them, can
> you check if my change in clearcase add command is ok?
>
> Emmanuel
Re: release plugin update for clearcase
Posted by Emmanuel Venisse <em...@venisse.net>.
I applied your new patch that contains some tabs and failure in add command test. I fixed them, can
you check if my change in clearcase add command is ok?
Emmanuel
Wim Deblauwe a écrit :
> Allright, here we go again. I have updated SCM-77 with a new patch.
> Hopefully this one will satisfy all your needs :)
>
> if there are any things I should change, please let me know.
>
> For the edit and unedit commands I have used the ScmFileStatus.UNKNOWN,
> because I don't know what I should use there. Maybe you guys can put
> something better in there.
>
> regards,
>
> Wim
>
> 2005/11/19, Emmanuel Venisse <emmanuel@venisse.net
> <ma...@venisse.net>>:
>
>
>
> Wim Deblauwe a écrit :
> > Hey Emmanuel,
> >
> > 1) I'm using TortoiseSVN to generate the patch. Maybe I did something
> > wrong, I never worked with it before.
>
> Strange, it works fine normally.
>
> >
> > 2) I sure can. Normally I just let IntelliJ IDEA do my
> formatting. Don't
> > know if you have the correct settings? Otherwise, i'll just fix
> the tabs.
>
> http://maven.apache.org/guides/development/guide-m2-development.html#Maven
> Code Style
>
> >
> > 3) Well, because I was unable to test and also I could not get the
> > logging to work (see one of my previous emails)
>
> clearcase commands print something to console, so, you can copy this
> output to text files and parse
> them with consumers. You don't need logger for tests, only some
> junit assert that verify your
> consumer is correct.
>
> >
> > 4) I haven't made testclasses yet. Should they really do clearcase
> > operations or check if the command line is like it should be?
>
> command line only and test consumers.
>
> >
> > I'm out of office now, so I will not be able to do these things
> until
> > monday.
> >
> > regards,
> >
> > Wim
> >
> > 2005/11/18, Emmanuel Venisse <emmanuel@venisse.net
> <ma...@venisse.net>
> > <mailto: emmanuel@venisse.net <ma...@venisse.net>>>:
> >
> > Some comments about your patch :
> > - How do you have generated it? some files are present more
> than one
> > time.
> > - Can you replace all tab character with 4 spaces?
> > - Why your consumers do nothing?
> > - Where are test classes?
> >
> > if you can recreate your patch and integrate all those
> remarks, it
> > will be more simple to read,
> > apply and maintain.
> >
> > Emmanuel
> >
> > Wim Deblauwe a écrit :
> > > Done: http://jira.codehaus.org/browse/MNG-1622
> > >
> > > I did not modify the release-plugin.
> > >
> > > regards,
> > >
> > > Wim
> > >
> > > 2005/11/18, Emmanuel Venisse < emmanuel@venisse.net
> <ma...@venisse.net>
> > <mailto:emmanuel@venisse.net <ma...@venisse.net>>
> > > <mailto:emmanuel@venisse.net <ma...@venisse.net>
> <mailto: emmanuel@venisse.net <ma...@venisse.net>>>>:
> > >
> > > Great.
> > >
> > > you can create a patch for maven-scm and attach to a
> jira issue.
> > > I'll review it (and perhaps Dan too).
> > >
> > > Do you have modify release-plugin?
> > >
> > > Wim Deblauwe a écrit :
> > > > Hi guys,
> > > >
> > > > I've implemented add, checkin, edit, remove, status and
> > tag for
> > > > clearcase. Is somebody willing to update the release
> > plugin so I can
> > > > test? Do I need to create a patch with all my current
> > changes for
> > > > someone to update the release plugin?
> > > >
> > > > regards,
> > > >
> > > > Wim
> > >
> > >
> >
> >
>
>
Re: release plugin update for clearcase
Posted by Emmanuel Venisse <em...@venisse.net>.
I didn't deploy it, we don't deploy snapshot version of plugins.
You need to checkout release plugin and build it.
I added clearcase dependency in it, so you don't need to add clearcase provider dep in its conf.
Emmanuel
Wim Deblauwe a écrit :
> On what server did you deploy? I tried today, but I did not get a new
> version. I got this in my pom.xml:
>
> <build>
> <plugins>
> <plugin>
> <artifactId>maven-release-plugin</artifactId>
> <dependencies>
> <dependency>
> <groupId>org.apache.maven.scm</groupId>
> <artifactId>maven-scm-provider-clearcase</artifactId>
> <version>1.0-beta-1-SNAPSHOT</version>
> </dependency>
> </dependencies>
> </plugin>
> </plugins>
> </build>
>
> maybe I need to specify a version for the release plugin?
>
> regards,
>
> Wim
>
> 2005/11/21, Wim Deblauwe <wim.deblauwe@gmail.com
> <ma...@gmail.com>>:
>
> I will test tomorrow, because the network here is extremely slow at
> the moment, don't know what is wrong.
>
> regards,
>
> Wim
>
> 2005/11/21, Emmanuel Venisse <emmanuel@venisse.net
> <ma...@venisse.net>>:
>
> it's deploy now.
>
> Emmanuel
>
> Wim Deblauwe a écrit :
>> I'm trying to build maven-scm-provider-clearcase, but I get the
>> following error:
>>
>> [INFO] Failed to resolve artifact.
>>
>> GroupId: org.apache.maven.scm
>> ArtifactId: maven-scm-test
>> Version: 1.0-beta-2-SNAPSHOT
>>
>> Reason: Unable to download the artifact from any repository
>> org.apache.maven.scm:maven-scm-test:1.0-beta-2-SNAPSHOT:jar
>>
>> from the specified remote repositories:
>> central (http://repo1.maven.org/maven2 ),
>> snapshots ( http://snapshots.maven.codehaus.org/maven2)
>>
>> On the release plugin: I will need to get it from svn and build it
>> myself I suppose? Or did you deploy it somewhere?
>>
>> regards,
>>
>> Wim
>>
>> 2005/11/21, Emmanuel Venisse <emmanuel@venisse.net
> <ma...@venisse.net>
>> <mailto: emmanuel@venisse.net <ma...@venisse.net>>>:
>>
>> I done changes.
>> You'll need to use -DuseEditMode=true with release:prepare
>>
>> Emmanuel
>>
>> Wim Deblauwe a écrit :
>> > Well, we discussed this already I thought. Before the
> release plugin
>> > changes the pom.xml, it needs to do a "edit" on the
> pom.xml. If you
>> > could change that, I'm very willing to test it out.
>> >
>> > regards,
>> >
>> > Wim
>> >
>> > 2005/11/21, Emmanuel Venisse < emmanuel@venisse.net
> <ma...@venisse.net>
>> <mailto:emmanuel@venisse.net <ma...@venisse.net>>
>> > <mailto:emmanuel@venisse.net
> <ma...@venisse.net> <mailto: emmanuel@venisse.net
> <ma...@venisse.net>>>>:
>> >
>> > I can do it, but i need to know the process for
> clearcase. It
>> will
>> > be better if you can update it
>> > yourself.
>> >
>> > Emmanuel
>> >
>> > Wim Deblauwe a écrit :
>> > > Hey,
>> > >
>> > > I noticed the patch has been applied, thank you
> for that.
>> What is
>> > next?
>> > > Do we have to make a request to update the
> release plugin
>> or are you
>> > > willing to update that?
>> > >
>> > > regards,
>> > >
>> > > Wim
>> > >
>> > > 2005/11/21, Wim Deblauwe < wim.deblauwe@gmail.com
> <ma...@gmail.com>
>> <mailto: wim.deblauwe@gmail.com
> <ma...@gmail.com>>
>> > <mailto:wim.deblauwe@gmail.com
> <ma...@gmail.com> <mailto: wim.deblauwe@gmail.com
> <ma...@gmail.com>> >
>> > > <mailto:wim.deblauwe@gmail.com
> <ma...@gmail.com>
>> <mailto: wim.deblauwe@gmail.com
> <ma...@gmail.com>> <mailto: wim.deblauwe@gmail.com
> <ma...@gmail.com>
>> <mailto:wim.deblauwe@gmail.com
> <ma...@gmail.com>>>>>:
>> > >
>> > > Allright, here we go again. I have updated
> SCM-77 with
>> a new
>> > patch.
>> > > Hopefully this one will satisfy all your needs :)
>> > >
>> > > if there are any things I should change,
> please let me
>> know.
>> > >
>> > > For the edit and unedit commands I have used the
>> > > ScmFileStatus.UNKNOWN, because I don't know
> what I
>> should use
>> > there.
>> > > Maybe you guys can put something better in there.
>> > >
>> > > regards,
>> > >
>> > > Wim
>> > >
>> > > 2005/11/19, Emmanuel Venisse
> <emmanuel@venisse.net <ma...@venisse.net>
>> <mailto: emmanuel@venisse.net <ma...@venisse.net>>
>> > <mailto:emmanuel@venisse.net
> <ma...@venisse.net> <mailto: emmanuel@venisse.net
> <ma...@venisse.net>>>
>> > > <mailto:emmanuel@venisse.net
> <ma...@venisse.net>
>> <mailto: emmanuel@venisse.net
> <ma...@venisse.net>> <mailto:emmanuel@venisse.net
> <ma...@venisse.net>
>> <mailto:emmanuel@venisse.net <ma...@venisse.net>>>>>:
>> > >
>> > >
>> > >
>> > > Wim Deblauwe a écrit :
>> > >> Hey Emmanuel,
>> > >>
>> > >> 1) I'm using TortoiseSVN to generate the patch.
> Maybe I did
>> > > something
>> > >> wrong, I never worked with it before.
>> > >
>> > > Strange, it works fine normally.
>> > >
>> > >>
>> > >> 2) I sure can. Normally I just let IntelliJ
> IDEA do my
>> > > formatting. Don't
>> > >> know if you have the correct settings?
> Otherwise, i'll just
>> > > fix the tabs.
>> > >
>> > >
>> >
>>
> http://maven.apache.org/guides/development/guide-m2-development.html#Maven
> <http://maven.apache.org/guides/development/guide-m2-development.html#Maven>
>> > > Code Style
>> > >
>> > >>
>> > >> 3) Well, because I was unable to test and also
> I could
>> not get the
>> > >> logging to work (see one of my previous emails)
>> > >
>> > > clearcase commands print something to
> console, so, you
>> > can copy
>> > > this output to text files and parse
>> > > them with consumers. You don't need
> logger for tests,
>> > only some
>> > > junit assert that verify your
>> > > consumer is correct.
>> > >
>> > >>
>> > >> 4) I haven't made testclasses yet. Should they
> really do
>> clearcase
>> > >> operations or check if the command line is like it
>> should be?
>> > >
>> > > command line only and test consumers.
>> > >
>> > >>
>> > >> I'm out of office now, so I will not be able to
> do these
>> > > things until
>> > >> monday.
>> > >>
>> > >> regards,
>> > >>
>> > >> Wim
>> > >>
>> > >> 2005/11/18, Emmanuel Venisse <
> emmanuel@venisse.net <ma...@venisse.net>
>> <mailto: emmanuel@venisse.net <ma...@venisse.net>>
>> > <mailto:emmanuel@venisse.net
> <ma...@venisse.net> <mailto: emmanuel@venisse.net
> <ma...@venisse.net>>>
>> > > <mailto:emmanuel@venisse.net
> <ma...@venisse.net>
>> <mailto: emmanuel@venisse.net
> <ma...@venisse.net>> <mailto:emmanuel@venisse.net
> <ma...@venisse.net>
>> <mailto:emmanuel@venisse.net <ma...@venisse.net>>>>
>> > >> <mailto: emmanuel@venisse.net
> <ma...@venisse.net>
>> <mailto: emmanuel@venisse.net
> <ma...@venisse.net>> <mailto:emmanuel@venisse.net
> <ma...@venisse.net>
>> <mailto:emmanuel@venisse.net <ma...@venisse.net>>>
>> > <mailto:emmanuel@venisse.net
> <ma...@venisse.net> <mailto: emmanuel@venisse.net
> <ma...@venisse.net>>
>> <mailto: emmanuel@venisse.net <ma...@venisse.net>
> <mailto:emmanuel@venisse.net <ma...@venisse.net>>>>>>:
>> > >>
>> > >> Some comments about your patch :
>> > >> - How do you have generated it? some files are
>> present more
>> > > than one
>> > >> time.
>> > >> - Can you replace all tab character with 4
> spaces?
>> > >> - Why your consumers do nothing?
>> > >> - Where are test classes?
>> > >>
>> > >> if you can recreate your patch and integrate
> all those
>> > > remarks, it
>> > >> will be more simple to read,
>> > >> apply and maintain.
>> > >>
>> > >> Emmanuel
>> > >>
>> > >> Wim Deblauwe a écrit :
>> > >> > Done:
> http://jira.codehaus.org/browse/MNG-1622
>> > > < http://jira.codehaus.org/browse/MNG-1622>
>> > >> >
>> > >> > I did not modify the release-plugin.
>> > >> >
>> > >> > regards,
>> > >> >
>> > >> > Wim
>> > >> >
>> > >> > 2005/11/18, Emmanuel Venisse <
>> emmanuel@venisse.net <ma...@venisse.net> <mailto:
> emmanuel@venisse.net <ma...@venisse.net>>
>> > <mailto:emmanuel@venisse.net
> <ma...@venisse.net> <mailto: emmanuel@venisse.net
> <ma...@venisse.net>>>
>> > > <mailto: emmanuel@venisse.net
> <ma...@venisse.net>
>> <mailto: emmanuel@venisse.net
> <ma...@venisse.net>> <mailto:emmanuel@venisse.net
> <ma...@venisse.net>
>> <mailto:emmanuel@venisse.net <ma...@venisse.net>>>>
>> > >> <mailto:emmanuel@venisse.net
> <ma...@venisse.net>
>> <mailto: emmanuel@venisse.net
> <ma...@venisse.net>> <mailto:emmanuel@venisse.net
> <ma...@venisse.net>
>> <mailto:emmanuel@venisse.net <ma...@venisse.net>>>
>> > <mailto:emmanuel@venisse.net
> <ma...@venisse.net> <mailto: emmanuel@venisse.net
> <ma...@venisse.net>>
>> <mailto:emmanuel@venisse.net <ma...@venisse.net>
> <mailto:emmanuel@venisse.net <ma...@venisse.net>>>>>
>> > >> > <mailto: emmanuel@venisse.net
> <ma...@venisse.net>
>> <mailto: emmanuel@venisse.net
> <ma...@venisse.net>> <mailto:emmanuel@venisse.net
> <ma...@venisse.net>
>> <mailto:emmanuel@venisse.net <ma...@venisse.net>>>
>> > > <mailto: emmanuel@venisse.net
> <ma...@venisse.net>
>> <mailto: emmanuel@venisse.net <ma...@venisse.net>>
>> > <mailto:emmanuel@venisse.net
> <ma...@venisse.net> <mailto:emmanuel@venisse.net
> <ma...@venisse.net>>>>
>> <mailto: emmanuel@venisse.net <ma...@venisse.net>
> <mailto: emmanuel@venisse.net <ma...@venisse.net>>
>> > <mailto: emmanuel@venisse.net
> <ma...@venisse.net> <mailto: emmanuel@venisse.net
> <ma...@venisse.net>>>
>> > > <mailto:emmanuel@venisse.net
> <ma...@venisse.net>
>> <mailto: emmanuel@venisse.net <ma...@venisse.net>>
>> > <mailto:emmanuel@venisse.net
> <ma...@venisse.net> <mailto:emmanuel@venisse.net
> <ma...@venisse.net>>>>>>>:
>> > >> >
>> > >> > Great.
>> > >> >
>> > >> > you can create a patch for maven-scm and
>> attach to a
>> > > jira issue.
>> > >> > I'll review it (and perhaps Dan too).
>> > >> >
>> > >> > Do you have modify release-plugin?
>> > >> >
>> > >> > Wim Deblauwe a écrit :
>> > >> > > Hi guys,
>> > >> > >
>> > >> > > I've implemented add, checkin,
> edit, remove,
>> > > status and
>> > >> tag for
>> > >> > > clearcase. Is somebody willing to
> update
>> the release
>> > >> plugin so I can
>> > >> > > test? Do I need to create a patch
> with all
>> my current
>> > >> changes for
>> > >> > > someone to update the release plugin?
>> > >> > >
>> > >> > > regards,
>> > >> > >
>> > >> > > Wim
>> > >> >
>> > >> >
>> > >>
>> > >>
>> > >
>> > >
>> > >
>> >
>> >
>>
>>
>
>
>
Re: release plugin update for clearcase
Posted by Wim Deblauwe <wi...@gmail.com>.
On what server did you deploy? I tried today, but I did not get a new
version. I got this in my pom.xml:
<build>
<plugins>
<plugin>
<artifactId>maven-release-plugin</artifactId>
<dependencies>
<dependency>
<groupId>org.apache.maven.scm</groupId>
<artifactId>maven-scm-provider-clearcase</artifactId>
<version>1.0-beta-1-SNAPSHOT</version>
</dependency>
</dependencies>
</plugin>
</plugins>
</build>
maybe I need to specify a version for the release plugin?
regards,
Wim
2005/11/21, Wim Deblauwe <wi...@gmail.com>:
>
> I will test tomorrow, because the network here is extremely slow at the
> moment, don't know what is wrong.
>
> regards,
>
> Wim
>
> 2005/11/21, Emmanuel Venisse <em...@venisse.net>:
> >
> > it's deploy now.
> >
> > Emmanuel
> >
> > Wim Deblauwe a écrit :
> > > I'm trying to build maven-scm-provider-clearcase, but I get the
> > > following error:
> > >
> > > [INFO] Failed to resolve artifact.
> > >
> > > GroupId: org.apache.maven.scm
> > > ArtifactId: maven-scm-test
> > > Version: 1.0-beta-2-SNAPSHOT
> > >
> > > Reason: Unable to download the artifact from any repository
> > > org.apache.maven.scm:maven-scm-test:1.0-beta-2-SNAPSHOT:jar
> > >
> > > from the specified remote repositories:
> > > central (http://repo1.maven.org/maven2),
> > > snapshots ( http://snapshots.maven.codehaus.org/maven2)
> > >
> > > On the release plugin: I will need to get it from svn and build it
> > > myself I suppose? Or did you deploy it somewhere?
> > >
> > > regards,
> > >
> > > Wim
> > >
> > > 2005/11/21, Emmanuel Venisse <emmanuel@venisse.net
> > > <ma...@venisse.net>>:
> > >
> > > I done changes.
> > > You'll need to use -DuseEditMode=true with release:prepare
> > >
> > > Emmanuel
> > >
> > > Wim Deblauwe a écrit :
> > > > Well, we discussed this already I thought. Before the release plugin
> >
> > > > changes the pom.xml, it needs to do a "edit" on the pom.xml. If you
> > > > could change that, I'm very willing to test it out.
> > > >
> > > > regards,
> > > >
> > > > Wim
> > > >
> > > > 2005/11/21, Emmanuel Venisse < emmanuel@venisse.net
> > > <mailto:emmanuel@venisse.net >
> > > > <mailto:emmanuel@venisse.net <ma...@venisse.net>>>:
> > > >
> > > > I can do it, but i need to know the process for clearcase. It
> > > will
> > > > be better if you can update it
> > > > yourself.
> > > >
> > > > Emmanuel
> > > >
> > > > Wim Deblauwe a écrit :
> > > > > Hey,
> > > > >
> > > > > I noticed the patch has been applied, thank you for that.
> > > What is
> > > > next?
> > > > > Do we have to make a request to update the release plugin
> > > or are you
> > > > > willing to update that?
> > > > >
> > > > > regards,
> > > > >
> > > > > Wim
> > > > >
> > > > > 2005/11/21, Wim Deblauwe < wim.deblauwe@gmail.com
> > > <ma...@gmail.com>
> > > > <mailto:wim.deblauwe@gmail.com <ma...@gmail.com> >
> > > > > <mailto:wim.deblauwe@gmail.com
> > > <ma...@gmail.com> <mailto: wim.deblauwe@gmail.com
> > > <ma...@gmail.com>>>>:
> > > > >
> > > > > Allright, here we go again. I have updated SCM-77 with
> > > a new
> > > > patch.
> > > > > Hopefully this one will satisfy all your needs :)
> > > > >
> > > > > if there are any things I should change, please let me
> > > know.
> > > > >
> > > > > For the edit and unedit commands I have used the
> > > > > ScmFileStatus.UNKNOWN, because I don't know what I
> > > should use
> > > > there.
> > > > > Maybe you guys can put something better in there.
> > > > >
> > > > > regards,
> > > > >
> > > > > Wim
> > > > >
> > > > > 2005/11/19, Emmanuel Venisse <emmanuel@venisse.net
> > > <ma...@venisse.net>
> > > > <mailto:emmanuel@venisse.net <ma...@venisse.net>>
> > > > > <mailto:emmanuel@venisse.net
> > > <ma...@venisse.net> <mailto:emmanuel@venisse.net
> > > <ma...@venisse.net>>>>:
> > > > >
> > > > >
> > > > >
> > > > > Wim Deblauwe a écrit :
> > > > >> Hey Emmanuel,
> > > > >>
> > > > >> 1) I'm using TortoiseSVN to generate the patch. Maybe I did
> > > > > something
> > > > >> wrong, I never worked with it before.
> > > > >
> > > > > Strange, it works fine normally.
> > > > >
> > > > >>
> > > > >> 2) I sure can. Normally I just let IntelliJ IDEA do my
> > > > > formatting. Don't
> > > > >> know if you have the correct settings? Otherwise, i'll just
> > > > > fix the tabs.
> > > > >
> > > > >
> > > >
> > > http://maven.apache.org/guides/development/guide-m2-development.html#Maven
> >
> > > > > Code Style
> > > > >
> > > > >>
> > > > >> 3) Well, because I was unable to test and also I could
> > > not get the
> > > > >> logging to work (see one of my previous emails)
> > > > >
> > > > > clearcase commands print something to console, so, you
> > > > can copy
> > > > > this output to text files and parse
> > > > > them with consumers. You don't need logger for tests,
> > > > only some
> > > > > junit assert that verify your
> > > > > consumer is correct.
> > > > >
> > > > >>
> > > > >> 4) I haven't made testclasses yet. Should they really do
> > > clearcase
> > > > >> operations or check if the command line is like it
> > > should be?
> > > > >
> > > > > command line only and test consumers.
> > > > >
> > > > >>
> > > > >> I'm out of office now, so I will not be able to do these
> > > > > things until
> > > > >> monday.
> > > > >>
> > > > >> regards,
> > > > >>
> > > > >> Wim
> > > > >>
> > > > >> 2005/11/18, Emmanuel Venisse < emmanuel@venisse.net
> > > <ma...@venisse.net>
> > > > <mailto:emmanuel@venisse.net <ma...@venisse.net>>
> > > > > <mailto:emmanuel@venisse.net
> > > <ma...@venisse.net> <mailto:emmanuel@venisse.net
> > > <ma...@venisse.net>>>
> > > > >> <mailto: emmanuel@venisse.net
> > > <ma...@venisse.net> <mailto:emmanuel@venisse.net
> > > <ma...@venisse.net>>
> > > > <mailto:emmanuel@venisse.net <ma...@venisse.net>
> > > <mailto: emmanuel@venisse.net <ma...@venisse.net>>>>>:
> > > > >>
> > > > >> Some comments about your patch :
> > > > >> - How do you have generated it? some files are
> > > present more
> > > > > than one
> > > > >> time.
> > > > >> - Can you replace all tab character with 4 spaces?
> > > > >> - Why your consumers do nothing?
> > > > >> - Where are test classes?
> > > > >>
> > > > >> if you can recreate your patch and integrate all those
> > > > > remarks, it
> > > > >> will be more simple to read,
> > > > >> apply and maintain.
> > > > >>
> > > > >> Emmanuel
> > > > >>
> > > > >> Wim Deblauwe a écrit :
> > > > >> > Done: http://jira.codehaus.org/browse/MNG-1622
> > > > > < http://jira.codehaus.org/browse/MNG-1622>
> > > > >> >
> > > > >> > I did not modify the release-plugin.
> > > > >> >
> > > > >> > regards,
> > > > >> >
> > > > >> > Wim
> > > > >> >
> > > > >> > 2005/11/18, Emmanuel Venisse <
> > > emmanuel@venisse.net <ma...@venisse.net>
> > > > <mailto:emmanuel@venisse.net <ma...@venisse.net>>
> > > > > <mailto: emmanuel@venisse.net
> > > <ma...@venisse.net> <mailto:emmanuel@venisse.net
> > > <ma...@venisse.net>>>
> > > > >> <mailto:emmanuel@venisse.net
> > > <ma...@venisse.net> <mailto:emmanuel@venisse.net
> > > <ma...@venisse.net>>
> > > > <mailto:emmanuel@venisse.net <ma...@venisse.net>
> > > <mailto:emmanuel@venisse.net <ma...@venisse.net>>>>
> > > > >> > <mailto: emmanuel@venisse.net
> > > <ma...@venisse.net> <mailto:emmanuel@venisse.net
> > > <ma...@venisse.net>>
> > > > > <mailto: emmanuel@venisse.net
> > > <ma...@venisse.net>
> > > > <mailto:emmanuel@venisse.net <ma...@venisse.net>>>
> > > <mailto: emmanuel@venisse.net <mailto: emmanuel@venisse.net>
> > > > <mailto: emmanuel@venisse.net <ma...@venisse.net>>
> > > > > <mailto:emmanuel@venisse.net
> > > <ma...@venisse.net>
> > > > <mailto:emmanuel@venisse.net <ma...@venisse.net>>>>>>:
> > > > >> >
> > > > >> > Great.
> > > > >> >
> > > > >> > you can create a patch for maven-scm and
> > > attach to a
> > > > > jira issue.
> > > > >> > I'll review it (and perhaps Dan too).
> > > > >> >
> > > > >> > Do you have modify release-plugin?
> > > > >> >
> > > > >> > Wim Deblauwe a écrit :
> > > > >> > > Hi guys,
> > > > >> > >
> > > > >> > > I've implemented add, checkin, edit, remove,
> > > > > status and
> > > > >> tag for
> > > > >> > > clearcase. Is somebody willing to update
> > > the release
> > > > >> plugin so I can
> > > > >> > > test? Do I need to create a patch with all
> > > my current
> > > > >> changes for
> > > > >> > > someone to update the release plugin?
> > > > >> > >
> > > > >> > > regards,
> > > > >> > >
> > > > >> > > Wim
> > > > >> >
> > > > >> >
> > > > >>
> > > > >>
> > > > >
> > > > >
> > > > >
> > > >
> > > >
> > >
> > >
> >
> >
>
Re: release plugin update for clearcase
Posted by Wim Deblauwe <wi...@gmail.com>.
I will test tomorrow, because the network here is extremely slow at the
moment, don't know what is wrong.
regards,
Wim
2005/11/21, Emmanuel Venisse <em...@venisse.net>:
>
> it's deploy now.
>
> Emmanuel
>
> Wim Deblauwe a écrit :
> > I'm trying to build maven-scm-provider-clearcase, but I get the
> > following error:
> >
> > [INFO] Failed to resolve artifact.
> >
> > GroupId: org.apache.maven.scm
> > ArtifactId: maven-scm-test
> > Version: 1.0-beta-2-SNAPSHOT
> >
> > Reason: Unable to download the artifact from any repository
> > org.apache.maven.scm:maven-scm-test:1.0-beta-2-SNAPSHOT:jar
> >
> > from the specified remote repositories:
> > central (http://repo1.maven.org/maven2),
> > snapshots (http://snapshots.maven.codehaus.org/maven2)
> >
> > On the release plugin: I will need to get it from svn and build it
> > myself I suppose? Or did you deploy it somewhere?
> >
> > regards,
> >
> > Wim
> >
> > 2005/11/21, Emmanuel Venisse <emmanuel@venisse.net
> > <ma...@venisse.net>>:
> >
> > I done changes.
> > You'll need to use -DuseEditMode=true with release:prepare
> >
> > Emmanuel
> >
> > Wim Deblauwe a écrit :
> > > Well, we discussed this already I thought. Before the release plugin
> > > changes the pom.xml, it needs to do a "edit" on the pom.xml. If you
> > > could change that, I'm very willing to test it out.
> > >
> > > regards,
> > >
> > > Wim
> > >
> > > 2005/11/21, Emmanuel Venisse < emmanuel@venisse.net
> > <ma...@venisse.net>
> > > <mailto:emmanuel@venisse.net <ma...@venisse.net>>>:
> > >
> > > I can do it, but i need to know the process for clearcase. It
> > will
> > > be better if you can update it
> > > yourself.
> > >
> > > Emmanuel
> > >
> > > Wim Deblauwe a écrit :
> > > > Hey,
> > > >
> > > > I noticed the patch has been applied, thank you for that.
> > What is
> > > next?
> > > > Do we have to make a request to update the release plugin
> > or are you
> > > > willing to update that?
> > > >
> > > > regards,
> > > >
> > > > Wim
> > > >
> > > > 2005/11/21, Wim Deblauwe < wim.deblauwe@gmail.com
> > <ma...@gmail.com>
> > > <mailto:wim.deblauwe@gmail.com <ma...@gmail.com> >
> > > > <mailto:wim.deblauwe@gmail.com
> > <ma...@gmail.com> <mailto:wim.deblauwe@gmail.com
> > <ma...@gmail.com>>>>:
> > > >
> > > > Allright, here we go again. I have updated SCM-77 with
> > a new
> > > patch.
> > > > Hopefully this one will satisfy all your needs :)
> > > >
> > > > if there are any things I should change, please let me
> > know.
> > > >
> > > > For the edit and unedit commands I have used the
> > > > ScmFileStatus.UNKNOWN, because I don't know what I
> > should use
> > > there.
> > > > Maybe you guys can put something better in there.
> > > >
> > > > regards,
> > > >
> > > > Wim
> > > >
> > > > 2005/11/19, Emmanuel Venisse <emmanuel@venisse.net
> > <ma...@venisse.net>
> > > <mailto:emmanuel@venisse.net <ma...@venisse.net>>
> > > > <mailto:emmanuel@venisse.net
> > <ma...@venisse.net> <mailto:emmanuel@venisse.net
> > <ma...@venisse.net>>>>:
> > > >
> > > >
> > > >
> > > > Wim Deblauwe a écrit :
> > > >> Hey Emmanuel,
> > > >>
> > > >> 1) I'm using TortoiseSVN to generate the patch. Maybe I did
> > > > something
> > > >> wrong, I never worked with it before.
> > > >
> > > > Strange, it works fine normally.
> > > >
> > > >>
> > > >> 2) I sure can. Normally I just let IntelliJ IDEA do my
> > > > formatting. Don't
> > > >> know if you have the correct settings? Otherwise, i'll just
> > > > fix the tabs.
> > > >
> > > >
> > >
> >
> http://maven.apache.org/guides/development/guide-m2-development.html#Maven
> > > > Code Style
> > > >
> > > >>
> > > >> 3) Well, because I was unable to test and also I could
> > not get the
> > > >> logging to work (see one of my previous emails)
> > > >
> > > > clearcase commands print something to console, so, you
> > > can copy
> > > > this output to text files and parse
> > > > them with consumers. You don't need logger for tests,
> > > only some
> > > > junit assert that verify your
> > > > consumer is correct.
> > > >
> > > >>
> > > >> 4) I haven't made testclasses yet. Should they really do
> > clearcase
> > > >> operations or check if the command line is like it
> > should be?
> > > >
> > > > command line only and test consumers.
> > > >
> > > >>
> > > >> I'm out of office now, so I will not be able to do these
> > > > things until
> > > >> monday.
> > > >>
> > > >> regards,
> > > >>
> > > >> Wim
> > > >>
> > > >> 2005/11/18, Emmanuel Venisse < emmanuel@venisse.net
> > <ma...@venisse.net>
> > > <mailto:emmanuel@venisse.net <ma...@venisse.net>>
> > > > <mailto:emmanuel@venisse.net
> > <ma...@venisse.net> <mailto:emmanuel@venisse.net
> > <ma...@venisse.net>>>
> > > >> <mailto: emmanuel@venisse.net
> > <ma...@venisse.net> <mailto:emmanuel@venisse.net
> > <ma...@venisse.net>>
> > > <mailto:emmanuel@venisse.net <ma...@venisse.net>
> > <mailto: emmanuel@venisse.net <ma...@venisse.net>>>>>:
> > > >>
> > > >> Some comments about your patch :
> > > >> - How do you have generated it? some files are
> > present more
> > > > than one
> > > >> time.
> > > >> - Can you replace all tab character with 4 spaces?
> > > >> - Why your consumers do nothing?
> > > >> - Where are test classes?
> > > >>
> > > >> if you can recreate your patch and integrate all those
> > > > remarks, it
> > > >> will be more simple to read,
> > > >> apply and maintain.
> > > >>
> > > >> Emmanuel
> > > >>
> > > >> Wim Deblauwe a écrit :
> > > >> > Done: http://jira.codehaus.org/browse/MNG-1622
> > > > < http://jira.codehaus.org/browse/MNG-1622>
> > > >> >
> > > >> > I did not modify the release-plugin.
> > > >> >
> > > >> > regards,
> > > >> >
> > > >> > Wim
> > > >> >
> > > >> > 2005/11/18, Emmanuel Venisse <
> > emmanuel@venisse.net <ma...@venisse.net>
> > > <mailto:emmanuel@venisse.net <ma...@venisse.net>>
> > > > <mailto: emmanuel@venisse.net
> > <ma...@venisse.net> <mailto:emmanuel@venisse.net
> > <ma...@venisse.net>>>
> > > >> <mailto:emmanuel@venisse.net
> > <ma...@venisse.net> <mailto:emmanuel@venisse.net
> > <ma...@venisse.net>>
> > > <mailto:emmanuel@venisse.net <ma...@venisse.net>
> > <mailto:emmanuel@venisse.net <ma...@venisse.net>>>>
> > > >> > <mailto: emmanuel@venisse.net
> > <ma...@venisse.net> <mailto:emmanuel@venisse.net
> > <ma...@venisse.net>>
> > > > <mailto: emmanuel@venisse.net
> > <ma...@venisse.net>
> > > <mailto:emmanuel@venisse.net <ma...@venisse.net>>>
> > <mailto: emmanuel@venisse.net <ma...@venisse.net>
> > > <mailto: emmanuel@venisse.net <ma...@venisse.net>>
> > > > <mailto:emmanuel@venisse.net
> > <ma...@venisse.net>
> > > <mailto:emmanuel@venisse.net <ma...@venisse.net>>>>>>:
> > > >> >
> > > >> > Great.
> > > >> >
> > > >> > you can create a patch for maven-scm and
> > attach to a
> > > > jira issue.
> > > >> > I'll review it (and perhaps Dan too).
> > > >> >
> > > >> > Do you have modify release-plugin?
> > > >> >
> > > >> > Wim Deblauwe a écrit :
> > > >> > > Hi guys,
> > > >> > >
> > > >> > > I've implemented add, checkin, edit, remove,
> > > > status and
> > > >> tag for
> > > >> > > clearcase. Is somebody willing to update
> > the release
> > > >> plugin so I can
> > > >> > > test? Do I need to create a patch with all
> > my current
> > > >> changes for
> > > >> > > someone to update the release plugin?
> > > >> > >
> > > >> > > regards,
> > > >> > >
> > > >> > > Wim
> > > >> >
> > > >> >
> > > >>
> > > >>
> > > >
> > > >
> > > >
> > >
> > >
> >
> >
>
>
Re: release plugin update for clearcase
Posted by Emmanuel Venisse <em...@venisse.net>.
it's deploy now.
Emmanuel
Wim Deblauwe a écrit :
> I'm trying to build maven-scm-provider-clearcase, but I get the
> following error:
>
> [INFO] Failed to resolve artifact.
>
> GroupId: org.apache.maven.scm
> ArtifactId: maven-scm-test
> Version: 1.0-beta-2-SNAPSHOT
>
> Reason: Unable to download the artifact from any repository
> org.apache.maven.scm:maven-scm-test:1.0-beta-2-SNAPSHOT:jar
>
> from the specified remote repositories:
> central (http://repo1.maven.org/maven2),
> snapshots (http://snapshots.maven.codehaus.org/maven2)
>
> On the release plugin: I will need to get it from svn and build it
> myself I suppose? Or did you deploy it somewhere?
>
> regards,
>
> Wim
>
> 2005/11/21, Emmanuel Venisse <emmanuel@venisse.net
> <ma...@venisse.net>>:
>
> I done changes.
> You'll need to use -DuseEditMode=true with release:prepare
>
> Emmanuel
>
> Wim Deblauwe a écrit :
> > Well, we discussed this already I thought. Before the release plugin
> > changes the pom.xml, it needs to do a "edit" on the pom.xml. If you
> > could change that, I'm very willing to test it out.
> >
> > regards,
> >
> > Wim
> >
> > 2005/11/21, Emmanuel Venisse < emmanuel@venisse.net
> <ma...@venisse.net>
> > <mailto:emmanuel@venisse.net <ma...@venisse.net>>>:
> >
> > I can do it, but i need to know the process for clearcase. It
> will
> > be better if you can update it
> > yourself.
> >
> > Emmanuel
> >
> > Wim Deblauwe a écrit :
> > > Hey,
> > >
> > > I noticed the patch has been applied, thank you for that.
> What is
> > next?
> > > Do we have to make a request to update the release plugin
> or are you
> > > willing to update that?
> > >
> > > regards,
> > >
> > > Wim
> > >
> > > 2005/11/21, Wim Deblauwe < wim.deblauwe@gmail.com
> <ma...@gmail.com>
> > <mailto:wim.deblauwe@gmail.com <ma...@gmail.com> >
> > > <mailto:wim.deblauwe@gmail.com
> <ma...@gmail.com> <mailto:wim.deblauwe@gmail.com
> <ma...@gmail.com>>>>:
> > >
> > > Allright, here we go again. I have updated SCM-77 with
> a new
> > patch.
> > > Hopefully this one will satisfy all your needs :)
> > >
> > > if there are any things I should change, please let me
> know.
> > >
> > > For the edit and unedit commands I have used the
> > > ScmFileStatus.UNKNOWN, because I don't know what I
> should use
> > there.
> > > Maybe you guys can put something better in there.
> > >
> > > regards,
> > >
> > > Wim
> > >
> > > 2005/11/19, Emmanuel Venisse <emmanuel@venisse.net
> <ma...@venisse.net>
> > <mailto:emmanuel@venisse.net <ma...@venisse.net>>
> > > <mailto:emmanuel@venisse.net
> <ma...@venisse.net> <mailto:emmanuel@venisse.net
> <ma...@venisse.net>>>>:
> > >
> > >
> > >
> > > Wim Deblauwe a écrit :
> > >> Hey Emmanuel,
> > >>
> > >> 1) I'm using TortoiseSVN to generate the patch. Maybe I did
> > > something
> > >> wrong, I never worked with it before.
> > >
> > > Strange, it works fine normally.
> > >
> > >>
> > >> 2) I sure can. Normally I just let IntelliJ IDEA do my
> > > formatting. Don't
> > >> know if you have the correct settings? Otherwise, i'll just
> > > fix the tabs.
> > >
> > >
> >
> http://maven.apache.org/guides/development/guide-m2-development.html#Maven
> > > Code Style
> > >
> > >>
> > >> 3) Well, because I was unable to test and also I could
> not get the
> > >> logging to work (see one of my previous emails)
> > >
> > > clearcase commands print something to console, so, you
> > can copy
> > > this output to text files and parse
> > > them with consumers. You don't need logger for tests,
> > only some
> > > junit assert that verify your
> > > consumer is correct.
> > >
> > >>
> > >> 4) I haven't made testclasses yet. Should they really do
> clearcase
> > >> operations or check if the command line is like it
> should be?
> > >
> > > command line only and test consumers.
> > >
> > >>
> > >> I'm out of office now, so I will not be able to do these
> > > things until
> > >> monday.
> > >>
> > >> regards,
> > >>
> > >> Wim
> > >>
> > >> 2005/11/18, Emmanuel Venisse < emmanuel@venisse.net
> <ma...@venisse.net>
> > <mailto:emmanuel@venisse.net <ma...@venisse.net>>
> > > <mailto:emmanuel@venisse.net
> <ma...@venisse.net> <mailto:emmanuel@venisse.net
> <ma...@venisse.net>>>
> > >> <mailto: emmanuel@venisse.net
> <ma...@venisse.net> <mailto:emmanuel@venisse.net
> <ma...@venisse.net>>
> > <mailto:emmanuel@venisse.net <ma...@venisse.net>
> <mailto: emmanuel@venisse.net <ma...@venisse.net>>>>>:
> > >>
> > >> Some comments about your patch :
> > >> - How do you have generated it? some files are
> present more
> > > than one
> > >> time.
> > >> - Can you replace all tab character with 4 spaces?
> > >> - Why your consumers do nothing?
> > >> - Where are test classes?
> > >>
> > >> if you can recreate your patch and integrate all those
> > > remarks, it
> > >> will be more simple to read,
> > >> apply and maintain.
> > >>
> > >> Emmanuel
> > >>
> > >> Wim Deblauwe a écrit :
> > >> > Done: http://jira.codehaus.org/browse/MNG-1622
> > > < http://jira.codehaus.org/browse/MNG-1622>
> > >> >
> > >> > I did not modify the release-plugin.
> > >> >
> > >> > regards,
> > >> >
> > >> > Wim
> > >> >
> > >> > 2005/11/18, Emmanuel Venisse <
> emmanuel@venisse.net <ma...@venisse.net>
> > <mailto:emmanuel@venisse.net <ma...@venisse.net>>
> > > <mailto: emmanuel@venisse.net
> <ma...@venisse.net> <mailto:emmanuel@venisse.net
> <ma...@venisse.net>>>
> > >> <mailto:emmanuel@venisse.net
> <ma...@venisse.net> <mailto:emmanuel@venisse.net
> <ma...@venisse.net>>
> > <mailto:emmanuel@venisse.net <ma...@venisse.net>
> <mailto:emmanuel@venisse.net <ma...@venisse.net>>>>
> > >> > <mailto: emmanuel@venisse.net
> <ma...@venisse.net> <mailto:emmanuel@venisse.net
> <ma...@venisse.net>>
> > > <mailto: emmanuel@venisse.net
> <ma...@venisse.net>
> > <mailto:emmanuel@venisse.net <ma...@venisse.net>>>
> <mailto: emmanuel@venisse.net <ma...@venisse.net>
> > <mailto: emmanuel@venisse.net <ma...@venisse.net>>
> > > <mailto:emmanuel@venisse.net
> <ma...@venisse.net>
> > <mailto:emmanuel@venisse.net <ma...@venisse.net>>>>>>:
> > >> >
> > >> > Great.
> > >> >
> > >> > you can create a patch for maven-scm and
> attach to a
> > > jira issue.
> > >> > I'll review it (and perhaps Dan too).
> > >> >
> > >> > Do you have modify release-plugin?
> > >> >
> > >> > Wim Deblauwe a écrit :
> > >> > > Hi guys,
> > >> > >
> > >> > > I've implemented add, checkin, edit, remove,
> > > status and
> > >> tag for
> > >> > > clearcase. Is somebody willing to update
> the release
> > >> plugin so I can
> > >> > > test? Do I need to create a patch with all
> my current
> > >> changes for
> > >> > > someone to update the release plugin?
> > >> > >
> > >> > > regards,
> > >> > >
> > >> > > Wim
> > >> >
> > >> >
> > >>
> > >>
> > >
> > >
> > >
> >
> >
>
>
Re: release plugin update for clearcase
Posted by Wim Deblauwe <wi...@gmail.com>.
Never mind on the error, found the problem.
2005/11/21, Wim Deblauwe <wi...@gmail.com>:
>
> I'm trying to build maven-scm-provider-clearcase, but I get the following
> error:
>
> [INFO] Failed to resolve artifact.
>
> GroupId: org.apache.maven.scm
> ArtifactId: maven-scm-test
> Version: 1.0-beta-2-SNAPSHOT
>
> Reason: Unable to download the artifact from any repository
> org.apache.maven.scm:maven-scm-test:1.0-beta-2-SNAPSHOT:jar
>
> from the specified remote repositories:
> central (http://repo1.maven.org/maven2),
> snapshots (http://snapshots.maven.codehaus.org/maven2)
>
> On the release plugin: I will need to get it from svn and build it myself
> I suppose? Or did you deploy it somewhere?
>
> regards,
>
> Wim
>
> 2005/11/21, Emmanuel Venisse <em...@venisse.net>:
> >
> > I done changes.
> > You'll need to use -DuseEditMode=true with release:prepare
> >
> > Emmanuel
> >
> > Wim Deblauwe a écrit :
> > > Well, we discussed this already I thought. Before the release plugin
> > > changes the pom.xml, it needs to do a "edit" on the pom.xml. If you
> > > could change that, I'm very willing to test it out.
> > >
> > > regards,
> > >
> > > Wim
> > >
> > > 2005/11/21, Emmanuel Venisse < emmanuel@venisse.net
> > > <ma...@venisse.net>>:
> > >
> > > I can do it, but i need to know the process for clearcase. It will
> > > be better if you can update it
> > > yourself.
> > >
> > > Emmanuel
> > >
> > > Wim Deblauwe a écrit :
> > > > Hey,
> > > >
> > > > I noticed the patch has been applied, thank you for that. What is
> > > next?
> > > > Do we have to make a request to update the release plugin or are you
> > > > willing to update that?
> > > >
> > > > regards,
> > > >
> > > > Wim
> > > >
> > > > 2005/11/21, Wim Deblauwe < wim.deblauwe@gmail.com
> > > <mailto:wim.deblauwe@gmail.com >
> > > > <mailto:wim.deblauwe@gmail.com <ma...@gmail.com>>>:
> > > >
> > > > Allright, here we go again. I have updated SCM-77 with a new
> > > patch.
> > > > Hopefully this one will satisfy all your needs :)
> > > >
> > > > if there are any things I should change, please let me know.
> > > >
> > > > For the edit and unedit commands I have used the
> > > > ScmFileStatus.UNKNOWN, because I don't know what I should use
> > > there.
> > > > Maybe you guys can put something better in there.
> > > >
> > > > regards,
> > > >
> > > > Wim
> > > >
> > > > 2005/11/19, Emmanuel Venisse <emmanuel@venisse.net
> > > <mailto:emmanuel@venisse.net >
> > > > <mailto:emmanuel@venisse.net <ma...@venisse.net>>>:
> > > >
> > > >
> > > >
> > > > Wim Deblauwe a écrit :
> > > >> Hey Emmanuel,
> > > >>
> > > >> 1) I'm using TortoiseSVN to generate the patch. Maybe I did
> > > > something
> > > >> wrong, I never worked with it before.
> > > >
> > > > Strange, it works fine normally.
> > > >
> > > >>
> > > >> 2) I sure can. Normally I just let IntelliJ IDEA do my
> > > > formatting. Don't
> > > >> know if you have the correct settings? Otherwise, i'll just
> > > > fix the tabs.
> > > >
> > > >
> > >
> > http://maven.apache.org/guides/development/guide-m2-development.html#Maven
> > > > Code Style
> > > >
> > > >>
> > > >> 3) Well, because I was unable to test and also I could not get the
> > > >> logging to work (see one of my previous emails)
> > > >
> > > > clearcase commands print something to console, so, you
> > > can copy
> > > > this output to text files and parse
> > > > them with consumers. You don't need logger for tests,
> > > only some
> > > > junit assert that verify your
> > > > consumer is correct.
> > > >
> > > >>
> > > >> 4) I haven't made testclasses yet. Should they really do clearcase
> > > >> operations or check if the command line is like it should be?
> > > >
> > > > command line only and test consumers.
> > > >
> > > >>
> > > >> I'm out of office now, so I will not be able to do these
> > > > things until
> > > >> monday.
> > > >>
> > > >> regards,
> > > >>
> > > >> Wim
> > > >>
> > > >> 2005/11/18, Emmanuel Venisse < emmanuel@venisse.net
> > > <ma...@venisse.net>
> > > > <mailto:emmanuel@venisse.net <ma...@venisse.net>>
> > > >> <mailto: emmanuel@venisse.net <ma...@venisse.net>
> > > <mailto:emmanuel@venisse.net <mailto: emmanuel@venisse.net>>>>:
> > > >>
> > > >> Some comments about your patch :
> > > >> - How do you have generated it? some files are present more
> > > > than one
> > > >> time.
> > > >> - Can you replace all tab character with 4 spaces?
> > > >> - Why your consumers do nothing?
> > > >> - Where are test classes?
> > > >>
> > > >> if you can recreate your patch and integrate all those
> > > > remarks, it
> > > >> will be more simple to read,
> > > >> apply and maintain.
> > > >>
> > > >> Emmanuel
> > > >>
> > > >> Wim Deblauwe a écrit :
> > > >> > Done: http://jira.codehaus.org/browse/MNG-1622
> > > > < http://jira.codehaus.org/browse/MNG-1622>
> > > >> >
> > > >> > I did not modify the release-plugin.
> > > >> >
> > > >> > regards,
> > > >> >
> > > >> > Wim
> > > >> >
> > > >> > 2005/11/18, Emmanuel Venisse < emmanuel@venisse.net
> > > <ma...@venisse.net>
> > > > <mailto: emmanuel@venisse.net <ma...@venisse.net>>
> > > >> <mailto:emmanuel@venisse.net <ma...@venisse.net>
> > > <mailto:emmanuel@venisse.net <ma...@venisse.net>>>
> > > >> > <mailto: emmanuel@venisse.net <ma...@venisse.net>
> > > > <mailto: emmanuel@venisse.net
> > > <ma...@venisse.net>> <mailto: emmanuel@venisse.net
> > > <mailto: emmanuel@venisse.net>
> > > > <mailto:emmanuel@venisse.net
> > > <ma...@venisse.net>>>>>:
> > > >> >
> > > >> > Great.
> > > >> >
> > > >> > you can create a patch for maven-scm and attach to a
> > > > jira issue.
> > > >> > I'll review it (and perhaps Dan too).
> > > >> >
> > > >> > Do you have modify release-plugin?
> > > >> >
> > > >> > Wim Deblauwe a écrit :
> > > >> > > Hi guys,
> > > >> > >
> > > >> > > I've implemented add, checkin, edit, remove,
> > > > status and
> > > >> tag for
> > > >> > > clearcase. Is somebody willing to update the release
> > > >> plugin so I can
> > > >> > > test? Do I need to create a patch with all my current
> > > >> changes for
> > > >> > > someone to update the release plugin?
> > > >> > >
> > > >> > > regards,
> > > >> > >
> > > >> > > Wim
> > > >> >
> > > >> >
> > > >>
> > > >>
> > > >
> > > >
> > > >
> > >
> > >
> >
> >
>
Re: release plugin update for clearcase
Posted by Wim Deblauwe <wi...@gmail.com>.
I'm trying to build maven-scm-provider-clearcase, but I get the following
error:
[INFO] Failed to resolve artifact.
GroupId: org.apache.maven.scm
ArtifactId: maven-scm-test
Version: 1.0-beta-2-SNAPSHOT
Reason: Unable to download the artifact from any repository
org.apache.maven.scm:maven-scm-test:1.0-beta-2-SNAPSHOT:jar
from the specified remote repositories:
central (http://repo1.maven.org/maven2),
snapshots (http://snapshots.maven.codehaus.org/maven2)
On the release plugin: I will need to get it from svn and build it myself I
suppose? Or did you deploy it somewhere?
regards,
Wim
2005/11/21, Emmanuel Venisse <em...@venisse.net>:
>
> I done changes.
> You'll need to use -DuseEditMode=true with release:prepare
>
> Emmanuel
>
> Wim Deblauwe a écrit :
> > Well, we discussed this already I thought. Before the release plugin
> > changes the pom.xml, it needs to do a "edit" on the pom.xml. If you
> > could change that, I'm very willing to test it out.
> >
> > regards,
> >
> > Wim
> >
> > 2005/11/21, Emmanuel Venisse <emmanuel@venisse.net
> > <ma...@venisse.net>>:
> >
> > I can do it, but i need to know the process for clearcase. It will
> > be better if you can update it
> > yourself.
> >
> > Emmanuel
> >
> > Wim Deblauwe a écrit :
> > > Hey,
> > >
> > > I noticed the patch has been applied, thank you for that. What is
> > next?
> > > Do we have to make a request to update the release plugin or are you
> > > willing to update that?
> > >
> > > regards,
> > >
> > > Wim
> > >
> > > 2005/11/21, Wim Deblauwe < wim.deblauwe@gmail.com
> > <ma...@gmail.com>
> > > <mailto:wim.deblauwe@gmail.com <ma...@gmail.com>>>:
> > >
> > > Allright, here we go again. I have updated SCM-77 with a new
> > patch.
> > > Hopefully this one will satisfy all your needs :)
> > >
> > > if there are any things I should change, please let me know.
> > >
> > > For the edit and unedit commands I have used the
> > > ScmFileStatus.UNKNOWN, because I don't know what I should use
> > there.
> > > Maybe you guys can put something better in there.
> > >
> > > regards,
> > >
> > > Wim
> > >
> > > 2005/11/19, Emmanuel Venisse <emmanuel@venisse.net
> > <ma...@venisse.net>
> > > <mailto:emmanuel@venisse.net <ma...@venisse.net>>>:
> > >
> > >
> > >
> > > Wim Deblauwe a écrit :
> > >> Hey Emmanuel,
> > >>
> > >> 1) I'm using TortoiseSVN to generate the patch. Maybe I did
> > > something
> > >> wrong, I never worked with it before.
> > >
> > > Strange, it works fine normally.
> > >
> > >>
> > >> 2) I sure can. Normally I just let IntelliJ IDEA do my
> > > formatting. Don't
> > >> know if you have the correct settings? Otherwise, i'll just
> > > fix the tabs.
> > >
> > >
> >
> http://maven.apache.org/guides/development/guide-m2-development.html#Maven
> > > Code Style
> > >
> > >>
> > >> 3) Well, because I was unable to test and also I could not get the
> > >> logging to work (see one of my previous emails)
> > >
> > > clearcase commands print something to console, so, you
> > can copy
> > > this output to text files and parse
> > > them with consumers. You don't need logger for tests,
> > only some
> > > junit assert that verify your
> > > consumer is correct.
> > >
> > >>
> > >> 4) I haven't made testclasses yet. Should they really do clearcase
> > >> operations or check if the command line is like it should be?
> > >
> > > command line only and test consumers.
> > >
> > >>
> > >> I'm out of office now, so I will not be able to do these
> > > things until
> > >> monday.
> > >>
> > >> regards,
> > >>
> > >> Wim
> > >>
> > >> 2005/11/18, Emmanuel Venisse < emmanuel@venisse.net
> > <ma...@venisse.net>
> > > <mailto:emmanuel@venisse.net <ma...@venisse.net>>
> > >> <mailto: emmanuel@venisse.net <ma...@venisse.net>
> > <mailto:emmanuel@venisse.net <ma...@venisse.net>>>>:
> > >>
> > >> Some comments about your patch :
> > >> - How do you have generated it? some files are present more
> > > than one
> > >> time.
> > >> - Can you replace all tab character with 4 spaces?
> > >> - Why your consumers do nothing?
> > >> - Where are test classes?
> > >>
> > >> if you can recreate your patch and integrate all those
> > > remarks, it
> > >> will be more simple to read,
> > >> apply and maintain.
> > >>
> > >> Emmanuel
> > >>
> > >> Wim Deblauwe a écrit :
> > >> > Done: http://jira.codehaus.org/browse/MNG-1622
> > > < http://jira.codehaus.org/browse/MNG-1622>
> > >> >
> > >> > I did not modify the release-plugin.
> > >> >
> > >> > regards,
> > >> >
> > >> > Wim
> > >> >
> > >> > 2005/11/18, Emmanuel Venisse < emmanuel@venisse.net
> > <ma...@venisse.net>
> > > <mailto: emmanuel@venisse.net <ma...@venisse.net>>
> > >> <mailto:emmanuel@venisse.net <ma...@venisse.net>
> > <mailto:emmanuel@venisse.net <ma...@venisse.net>>>
> > >> > <mailto: emmanuel@venisse.net <ma...@venisse.net>
> > > <mailto:emmanuel@venisse.net
> > <ma...@venisse.net>> <mailto: emmanuel@venisse.net
> > <ma...@venisse.net>
> > > <mailto:emmanuel@venisse.net
> > <ma...@venisse.net>>>>>:
> > >> >
> > >> > Great.
> > >> >
> > >> > you can create a patch for maven-scm and attach to a
> > > jira issue.
> > >> > I'll review it (and perhaps Dan too).
> > >> >
> > >> > Do you have modify release-plugin?
> > >> >
> > >> > Wim Deblauwe a écrit :
> > >> > > Hi guys,
> > >> > >
> > >> > > I've implemented add, checkin, edit, remove,
> > > status and
> > >> tag for
> > >> > > clearcase. Is somebody willing to update the release
> > >> plugin so I can
> > >> > > test? Do I need to create a patch with all my current
> > >> changes for
> > >> > > someone to update the release plugin?
> > >> > >
> > >> > > regards,
> > >> > >
> > >> > > Wim
> > >> >
> > >> >
> > >>
> > >>
> > >
> > >
> > >
> >
> >
>
>
Re: release plugin update for clearcase
Posted by Emmanuel Venisse <em...@venisse.net>.
I done changes.
You'll need to use -DuseEditMode=true with release:prepare
Emmanuel
Wim Deblauwe a écrit :
> Well, we discussed this already I thought. Before the release plugin
> changes the pom.xml, it needs to do a "edit" on the pom.xml. If you
> could change that, I'm very willing to test it out.
>
> regards,
>
> Wim
>
> 2005/11/21, Emmanuel Venisse <emmanuel@venisse.net
> <ma...@venisse.net>>:
>
> I can do it, but i need to know the process for clearcase. It will
> be better if you can update it
> yourself.
>
> Emmanuel
>
> Wim Deblauwe a écrit :
> > Hey,
> >
> > I noticed the patch has been applied, thank you for that. What is
> next?
> > Do we have to make a request to update the release plugin or are you
> > willing to update that?
> >
> > regards,
> >
> > Wim
> >
> > 2005/11/21, Wim Deblauwe < wim.deblauwe@gmail.com
> <ma...@gmail.com>
> > <mailto:wim.deblauwe@gmail.com <ma...@gmail.com>>>:
> >
> > Allright, here we go again. I have updated SCM-77 with a new
> patch.
> > Hopefully this one will satisfy all your needs :)
> >
> > if there are any things I should change, please let me know.
> >
> > For the edit and unedit commands I have used the
> > ScmFileStatus.UNKNOWN, because I don't know what I should use
> there.
> > Maybe you guys can put something better in there.
> >
> > regards,
> >
> > Wim
> >
> > 2005/11/19, Emmanuel Venisse <emmanuel@venisse.net
> <ma...@venisse.net>
> > <mailto:emmanuel@venisse.net <ma...@venisse.net>>>:
> >
> >
> >
> > Wim Deblauwe a écrit :
> >> Hey Emmanuel,
> >>
> >> 1) I'm using TortoiseSVN to generate the patch. Maybe I did
> > something
> >> wrong, I never worked with it before.
> >
> > Strange, it works fine normally.
> >
> >>
> >> 2) I sure can. Normally I just let IntelliJ IDEA do my
> > formatting. Don't
> >> know if you have the correct settings? Otherwise, i'll just
> > fix the tabs.
> >
> >
> http://maven.apache.org/guides/development/guide-m2-development.html#Maven
> > Code Style
> >
> >>
> >> 3) Well, because I was unable to test and also I could not get the
> >> logging to work (see one of my previous emails)
> >
> > clearcase commands print something to console, so, you
> can copy
> > this output to text files and parse
> > them with consumers. You don't need logger for tests,
> only some
> > junit assert that verify your
> > consumer is correct.
> >
> >>
> >> 4) I haven't made testclasses yet. Should they really do clearcase
> >> operations or check if the command line is like it should be?
> >
> > command line only and test consumers.
> >
> >>
> >> I'm out of office now, so I will not be able to do these
> > things until
> >> monday.
> >>
> >> regards,
> >>
> >> Wim
> >>
> >> 2005/11/18, Emmanuel Venisse < emmanuel@venisse.net
> <ma...@venisse.net>
> > <mailto:emmanuel@venisse.net <ma...@venisse.net>>
> >> <mailto: emmanuel@venisse.net <ma...@venisse.net>
> <mailto:emmanuel@venisse.net <ma...@venisse.net>>>>:
> >>
> >> Some comments about your patch :
> >> - How do you have generated it? some files are present more
> > than one
> >> time.
> >> - Can you replace all tab character with 4 spaces?
> >> - Why your consumers do nothing?
> >> - Where are test classes?
> >>
> >> if you can recreate your patch and integrate all those
> > remarks, it
> >> will be more simple to read,
> >> apply and maintain.
> >>
> >> Emmanuel
> >>
> >> Wim Deblauwe a écrit :
> >> > Done: http://jira.codehaus.org/browse/MNG-1622
> > < http://jira.codehaus.org/browse/MNG-1622>
> >> >
> >> > I did not modify the release-plugin.
> >> >
> >> > regards,
> >> >
> >> > Wim
> >> >
> >> > 2005/11/18, Emmanuel Venisse < emmanuel@venisse.net
> <ma...@venisse.net>
> > <mailto: emmanuel@venisse.net <ma...@venisse.net>>
> >> <mailto:emmanuel@venisse.net <ma...@venisse.net>
> <mailto:emmanuel@venisse.net <ma...@venisse.net>>>
> >> > <mailto: emmanuel@venisse.net <ma...@venisse.net>
> > <mailto:emmanuel@venisse.net
> <ma...@venisse.net>> <mailto: emmanuel@venisse.net
> <ma...@venisse.net>
> > <mailto:emmanuel@venisse.net
> <ma...@venisse.net>>>>>:
> >> >
> >> > Great.
> >> >
> >> > you can create a patch for maven-scm and attach to a
> > jira issue.
> >> > I'll review it (and perhaps Dan too).
> >> >
> >> > Do you have modify release-plugin?
> >> >
> >> > Wim Deblauwe a écrit :
> >> > > Hi guys,
> >> > >
> >> > > I've implemented add, checkin, edit, remove,
> > status and
> >> tag for
> >> > > clearcase. Is somebody willing to update the release
> >> plugin so I can
> >> > > test? Do I need to create a patch with all my current
> >> changes for
> >> > > someone to update the release plugin?
> >> > >
> >> > > regards,
> >> > >
> >> > > Wim
> >> >
> >> >
> >>
> >>
> >
> >
> >
>
>
Re: Re: release plugin update for clearcase
Posted by Wim Deblauwe <wi...@gmail.com>.
Well, we discussed this already I thought. Before the release plugin changes
the pom.xml, it needs to do a "edit" on the pom.xml. If you could change
that, I'm very willing to test it out.
regards,
Wim
2005/11/21, Emmanuel Venisse <em...@venisse.net>:
>
> I can do it, but i need to know the process for clearcase. It will be
> better if you can update it
> yourself.
>
> Emmanuel
>
> Wim Deblauwe a écrit :
> > Hey,
> >
> > I noticed the patch has been applied, thank you for that. What is next?
> > Do we have to make a request to update the release plugin or are you
> > willing to update that?
> >
> > regards,
> >
> > Wim
> >
> > 2005/11/21, Wim Deblauwe <wim.deblauwe@gmail.com
> > <ma...@gmail.com>>:
> >
> > Allright, here we go again. I have updated SCM-77 with a new patch.
> > Hopefully this one will satisfy all your needs :)
> >
> > if there are any things I should change, please let me know.
> >
> > For the edit and unedit commands I have used the
> > ScmFileStatus.UNKNOWN, because I don't know what I should use there.
> > Maybe you guys can put something better in there.
> >
> > regards,
> >
> > Wim
> >
> > 2005/11/19, Emmanuel Venisse <emmanuel@venisse.net
> > <ma...@venisse.net>>:
> >
> >
> >
> > Wim Deblauwe a écrit :
> >> Hey Emmanuel,
> >>
> >> 1) I'm using TortoiseSVN to generate the patch. Maybe I did
> > something
> >> wrong, I never worked with it before.
> >
> > Strange, it works fine normally.
> >
> >>
> >> 2) I sure can. Normally I just let IntelliJ IDEA do my
> > formatting. Don't
> >> know if you have the correct settings? Otherwise, i'll just
> > fix the tabs.
> >
> >
> http://maven.apache.org/guides/development/guide-m2-development.html#Maven
> > Code Style
> >
> >>
> >> 3) Well, because I was unable to test and also I could not get the
> >> logging to work (see one of my previous emails)
> >
> > clearcase commands print something to console, so, you can copy
> > this output to text files and parse
> > them with consumers. You don't need logger for tests, only some
> > junit assert that verify your
> > consumer is correct.
> >
> >>
> >> 4) I haven't made testclasses yet. Should they really do clearcase
> >> operations or check if the command line is like it should be?
> >
> > command line only and test consumers.
> >
> >>
> >> I'm out of office now, so I will not be able to do these
> > things until
> >> monday.
> >>
> >> regards,
> >>
> >> Wim
> >>
> >> 2005/11/18, Emmanuel Venisse < emmanuel@venisse.net
> > <ma...@venisse.net>
> >> <mailto: emmanuel@venisse.net <ma...@venisse.net>>>:
> >>
> >> Some comments about your patch :
> >> - How do you have generated it? some files are present more
> > than one
> >> time.
> >> - Can you replace all tab character with 4 spaces?
> >> - Why your consumers do nothing?
> >> - Where are test classes?
> >>
> >> if you can recreate your patch and integrate all those
> > remarks, it
> >> will be more simple to read,
> >> apply and maintain.
> >>
> >> Emmanuel
> >>
> >> Wim Deblauwe a écrit :
> >> > Done: http://jira.codehaus.org/browse/MNG-1622
> > <http://jira.codehaus.org/browse/MNG-1622>
> >> >
> >> > I did not modify the release-plugin.
> >> >
> >> > regards,
> >> >
> >> > Wim
> >> >
> >> > 2005/11/18, Emmanuel Venisse < emmanuel@venisse.net
> > <ma...@venisse.net>
> >> <mailto:emmanuel@venisse.net <ma...@venisse.net>>
> >> > <mailto:emmanuel@venisse.net
> > <ma...@venisse.net> <mailto: emmanuel@venisse.net
> > <ma...@venisse.net>>>>:
> >> >
> >> > Great.
> >> >
> >> > you can create a patch for maven-scm and attach to a
> > jira issue.
> >> > I'll review it (and perhaps Dan too).
> >> >
> >> > Do you have modify release-plugin?
> >> >
> >> > Wim Deblauwe a écrit :
> >> > > Hi guys,
> >> > >
> >> > > I've implemented add, checkin, edit, remove,
> > status and
> >> tag for
> >> > > clearcase. Is somebody willing to update the release
> >> plugin so I can
> >> > > test? Do I need to create a patch with all my current
> >> changes for
> >> > > someone to update the release plugin?
> >> > >
> >> > > regards,
> >> > >
> >> > > Wim
> >> >
> >> >
> >>
> >>
> >
> >
> >
>
>
Re: Re: release plugin update for clearcase
Posted by Emmanuel Venisse <em...@venisse.net>.
I can do it, but i need to know the process for clearcase. It will be better if you can update it
yourself.
Emmanuel
Wim Deblauwe a écrit :
> Hey,
>
> I noticed the patch has been applied, thank you for that. What is next?
> Do we have to make a request to update the release plugin or are you
> willing to update that?
>
> regards,
>
> Wim
>
> 2005/11/21, Wim Deblauwe <wim.deblauwe@gmail.com
> <ma...@gmail.com>>:
>
> Allright, here we go again. I have updated SCM-77 with a new patch.
> Hopefully this one will satisfy all your needs :)
>
> if there are any things I should change, please let me know.
>
> For the edit and unedit commands I have used the
> ScmFileStatus.UNKNOWN, because I don't know what I should use there.
> Maybe you guys can put something better in there.
>
> regards,
>
> Wim
>
> 2005/11/19, Emmanuel Venisse <emmanuel@venisse.net
> <ma...@venisse.net>>:
>
>
>
> Wim Deblauwe a écrit :
>> Hey Emmanuel,
>>
>> 1) I'm using TortoiseSVN to generate the patch. Maybe I did
> something
>> wrong, I never worked with it before.
>
> Strange, it works fine normally.
>
>>
>> 2) I sure can. Normally I just let IntelliJ IDEA do my
> formatting. Don't
>> know if you have the correct settings? Otherwise, i'll just
> fix the tabs.
>
> http://maven.apache.org/guides/development/guide-m2-development.html#Maven
> Code Style
>
>>
>> 3) Well, because I was unable to test and also I could not get the
>> logging to work (see one of my previous emails)
>
> clearcase commands print something to console, so, you can copy
> this output to text files and parse
> them with consumers. You don't need logger for tests, only some
> junit assert that verify your
> consumer is correct.
>
>>
>> 4) I haven't made testclasses yet. Should they really do clearcase
>> operations or check if the command line is like it should be?
>
> command line only and test consumers.
>
>>
>> I'm out of office now, so I will not be able to do these
> things until
>> monday.
>>
>> regards,
>>
>> Wim
>>
>> 2005/11/18, Emmanuel Venisse < emmanuel@venisse.net
> <ma...@venisse.net>
>> <mailto: emmanuel@venisse.net <ma...@venisse.net>>>:
>>
>> Some comments about your patch :
>> - How do you have generated it? some files are present more
> than one
>> time.
>> - Can you replace all tab character with 4 spaces?
>> - Why your consumers do nothing?
>> - Where are test classes?
>>
>> if you can recreate your patch and integrate all those
> remarks, it
>> will be more simple to read,
>> apply and maintain.
>>
>> Emmanuel
>>
>> Wim Deblauwe a écrit :
>> > Done: http://jira.codehaus.org/browse/MNG-1622
> <http://jira.codehaus.org/browse/MNG-1622>
>> >
>> > I did not modify the release-plugin.
>> >
>> > regards,
>> >
>> > Wim
>> >
>> > 2005/11/18, Emmanuel Venisse < emmanuel@venisse.net
> <ma...@venisse.net>
>> <mailto:emmanuel@venisse.net <ma...@venisse.net>>
>> > <mailto:emmanuel@venisse.net
> <ma...@venisse.net> <mailto: emmanuel@venisse.net
> <ma...@venisse.net>>>>:
>> >
>> > Great.
>> >
>> > you can create a patch for maven-scm and attach to a
> jira issue.
>> > I'll review it (and perhaps Dan too).
>> >
>> > Do you have modify release-plugin?
>> >
>> > Wim Deblauwe a écrit :
>> > > Hi guys,
>> > >
>> > > I've implemented add, checkin, edit, remove,
> status and
>> tag for
>> > > clearcase. Is somebody willing to update the release
>> plugin so I can
>> > > test? Do I need to create a patch with all my current
>> changes for
>> > > someone to update the release plugin?
>> > >
>> > > regards,
>> > >
>> > > Wim
>> >
>> >
>>
>>
>
>
>
Re: release plugin update for clearcase
Posted by Wim Deblauwe <wi...@gmail.com>.
Hey,
I noticed the patch has been applied, thank you for that. What is next? Do
we have to make a request to update the release plugin or are you willing to
update that?
regards,
Wim
2005/11/21, Wim Deblauwe <wi...@gmail.com>:
>
> Allright, here we go again. I have updated SCM-77 with a new patch.
> Hopefully this one will satisfy all your needs :)
>
> if there are any things I should change, please let me know.
>
> For the edit and unedit commands I have used the ScmFileStatus.UNKNOWN,
> because I don't know what I should use there. Maybe you guys can put
> something better in there.
>
> regards,
>
> Wim
>
> 2005/11/19, Emmanuel Venisse <em...@venisse.net>:
> >
> >
> >
> > Wim Deblauwe a écrit :
> > > Hey Emmanuel,
> > >
> > > 1) I'm using TortoiseSVN to generate the patch. Maybe I did something
> > > wrong, I never worked with it before.
> >
> > Strange, it works fine normally.
> >
> > >
> > > 2) I sure can. Normally I just let IntelliJ IDEA do my formatting.
> > Don't
> > > know if you have the correct settings? Otherwise, i'll just fix the
> > tabs.
> >
> >
> > http://maven.apache.org/guides/development/guide-m2-development.html#MavenCode Style
> >
> > >
> > > 3) Well, because I was unable to test and also I could not get the
> > > logging to work (see one of my previous emails)
> >
> > clearcase commands print something to console, so, you can copy this
> > output to text files and parse
> > them with consumers. You don't need logger for tests, only some junit
> > assert that verify your
> > consumer is correct.
> >
> > >
> > > 4) I haven't made testclasses yet. Should they really do clearcase
> > > operations or check if the command line is like it should be?
> >
> > command line only and test consumers.
> >
> > >
> > > I'm out of office now, so I will not be able to do these things until
> > > monday.
> > >
> > > regards,
> > >
> > > Wim
> > >
> > > 2005/11/18, Emmanuel Venisse <emmanuel@venisse.net
> > > <mailto: emmanuel@venisse.net>>:
> > >
> > > Some comments about your patch :
> > > - How do you have generated it? some files are present more than one
> > > time.
> > > - Can you replace all tab character with 4 spaces?
> > > - Why your consumers do nothing?
> > > - Where are test classes?
> > >
> > > if you can recreate your patch and integrate all those remarks, it
> > > will be more simple to read,
> > > apply and maintain.
> > >
> > > Emmanuel
> > >
> > > Wim Deblauwe a écrit :
> > > > Done: http://jira.codehaus.org/browse/MNG-1622
> > > >
> > > > I did not modify the release-plugin.
> > > >
> > > > regards,
> > > >
> > > > Wim
> > > >
> > > > 2005/11/18, Emmanuel Venisse < emmanuel@venisse.net
> > > <ma...@venisse.net>
> > > > <mailto:emmanuel@venisse.net <mailto: emmanuel@venisse.net>>>:
> > > >
> > > > Great.
> > > >
> > > > you can create a patch for maven-scm and attach to a jira issue.
> > > > I'll review it (and perhaps Dan too).
> > > >
> > > > Do you have modify release-plugin?
> > > >
> > > > Wim Deblauwe a écrit :
> > > > > Hi guys,
> > > > >
> > > > > I've implemented add, checkin, edit, remove, status and
> > > tag for
> > > > > clearcase. Is somebody willing to update the release
> > > plugin so I can
> > > > > test? Do I need to create a patch with all my current
> > > changes for
> > > > > someone to update the release plugin?
> > > > >
> > > > > regards,
> > > > >
> > > > > Wim
> > > >
> > > >
> > >
> > >
> >
> >
>
Re: release plugin update for clearcase
Posted by Wim Deblauwe <wi...@gmail.com>.
Allright, here we go again. I have updated SCM-77 with a new patch.
Hopefully this one will satisfy all your needs :)
if there are any things I should change, please let me know.
For the edit and unedit commands I have used the ScmFileStatus.UNKNOWN,
because I don't know what I should use there. Maybe you guys can put
something better in there.
regards,
Wim
2005/11/19, Emmanuel Venisse <em...@venisse.net>:
>
>
>
> Wim Deblauwe a écrit :
> > Hey Emmanuel,
> >
> > 1) I'm using TortoiseSVN to generate the patch. Maybe I did something
> > wrong, I never worked with it before.
>
> Strange, it works fine normally.
>
> >
> > 2) I sure can. Normally I just let IntelliJ IDEA do my formatting. Don't
> > know if you have the correct settings? Otherwise, i'll just fix the
> tabs.
>
> http://maven.apache.org/guides/development/guide-m2-development.html#MavenCode Style
>
> >
> > 3) Well, because I was unable to test and also I could not get the
> > logging to work (see one of my previous emails)
>
> clearcase commands print something to console, so, you can copy this
> output to text files and parse
> them with consumers. You don't need logger for tests, only some junit
> assert that verify your
> consumer is correct.
>
> >
> > 4) I haven't made testclasses yet. Should they really do clearcase
> > operations or check if the command line is like it should be?
>
> command line only and test consumers.
>
> >
> > I'm out of office now, so I will not be able to do these things until
> > monday.
> >
> > regards,
> >
> > Wim
> >
> > 2005/11/18, Emmanuel Venisse <emmanuel@venisse.net
> > <ma...@venisse.net>>:
> >
> > Some comments about your patch :
> > - How do you have generated it? some files are present more than one
> > time.
> > - Can you replace all tab character with 4 spaces?
> > - Why your consumers do nothing?
> > - Where are test classes?
> >
> > if you can recreate your patch and integrate all those remarks, it
> > will be more simple to read,
> > apply and maintain.
> >
> > Emmanuel
> >
> > Wim Deblauwe a écrit :
> > > Done: http://jira.codehaus.org/browse/MNG-1622
> > >
> > > I did not modify the release-plugin.
> > >
> > > regards,
> > >
> > > Wim
> > >
> > > 2005/11/18, Emmanuel Venisse < emmanuel@venisse.net
> > <ma...@venisse.net>
> > > <mailto:emmanuel@venisse.net <ma...@venisse.net>>>:
> > >
> > > Great.
> > >
> > > you can create a patch for maven-scm and attach to a jira issue.
> > > I'll review it (and perhaps Dan too).
> > >
> > > Do you have modify release-plugin?
> > >
> > > Wim Deblauwe a écrit :
> > > > Hi guys,
> > > >
> > > > I've implemented add, checkin, edit, remove, status and
> > tag for
> > > > clearcase. Is somebody willing to update the release
> > plugin so I can
> > > > test? Do I need to create a patch with all my current
> > changes for
> > > > someone to update the release plugin?
> > > >
> > > > regards,
> > > >
> > > > Wim
> > >
> > >
> >
> >
>
>
Re: release plugin update for clearcase
Posted by Emmanuel Venisse <em...@venisse.net>.
Wim Deblauwe a écrit :
> Hey Emmanuel,
>
> 1) I'm using TortoiseSVN to generate the patch. Maybe I did something
> wrong, I never worked with it before.
Strange, it works fine normally.
>
> 2) I sure can. Normally I just let IntelliJ IDEA do my formatting. Don't
> know if you have the correct settings? Otherwise, i'll just fix the tabs.
http://maven.apache.org/guides/development/guide-m2-development.html#Maven Code Style
>
> 3) Well, because I was unable to test and also I could not get the
> logging to work (see one of my previous emails)
clearcase commands print something to console, so, you can copy this output to text files and parse
them with consumers. You don't need logger for tests, only some junit assert that verify your
consumer is correct.
>
> 4) I haven't made testclasses yet. Should they really do clearcase
> operations or check if the command line is like it should be?
command line only and test consumers.
>
> I'm out of office now, so I will not be able to do these things until
> monday.
>
> regards,
>
> Wim
>
> 2005/11/18, Emmanuel Venisse <emmanuel@venisse.net
> <ma...@venisse.net>>:
>
> Some comments about your patch :
> - How do you have generated it? some files are present more than one
> time.
> - Can you replace all tab character with 4 spaces?
> - Why your consumers do nothing?
> - Where are test classes?
>
> if you can recreate your patch and integrate all those remarks, it
> will be more simple to read,
> apply and maintain.
>
> Emmanuel
>
> Wim Deblauwe a écrit :
> > Done: http://jira.codehaus.org/browse/MNG-1622
> >
> > I did not modify the release-plugin.
> >
> > regards,
> >
> > Wim
> >
> > 2005/11/18, Emmanuel Venisse < emmanuel@venisse.net
> <ma...@venisse.net>
> > <mailto:emmanuel@venisse.net <ma...@venisse.net>>>:
> >
> > Great.
> >
> > you can create a patch for maven-scm and attach to a jira issue.
> > I'll review it (and perhaps Dan too).
> >
> > Do you have modify release-plugin?
> >
> > Wim Deblauwe a écrit :
> > > Hi guys,
> > >
> > > I've implemented add, checkin, edit, remove, status and
> tag for
> > > clearcase. Is somebody willing to update the release
> plugin so I can
> > > test? Do I need to create a patch with all my current
> changes for
> > > someone to update the release plugin?
> > >
> > > regards,
> > >
> > > Wim
> >
> >
>
>
Re: release plugin update for clearcase
Posted by Wim Deblauwe <wi...@gmail.com>.
Hey Emmanuel,
1) I'm using TortoiseSVN to generate the patch. Maybe I did something wrong,
I never worked with it before.
2) I sure can. Normally I just let IntelliJ IDEA do my formatting. Don't
know if you have the correct settings? Otherwise, i'll just fix the tabs.
3) Well, because I was unable to test and also I could not get the logging
to work (see one of my previous emails)
4) I haven't made testclasses yet. Should they really do clearcase
operations or check if the command line is like it should be?
I'm out of office now, so I will not be able to do these things until
monday.
regards,
Wim
2005/11/18, Emmanuel Venisse <em...@venisse.net>:
>
> Some comments about your patch :
> - How do you have generated it? some files are present more than one time.
> - Can you replace all tab character with 4 spaces?
> - Why your consumers do nothing?
> - Where are test classes?
>
> if you can recreate your patch and integrate all those remarks, it will be
> more simple to read,
> apply and maintain.
>
> Emmanuel
>
> Wim Deblauwe a écrit :
> > Done: http://jira.codehaus.org/browse/MNG-1622
> >
> > I did not modify the release-plugin.
> >
> > regards,
> >
> > Wim
> >
> > 2005/11/18, Emmanuel Venisse <emmanuel@venisse.net
> > <ma...@venisse.net>>:
> >
> > Great.
> >
> > you can create a patch for maven-scm and attach to a jira issue.
> > I'll review it (and perhaps Dan too).
> >
> > Do you have modify release-plugin?
> >
> > Wim Deblauwe a écrit :
> > > Hi guys,
> > >
> > > I've implemented add, checkin, edit, remove, status and tag for
> > > clearcase. Is somebody willing to update the release plugin so I can
> > > test? Do I need to create a patch with all my current changes for
> > > someone to update the release plugin?
> > >
> > > regards,
> > >
> > > Wim
> >
> >
>
>
Re: release plugin update for clearcase
Posted by Emmanuel Venisse <em...@venisse.net>.
Some comments about your patch :
- How do you have generated it? some files are present more than one time.
- Can you replace all tab character with 4 spaces?
- Why your consumers do nothing?
- Where are test classes?
if you can recreate your patch and integrate all those remarks, it will be more simple to read,
apply and maintain.
Emmanuel
Wim Deblauwe a écrit :
> Done: http://jira.codehaus.org/browse/MNG-1622
>
> I did not modify the release-plugin.
>
> regards,
>
> Wim
>
> 2005/11/18, Emmanuel Venisse <emmanuel@venisse.net
> <ma...@venisse.net>>:
>
> Great.
>
> you can create a patch for maven-scm and attach to a jira issue.
> I'll review it (and perhaps Dan too).
>
> Do you have modify release-plugin?
>
> Wim Deblauwe a écrit :
> > Hi guys,
> >
> > I've implemented add, checkin, edit, remove, status and tag for
> > clearcase. Is somebody willing to update the release plugin so I can
> > test? Do I need to create a patch with all my current changes for
> > someone to update the release plugin?
> >
> > regards,
> >
> > Wim
>
>
Re: release plugin update for clearcase
Posted by Wim Deblauwe <wi...@gmail.com>.
Done: http://jira.codehaus.org/browse/MNG-1622
I did not modify the release-plugin.
regards,
Wim
2005/11/18, Emmanuel Venisse <em...@venisse.net>:
>
> Great.
>
> you can create a patch for maven-scm and attach to a jira issue. I'll
> review it (and perhaps Dan too).
>
> Do you have modify release-plugin?
>
> Wim Deblauwe a écrit :
> > Hi guys,
> >
> > I've implemented add, checkin, edit, remove, status and tag for
> > clearcase. Is somebody willing to update the release plugin so I can
> > test? Do I need to create a patch with all my current changes for
> > someone to update the release plugin?
> >
> > regards,
> >
> > Wim
>
>
Re: release plugin update for clearcase
Posted by Emmanuel Venisse <em...@venisse.net>.
Great.
you can create a patch for maven-scm and attach to a jira issue. I'll review it (and perhaps Dan too).
Do you have modify release-plugin?
Wim Deblauwe a écrit :
> Hi guys,
>
> I've implemented add, checkin, edit, remove, status and tag for
> clearcase. Is somebody willing to update the release plugin so I can
> test? Do I need to create a patch with all my current changes for
> someone to update the release plugin?
>
> regards,
>
> Wim