fix dangling link

Message ID 1300143602-7569-1-git-send-email-janne-libav@jannau.net
State Committed
Commit e1c8ba61def0e7ad4e9420183e1111e43200c6e6
Headers show

Commit Message

Janne Grunau March 14, 2011, 11 p.m.
---
 src/index |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

Comments

Mans Rullgard March 14, 2011, 11:23 p.m. | #1
Janne Grunau <janne-libav@jannau.net> writes:

> ---
>  src/index |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/src/index b/src/index
> index f50eabd..7e3ba7d 100644
> --- a/src/index
> +++ b/src/index
> @@ -4,7 +4,7 @@
>  <p>
>  <b>Libav</b> is a complete, cross-platform solution to record, convert and
>  stream audio and video. It includes <b>libavcodec</b> - the leading audio/video
> -codec library. See the <a href="libav-doc.html">documentation</a> for a
> +codec library. See the <a href="ffmpeg.html">documentation</a> for a
>  complete feature list and the <a href="changelog.html">Changelog</a> for recent
>  changes.
>  </p>

Shouldn't we rather fix the filename?
Janne Grunau March 15, 2011, 11:06 a.m. | #2
On Mon, Mar 14, 2011 at 11:23:39PM +0000, Måns Rullgård wrote:
> Janne Grunau <janne-libav@jannau.net> writes:
> 
> > ---
> >  src/index |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> >
> > diff --git a/src/index b/src/index
> > index f50eabd..7e3ba7d 100644
> > --- a/src/index
> > +++ b/src/index
> > @@ -4,7 +4,7 @@
> >  <p>
> >  <b>Libav</b> is a complete, cross-platform solution to record, convert and
> >  stream audio and video. It includes <b>libavcodec</b> - the leading audio/video
> > -codec library. See the <a href="libav-doc.html">documentation</a> for a
> > +codec library. See the <a href="ffmpeg.html">documentation</a> for a
> >  complete feature list and the <a href="changelog.html">Changelog</a> for recent
> >  changes.
> >  </p>
> 
> Shouldn't we rather fix the filename?

I think it is a dangling link to an old filename of the ffmpeg app
documentation. Linking documentation.html migh be an alternative but it
doesn't match the description of the link.

Janne
Diego Elio Pettenò March 15, 2011, 7:22 p.m. | #3
Il giorno mar, 15/03/2011 alle 12.06 +0100, Janne Grunau ha scritto:
> 
> I think it is a dangling link to an old filename of the ffmpeg app
> documentation. Linking documentation.html migh be an alternative but
> it
> doesn't match the description of the link. 

Do we have generated Doxygen documentation already?
Diego Biurrun March 15, 2011, 11:23 p.m. | #4
On Tue, Mar 15, 2011 at 12:00:02AM +0100, Janne Grunau wrote:
> 
> --- a/src/index
> +++ b/src/index
> @@ -4,7 +4,7 @@
>  <b>Libav</b> is a complete, cross-platform solution to record, convert and
>  stream audio and video. It includes <b>libavcodec</b> - the leading audio/video
> -codec library. See the <a href="libav-doc.html">documentation</a> for a
> +codec library. See the <a href="ffmpeg.html">documentation</a> for a
>  complete feature list and the <a href="changelog.html">Changelog</a> for recent
>  changes.

This is wrong, it should link to the generated-from-doxy ffmpeg-doc.html.

Diego
Diego Biurrun March 15, 2011, 11:27 p.m. | #5
On Wed, Mar 16, 2011 at 12:23:38AM +0100, Diego Biurrun wrote:
> On Tue, Mar 15, 2011 at 12:00:02AM +0100, Janne Grunau wrote:
> > 
> > --- a/src/index
> > +++ b/src/index
> > @@ -4,7 +4,7 @@
> >  <b>Libav</b> is a complete, cross-platform solution to record, convert and
> >  stream audio and video. It includes <b>libavcodec</b> - the leading audio/video
> > -codec library. See the <a href="libav-doc.html">documentation</a> for a
> > +codec library. See the <a href="ffmpeg.html">documentation</a> for a
> >  complete feature list and the <a href="changelog.html">Changelog</a> for recent
> >  changes.
> 
> This is wrong, it should link to the generated-from-doxy ffmpeg-doc.html.

Sorry, I got confused, patch is OK.

Diego

Patch

diff --git a/src/index b/src/index
index f50eabd..7e3ba7d 100644
--- a/src/index
+++ b/src/index
@@ -4,7 +4,7 @@ 
 <p>
 <b>Libav</b> is a complete, cross-platform solution to record, convert and
 stream audio and video. It includes <b>libavcodec</b> - the leading audio/video
-codec library. See the <a href="libav-doc.html">documentation</a> for a
+codec library. See the <a href="ffmpeg.html">documentation</a> for a
 complete feature list and the <a href="changelog.html">Changelog</a> for recent
 changes.
 </p>