Script 'mail_helper' called by obssrc
Hello community,

here is the log from the commit of package openscad for openSUSE:Factory 
checked in at 2022-07-05 12:09:54
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/openscad (Old)
 and      /work/SRC/openSUSE:Factory/.openscad.new.1548 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "openscad"

Tue Jul  5 12:09:54 2022 rev:12 rq:986747 version:2021.01

Changes:
--------
--- /work/SRC/openSUSE:Factory/openscad/openscad.changes        2022-02-07 
23:39:08.954009751 +0100
+++ /work/SRC/openSUSE:Factory/.openscad.new.1548/openscad.changes      
2022-07-05 12:10:27.984641719 +0200
@@ -1,0 +2,18 @@
+Mon Jul  4 10:02:04 UTC 2022 - Christophe Giboudeaux <[email protected]>
+
+- Rebase fix_build_with_cgal-5.4.patch
+- Spec cleanup
+
+-------------------------------------------------------------------
+Sun Apr  3 07:42:33 UTC 2022 - Samu Voutilainen <[email protected]>
+
+- Disable build with GCAL-5.4 or newer.
+  Upstream has dropped pre-cgal-5.0 support and with that, reworked
+  CGAL integration fully. Backporting relevant patches does not
+  make sense.
+- Add patch to fix compilation with CGAL-5.4
+  + fix_build_with_cgal-5.4.patch
+- Add patch to fix compilation with newer Boost versions
+  + fix_build_issue_with_overloaded_join.patch
+
+-------------------------------------------------------------------

New:
----
  fix_build_issue_with_overloaded_join.patch
  fix_build_with_cgal-5.4.patch

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ openscad.spec ++++++
--- /var/tmp/diff_new_pack.tBsysw/_old  2022-07-05 12:10:28.540642518 +0200
+++ /var/tmp/diff_new_pack.tBsysw/_new  2022-07-05 12:10:28.544642524 +0200
@@ -24,22 +24,23 @@
 Group:          Productivity/Graphics/CAD
 URL:            https://www.openscad.org/
 Source:         https://files.openscad.org/%{name}-%{version}.src.tar.gz
-Patch1:         fix_build_with_cgal-5.3.patch
-Patch2:         CVE-2022-0496.patch
-Patch3:         CVE-2022-0497.patch
+Patch0:         fix_build_with_cgal-5.3.patch
+Patch1:         CVE-2022-0496.patch
+Patch2:         CVE-2022-0497.patch
+Patch3:         fix_build_with_cgal-5.4.patch
+Patch4:         fix_build_issue_with_overloaded_join.patch
 BuildRequires:  bison
 BuildRequires:  double-conversion-devel
-BuildRequires:  eigen3-devel
 BuildRequires:  flex
-BuildRequires:  fontconfig-devel
-BuildRequires:  freetype2-devel
 BuildRequires:  gcc-c++
-BuildRequires:  harfbuzz-devel
 BuildRequires:  libboost_filesystem-devel
 BuildRequires:  libboost_program_options-devel
 BuildRequires:  libboost_regex-devel
 BuildRequires:  libboost_thread-devel
-BuildRequires:  libcgal-devel
+# Upstream has dropped pre-cgal-5.0 support and with that, reworked
+# CGAL integration fully.
+# Backporting relevant patches does not make sense.
+BuildRequires:  libcgal-devel > 5.0
 BuildRequires:  libqscintilla-qt5-devel
 BuildRequires:  libspnav-devel
 BuildRequires:  memory-constraints
@@ -48,10 +49,17 @@
 BuildRequires:  pkgconfig(Qt5Concurrent)
 BuildRequires:  pkgconfig(Qt5Core)
 BuildRequires:  pkgconfig(Qt5DBus)
+BuildRequires:  pkgconfig(Qt5Gamepad)
 BuildRequires:  pkgconfig(Qt5Gui)
 BuildRequires:  pkgconfig(Qt5Multimedia)
+BuildRequires:  pkgconfig(Qt5Network)
 BuildRequires:  pkgconfig(Qt5OpenGL)
 BuildRequires:  pkgconfig(Qt5Widgets)
+BuildRequires:  pkgconfig(cairo) > 1.14
+BuildRequires:  pkgconfig(eigen3)
+BuildRequires:  pkgconfig(fontconfig)
+BuildRequires:  pkgconfig(freetype2)
+BuildRequires:  pkgconfig(harfbuzz)
 BuildRequires:  pkgconfig(libxml-2.0)
 BuildRequires:  pkgconfig(libzip)
 # With v2019.05, openGL is required but Arm uses openGL ES
