Merge lp://staging/~bregma/geis/pygies-python3 into lp://staging/~themuso/geis/python3
Proposed by
Stephen M. Webb
| Status: | Merged |
|---|---|
| Merged at revision: | 294 |
| Proposed branch: | lp://staging/~bregma/geis/pygies-python3 |
| Merge into: | lp://staging/~themuso/geis/python3 |
| Diff against target: |
93 lines (+11/-10) 2 files modified
python/Makefile.am (+1/-1) python/geis/geis_v2.py (+10/-9) |
| To merge this branch: | bzr merge lp://staging/~bregma/geis/pygies-python3 |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Luke Yelavich | Pending | ||
|
Review via email:
|
|||
Description of the change
(1) Strings returned through the ctypes interface were byte strings and needed to be decoded as utf-8 (otherwise a mismatch occurs during dictionary lookups).
(2) Wrong variable name was used in the link line in Makefile.am so no python runtime was linked in (problem may only show up using the GNU gold linker)
To post a comment you must log in.
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
