Source: blockdiag
Version: 1.1.6-1
Severity: serious
Justification: fails to build from source
blockdiag FTBFS:
| set -e; \
| PYTHONPATH=/build/blockdiag-VLK4f1/blockdiag-1.1.6/src nosetests -d
|
................................................................................................................................................................................................................................................................F..........................................
| ======================================================================
| FAIL: blockdiag.tests.test_pep8.test_pep8
| ----------------------------------------------------------------------
| Traceback (most recent call last):
| File "/usr/lib/python2.7/dist-packages/nose/case.py", line 197, in runTest
| self.test(*self.arg)
| File
"/build/blockdiag-VLK4f1/blockdiag-1.1.6/src/blockdiag/tests/test_pep8.py",
line 38, in test_pep8
| assert errors + warnings == 0, message
| AssertionError: pep8: 21 errors / 0 warnings
| 0 = <module 'pep8' from
'/usr/lib/python2.7/dist-packages/pep8.pyc'>.get_count('W')
| 'pep8: 21 errors / 0 warnings' = 'pep8: %d errors / %d warnings' % (21, 0)
| print 'pep8: 21 errors / 0 warnings'
| >> assert 21 + 0 == 0, 'pep8: 21 errors / 0 warnings'
|
| -------------------- >> begin captured stdout << ---------------------
| /build/blockdiag-VLK4f1/blockdiag-1.1.6/src/blockdiag/builder.py:505:51: E272
multiple spaces before keyword
| for edge in (e for e in self.diagram.edges if e.style != 'none'):
| ^
| /build/blockdiag-VLK4f1/blockdiag-1.1.6/src/blockdiag/builder.py:509:48: E272
multiple spaces before keyword
| for edge in (e for e in group.edges if e.style != 'none'):
| ^
| /build/blockdiag-VLK4f1/blockdiag-1.1.6/src/blockdiag/drawer.py:78:38: E272
multiple spaces before keyword
| for edge in (e for e in edges if e.style != 'none'):
| ^
| /build/blockdiag-VLK4f1/blockdiag-1.1.6/src/blockdiag/elements.py:591:74:
E502 the backslash is redundant between brackets
| kwargs = dict([str(unquote(attr.name)), unquote(attr.value)] \
| ^
| /build/blockdiag-VLK4f1/blockdiag-1.1.6/src/blockdiag/metrics.py:45:26: E712
comparison to False should be 'if cond is False:' or 'if not cond:'
| if self.stroking == False:
| ^
|
/build/blockdiag-VLK4f1/blockdiag-1.1.6/src/blockdiag/imagedraw/simplesvg.py:206:52:
E502 the backslash is redundant between brackets
| self.path.append('A%s,%s %s %s %s %s %s' % \
| ^
|
/build/blockdiag-VLK4f1/blockdiag-1.1.6/src/blockdiag/imagedraw/simplesvg.py:210:52:
E502 the backslash is redundant between brackets
| self.path.append('a%s,%s %s %s %s %s %s' % \
| ^
|
/build/blockdiag-VLK4f1/blockdiag-1.1.6/src/blockdiag/imagedraw/filters/linejump.py:131:52:
E712 comparison to True should be 'if cond is True:' or 'if cond:'
| if 'jump' in kwargs and kwargs['jump'] == True:
| ^
|
/build/blockdiag-VLK4f1/blockdiag-1.1.6/src/blockdiag/noderenderer/__init__.py:104:27:
E711 comparison to None should be 'if cond is not None:'
| if self.node.icon != None and kwargs.get('shadow') != True:
| ^
|
/build/blockdiag-VLK4f1/blockdiag-1.1.6/src/blockdiag/noderenderer/__init__.py:119:31:
E711 comparison to None should be 'if cond is not None:'
| if self.node.numbered != None and kwargs.get('shadow') != True:
| ^
|
/build/blockdiag-VLK4f1/blockdiag-1.1.6/src/blockdiag/tests/test_builder_edge.py:49:31:
E711 comparison to None should be 'if cond is None:'
| assert edge.thick == None
| ^
|
/build/blockdiag-VLK4f1/blockdiag-1.1.6/src/blockdiag/tests/test_builder_edge.py:57:31:
E711 comparison to None should be 'if cond is None:'
| assert edge.thick == None
| ^
|
/build/blockdiag-VLK4f1/blockdiag-1.1.6/src/blockdiag/tests/test_utils_fontmap.py:291:52:
E502 the backslash is redundant between brackets
| _config = ("[fontmap]\nserif-bold: %s\n" + \
| ^
| /build/blockdiag-VLK4f1/blockdiag-1.1.6/src/blockdiag/tests/utils.py:85:73:
E502 the backslash is redundant between brackets
| raise RuntimeError('edge (%s -> %s) is not found' % \
| ^
| /build/blockdiag-VLK4f1/blockdiag-1.1.6/src/blockdiag/tests/utils.py:89:50:
E272 multiple spaces before keyword
| for node in (n for n in diagram.nodes if n.drawable):
| ^
|
/build/blockdiag-VLK4f1/blockdiag-1.1.6/src/blockdiag/utils/TextFolder.py:47:34:
E272 multiple spaces before keyword
| return len([x for x in string if is_zenkaku(x)])
| ^
|
/build/blockdiag-VLK4f1/blockdiag-1.1.6/src/blockdiag/utils/TextFolder.py:61:34:
E272 multiple spaces before keyword
| return len([x for x in string if not is_zenkaku(x)])
| ^
|
/build/blockdiag-VLK4f1/blockdiag-1.1.6/src/blockdiag/utils/bootstrap.py:111:77:
E502 the backslash is redundant between brackets
| help='do not make transparent background of diagram ' +\
| ^
| /build/blockdiag-VLK4f1/blockdiag-1.1.6/src/blockdiag/utils/ellipse.py:27:58:
E502 the backslash is redundant between brackets
| phi += du / math.sqrt((a * math.sin(phi)) ** 2 + \
| ^
|
/build/blockdiag-VLK4f1/blockdiag-1.1.6/src/blockdiag/utils/rst/directives.py:233:56:
E272 multiple spaces before keyword
| descriptions = [n.to_desctable() for n in nodes if n.drawable]
| ^
|
/build/blockdiag-VLK4f1/blockdiag-1.1.6/src/blockdiag/utils/rst/directives.py:255:56:
E272 multiple spaces before keyword
| descriptions = [e.to_desctable() for e in edges if e.style != 'none']
| ^
| 8 E272 multiple spaces before keyword
| 7 E502 the backslash is redundant between brackets
| 4 E711 comparison to None should be 'if cond is not None:'
| 2 E712 comparison to False should be 'if cond is False:' or 'if not
cond:'
| pep8: 21 errors / 0 warnings
|
| --------------------- >> end captured stdout << ----------------------
|
| ----------------------------------------------------------------------
| Ran 299 tests in 10.245s
|
| FAILED (failures=1)
| make[1]: *** [override_dh_auto_test] Error 1
--
Jakub Wilk
--
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]
