Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 14 Jan 2003 12:08:21 +0200
From:      Sheldon Hearn <sheldonh@starjuice.net>
To:        Matt Dillon <dillon@FreeBSD.org>
Cc:        cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sbin/dump Makefile cache.c dump.8 dump.h main.c traverse.c
Message-ID:  <20030114100821.GE61405@starjuice.net>
In-Reply-To: <200301131942.h0DJgfbN010760@repoman.freebsd.org>
References:  <200301131942.h0DJgfbN010760@repoman.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On (2003/01/13 11:42), Matt Dillon wrote:

>   Modified files:
>     sbin/dump            Makefile dump.8 dump.h main.c traverse.c 
>   Added files:
>     sbin/dump            cache.c 
>   Log:
>   Add a caching option to dump.  Use -C.  Note that NetBSD has a caching option
>   called -r but it takes 512 byte blocks instead of megabytes, and I felt a
>   megabytes specification would be far more useful so I did not use the same
>   option character.

[...]

First, let me say that this looks very cool.

>   MFC after:      3 days

However, I'd like you to wait with respect to merging.

I don't want this option to hit production lines until it's clear that
the compatibility camp is okay with offering similar functionality to
that found in NetBSD, without using the same option.

For example, if OpenBSD also add NetBSD's -r, then we're the odd man
out.  If the only justification is the one you gave in your commit log
(megabytes are easier to work with than 512 byte blocks), then I suspect
it might be worthwhile to get in line with NetBSD before merging.

So could you hold off for, say 2 weeks, before merging?  I think this
will allow enough time for the issue to be discussed and resolved.

Ciao,
Sheldon.

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20030114100821.GE61405>