@@ -64,24 +72,24 @@
 aspects, e.g. modelling of machine parts.
 
 %prep
-%setup -q
-
-%patch1 -p1
-%patch2 -p1
-%patch3 -p1
+%autosetup -p1
 
 %build
-%qmake5 PREFIX=%{_prefix} CONFIG+=qopenglwidget CONFIG+=c++14
+%qmake5 PREFIX=%{_prefix} CONFIG+=qopenglwidget
+
 # As of 08.05.2021, memoryperjob constraint is not working correctly,
 # so limit memory per job here.
 %limit_build -m 2500
 %make_build
 
 %install
-make INSTALL_ROOT=%{buildroot} install
+%qmake5_install
+
 install -D -m 0644 doc/openscad.1 %{buildroot}%{_mandir}/man1/openscad.1
+
 # remove bundled liberation fonts
-rm -rf %{buildroot}%{_datadir}/openscad/fonts
+rm -r %{buildroot}%{_datadir}/openscad/fonts
+
 %find_lang %{name}
 
 rm %{buildroot}%{_datadir}/openscad/libraries/MCAD/.gitignore

++++++ fix_build_issue_with_overloaded_join.patch ++++++
commit 9b79576c1ee9d57d0f4a5de5c1365bb87c548f36
Author: Torsten Paul <[email protected]>
Date:   Wed Feb 2 00:50:43 2022 +0100
Subject: Fix build issue with overloaded join().
Upstream: yes

Index: openscad-2021.01/src/openscad.cc
===================================================================
--- openscad-2021.01.orig/src/openscad.cc
+++ openscad-2021.01/src/openscad.cc
@@ -65,7 +65,6 @@
 #include <chrono>
 #include <boost/algorithm/string.hpp>
 #include <boost/algorithm/string/split.hpp>
-#include <boost/algorithm/string/join.hpp>
 #include <boost/range/adaptor/transformed.hpp>
 #include <boost/program_options.hpp>
 #include <boost/filesystem.hpp>
@@ -307,7 +306,7 @@ void set_render_color_scheme(const std::
        }

        if (exit_if_not_found) {
-               
LOG(message_group::None,Location::NONE,"",(boost::join(ColorMap::inst()->colorSchemeNames(),
 "\n")));
+               
LOG(message_group::None,Location::NONE,"",(boost::algorithm::join(ColorMap::inst()->colorSchemeNames(),
 "\n")));

                exit(1);
        } else {
@@ -885,7 +884,7 @@ struct CommaSeparatedVector
 };

 template <class Seq, typename ToString>
-std::string join(const Seq &seq, const std::string &sep, const ToString 
&toString)
+std::string str_join(const Seq &seq, const std::string &sep, const ToString 
&toString)
 {
     return boost::algorithm::join(boost::adaptors::transform(seq, toString), 
sep);
 }
