[Bug 168339] Review Request: libbinio
bugzilla at redhat.com
bugzilla at redhat.com
Thu Sep 15 06:56:47 UTC 2005
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: libbinio
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=168339
------- Additional Comments From rc040203 at freenet.de 2005-09-15 02:56 EST -------
NEEDSWORK:
* BuildRequires: libstdc++-devel
^^^^ This is probably superfluous.
* BuildRequires: info
^^^^ This is probably wrong.
info is an *.info reader. *.infos normally are generated by makeinfo (from the
texinfo packages). Packages complying to the GNU Standards ship prebuilt *.infos
(Which AFAIS applies, here), so neither info nor texinfo should be required for
building.
* %{_includedir}/*.h
I hate packages installing their headers to /usr/include, because they pollute
the system include path. Instead, I recommend to install them to a subdirectory
of /usr/include, e.g.
%configure --includedir=%{_include}/binio
...
%{_includedir}/binio
...
* Your %post script is non functional.
You can't use -p /sbin/ldconfig there because you are trying to install infos at
the same time. You'll have to use something along these lines:
Requires(post): /sbin/ldconfig
Requires(post): /sbin/install-info
%post
/sbin/ldconfig
/sbin/install-info .....
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
More information about the fedora-extras-list
mailing list