FreeBSD Port: openldap-server-2.4.10

[Mikhail Goriachev]

Quoting Peter Pentchev <roam at ringlet.net>:

> On Fri, Jul 04, 2008 at 10:34:32PM -0400, Mikhail Goriachev wrote:
>> Quoting Peter Pentchev <roam at ringlet.net>:
>>
>> > On Fri, Jul 04, 2008 at 01:22:15PM -0400, Mikhail Goriachev wrote:
>> >> Hi,
>> > [snip]
>> >> I slapped together a workaround. Here's a "patch", maybe the idea of
>> >> it will be of some use.
>> >
>> > Just a minor comment on the patch:
>> >
>> >> +DBDIR=`grep directory /usr/local/etc/openldap/slapd.conf | awk '{
>> >> print $2 }'`
>> >
>> > This is better written as
>> >
>> >   awk '/directory/ {print $2}' /usr/local/etc/openldap/slapd.conf
>>
>> Nice one!
>>
>> > or possibly (I'm not quite familiar with the slapd.conf syntax) even:
>> >
>> >   awk '$1 == "directory" {print $2}' /usr/local/etc/openldap/slapd.conf
>>
>> They both work but I like the first one more.
>
> Actually the second one might be better - the first one may be confused
> by the string "directory" appearing either in a comment or in the string
> value of some other parameter or even in the *name* of some other
> parameter.


Oh, didn't really see the meaning of the second line before. Thanks a  
lot for that.


And the patch goes again:


--- slapd.sh.in.orig    2008-07-05 18:47:30.000000000 +0000
+++ slapd.sh.in 2008-07-05 18:53:00.000000000 +0000
@@ -39,6 +39,8 @@

  # extract user and group, adjust ownership of directories and database

+DBDIR=`awk '$1 == "directory" {print $2}'  
"%%PREFIX%%/etc/openldap/slapd.conf"`
+
  start_precmd()
  {
    local slapd_ownername slapd_groupname
@@ -48,8 +50,8 @@
      ;;
    *)
      chown "$slapd_owner" "%%LDAP_RUN_DIR%%"
-    chown -RL "$slapd_owner" "%%DATABASEDIR%%"
-    chmod 700 "%%DATABASEDIR%%"
+    chown -RL "$slapd_owner" "${DBDIR}"
+    chmod 700 "${DBDIR}"
      chown "$slapd_owner" "%%PREFIX%%/etc/openldap/slapd.conf"

      slapd_ownername="${slapd_owner%:*}"





Regards,
Mikhail.


-- 
Mikhail Goriachev
Webanoide