Uploaded image for project: 'Maven SCM'
  1. Maven SCM
  2. SCM-553

release:prepare not working with synergy scm-provider

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 1.3
    • 1.5
    • None
    • found and fixed on

      windows xp, service pack 3
      Synergy 6.5 build 4096
      java 1.6.0_18
      maven 3.0-alpha6
    • Patch

    Description

      All the scm commands, as implemented in the synergy scm-provider, assume a default (synergy-)task or try to set one. However, all commands also stop the synergy session after they finish. This is no problem, when each command is run by itself.

      But, stopping the session also causes Synergy (at least our version 6.5 build 4096) to de-select the default task. This causes mvn release:prepare to fail, as the checked out and modified poms cannot be checked in because no default task (which is required by the check in command, as it is currently implemented) is set.
      Also, when tagging a version (which is done in Synergy by creating a baseline) the so-called prep-project is not updated, which means that the changed poms are not included in the baseline.

      I have attached a patch which fixes these issues for me and keeps the old behaviour largely unchanged so that people, who only use the provider to issue atomic scm-commands should encounter no changes EXCEPT one:

      Now, when a tag is to be created, the provider will always try to update (reconfigure) the project as specified by the scm-url (in the pom, usually the prep-project). This means, that all changes since the last update will automatically be included in the tagged version. Most of the time this is what you want, but it may be an issue if you end up including untested code. The easiest "workaround" is to declare a general code-freeze when tagging a version.

      Attachments

        1. maven-scm-provider-synergy.patch
          27 kB
          Jan Malcomess

        Activity

          People

            olamy Olivier Lamy
            jmalcome Jan Malcomess
            Votes:
            1 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: