Re: [FFmpeg-devel] [PATCH] avutil/opencl: don't include config.h

2015-01-12 Thread James Almer
On 12/01/15 6:18 AM, Wei Gao wrote: > 2015-01-09 5:54 GMT+08:00 Thilo Borgmann : > >> Am 08.01.15 17:43, schrieb James Almer: >>> On 08/01/15 4:52 AM, Wei Gao wrote: 2015-01-08 10:10 GMT+08:00 James Almer : > On 07/01/15 10:55 PM, Wei Gao wrote: >> 2015-01-08 8:42 GMT+08:00 Micha

Re: [FFmpeg-devel] [PATCH] avutil/opencl: don't include config.h

2015-01-12 Thread Wei Gao
2015-01-09 5:54 GMT+08:00 Thilo Borgmann : > Am 08.01.15 17:43, schrieb James Almer: > > On 08/01/15 4:52 AM, Wei Gao wrote: > >> 2015-01-08 10:10 GMT+08:00 James Almer : > >> > >>> On 07/01/15 10:55 PM, Wei Gao wrote: > 2015-01-08 8:42 GMT+08:00 Michael Niedermayer : > > > On Wed, J

Re: [FFmpeg-devel] [PATCH] avutil/opencl: don't include config.h

2015-01-08 Thread Thilo Borgmann
Am 08.01.15 17:43, schrieb James Almer: > On 08/01/15 4:52 AM, Wei Gao wrote: >> 2015-01-08 10:10 GMT+08:00 James Almer : >> >>> On 07/01/15 10:55 PM, Wei Gao wrote: 2015-01-08 8:42 GMT+08:00 Michael Niedermayer : > On Wed, Jan 07, 2015 at 05:44:41PM -0300, James Almer wrote: >> O

Re: [FFmpeg-devel] [PATCH] avutil/opencl: don't include config.h

2015-01-08 Thread James Almer
On 08/01/15 4:52 AM, Wei Gao wrote: > 2015-01-08 10:10 GMT+08:00 James Almer : > >> On 07/01/15 10:55 PM, Wei Gao wrote: >>> 2015-01-08 8:42 GMT+08:00 Michael Niedermayer : >>> On Wed, Jan 07, 2015 at 05:44:41PM -0300, James Almer wrote: > On 06/01/15 11:27 PM, Michael Niedermayer wrote:

Re: [FFmpeg-devel] [PATCH] avutil/opencl: don't include config.h

2015-01-07 Thread Wei Gao
2015-01-08 10:10 GMT+08:00 James Almer : > On 07/01/15 10:55 PM, Wei Gao wrote: > > 2015-01-08 8:42 GMT+08:00 Michael Niedermayer : > > > >> On Wed, Jan 07, 2015 at 05:44:41PM -0300, James Almer wrote: > >>> On 06/01/15 11:27 PM, Michael Niedermayer wrote: > On Tue, Jan 06, 2015 at 01:56:01AM

Re: [FFmpeg-devel] [PATCH] avutil/opencl: don't include config.h

2015-01-07 Thread James Almer
On 07/01/15 10:55 PM, Wei Gao wrote: > 2015-01-08 8:42 GMT+08:00 Michael Niedermayer : > >> On Wed, Jan 07, 2015 at 05:44:41PM -0300, James Almer wrote: >>> On 06/01/15 11:27 PM, Michael Niedermayer wrote: On Tue, Jan 06, 2015 at 01:56:01AM -0300, James Almer wrote: > It's not an installe

Re: [FFmpeg-devel] [PATCH] avutil/opencl: don't include config.h

2015-01-07 Thread Wei Gao
2015-01-08 8:42 GMT+08:00 Michael Niedermayer : > On Wed, Jan 07, 2015 at 05:44:41PM -0300, James Almer wrote: > > On 06/01/15 11:27 PM, Michael Niedermayer wrote: > > > On Tue, Jan 06, 2015 at 01:56:01AM -0300, James Almer wrote: > > >> It's not an installed header. Use libavutil/avconfig.h inste

Re: [FFmpeg-devel] [PATCH] avutil/opencl: don't include config.h

2015-01-07 Thread Michael Niedermayer
On Wed, Jan 07, 2015 at 05:44:41PM -0300, James Almer wrote: > On 06/01/15 11:27 PM, Michael Niedermayer wrote: > > On Tue, Jan 06, 2015 at 01:56:01AM -0300, James Almer wrote: > >> It's not an installed header. Use libavutil/avconfig.h instead. > > > > assuming noone objects to this "hack" ... ug

Re: [FFmpeg-devel] [PATCH] avutil/opencl: don't include config.h

2015-01-07 Thread James Almer
On 06/01/15 11:27 PM, Michael Niedermayer wrote: > On Tue, Jan 06, 2015 at 01:56:01AM -0300, James Almer wrote: >> It's not an installed header. Use libavutil/avconfig.h instead. > > assuming noone objects to this "hack" ... ugly but it should fix the > build so ok Yes, i agree it's kinda ugly. W

Re: [FFmpeg-devel] [PATCH] avutil/opencl: don't include config.h

2015-01-06 Thread Wei Gao
2015-01-06 12:56 GMT+08:00 James Almer : > It's not an installed header. Use libavutil/avconfig.h instead. > > Signed-off-by: James Almer > --- > Untested as i don't have an OpenCL SDK installed. > > configure | 2 +- > libavutil/opencl.h | 2 +- > 2 files changed, 2 insertions(+), 2 de

Re: [FFmpeg-devel] [PATCH] avutil/opencl: don't include config.h

2015-01-06 Thread Michael Niedermayer
On Tue, Jan 06, 2015 at 01:56:01AM -0300, James Almer wrote: > It's not an installed header. Use libavutil/avconfig.h instead. assuming noone objects to this "hack" ... ugly but it should fix the build so ok [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Those

[FFmpeg-devel] [PATCH] avutil/opencl: don't include config.h

2015-01-05 Thread James Almer
It's not an installed header. Use libavutil/avconfig.h instead. Signed-off-by: James Almer --- Untested as i don't have an OpenCL SDK installed. configure | 2 +- libavutil/opencl.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/configure b/configure index 50458