Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update to xdg-open 1.1.1 #46

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

plynchnlm
Copy link

I downloaded xdg-utils and built version 1.1.1. The manual tests for node-open pass, though I did try running them as "sudo" as recommended.

@pwnall
Copy link
Owner

pwnall commented Nov 18, 2015

I'd be happy to update to a new stable version of xdg-open. Can you please tell me the steps you took to build this, so I can verify the source?

Thank you very much!

@plynchnlm
Copy link
Author

Sure:

  1. clone the xdg-utils repository (http://cgit.freedesktop.org/xdg/xdg-utils/ -- see links at the bottom) and check out version 1.1.1

  2. edit scripts/xdg-utils-common.in to fix the version number string. You'll see a line with echo "@NAME@ 1.1.0 rc3". Change that to 1.1.1. I filed a bugzilla report about that, and heard back that it will be fixed with the next version.

  3. Ignore what the README says about not needing to run make. The files in the scripts directory are not ready for running.

  4. Run ./configure, and then make. I can't remember if I had to run make install. If you need to do that, when you run ./configure add --prefix=somewhere to set your installation path, so it doesn't try to put stuff in /usr/bin. I do remember having to install a package for xmlto, and the lynx text browser to get make to run properly.

If you run into difficulty, you could also just copy opn's xdg-open, which also claims to be 1.1.0 rc3. (It is not the same as 1.1.1, though, but should be at least 1.1.0 rc3.)

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