Re: [libav-devel] [PATCH] build: Add support for travis CI

2015-08-29 Thread Luca Barbato
On 29/08/15 21:37, James Almer wrote: > I've seen people create pull requests with nonsense commits in some > projects when what they intended was to report a bug or request a > feature. I'm not sure if this bridge is a good idea. If they apply and pass make fate they get forwarded otherwise not,

Re: [libav-devel] [PATCH] lavr: Remove unused label

2015-08-29 Thread Luca Barbato
On 29/08/15 21:20, Vittorio Giovara wrote: > --- > Leftover from the removal. > Vittorio > > libavresample/resample.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/libavresample/resample.c b/libavresample/resample.c > index b307487..9c447d3 100644 > --- a/libavresample/resample.c > +++

Re: [libav-devel] [PATCH] des: Add missing include

2015-08-29 Thread Martin Storsjö
On Sat, 29 Aug 2015, Vittorio Giovara wrote: --- Related to In file included from libavutil/des.h.c:1: /Users/GiovaraV/src/libav/libavutil/des.h:33:5: warning: 'FF_API_CRYPTO_CONTEXT' is not defined, evaluates to 0 [-Wundef] #if FF_API_CRYPTO_CONTEXT ^ Vittorio libavutil/des.h |

Re: [libav-devel] [PATCH] libfdk-aacdec: Remove unused variable

2015-08-29 Thread Martin Storsjö
On Sat, 29 Aug 2015, Vittorio Giovara wrote: --- libavcodec/libfdk-aacdec.c | 1 - 1 file changed, 1 deletion(-) diff --git a/libavcodec/libfdk-aacdec.c b/libavcodec/libfdk-aacdec.c index 5ac8fb7..cdf7a05 100644 --- a/libavcodec/libfdk-aacdec.c +++ b/libavcodec/libfdk-aacdec.c @@ -209,7 +209,6 @

Re: [libav-devel] [PATCH] ismindex: Use the correct abs() version

2015-08-29 Thread Martin Storsjö
On Sat, 29 Aug 2015, Vittorio Giovara wrote: --- Related to ./libav/tools/ismindex.c:378:29: warning: absolute value function 'abs' given an argument of type 'long long' but has parameter of type 'int' which may cause truncation of value [-Wabsolute-value] if (duration > 0 && a

[libav-devel] [PATCH] libfdk-aacdec: Remove unused variable

2015-08-29 Thread Vittorio Giovara
--- libavcodec/libfdk-aacdec.c | 1 - 1 file changed, 1 deletion(-) diff --git a/libavcodec/libfdk-aacdec.c b/libavcodec/libfdk-aacdec.c index 5ac8fb7..cdf7a05 100644 --- a/libavcodec/libfdk-aacdec.c +++ b/libavcodec/libfdk-aacdec.c @@ -209,7 +209,6 @@ static av_cold int fdk_aac_decode_init(AVCod

[libav-devel] [PATCH] lavr: Remove unused label

2015-08-29 Thread Vittorio Giovara
--- Leftover from the removal. Vittorio libavresample/resample.c | 1 - 1 file changed, 1 deletion(-) diff --git a/libavresample/resample.c b/libavresample/resample.c index b307487..9c447d3 100644 --- a/libavresample/resample.c +++ b/libavresample/resample.c @@ -256,7 +256,6 @@ int avresample_se

[libav-devel] [PATCH] ismindex: Use the correct abs() version

2015-08-29 Thread Vittorio Giovara
--- Related to ./libav/tools/ismindex.c:378:29: warning: absolute value function 'abs' given an argument of type 'long long' but has parameter of type 'int' which may cause truncation of value [-Wabsolute-value] if (duration > 0 && abs(duration - track->offsets[i].duration) > 3

[libav-devel] [PATCH] des: Add missing include

2015-08-29 Thread Vittorio Giovara
--- Related to In file included from libavutil/des.h.c:1: /Users/GiovaraV/src/libav/libavutil/des.h:33:5: warning: 'FF_API_CRYPTO_CONTEXT' is not defined, evaluates to 0 [-Wundef] #if FF_API_CRYPTO_CONTEXT ^ Vittorio libavutil/des.h | 1 + 1 file changed, 1 insertion(+) diff -

Re: [libav-devel] [PATCH] build: Add support for travis CI

2015-08-29 Thread Martin Storsjö
On Sat, 29 Aug 2015, Luca Barbato wrote: From: Federico Tomassetti It is useful to support an eventual staging branch and to have a first layer fate-check on github pull requests. --- Now updated to be much faster, thanks again to Federico for his effort. .travis.yml | 19 +++

Re: [libav-devel] [PATCH] pixdesc: extend size of the AVPixFmtDescriptor.flags field

2015-08-29 Thread Luca Barbato
On 29/08/15 19:58, wm4 wrote: I'd consider part of the current bump and I do like it. I'd push it tomorrow if somebody won't burn me for forgetting it. Thanks for reminding me. lu ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.

[libav-devel] [PATCH] pixdesc: extend size of the AVPixFmtDescriptor.flags field

2015-08-29 Thread wm4
--- Also adding some doxygen. Not bothering with a bump. --- doc/APIchanges | 3 +++ libavutil/pixdesc.h | 6 +- 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/doc/APIchanges b/doc/APIchanges index e500ac8..6a3d457 100644 --- a/doc/APIchanges +++ b/doc/APIchanges @@ -13,6 +

Re: [libav-devel] [PATCH] build: Add support for travis CI

2015-08-29 Thread Luca Barbato
On 29/08/15 18:28, Timothy Gu wrote: > Also… > Locally updated according to your sprunge, thanks a lot! lu ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH] build: Add support for travis CI

2015-08-29 Thread Luca Barbato
On 29/08/15 17:36, Timothy Gu wrote: > Maybe -j 8 is a little bit too aggressive? We tested it and seemed near optimal, most of the tests are 50%+ setup time (thus why I stopped using perf with them). > Also, how will this be integrated into the existing FATE framework? Or will > it not? Travis

Re: [libav-devel] [PATCH] build: Add support for travis CI

2015-08-29 Thread Timothy Gu
Also… On Sat, Aug 29, 2015 at 8:00 AM Luca Barbato wrote: > diff --git a/.travis.yml b/.travis.yml > new file mode 100644 > index 000..773327e > --- /dev/null > +++ b/.travis.yml > @@ -0,0 +1,19 @@ > +language: c > + sudo: false > This property and all the ones below should be on the same i

Re: [libav-devel] [PATCH] build: Add support for travis CI

2015-08-29 Thread Timothy Gu
On Sat, Aug 29, 2015 at 8:00 AM Luca Barbato wrote: > From: Federico Tomassetti > > It is useful to support an eventual staging branch and to have a first > layer fate-check on github pull requests. > --- > > Now updated to be much faster, thanks again to Federico for his > effort. > I was goin

[libav-devel] [PATCH] build: Add support for travis CI

2015-08-29 Thread Luca Barbato
From: Federico Tomassetti It is useful to support an eventual staging branch and to have a first layer fate-check on github pull requests. --- Now updated to be much faster, thanks again to Federico for his effort. .travis.yml | 19 +++ 1 file changed, 19 insertions(+) create

[libav-devel] [PATCH] build: Add support for travis CI

2015-08-29 Thread Luca Barbato
From: Federico Tomassetti It is useful to support an eventual staging branch and to have a first layer fate-check on github pull requests. --- And would be nice to have the pull-request -> mailing list bridge deployed soon =P .travis.yml | 11 +++ 1 file changed, 11 insertions(+) crea