patchConcurrent Versions System - Patches: patch #5053, adds new cvs command, 'passwd', to...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

patch #5053: adds new cvs command, 'passwd', to change cvs password using cvs client

Submitter:  None
Submitted:  Wed 26 Apr 2006 07:03:32 AM UTC
   
 
Category:  Feature (patch attached) Priority:  5 - Normal
Status:  None Privacy:  Public
Assigned to:  None Originator Email:  -email is unavailable-
Open/Closed:  Open Fixed Release:  None
Fixed Feature Release:  None

Sat 29 Apr 2006 06:28:10 PM UTC, comment #2: 

I was just looking over at the CVSNT source and they already
provide a 'passwd' command. So, it would probably be good to
ensure that anything added to CVS will either interoperate
correctly, or be called something else completely. I suggest
you update your patch so that it avoids the name "passwd" at
present to avoid interoperation problems with CVSNT clients
and servers.

Options on the CVSNT 'passwd' command are apparently as
follows:

  -a              Add user
  -x              Disable user
  -X              Delete user
  -r real_user    Alias username to real system user
  -R              Remove alias to real system user
  -D domain       Use domain password

You may use the command:

    cvs -d :pserver:cvs@cvs.cvsnt.org:/usr/local/cvs checkout cvsnt

to checkout the CVSNT sources. Documentation of the CVSNT
client/server protocol should be in doc/cvsclient.dbk (XML
document text).

Note that it will also be desirable for your patch to add
the documentation for the 'passwd' protocol to the
doc/cvsclient.texi file.

Mark D. Baushke <mdb>
Group administrator
Thu 27 Apr 2006 01:58:38 AM UTC, comment #1: 

I have replied in private e-mail with suggestions about
coding style and reported some bugs with this patch.

The biggest problem with the patch as provided is that
is assumes that flock() and LOCK_EX exist on all platforms
and this is a bad assumption. For example, Solaris 9 does
not provide this functionality. It may be desirable to look
at how the GNULIB getpass function does exclusivity locking
as an alternative approach.

By the way, the patch is no patch presently attached to this
case.

Mark D. Baushke <mdb>
Group administrator
Wed 26 Apr 2006 07:03:32 AM UTC, original submission:  

The cvs-passwd patch is meant add new command named
'passwd' to the cvs; this is to facilitate the cvs user,
to change her cvs pserver password, at her will, by
using the cvs command line client program.

The patch is been created on my machine running FC3 and is made
for cvs-1.12.13 source tree.

Anonymous

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attached Files
file #12328:  cvs-passwd.patch added by None (54KiB - text/x-patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

CC list is empty

 

There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

 

Follows 1 latest change.

Date Changed by Updated Field Previous Value => Replaced by
2007-03-29 None Attached File- Added cvs-passwd.patch, #12328

Back to the top

Powered by Savane 3.13-ec04.
Corresponding source code