Skip to content

Conversation

@jolros
Copy link

@jolros jolros commented Dec 19, 2013

Hi,

I thought this was a great idea for a utility, thanks! I wanted to avoid file cleanup and a manual call to "qlmanage" or "open" in my own process, so I added an option (-p) that runs Quicklook on a temporary file instead of printing to STDOUT. It's backwards compatible with the existing command usage.

No worries if you don't want this functionality, but it was useful for me.

(I deleted my initial pull request and added this one in order to include documentation in the README file)

Thanks again.

@Profpatsch
Copy link

It’s not a good idea to include calls to a specific program. That’s what the command line and stdout is for.

@Profpatsch
Copy link

Tip: all those lines of code can be replaced by:

#!/bin/bash

plot "$@" | qlmanage -p -

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants