Project

General

Profile

Actions

Patch #3159

closed

Mercurial (Hg) integration doesn't work if redmine is installed in folder path containing space

Added by Afriza N. Arief about 15 years ago. Updated about 15 years ago.

Status:
Closed
Priority:
High
Assignee:
-
Category:
SCM
Target version:
Start date:
2009-04-11
Due date:
% Done:

0%

Estimated time:

Description

Mercurial (Hg) integration doesn't work if redmine is installed in folder path containing space. The problem is because the template file path is not properly quoted. (e.g. "C:/Program Files/BitNami Redmine Stack/apps/redmine/lib/redmine/scm/adapters/mercurial/hg-template-1.0.tmpl" )

affected branch: 0.8-stable
affected revision: svn r2670
affected file:
http://www.redmine.org/projects/redmine/repository/revisions/2670/entry/branches/0.8-stable/lib/redmine/scm/adapters/mercurial_adapter.rb
affected line: 108

Note: I am beginner in Ruby and redmine. The patch I attached will only quote the path and won't convert forward slash to backslash. Please adjust the patch as deemed necessary.


Files

Actions #1

Updated by Afriza N. Arief about 15 years ago

  • % Done changed from 100 to 0
Actions #2

Updated by Afriza N. Arief about 15 years ago

the syndrome is the following error when trying to access Mercurial repository

undefined method `last’ for nil:NilClass
RAILS_ROOT: C:/Program Files/BitNami Redmine Stack/apps/redmine
lib/redmine/scm/adapters/mercurial_adapter.rb:72:in `info’
lib/redmine/scm/adapters/abstract_adapter.rb:150:in `retrieve_root_url’
lib/redmine/scm/adapters/abstract_adapter.rb:54:in `initialize’
app/models/repository.rb:41:in `new’
app/models/repository.rb:41:in `scm’
app/models/repository/mercurial.rb:56:in `fetch_changesets’
app/controllers/repositories_controller.rb:69:in `show’

Actions #4

Updated by Jean-Philippe Lang about 15 years ago

  • Status changed from New to Resolved
  • Target version set to 0.8.4

Fix committed in r2679.
Thanks for pointing this out.

Actions #5

Updated by Jean-Philippe Lang about 15 years ago

  • Status changed from Resolved to Closed

Merged in 0.8-stable in r2681.

Actions

Also available in: Atom PDF