Summary

Fix references to non-existant variables in rbt patch.

Review Request #4022 — Created April 5, 2013 and submitted April 8, 2013, 3:30 p.m.

Information

Owner

sparkiegeek

Repository

RBTools

Branch

master

Bugs

Depends On

Reviewers

Groups

rbtools

People

Description

rbt patch in RBTools 0.5 is pretty broken - it was using attributes which didn't exist (Patch.repository_info and Patch.tool instead of the local variables that it creates).

I was getting the following error messages before these fixes (not all at once)

CRITICAL:root:'Patch' object has no attribute 'repository_info'
CRITICAL:root:'Patch' object has no attribute 'tool'
CRITICAL:root:'NoneType' object has no attribute 'get_diffs'

Testing Done

ran rbt patch successfully
flake8 patch.py

This is a review from Review Bot.
  Tool: PEP8 Style Checker
  Processed Files:
    rbtools/commands/patch.py
  Ignored Files:

Ship it!

```
Ship It!
```

Status: Closed (submitted)

Change Summary:

Pushed to master (bd1438640fa3c4c1e7ec66c07e069c6f5f54973a). Thanks!