Simple patch for pts.c

Jeffrey Hutzelman jhutz at cmu.edu
Mon Nov 20 23:55:15 CET 2006



On Thursday, November 16, 2006 01:26:39 PM +0100 Tomas Olsson 
<tol at it.su.se> wrote:

> Jeffrey Hutzelman <jhutz at cmu.edu> writes:
>
>> On Friday, November 10, 2006 10:45:20 AM +0100 Tomas Olsson
>> <tol at it.su.se>  wrote:
>>
>> > Thanks. The original mistake was most likely in that this is the only
>> > place where the host argument was added after 'cell', so I'd prefer
>> > just reordering the command's arguments for consistency. Would that
>> > cause any problems (like widely deployed arla-dependent scripts for
>> > example)?
>> >
>> > It would also make it slightly less different from the OpenAFS version,
>> > which is good.
>>
>> I'm not sure I follow this.  OpenAFS doesn't have a -host argument to
>> pts  chown, or any other pts subcommand.  Maximal compatibility with
>> OpenAFS  would require putting any new arguments last, and preferably
>> requiring that  the corresponding switches be given explicitly.
>>
> Good catch. It was meant to read "...was added after 'cell'".
>
> You're very correct, hopefully what went into CVS was almost as
> correct. We ended up with
>   pts chown [-name] user/group [-owner] owner [-cell cellname] [-host
> server] [-noauth] which should match the other commands that got a 'host'
> arg back in the good ol' days.

Yup, that looks good.


More information about the Arla-drinkers mailing list