git: b8e8a959d729 - main - audio/fossmixer: fix illegal instruction error on some hosts

[Alexey Dokuchaev]

The branch main has been updated by danfe:

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

commit b8e8a959d729468d50b9399483bc6b6ba47cfac4
Author:     Alexey Dokuchaev <danfe at FreeBSD.org>
AuthorDate: 2021-08-01 14:20:22 +0000
Commit:     Alexey Dokuchaev <danfe at FreeBSD.org>
CommitDate: 2021-08-01 14:23:13 +0000

    audio/fossmixer: fix illegal instruction error on some hosts
    
    Previously, configure script was passing certain optimization options
    by default; now it is protected by --enable-native switch which users
    have to specify explicitly:
    
      -C_FLAGS=$C_FLAGS" -O3 -ffast-math `/bin/sh ./build/opt.sh \"$CC\" -march=native`"
      +C_FLAGS=$C_FLAGS" -O2"
    
    Take one step further and replace -O2 with our default CFLAGS which are
    essentially the same, but safer (https://blog.regehr.org/archives/1307).
    
    We must, however, force -msse2 on i386 for the build to succeed there;
    other architectures might or might not need similar treatment as well.
    
    PR:             257520
    Submitted by:   maintainer (who is also upstream author)
---
 audio/fossmixer/Makefile | 6 ++++--
 audio/fossmixer/distinfo | 6 +++---
 2 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/audio/fossmixer/Makefile b/audio/fossmixer/Makefile
index f5ea00bd8225..6c421e1e1220 100644
--- a/audio/fossmixer/Makefile
+++ b/audio/fossmixer/Makefile
@@ -1,9 +1,9 @@
 PORTNAME=	fossmixer
 PORTVERSION=	0.0.3
-PORTREVISION=	1
+PORTREVISION=	2
 CATEGORIES=	audio
 MASTER_SITES=	SF/detroit
-DISTNAME=	detroit-0.2.2
+DISTNAME=	detroit-0.2.3
 
 MAINTAINER=	salojan at goto10.co
 COMMENT=	Graphical mixer application for OSS
@@ -40,6 +40,7 @@ CONFIGURE_ARGS=	--disable-bob \
 
 ALL_TARGET=
 INSTALL_TARGET=	install-strip
+CFLAGS_i386=	-msse2
 
 post-patch:
 	@(cd ${WRKSRC}/apps && ./build.sh fossmixer)
@@ -50,6 +51,7 @@ post-patch:
 
 	@${REINPLACE_CMD} -e "s|^.*bobs.*$$||" ${WRKSRC}/Makefile.in
 	@${REINPLACE_CMD} -e "s|@echo Compiling engine/$$< ; ||" ${WRKSRC}/engine/Makefile.in
+	@${REINPLACE_CMD} -e "s|-O2|${CFLAGS}|" ${WRKSRC}/configure
 
 post-install:
 	@for i in conf res; do \
diff --git a/audio/fossmixer/distinfo b/audio/fossmixer/distinfo
index 4c2ae07bfd67..b4aedbe004d5 100644
--- a/audio/fossmixer/distinfo
+++ b/audio/fossmixer/distinfo
@@ -1,3 +1,3 @@
-TIMESTAMP = 1573152492
-SHA256 (detroit-0.2.2.tar.xz) = c39850f0a90ef605aa6a818503ff6cf22a5acdd52a1e68a19a6eb7c6d04f4787
-SIZE (detroit-0.2.2.tar.xz) = 22256672
+TIMESTAMP = 1627745757
+SHA256 (detroit-0.2.3.tar.xz) = b56f1f4b32eb3c39b428d944a05b65cd17a4328e1b6bb577bd5de1d9f0d55618
+SIZE (detroit-0.2.3.tar.xz) = 22256400