Closed
Bug 229991
Opened 21 years ago
Closed 10 years ago
context=file fails with :ext: style cvsroot
Categories
(Bugzilla :: Attachments & Requests, defect)
Bugzilla
Attachments & Requests
Tracking
()
RESOLVED
INVALID
People
(Reporter: joseph, Unassigned)
References
Details
User-Agent: Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.1; .NET CLR 1.1.4322)
Build Identifier:
Attempting to use the 'file' link in the patchviewer gives:
Could not check out foo/bar.c r1.159 from :ext:cvs:/export/cvs
at /usr/local/share/perl/5.6.1/PatchReader/AddCVSContext.pm line 191.
The webserver log says:
cvs [checkout aborted]: cannot exec rsh: No such file or directory
cvs [checkout aborted]: end of file from server (consult above messages if any)
[Sat Jan 3 22:39:36 2004] attachment.cgi: Could not check out foo/bar.c r1.159
from :ext:cvs:/export/cvs
at /usr/local/share/perl/5.6.1/PatchReader/AddCVSContext.pm line 191.
This is with patchreader version 0.9.2. rsh is in /usr/bin/rsh. bugzilla
version is the current HEAD.
A workaround is:
Index: globals.pl
===================================================================
RCS file: /cvsroot/mozilla/webtools/bugzilla/globals.pl,v
retrieving revision 1.251
diff -u -r1.251 globals.pl
--- globals.pl 22 Nov 2003 03:50:38 -0000 1.251
+++ globals.pl 3 Jan 2004 22:56:53 -0000
@@ -85,7 +85,7 @@
# Cwd.pm in perl 5.6.1 gives a warning if $::ENV{'PATH'} isn't defined
# Set this to '' so that we don't get warnings cluttering the logs on every
# system call
-$::ENV{'PATH'} = '';
+$::ENV{'PATH'} = '/usr/bin';
# Ignore SIGTERM and SIGPIPE - this prevents DB corruption. If the user closes
# their browser window while a script is running, the webserver sends these
This seems unlikely to be the proper fix though!
Reproducible: Always
Steps to Reproduce:
Updated•19 years ago
|
QA Contact: mattyt-bugzilla → default-qa
Updated•18 years ago
|
Assignee: myk → attach-and-request
Updated•18 years ago
|
Blocks: bz-patch
OS: Windows XP → All
Hardware: PC → All
Summary: patchviewer context=file fails with :ext: style cvsroot → [PatchReader] context=file fails with :ext: style cvsroot
Comment 3•17 years ago
|
||
It's probably a safer fix to put a path to rsh or ssh in localconfig and set the CVS_RSH environment variable appropriately based on it. This is something that could be done without pushing upstream to PatchReader (unless it's for PatchReader's docs).
Status: UNCONFIRMED → NEW
Ever confirmed: true
Summary: [PatchReader] context=file fails with :ext: style cvsroot → context=file fails with :ext: style cvsroot
Comment 4•10 years ago
|
||
CVS-specific code has been removed from Bugzilla 5.0, see bug 1068494. This feature no longer exists.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → INVALID
You need to log in
before you can comment on or make changes to this bug.
Description
•