Description
Jan Beich
2017-01-18 11:52:15 UTC
The fix here would probably be similar to bug 216253. Created attachment 179176 [details]
proposed fix: PyList_SET_ITEM -> PyList_SetItem
Comment on attachment 179176 [details] proposed fix: PyList_SET_ITEM -> PyList_SetItem Looks OK and builds fine[1]. Can you land under "portmgr blanket" as a maintainer approval? Also add "MFH: 2017Q1" to commit message. [1] 10.3 i386: http://sprunge.us/NETT /projects/clang400-import i386: http://sprunge.us/JjTD A commit references this bug: Author: tcberner Date: Sat Jan 21 22:10:43 UTC 2017 New revision: 432089 URL: https://svnweb.freebsd.org/changeset/ports/432089 Log: Fix graphics/py-poppler-qt4: fails to build with clang 4.0 PyList_SET_ITEM() is supposed to be void, but as a macro it's an expression and has a pointer value. Clang 4.0 trips on the bogus pointer comparison; given the comparison, it should be PyList_SetItem() which returns -1 on failure. PR: 216228 Approved by: portmgr blanket, rakuco (mentor) MFH: 2017Q1 Changes: head/graphics/py-poppler-qt4/Makefile head/graphics/py-poppler-qt4/files/ head/graphics/py-poppler-qt4/files/patch-types.sip Still compiles with clang 3.9.1 so fix should be ok ... A commit references this bug: Author: tcberner Date: Mon Jan 23 20:40:08 UTC 2017 New revision: 432289 URL: https://svnweb.freebsd.org/changeset/ports/432289 Log: MFH: r432089 Fix graphics/py-poppler-qt4: fails to build with clang 4.0 PyList_SET_ITEM() is supposed to be void, but as a macro it's an expression and has a pointer value. Clang 4.0 trips on the bogus pointer comparison; given the comparison, it should be PyList_SetItem() which returns -1 on failure. PR: 216228 Approved by: portmgr blanket, rakuco (mentor) Approved by: ports-secteam (junovitch), rakuco (mentor) Changes: _U branches/2017Q1/ branches/2017Q1/graphics/py-poppler-qt4/Makefile branches/2017Q1/graphics/py-poppler-qt4/files/ |