Cargo

Create a m2 plugin

Details

  • Type: New Feature New Feature
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 0.1-maven2
  • Component/s: Maven2
  • Labels:
    None
  • Number of attachments :
    2

Description

Early version almost ready, will finish & submit when I have time..

Activity

Hide
Vincent Massol added a comment -

Mark,

I have started implementing the config defined on http://docs.codehaus.org/display/CARGO/Maven2+plugin. I'm attaching what I've done today in maven2-20051027-vma.zip. It's not functional but I wanted to give you feedback about the direction I'm going in. I'm not at a stage where I can check in. As you can see I have removed some or your mojos but that's just temporarily. I have just deleted them locally so that I can build the new version.

Here are the improvements/issues:

  • m2 doesn't allow to put the configuration POJOs in a directory different from where the mojos are. Jason Van Zyl said he wanted to support this in the future
  • m2 doesn't allow to name the config element differently (for example Configuration --> ConfigurationElement, etc). Thus the code is a little bit difficult to read. Jason said he'll fix that.
  • I still need to map all the monitors, system properties, extra classpaths, etc
  • It doesn't work yet but it shouldn't be too far...
  • Our Ant code uses almost exactly the same configuration POJOs classes. There's code sharing that we could do but I'm not sure it'll always work (for example what happens if we change the Ant config format).

I'll continue working on this tomorrow.

Let me know what you think!

Thanks

Show
Vincent Massol added a comment - Mark, I have started implementing the config defined on http://docs.codehaus.org/display/CARGO/Maven2+plugin. I'm attaching what I've done today in maven2-20051027-vma.zip. It's not functional but I wanted to give you feedback about the direction I'm going in. I'm not at a stage where I can check in. As you can see I have removed some or your mojos but that's just temporarily. I have just deleted them locally so that I can build the new version. Here are the improvements/issues:
  • m2 doesn't allow to put the configuration POJOs in a directory different from where the mojos are. Jason Van Zyl said he wanted to support this in the future
  • m2 doesn't allow to name the config element differently (for example Configuration --> ConfigurationElement, etc). Thus the code is a little bit difficult to read. Jason said he'll fix that.
  • I still need to map all the monitors, system properties, extra classpaths, etc
  • It doesn't work yet but it shouldn't be too far...
  • Our Ant code uses almost exactly the same configuration POJOs classes. There's code sharing that we could do but I'm not sure it'll always work (for example what happens if we change the Ant config format).
I'll continue working on this tomorrow. Let me know what you think! Thanks
Hide
Vincent Massol added a comment -

My first try at implementing the config defined on http://docs.codehaus.org/display/CARGO/Maven2+plugin

Show
Vincent Massol added a comment - My first try at implementing the config defined on http://docs.codehaus.org/display/CARGO/Maven2+plugin
Hide
Vincent Massol added a comment -

Here's a working version. It supports starting and stopping containers.

Mark, if you're ok I'd like to commit this and then work on adding the deployers again. WDYT?

Show
Vincent Massol added a comment - Here's a working version. It supports starting and stopping containers. Mark, if you're ok I'd like to commit this and then work on adding the deployers again. WDYT?
Hide
Mark Hobson added a comment -

Sure, please do - looks good to me. I'll have some time this weekend to help out, so best get in sync before we proceed along divergent paths Chat then.

Show
Mark Hobson added a comment - Sure, please do - looks good to me. I'll have some time this weekend to help out, so best get in sync before we proceed along divergent paths Chat then.
Hide
Vincent Massol added a comment -

First implementation done.

Show
Vincent Massol added a comment - First implementation done.
Hide
Vincent Massol added a comment -

reopen to change component to maven2

Show
Vincent Massol added a comment - reopen to change component to maven2
Hide
Vincent Massol added a comment -

changed component to maven2

Show
Vincent Massol added a comment - changed component to maven2
Hide
Vincent Massol added a comment -

Changed component to maven2

Show
Vincent Massol added a comment - Changed component to maven2
Hide
Vincent Massol added a comment -

Reopen to switch fixfor version to 0.1-maven2

Show
Vincent Massol added a comment - Reopen to switch fixfor version to 0.1-maven2
Hide
Vincent Massol added a comment -

Changed fixfor version to 0.1-maven2

Show
Vincent Massol added a comment - Changed fixfor version to 0.1-maven2

People

Vote (1)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: