Skip to content

Conversation

luke-gru
Copy link
Contributor

@luke-gru luke-gru commented Jun 6, 2011

I added the xdg-open system call because I think it might be more prevalent than the BROWSER env variable.
And I threw in Debian's default browser symlink as well, for good measure?

Let me know what you think, and thanks for gist, it's very useful!

@indirect
Copy link
Collaborator

indirect commented Jun 6, 2011

I don't use Linux myself, but if someone who does could test this and give it the thumbs up, I would be happy to pull it in.

On Jun 5, 2011, at 5:40 PM, luke-gru reply@reply.github.com wrote:

I added the xdg-open system call because I think it might be more prevalent than the BROWSER env variable.
And I threw in Debian's default browser symlink as well, for good measure?

Let me know what you think, and thanks for gist, it's very useful!

Reply to this email directly or view it on GitHub:
#50

@luke-gru
Copy link
Contributor Author

luke-gru commented Jun 6, 2011

Yeah, for what it's worth I tested it on my computers and it worked. But yeah, if someone else can test it that'd be cool.

@luke-gru luke-gru merged commit 627ac48 into defunkt:master Jun 19, 2011
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