Add support for ClearCase lbtype comparison

Review Request #6500 — Created Oct. 24, 2014 and submitted

Information

RBTools
release-0.6.x
ea7f58d...

Reviewers

Identify all elements belonging to previous and current labels thanks to cleartool find -exec cleartool desc command.
Then for each element, attach these versions or default one "/main/0" (when a file was added or removed).
Add only versions not matching to the changeset.
When one label is provided, set a temporary label on current directory before proceeding to label comparison. This label is removed when a error happen eg. current directory contains checked out elements.

tested done within a clearcase dynamic view and a snapshot view from a linux machine

rbt diff generation between one label and current => OK
rbt diff generation between two labels => OK

rbt post on a reviewboard server 2.0.0 => OK
rbt post on a reviewboard server 1.7.6 with this patch using rbtools release-0.6.2 + 5defd0c + 30b0194

error message when root dir is checked out => OK
error message when any element under root dir is checked out => OK

Label comparison failed because:
ClearCase backend cannot set label when some elements are checked out:
<element xxx>

Description From Last Updated

Col: 54 E502 the backslash is redundant between brackets

reviewbotreviewbot

Col: 31 E127 continuation line over-indented for visual indent

reviewbotreviewbot

Col: 53 E502 the backslash is redundant between brackets

reviewbotreviewbot

Col: 21 E123 closing bracket does not match indentation of opening bracket's line

reviewbotreviewbot

Col: 80 E501 line too long (83 > 79 characters)

reviewbotreviewbot

Col: 80 E501 line too long (85 > 79 characters)

reviewbotreviewbot

Col: 51 E128 continuation line under-indented for visual indent

reviewbotreviewbot

Col: 80 E501 line too long (80 > 79 characters)

reviewbotreviewbot

Col: 21 E128 continuation line under-indented for visual indent

reviewbotreviewbot

Col: 29 E126 continuation line over-indented for hanging indent

reviewbotreviewbot

Col: 29 E126 continuation line over-indented for hanging indent

reviewbotreviewbot

Col: 80 E501 line too long (81 > 79 characters)

reviewbotreviewbot

Col: 80 E501 line too long (81 > 79 characters)

reviewbotreviewbot
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        rbtools/clients/clearcase.py
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        rbtools/clients/clearcase.py
    
    
  2. rbtools/clients/clearcase.py (Diff revision 1)
     
     
    Show all issues
    Col: 54
     E502 the backslash is redundant between brackets
    
  3. rbtools/clients/clearcase.py (Diff revision 1)
     
     
    Show all issues
    Col: 31
     E127 continuation line over-indented for visual indent
    
  4. rbtools/clients/clearcase.py (Diff revision 1)
     
     
    Show all issues
    Col: 53
     E502 the backslash is redundant between brackets
    
  5. rbtools/clients/clearcase.py (Diff revision 1)
     
     
    Show all issues
    Col: 21
     E123 closing bracket does not match indentation of opening bracket's line
    
  6. rbtools/clients/clearcase.py (Diff revision 1)
     
     
    Show all issues
    Col: 80
     E501 line too long (83 > 79 characters)
    
  7. rbtools/clients/clearcase.py (Diff revision 1)
     
     
    Show all issues
    Col: 80
     E501 line too long (85 > 79 characters)
    
  8. rbtools/clients/clearcase.py (Diff revision 1)
     
     
    Show all issues
    Col: 51
     E128 continuation line under-indented for visual indent
    
  9. rbtools/clients/clearcase.py (Diff revision 1)
     
     
    Show all issues
    Col: 80
     E501 line too long (80 > 79 characters)
    
  10. rbtools/clients/clearcase.py (Diff revision 1)
     
     
    Show all issues
    Col: 21
     E128 continuation line under-indented for visual indent
    
  11. 
      
DE
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        rbtools/clients/clearcase.py
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        rbtools/clients/clearcase.py
    
    
  2. rbtools/clients/clearcase.py (Diff revision 2)
     
     
    Show all issues
    Col: 29
     E126 continuation line over-indented for hanging indent
    
  3. rbtools/clients/clearcase.py (Diff revision 2)
     
     
    Show all issues
    Col: 29
     E126 continuation line over-indented for hanging indent
    
  4. 
      
DE
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        rbtools/clients/clearcase.py
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        rbtools/clients/clearcase.py
    
    
  2. rbtools/clients/clearcase.py (Diff revision 3)
     
     
    Show all issues
    Col: 80
     E501 line too long (81 > 79 characters)
    
    1. revert indentation, IMHO would rather like a too long line (few characters) rather than a bad indentation

  3. rbtools/clients/clearcase.py (Diff revision 3)
     
     
    Show all issues
    Col: 80
     E501 line too long (81 > 79 characters)
    
  4. 
      
david
  1. Ship It!

  2. 
      
DE
Review request changed
Status:
Completed
Change Summary:
Pushed to release-0.6.x (22aaf41)