[rrd-developers] [PATCH] rrd_dump and XML schema definition

[Tobias Oetiker]

Today Sebastian Harl wrote:

> Hi,
>
> On Tue, Oct 28, 2008 at 01:40:37AM -0700, tlindenmann wrote:
> >    - change the argument "[--no-header|-n]" to [--header|-h {xsd,dtd}]
> > (default "no header output")
>
> Hrm ... this breaks backwards compatibility of the user interface and it
> changes the current default behavior. I'd rather suggest to do the
> following:
>
>  - keep the --no-header|-n option
>  - create a DTD header by default
>  - use --header|-h to change that default
>  - complain if both, -n and -h are used

you wouldn't even have to complain since having a different
default is fine ...

>
> This should preserve the old behavior, thus not breaking possibly
> existing scripts and you may _explicitly_ specify whether to create or
> not to create a header, which imho is a much nicer interface.

fine ... with me ...

cheers
tobi


> Comments?

>
> Cheers,
> Sebastian
>
>

-- 
Tobi Oetiker, OETIKER+PARTNER AG, Aarweg 15 CH-4600 Olten, Switzerland
http://it.oetiker.ch tobi at oetiker.ch ++41 62 775 9902 / sb: -9900