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 --disable-

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 f

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, the

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 in

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 nor

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 attac

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 Acce

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 use

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 --disable-hwacce

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 c