Re: add cvs -W option to disable -R/CVSREADONLYFS

2004-03-12 Thread Larry Jones
Norikatsu Shigemura writes:
> 
>   Although I am checkout-ing well by using cvs -R in ~/.cvsrc,
>   when I want to commit, It is prevented by -R.  So I wanted to
>   disable this feature.

Use the global -f option to ignore ~/.cvsrc.

-Larry Jones

Ever notice how tense grown-ups get when they're recreating? -- Calvin
___
[EMAIL PROTECTED] mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-hackers
To unsubscribe, send any mail to "[EMAIL PROTECTED]"


Re: add cvs -W option to disable -R/CVSREADONLYFS

2004-03-09 Thread Norikatsu Shigemura
On Tue, 9 Mar 2004 15:50:28 +0200
Ruslan Ermilov <[EMAIL PROTECTED]> wrote:
> > Although I am checkout-ing well by using cvs -R in ~/.cvsrc,
> > when I want to commit, It is prevented by -R.  So I wanted to
> > disable this feature.  Although control by the environment
> > variable CVSREADONLYFS was also considered, honestly it is
> > troublesome.
> I also have -R in most of my ~/.cvsrc's, and when I need to commit,
> I just use "cvs -f" (for local repos).  For remote repos, -R is not

Ah, I didn't notice it!  It's a good idea.

> a problem (it's not passed on the wire) unless you happen to have
> it set in ~/.cvsrc on the repository host.  Even in this case, it

Yes, so I did not care.  Recently I noticed.

> is still possible to commit by setting CVS_SERVER="cvs -f" in the
> environment of the committing host.

I almost used and tuned for ports.  But now, I'm using local
repositry for linuxpluginwrapper and my products.  I'm tired
to change mode:-).
___
[EMAIL PROTECTED] mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-hackers
To unsubscribe, send any mail to "[EMAIL PROTECTED]"


Re: add cvs -W option to disable -R/CVSREADONLYFS

