fix dangling link

diff mbox

---
 src/index |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

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?

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

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?

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

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