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 140948 - NB offers and does port setting for ext protocol
Summary: NB offers and does port setting for ext protocol
Status: REOPENED
Alias: None
Product: versioncontrol
Classification: Unclassified
Component: CVS (show other bugs)
Version: 6.x
Hardware: All Windows XP
: P3 blocker (vote)
Assignee: issues@versioncontrol
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-07-21 09:55 UTC by giorgio42
Modified: 2010-07-30 09:55 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description giorgio42 2008-07-21 09:55:02 UTC
This is a bug in an NB65M1, but can probably found in all version at least since 6.0.

I inadvertently put the standard pserver port 2401 into a ext: protocol setup (the GUI offers this).
Then I added several projects to my CVS repository. Now when I issue any cvs command from the command line, I get a slew
of warnings about port setting not supported with the ext: protocol. I tested this with cvs 1.11.22 and 1.12.12.

This is probably also a bug in the UI spec.
P2, because in this way NB allows or even encourages me to create corrupted data.


(Here a snippet of the warnings, one warning per CVS subdirectory):
cvs update: ignoring CVS/Root because it does not contain a valid root.
cvs update: CVSROOT port specification is only valid for gserver, kserver,
cvs update: and pserver connection methods.
cvs update: in directory webapp/WEB-INF:
cvs update: ignoring CVS/Root because it does not contain a valid root.
cvs update: CVSROOT port specification is only valid for gserver, kserver,
cvs update: and pserver connection methods.
cvs update: in directory webapp/WEB-INF/classes:
cvs update: ignoring CVS/Root because it does not contain a valid root.
cvs update: CVSROOT port specification is only valid for gserver, kserver,
cvs update: and pserver connection methods.
cvs update: in directory webapp/WEB-INF/classes:
cvs update: ignoring CVS/Root because it does not contain a valid root.
cvs update: CVSROOT port specification is only valid for gserver, kserver,
cvs update: and pserver connection methods.
... and so on ...

Georg
Comment 1 Maros Sandor 2008-07-22 08:00:48 UTC
What is your cvs root string then?
A simple fix in your case is to run CVS/Change CVS Root and enter correct cvs root there.
We'll add a check for that.
Comment 2 giorgio42 2008-07-22 21:33:51 UTC
Maros,

the root string is like :ext:itsme@where.ami.org:2401:/cvs/foo/bar. The :2401 part is obviously forbidden with :ext:.

I was aware of the change root functionality and I used it on my windows PC, but my Solaris box is so underpowered (Sun
Blade 100 with 512 MB RAM...) that it was faster to fix the problem using bash and sed instead of waiting for NB6.1 to
do the job.

Thanks,
Georg

Comment 3 Maros Sandor 2008-07-31 12:29:31 UTC
Added checks for port number
ce0946917366
Comment 4 Quality Engineering 2008-07-31 15:44:14 UTC
Integrated into 'main-golden', available in build *200807311401* on http://bits.netbeans.org/dev/nightly/
Changeset: http://hg.netbeans.org/main/rev/ce0946917366
User: Maros Sandor <msandor@netbeans.org>
Log: #140948: Checking for correct CVSROOT format: port is only allowed for pserver method
Comment 5 Maros Sandor 2008-08-18 15:29:35 UTC
I had to roll the fix back since it caused a regression, see Issue #143878.
c8071affabc4