Merge lp://staging/~brendan-donegan/checkbox/checkbox_touch_py_init into lp://staging/checkbox

Proposed by Brendan Donegan
Status: Merged
Approved by: Maciej Kisielewski
Approved revision: 4156
Merged at revision: 4191
Proposed branch: lp://staging/~brendan-donegan/checkbox/checkbox_touch_py_init
Merge into: lp://staging/checkbox
Diff against target: 21 lines (+17/-0)
1 file modified
checkbox-touch/py/__init__.py (+17/-0)
To merge this branch: bzr merge lp://staging/~brendan-donegan/checkbox/checkbox_touch_py_init
Reviewer Review Type Date Requested Status
Maciej Kisielewski (community) Approve
Paul Larson Approve
Review via email: mp+282195@code.staging.launchpad.net

Description of the change

When I was trying to use the build-me script I kept hitting:

Traceback (most recent call last):
   File "./build-me", line 14, in <module>
   from py.embedded_providers import EmbeddedProvider1PlugInCollection
ImportError: No module named 'py.embedded_providers'

This turned out to be because there was no __init__.py in checkbox-touch/py

Theoretically there shouldn't need to be, but to be safe and avoid anyone else having the same problem, let's just add it there.

To post a comment you must log in.
Revision history for this message
Paul Larson (pwlars) wrote :

Shouldn't be any harm in this, +1

review: Approve
Revision history for this message
Po-Hsu Lin (cypressyew) wrote :

Interesting, I tried this on Xenial with the patch, but it works fine with ./build-me in checkbox-touch directory.

Revision history for this message
Maciej Kisielewski (kissiel) wrote :

+1
Let's land it

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