This would make ffmpeg fail on VS versions prior to VS 2013, for those using the BUILD.gn file, which I assume is not many. It would be easy to add additional conditionals for earlier versions, but if it's just for Chromium then 2013/2015 is sufficient.
VS versions after 2015 will work automatically, which is why I did the conditional as shown. On Tue, Nov 24, 2015 at 4:01 PM, compn <te...@mi.rr.com> wrote: > On Tue, 24 Nov 2015 14:18:47 -0800 > Bruce Dawson <brucedaw...@google.com> wrote: > > > The attached patch lets ffmpeg build with both VS 2013 and VS 2015. > > > > Comments? > > > > not sure if ffmpeg is supporting VS 2010, but does this work on VS2010 > (or any other VS besides 2013/2015) after your changes? > > thanks > -compn > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > -- Bruce Dawson _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
