public inbox for git-commits@fedoraproject.org
help / color / mirror / Atom feed
From: Attila Kovacs <attipaci@gmail.com>
To: git-commits@fedoraproject.org
Subject: [rpms/libxchange] rawhide: Include upstream commits post release
Date: Tue, 09 Jun 2026 08:13:53 GMT [thread overview]
Message-ID: <178099283371.1.12991857239723918279.rpms-libxchange-e588e5884ead@fedoraproject.org> (raw)
A new commit has been pushed.
Repo : rpms/libxchange
Branch : rawhide
Commit : e588e5884eadf610083bbeb8a644c80ec6daf0f1
Author : Attila Kovacs <attipaci@gmail.com>
Date : 2026-06-09T10:11:47+02:00
Stats : +202/-1 in 2 file(s)
URL : https://src.fedoraproject.org/rpms/libxchange/c/e588e5884eadf610083bbeb8a644c80ec6daf0f1?branch=rawhide
Log:
Include upstream commits post release
---
diff --git a/0001.patch b/0001.patch
new file mode 100644
index 0000000..b932cc9
--- /dev/null
+++ b/0001.patch
@@ -0,0 +1,200 @@
+From 60ccc9c7a1650115e40f691033b4d79f792de0c6 Mon Sep 17 00:00:00 2001
+From: Attila Kovacs <attipaci@gmail.com>
+Date: Mon, 8 Jun 2026 21:36:20 +0200
+Subject: [PATCH 1/3] Add xIsDebug()
+
+---
+ CHANGELOG.md | 8 ++++++++
+ include/xchange.h | 1 +
+ src/xchange.c | 18 +++++++++++++++---
+ 3 files changed, 24 insertions(+), 3 deletions(-)
+
+diff --git a/CHANGELOG.md b/CHANGELOG.md
+index d33e3f9..6f32196 100644
+--- a/CHANGELOG.md
++++ b/CHANGELOG.md
+@@ -7,6 +7,14 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
+ [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
+
+
++## [Unreleased]
++
++### Added
++
++ - Added `xIsDebug()` function to check on `xDebug`. While the global variable is fine in most cases, they are
++ problematic for Windows DLLs. It's better to use purely functional access instead.
++
++
+ ## [1.2.0] - 2026-06-08
+
+ High priority bug fixes and version bump.
+diff --git a/include/xchange.h b/include/xchange.h
+index 9ace22e..bb00226 100644
+--- a/include/xchange.h
++++ b/include/xchange.h
+@@ -240,6 +240,7 @@ extern boolean xDebug; ///< Switch to enable debugging (very verbose) o
+ // In xutil.c ------------------------------------------------>
+ boolean xIsVerbose();
+ void xSetVerbose(boolean value);
++boolean xIsDebug();
+ void xSetDebug(boolean value);
+ int xError(const char *fn, int code);
+ const char *xErrorDescription(int code);
+diff --git a/src/xchange.c b/src/xchange.c
+index ca95a9e..ecd8644 100644
+--- a/src/xchange.c
++++ b/src/xchange.c
+@@ -129,7 +129,6 @@ int x_warn(const char *from, const char *desc, ...) {
+ * @return TRUE (1) if verbosity is enabled, or else FALSE (0).
+ *
+ * @sa xSetVerbose()
+- * @sa xSetDebug()
+ */
+ boolean xIsVerbose() {
+ return xVerbose;
+@@ -140,18 +139,31 @@ boolean xIsVerbose() {
+ *
+ * @param value TRUE (non-zero) to enable verbose output, or else FALSE (0).
+ *
+- * @sa xIsVerbose()
++ * @sa xIsVerbose(), xSetDebug()
+ */
+ void xSetVerbose(boolean value) {
+ xVerbose = value ? TRUE : FALSE;
+ }
+
++/**
++ * Checks if debug output is enabled
++ *
++ * @return TRUE (1) if debug output is enabled, otherwise FALSE (0).
++ *
++ * @since 1.2.1
++ *
++ * @sa xSetDebug()
++ */
++boolean xIsDebug() {
++ return xDebug;
++}
++
+ /**
+ * Enables or disables debugging output.
+ *
+ * @param value TRUE (non-zero) to enable verbose output, or else FALSE (0).
+ *
+- * @sa xSetVerbose()
++ * @sa xIsDebug(), xSetVerbose()
+ */
+ void xSetDebug(boolean value) {
+ xDebug = value ? TRUE : FALSE;
+--
+2.54.0
+
+
+From 30005906bc0101b7b683187fbb6cbd97ddb621ba Mon Sep 17 00:00:00 2001
+From: Attila Kovacs <attipaci@gmail.com>
+Date: Mon, 8 Jun 2026 21:37:19 +0200
+Subject: [PATCH 2/3] Don't check for math lib in package config
+
+---
+ CHANGELOG.md | 6 ++++++
+ cmake/xchangeConfig.cmake.in | 8 --------
+ 2 files changed, 6 insertions(+), 8 deletions(-)
+
+diff --git a/CHANGELOG.md b/CHANGELOG.md
+index 6f32196..7c09af6 100644
+--- a/CHANGELOG.md
++++ b/CHANGELOG.md
+@@ -9,6 +9,12 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
+
+ ## [Unreleased]
+
++### Fixed
++
++ - CMake `xchangeConfig` to skip requiting math lib for non-Windows platforms in general, since it's can fail if the
++ math library is not in the search path, but in the build path, such as in case of some cross builds (see e.g. the
++ vcpkg Android builds)
++
+ ### Added
+
+ - Added `xIsDebug()` function to check on `xDebug`. While the global variable is fine in most cases, they are
+diff --git a/cmake/xchangeConfig.cmake.in b/cmake/xchangeConfig.cmake.in
+index 1742cc7..a8060b0 100644
+--- a/cmake/xchangeConfig.cmake.in
++++ b/cmake/xchangeConfig.cmake.in
+@@ -7,14 +7,6 @@ include(CMakeFindDependencyMacro)
+ # Include targets
+ include("${CMAKE_CURRENT_LIST_DIR}/xchangeTargets.cmake")
+
+-# Find math library if needed
+-if(NOT WIN32)
+- find_library(MATH_LIBRARY m)
+- if(NOT MATH_LIBRARY)
+- message(FATAL_ERROR "Math library not found")
+- endif()
+-endif()
+-
+ # Get include directories from target properties
+ get_target_property(xchange_INCLUDE_DIRS xchange::core INTERFACE_INCLUDE_DIRECTORIES)
+
+--
+2.54.0
+
+
+From b9c3f6ddc48fad6059929474e42796324c281cb8 Mon Sep 17 00:00:00 2001
+From: Attila Kovacs <attipaci@gmail.com>
+Date: Mon, 8 Jun 2026 21:44:31 +0200
+Subject: [PATCH 3/3] Reference functions not vars in xvprintf() and xdprintf()
+ macros
+
+---
+ CHANGELOG.md | 9 +++++++--
+ include/xchange.h | 4 ++--
+ 2 files changed, 9 insertions(+), 4 deletions(-)
+
+diff --git a/CHANGELOG.md b/CHANGELOG.md
+index 7c09af6..d1ca571 100644
+--- a/CHANGELOG.md
++++ b/CHANGELOG.md
+@@ -9,10 +9,12 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
+
+ ## [Unreleased]
+
++Upcoming bug-fix release, possbily as early as 1 August 2026.
++
+ ### Fixed
+
+- - CMake `xchangeConfig` to skip requiting math lib for non-Windows platforms in general, since it's can fail if the
+- math library is not in the search path, but in the build path, such as in case of some cross builds (see e.g. the
++ - CMake `xchangeConfig` to skip requiring math lib for non-Windows platforms in general, since it's can fail if the
++ math library is in the build path, but not in the search path, such as in case of some cross builds (see e.g. the
+ vcpkg Android builds)
+
+ ### Added
+@@ -20,6 +22,9 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
+ - Added `xIsDebug()` function to check on `xDebug`. While the global variable is fine in most cases, they are
+ problematic for Windows DLLs. It's better to use purely functional access instead.
+
++ - `xvprintf()` / `xdprintf()` macros to reference functions instead of global vars (see above comment on Windows
++ DLLs.
++
+
+ ## [1.2.0] - 2026-06-08
+
+diff --git a/include/xchange.h b/include/xchange.h
+index bb00226..1dd1c3c 100644
+--- a/include/xchange.h
++++ b/include/xchange.h
+@@ -234,8 +234,8 @@ typedef struct {
+ extern boolean xVerbose; ///< Switch to enable verbose console output for XChange operations.
+ extern boolean xDebug; ///< Switch to enable debugging (very verbose) output for XChange operations.
+
+-#define xvprintf if(xVerbose) printf ///< Use for generating verbose output
+-#define xdprintf if(xDebug) printf ///< Use for generating debug output
++#define xvprintf if(xIsVerbose()) printf ///< Use for generating verbose output
++#define xdprintf if(xIsDebug()) printf ///< Use for generating debug output
+
+ // In xutil.c ------------------------------------------------>
+ boolean xIsVerbose();
+--
+2.54.0
+
diff --git a/libxchange.spec b/libxchange.spec
index 1cba31c..03df929 100644
--- a/libxchange.spec
+++ b/libxchange.spec
@@ -7,6 +7,7 @@ Summary: Structured data representation and JSON support for C/C++
License: Unlicense
URL: https://sigmyne.github.io/xchange
Source0: https://github.com/Sigmyne/xchange/archive/refs/tags/v%{upstream_version}.tar.gz
+Patch0: 0001.patch
BuildRequires: gcc
BuildRequires: cmake
BuildRequires: sed
@@ -36,7 +37,7 @@ This package provides HTML documentation and examples for the xchange C/C++
library. The HTML API documentation can also be used with the Eclipse IDE.
%prep
-%autosetup -n xchange-%{upstream_version}
+%autosetup -p1 -n xchange-%{upstream_version}
%build
reply other threads:[~2026-06-09 8:13 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=178099283371.1.12991857239723918279.rpms-libxchange-e588e5884ead@fedoraproject.org \
--to=attipaci@gmail.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