Bug 285593 - devel/mimalloc: Overrides framework CFLAGS
Summary: devel/mimalloc: Overrides framework CFLAGS
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Emanuel Haupt
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2025-03-23 01:13 UTC by Daniel Engberg
Modified: 2025-03-29 13:30 UTC (History)
0 users

See Also:
bugzilla: maintainer-feedback? (ehaupt)

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Engberg freebsd_committer freebsd_triage 2025-03-23 01:13:15 UTC 
The port overrides framework option CPUTYPE which isn't desirable, having a quick look it seems like you should disable MI_OPT_ARCH and enable MI_OPT_SIMD with C/CXXFLAGS for it to work as expected.
Another option would be removing MI_OPT_ARCH_FLAGS in CMakeLists.txt
Comment 1 commit-hook freebsd_committer freebsd_triage 2025-03-29 13:28:52 UTC 
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=8ecb76bb8a7bcfe804ae528045db2cdefb0e2d63

commit 8ecb76bb8a7bcfe804ae528045db2cdefb0e2d63
Author:     Emanuel Haupt <ehaupt@FreeBSD.org>
AuthorDate: 2025-03-29 13:27:35 +0000
Commit:     Emanuel Haupt <ehaupt@FreeBSD.org>
CommitDate: 2025-03-29 13:27:35 +0000

    devel/mimalloc: Update to 3.0.3

    * Update to 3.0.3
    * Disable optimizations [1]

    PR:             285593 [1]

 devel/mimalloc/Makefile                            |  2 +-
 devel/mimalloc/distinfo                            |  6 +--
 devel/mimalloc/files/patch-CMakeLists.txt          | 62 ++++++++++++++++++----
 .../files/patch-src_prim_unix_prim.c (gone)        | 12 -----
 devel/mimalloc/pkg-plist                           |  1 +
 5 files changed, 58 insertions(+), 25 deletions(-)
Comment 2 Emanuel Haupt freebsd_committer freebsd_triage 2025-03-29 13:30:04 UTC 
Committed, together with an update. Thanks.