Conversion fails with chained custom commands

Hi!
The output seems like the commands you'd typically send to FFmpeg when converting a file: checks of file size, file type etc, and then the actual conversion. The strange thing is that only the actual converting command (/usr/local/bin/ffmpeg -i sites/default/files/justdoit_0.mov -ab 64 -ar 44100 -b 1000 -r 29 -nr 1000 -g 500 -s 704:576 -qmax 2 /tmp/justdoit_0.flv; /usr/bin/flvtool2 -U /tmp/justdoit_0.flv) should show up in that location. What's also strange is that the commands are glued together without whitespace/blank lines separating them. Or is that just a copy-paste problem?

Printing out the $command variable should give you exactly the same information that you'd get from the debug/logging feature. Please check there too and report back about what commands run and if they are properly separated.

To clarify, when I was talking about "printing out $command" I was referring to the line of code you got from Arthur. There is a debug setting which will let you see the contents of that very same variable in the watchdog. So my question was, does it show the same values there?

You write in your email that the commands above are output exactly like that, glued together. This is a mystery to me.

I think in order to debug this we need to look at your configuration. You can help by adding yet another line of code to the one you got from Arthur:

drupal_set_message(print_r($configuration, true));
drupal_set_message(print_r($command, true));

These two lines should come after the line with $command = implode(" ", $options);.

And to be clear: We want the $command variable to contain only this part:

-i sites/default/files/justdoit_0.mov -ab 64 -ar 44100 -b 1000 -r 29 -nr 1000 -g 500 -s 704:576 -qmax 2 /tmp/justdoit_0.flv; /usr/bin/flvtool2 -U /tmp/justdoit_0.flv

Note: the following part should *not* be included in your custom command setting: /usr/local/bin/ffmpeg

That's about as much as I can do right now. Please report back and I'll check on your progress as soon as I can.

That looks good actually (except for the "failed unexpectedly" part and the "1" messages). Have you double checked FFmpeg Wrapper's setting (/usr/bin/ffmpeg or similar)? If you add this part to the command you should be able to run that in a terminal and see what happens:

/usr/bin/ffmpeg -i sites/default/files/justdoit_3.mov -ab 64 -ar 44100 -b 1000 -r 29 -nr 1000 -g 500 -s 704:576 -qmax 2 /tmp/justdoit_3.flv; /usr/bin/flvtool2 -U /tmp/justdoit_3.flv

But maybe you've already done that? If that runs successfully there is probably a problem with file permissions or PHP security settings. You can also try FFmpeg Wrapper's testing feature, which should give you a more "real world" test environment.

The "1" messages have to come from another drupal_set_message() call somewhere. If not, something's very broken. Do you know where they come from?

The second screenshot shows the same wierd text you pasted in your first post. Does that still show up? To me that looks like there is some extra print commands or similar in ffmpeg_wrapper.module. Maybe it's time to install a fresh copy of these modules?

One last thing, take a look in /tmp and make sure that justdoit_3.flv hasn't actually been converted. There could be a problem with the error reporting.

OK, we're making progress.

It seems like the second command you put into your custom command field does something to FFmpeg Wrapper's result handling. I think the problem is that the shell output from FFmpeg is lost and replaced by that of flvtool (which is empty?). Also, you say that the output file is actually generated sucessfully.

If this is true, you should be able to make FFmpeg Converter work by changing its $output check so that it's strictly typed. The conversion function in FFmpeg Wrapper returns false on failure, and the command's output on success. If the output is empty, that is equal to false in the PHP world.

So on this row, just a few rows below where you put the other stuff before:

if ($output == false || !file_exists($output_file) || filesize($output_file) == 0) {

we'll add an equal sign like this:

if ($output === false || !file_exists($output_file) || filesize($output_file) == 0) {

See if that works.

Great! I'm committing the change in #8.

I'll create a new issue for your second problem.