This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 11891 - CVS Edit / Unedit menu items are gone
Summary: CVS Edit / Unedit menu items are gone
Status: RESOLVED DUPLICATE of bug 70639
Alias: None
Product: versioncontrol
Classification: Unclassified
Component: CVS (show other bugs)
Version: 3.x
Hardware: Sun Solaris
: P3 blocker (vote)
Assignee: issues@versioncontrol
URL:
Keywords: UI
Depends on:
Blocks:
 
Reported: 2001-05-02 20:27 UTC by Daniel Kim
Modified: 2007-01-04 17:14 UTC (History)
1 user (show)

See Also:
Issue Type: ENHANCEMENT
Exception Reporter:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Kim 2001-05-02 20:27:42 UTC
For build 010427, the user can no longer access CVS Client > Edit / Unedit
commands (those menu items are completely missing).  Not clear where the user
needs to go in the IDE to set the file as editable.

If this is a temporary disabling, please advise.
Comment 1 Milos Kleint 2001-05-03 07:15:21 UTC
Well. The cvs client (the java implementation of the cvs client and the module 
built on top of it) doesn't support edit command yet. And probably never will 
since it's not possible to set/reset read-only flag on a file from java.
If you want to you the edit command, please mount a command-line cvs support 
module.. (you can decide that in the mounting wizard.)
Comment 2 Michael Ottati 2001-05-03 18:05:40 UTC
The CVS edit command implies more than just setting the file to be writable.
You have not discussed the notification aspect of informing others that you are
working on the file. Editors/Watches etc.

I agree that Java does not support unsetting the read only flag. 

In: http://developer.java.sun.com/developer/bugParade/bugs/4167472.html
JavaSoft states:

QUOTE
In the absence of a way to make an existing file writable, you can always
delete the read-only file and rewrite a new, writable version.  This is what 
many source-code-control systems do anyway.
ENDQUOTE

It seems to me that it is a pretty big deal that we do not support edit in the 
default CVS implementation. Both the submitter and I were surprised by the fact
that the default behavior changed from what it used to be. This is due to 
changing the default client type, something that we now understand. In any 
event, not having anything for edit is confusing, and it seems like it could be 
worked around with the delete/rewrite trick discussed above.





Comment 3 Milos Kleint 2001-05-03 18:30:36 UTC
Cool. I didn't realize the trick with deleting RO and writing RW file.
If this works, then the cvs edit could be supported.

Informing others is the job for the server (as far as I understand it right now 
- didn't take a closer look at the cvs protocol for the edit/watch commands)

It doesn't seem as such a big deal to me actually. The set of commands that is 
supported should be part of the documentation I think. The javacvs module was 
picked as default because it doesn't need a platform dependant cvs.exe file. 
There are some other limitations to it, but for the :pserver: access it was 
decided to have the javacvs as default. It might be a wrong decision.
However
cvs edit and everything that 
wraps around it (watchers/ watch/...) is not the default cvs behaviour. (IMHO). 
This is something that was added to satisfy the " everyone reads, one modifies" 
approach that is not native to cvs. That's it was decided to implement other 
commands before edit etc.


Comment 4 Milos Kleint 2001-05-03 19:05:17 UTC
I've read the entry in the bugparade and the workaround with deleting is not 
100% bulletproof (thus not ideal solution).. It depends on the platform. :(
Comment 5 Jan Chalupa 2001-11-27 15:13:02 UTC
Target milestone -> 3.3.1.
Comment 6 Filip Rachunek 2002-04-30 15:37:40 UTC
UI keyword added.
Comment 7 Marek Grummich 2002-07-22 10:40:10 UTC
Set target milestone to TBD
Comment 8 Marek Grummich 2002-07-22 10:41:29 UTC
Set target milestone to TBD
Comment 9 Maros Sandor 2006-12-18 15:42:13 UTC

*** This issue has been marked as a duplicate of 70639 ***