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

Volker Fröhlich <volke...@gmx.at> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |volke...@gmx.at

--- Comment #1 from Volker Fröhlich <volke...@gmx.at> ---
Did you try to submit your patch?

Don't the other methods work that are described in
http://fedoraproject.org/wiki/Packaging:Guidelines#Removing_Rpath ?

rm -rf %{buildroot} is not necessary.

It must be:

%post -p /sbin/ldconfig
%postun -p /sbin/ldconfig

I noticed the PDFs make up most of the size of the package. Consider a
separated -doc sub-package, as mentioned here:
http://fedoraproject.org/wiki/Packaging:Guidelines#Documentation

THANKS, NEWS and the examples directory could also be included.

I'd like to recommend registering the libray on http://upstream-tracker.org to
keep track of ABI breakage.

You can remove the trailing slash from the URL, if you want.

Odd findings: "ld_config" sounds a lot like ldconfig. The version numbers that
is part of the include-directories also surprised me.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to