Skip to content

Commit

Permalink
Merge pull request #903 from kentsommer/patch-1
Browse files Browse the repository at this point in the history
Fix python3 support for builder.py
  • Loading branch information
dirk-thomas authored Dec 8, 2017
2 parents a7f81d0 + d76ba7c commit 80005c1
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions python/catkin/builder.py
Original file line number Diff line number Diff line change
Expand Up @@ -274,11 +274,11 @@ def get_multiarch():
p = subprocess.Popen(
['gcc', '-print-multiarch'],
stdout=subprocess.PIPE, stderr=subprocess.PIPE)
out, err = p.communicate()
out = p.communicate()[0].decode('utf-8')
if p.returncode != 0:
out, err = subprocess.Popen(
out = subprocess.Popen(
['dpkg-architecture', '-qDEB_HOST_MULTIARCH'],
stdout=subprocess.PIPE, stderr=subprocess.PIPE).communicate()
stdout=subprocess.PIPE, stderr=subprocess.PIPE).communicate()[0].decode('utf-8')

# be sure of returning empty string or a valid multiarch tuple format
assert(not out.strip() or out.strip().count('-') == 2)
Expand Down

0 comments on commit 80005c1

Please sign in to comment.