[Date Prev][Date Next] [Chronological] [Thread] [Top]

Re: (ITS#3705) enforce strict parsing of slapd.conf



hyc@symas.com wrote:

>ando@sys-net.it wrote:
>  
>
>>I have committed a patch to HEAD that enforces strict parsing of
>>slapd.conf; it's enabled when LDAP_DEVEL is #define'd.  It passes the
>>whole test suite of HEAD.  If there's consensus, I'd enable it by default.
>>
>>    
>>
>I'm seeing an immediate failure in test000 due to bad parsing of the 
>copyright header in slapd.1.conf. No one else running into this?
>
>  
>
This is what the top of testrun/slapd.1.conf looks like:

# stand-alone slapd config -- for testing (with indexing)
# $OpenLDAP: pkg/ldap/tests/data/slapd-schema.conf,v 1.20.2.4 2003/12/15 
22:05:29 kurt Exp $
 This work is part of OpenLDAP Software <http://www.openldap.org/>.

 Copyright 1998-2005 The OpenLDAP Foundation.
 All rights reserved.

 Redistribution and use in source and binary forms, with or without
 modification, are permitted only as authorized by the OpenLDAP
 Public License.

 A copy of this license is available in the file LICENSE in the
 top-level directory of the distribution or, alternatively, at
 <http://www.OpenLDAP.org/license.html>.

#
include ./schema/core.schema

Note the double '##' copyright comments have been deleted by the conf.sh 
script. It appears to do this because it substitutes
    s/^#${RELAY}#//
and $RELAY is not defined by default.

-- 
  -- Howard Chu
  Chief Architect, Symas Corp.       Director, Highland Sun
  http://www.symas.com               http://highlandsun.com/hyc
  Symas: Premier OpenSource Development and Support