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

Check that lsb-release is installed #1

Closed
wants to merge 1 commit into from

Conversation

rdoursenaud
Copy link

lsb_release may not installed leading to a corrupt apt list file.
Let's install it!

@rvagg
Copy link
Contributor

rvagg commented Aug 16, 2014

I've been wondering if it's safe enough to just inspect /etc/debian_version when lsb_release isn't installed, since it's only Debian on which this is a problem, it might be worth us exploring that possibility first.

@rvagg
Copy link
Contributor

rvagg commented Aug 16, 2014

In fact, installing lsb_release and running it on Debian gives me:

No LSB modules are available.

So we're going to need to be cleverer anyway!

@chrislea
Copy link
Contributor

That's what lsb_release does with no arguments. Try

lsb_release -c -s

@rdoursenaud
Copy link
Author

lsb_release is by far the most reliable way to determine a linux distro flavor.
I don't see any drawbacks.

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.

3 participants