checkdeps.py broken

[PeterSurda]

Traceback (most recent call last):
  File "checkdeps.py", line 142, in <module>
    if x in EXTRAS_REQUIRE_DEPS
KeyError: None

[PeterSurda]

At least on Debian it's still broken after e1d2ead, haven't tested others.

[g1itch]

Sad

[PeterSurda]

What OS did you test it on?

[PeterSurda]

It probably works now with Ubuntu for example, but on Debian this is in the os-releases:

NAME="Debian GNU/Linux"

and it looks like that's parsed incorrectly, maybe it should be added to the OS_RELEASE dict.

[g1itch]

@PeterSurda Gentoo/Calculate, Ubuntu:Xenial and Ubuntu:Trusy - on Travis CI.

[g1itch]

wow

[g1itch]

I really forgot Debian, excuse me please.
Possible further changes:

diff --git a/checkdeps.py b/checkdeps.py
index f261c924..4115c5a8 100755
--- a/checkdeps.py
+++ b/checkdeps.py
@@ -136,6 +136,8 @@ if prereqs:
 OPSYS = detectOS()
 CMD = PACKAGE_MANAGER[OPSYS] if OPSYS in PACKAGE_MANAGER else 'UNKNOWN_INSTALLER'
 for lhs, rhs in EXTRAS_REQUIRE.items():
+    if OPSYS is None:
+        break
     if rhs and any([
         EXTRAS_REQUIRE_DEPS[x][OPSYS]
         for x in rhs
diff --git a/src/depends.py b/src/depends.py
index 39f5e759..d284a11b 100755
--- a/src/depends.py
+++ b/src/depends.py
@@ -29,6 +29,7 @@ logger.addHandler(handler)
 logger.setLevel(logging.ERROR)
 
 OS_RELEASE = {
+    "Debian GNU/Linux".lower(): "Debian",
     "fedora": "Fedora",
     "opensuse": "openSUSE",
     "ubuntu": "Ubuntu",

Maybe it was my mistake to agree with creation of OS_RELEASE dict.