This is the mail archive of the
binutils@sources.redhat.com
mailing list for the binutils project.
Re: NEWS link for gas
On Tue, May 25, 2004 at 11:53:29PM -0400, Ian Lance Taylor wrote:
> 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.
Thanks! This used to work; I'm guessing the recent CVS changes on
sourceware broke it (but it broke at least a week or so ago). Anyway,
it works now.
--
Daniel Jacobowitz