I'm going to go ahead and just apologize in advance for not splitting these off into several separate issues - I'll try to do so when I get a chance later...
But there's not been much more since the beginning of the year about some testing of mp4 encoding support, and I was particularly interested in this feature, so I figured I'd give it a shot.

Anyhow, I've got a patch that I'm toying with at the moment that adds reasonably complete support for selecting between FLV and MP4 encoding...
It could definitely use some polish (probably a great deal of polish) but I figured people might be interested.

The obnoxious part is that it also contains several bug fixes and feature requests that should probably be separate issues:
(off the top of my head at the moment)
- Correct '@thumbnail' parsing in the get_flashvars with FlashVideo CCK enabled and a path for the thumbnail other than matching the video files
- Updated regex value in _flashvideo_video_resize to work with current releases of ffmpeg, should fall back gracefully
- Added support for qt-faststart when using mp4 extension (should probably be reworked into a formatter string to a metadata handler?)
- Added an option to force execution of the conversion process on files with flash capable extensions (avoiding issues with ffmpeg's qt-faststart)
- Contains the fix posted by ore for flashvideo_get_size in #622386: Support for high and low quality encoded files

This may largely end up being deprecated soon enough, but I figured I'd toss it out there anyhow.

Comments

Vector-’s picture

Vector-’s picture

StatusFileSize
new10.37 KB

oops, _flashvideo_process_postop also needs to know about the Force Conversion flag...

Here's another patch with that included

aznboy’s picture

Looks promising. Will try it out and report back.

aznboy’s picture

Works pretty well. Had to change '/' for windows localhost but overall not problems with 2 encodes.

Tried converting to flv and h264 and was successful.

aznboy’s picture

One thing I found and not sure if this is a flashvideo problem or the patch but on qt-faststart if you send files to the batch folder only the first file will run through the qt-faststart while the other files will not so they have to be fully downloaded to view.

I had mine setup with cck fields, run 5 files during cron, and was uploading files to a video import file to run in batches.

Dokuro’s picture

Vector, just found this, you are an all-star, the qt-faststart is key. I will talk to Travis about this and I think it should go into the latest dev, I am testing it out now.