)]}'
{"nova/compat/flagfile.py":[{"author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"change_message_id":"ac36ed2426cbf7a6d76e110038b656f420b3ec61","unresolved":false,"context_lines":[{"line_number":99,"context_line":""},{"line_number":100,"context_line":"    if args and not args[0].startswith(\u0027--\u0027):"},{"line_number":101,"context_line":"        # This is a config file, not a flagfile, so return it."},{"line_number":102,"context_line":"        return [\u0027--config-file\u003d\u0027 + flagfile] + argp[1:]"},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"    #"},{"line_number":105,"context_line":"    # We\u0027re recursing here to convert any --flagfile arguments"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAEn%2F%2Fx18%3D","line":102,"updated":"2012-03-06 01:07:54.000000000","message":"Note that this fails if your nova.conf in flag file format starts with a # comment or a newline...  The test you added would still pass if you had a flag file containing \"\\n\\n--foo\u003dbar\" or \"# woot\\n--foo\u003dbar\"","commit_id":"2c37e5637d4b40df3c0eae48b2b55f64120dc2fe"},{"author":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"change_message_id":"dc6457e029908e2defb9f85e32338c279827c085","unresolved":false,"context_lines":[{"line_number":99,"context_line":""},{"line_number":100,"context_line":"    if args and not args[0].startswith(\u0027--\u0027):"},{"line_number":101,"context_line":"        # This is a config file, not a flagfile, so return it."},{"line_number":102,"context_line":"        return [\u0027--config-file\u003d\u0027 + flagfile] + argp[1:]"},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"    #"},{"line_number":105,"context_line":"    # We\u0027re recursing here to convert any --flagfile arguments"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAEn%2F%2Fx1o%3D","line":102,"in_reply_to":"AAAAEn%2F%2Fx18%3D","updated":"2012-03-06 01:17:00.000000000","message":"_read_lines above strips out # and // lines. I put comments in the test to ensure that it still works.","commit_id":"2c37e5637d4b40df3c0eae48b2b55f64120dc2fe"},{"author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"change_message_id":"fd741fe8b9f2809c88082113f9b02d0efd5a7fcb","unresolved":false,"context_lines":[{"line_number":99,"context_line":""},{"line_number":100,"context_line":"    if args and not args[0].startswith(\u0027--\u0027):"},{"line_number":101,"context_line":"        # This is a config file, not a flagfile, so return it."},{"line_number":102,"context_line":"        return [\u0027--config-file\u003d\u0027 + flagfile] + argp[1:]"},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"    #"},{"line_number":105,"context_line":"    # We\u0027re recursing here to convert any --flagfile arguments"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAEn%2F%2Fx1c%3D","line":102,"in_reply_to":"AAAAEn%2F%2Fx1o%3D","updated":"2012-03-06 01:23:12.000000000","message":"Aha..  So it does.  What\u0027s missing is a test that contains \"\\n\\n# comment\\n--foo\u003dbar\\n\" that shows it got converted to config file format correctly still... then I guess it would have been obvious that this didn\u0027t break that.  But... your reply is good enough for me.","commit_id":"2c37e5637d4b40df3c0eae48b2b55f64120dc2fe"}]}