NEWS link for gas

NEWS link for gas

[Matt Davis]

the NEWS link for gas near the top of the page gave me a CVS error.

NEWS link for gas

[Matt Davis]

the NEWS link for gas near the top of the page gave me a CVS error.

Re: NEWS link for gas

[Ben Elliston]

Matt Davis <deadsmith@mail.utexas.edu> writes:

> the NEWS link for gas near the top of the page gave me a CVS error.

It would best if the CVS revisions were not hardwired in the web page,
but if the revision tag for the current release were used instead.
Something like:

--- index.html~	2004-05-21 10:51:26.000000000 +1000
+++ index.html	2004-05-21 10:55:43.000000000 +1000
@@ -49,9 +49,9 @@
 
 <P>
 The latest release of GNU binutils is 2.15.  The various NEWS files
-(<A href="http://sources.redhat.com/cgi-bin/cvsweb.cgi/~checkout~/src/binutils/NEWS?rev=1.42&content-type=text/plain&cvsroot=src">binutils</A>,
- <A href="http://sources.redhat.com/cgi-bin/cvsweb.cgi/~checkout~/src/gas/NEWS?rev=1.40.6.2&content-type=text/plain&cvsroot=src">gas</A>, and
- <A href="http://sources.redhat.com/cgi-bin/cvsweb.cgi/~checkout~/src/ld/NEWS?rev=1.55.6.1&content-type=text/plain&cvsroot=src">ld</A>)
+(<A href="http://sources.redhat.com/cgi-bin/cvsweb.cgi/~checkout~/src/binutils/NEWS?only_with_tag=binutils-2_15&content-type=text/plain&cvsroot=src">binutils</A>,
+ <A href="http://sources.redhat.com/cgi-bin/cvsweb.cgi/~checkout~/src/gas/NEWS?only_with_tag=binutils-2_15&content-type=text/plain&cvsroot=src">gas</A>, and
+ <A href="http://sources.redhat.com/cgi-bin/cvsweb.cgi/~checkout~/src/ld/NEWS?only_with_tag=binutils-2_15&content-type=text/plain&cvsroot=src">ld</A>)
 have details of what has changed in this release.
 
 <P>

Ben

Re: NEWS link for gas

[Daniel Jacobowitz]

On Fri, May 21, 2004 at 10:57:29AM +1000, Ben Elliston wrote:
> Matt Davis <deadsmith@mail.utexas.edu> writes:
> 
> > the NEWS link for gas near the top of the page gave me a CVS error.
> 
> It would best if the CVS revisions were not hardwired in the web page,
> but if the revision tag for the current release were used instead.
> Something like:

That's fine if you want to make the change, and assuming it works.  But
did you try the existing links?  I got an error that definitely was not
there yesterday.

-- 
Daniel Jacobowitz

Re: NEWS link for gas

[Ben Elliston]

Daniel Jacobowitz <drow@false.org> writes:

> That's fine if you want to make the change, and assuming it works.
> But did you try the existing links?  I got an error that definitely
> was not there yesterday.

I did try the links and they failed.  I didn't bother working out why,
although I suppose that wouldn't hurt ;-)

Ben

Re: NEWS link for gas

[Daniel Jacobowitz]

On Fri, May 21, 2004 at 11:35:16AM +1000, Ben Elliston wrote:
> Daniel Jacobowitz <drow@false.org> writes:
> 
> > That's fine if you want to make the change, and assuming it works.
> > But did you try the existing links?  I got an error that definitely
> > was not there yesterday.
> 
> I did try the links and they failed.  I didn't bother working out why,
> although I suppose that wouldn't hurt ;-)

Easy: I typo'd.  I switched the revision numbers for gas and ld.

Anyway, please do check in your change.  Then I'll remember to
reference the tag next release also.

-- 
Daniel Jacobowitz

Re: NEWS link for gas

[Ben Elliston]

Daniel Jacobowitz <drow@false.org> writes:

> Anyway, please do check in your change.  Then I'll remember to
> reference the tag next release also.

I'm afraid I don't have write access to /cvs/binutils/htdocs.

Ben

Re: NEWS link for gas

[Daniel Jacobowitz]

On Fri, May 21, 2004 at 12:57:18PM +1000, Ben Elliston wrote:
> Daniel Jacobowitz <drow@false.org> writes:
> 
> > Anyway, please do check in your change.  Then I'll remember to
> > reference the tag next release also.
> 
> I'm afraid I don't have write access to /cvs/binutils/htdocs.

Your URLs now don't work for me either.  Something is wrong with
cvsweb.

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?

-- 
Daniel Jacobowitz

Re: NEWS link for gas

[Ian Lance Taylor]

Daniel Jacobowitz <drow@false.org> 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.

Ian

Re: NEWS link for gas

[Ian Lance Taylor]

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

> Daniel Jacobowitz <drow@false.org> 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>.)

Ian

Re: NEWS link for gas

[Ian Lance Taylor]

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

Re: NEWS link for gas

[Daniel Jacobowitz]

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

Re: NEWS link for gas

[Daniel Jacobowitz]

On Fri, May 21, 2004 at 10:57:29AM +1000, Ben Elliston wrote:
> Matt Davis <deadsmith@mail.utexas.edu> writes:
> 
> > the NEWS link for gas near the top of the page gave me a CVS error.
> 
> It would best if the CVS revisions were not hardwired in the web page,
> but if the revision tag for the current release were used instead.
> Something like:
> 
> --- index.html~	2004-05-21 10:51:26.000000000 +1000
> +++ index.html	2004-05-21 10:55:43.000000000 +1000
> @@ -49,9 +49,9 @@
>  
>  <P>
>  The latest release of GNU binutils is 2.15.  The various NEWS files
> -(<A href="http://sources.redhat.com/cgi-bin/cvsweb.cgi/~checkout~/src/binutils/NEWS?rev=1.42&content-type=text/plain&cvsroot=src">binutils</A>,
> - <A href="http://sources.redhat.com/cgi-bin/cvsweb.cgi/~checkout~/src/gas/NEWS?rev=1.40.6.2&content-type=text/plain&cvsroot=src">gas</A>, and
> - <A href="http://sources.redhat.com/cgi-bin/cvsweb.cgi/~checkout~/src/ld/NEWS?rev=1.55.6.1&content-type=text/plain&cvsroot=src">ld</A>)
> +(<A href="http://sources.redhat.com/cgi-bin/cvsweb.cgi/~checkout~/src/binutils/NEWS?only_with_tag=binutils-2_15&content-type=text/plain&cvsroot=src">binutils</A>,
> + <A href="http://sources.redhat.com/cgi-bin/cvsweb.cgi/~checkout~/src/gas/NEWS?only_with_tag=binutils-2_15&content-type=text/plain&cvsroot=src">gas</A>, and
> + <A href="http://sources.redhat.com/cgi-bin/cvsweb.cgi/~checkout~/src/ld/NEWS?only_with_tag=binutils-2_15&content-type=text/plain&cvsroot=src">ld</A>)
>  have details of what has changed in this release.

Unfortunately, if this ever worker before, it doesn't work now.  I had
to revert to using revision numbers for 2.16.  The failure mode may not
have been obvious; it works, but you get HEAD's NEWS instead.

-- 
Daniel Jacobowitz
CodeSourcery, LLC