Dear Ali,
> Is the Turing Codec interface patch going to be pushed after the fix of flag
> problem ?
The problem with the static flag is already fixed in the current version of the
patch (v12). If you want to test it, you can manually copy the patch and apply
it on top of the latest ffmpeg HEA
think this was the only blocker with this latest version of the patch?
Thanks,
----
Saverio Blasi
Senior Research Engineer
BBC Research & Development
Centre House|56 Wood Lane|London|W12 7SB
___
>>In other words, it's just an example in the wiki. It's one way to do it, but
>>that doesn't mean it's *the* way to do it. Building against dynamic libs is
>>certainly much more common if those are (at least for the most part) provided
>>by the "system" (distribution or what have you).
>>
>>Bot
>>2. Not used --pkg-config-flags--static and thus missed including the
>>(unfortunately non-namespaced) embedded boost libraries.
>>
>>I followed the example instructions in
>>https://trac.ffmpeg.org/wiki/CompilationGuide/Ubuntu#ffmpeg (at the top of
>>the page it says "Compile FFmpeg on Ubuntu,
2. Not used --pkg-config-flags--static and thus missed including the
(unfortunately non-namespaced) embedded boost libraries.
I followed the example instructions in
https://trac.ffmpeg.org/wiki/CompilationGuide/Ubuntu#ffmpeg (at the top of the
page it says "Compile FFmpeg on Ubuntu, Debian, or
>>So I did try to build this, but the ffmpeg configure check apparently fails
>>at picking it up and dumps 3.5MB of C++ linker errors into the config.log.
>>And I have no idea why it does that. configure actually prints "ERROR:
>>libturing not found using pkg-config", but the .pc part seems to b
- This patch contains the changes to interface the Turing codec
(http://turingcodec.org/) with ffmpeg. The patch was modified to address the
comments in the review as follows:
- Added a pkg-config file to list all dependencies required by libturing.
This should address the issue pointed out by
Dear all,
>> A build failure seems a bit like a deal breaker
Sorry for this. I have just pushed a fix to the Turing repository to remove
bool type from the libturing APIs. The patch builds and runs fine now in our
tests.
Thanks,
Saverio
___
ffmpeg-de
- This patch contains the changes to interface the Turing codec
(http://turingcodec.org/) with ffmpeg. The patch was modified to address the
comments in the review as follows:
- Added a pkg-config file to list all dependencies required by libturing.
This should address the issue pointed out by
- This patch contains the changes to interface the Turing codec
(http://turingcodec.org/) with ffmpeg. The patch was modified to address the
comments in the review as follows:
- Added a pkg-config file to list all dependencies required by libturing.
This should address the issue pointed out by
Hello,
Could you please provide us with some feedback on this patch?
Thanks,
Saverio
-Original Message-
From: Saverio Blasi [mailto:saverio.bl...@bbc.co.uk]
Sent: 09 May 2017 11:08
To: ffmpeg-devel@ffmpeg.org
Cc: Saverio Blasi
Subject: [PATCH v10] - Added Turing codec interface for
...@ffmpeg.org] On Behalf Of
Saverio Blasi
Sent: 11 May 2017 08:21
To: FFmpeg development discussions and patches
Subject: Re: [FFmpeg-devel] [PATCH v10] - Added Turing codec interface for
ffmpeg
Dear all,
First of all thank you very much for all the help you provided with this patch.
We are now at the
Dear all,
First of all thank you very much for all the help you provided with this patch.
We are now at the tenth iteration, and the patch greatly improved since the
first time we submitted it. In fact most of the lines of code in the current
version of the patch were suggested by you or revise
Dear Michael,
>> Why do you pass a single string of all options and then manually parse all
>> options out of it instead of specifying the options each in teh AVOption
>> array ?
We prefer to pass a single string so that we can be more flexible how we then
parse it and what to do with it. T
- This patch contains the changes to interface the Turing codec
(http://turingcodec.org/) with ffmpeg. The patch was modified to address the
comments in the review as follows:
- Added a pkg-config file to list all dependencies required by libturing.
This should address the issue pointed out by
[PATCH v9] - Added Turing codec interface for
ffmpeg
2017-04-28 17:18 GMT+02:00 Saverio Blasi :
> The patch applies nicely to my local ffmpeg repository.
Please confirm that tools/patcheck does not report any issues
(trailing whitespace cannot be committed to our repository).
> This e-
[mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of
Michael Niedermayer
Sent: 26 April 2017 14:02
To: FFmpeg development discussions and patches
Subject: Re: [FFmpeg-devel] [PATCH v9] - Added Turing codec interface for ffmpeg
On Wed, Apr 26, 2017 at 07:55:59AM +, Saverio Blasi wrote
best,
Saverio Blasi, PhD
Senior Research Engineer
BBC Research & Development
Centre House|56 Wood Lane|London|W12 7SB
-Original Message-
From: Saverio Blasi [mailto:saverio.bl...@bbc.co.uk]
Sent: 24 April 2017 14:29
To: ffmpeg-d
- This patch contains the changes to interface the Turing codec
(http://turingcodec.org/) with ffmpeg. The patch was modified to address the
comments in the review as follows:
- Added a pkg-config file to list all dependencies required by libturing.
This should address the issue pointed out by
- This patch contains the changes to interface the Turing codec
(http://turingcodec.org/) with ffmpeg. The patch was modified to address the
comments in the review as follows:
- Added a pkg-config file to list all dependencies required by libturing.
This should address the issue pointed out by
are completely transparent to our ffmpeg patch, which
remains valid at v7.
We would appreciate comments on this.
Thanks,
All the best,
Saverio Blasi, PhD
Senior Research Engineer
BBC Research & Development
Centre House|56 Wood Lane|Lo
interface for ffmpeg
On 27/02/17 08:29, Saverio Blasi wrote:
>> Right, I had a bit more of a look at it and was able to "fix" it by adding
>> the boost libraries to the ffmpeg configure line as well.
>>
>> Actually running it it all looks good to me, if as fast
y to know the patch will be integrated.
Regarding the problems you mention, we will have a look at the output of
valgrind, and are also working on speeding up the encoder.
All the best,
----
Saverio Blasi, PhD
Senior Research Engineer
BBC Research
the best,
Saverio Blasi, PhD
Senior Research Engineer
BBC Research & Development
Centre House|56 Wood Lane|London|W12 7SB
-Original Message-
From: Saverio Blasi [mailto:saverio.bl...@bbc.co.uk]
Sent: 13 February 2017 13:3
> Vertical indentation please.
Thanks for this feedback. We have recently circulated a new version of the
patch (v7) which fixes these indentation issues.
All the best,
----
Saverio Blasi, PhD
Senior Research Engineer
BBC Research & Deve
the best,
----
Saverio Blasi, PhD
Senior Research Engineer
BBC Research & Development
Centre House|56 Wood Lane|London|W12 7SB
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/
- This patch contains the changes to interface the Turing codec
(http://turingcodec.org/) with ffmpeg. The patch was modified to address the
comments in the review as follows:
- Added a pkg-config file to list all dependencies required by libturing.
This should address the issue pointed out by
eg-devel-boun...@ffmpeg.org] On Behalf Of wm4
Sent: 08 February 2017 09:28
To: ffmpeg-devel@ffmpeg.org
Subject: Re: [FFmpeg-devel] [PATCH v5] Added Turing codec interface for ffmpeg
On Wed, 8 Feb 2017 08:41:56 +
Saverio Blasi wrote:
> - This patch contains the changes to interface the
- This patch contains the changes to interface the Turing codec
(http://turingcodec.org/) with ffmpeg. The patch was modified to address the
comments in the review as follows:
- Added a pkg-config file to list all dependencies required by libturing.
This should address the issue pointed out by
fmpeg-devel-boun...@ffmpeg.org] On Behalf Of Mark
Thompson
Sent: 08 February 2017 13:06
To: FFmpeg development discussions and patches
Subject: Re: [FFmpeg-devel] [PATCH v5] Added Turing codec interface for ffmpeg
On 08/02/17 08:41, Saverio Blasi wrote:
> - This patch contains the changes to int
- This patch contains the changes to interface the Turing codec
(http://turingcodec.org/) with ffmpeg. The patch was modified to address the
comments in the review as follows:
- Added a pkg-config file to list all dependencies required by libturing.
This should address the issue pointed out by
31 matches
Mail list logo