Accepting request 999695 from home:sfalken:branches:X11:LXQt

- Add fix-no-undefined-builderror.patch, to correct failing build.  [gh#lxqt/screengrab#310]
- Cleaned up %build section
- Modified %files, due to new additions from upstream

OBS-URL: https://build.opensuse.org/request/show/999695
OBS-URL: https://build.opensuse.org/package/show/X11:LXQt/screengrab?expand=0&rev=35

fix-no-undefined-builderror.patch

@@ -0,0 +1,18 @@
+diff -ruNp screengrab-orig/src/modules/extedit/CMakeLists.txt screengrab/src/modules/extedit/CMakeLists.txt
+--- screengrab-orig/src/modules/extedit/CMakeLists.txt
++++ screengrab/src/modules/extedit/CMakeLists.txt
+@@ -24,12 +24,10 @@ qt5_translation_loader(extedit_QM_LOADER
+ )
+ 
+ add_library(extedit
+-    SHARED
++    STATIC
+         ${extedit_SRC}
+         ${extedit_QMS}
+         ${extedit_QM_LOADER}
+ )
+-set_property (TARGET extedit PROPERTY SOVERSION 1.0.0)
+-install(TARGETS extedit DESTINATION ${SG_LIBDIR})
+ 
+ target_link_libraries(extedit Qt5::Widgets Qt5::X11Extras Qt5Xdg)
+

screengrab.changes

@@ -1,3 +1,10 @@
+-------------------------------------------------------------------
+Sun Aug 28 18:27:32 UTC 2022 - Shawn Dunn <sfalken@cloverleaf-linux.org>
+
+- Add fix-no-undefined-builderror.patch, to correct failing build.  [gh#lxqt/screengrab#310]
+- Cleaned up %build section
+- Modified %files, due to new additions from upstream
+
 -------------------------------------------------------------------
 Fri Apr 29 06:31:47 UTC 2022 - Michael Vetter <mvetter@suse.com>
 

screengrab.spec

@@ -27,6 +27,8 @@ Source:         https://github.com/lxqt/screengrab/releases/download/%{version}/
 Source1:        https://github.com/lxqt/screengrab/releases/download/%{version}/%{name}-%{version}.tar.xz.asc
 Source2:        %{name}.keyring
 Patch0:         screengrab-link.patch
+# PATCH-FIX-UPSTREAM fix-no-undefined-builderror.patch gh#lxqt/screengrab#310
+Patch1:         fix-no-undefined-builderror.patch
 BuildRequires:  cmake >= 3.1.0
 BuildRequires:  gcc-c++
 BuildRequires:  hicolor-icon-theme
@@ -56,17 +58,11 @@ Screenshot taker with the ability to publish them via hosting services.
 %prep
 %setup -q
 %patch0 -p1
+%patch1 -p1
 
 %build
-# seems like their cmake doesnt do this
+%cmake
-#SUSE_ASNEEDED=0
+%cmake_build
-CXXFLAGS="%{optflags} -I%{_includedir}/qt5xdg"
-%cmake \
-    -DCMAKE_SKIP_RPATH:BOOL=ON \
-    -DSG_DBUS_NOTIFY=ON \
-    -DSG_EXT_UPLOADS=OFF \
-    -DSG_GLOBALSHORTCUTS=OFF \
-    -DUPDATE_TRANSLATIONS=OFF
 
 %install
 %cmake_install
@@ -81,6 +77,7 @@ rm -rf %{buildroot}%{_datadir}/doc
 %{_datadir}/applications/%{name}.desktop
 %{_datadir}/icons/hicolor/*/apps/%{name}.??g
 %{_datadir}/screengrab/screengrab.conf
+%{_datadir}/metainfo/%{name}.metainfo.xml
 
 %files lang -f %{name}.lang
 %dir %{_datadir}/%{name}