This is the mail archive of the binutils@sources.redhat.com mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: NEWS link for gas


Ian Lance Taylor <ian@wasabisystems.com> writes:

> > > Overseers, any idea why:
> > >   http://sources.redhat.com/cgi-bin/cvsweb.cgi/~checkout~/src/gas/NEWS?only_with_tag=binutils-2_15&content-type=text/plain&cvsroot=src
> > > 
> > > returns a server error?
> > 
> > Because it should be this:
> > 
> > http://sources.redhat.com/cgi-bin/cvsweb.cgi/~checkout~src/gas/NEWS?only_with_tag=binutils-2_15&content-type=text/plain&cvsroot=src
> > 
> > Dunno why the cvsweb download link is broken.
> 
> Actually, I'm not sure that link is right either, although it does
> work.  Perhaps cvsweb.cgi should be removing the '/' in
> "/src/gas/NEWS" before it calls CVS.
> 
> It appears that CVS did change to detect this case:
> 
> 2003-11-18  Derek Price  <derek@ximbiot.com>
> 
> 	* modules.c (do_module): Reject absolute paths.
> 	(Report and suggested fix from Tony Hoyle <tmh@nodomain.org>.)

OK, I did a quick patch to cvsweb.cgi to make this work again.  Your
original URL should also now work, and is probably the right one to
use.

I suppose that we should update cvsweb.cgi at some point.  There are
certainly newer versions out there.

Ian


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]