makeqctoolsreport - fix transcode function and add shebang

[kieranjol]

Initially ffmpegstring was a string, but subprocess.call requires a list of arguments. Here was the error that I kept getting:

Traceback (most recent call last):
  File "makeqctoolsreport.py", line 81, in <module>
    transcode()
  File "makeqctoolsreport.py", line 60, in transcode
    subprocess.call(ffmpegstring)
  File "/usr/local/Cellar/python/2.7.11/Frameworks/Python.framework/Versions/2.7/lib/python2.7/subprocess.py", line 522, in call
    return Popen(*popenargs, **kwargs).wait()
  File "/usr/local/Cellar/python/2.7.11/Frameworks/Python.framework/Versions/2.7/lib/python2.7/subprocess.py", line 710, in __init__
    errread, errwrite)
  File "/usr/local/Cellar/python/2.7.11/Frameworks/Python.framework/Versions/2.7/lib/python2.7/subprocess.py", line 1335, in _execute_child
    raise child_exception
OSError: [Errno 2] No such file or directory

This pull request turns the command into a list. Because inputCodec and filterString can be empty, it causes the ffmpeg command to fail, which is why the if statement is there.

Why is the transcode necessary at all? Can't you just run ffprobe on the source file directly?

Also I added a shebang for good measure.

[kieranjol]

Oh, I also removed the return from the function. As the only action of the function is to create a file, I don't think that the function needs to return anything.

[FutureDays]

Kieran, I'll fix that transcode to only go if the return from ffprobe is j2k

[kieranjol]

Ah cool!