[Ffmpeg-devel-irc] ffmpeg-devel.log.20160522

[burek]

[00:17:45 CEST] <Illya> michaelni: does it break the build with patch #1 as well?
[00:18:19 CEST] <Illya> SDL patch, I mean.
[00:29:47 CEST] <michaelni> Illya, just tried, fails too (see ML)
[00:29:59 CEST] <michaelni> i mean patch 1+2
[00:30:14 CEST] <michaelni> ill try the older patch in a moment
[00:32:31 CEST] <Illya> if they still dont work: https://gist.github.com/c1b5dbd2882f5580e17dae6f9269612f this should, and if this doesnt, then there's something very wrong (probably on my end, though)
[00:37:31 CEST] <michaelni> Illya, just to be sure this isnt missed, this is a ubuntu 12.04 LTS i think  (aka a somewhat older OS)
[00:38:57 CEST] <Illya> also make sure that you have sdl2 installed (you probably do, just checking). I mean.... it should work, SDL is pretty cross platform
[00:42:02 CEST] <michaelni> Illya, i doubt the box has sdl2 but build of plain ./configure broke but then maybe i installed sdl2 at some point and forgot
[00:44:00 CEST] <Illya> I didn't see a configure error in your log? Was this afterwards?
[00:46:11 CEST] <michaelni> theres no error from configure
[00:47:33 CEST] <michaelni> theres also no sdl2-config
[00:48:17 CEST] <michaelni> !HAVE_SDL2=yes
[00:48:24 CEST] <michaelni> CONFIG_SDL2_OUTDEV=yes
[00:48:37 CEST] <michaelni> thats a odd combination
[00:49:00 CEST] <Illya> err. oh, yes i see the issue now
[00:49:17 CEST] <Illya> I uninstalled sdl2 and sdl2 outdev is still enabled
[00:52:33 CEST] <Illya> michaelni: ok, fixing that now. And, with sdl2 installed, how does it do?
[00:53:02 CEST] <michaelni> i dont think 12.04 LTS has sdl2
[00:53:17 CEST] <JEEB> it's old enough to barely be in support now
[00:53:25 CEST] <michaelni> i could test on another box of course
[00:53:26 CEST] <JEEB> will be out next year
[00:53:41 CEST] <JEEB> so yeah, you'd have to build SDL2 into another prefix
[00:53:43 CEST] <JEEB> to test it
[00:59:17 CEST] <Illya> michaelni: this is the updated patch, which doesnt break configure (hopefully) anymore. https://gist.github.com/adc405000493a9f26e568ea3ae0468ef I wasn't checking for the sdl2 outdev's dependencies, must have missed that earlier
[01:06:39 CEST] <michaelni> Illya, on old ubuntu ffplay isnt build anymore build seems working thogh for the rest, on new ubuntu ffplay doesnt work, doesnt recognize "q" for example
[01:06:42 CEST] <michaelni> "The key you just pressed is not recognized by SDL. To help get this fixed, please report this to the SDL mailing list <sdl at libsdl.org> X11 KeyCode 24 (16), X11 KeySym 0x71 (q)."
[01:09:53 CEST] <Illya> FFplay won't build without sdl2 since it's been converted from sdl to sdl2, and since you dont have sdl2 on the old ubuntu it cant build, and the first patch is actually Marton's, but mine just requires it (hence me bundling it with mine all the time) so I haven't really delved into ffplay much
[01:47:29 CEST] <michaelni> would it be difficult/messy to keep SDL1 support until distros with only SDL1 are dead?
[02:18:23 CEST] <Illya> michaelni: probably
[02:19:44 CEST] <Illya> For ffplay, most definitely, the patch looks like it changes quite a lot. For the outdevs, not at all.
[02:32:43 CEST] <llogan> andrey_utkin: approved. you can ping me next time, but it may take me a while as I was afk for 7 hours.
[02:46:15 CEST] <cone-279> ffmpeg 03Michael Niedermayer 07master:ea88dc3e8e2c: avformat/avienc: Rename reshuffles return variable to ensure it is not mixed up
[02:46:15 CEST] <cone-279> ffmpeg 03Michael Niedermayer 07master:9d5894b2399e: avformat/avienc: Fix memleaks on errors
[02:53:19 CEST] <Illya> michaelni: not sure how to proceed with this. I guess I could still make a patch to add the sdl2 outdev independently to ffplay, idk. Although, it's not like it 100% works either, there is still one major bug which I'm banging my head against the wall about.
[03:02:09 CEST] <michaelni> Illya, iam just speaking for myself but wouldnt it be trivial to keep ffplay_sdl1.c ? (though thats something marton would need to decide if he wants that)
[03:02:46 CEST] <Illya> oh. well... you could do that, I thought you mean keeping both in the same file, using macros to compile each one
[03:03:39 CEST] <michaelni> also either way if its easy to make dev and ffplay more independant that probably doest hurt
[03:04:57 CEST] <michaelni> why are they dependant ?
[03:05:10 CEST] <Illya> I can submit the dev as a separate thing, but it would require a patch for configure which would suit ffplay as well, although I think I could just squash the configure commits and then remove the ffplay parts
[03:05:38 CEST] <michaelni> so its just reordering a few hunks ? if so that seems like it should be done anyway
[03:05:50 CEST] <michaelni> reordring between commits that is
[03:06:15 CEST] <Illya> michaelni: it was mostly just for keeping things together, I think that Marton didn't want to submit the ffplay patch until there was an sdl2 outdev.
[03:06:47 CEST] <Illya> I mean, his sdl2 ffplay patch has been around for a while (2014 iirc)
[03:07:42 CEST] <michaelni> i was just thinking why its not 1 patch for build sys and 1 for outdev and one for ffplay 
[03:08:29 CEST] <Illya> not sure, but it would make sense to do it like that
[03:52:27 CEST] <cone-279> ffmpeg 03Michael Niedermayer 07master:2062d51275bd: avformat/matroskaenc: wrap par->format use as sample format under codec_type == AVMEDIA_TYPE_AUDI
[03:52:28 CEST] <cone-279> ffmpeg 03Michael Niedermayer 07master:566be4f9e886: avformat/matroskaenc: reindent after last commit
[04:32:31 CEST] <rcombs> AVMEDIA_TYPE_AUDI, not to be confused with AVMEDIA_TYPE_CHVY or AVMEDIA_TYPE_TYTA
[10:32:20 CEST] <andrey_utkin> thanks for approving email, llogan.
[11:58:40 CEST] <j-b> 'morning
[18:13:30 CEST] <lilibox> hi
[18:15:04 CEST] <Compn> helllloooo
[18:15:23 CEST] <Compn> lilibox : we may not be here immediately to take your call, but we will see it later
[18:15:33 CEST] <Compn> so please ask your question and stick around for answer
[20:33:49 CEST] <Shiz> lol
[20:34:39 CEST] <Shiz> why did cehoyos edit the link out
[20:50:49 CEST] <nevcairiel> you mean the automatic wiki link? why should that be there?
[20:51:05 CEST] <nevcairiel> it links any camelcase word, which is kinda stupid
[20:51:14 CEST] <Shiz> no
[20:51:17 CEST] <Shiz> he edited it twice
[20:51:28 CEST] <Shiz> there was a link to the MPC-BE source on sourceforge first
[20:55:26 CEST] <durandal_1707> Shiz: mpc-be supports that files?
[20:57:12 CEST] <JEEB> it has a hacked gabest's mpeg-ps demuxer so it might support it
[20:57:21 CEST] <JEEB> GPL code of course, so lol
[21:02:01 CEST] <kierank> gabest
[21:02:05 CEST] <kierank> haven't heard that name for a while
[21:02:12 CEST] <JEEB> he pops up in random places
[21:02:21 CEST] <nevcairiel> its been like 10 years since he was active in multimedia
[21:02:24 CEST] <JEEB> I think a few years back he posted on Avery Lee's blog
[21:02:48 CEST] <JEEB> he went to opengl programming with emulators after leaving guliverkli
[21:03:03 CEST] <JEEB> and then... he just floats on the internets
[21:06:47 CEST] <Shiz> durandal_1707: yes
[22:59:29 CEST] <cone-354> ffmpeg 03Paul B Mahol 07master:5ce1d089f01a: avcodec/mvcdec: simplify, no need to use reget buffer
[23:06:35 CEST] <cone-354> ffmpeg 03Piotr Bandurski 07master:854c405dab23: avcodec/iff: mention RGB8/RGBN decoder
[23:34:49 CEST] <cone-354> ffmpeg 03Paul B Mahol 07master:cd506c279229: avcodec/truemotion2rt: also export key frame info
[00:00:00 CEST] --- Mon May 23 2016