[Bug 1069243] Review Request: ccaudio2 - C++ class framework for telephonic audio applications

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1069243



--- Comment #12 from Michael Schwendt <bugs.michael@xxxxxxx> ---
> * Thu Apr 24 2014 David Sugar - 2.1.1-3
> - Make sure we install stripped binaries and remove rpath.

You don't want to strip the binaries. The -debuginfo package generation needs
unstripped binaries.


> BuildRequires: pkgconfig

https://fedoraproject.org/wiki/Packaging:Guidelines#BuildRequires_based_on_pkg-config


> %package devel
> Requires: %{name}%{?_isa} = %{version}-%{release}
> Requires: %{name}-bin%{?_isa} = %{version}-%{release}

Some redundancy there. If -bin subpkg already requires base package, you don't
need the first Requires here. Plus, it's questionable that the -devel package
pulls in the -bin package. Do all developers need those tools?


> Summary: Headers for building ccaudio applications

Summary: Files for developing with %{name}

because the package includes more than headers.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review





[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]