Maven 1.x Changelog Plugin

MKS Support

Details

  • Type: New Feature New Feature
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 1.9
  • Component/s: None
  • Labels:
    None
  • Number of attachments :
    3

Description

I write a patch for the maven-1 changelog Plugin to support the MKS scm (http://www.mks.com/products/sie). I hope you will add it to your sources. How can i send you the patch? I attach the changed files?

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Arnaud Heritier added a comment -

It's easier for us if :

  • you checkout the plugin sources with a subversion client
  • you apply your modifications and add your new files
  • you generate a patch
  • you attach it to this issue
Show
Arnaud Heritier added a comment - It's easier for us if :
  • you checkout the plugin sources with a subversion client
  • you apply your modifications and add your new files
  • you generate a patch
  • you attach it to this issue
Hide
Permalink
Christoph Jerolimov added a comment -

Okay, here's a svn diff.

Show
Christoph Jerolimov added a comment - Okay, here's a svn diff.
Hide
Permalink
Arnaud Heritier added a comment -

Thanks a lot.
We'll review it ASAP.

Show
Arnaud Heritier added a comment - Thanks a lot. We'll review it ASAP.
Hide
Permalink
Lukas Theussl added a comment -

Thanks for your contribution Christoph!

There are two things missing before we can apply your patch:

  • some test cases that show that your code works (in src/test)
  • documentation updates (in xdocs/)

Do you think you can do that?

Show
Lukas Theussl added a comment - Thanks for your contribution Christoph! There are two things missing before we can apply your patch:
  • some test cases that show that your code works (in src/test)
  • documentation updates (in xdocs/)
Do you think you can do that?
Hide
Permalink
Christoph Jerolimov added a comment -

Hello Lukas, yes i think i can do that.

Here the new version of my diff: "mks.diff"

  • Its only update the informationen which scm the changelog plugin assits. (A "real" documentation do not exist, or?)
  • It include an mks-logfile and a TestCase.

Bye

Show
Christoph Jerolimov added a comment - Hello Lukas, yes i think i can do that. Here the new version of my diff: "mks.diff"
  • Its only update the informationen which scm the changelog plugin assits. (A "real" documentation do not exist, or?)
  • It include an mks-logfile and a TestCase.
Bye
Hide
Permalink
Lukas Theussl added a comment -

Excellent, thanks a lot Christoph!
I have reviewed your patch and it looks good. I will commit it after a few cosmetic changes. Just one note to be on the safe side: one of the source files does not have a copyright notice. I assume that you are aware that by submitting a patch here, you agree to put it under the ASL2 license.

Show
Lukas Theussl added a comment - Excellent, thanks a lot Christoph! I have reviewed your patch and it looks good. I will commit it after a few cosmetic changes. Just one note to be on the safe side: one of the source files does not have a copyright notice. I assume that you are aware that by submitting a patch here, you agree to put it under the ASL2 license.
Hide
Permalink
Lukas Theussl added a comment -

Patch applied. Thanks!

Show
Lukas Theussl added a comment - Patch applied. Thanks!

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved:
Atlassian JIRA (v4.4.3#663-r165197) | Report a problem
Powered by a free Atlassian JIRA open source license for Codehaus. Try JIRA - bug tracking software for your team.