@@ -947,7 +946,7 @@ int main(int argc, char **argv)
                ("P,P", po::value<string>(), "customizer parameter set")
 #ifdef ENABLE_EXPERIMENTAL
                ("enable", po::value<vector<string>>(), ("enable experimental 
features: " +
-                                                         
join(boost::make_iterator_range(Feature::begin(), Feature::end()), " | ",
+                                                         
str_join(boost::make_iterator_range(Feature::begin(), Feature::end()), " | ",
                                                               [](const Feature 
*feature) {
                                                                   return 
feature->get_name();
                                                               }) +
@@ -964,11 +963,11 @@ int main(int argc, char **argv)
                ("render", po::value<string>()->implicit_value(""), "for full 
geometry evaluation when exporting png")
                ("preview", po::value<string>()->implicit_value(""), 
"[=throwntogether] -for ThrownTogether preview png")
                ("animate", po::value<unsigned>(), "export N animated frames")
-               ("view", po::value<CommaSeparatedVector>(), ("=view options: " 
+ boost::join(viewOptions.names(), " | ")).c_str())
+               ("view", po::value<CommaSeparatedVector>(), ("=view options: " 
+ boost::algorithm::join(viewOptions.names(), " | ")).c_str())
                ("projection", po::value<string>(), "=(o)rtho or (p)erspective 
when exporting png")
                ("csglimit", po::value<unsigned int>(), "=n -stop rendering at 
n CSG elements when exporting png")
                ("colorscheme", po::value<string>(), ("=colorscheme: " +
-                                                     
join(ColorMap::inst()->colorSchemeNames(), " | ",
+                                                     
str_join(ColorMap::inst()->colorSchemeNames(), " | ",
                                                           [](const 
std::string& colorScheme) {
                                                               return 
(colorScheme == ColorMap::inst()->defaultColorSchemeName() ? "*" : "") + 
colorScheme;
                                                           }) +

++++++ fix_build_with_cgal-5.3.patch ++++++
--- /var/tmp/diff_new_pack.tBsysw/_old  2022-07-05 12:10:28.584642581 +0200
+++ /var/tmp/diff_new_pack.tBsysw/_new  2022-07-05 12:10:28.588642587 +0200
@@ -6,11 +6,11 @@
 
     Fix build failure with "generic_print_polyhedron" on CGAL-5.3.
 
-diff --git a/src/cgalutils-polyhedron.cc b/src/cgalutils-polyhedron.cc
-index 0e5c72c36..14bddb7f4 100644
---- a/src/cgalutils-polyhedron.cc
-+++ b/src/cgalutils-polyhedron.cc
-@@ -335,19 +335,6 @@ namespace CGALUtils {
+Index: openscad-2021.01/src/cgalutils-polyhedron.cc
+===================================================================
+--- openscad-2021.01.orig/src/cgalutils-polyhedron.cc
++++ openscad-2021.01/src/cgalutils-polyhedron.cc
+@@ -337,19 +337,6 @@ namespace CGALUtils {
      }
        };
  
@@ -30,11 +30,11 @@
  }; // namespace CGALUtils
  
  #endif /* ENABLE_CGAL */
-diff --git a/src/cgalutils.h b/src/cgalutils.h
-index 231762ebc..7da5a25ca 100644
---- a/src/cgalutils.h
-+++ b/src/cgalutils.h
-@@ -33,7 +33,6 @@ namespace CGALUtils {
+Index: openscad-2021.01/src/cgalutils.h
+===================================================================
+--- openscad-2021.01.orig/src/cgalutils.h
++++ openscad-2021.01/src/cgalutils.h
+@@ -45,7 +45,6 @@ namespace CGALUtils {
        bool is_approximately_convex(const PolySet &ps);
        Geometry const* applyMinkowski(const Geometry::Geometries &children);
  

++++++ fix_build_with_cgal-5.4.patch ++++++
Date: Sun, 03 Apr 2022 11:15:44 +0300
Subject: Fix build with CGAL-5.4
Upstream: modified from upstream commits
- c32efe043a65b7fd761751c8edb56f8deb6a9ed5
- 71f2831c0484c3f35cbf44e1d1dc2c857384100b

diff --git a/src/cgalutils-tess.cc b/src/cgalutils-tess.cc
index ec1cc1e..fb709fc 100644
--- a/src/cgalutils-tess.cc
+++ b/src/cgalutils-tess.cc
@@ -6,10 +6,12 @@
 #pragma push_macro("NDEBUG")
 #undef NDEBUG
 #include <CGAL/Constrained_Delaunay_triangulation_2.h>
-#if CGAL_VERSION_NR >= CGAL_VERSION_NUMBER(4,11,0)
-  #include <CGAL/Triangulation_2_projection_traits_3.h>
+#if CGAL_VERSION_NR < CGAL_VERSION_NUMBER(5, 4, 0)
+#include <CGAL/Triangulation_2_projection_traits_3.h>
+typedef CGAL::Triangulation_2_filtered_projection_traits_3<K> Projection;
 #else
-  #include <CGAL/Triangulation_2_filtered_projection_traits_3.h>
+#include <CGAL/Projection_traits_3.h>
+typedef CGAL::Filtered_projection_traits_3<K> Projection;
 #endif
 #include <CGAL/Triangulation_face_base_with_info_2.h>
 #pragma pop_macro("NDEBUG")
@@ -19,7 +21,6 @@ struct FaceInfo {
   bool in_domain() { return nesting_level%2 == 1; }
 };
 
-typedef CGAL::Triangulation_2_filtered_projection_traits_3<K> Projection;
 typedef CGAL::Triangulation_face_base_with_info_2<FaceInfo, K> Fbb;
 typedef CGAL::Triangulation_data_structure_2<
        CGAL::Triangulation_vertex_base_2<Projection>,

Reply via email to