Project

General

Profile

Actions

Patch #4113

closed

Directory listing with CVS v1.11 patch

Added by alexander smishlajev over 14 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
SCM
Target version:
-
Start date:
2009-10-26
Due date:
% Done:

0%

Estimated time:

Description

Directory listing with CVS SCM uses rls command which was not available in CVS versons prior to 1.12.

Could the CVS adapter use cvs -q -n rlog -R to get the repository contents listing?


Files

cvs_adapter.rb.patch (4.07 KB) cvs_adapter.rb.patch alexander smishlajev, 2009-10-27 11:09
Actions #1

Updated by Jean-Philippe Lang over 14 years ago

  • Status changed from New to Closed
  • Resolution set to Invalid

Closing this defect since RedmineRepositories clearly states that cvs 1.12 is required.
Feel free to submit a patch.

Actions #2

Updated by alexander smishlajev over 14 years ago

Well, but the "stable" CVS version still is 1.11, and 1.11 is shipped with the latest production releases of FreeBSD.

The patch is attached.

I admit that the variant requiring CVS version 1.12 is much more effective. Perhaps Redmine could check the cvs version and then choose which code to run?

Actions #3

Updated by Jean-Philippe Lang over 14 years ago

  • Tracker changed from Defect to Patch
Actions #4

Updated by Jean-Philippe Lang over 14 years ago

  • Subject changed from Repository browsing fails with CVS v1.11 to Directory listing with CVS v1.11 patch
  • Category set to SCM
  • Status changed from Reopened to New
Actions #5

Updated by Gareth Sylvester-Bradley over 13 years ago

FWIW, I've now been using this patch successfully for several months to enable repository browsing for our CVS 1.11 server. Thanks!

This week I just needed to tweak the regular expression that parses the revision info very slightly to play nice with patch #6090 to make downloading binary files from a CVS repository work when running Redmine on Windows.

Change line 65 from:

          re_loginfo = "revision ([0-9.]+)\s*\n"
to:
          re_loginfo = "revision ([0-9.]+)\s*\r?\n"

Actions #6

Updated by Go MAEDA over 2 years ago

  • Status changed from New to Closed

Closing this issue because CVS 1.11 is too old.

Actions

Also available in: Atom PDF