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:

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).

CVS-specific code has been removed from Bugzilla 5.0, see bug 1068494. This feature no longer exists.