[cyrusimap/cyrus-sasl] configure.ac: add warning about plugindir mismatch (issue #473) (2ea0649)
jan parcel
jan.parcel at oracle.com
Fri Sep 15 12:43:15 EDT 2017
On 9/15/2017 9:10 AM, flowerysong wrote:
>
> I'm not sure what the utility is to having a hard-coded default
> install location that doesn't respect the setting from
> --with-plugindir; it would seem better to me to change
> https://github.com/cyrusimap/cyrus-sasl/blob/2ea0649f5cafce4f0b597d82f7acca7721e96cc8/plugins/Makefile.am#L66
> to set sasldir to the same value as plugindir. People who need the
> current behaviour would be able to override sasldir with e.g. |make
> sasldir=/usr/lib/sasl2| (an approach we currently use to make sure
> they're installed to plugindir) but it would be more difficult for
> people to end up with a broken install.
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/cyrusimap/cyrus-sasl/commit/2ea0649f5cafce4f0b597d82f7acca7721e96cc8#commitcomment-24326253>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AWZUI8FNnnQBYCtKaDik6XMgWmrxhFkzks5siqExgaJpZM4PZMes>.
>
For me, that is the issue....I need sasldir and plugindir to match, but
there is no way to set sasldir, so I had to patch it to be settable.
My patch also checks that the 2 match, INSTEAD of comparing to something
hard-coded. Having them match anyway would fix that.
- @if test "$(plugindir)" != "$(prefix)/lib/sasl2"; then \
+ @if [ "X$(sasldir)" != "X$(plugindir)" ] ; then \
Here's what I do in Makefile after patching:
SASL2.32 = $(USRLIBDIR)/sasl2
SASL2.64 = $(USRLIBDIR)/sasl2/$(MACH64)
CONFIGURE_OPTIONS.32 += --with-plugindir=$(SASL2.32)
CONFIGURE_OPTIONS.64 += --with-plugindir=$(SASL2.64)
CONFIGURE_OPTIONS.32 += --with-sasldir=$(SASL2.32)
CONFIGURE_OPTIONS.64 += --with-sasldir=$(SASL2.64)
That's a lot of trouble to go to just to get them to install and be
looked for in the same place.
--
Jan Parcel, Software Developer
Oracle Systems Server & Cloud Engineering
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.andrew.cmu.edu/pipermail/cyrus-sasl/attachments/20170915/344fc17f/attachment.html>
More information about the Cyrus-sasl
mailing list