)]}'
{"monasca_setup/main.py":[{"author":{"_account_id":2419,"name":"Roland Hochmuth","email":"roland.hochmuth@hpe.com","username":"roland-hochmuth-s"},"change_message_id":"3ea00c460a88fe92623cf1903ea634783c2a329a","unresolved":false,"context_lines":[{"line_number":105,"context_line":"    detected_os \u003d platform.system()"},{"line_number":106,"context_line":"    if detected_os \u003d\u003d \u0027Linux\u0027:"},{"line_number":107,"context_line":"        linux_flavor \u003d platform.linux_distribution()[0]"},{"line_number":108,"context_line":"        if \u0027Ubuntu\u0027 in linux_flavor or \u0027debian\u0027 in linux_flavor:"},{"line_number":109,"context_line":"            for package in [\u0027coreutils\u0027]:"},{"line_number":110,"context_line":"                # Check for required dependencies for system checks"},{"line_number":111,"context_line":"                try:"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a6ced46_fa061b1c","line":108,"updated":"2015-03-25 19:12:22.000000000","message":"Wouldn\u0027t this be better as\n\nif flavor in support_linux_flavors\n\nif flavor in [\u0027Ubuntu\u0027, \u0027debian\u0027]","commit_id":"1ad8fe6bdd04504247a07db9be24c9851b58ff4d"},{"author":{"_account_id":11094,"name":"Tim Kuhlman","email":"timothy.kuhlman@canonical.com","username":"kuhlmant"},"change_message_id":"88d249d9f8e96b8db4462d0f0bdb54654a30f648","unresolved":false,"context_lines":[{"line_number":105,"context_line":"    detected_os \u003d platform.system()"},{"line_number":106,"context_line":"    if detected_os \u003d\u003d \u0027Linux\u0027:"},{"line_number":107,"context_line":"        linux_flavor \u003d platform.linux_distribution()[0]"},{"line_number":108,"context_line":"        if \u0027Ubuntu\u0027 in linux_flavor or \u0027debian\u0027 in linux_flavor:"},{"line_number":109,"context_line":"            for package in [\u0027coreutils\u0027]:"},{"line_number":110,"context_line":"                # Check for required dependencies for system checks"},{"line_number":111,"context_line":"                try:"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a6ced46_1a0bf77d","line":108,"in_reply_to":"1a6ced46_fa061b1c","updated":"2015-03-25 19:27:30.000000000","message":"I agree, linux_flavor looks to be just a string so syntax like \u0027flavor in [\u0027Ubnutu\u0027, \u0027debian\u0027]) is cleaner.","commit_id":"1ad8fe6bdd04504247a07db9be24c9851b58ff4d"}]}
