Hello community, here is the log from the commit of package rttr for openSUSE:Factory checked in at 2019-07-08 15:10:53 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/rttr (Old) and /work/SRC/openSUSE:Factory/.rttr.new.4615 (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Package is "rttr" Mon Jul 8 15:10:53 2019 rev:4 rq:713654 version:0.9.6 Changes: -------- --- /work/SRC/openSUSE:Factory/rttr/rttr.changes 2019-05-10 09:19:52.284472310 +0200 +++ /work/SRC/openSUSE:Factory/.rttr.new.4615/rttr.changes 2019-07-08 15:10:55.611339436 +0200 @@ -1,0 +2,11 @@ +Thu Jun 27 14:25:10 UTC 2019 - Luca Beltrame <lbeltrame@kde.org> + +- Disable tests to allow the package to build under GCC 9 + +------------------------------------------------------------------- +Tue Jun 4 02:54:17 UTC 2019 - Dan Weatherill <plasteredparrot@gmail.com> + +- remove_pessimizing_moves.patch: remove NRVO pessimizing moves + (fixes a warning on gcc 9 to fix current Factory build) + +------------------------------------------------------------------- New: ---- remove_pessimizing_moves.patch ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ rttr.spec ++++++ --- /var/tmp/diff_new_pack.ljLs1j/_old 2019-07-08 15:10:56.627340411 +0200 +++ /var/tmp/diff_new_pack.ljLs1j/_new 2019-07-08 15:10:56.631340414 +0200 @@ -36,6 +36,8 @@ Patch4: fix-include-permissions.patch # PATCH-FIX-UPSTREAM lp64.patch Support all LP64 architectures (https://github.com/rttrorg/rttr/pull/231) Patch5: lp64.patch +#PATCH-FIX-UPSTREAM remove_pessimizing_moves.patch remove pessimizing move calls (https://github.com/rttrorg/rttr/pull/243) +Patch6: remove_pessimizing_moves.patch BuildRequires: cmake BuildRequires: dos2unix BuildRequires: doxygen @@ -57,10 +59,15 @@ %patch4 -p1 %patch5 -p1 +#only apply move elision to gcc versions that support it +%if 0%{?suse_version} >= 1500 +%patch6 -p1 +%endif + %build find . -type f -exec chmod a-x "{}" + dos2unix README.md -%cmake -DBUILD_BENCHMARKS=OFF -DCMAKE_INSTALL_CMAKEDIR=cmake +%cmake -DBUILD_BENCHMARKS=OFF -DCMAKE_INSTALL_CMAKEDIR=cmake -DBUILD_UNIT_TESTS=OFF #make unit tests pass export LD_LIBRARY_PATH=%{_builddir}/rttr-%{version}/build/lib/ %make_jobs ++++++ remove_pessimizing_moves.patch ++++++ Index: rttr-0.9.6/src/rttr/detail/registration/bind_impl.h =================================================================== --- rttr-0.9.6.orig/src/rttr/detail/registration/bind_impl.h +++ rttr-0.9.6/src/rttr/detail/registration/bind_impl.h @@ -289,7 +289,7 @@ class registration::bind<detail::ctor_fu std::move(get_metadata(std::forward<Args>(args)...)), std::move(get_default_args<type_list<Acc_Func>, function_type>(std::forward<Args>(args)...)), std::move(create_param_infos<type_list<F>, function_type>(std::forward<Args>(args)...))); - return std::move(ctor); + return ctor; } public: bind(const std::shared_ptr<detail::registration_executer>& reg_exec, F func) @@ -386,7 +386,7 @@ class registration::bind<detail::prop, C detail::map_access_level_to_enum<acc_level>::value, getter_policy, setter_policy, Metadata_Count>>(name, type::get<Class_Type>(), acc, std::move(metadata_list)); - return std::move(prop); + return prop; } public: @@ -476,7 +476,7 @@ class registration::bind<detail::prop, C getter_policy, setter_policy, Metadata_Count>>(name, type::get<Class_Type>(), getter, setter, std::move(metadata_list)); - return std::move(prop); + return prop; } public: @@ -566,7 +566,7 @@ class registration::bind<detail::prop_re getter_policy, default_setter_policy, Metadata_Count>>(name, type::get<Class_Type>(), acc, std::move(metadata_list)); - return std::move(prop); + return prop; } public: @@ -659,7 +659,7 @@ class registration::bind<detail::meth, C std::move(get_metadata(std::forward<Args>(args)...)), std::move(get_default_args<type_list<Acc_Func>, function_type>(std::forward<Args>(args)...)), std::move(create_param_infos<type_list<F>, function_type>(std::forward<Args>(args)...)) ); - return std::move(meth); + return meth; } template<typename Policy, std::size_t Metadata_Count, typename...TArgs, typename...Param_Args> @@ -770,7 +770,7 @@ class registration::bind<detail::enum_, std::move(get_metadata(std::forward<Args>(args)...))); - return std::move(enum_wrapper); + return enum_wrapper; } public: