Accepting request 295640 from Base:System

1

OBS-URL: https://build.opensuse.org/request/show/295640
OBS-URL: https://build.opensuse.org/package/show/openSUSE:Factory/powertop?expand=0&rev=38

powertop.changes

@@ -1,3 +1,17 @@
+-------------------------------------------------------------------
+Sun Apr 12 08:50:44 UTC 2015 - mpluskal@suse.com
+
+- Add systemd dependencies
+- Add pre/post macros for service
+- Add rcservice symlink
+- Remove double depenency on pkg-config
+- Cleanup spec file with spec-cleaner
+
+-------------------------------------------------------------------
+Sat Apr 11 04:56:30 UTC 2015 - hillwood@linuxfans.org
+
+- Add a systemd script. 
+
 -------------------------------------------------------------------
 Sun Feb  8 22:02:17 UTC 2015 - p.drouand@gmail.com
 

powertop.service

@@ -0,0 +1,11 @@
+[Unit]
+Description=Extend the battery life of laptop
+
+[Service]
+Type=oneshot
+Environment="TERM=xterm"
+ExecStart=/usr/sbin/powertop --auto-tune
+
+[Install]
+WantedBy=multi-user.target
+

powertop.spec

@@ -1,7 +1,7 @@
 #
 # spec file for package powertop
 #
-# Copyright (c) 2015 SUSE LINUX Products GmbH, Nuernberg, Germany.
+# Copyright (c) 2015 SUSE LINUX GmbH, Nuernberg, Germany.
 #
 # All modifications and additions to the file contributed by third parties
 # remain the property of their copyright owners, unless otherwise agreed
@@ -17,14 +17,15 @@
 
 
 Name:           powertop
-Url:            https://01.org/powertop/
+Version:        2.7
+Release:        0
 #Git-Clone:	git://github.com/fenrus75/powertop
 Summary:        A Linux Tool to Find out What is Using Power on a Laptop
 License:        GPL-2.0+
 Group:          System/Monitoring
-Version:        2.7
+Url:            https://01.org/powertop/
-Release:        0
+Source0:        https://01.org/sites/default/files/downloads/%{name}/%{name}-%{version}.tar.gz
-Source:         https://01.org/sites/default/files/downloads/%name/%name-%version.tar.gz
+Source1:        powertop.service
 Patch2:         powertop-1.98-always-create-params.patch
 Patch3:         powertop-fix-crash-with-autotune.patch
 Patch5:         powertop-no-date.patch
@@ -32,13 +33,15 @@ BuildRequires:  gcc-c++
 BuildRequires:  gettext
 BuildRequires:  ncurses-devel
 BuildRequires:  pkg-config
-BuildRequires:  pkgconfig
+BuildRequires:  systemd-rpm-macros
 BuildRequires:  pkgconfig(libnl-3.0)
 BuildRequires:  pkgconfig(libnl-genl-3.0)
 BuildRequires:  pkgconfig(libpci)
 BuildRequires:  pkgconfig(zlib)
+Requires:       xterm
 Recommends:     %{name}-lang
 BuildRoot:      %{_tmppath}/%{name}-%{version}-build
+%{?systemd_requires}
 
 %description
 PowerTOP is a program that collects the various pieces of information
@@ -61,25 +64,41 @@ find . -name '*.o' -delete
 make %{?_smp_mflags}
 
 %install
-make install DESTDIR="%buildroot"
+make DESTDIR=%{buildroot} install %{?_smp_mflags}
 install -Dd %{buildroot}%{_localstatedir}/cache/powertop
 touch %{buildroot}%{_localstatedir}/cache/powertop/{saved_parameters.powertop,saved_results.powertop}
+install -d %{buildroot}%{_unitdir}
+install -m 644 %{SOURCE1} %{buildroot}%{_unitdir}
+ln -s %{_sbindir}/service %{buildroot}%{_sbindir}/rc%{name}
 %find_lang %{name}
 
+%pre
+%service_add_pre %{name}.service
+
 %post
+%service_add_post %{name}.service
 # Hack for powertop not to show warnings on first start
 touch %{_localstatedir}/cache/powertop/saved_parameters.powertop
 touch %{_localstatedir}/cache/powertop/saved_results.powertop
 
+%preun
+%service_del_preun %{name}.service
+
+%postun
+%service_del_postun %{name}.service
+
 %files
 %defattr(-,root,root)
 %doc COPYING README
 %dir %{_localstatedir}/cache/powertop
 %ghost %{_localstatedir}/cache/powertop/saved_parameters.powertop
 %ghost %{_localstatedir}/cache/powertop/saved_results.powertop
-%_sbindir/%name
+%{_sbindir}/%{name}
 %{_mandir}/man8/powertop.8.gz
+%{_unitdir}/%{name}.service
+%{_sbindir}/rc%{name}
 
 %files lang -f %{name}.lang
+%defattr(-,root,root)
 
 %changelog