On 28/12/2018 02:48, Bruce Dubbs via blfs-dev wrote: > On 12/27/2018 02:20 AM, Pierre Labastie via blfs-dev wrote: >> Hi, >> >> ATM, the book stipulates that Python2 is required for LLVM. But I've been >> able >> to build and test LLVM-7.0.1 with only Python3 installed on the system: only >> 11 tests out of more than 33000 fail, and missing python(2) seems to lead to >> only 2 failures out of 11. >> >> I've not tried to build the doc, but since sphinx is Python3 compatible, I >> guess this should not be a problem. >> >> The only reason to want Python2 for llvm are the tools: a lot of programs in >> the utils directory still have #!/usr/bin/env python (or python2.7). >> >> So I want to change Python2 to "optional (run-time, for using some tools)". >> Is >> it OK with you? > > Before I went on my short vacation, I updated llvm to version 7.0.1. I was > doing some updates on my laptop also in addition to my regular development > system and built it without P2. I intended to move P2 to optional. I just > missed it. > > I was thinking about adding a comment that it is not used if P3 is present. > Is there something you know about in llvm that explicitly needs P2? >
As said, some of the tools in /usr/bin or /usr/share seem to require P2. Four tools explicitely have "#!/usr/bin/env/ python2.7", which could mean they explicitely require P2: /usr/share/opt-viewer/opt-diff.py /usr/share/opt-viewer/opt-stats.py /usr/share/opt-viewer/opt-viewer.py /usr/share/opt-viewer/optrecord.py Some have "#!/usr/bin/env python", which does not mean that they are not compatible with P3 (but looking at how they use the "print" function, they aren't): /usr/bin/git-clang-format /usr/bin/hmaptool /usr/bin/scan-view /usr/share/clang/clang-format-diff.py /usr/share/scan-view/Reporter.py /usr/share/scan-view/startfile.py (list possibly incomplete) So, P2 is optional for the build an for antyhing in the book, but if people want to use llvm as their development platform (which might mean using opt-viewer and scan-view), I think they need P2 ATM. Pierre -- http://lists.linuxfromscratch.org/listinfo/blfs-dev FAQ: http://www.linuxfromscratch.org/blfs/faq.html Unsubscribe: See the above information page
