Bug 222380
| Summary: | devel/llvm*: Build and install pp-trace executable when building llvm with extras selected | ||||||
|---|---|---|---|---|---|---|---|
| Product: | Ports & Packages | Reporter: | bye <owen94012> | ||||
| Component: | Individual Port(s) | Assignee: | Brooks Davis <brooks> | ||||
| Status: | Closed Not Accepted | ||||||
| Severity: | Affects Only Me | CC: | brooks | ||||
| Priority: | --- | Keywords: | easy, feature, needs-patch, needs-qa | ||||
| Version: | Latest | Flags: | koobs:
maintainer-feedback?
(brooks) |
||||
| Hardware: | Any | ||||||
| OS: | Any | ||||||
| Attachments: |
|
||||||
Assign to maintainer. Patch should include changes for all devel/llvm* ports. This works, but will cause a ton of rebuilding globally if I commit. Are there other tools you'd like to add to the set? A commit references this bug: Author: brooks Date: Tue Sep 19 23:44:36 UTC 2017 New revision: 450176 URL: https://svnweb.freebsd.org/changeset/ports/450176 Log: Disable LLDB builds on 10.x as they don't link. Attempt to correct default targets. [0] Fix more references to clang-format50 and use a more future proof patching approach. [1] Build and install pp-trace with the EXTRAS target. PR: 219779 [0], 220995 [1], 222380 [2] Changes: head/devel/llvm50/Makefile head/devel/llvm50/files/clang-patch-tools_clang_tools_clang-format_clang-format-sublime.py head/devel/llvm50/files/clang-patch-tools_clang_tools_clang-format_clang-format.py head/devel/llvm50/files/clang-patch-tools_clang_tools_clang-format_git-clang-format head/devel/llvm50/pkg-plist A commit references this bug: Author: brooks Date: Wed Sep 20 01:33:42 UTC 2017 New revision: 450183 URL: https://svnweb.freebsd.org/changeset/ports/450183 Log: Don't install a wrapper for the not installed pp-trace program. GC an obsolete patch. PR: 222380 Changes: head/devel/llvm50/Makefile head/devel/llvm50/files/clang-patch-svn-r280672 head/devel/llvm50/pkg-plist I've reverted the previous change. The patch installs a wrapper for pp-trace which is not built so the wrapper is useless. I'd be happy to accept a patch that actually builds and installs pp-trace, but this didn't do it and I'm insufficiently motivated to figure out the cmake bits to do it my self. |
Created attachment 186445 [details] devel-llvm40-makefile-diff /devel/llvm40 has a some extra tools; pp-trace is one of them. There's a Makefile diff attached. Making that change then the pp-trace executable will be built, then it should be easy to put it in the proper place with the install command.