プロジェクト

全般

プロフィール

Vote #63412

完了

CVS: add support for modules names with spaces

Admin Redmine さんが3年以上前に追加. 3年以上前に更新.

ステータス:
Closed
優先度:
通常
担当者:
-
カテゴリ:
SCM_3
対象バージョン:
開始日:
2008/06/12
期日:
進捗率:

0%

予定工数:
category_id:
3
version_id:
2
issue_org_id:
1434
author_id:
1233
assigned_to_id:
0
comments:
4
status_id:
5
tracker_id:
1
plus1:
0
affected_version:
closed_on:
affected_version_id:
3
ステータス-->[Closed]

説明

redmine fails to import changes from CVS module with space in the name:
For example module: "module with spaces" - redmine generate command which makes cvs to treat the module name as several modules separated with spaces.

Solution is to put module name into the quotes:

Here is patch to fix this:
file: lib/redmine/scm/adapters/cvs_adapter.rb

68c68
<           cmd << " #{path_with_project}"
---
>           cmd << ' "'+"#{path_with_project}"+'"'
113c113
<           cmd << " #{path_with_project}"
---
>           cmd << ' "'+"#{path_with_project}"+'"'

journals

Here is full fix - for all cvs 8 calls the adapter does:

<pre>
68c68
< cmd << " #{path_with_project}"
---
> cmd << ' "'+"#{path_with_project}"+'"'
113c113
< cmd << " #{path_with_project}"
---
> cmd << ' "'+"#{path_with_project}"+'"'
233c233
< cmd = "#{CVS_BIN} -d #{root_url} rdiff -u -r#{identifier_to} -r#{identifier_from} #{path_with_project}"
---
> cmd = "#{CVS_BIN} -d #{root_url} rdiff -u -r#{identifier_to} -r#{identifier_from} \"#{path_with_project}\""
248c248
< cmd = "#{CVS_BIN} -d #{root_url} co -r#{identifier} -p #{path_with_project}"
---
> cmd = "#{CVS_BIN} -d #{root_url} co -r#{identifier} -p \"#{path_with_project}\""
261c261
< cmd = "#{CVS_BIN} -d #{root_url} rannotate -r#{identifier} #{path_with_project}"
---
> cmd = "#{CVS_BIN} -d #{root_url} rannotate -r#{identifier} \"#{path_with_project}\""
</pre>
--------------------------------------------------------------------------------
Fixed in r1527 using the shell_quote method instead.
--------------------------------------------------------------------------------

--------------------------------------------------------------------------------

--------------------------------------------------------------------------------


related_issues

duplicates,Closed,1642,CVS: add support for modules names with spaces (was #1434).

Admin Redmine さんが3年以上前に更新

  • カテゴリSCM_3 にセット
  • 対象バージョン0.8_2 にセット

他の形式にエクスポート: Atom PDF

いいね!0
いいね!0