Re: [www/rt] A flavor for LDAP?

On 2018/03/08 13:08, William Leuschner wrote:
> Here's a patch that adds a short message to the README explaining how to
> add LDAP support to RT. Also, when I tested the packaging, I noticed
> that the configure script was being given some GNU-specific arguments
> that it didn't understand, so I changed the configure script style from
> GNU to simple in addition to bumping the revision.

This shouldn't be CONFIGURE_STYLE=simple - CONFIGURE_STYLE=gnu is
correct already.

> diff -u -p -r1.40 Makefile
> --- Makefile 11 Jan 2018 19:27:12 -0000 1.40
> +++ Makefile 8 Mar 2018 17:50:50 -0000
> @@ -4,7 +4,7 @@ COMMENT= industrial-grade ticketing sys
>
> DISTNAME= rt-4.4.2
> CATEGORIES= www
> -REVISION= 0
> +REVISION= 1
>
> MAINTAINER= Robert Nagy <robert@openbsd.org>
>
> @@ -107,7 +107,7 @@ NO_BUILD= Yes
>
> PKG_ARCH= *
>
> -CONFIGURE_STYLE= gnu
> +CONFIGURE_STYLE= simple
>
> CONFIGURE_ENV+= LDFLAGS="-L${LOCALBASE}/lib"
>
> Index: pkg/README
> ===================================================================
> RCS file: /cvs/ports/www/rt/pkg/README,v
> retrieving revision 1.1
> diff -u -p -r1.1 README
> --- pkg/README 14 Jun 2011 13:19:33 -0000 1.1
> +++ pkg/README 8 Mar 2018 17:50:50 -0000
> @@ -20,3 +20,8 @@ ${PREFIX}/share/doc/rt3/README
> If you are upgrading your installation, don't forget to run:
> # rt-setup-database --dba <yourdbauser> --prompt-for-dba-password \
> --action upgrade
> +
> +If you plan to connect your rt installation to an LDAP domain,
> +install p5-ldap and then run
> +
> +# rt-test-dependencies

Is it actually required to run rt-test-dependencies? I haven't used rt
for some years but from what I remember it's just a checker and doesn't
make any actual changes?