Gordon Messmer writes:
While working on updating the FreeBSD port, I noticed a few patches that may be appropriate for merging.First, the test for -lcourier-unicode seems to be slightly deficient. All of the AC_CHECK_LIB checks successfully locate libs in /usr/local/lib, but AC_LINK_IFELSE doesn't seem to. I think that's because that part of configure.ac stashes $LIBS where -L/usr/local/lib will be? Regardless, maybe this section can be refactored to use AC_CHECK_LIB...
That particular tests checks not only for the library itself being installed, but also for the presence of the header files.
I'll replace that with both AC_CHECK_LIB and AC_CHECK_HEADER.
Second, courier/perms.sh.in sets a number of executables to 555, and others to 755. The FreeBSD maintainer makes those consistent, removing write access.
Adjusting the permissions on the individual executables seems fine. However the entries for the directories have to stay there. The default configuration installs Courier completely inside the /usr/lib/courier tree, so all of these directories get created, and this sets the right permissions on them.
pgpQiq0D1laYg.pgp
Description: PGP signature
------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, SlashDot.org! http://sdm.link/slashdot
_______________________________________________ courier-users mailing list courier-users@lists.sourceforge.net Unsubscribe: https://lists.sourceforge.net/lists/listinfo/courier-users
