On Thu, 15 Jun 2017, Ryan Tandy wrote:

> from before my time. I don't have the energy for a wholesale update of the
> code style, but I'm updating pieces to a style like you suggest as I touch

Ah, okay.

> them. The diff was meant as an example of a very minimal change that would be
> suitable for backporting to stable, or applying to a customer system in your
> case.

Well, local reproduction test system for a customer system, but I agree
as to stable.

> I have to check more closely but I think the lack of quoting around the second
> grep may be intentional. I have a vague recollection of a case where it was
> valid to have multiple databases configured for the same suffix.

OK.

> Possibly less portable, but for scripts targeted at Debian I've generally used
> a perl one-liner:
> 
> perl -00 -pe 's/\n[ \t]//g; s/^#.*\n//mg' "$@"

Oh, nice, thanks for sharing.

> In any case I'm trying to (gradually...) update the maintainer scripts to use
> slapcat instead of parsing the config backend files directly, which comes with
> a '-o ldif-wrap=no' option.

That makes sense, yes…

bye,
//mirabilos
-- 
tarent solutions GmbH
Rochusstraße 2-4, D-53123 Bonn • http://www.tarent.de/
Tel: +49 228 54881-393 • Fax: +49 228 54881-235
HRB 5168 (AG Bonn) • USt-ID (VAT): DE122264941
Geschäftsführer: Dr. Stefan Barth, Kai Ebenrett, Boris Esser, Alexander Steeg

Reply via email to