Re: slapadd shows wrong error message on non-readable ldif file (ITS#4044)

[ando@sys-net.it]

>>
>>> On Tue, Sep 27, 2005 at 02:25:30PM +0000, Howard Chu wrote:
>>>> The error message is correct. See the slapadd(1) manpage, the "-f"
>>>> option is for
>>>> specifying a config file, and you have specified an invalid config
>>>> file.
>>>> You
>>>> want the "-l" option for specifying an LDIF input file. As there is no
>>>> software
>>>> bug here, this ITS will be closed.
>>>
>>> Yes, I understand the invocation error.  But compare the error message
>>> with the one given by -l on an unreadable file: that one _does_ tell
>>> me the file is unreadable.  There is still an erroneous error message
>>> in the -f case.
>>
>> With HEAD code (where LDAP_DEVEL is enabled, implying
>> SLAPD_CONF_UNKNOWN_BAILOUT is enabled as well), invoking slaptools with
>> an
>> invalid config file results in
>>
>> ando@ando:> slapadd -f data/test-ordered.ldif
>> slapadd: bad configuration file!
>
> This is still not an adequate message when the file is read-only and
> slappadd cannot tell anything about its contents.

I don't quite understand the "read-only" issue; what do you expect slapadd
to do with both the configuration and the LDIF files, apart from reading
them?  And what would be the appropriate message if one passes garbage to
the switch that's supposed to indicate a configuration file?

p.

-- 
Pierangelo Masarati
mailto:pierangelo.masarati@sys-net.it


    SysNet - via Dossi,8 27100 Pavia Tel: +390382573859 Fax: +390382476497