Project

General

Profile

Actions

Defect #6706

closed

Resolving issues with the commit message produces the wrong comment with CVS

Added by Jim Naslund over 13 years ago. Updated about 13 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Toshi MARUYAMA
Category:
SCM
Target version:
Start date:
2010-10-20
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed
Affected version:

Description

The comment redmine automatically closes the issue with will say something like 'Applied in changeset r_4.' instead of the actual changeset number.


Files

cvs-close-text.png (74.7 KB) cvs-close-text.png Toshi MARUYAMA, 2010-11-17 08:45
cvs-issue.png (77.8 KB) cvs-issue.png Toshi MARUYAMA, 2011-02-07 12:46
cvs-revision.png (46.4 KB) cvs-revision.png Toshi MARUYAMA, 2011-02-07 12:46

Related issues

Related to Redmine - Defect #996: CVS view ends up with "duplicate key violates unique constraint" after 0.7.0RC1 updateClosedToshi MARUYAMA2008-04-04

Actions
Actions #1

Updated by Jean-Baptiste Barth over 13 years ago

You mean there's a "_" in the message that shouldn't be here ? And can you test if it's still the case in 1.0.x versions ?

Actions #2

Updated by Jim Naslund over 13 years ago

Sorry, I should of clarified that -- no. Our actual revision numbers are in the thousands, but the numbers in the update message are always small. It looks like they are maybe temporary revisions used to group the CVS file changes into one changeset. I've also noticed that if I refresh the activity page while redmine is polling the respository the same 'changeset r_4' messages will appear. If I refresh after it is done they end up as real changeset numbers e.g. r9779.

I can't easily check the 1.0.x versions without upgrading our server.

Actions #3

Updated by Toshi MARUYAMA over 13 years ago

I confirmed svn trunk has this problem.

I attach an image.

The reason is following.

  1. cvs adapter creates changeset with temporary revison "_X" at source:tags/1.0.3/app/models/repository/cvs.rb#L121
  2. Changeset.after_create() is called and new journal is added at source:tags/1.0.3/app/models/changeset.rb#L76
  3. cvs adapter changes revisons "_X" to "Y" at source:tags/1.0.3/app/models/repository/cvs.rb#L148
Actions #4

Updated by Toshi MARUYAMA about 13 years ago

  • Category set to SCM
Actions #5

Updated by Toshi MARUYAMA about 13 years ago

  • Status changed from New to 7
  • Assignee set to Toshi MARUYAMA
Actions #6

Updated by Toshi MARUYAMA about 13 years ago

To fix this issue, I generate pseudo scmid "branch_name-YYYYMMDD-HHMMSS".
Redmine generates pseudo revision number, and using this number for auto close issue text is very difficult.
Redmine supports scmid of Git, Mercurial, Bazaar and Darcs.
Redmine supports "commit:scmid" in wiki. You can use first some letters of scmid.
For example, "commit:branch_name-YYYYMMDD-HHMM".

Actions

Also available in: Atom PDF