public inbox for git-commits@fedoraproject.org
help / color / mirror / Atom feed
From: William Cohen <wcohen@redhat.com>
To: git-commits@fedoraproject.org
Subject: [rpms/memstomp] rawhide: Fix FTBFS in Fedora rawhide/f45 (#2504325)
Date: Tue, 04 Aug 2026 13:28:33 GMT	[thread overview]
Message-ID: <178585011364.1.523216787817342827.rpms-memstomp-6712901cdbda@fedoraproject.org> (raw)

A new commit has been pushed.

Repo   : rpms/memstomp
Branch : rawhide
Commit : 6712901cdbda39d5adb2ae919e0762b1fcf66630
Author : William Cohen <wcohen@redhat.com>
Date   : 2026-08-04T09:27:03-04:00
Stats  : +41/-5 in 2 file(s)
URL    : https://src.fedoraproject.org/rpms/memstomp/c/6712901cdbda39d5adb2ae919e0762b1fcf66630?branch=rawhide

Log:
Fix FTBFS in Fedora rawhide/f45 (#2504325)

---
diff --git a/memstomp-undefs.patch b/memstomp-undefs.patch
new file mode 100644
index 0000000..8bdc2c1
--- /dev/null
+++ b/memstomp-undefs.patch
@@ -0,0 +1,28 @@
+From e2b2e970b78892706ca7f3d211d294c15ac29d61 Mon Sep 17 00:00:00 2001
+From: William Cohen <wcohen@redhat.com>
+Date: Mon, 3 Aug 2026 23:25:51 -0400
+Subject: [PATCH] Add some missing undefs to memstomp.c
+
+With newer compilers the build ends up generating errors for memchr and strstr
+in memstmop.c.  Add additional undefs to eliminate those errors.
+---
+ memstomp.c | 2 ++
+ 1 file changed, 2 insertions(+)
+
+diff --git a/memstomp.c b/memstomp.c
+index cb2bafc..c3b9a9c 100644
+--- a/memstomp.c
++++ b/memstomp.c
+@@ -71,7 +71,9 @@
+ #undef strcspn
+ #undef strspn
+ #undef strpbrk
++#undef strstr
+ #undef strtok_r
++#undef memchr
+ #undef memcmp
+ 
+ static bool abrt_trap = false;
+-- 
+2.55.0
+

diff --git a/memstomp.spec b/memstomp.spec
index 975ac11..0bdfdd7 100644
--- a/memstomp.spec
+++ b/memstomp.spec
@@ -2,23 +2,23 @@
 
 Name:		memstomp
 Version:	0.1.4
-Release:	46%{?dist}
+Release:	47%{?dist}
 Summary:	Warns of memory argument overlaps to various functions
 # The entire source code is LGPLV3+ with the exception of backtrace-symbols.c which
 # is GPLv2+ by way of being a hacked up old version of binutils's addr2line.
 # backtrace-symbols.c is built into an independent .so to avoid license contamination
 # Automatically converted from old format: LGPLv3+ and GPLv2+ - review is highly recommended.
 License:	LGPL-3.0-or-later AND GPL-2.0-or-later
-URL:		git://fedorapeople.org/home/fedora/wcohen/public_git/memstomp
+URL:		https://github.com/wcohen/memstomp.git
 # The source for this package was pulled from upstream's vcs.  Use the
 # following commands to generate the tarball:
 # git glone git://fedorapeople.org/home/fedora/wcohen/public_git/memstomp
 # cd memstomp
 # git archive --prefix memstomp-0.1.4-38573e7d/ master | gzip > memstomp-0.1.4-3867e37d.tar.gz
-Source0:	%{name}-%{version}-%{githash}.tar.gz
+Source0:	https://github.com/wcohen/memstomp/archive/%{githash}/%{name}-%{version}-%{githash}.tar.gz
 Requires:	util-linux
-BuildRequires: make
-BuildRequires:  gcc
+BuildRequires:	make
+BuildRequires:	gcc
 BuildRequires:	binutils-devel autoconf automake dejagnu
 
 Patch0: memstomp-testsuite.patch
@@ -32,6 +32,7 @@ Patch7: memstomp-implicit-int.patch
 Patch8: bfd-api-change.patch
 Patch9: memstomp-PTR.patch
 Patch10: memstomp-sframe.patch
+Patch11: memstomp-undefs.patch
 
 
 %description 
@@ -54,9 +55,12 @@ overlapping memory arguments to certain library calls.
 %patch -P8 -p1
 %patch -P9 -p1
 %patch -P10 -p1
+%patch -P11 -p1
 
 
 %build
+%set_build_flags
+CFLAGS="$CFLAGS -lzstd"
 autoreconf
 %configure
 # We force -O0 here because memstomp essentially relies on GCC
@@ -78,6 +82,10 @@ make install DESTDIR=$RPM_BUILD_ROOT
 %{_mandir}/man1/memstomp.1.gz
 
 %changelog
+* Tue Aug 04 2026 William Cohen <wcohen@redhat.com> - 0.1.4-47
+- Add zstd in configure tests to fix FTBFS. (rhbz#2504325)
+- Clean up rpmlint complaints in memstomp.spec.
+
 * Thu Jul 16 2026 Fedora Release Engineering <releng@fedoraproject.org> - 0.1.4-46
 - Rebuilt for https://fedoraproject.org/wiki/Fedora_45_Mass_Rebuild
 

                 reply	other threads:[~2026-08-04 13:28 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=178585011364.1.523216787817342827.rpms-memstomp-6712901cdbda@fedoraproject.org \
    --to=wcohen@redhat.com \
    --cc=git-commits@fedoraproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox