Accepting request 1227763 from science

OBS-URL: https://build.opensuse.org/request/show/1227763
OBS-URL: https://build.opensuse.org/package/show/openSUSE:Factory/lalsimulation?expand=0&rev=12

lalsimulation-swig-stringval-not-value.patch

@@ -0,0 +1,33 @@
+From 9dba245ab3692ecf691247a442704f13c075ed34 Mon Sep 17 00:00:00 2001
+From: Karl Wette <karl.wette@ligo.org>
+Date: Thu, 24 Oct 2024 18:06:30 +1100
+Subject: [PATCH] common/swig/generate_swig_iface.py: prefer "stringval" before
+ "value" attribute
+
+- As of SWIG 4.3.0 "value" attribute contains quote characters
+- "stringval" attribute appears to still contain the raw string
+---
+ common/swig/generate_swig_iface.py | 6 ++++--
+ 1 file changed, 4 insertions(+), 2 deletions(-)
+
+diff --git a/common/swig/generate_swig_iface.py b/common/swig/generate_swig_iface.py
+index 530b026d5d..eaee2d41ce 100644
+--- a/common/swig/generate_swig_iface.py
++++ b/common/swig/generate_swig_iface.py
+@@ -195,9 +195,11 @@ class SwigSymbols(object):
+         """Parse a swiglal macro
+         """
+         cdecl_name = get_swig_attr(cdecl, 'name')
+-        cdecl_value = get_swig_attr(cdecl, 'value')
++        cdecl_value = get_swig_attr(cdecl, 'stringval')
+         if not cdecl_value:
+-            raise ValueError("cdecl '{}' has no value".format(cdecl_name))
++            cdecl_value = get_swig_attr(cdecl, 'value')
++            if not cdecl_value:
++                raise ValueError("cdecl '{}' has no value".format(cdecl_name))
+         macro = re.sub(r'\s', '', cdecl_value)
+         if cdecl_name == '__swiglal__':
+             if macro in clear_macros:
+-- 
+GitLab
+

lalsimulation.changes

@@ -1,3 +1,15 @@
+-------------------------------------------------------------------
+Sun Dec  1 11:27:36 UTC 2024 - Atri Bhattacharya <badshah400@gmail.com>
+
+- Add lalsimulation-swig-stringval-not-value.patch: For
+  compatibility with swig 4.3.0, prefer 'stringval' before 'value'
+  attribute (upstream commit 9dba245a).
+
+-------------------------------------------------------------------
+Mon Nov 18 12:43:36 UTC 2024 - Atri Bhattacharya <badshah400@gmail.com>
+
+- Fix use of %requires_eq with respect to octave.
+
 -------------------------------------------------------------------
 Wed Sep 11 17:26:10 UTC 2024 - Atri Bhattacharya <badshah400@gmail.com>
 

lalsimulation.spec

@@ -31,6 +31,8 @@ Summary:        LSC Algorithm Simulation Library
 License:        GPL-2.0-only
 URL:            https://wiki.ligo.org/Computing/DASWG/LALSuite
 Source:         https://software.igwn.org/sources/source/lalsuite/%{name}-%{version}.tar.xz
+# PATCH-FIX-UPSTREAM
+Patch0:         https://git.ligo.org/lscsoft/lalsuite/-/commit/9dba245ab3692ecf691247a442704f13c075ed34.patch#/lalsimulation-swig-stringval-not-value.patch
 BuildRequires:  %{python_module devel}
 BuildRequires:  %{python_module lal >= 7.2.0}
 BuildRequires:  %{python_module numpy >= 1.7}
@@ -93,6 +95,7 @@ BuildArch:      noarch
 This package provides the data files used when running analyses involving
 lalsimulation.
 
+%if %{with octave}
 %package -n octave-lalsimulation
 Summary:        Octave bindings for LALSimulation
 Requires:       %{name}-data = %{version}
@@ -101,8 +104,10 @@ Requires:       octave-lal
 
 %description -n octave-lalsimulation
 This package provides the necessary files for using LALSimulation with octave.
+%endif
 
 %prep
+# Upstream commits are -p1 against the full lalsuite, but -p2 against individual lal* pkgs
 %autosetup -p2
 
 %build