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.
─────────────────
William Leuschner
wel2...@rit.edu
Index: Makefile
===================================================================
RCS file: /cvs/ports/www/rt/Makefile,v
retrieving revision 1.40
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 <rob...@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
On 3/7/2018 8:12 PM, Stuart Henderson wrote:
> On 2018/03/07 20:01, William Leuschner wrote:
>>
>>> On 7 Mar, 2018, at 19:09, Stuart Henderson <s...@spacehopper.org> wrote:
>>>
>>>
>>> That is --enable-externalauth, isn't it?
>>
>> Yup, that’s the one.
>>
>>> i.e. the only thing it does is set a default in rt-test-dependencies
>>> which you can override with "rt-test-dependencies --with-externalauth"
>>> anyway. It doesn't turn any of the rest of the code on or off.
>>
>> Oh, I didn’t know that — I just looked at the configure script to see what
>> option I had to pass to get it to enable LDAP. I’ll give that a try.
>>
>> Would it be helpful for me to provide a patch that explains this in the
>> post-install message, so that other people who want to do the same thing
>> don’t have to rediscover all this?
>>
>> - William
>
> pkg/README would be the place to add it - this is the file that gets
> installed to /usr/local/share/doc/pkg-readmes/rt-$VERSION which is
> mentioned by pkg_add post-install.
>
> Changing this would also need REVISION www/rt/Makefile to be bumped.
>
