Bug 268745
| Summary: | devel/creduce: Update to latest commit (20220927) | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | Ports & Packages | Reporter: | Daniel Engberg <diizzy> | ||||||||
| Component: | Individual Port(s) | Assignee: | Daniel Engberg <diizzy> | ||||||||
| Status: | Closed FIXED | ||||||||||
| Severity: | Affects Only Me | CC: | brooks, diizzy, pkubaj, tcberner | ||||||||
| Priority: | --- | Flags: | bugzilla:
maintainer-feedback?
(swills) |
||||||||
| Version: | Latest | ||||||||||
| Hardware: | Any | ||||||||||
| OS: | Any | ||||||||||
| Attachments: |
|
||||||||||
I'll try it in Poudriere too once by buildboxes are available Pitor, is the PPC stuff still needed? Created attachment 239253 [details]
Patch for creduce v2
Add flex as build dependency
BROKEN_FreeBSD_12_powerpc64 is probably still needed, but I doubt anyone still uses powerpc64 with 12, given the vast improvements done in 13. I don't and I haven't seen any ports fixes for powerpc64 on 12 in the past year or so. -mabi=elfv2 is implicitly set on powerpc64 by LLVM since 9.0, it was set here because this port uses 8.0. And I just tested that LLD_UNSAFE for powerpc is not needed anymore with your patch. patch barfed on the removal of devel/creduce/files/builtin_is_constant_evaluated.h, but after manually applying, this looks good and worked with a simple example. Thanks! Created attachment 239263 [details]
Patch for creduce v3
Add license info
Add USES= compiler:c++14-lang
Adjust PowerPC related build settings based on pkubaj's feedback
Use ${LLVM_PREFIX} as suggested by tcberner
Poudriere testport OK 12.3-RELEASE (amd64) Poudriere testport OK 13.1-RELEASE (i386) A commit in branch main references this bug: URL: https://cgit.FreeBSD.org/ports/commit/?id=be17d4ad221b313ecad86d0f42cea4b1c8b19d74 commit be17d4ad221b313ecad86d0f42cea4b1c8b19d74 Author: Daniel Engberg <diizzy@FreeBSD.org> AuthorDate: 2023-01-23 09:58:51 +0000 Commit: Daniel Engberg <diizzy@FreeBSD.org> CommitDate: 2023-01-23 09:58:56 +0000 devel/creduce: Update to latest commit (20220927) and unbreak Update to latest commit to support LLVM 15 Convert to CMake and adjust dependencies PR: 268745 Reviewed by: pkubaj, tcberner Tested by: pkubaj, brooks Approved by: portmgr (maintainer timeout, 2+ weeks) devel/creduce/Makefile | 55 +++++++++------------- devel/creduce/distinfo | 6 +-- .../files/builtin_is_constant_evaluated.h (gone) | 1 - devel/creduce/files/patch-CMakeLists.txt (new) | 16 +++++++ 4 files changed, 41 insertions(+), 37 deletions(-) |
Created attachment 239252 [details] Patch for creduce * Update to latest commmit to support LLVM 15 * Convert to CMake * Adjust dependencies Grepped source code but I can't find any obvious reference to p5-Benchmark-Timer and p5-Sys-CPU. Compile tested on FreeBSD 13.1-STABLE (amd64) (make, make check-plist)