)]}'
{"lib/swift":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"017398a0c4d1c7e13be693453102b12b1508b159","unresolved":false,"context_lines":[{"line_number":160,"context_line":"# Test if any Swift services are enabled"},{"line_number":161,"context_line":"# is_swift_enabled"},{"line_number":162,"context_line":"function is_swift_enabled {"},{"line_number":163,"context_line":"    [[ ,${ENABLED_SERVICES} \u003d~ ,\"^s-\" ]] \u0026\u0026 return 0"},{"line_number":164,"context_line":"    return 1"},{"line_number":165,"context_line":"}"},{"line_number":166,"context_line":""}],"source_content_type":"text/x-swift","patch_set":2,"id":"9abb7d3a_21eb1ab2","line":163,"updated":"2016-05-30 05:15:46.000000000","message":"this is why we just the trick with the leading \",\" -- so that it\u0027s always \",s-*\" that is matched.\n\nAre you seeing a problem?  We fixed a similar issue in I9deee735812cde44ea5140b1ad76848f02576609","commit_id":"965cacdb96f1735ae0fb214e15cefe241b58e8d1"},{"author":{"_account_id":12175,"name":"Eli Qiao","email":"qiaoliyong@gmail.com","username":"Eli"},"change_message_id":"90e182b723bc13b5744704acd850f8a9ab9778a5","unresolved":false,"context_lines":[{"line_number":160,"context_line":"# Test if any Swift services are enabled"},{"line_number":161,"context_line":"# is_swift_enabled"},{"line_number":162,"context_line":"function is_swift_enabled {"},{"line_number":163,"context_line":"    [[ ,${ENABLED_SERVICES} \u003d~ ,\"^s-\" ]] \u0026\u0026 return 0"},{"line_number":164,"context_line":"    return 1"},{"line_number":165,"context_line":"}"},{"line_number":166,"context_line":""}],"source_content_type":"text/x-swift","patch_set":2,"id":"9abb7d3a_7c978590","line":163,"in_reply_to":"9abb7d3a_21395a42","updated":"2016-05-30 05:42:44.000000000","message":"with I9deee735812cde44ea5140b1ad76848f02576609, it works, I don\u0027t think we need this patch anymore","commit_id":"965cacdb96f1735ae0fb214e15cefe241b58e8d1"},{"author":{"_account_id":12175,"name":"Eli Qiao","email":"qiaoliyong@gmail.com","username":"Eli"},"change_message_id":"b4a610cfd34675535437ada0d69d10474436b0b0","unresolved":false,"context_lines":[{"line_number":160,"context_line":"# Test if any Swift services are enabled"},{"line_number":161,"context_line":"# is_swift_enabled"},{"line_number":162,"context_line":"function is_swift_enabled {"},{"line_number":163,"context_line":"    [[ ,${ENABLED_SERVICES} \u003d~ ,\"^s-\" ]] \u0026\u0026 return 0"},{"line_number":164,"context_line":"    return 1"},{"line_number":165,"context_line":"}"},{"line_number":166,"context_line":""}],"source_content_type":"text/x-swift","patch_set":2,"id":"9abb7d3a_21395a42","line":163,"in_reply_to":"9abb7d3a_21eb1ab2","updated":"2016-05-30 05:22:10.000000000","message":"I see that if we add new service like \"xxxs-api\", is_swift_enabled will return true, just check my devstack version, it doesn\u0027t container fix I9deee735812cde44ea5140b1ad76848f02576609, I will do a test with that fix and get back here.","commit_id":"965cacdb96f1735ae0fb214e15cefe241b58e8d1"}]}