Bug 69854 - Bug fix devel/cvsweb
Summary: Bug fix devel/cvsweb
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Akinori MUSHA
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-07-31 21:30 UTC by mark
Modified: 2004-08-12 14:20 UTC (History)
0 users

See Also:


Attachments
cvsweb.conf.dist.diff (503 bytes, patch)
2004-07-31 21:30 UTC, mark
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description mark 2004-07-31 21:30:20 UTC
Surfing in the my cvs repository by devel/cvsweb I've noted
when I download something I get this error from httpd:

.....
Error: Unexpected output from cvs co: /usr/bin/cvs: invalid
option -- l Usage: cvs [cvs-options] command
[command-options-and-arguments] where cvs-options are -q, -n,
etc.
[...]

I realize "cvs co" doesn't accept the -l option.

root@remotelab # cvs -v

Concurrent Versions System (CVS) '1.11.17'-FreeBSD (client/server)

How-To-Repeat: Install devel/cvsweb port and use it.
Comment 1 Ying-Chieh Liao freebsd_committer freebsd_triage 2004-08-02 03:38:47 UTC
Responsible Changed
From-To: freebsd-ports-bugs->freebsd-cvsweb

over to maintainer
Comment 2 scop freebsd_committer freebsd_triage 2004-08-09 16:59:55 UTC
Responsible Changed
From-To: freebsd-cvsweb->knu

(Over to knu who maintains the cvsweb* ports.) 

This is a known issue.  The current version of cvsweb (3.0.1, 
available as devel/cvsweb3, I'd recommend switching to it) 
documents this a bit better: 

# Options to pass to cvs(1). 
# For cvs versions 1.11 to 1.11.6 (broken in < 1.11, removed in 1.11.7), you 
# can use the '-l' option to prevent cvs from writing to the history file. 
# For other cvs versions, either suppress history logging by using the 
# LogHistory parameter in CVSROOT/config or make sure that the CVSweb user 
# can read and write to CVSROOT/history. 

knu, perhaps it would make sense to apply the patch from this PR 
to the current devel/cvsweb port.  However, not using -l may require 
tweaking of things according to the above.  Your call...
Comment 3 Akinori MUSHA freebsd_committer freebsd_triage 2004-08-12 14:19:00 UTC
State Changed
From-To: open->closed

The obsolete option has been removed from the default value of @cvs_options 
as suggested.  Thanks.