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.

Sign up for GitHub

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

Jump to bottom

debian stable requires lsb_release #6

Closed
avsm opened this issue Mar 20, 2015 · 0 comments · Fixed by #7
Closed

debian stable requires lsb_release #6

avsm opened this issue Mar 20, 2015 · 0 comments · Fixed by #7

Comments

@avsm
Copy link
Contributor

avsm commented Mar 20, 2015

This is a depext outside of normal OPAM's, so might be worth trying a heuristic if it doesnt exist. Its not installed by default in Debian Wheezy afaict.
avsm added a commit to avsm/opam-depext that referenced this issue Mar 21, 2015
@avsm
Fix has_command by not going through a subshell
66cb933 
```
$ command -v fooooo
$ echo $?
1
$ /bin/sh -c command -v foooo
$ echo $?
0
```

We now just call `command` directly, which fixes ocaml-opam#6
@avsm avsm mentioned this issue Mar 21, 2015
Merged
@AltGr AltGr closed this as completed in #7 Mar 23, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix has_command, CentOS detection, source installation and add OS update support avsm/opam-depext
1 participant
@avsm