What version are you running?
RBTools-0.6-py2.6.egg
What's the URL of the page containing the problem?
n/a
What steps will reproduce the problem?
1. using perforce, add a new file (p4 add myfile; p4 submit) (change #1)
2. branch the file, p4 integ myfile myfile2 (or whatever); p4 submit (change #2)
3. rbt post 2
What is the expected output? What do you see instead?
Expected: A review is created and the diff submitted.
Observed: CRITICAL: Unsupported action type "branch" for myfile2
What operating system are you using? What browser?
submitting from RHEL Linux 5.5; browser - any.
Please provide any additional information below.
This appears to be due to the following in clients/perforce.py (around line 644, in method _compute_range_changes:
if action not in ('edit', 'integrate', 'add', 'delete',
'move/add', 'move/delete'):
raise Exception('Unsupported action type "%s" for %s' %
(action, depot_file))
if action == 'integrate':
action = 'edit'
elif action == 'branch':
action = 'add'
Locally we have resolved the issue (successfully submitting the review) with the following naive change, replacing the above with:
if action == 'integrate':
action = 'edit'
elif action == 'branch':
action = 'add'
if action not in ('edit', 'add', 'delete',
'move/add', 'move/delete'):
raise Exception('Unsupported action type "%s" for %s' %
(action, depot_file))
complete output of example:
$ tools/temp>echo data > a_file_please_ignore
$ tools/temp>p4 add a_file_please_ignore
//depot/<snip>/a_file_please_ignore#1 - opened for add
$ tools/temp>p4 submit a_file_please_ignore
Change 1205953 created with 1 open file(s).
Submitting change 1205953.
Locking 1 files ...
Change 1205953 submitted.
$ tools/temp>p4 integ a_file_please_ignore a_file_please_ignore.2
//depot/<snip>/a_file_please_ignore.2#1 - branch/sync from //depot/<snip>/a_file_please_ignore#1
$ tools/temp>p4 submit a_file_please_ignore.2
Change 1205956 created with 1 open file(s).
Submitting change 1205956.
Locking 1 files ...
branch //depot/<snip>/a_file_please_ignore.2#1
Change 1205956 submitted.
$ tools/temp>rbt post 1205956
ERROR: Could not load SCM Client "svn": No module named etree
ERROR: Could not load SCM Client "git": No module named etree
ERROR: Could not load SCM Client "mercurial": No module named uuid
Generating diff for range of submitted changes: 1205955 to 1205956
CRITICAL: Unsupported action type "branch" for //depot/<snip>/a_file_please_ignore.2
with the naive fix:
$ tools/temp>rbt post 1205956
ERROR: Could not load SCM Client "svn": No module named etree
ERROR: Could not load SCM Client "git": No module named etree
ERROR: Could not load SCM Client "mercurial": No module named uuid
Generating diff for range of submitted changes: 1205955 to 1205956
Review request #1421 posted.
...
Draft diff
This diff is part of your current draft. Other users will not see this diff until you publish your draft.
Files Changed:
//depot/<snip>/a_file_please_ignore.2: 1 change [ new content ]
