Fixes for UTI file types fixes #1428, #1427

[pjrobertson]

Seems liks QSUTIForExtensionOrType() isn't that reliable for getting the UTI of a file, it was return dyn.agk8u for public.executable files, whereas QSUTIWithLSInfoRec() was doing the right thing, so I switched to that and added a comment

This is against release since this bug pretty much cripples the Terminal Plugin :)

OK seems like GitHub doesn't like chrome. This is against release

[skurfer]

Release? It was all done! 💩

😉

[skurfer]

This doesn't fix the fact that arguments from the third pane are ignored, but I gather it isn't supposed to.

[pjrobertson]

yeah, that's probably something else. I hadn't looked into it since I don't
have any scripts that take arguments, I guess I could just use something
from /usr/bin though - duh!

Is it too late for release then? If so we'll have to wait for a point
release (We could do a quick one after ß72

On 20 March 2013 01:49, Rob McBroom notifications@github.com wrote:

This doesn't fix the fact that arguments from the third pane are ignored,
but I gather it isn't supposed to.

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/1430#issuecomment-15135255
.

[skurfer]

Not too late, just more work. It'll go in.

[pjrobertson]

k sorry for the extra work :'(

On 20 March 2013 21:36, Rob McBroom notifications@github.com wrote:

Not too late, just more work. It'll go in.

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/1430#issuecomment-15179165
.