cyrusimap / cyrus-sasl

Other
134 stars 151 forks source link

`SASL_PATH` no longer has any effect but is still documented #852

Open flowerysong opened 4 months ago

flowerysong commented 4 months ago

In 79827fbe559318553d6ccdf5ec37ae6778b4c367 (part of #745) SASL_PLUGINDIR replaced SASL_PATH as the environment variable used to configure the plugin directory at runtime. It's not clear that this was a deliberate change or what motivated it, since it's not mentioned in the commit message.

SASL_PATH is still referenced multiple times in the documentation, e.g. https://github.com/cyrusimap/cyrus-sasl/blob/537af16986caff73ef45ab54a5a020bfb636ddaf/docsrc/sasl/installation.rst?plain=1#L44 and https://github.com/cyrusimap/cyrus-sasl/blob/537af16986caff73ef45ab54a5a020bfb636ddaf/docsrc/sasl/quickstart.rst?plain=1#L61

It also still has a vestigial presence in configure.ac, though nothing references this definition: https://github.com/cyrusimap/cyrus-sasl/blob/537af16986caff73ef45ab54a5a020bfb636ddaf/configure.ac#L1391