Re: unclarities in message

2007-05-16 Thread Jim Meyering
Benno Schulenberg <[EMAIL PROTECTED]> wrote: > Jim Meyering wrote: >> Benno Schulenberg <[EMAIL PROTECTED]> wrote: >> > -{"no-update", no_argument, NULL, 'd'}, >> > +{"dry-run", no_argument, NULL, 'd'}, >> >> Looks good, but do not remove support for --no-update just yet. >> Instead, mark i

Re: unclarities in message

2007-05-16 Thread Benno Schulenberg
Jim Meyering wrote: > Benno Schulenberg <[EMAIL PROTECTED]> wrote: > > -{"no-update", no_argument, NULL, 'd'}, > > +{"dry-run", no_argument, NULL, 'd'}, > > Looks good, but do not remove support for --no-update just yet. > Instead, mark it as deprecated, and with a comment suggesting > when

Re: unclarities in message

2007-05-16 Thread Jim Meyering
Benno Schulenberg <[EMAIL PROTECTED]> wrote: > diff --git a/partprobe/partprobe.c b/partprobe/partprobe.c > index 1659100..e79193a 100644 > --- a/partprobe/partprobe.c > +++ b/partprobe/partprobe.c > @@ -57,7 +57,7 @@ > > static struct option const long_options[] = >{ > -{"no-update", no_a

Re: unclarities in message

2007-05-16 Thread Benno Schulenberg
Otavio Salvador wrote: > Benno Schulenberg <[EMAIL PROTECTED]> writes: > > Better use the same word in both cases then: > > > > Inform the operating system about partition table changes. > > > > -d, --dry-run do not actually inform the operating system > > Ack. Can you provide a patch for i

Re: unclarities in message

2007-05-16 Thread Otavio Salvador
Benno Schulenberg <[EMAIL PROTECTED]> writes: > Hi, > > In partprobe/partprobe.c it says: > > Inform the OS of partition table changes. > > -d, --no-update don't update the kernel > > To me "updating the kernel" means to install a new version of the > kernel. But it seems that "don't i