set format based on extension

[urg]

There appears to be a bug where if you set the output file with extension and give an input file, the output file will be generated only as a pdf. This is a quick fix to get it working, but don't know the script very well, so perhaps there's a better place to add this fix.

[skywinder]

Nice addition! @dagwieers I think, it makes sense to merge it. Thoughts?

[dagwieers]

Looks fine, however I would use.lstrip('.') instead. And maybe document the surrounding code as well ;-) I will amend the change after merging.

Thanks !

[dagwieers]

Done. 291895b

[skywinder]

@dagwieers is back! Hooray! 👍

[dagwieers]

@skywinder I was flooded by lots of unoconv PR closings, so you did get my attention ;-)

PS I wouldn't mind if you want to get more involved and merge PRs or do releases.

[ghost]

This is a late comment, but this broke my system. I recently upgraded my server to the next Ubuntu LTS and with it came the new Unoconv.

My system makes a tempfile with a random name and without extension, then called the unoconv binary and supplied the path to the --output parameter. Then my system went looking for this file, which wasn't there because it was looking for /tmp/tempfile_TQC4Nd and unoconv wrote to /tmp/tempfile_TQC4Nd.csv.

I worked around this by doing the following:
(unoconv -vvvvvvvvvvvvvvvv --format csv --stdout infile.xls > /tmp/tempfile_TQC4Nd) 2>&1

I'm using a subshell here so I can redirect the stderr messages to the stdout without it ending up in the converted file (and log the debug messages).

For the record, this is not a complaint (Unoconv has made my life so much easier when dealing with awful Office file formats), I just wanted to say that sometimes changes that seem low impact can indeed break other peoples systems. And if there is someone else who ran into this problem, they can use my solution.