packaging: spec cleanups

- whitespace cleanups
- always use pkgconfig() in BuildRequires where possible
- create a docs subpackage for the man page
- fix RPM group tag

Change-Id: Ifa41e9f3a1bd0907a7c05e09dbf0dd1ae684de6f
Signed-off-by: Patrick McCarty <patrick.mccarty@linux.intel.com>

diff --git a/packaging/powertop.spec b/packaging/powertop.spec
index c0466f1..402f1e3 100644 (file)
--- a/packaging/powertop.spec
+++ b/packaging/powertop.spec
@@ -2,20 +2,19 @@ Name:           powertop
 Version:        2.3
 Release:        1
 License:        GPL-2.0
-Summary:        PowerTop is tool that is used for power diagnostics
+Summary:        A tool that is used for power diagnostics
 Url:            http://www.01.org
-Group:          Platform Development/Tools
+Group:          System/Utilities
 Source0:        https://01.org/powertop/sites/default/files/downloads/%{name}-%{version}.tar.gz
-Source1001:    powertop.manifest
-
+Source1001:     %{name}.manifest
 BuildRequires:  gettext
-BuildRequires:  zlib-devel
 BuildRequires:  pkgconfig(libnl-1)
 BuildRequires:  pkgconfig(libpci)
 BuildRequires:  pkgconfig(ncurses)
+BuildRequires:  pkgconfig(zlib)
 
 %description
-PowerTop is tool that detects which Linux programs
+PowerTop is a tool that detects which Linux programs
 and kernel tunables are resulting in the largest
 power consumption and use of battery time. By
 fixing (or closing) these applications or
@@ -29,7 +28,6 @@ laptop.
 cp %{SOURCE1001} .
 
 %build
-
 %configure --disable-static
 make %{?_smp_mflags}
 
@@ -38,11 +36,11 @@ make %{?_smp_mflags}
 
 %find_lang %{name}
 
+%docs_package
+
 %lang_package
 
 %files
 %manifest %{name}.manifest
-%defattr(-,root,root,-)
 %license COPYING
 %{_sbindir}/powertop
-%{_mandir}/man8/powertop.8.gz