2004-03-09 Thread Ruslan Ermilov
On Tue, Mar 09, 2004 at 09:33:44PM +0900, Norikatsu Shigemura wrote:
> On Tue, 9 Mar 2004 20:17:34 +0900 (JST)
> Norikatsu Shigemrua <[EMAIL PROTECTED]> wrote:
> > >Synopsis:  add cvs -W option to disable -R/CVSREADONLYFS
> > >Description:
> > I added a feature which is to nagate a effect of -R/CVSREADONLYFS
> > to cvs as -W option.  Please review and commit.
> 
>   Although I am checkout-ing well by using cvs -R in ~/.cvsrc,
>   when I want to commit, It is prevented by -R.  So I wanted to
>   disable this feature.  Although control by the environment
>   variable CVSREADONLYFS was also considered, honestly it is
>   troublesome.
> 
I also have -R in most of my ~/.cvsrc's, and when I need to commit,
I just use "cvs -f" (for local repos).  For remote repos, -R is not
a problem (it's not passed on the wire) unless you happen to have
it set in ~/.cvsrc on the repository host.  Even in this case, it
is still possible to commit by setting CVS_SERVER="cvs -f" in the
environment of the committing host.


Cheers,
-- 
Ruslan Ermilov
FreeBSD committer
[EMAIL PROTECTED]


pgp0.pgp
Description: PGP signature


Re: add cvs -W option to disable -R/CVSREADONLYFS

2004-03-09 Thread Norikatsu Shigemura
On Tue, 9 Mar 2004 20:17:34 +0900 (JST)
Norikatsu Shigemrua <[EMAIL PROTECTED]> wrote:
> >Synopsis:    add cvs -W option to disable -R/CVSREADONLYFS
> >Description:
>   I added a feature which is to nagate a effect of -R/CVSREADONLYFS
>   to cvs as -W option.  Please review and commit.

Although I am checkout-ing well by using cvs -R in ~/.cvsrc,
when I want to commit, It is prevented by -R.  So I wanted to
disable this feature.  Although control by the environment
variable CVSREADONLYFS was also considered, honestly it is
troublesome.

I was pointed out by some people.
1. What to control by alias?  Yes, I forgot it.  But I think
   it is troublesome.
2. There is a different from -R -W and -W -R.  Is this OK?
3. This patch is not commited by cvshome.  So you should
   avoid description of -W option?  I think so.
___
[EMAIL PROTECTED] mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-hackers
To unsubscribe, send any mail to "[EMAIL PROTECTED]"


Re: add cvs -W option to disable -R/CVSREADONLYFS

2004-03-09 Thread Norikatsu Shigemura
Hi peter.

On Tue, 9 Mar 2004 20:17:34 +0900 (JST)
Norikatsu Shigemrua <[EMAIL PROTECTED]> wrote:
> >Synopsis:    add cvs -W option to disable -R/CVSREADONLYFS
> >Description:
>   I added a feature which is to nagate a effect of -R/CVSREADONLYFS
>   to cvs as -W option.  Please review and commit.
> >How-To-Repeat:
>   N/A
> >Fix:
>   Following patches are for cvs-1.12.5.

I wrote for FreeBSD's cvs.  Please, please!
- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
Index: src/contrib/cvs/man/cvs.1
===
RCS file: /home/ncvs/src/contrib/cvs/man/cvs.1,v
retrieving revision 1.20
diff -u -r1.20 cvs.1
--- src/contrib/cvs/man/cvs.1   13 Nov 2003 17:57:53 -  1.20
+++ src/contrib/cvs/man/cvs.1   9 Mar 2004 10:44:33 -
@@ -244,7 +244,8 @@
 .SM CVSREADONLYFS
 environment variable is set.  Using
 .B \-R
-can also considerably speed up checkout's over NFS.
+can also considerably speed up checkout's over NFS.  If you want to disable
+this feature, you should specify -W option.
 .TP
 .B \-v [ --version ]
 Displays version and copyright information for
@@ -255,6 +256,10 @@
 Overrides the setting of the
 .SM CVSREAD
 environment variable.
+.TP
+.B \-W
+Turns off read-only repository mode.  This is default mode, but it
+typically uses to negate the effect of -R option.
 .TP
 .B \-g
 Forces group-write perms on working files.  This option is typically
Index: src/contrib/cvs/src/main.c
===
RCS file: /home/ncvs/src/contrib/cvs/src/main.c,v
retrieving revision 1.23
diff -u -r1.23 main.c
--- src/contrib/cvs/src/main.c  7 Jul 2003 19:15:36 -   1.23
+++ src/contrib/cvs/src/main.c  9 Mar 2004 10:44:33 -
@@ -256,6 +256,7 @@
 "-n   Do not execute anything that will change the disk.\n",
 "-t   Show trace of program execution -- try with -n.\n",
 "-R   Assume repository is read-only, such as CDROM\n",
+"-W   Assume repository is writable (default).\n",
 "-v   CVS version and copyright.\n",
 "-T tmpdirUse 'tmpdir' for temporary files.\n",
 "-e editorUse 'editor' for editing log information.\n",
@@ -413,7 +414,7 @@
 int help = 0;  /* Has the user asked for help?  This
   lets us support the `cvs -H cmd'
   convention to give help for cmd. */
-static const char short_options[] = "+QqgrwtnRlvb:T:e:d:Hfz:s:xaU";
+static const char short_options[] = "+QqgrwtnRWlvb:T:e:d:Hfz:s:xaU";
 static struct option long_options[] =
 {
 {"help", 0, NULL, 'H'},
@@ -556,6 +557,10 @@
break;
case 'R':
readonlyfs = 1;
+   logoff = 1;
+   break;
+   case 'W':
+   readonlyfs = 0;
logoff = 1;
break;
case 'n':
- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
___
[EMAIL PROTECTED] mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-hackers
To unsubscribe, send any mail to "[EMAIL PROTECTED]"


add cvs -W option to disable -R/CVSREADONLYFS

2004-03-09 Thread Norikatsu Shigemrua
>Submitter-Id:  net
>Originator:Norikatsu Shigemura
>Organization:  net
>Confidential:  no
>Synopsis:  add cvs -W option to disable -R/CVSREADONLYFS
>Severity:  non-critical
>Priority:  low
>Category:  cvs
>Class: change-request
>Release:   1.12.5
>Environment:
System: FreeBSD melfina.ninth-nine.com 5.2-CURRENT FreeBSD 5.2-CURRENT #12: Sun Mar  7 
21:11:13 JST 2004 [EMAIL PROTECTED]:/usr/obj/usr/src/sys/MELFINA i386

>Description:
I added a feature which is to nagate a effect of -R/CVSREADONLYFS
to cvs as -W option.  Please review and commit.
>How-To-Repeat:
N/A
>Fix:
Following patches are for cvs-1.12.5.
- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
--- cvs-1.12.5/src/main.c.orig  Wed Nov 26 01:11:47 2003
+++ cvs-1.12.5/src/main.c   Tue Mar  9 19:50:09 2004
@@ -252,6 +252,7 @@ static const char *const opt_usage[] =
 "-t   Show trace of program execution (repeat for more\n",
 " verbosity) -- try with -n.\n",
 "-R   Assume repository is read-only, such as CDROM\n",
+"-W   Assume repository is writable (default).\n",
 "-v   CVS version and copyright.\n",
 "-T tmpdirUse 'tmpdir' for temporary files.\n",
 "-e editorUse 'editor' for editing log information.\n",
@@ -423,7 +424,7 @@ main (int argc, char **argv)
 int help = 0;  /* Has the user asked for help?  This
   lets us support the `cvs -H cmd'
   convention to give help for cmd. */
-static const char short_options[] = "+QqrwtnRvb:T:e:d:Hfz:s:xa";
+static const char short_options[] = "+QqrwtnRWvb:T:e:d:Hfz:s:xa";
 static struct option long_options[] =
 {
 {"help", 0, NULL, 'H'},
@@ -563,6 +564,10 @@ main (int argc, char **argv)
break;
case 'R':
readonlyfs = -1;
+   logoff = 1;
+   break;
+   case 'W':
+   readonlyfs = 0;
logoff = 1;
break;
case 'n':
--- cvs-1.12.5/man/cvs.1.orig   Sat Mar 29 01:02:33 2003
+++ cvs-1.12.5/man/cvs.1Tue Mar  9 19:51:21 2004
@@ -243,7 +243,8 @@ Same effect as if the
 .SM CVSREADONLYFS
 environment variable is set.  Using
 .B \-R
-can also considerably speed up checkout's over NFS.
+can also considerably speed up checkout's over NFS.  If you want to disable
+this feature, you should specify -W option.
 .TP
 .B \-v [ --version ]
 Displays version and copyright information for
@@ -254,6 +255,10 @@ Makes new working files read-write (defa
 Overrides the setting of the
 .SM CVSREAD
 environment variable.
+.TP
+.B \-W
+Turns off read-only repository mode.  This is default mode, but it
+typically uses to negate the effect of -R option.
 .TP
 .B \-x
 Encrypt all communication between the client and the server.  As of
--- cvs-1.12.5/doc/cvs.texinfo.orig Wed Dec 10 05:09:32 2003
+++ cvs-1.12.5/doc/cvs.texinfo  Tue Mar  9 19:53:16 2004
@@ -8151,7 +8151,12 @@ repository.
 
 Same effect as if the @code{CVSREADONLYFS} environment
 variable is set. Using @samp{-R} can also considerably
-speed up checkout's over NFS.
+speed up checkout's over NFS.  If you want to disable
+this feature, you should specify @samp{-W} option.
+
[EMAIL PROTECTED] -W
+Turns off read-only repository mode.  This is default mode,
+but it typically uses to negate the effect of @samp{-R} option.
 
 @cindex Read-only mode
 @item -n
- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
___
[EMAIL PROTECTED] mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-hackers
To unsubscribe, send any mail to "[EMAIL PROTECTED]"