)]}'
{"devstack-vm-gate.sh":[{"author":{"_account_id":6609,"name":"Elizabeth K. Joseph","email":"lyz@princessleia.com","username":"lyz"},"change_message_id":"b0bf087e8e6d3571f179391b12ddfd2b72b4add1","unresolved":false,"context_lines":[{"line_number":49,"context_line":"    fi"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"    # Install PyYaml for test-matrix.py"},{"line_number":52,"context_line":"    if is_ubuntu_or_debian; then"},{"line_number":53,"context_line":"        sudo apt-get update"},{"line_number":54,"context_line":"        sudo apt-get install python-yaml"},{"line_number":55,"context_line":"    elif is_fedora; then"}],"source_content_type":"text/x-sh","patch_set":6,"id":"5a890539_b03fff23","line":52,"updated":"2014-12-04 22:01:37.000000000","message":"Do you instead mean uses_debs here?","commit_id":"8b8fe44d9cf227f8e6c59b39f505e5307d38d04c"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"46ba077a9ff5c37a6e01ba1eb4dd2baf73bfd46e","unresolved":false,"context_lines":[{"line_number":49,"context_line":"    fi"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"    # Install PyYaml for test-matrix.py"},{"line_number":52,"context_line":"    if is_ubuntu_or_debian; then"},{"line_number":53,"context_line":"        sudo apt-get update"},{"line_number":54,"context_line":"        sudo apt-get install python-yaml"},{"line_number":55,"context_line":"    elif is_fedora; then"}],"source_content_type":"text/x-sh","patch_set":6,"id":"5a890539_0a17c5ee","line":52,"updated":"2014-12-04 18:14:24.000000000","message":"so where is this defined now?","commit_id":"8b8fe44d9cf227f8e6c59b39f505e5307d38d04c"},{"author":{"_account_id":6133,"name":"yolanda.robla","email":"yroblamo@redhat.com","username":"yolanda.robla"},"change_message_id":"585a7a92d97d0a515d5f36e06340d45f6974624c","unresolved":false,"context_lines":[{"line_number":49,"context_line":"    fi"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"    # Install PyYaml for test-matrix.py"},{"line_number":52,"context_line":"    if is_ubuntu_or_debian; then"},{"line_number":53,"context_line":"        sudo apt-get update"},{"line_number":54,"context_line":"        sudo apt-get install python-yaml"},{"line_number":55,"context_line":"    elif is_fedora; then"}],"source_content_type":"text/x-sh","patch_set":6,"id":"5a890539_996f7efc","line":52,"in_reply_to":"5a890539_b03fff23","updated":"2014-12-05 09:23:00.000000000","message":"Done","commit_id":"8b8fe44d9cf227f8e6c59b39f505e5307d38d04c"}],"functions.sh":[{"author":{"_account_id":6609,"name":"Elizabeth K. Joseph","email":"lyz@princessleia.com","username":"lyz"},"change_message_id":"eab048202e5fd9c5b969b5a828a95295cc3a8c39","unresolved":false,"context_lines":[{"line_number":28,"context_line":"}"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"function is_debian {"},{"line_number":31,"context_line":"    lsb_release -i 2\u003e/dev/null | grep -iq \"debian\""},{"line_number":32,"context_line":"}"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"function function_exists {"}],"source_content_type":"text/x-sh","patch_set":1,"id":"9aa7fdbe_cc44301d","line":31,"updated":"2014-10-25 14:29:21.000000000","message":"The lsb-base package is not default on Debian images, so we may need to check for this first.","commit_id":"e9a944e57508c2333d65001f7f1d31699e1f2a5c"},{"author":{"_account_id":6609,"name":"Elizabeth K. Joseph","email":"lyz@princessleia.com","username":"lyz"},"change_message_id":"dd591ab1e8fb4de48da7ac70fee87466d37c92d1","unresolved":false,"context_lines":[{"line_number":28,"context_line":"}"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"function is_debian {"},{"line_number":31,"context_line":"    lsb_release -i 2\u003e/dev/null | grep -iq \"debian\""},{"line_number":32,"context_line":"}"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"function function_exists {"}],"source_content_type":"text/x-sh","patch_set":1,"id":"9aa7fdbe_9f43ed11","line":31,"in_reply_to":"9aa7fdbe_66d438ec","updated":"2014-10-28 17:02:34.000000000","message":"Sure, that could work too.","commit_id":"e9a944e57508c2333d65001f7f1d31699e1f2a5c"},{"author":{"_account_id":6133,"name":"yolanda.robla","email":"yroblamo@redhat.com","username":"yolanda.robla"},"change_message_id":"d7d44bf8caed9ce35911de995e48c229ece99284","unresolved":false,"context_lines":[{"line_number":28,"context_line":"}"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"function is_debian {"},{"line_number":31,"context_line":"    lsb_release -i 2\u003e/dev/null | grep -iq \"debian\""},{"line_number":32,"context_line":"}"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"function function_exists {"}],"source_content_type":"text/x-sh","patch_set":1,"id":"9aa7fdbe_66d438ec","line":31,"in_reply_to":"9aa7fdbe_cc44301d","updated":"2014-10-27 08:51:33.000000000","message":"Shall we better rely on a cat /etc/*-release ? Only for Debian?","commit_id":"e9a944e57508c2333d65001f7f1d31699e1f2a5c"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"7f20da925b74b551bfb83ef232966564ee7e578b","unresolved":false,"context_lines":[{"line_number":29,"context_line":""},{"line_number":30,"context_line":"function is_debian {"},{"line_number":31,"context_line":"    # do not rely on lsb_release because it may be not installed by default"},{"line_number":32,"context_line":"    cat /etc/*-release | grep DISTRIB_ID 2\u003e/dev/null | grep -iq \"debian\""},{"line_number":33,"context_line":"}"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"function function_exists {"}],"source_content_type":"text/x-sh","patch_set":2,"id":"5a890539_562fa5f6","line":32,"updated":"2014-11-18 20:37:34.000000000","message":"looking at [1], isn\u0027t this just \"ID\"?\n\n[1] http://sources.debian.net/src/base-files/7.10/etc/os-release/","commit_id":"5cb542ec264237b8907ee2d889e58cdc59d56e33"},{"author":{"_account_id":6133,"name":"yolanda.robla","email":"yroblamo@redhat.com","username":"yolanda.robla"},"change_message_id":"f6e780540de4610f1be6868689f4df66d33aebeb","unresolved":false,"context_lines":[{"line_number":29,"context_line":""},{"line_number":30,"context_line":"function is_debian {"},{"line_number":31,"context_line":"    # do not rely on lsb_release because it may be not installed by default"},{"line_number":32,"context_line":"    cat /etc/*-release | grep DISTRIB_ID 2\u003e/dev/null | grep -iq \"debian\""},{"line_number":33,"context_line":"}"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"function function_exists {"}],"source_content_type":"text/x-sh","patch_set":2,"id":"5a890539_875f0775","line":32,"in_reply_to":"5a890539_562fa5f6","updated":"2014-11-19 17:42:17.000000000","message":"You are totally right. I was assuming that the file was the same as used in Ubuntu, with different settings, but it\u0027s not the case. Thx for the tip!","commit_id":"5cb542ec264237b8907ee2d889e58cdc59d56e33"}]}