release:prepare fails when CVS tagging gives a warning

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Cannot Reproduce
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: None
  • Labels:
    None
  • Environment:
    Maven 2, release plugin version 2.0-beta-3. Scm url is scm:cvs:ext:${user.name}@_HOSTNAME_:_CVSROOT_. The host that runs the release is on Linux and the CVS repository is on Linux.

Description

The release:prepare target fails with

[INFO] Tagging release with the label ADMIN_CONSOLE_3_2_8.
USERNAME@HOSTNAME's password:
Provider message:
The cvs tag command failed.
Command output:
Warning: No xauth data; using fake authentication data for X11 forwarding.
cvs tag: pom.xml is locally modified

Looking through the code it seems like the release fails beacuse the cvs command returns a status different from successful.

  1. Text File
    howto.apt-patch.txt
    08/Sep/06 1:55 PM
    1.0 kB
    Todd Nine
  2. Text File
    howto.apt-patch.txt
    08/Sep/06 1:55 PM
    1.0 kB
    Todd Nine
  3. Text File
    ScmCommitPhase-patch.txt
    08/Sep/06 1:55 PM
    10 kB
    Todd Nine
  4. Text File
    ScmCommitPhaseTest-patch.txt
    08/Sep/06 1:55 PM
    23 kB
    Todd Nine

Issue Links

is duplicated by

Bug - A problem which impairs or prevents the functions of the product. MRELEASE-94 Modified Parent POM is not commited

  • Blocker - Blocks development and/or testing work, production could not run
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Dan Tran added a comment -

you mean the tag went thru and release prepare fails?

Show
Dan Tran added a comment - you mean the tag went thru and release prepare fails?
Hide
Permalink
Brett Porter added a comment -

it looks like the tag command is actually failing, not warning

Show
Brett Porter added a comment - it looks like the tag command is actually failing, not warning
Hide
Permalink
Todd Nine added a comment -

Updated settings property to be more generic

Show
Todd Nine added a comment - Updated settings property to be more generic

People

  • Assignee:
    Brett Porter
    Reporter:
    Ørjan Austvold
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: