Architecture specific change in rpms/openbabel.git - Arch-excludes - Fedora mailing-lists

1 Oct 2022

The package rpms/openbabel.git has added or updated architecture specific content in its
spec file (ExclusiveArch/ExcludeArch or %ifarch/%ifnarch) in commit(s):
https://src.fedoraproject.org/cgit/rpms/openbabel.git/commit/?id=28073a00267....
Change:
+%ifarch %{power64}
Thanks.
Full change:
============
commit f58d73c656a50c6787797426a8b97c8b250063ad
Author: Antonio Trande sagitter@fedoraproject.org
Date:   Thu Sep 29 21:58:25 2022 +0200
Fix rhbz#2130870

diff --git a/openbabel-plugindir.patch b/openbabel-plugindir.patch
index beb73a3..30e15cf 100644
--- a/openbabel-plugindir.patch
+++ b/openbabel-plugindir.patch
@@ -10,26 +10,21 @@ diff -up a/CMakeLists.txt.plugindir b/CMakeLists.txt
# This should be phased out in preference for just using the target name
  set(BABEL_LIBRARY openbabel)
-@@ -276,7 +276,7 @@
-     }
-    " SCANDIR_NEEDS_CONST)
- 
--  set(OB_MODULE_PATH "${CMAKE_INSTALL_PREFIX}/${OB_PLUGIN_INSTALL_DIR}")
-+  set(OB_MODULE_PATH "${LIB_INSTALL_DIR}/openbabel3")
- 
-   # Add some visibility support when using GCC
-   # note: Altough MinGW g++ 4.4 passes this test, visibility can't be used
-@@ -751,11 +751,11 @@
+@@ -750,7 +750,7 @@
+   "${openbabel_BINARY_DIR}/OpenBabel3ConfigVersion.cmake" @ONLY)
# Now to configure the installed config file.
- set(OB_CONFIG_DIR "${LIB_INSTALL_DIR}/cmake/openbabel3")
--set(OpenBabel3_INCLUDE_DIRS "${OpenBabel3_INSTALL_PREFIX}/${OB_INCLUDE_DIRS}")
-+set(OpenBabel3_INCLUDE_DIRS "${INCLUDE_INSTALL_DIR}/openbabel3")
+-set(OB_CONFIG_DIR "${LIB_INSTALL_DIR}/cmake/openbabel3")
++set(OB_CONFIG_DIR "lib${LIB_SUFFIX}/cmake/openbabel3")
+ set(OpenBabel3_INCLUDE_DIRS "${OpenBabel3_INSTALL_PREFIX}/${OB_INCLUDE_DIRS}")
  set(OpenBabel3_LIBRARIES "$<TARGET_FILE:openbabel>")
- set(OB_EXPORTS_FILE
--  "${OpenBabel3_INSTALL_PREFIX}/${OB_CONFIG_DIR}/OpenBabel3_EXPORTS.cmake")
-+  "${LIB_INSTALL_DIR}/cmake/openbabel3/OpenBabel3_EXPORTS.cmake")
- set(REL_REF)
- if(NOT WIN32)
-   set(REL_REF "/../../..")
+--- a/OpenBabel3Config.cmake.orig.in	2020-05-08 17:38:02.000000000 +0200
++++ b/OpenBabel3Config.cmake.in	2022-09-29 21:36:17.995630395 +0200
+@@ -1,5 +1,5 @@
+ # The OpenBabel3 config file. To get the targets include the exports file.
+-get_filename_component(OpenBabel3_INSTALL_PREFIX "${OpenBabel3_DIR}@REL_REF@"
++get_filename_component(OpenBabel3_INSTALL_PREFIX "/usr"
+   ABSOLUTE)
+ 
+ set(OpenBabel3_VERSION_MAJOR   "@BABEL_MAJ_VER@")
diff --git a/openbabel.spec b/openbabel.spec
index ecf3911..bf76e56 100644
--- a/openbabel.spec
+++ b/openbabel.spec
@@ -3,7 +3,7 @@
 # we don't want to provide private Perl or Python extension libs
 %global __provides_exclude_from ^(%{perl_vendorarch}/auto|%{python3_sitearch})/.*\.so$
-# Avoid LTO flags in these architectures
+# Avoid LTO flags in these architectures:
 # eigen3/Eigen/src/Core/arch/AltiVec/MatrixProduct.h:1199:26:
 # error: inlining failed in call to 'always_inline' 'Eigen::internal::bload<Eigen::internal::blas_data_mapper<double, long, 0, 0, 1>, double __vector(2), long, 2l, 0, 
 %if 0%{?rhel}
@@ -14,7 +14,7 @@
Name: openbabel
 Version: 3.1.1
-Release: 14%{?dist}
+Release: 15%{?dist}
 Summary: Chemistry software file format converter
 License: GPLv2
 URL: https://openbabel.org/
@@ -25,7 +25,7 @@ Source2: openbabel-inchi-license-lgpl-2.1.txt
 # fix perl modules install path
 Patch0: %{name}-perl.patch
-# fix plugin directory location (#680292, patch by lg)
+# fix openbabel version + cmake config files
 Patch1: %{name}-plugindir.patch
# fix SWIG_init even when not using swig (#772149)
@@ -225,6 +225,7 @@ export CXXFLAGS="%{optflags} -DEIGEN_ALTIVEC_DISABLE_MMA"
  -DRUBY_BINDINGS:BOOL=ON \
  -DWITH_MAEPARSER:BOOL=OFF \
  -DWITH_COORDGEN:BOOL=OFF \
+ -DOB_PLUGIN_INSTALL_DIR:PATH=%{_lib}/openbabel3 \
 %if 0%{?rhel}
  -DOPENBABEL_USE_SYSTEM_INCHI=false \
 %else
@@ -302,10 +303,10 @@ ctest3 -j1 --force-new-ctest-process
 %exclude %{_mandir}/man1/obgui.1*
%files devel
-%{_includedir}/%{name}3
+%{_includedir}/%{name}3/
 %{_libdir}/libopenbabel.so
 %{_libdir}/libopenbabel.so.7
-%{_libdir}/pkgconfig/*.pc
+%{_libdir}/pkgconfig/openbabel-3.pc
 %{_libdir}/cmake/openbabel3/
 %if 0%{?rhel}
 %{_libdir}/libinchi.so
@@ -325,7 +326,7 @@ ctest3 -j1 --force-new-ctest-process
 %license COPYING
 %doc THANKS AUTHORS authors.txt README.md
 %{_datadir}/%{name}/
-%{_libdir}/%{name}/
+%{_libdir}/%{name}3/
 %{_libdir}/libopenbabel.so.7.0.0
 %if 0%{?rhel}
 %license inchi-license-lgpl-2.1.txt
@@ -345,6 +346,10 @@ ctest3 -j1 --force-new-ctest-process
 %{ruby_vendorarchdir}/openbabel.so
%changelog
+* Thu Sep 29 2022 Antonio Trande sagitter@fedoraproject.org - 3.1.1-15
+- Fix openbabel version in CMakeLists.txt
+- Fix rhbz#2130870
+
 * Tue Sep 27 2022 Antonio Trande sagitter@fedoraproject.org - 3.1.1-14
 - Fix EPEL builds
commit 28073a00267c3c42ba3c737a133a832654a30dda
Author: Antonio Trande sagitter@fedoraproject.org
Date:   Tue Sep 27 20:14:27 2022 +0200
Fix EPEL builds
diff --git a/openbabel.spec b/openbabel.spec
index a111186..ecf3911 100644
--- a/openbabel.spec
+++ b/openbabel.spec
@@ -3,9 +3,18 @@
 # we don't want to provide private Perl or Python extension libs
 %global __provides_exclude_from ^(%{perl_vendorarch}/auto|%{python3_sitearch})/.*\.so$
+# Avoid LTO flags in these architectures
+# eigen3/Eigen/src/Core/arch/AltiVec/MatrixProduct.h:1199:26:
+# error: inlining failed in call to 'always_inline' 'Eigen::internal::bload<Eigen::internal::blas_data_mapper<double, long, 0, 0, 1>, double __vector(2), long, 2l, 0, 
+%if 0%{?rhel}
+%ifarch %{power64}
+%define _lto_cflags %{nil}
+%endif
+%endif
+
 Name: openbabel
 Version: 3.1.1
-Release: 13%{?dist}
+Release: 14%{?dist}
 Summary: Chemistry software file format converter
 License: GPLv2
 URL: https://openbabel.org/
@@ -58,9 +67,11 @@ BuildRequires: eigen3-devel
 BuildRequires: gcc-c++
 %if 0%{?fedora}
 BuildRequires: inchi-devel >= 1.0.3
+BuildRequires: wxGTK-devel
+%else
+BuildRequires: wxGTK3-devel
 %endif
 BuildRequires: libxml2-devel
-BuildRequires: wxGTK-devel
 BuildRequires: ImageMagick
 BuildRequires: rapidjson-devel
 Requires:      %{name}-libs%{?_isa} = %{version}-%{release}
@@ -334,6 +345,9 @@ ctest3 -j1 --force-new-ctest-process
 %{ruby_vendorarchdir}/openbabel.so
%changelog
+* Tue Sep 27 2022 Antonio Trande sagitter@fedoraproject.org - 3.1.1-14
+- Fix EPEL builds
+
 * Tue Sep 27 2022 Antonio Trande sagitter@fedoraproject.org - 3.1.1-13
 - New rebuild

    