Accepting request 1236929 from home:alessio.biancalana:branches:utilities

- Add %pre hook
- Get rid of unneeded attack surface
- Comply with logrotate config naming convention

OBS-URL: https://build.opensuse.org/request/show/1236929
OBS-URL: https://build.opensuse.org/package/show/utilities/below?expand=0&rev=2

below.changes

@@ -1,3 +1,10 @@
+-------------------------------------------------------------------
+Fri Jan 10 18:40:14 UTC 2025 - Alessio Biancalana <alessio.biancalana@suse.com>
+
+- Add %pre hook
+- Get rid of unneeded attack surface
+- Comply with logrotate config naming convention
+
 -------------------------------------------------------------------
 Tue Dec 31 14:16:38 UTC 2024 - Alessio Biancalana <alessio.biancalana@suse.com>
 

below.spec

@@ -45,6 +45,9 @@ below is an interactive tool to view and record historical system data. It has s
 
 below does not have support for cgroup1.
 
+%pre
+%service_add_pre %{name}.service
+
 %post
 %service_add_post %{name}.service
 
@@ -64,17 +67,17 @@ below does not have support for cgroup1.
 
 %install
 install -D -m0644 %{_builddir}/%{name}-%{version}%{_sysconfdir}/%{name}.service %{buildroot}%{_unitdir}/%{name}.service
-install -D -p -m0644 %{_builddir}/%{name}-%{version}%{_sysconfdir}/logrotate.conf %{buildroot}%{_sysconfdir}/logrotate.d/%{name}.conf
-install -d -m1777 %{buildroot}%{_localstatedir}/log/%{name}
+install -D -p -m0644 %{_builddir}/%{name}-%{version}%{_sysconfdir}/logrotate.conf %{buildroot}%{_sysconfdir}/logrotate.d/%{name}
 install -D -d -m0755 %{buildroot}%{_bindir}
 install -m0755 %{_builddir}/%{name}-%{version}/target/release/%{name} %{buildroot}%{_bindir}/%{name}
+install -d -m1755 %{buildroot}%{_localstatedir}/log/%{name}
 
 %files
 %license LICENSE
 %{_bindir}/%{name}
 %{_unitdir}/%{name}.service
 %dir %{_sysconfdir}/logrotate.d
-%config(noreplace) %{_sysconfdir}/logrotate.d/%{name}.conf
 %dir %{_localstatedir}/log/%{name}
+%config(noreplace) %{_sysconfdir}/logrotate.d/%{name}
 
 %changelog