Merge lp://qastaging/~yofel/ubuntu/maverick/samba/lp-623798 into lp://qastaging/ubuntu/maverick/samba

Proposed by Philip Muškovac
Status: Merged
Merge reported by: Mathias Gug
Merged at revision: not available
Proposed branch: lp://qastaging/~yofel/ubuntu/maverick/samba/lp-623798
Merge into: lp://qastaging/ubuntu/maverick/samba
Diff against target: 44 lines (+9/-3)
2 files modified
debian/changelog (+6/-0)
debian/source_samba.py (+3/-3)
To merge this branch: bzr merge lp://qastaging/~yofel/ubuntu/maverick/samba/lp-623798
Reviewer Review Type Date Requested Status
Mathias Gug Approve
Review via email: mp+34363@code.qastaging.launchpad.net

Description of the change

Fix a typo in the apport hook that causes it to crash when choosing 'others' for the client question:

*** Which clients are failing to connect?

Choices:
  1: Windows
  2: Ubuntu
  3: Both
  4: Other
  C: Cancel
Please choose (1/2/3/4/C):4
hook /usr/share/apport/package-hooks//source_samba.py crashed:
Traceback (most recent call last):
  File "/usr/lib/python2.6/dist-packages/apport/report.py", line 636, in add_hooks_info
    symb['add_info'](self, ui)
  File "/usr/share/apport/package-hooks//source_samba.py", line 95, in add_info
    if repsonse[0] == 3:
NameError: global name 'repsonse' is not defined

To post a comment you must log in.
Revision history for this message
Mathias Gug (mathiaz) wrote :

Thanks. I've updated the diff against the latest version of the samba package in maverick and uploaded it to maverick.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: