)]}'
{"functions.sh":[{"author":{"_account_id":6473,"name":"Sunil Thaha","email":"sthaha@redhat.com","username":"sthaha"},"change_message_id":"127b22f50bef66e82cf90cda305dd1c6f7df4df5","unresolved":false,"context_lines":[{"line_number":355,"context_line":"    # This is necessary to keep sudo from complaining"},{"line_number":356,"context_line":"    fix_etc_hosts"},{"line_number":357,"context_line":""},{"line_number":358,"context_line":"    # Detect OS type"},{"line_number":359,"context_line":"    # Ubuntu has an lsb_release command which allows us to detect if it is Ubuntu"},{"line_number":360,"context_line":"    if [ \"$DEVSTACK_GATE_VIRT_DRIVER\" \u003d\u003d \"libvirt\" ]; then"},{"line_number":361,"context_line":"        if lsb_release -i 2\u003e/dev/null | grep -iq ubuntu"}],"source_content_type":"text/x-sh","patch_set":3,"id":"7adec928_bf1a4115","side":"PARENT","line":358,"updated":"2014-05-09 05:10:26.000000000","message":"I prefer to retain the nested if since checking for another distro will have to switch to a nested if\n\nif [ libvirt ]; then\n   ... distro_common_libvirt\n   if is_ubuntu; then\n      ... \n   elif is_fedora; then\n      ...\n   fi\nfi","commit_id":"635db7bc0c56da62a17e7ba6058c34a0eca63005"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"65fc83b869818da02bfc3bca6d4a539d8969a615","unresolved":false,"context_lines":[{"line_number":365,"context_line":"    # This is necessary to keep sudo from complaining"},{"line_number":366,"context_line":"    fix_etc_hosts"},{"line_number":367,"context_line":""},{"line_number":368,"context_line":"    if is_ubuntu \u0026\u0026 [ \"$DEVSTACK_GATE_VIRT_DRIVER\" \u003d\u003d \"libvirt\" ]; then"},{"line_number":369,"context_line":"        # Temporary work around to install a patched libvirt 0.9.8+"},{"line_number":370,"context_line":"        enable_latest_libvirt"},{"line_number":371,"context_line":"    fi"}],"source_content_type":"text/x-sh","patch_set":3,"id":"7adec928_a5ace0a1","line":368,"updated":"2014-05-09 20:20:55.000000000","message":"this is being removed in Id094a7e3e3004acd3d47b7fe34161ed2995e095b","commit_id":"0b9d5e9b8cbc5e24684d90b61a2c42f713f5f7f0"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"6ccb086bb354fe2cfd0c89208b1de56f998e6117","unresolved":false,"context_lines":[{"line_number":365,"context_line":"    # This is necessary to keep sudo from complaining"},{"line_number":366,"context_line":"    fix_etc_hosts"},{"line_number":367,"context_line":""},{"line_number":368,"context_line":"    if is_ubuntu \u0026\u0026 [ \"$DEVSTACK_GATE_VIRT_DRIVER\" \u003d\u003d \"libvirt\" ]; then"},{"line_number":369,"context_line":"        # Temporary work around to install a patched libvirt 0.9.8+"},{"line_number":370,"context_line":"        enable_latest_libvirt"},{"line_number":371,"context_line":"    fi"}],"source_content_type":"text/x-sh","patch_set":3,"id":"7adec928_872a7e2a","line":368,"in_reply_to":"7adec928_a5ace0a1","updated":"2014-05-12 00:32:35.000000000","message":"Done","commit_id":"0b9d5e9b8cbc5e24684d90b61a2c42f713f5f7f0"},{"author":{"_account_id":6473,"name":"Sunil Thaha","email":"sthaha@redhat.com","username":"sthaha"},"change_message_id":"8212d149b617c81639fe4e975eec1f4cbcc88632","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"# Distro check functions"},{"line_number":20,"context_line":"function is_fedora {"},{"line_number":21,"context_line":"    lsb_release -i 2\u003e/dev/null | grep -iq \"fedora\""},{"line_number":22,"context_line":"}"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"function is_ubuntu {"}],"source_content_type":"text/x-sh","patch_set":4,"id":"7adec928_6ae8d9b4","line":21,"updated":"2014-05-12 04:53:40.000000000","message":"How about:\n\n\nfunction is_fedora {\n  [[ $(lsb_release -i) \u003d~ \"[fF]edora\" ]]\n}","commit_id":"882690fcce6674ab92e7593938d7c2da0a0fd2f6"},{"author":{"_account_id":6473,"name":"Sunil Thaha","email":"sthaha@redhat.com","username":"sthaha"},"change_message_id":"8212d149b617c81639fe4e975eec1f4cbcc88632","unresolved":false,"context_lines":[{"line_number":22,"context_line":"}"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"function is_ubuntu {"},{"line_number":25,"context_line":"    lsb_release -i 2\u003e/dev/null | grep -iq \"ubuntu\""},{"line_number":26,"context_line":"}"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":""}],"source_content_type":"text/x-sh","patch_set":4,"id":"7adec928_eadb0919","line":25,"updated":"2014-05-12 04:53:40.000000000","message":"same as above: \n\nfunction is_ubuntu {\n [[ $(lsb_release -i) \u003d~ \"[Uu]buntu\" ]]\n}","commit_id":"882690fcce6674ab92e7593938d7c2da0a0fd2f6"}]}