diff mbox series

[FFmpeg-devel] doc/general_contents: Fix dead links

Message ID d349d43346c90f7211cc2b5a7fefe878@e.email
State New
Headers show
Series [FFmpeg-devel] doc/general_contents: Fix dead links | expand

Checks

Context Check Description
andriy/commit_summary warning Please wrap lines in the body of the commit message between 60 and 72 characters.
andriy/x86_make success Make finished
andriy/x86_make_fate success Make fate finished
andriy/PPC64_make success Make finished
andriy/PPC64_make_fate success Make fate finished

Commit Message

Mapul Bhola Aug. 16, 2021, 4:22 p.m. UTC
The x265 configuration and installation guide has moved to Bitbucket so the updated link reflects that one.

As the openCV site currently only has docs for opencv3 and the filter in libavfilter is an opencv2 one an archived link to the old cv2 documentation is included.
---
 doc/filters.texi          | 2 +-
 doc/general_contents.texi | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

Comments

Gyan Doshi Aug. 20, 2021, 12:05 p.m. UTC | #1
On 2021-08-16 09:52 pm, Mapul Bhola wrote:
> The x265 configuration and installation guide has moved to Bitbucket so the updated link reflects that one.
>
> As the openCV site currently only has docs for opencv3 and the filter in libavfilter is an opencv2 one an archived link to the old cv2 documentation is included.
> ---
>   doc/filters.texi          | 2 +-
>   doc/general_contents.texi | 2 +-
>   2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/doc/filters.texi b/doc/filters.texi
> index eaf23e3736..2f2722383e 100644
> --- a/doc/filters.texi
> +++ b/doc/filters.texi
> @@ -15484,7 +15484,7 @@ values are assumed.
>   
>   Refer to the official libopencv documentation for more precise
>   information:
> -@url{http://docs.opencv.org/master/modules/imgproc/doc/filtering.html}
> +@url{http://ghostarchive.org/archive/Wssn7}

Isn't this one better:

http://www.opencv.org.cn/opencvdoc/2.3.2/html/modules/imgproc/doc/filtering.html

>   
>   Several libopencv filters are supported; see the following subsections.
>   
> diff --git a/doc/general_contents.texi b/doc/general_contents.texi
> index 354899ad17..8664639f6b 100644
> --- a/doc/general_contents.texi
> +++ b/doc/general_contents.texi
> @@ -304,7 +304,7 @@ details), you must upgrade FFmpeg's license to GPL in order to use it.
>   
>   FFmpeg can make use of the x265 library for HEVC encoding.
>   
> -Go to @url{http://x265.org/developers.html} and follow the instructions
> +Go to @url{https://bitbucket.org/multicoreware/x265_git/wiki/Home} and follow the instructions
>   for installing the library. Then pass @code{--enable-libx265} to configure
>   to enable it.

I think this one is better:

https://bitbucket.org/multicoreware/x265_git/src/master/build/README.txt

Regards,
Gyan
Mapul Bhola Aug. 20, 2021, 12:11 p.m. UTC | #2
August 20, 2021 8:05 AM, "Gyan Doshi" <ffmpeg@gyani.pro> wrote:

> On 2021-08-16 09:52 pm, Mapul Bhola wrote:
> 
>> The x265 configuration and installation guide has moved to Bitbucket so the updated link reflects
>> that one.
>> 
>> As the openCV site currently only has docs for opencv3 and the filter in libavfilter is an opencv2
>> one an archived link to the old cv2 documentation is included.
>> ---
>> doc/filters.texi | 2 +-
>> doc/general_contents.texi | 2 +-
>> 2 files changed, 2 insertions(+), 2 deletions(-)
>> 
>> diff --git a/doc/filters.texi b/doc/filters.texi
>> index eaf23e3736..2f2722383e 100644
>> --- a/doc/filters.texi
>> +++ b/doc/filters.texi
>> @@ -15484,7 +15484,7 @@ values are assumed.
>>> Refer to the official libopencv documentation for more precise
>> information:
>> -@url{http://docs.opencv.org/master/modules/imgproc/doc/filtering.html}
>> +@url{http://ghostarchive.org/archive/Wssn7}
> 
> Isn't this one better:
> 
> http://www.opencv.org.cn/opencvdoc/2.3.2/html/modules/imgproc/doc/filtering.html
Yes, I didn't see that for some reason on the original site. Will resubmit patch with the link you had.
> 
>>> Several libopencv filters are supported; see the following subsections.
>>> diff --git a/doc/general_contents.texi b/doc/general_contents.texi
>> index 354899ad17..8664639f6b 100644
>> --- a/doc/general_contents.texi
>> +++ b/doc/general_contents.texi
>> @@ -304,7 +304,7 @@ details), you must upgrade FFmpeg's license to GPL in order to use it.
>>> FFmpeg can make use of the x265 library for HEVC encoding.
>>> -Go to @url{http://x265.org/developers.html} and follow the instructions
>> +Go to @url{https://bitbucket.org/multicoreware/x265_git/wiki/Home} and follow the instructions
>> for installing the library. Then pass @code{--enable-libx265} to configure
>> to enable it.
> 
> I think this one is better:
> 
> https://bitbucket.org/multicoreware/x265_git/src/master/build/README.txt
Agreed, will use this as well.
> 
> Regards,
> Gyan
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
diff mbox series

Patch

diff --git a/doc/filters.texi b/doc/filters.texi
index eaf23e3736..2f2722383e 100644
--- a/doc/filters.texi
+++ b/doc/filters.texi
@@ -15484,7 +15484,7 @@  values are assumed.
 
 Refer to the official libopencv documentation for more precise
 information:
-@url{http://docs.opencv.org/master/modules/imgproc/doc/filtering.html}
+@url{http://ghostarchive.org/archive/Wssn7}
 
 Several libopencv filters are supported; see the following subsections.
 
diff --git a/doc/general_contents.texi b/doc/general_contents.texi
index 354899ad17..8664639f6b 100644
--- a/doc/general_contents.texi
+++ b/doc/general_contents.texi
@@ -304,7 +304,7 @@  details), you must upgrade FFmpeg's license to GPL in order to use it.
 
 FFmpeg can make use of the x265 library for HEVC encoding.
 
-Go to @url{http://x265.org/developers.html} and follow the instructions
+Go to @url{https://bitbucket.org/multicoreware/x265_git/wiki/Home} and follow the instructions
 for installing the library. Then pass @code{--enable-libx265} to configure
 to enable it.