Re: [FFmpeg-devel] [PATCH] configure: Allow users to disable all hwaccel libraries

2017-12-10 Thread Gyan Doshi
On 12/11/2017 12:06 AM, Mark Thompson wrote: --disable-hwaccels does not disable hardware codecs: it disables hwaccels only, decoders and encoders will still be present. If you want to disable decoder and encoder wrappers as well then you will need to either use --disable-decoders

Re: [FFmpeg-devel] [PATCH] configure: Allow users to disable all hwaccel libraries

2017-12-10 Thread Mark Thompson
On 10/12/17 18:22, Gyan Doshi wrote: > On 12/10/2017 10:44 PM, Mark Thompson wrote: > >> Can you explain what you actually want to achieve here?  It sounds more like >> you want --disable-autodetect (hardware libraries are not the only ones >> which can give surprise dependencies). > > If the

Re: [FFmpeg-devel] [PATCH] configure: Allow users to disable all hwaccel libraries

2017-12-10 Thread Gyan Doshi
On 12/10/2017 10:44 PM, Mark Thompson wrote: Can you explain what you actually want to achieve here? It sounds more like you want --disable-autodetect (hardware libraries are not the only ones which can give surprise dependencies). If the ffmpeg HW decoder/encoder wrappers are disabled,

Re: [FFmpeg-devel] [PATCH] configure: Allow users to disable all hwaccel libraries

2017-12-10 Thread Mark Thompson
On 07/12/17 09:31, Gyan Doshi wrote: > On 12/7/2017 12:59 PM, Tobias Rapp wrote: >> >> Why not add the HWACCEL_AUTODETECT_LIBRARY_LIST to --disable-hwaccels? > > That is a better choice but there wasn't a bespoke case for > '--disable-hwaccels', so originally ignored that possibility. Added it

Re: [FFmpeg-devel] [PATCH] configure: Allow users to disable all hwaccel libraries

2017-12-10 Thread Carl Eugen Hoyos
2017-12-10 13:00 GMT+01:00 Gyan Doshi : > Ping for the patch Threaten to apply and wait a few days. Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] configure: Allow users to disable all hwaccel libraries

2017-12-10 Thread Gyan Doshi
On 12/8/2017 6:18 PM, Gyan Doshi wrote: On 12/8/2017 4:56 PM, Tobias Rapp wrote: includes autodetected hwaccels, but that's just my personal opinion. I leave it to the maintainer of the file to decide what to push. Speaking of which, who is the maintainer of configure? Neither the file

Re: [FFmpeg-devel] [PATCH] configure: Allow users to disable all hwaccel libraries

2017-12-08 Thread Gyan Doshi
On 12/8/2017 4:56 PM, Tobias Rapp wrote: includes autodetected hwaccels, but that's just my personal opinion. I leave it to the maintainer of the file to decide what to push. Speaking of which, who is the maintainer of configure? Neither the file nor MAINTAINERS has a name. Regards, Gyan

Re: [FFmpeg-devel] [PATCH] configure: Allow users to disable all hwaccel libraries

2017-12-08 Thread Tobias Rapp
On 07.12.2017 10:31, Gyan Doshi wrote: On 12/7/2017 12:59 PM, Tobias Rapp wrote: Why not add the HWACCEL_AUTODETECT_LIBRARY_LIST to --disable-hwaccels? That is a better choice but there wasn't a bespoke case for '--disable-hwaccels', so originally ignored that possibility. Added it in

Re: [FFmpeg-devel] [PATCH] configure: Allow users to disable all hwaccel libraries

2017-12-07 Thread Gyan Doshi
On 12/7/2017 12:59 PM, Tobias Rapp wrote: Why not add the HWACCEL_AUTODETECT_LIBRARY_LIST to --disable-hwaccels? That is a better choice but there wasn't a bespoke case for '--disable-hwaccels', so originally ignored that possibility. Added it in attached (new) patch. Either one - this or

Re: [FFmpeg-devel] [PATCH] configure: Allow users to disable all hwaccel libraries

2017-12-06 Thread Tobias Rapp
On 30.11.2017 11:58, Gyan Doshi wrote: On 11/30/2017 4:19 PM, Clément Bœsch wrote: Is --disable-autodetect broken/not complete for you? This disables more than just the hwaccel libs. I've tested my patch locally and configure lists no entries under External Libraries Providing Hardware

Re: [FFmpeg-devel] [PATCH] configure: Allow users to disable all hwaccel libraries

2017-12-06 Thread Gyan Doshi
On 12/4/2017 9:47 AM, Gyan Doshi wrote: On 11/30/2017 3:58 PM, Gyan Doshi wrote: I added --disable-hwaccels during configure and noticed that all autodetected hwaccel libs were still being linked - 6 in my case, which makes for a more bloated binary than warranted. Turns out the option only

Re: [FFmpeg-devel] [PATCH] configure: Allow users to disable all hwaccel libraries

2017-12-03 Thread Gyan Doshi
On 11/30/2017 3:58 PM, Gyan Doshi wrote: I added --disable-hwaccels during configure and noticed that all autodetected hwaccel libs were still being linked - 6 in my case, which makes for a more bloated binary than warranted. Turns out the option only disables native components which make

Re: [FFmpeg-devel] [PATCH] configure: Allow users to disable all hwaccel libraries

2017-11-30 Thread Gyan Doshi
On 11/30/2017 4:19 PM, Clément Bœsch wrote: Is --disable-autodetect broken/not complete for you? This disables more than just the hwaccel libs. I've tested my patch locally and configure lists no entries under External Libraries Providing Hardware Acceleration, unlike with

Re: [FFmpeg-devel] [PATCH] configure: Allow users to disable all hwaccel libraries

2017-11-30 Thread Clément Bœsch
On Thu, Nov 30, 2017 at 03:58:21PM +0530, Gyan Doshi wrote: > I added --disable-hwaccels during configure and noticed that all > autodetected hwaccel libs were still being linked - 6 in my case, which > makes for a more bloated binary than warranted. > > Turns out the option only disables native

[FFmpeg-devel] [PATCH] configure: Allow users to disable all hwaccel libraries

2017-11-30 Thread Gyan Doshi
I added --disable-hwaccels during configure and noticed that all autodetected hwaccel libs were still being linked - 6 in my case, which makes for a more bloated binary than warranted. Turns out the option only disables native components which make use of the hwaccel libs. The reporter of