On 13 Mar 2015, at 12:34, Reindl Harald h.reindl@thelounge.net wrote:
Am 13.03.2015 um 11:23 schrieb Timo Sirainen:
On 12 Mar 2015, at 21:09, Reindl Harald h.reindl@thelounge.net wrote:
/usr/lib64/dovecot/stats/libstats_mail.so
why in the world a new sub-directory containing just one so-file enforcing pakcage buildsers to change SPEC files?
So that external plugins can add more files in there and extend the available statistics
but they can also go to /usr/lib64/dovecot/ http://fedoraproject.org/wiki/Packaging:Guidelines#Beware_of_Rpath
These are plugins, not libraries. They don't need rpaths. The reason for the new directory's existence is exactly the same as for the already existing plugin directories:
$lib/dovecot/auth/ $lib/dovecot/doveadm/ $lib/dovecot/settings/ $lib/dovecot/sieve/
So auth, doveadm, settings and sieve all work by loading all the plugins from inside the directories. If I changed the stats to work differently, e.g. load $lib/dovecot/libstats_*.so it would work differently than everything else, which would be confusing. For consistency it would be better then to get rid of the above directories as well. But changing that would break backwards compatibility with external plugins (e.g. with Sieve).