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 <[email protected]> > > @@ -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?
