jira.codehaus.org

  • Log In Access more options
    • Online Help
    • Keyboard Shortcuts
    • About JIRA
    • JIRA Credits
    • What?s New
  • Dashboards Access more options (Alt+d)
  • Projects Access more options (Alt+p)
  • Issues Access more options (Alt+i)
Signup
Maven 1.x Changelog Plugin
  • Maven 1.x Changelog Plugin
  • MPCHANGELOG-80

MKS Support

  • Log In
  • Views
    • XML
    • Word
    • Printable

Details

  • Type: New Feature New Feature
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 1.9
  • 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?

  • Options
    • Sort By Name
    • Sort By Date
    • Ascending
    • Descending
    • Download All

Attachments

  1. File
    changelog-mks.diff
    13/Jan/06 4:36 AM
    12 kB
    Christoph Jerolimov
  2. Hide
    Zip Archive
    changelog-mks-support.zip
    13/Jan/06 3:22 AM
    9 kB
    Christoph Jerolimov
    1. Java Source File
      changelog/src/.../changelog/ChangeLog.java 19 kB
    2. Java Source File
      changelog/src/.../MksChangeLogFactory.java 1 kB
    3. Java Source File
      changelog/src/.../MksChangeLogGenerator.java 2 kB
    4. Java Source File
      changelog/src/.../MksChangeLogParser.java 7 kB
    Download Zip
    Show
    Zip Archive
    changelog-mks-support.zip
    13/Jan/06 3:22 AM
    9 kB
    Christoph Jerolimov
  3. File
    mks.diff
    19/Jan/06 9:36 AM
    20 kB
    Christoph Jerolimov

Activity

Ascending order - Click to sort in descending order
  • All
  • Comments
  • Work Log
  • History
  • Activity
Hide
Permalink
Arnaud Heritier added a comment - 13/Jan/06 3:59 AM

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 - 13/Jan/06 3:59 AM 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 - 13/Jan/06 4:36 AM

Okay, here's a svn diff.

Show
Christoph Jerolimov added a comment - 13/Jan/06 4:36 AM Okay, here's a svn diff.
Hide
Permalink
Arnaud Heritier added a comment - 13/Jan/06 4:58 AM

Thanks a lot.
We'll review it ASAP.

Show
Arnaud Heritier added a comment - 13/Jan/06 4:58 AM Thanks a lot. We'll review it ASAP.
Hide
Permalink
Lukas Theussl added a comment - 13/Jan/06 3:10 PM

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 - 13/Jan/06 3:10 PM 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 - 19/Jan/06 9:36 AM

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 - 19/Jan/06 9:36 AM 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 - 19/Jan/06 1:06 PM

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 - 19/Jan/06 1:06 PM 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 - 19/Jan/06 1:29 PM

Patch applied. Thanks!

Show
Lukas Theussl added a comment - 19/Jan/06 1:29 PM Patch applied. Thanks!

People

  • Assignee:
    Unassigned
    Reporter:
    Christoph Jerolimov
Vote (0)
Watch (0)

Dates

  • Created:
    13/Jan/06 3:22 AM
    Updated:
    19/Jan/06 1:29 PM
    Resolved:
    19/Jan/06 1:29 PM
  • Atlassian JIRA (v5.2.7#850-sha1:b2af0c8)
  • Report a problem
  • Powered by a free Atlassian JIRA open source license for Codehaus. Try JIRA - bug tracking software for your team.