[pnmixer] Build with $RPM_OPT_FLAGS (#800642)

Christoph Wickert cwickert at fedoraproject.org
Sat Mar 10 20:40:11 UTC 2012


commit e4bd9869b1dfb696f6d8ffdc8d2dbb9c5ecf19fd
Author: Christoph Wickert <cwickert at fedoraproject.org>
Date:   Sat Mar 10 21:40:07 2012 +0100

    Build with $RPM_OPT_FLAGS (#800642)
    
    - Minor fixes for desktop file

 pnmixer.spec |   14 ++++++++++++--
 1 files changed, 12 insertions(+), 2 deletions(-)
---
diff --git a/pnmixer.spec b/pnmixer.spec
index f3b8e37..7ef86c0 100644
--- a/pnmixer.spec
+++ b/pnmixer.spec
@@ -2,7 +2,7 @@
 
 Name:           pnmixer
 Version:        0.5.1
-Release:        2%{?dist}
+Release:        3%{?dist}
 Summary:        Lightweight mixer applet
 
 Group:          Applications/Multimedia
@@ -54,7 +54,9 @@ NOCONFIGURE=yes ./autogen.sh
 
 %build
 %configure
-make %{?_smp_mflags}
+# CFLAGS... is only needed because of #800642
+# https://github.com/nicklan/pnmixer/issues/19
+make %{?_smp_mflags} CFLAGS='%{optflags}'
 
 
 %install
@@ -64,9 +66,13 @@ make install DESTDIR=%{buildroot} INSTALL='install -p'
 # Don't install a launcher in the menu but in autostart
 rm -rf %{buildroot}%{_datadir}/applications/
 
+# minor fixes for the desktop file
+# upstream https://github.com/nicklan/pnmixer/issues/18
 desktop-file-install \
     --dir=%{buildroot}%{_sysconfdir}/xdg/autostart \
+    --add-category="Audio" \
     --add-category="TrayIcon" \
+    --remove-category="Application" \
     --add-only-show-in="XFCE;LXDE;" \
     data/pnmixer.desktop
 
@@ -85,6 +91,10 @@ rm -rf %{buildroot}
 %{_datadir}/%{name}/
 
 %changelog
+* Sat Mar 10 2012 Christoph Wickert <cwickert at fedoraproject.org> - 0.5.1-3
+- Build with $RPM_OPT_FLAGS (#800642)
+- Minor fixes for desktop file
+
 * Sun Mar 04 2012 Christoph Wickert <cwickert at fedoraproject.org> - 0.5.1-2
 - Fix launcher icon
 - Don't install a launcher in the application menu


More information about the scm-commits mailing list