Project

General

Profile

Actions

Defect #28920

closed

Redmine::VERSION::revision should take subversion_command setting into account

Added by Go MAEDA almost 6 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Administration
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed
Affected version:

Description

The revision number may not be shown on admin_info page because the execution of the svn command fails in some cases.

Redmine::VERSION::revision method should take scm_subversion_command setting into account.

Index: lib/redmine/version.rb
===================================================================
--- lib/redmine/version.rb    (revision 17354)
+++ lib/redmine/version.rb    (working copy)
@@ -17,7 +17,7 @@
       if File.directory?(File.join(Rails.root, '.svn'))
         begin
           path = Redmine::Scm::Adapters::AbstractAdapter.shell_quote(Rails.root.to_s)
-          if `svn info --xml #{path}` =~ /revision="(\d+)"/
+          if `#{Redmine::Scm::Adapters::SubversionAdapter.client_command} info --xml #{path}` =~ /revision="(\d+)"/
             return $1.to_i
           end
         rescue

Related issues

Related to Redmine - Feature #7517: Configurable path of executable for scm adaptersClosedToshi MARUYAMA2011-02-02

Actions
Related to Redmine - Defect #32110: "already initialized constant Redmine::Scm::Adapters::SubversionAdapter::SVN_BIN" warning when executing rake tasksClosedGo MAEDA

Actions
Actions #1

Updated by Go MAEDA almost 6 years ago

  • Related to Feature #7517: Configurable path of executable for scm adapters added
Actions #2

Updated by Go MAEDA almost 6 years ago

  • Target version set to 4.1.0
Actions #3

Updated by Go MAEDA almost 6 years ago

  • Subject changed from Redmine::VERSION::revision method may fail due to hardcoded svn command name to Redmine::VERSION::revision should take subversion_command_setting into account
  • Status changed from New to Closed
  • Assignee set to Go MAEDA
  • Target version changed from 4.1.0 to 4.0.0
  • Resolution set to Fixed

Committed. The patch was provided by Yuichi Harada. Thank you for the contribution.

Actions #4

Updated by Go MAEDA over 5 years ago

  • Subject changed from Redmine::VERSION::revision should take subversion_command_setting into account to Redmine::VERSION::revision should take subversion_command setting into account
Actions #5

Updated by Go MAEDA over 4 years ago

  • Related to Defect #32110: "already initialized constant Redmine::Scm::Adapters::SubversionAdapter::SVN_BIN" warning when executing rake tasks added
Actions

Also available in: Atom PDF