)]}'
{"rally/ui/utils.py":[{"author":{"_account_id":7369,"name":"Sergey Skripnick","email":"sskripnick@mirantis.com","username":"redixin"},"change_message_id":"cb3c912367ce36385ff03d2b8a66044a5d1ddddf","unresolved":false,"context_lines":[{"line_number":26,"context_line":"    \"Where key-1,value-1 and key-2,value-2 are key pairs of template.\")"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"def get_template(template_path):"},{"line_number":30,"context_line":"    import mako.lookup"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"    templates_dir \u003d os.path.join(os.path.dirname(__file__), \"templates\")"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a26ad4f_4a3a1a38","line":29,"updated":"2015-10-28 13:46:12.000000000","message":"Not sure if it will work if we just install rally from pypi.\n\nThe only way for working with resources is pkgutil. Did you tried to use it?","commit_id":"70eb75d9ca25ff3d35f255e25f354f343303aded"},{"author":{"_account_id":10475,"name":"Alexander Maretskiy","email":"amaretskiy@mirantis.com","username":"maretskiy"},"change_message_id":"598f8b2457d0f907454c897edfafedc659aedc85","unresolved":false,"context_lines":[{"line_number":26,"context_line":"    \"Where key-1,value-1 and key-2,value-2 are key pairs of template.\")"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"def get_template(template_path):"},{"line_number":30,"context_line":"    import mako.lookup"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"    templates_dir \u003d os.path.join(os.path.dirname(__file__), \"templates\")"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a26ad4f_6a7916fc","line":29,"in_reply_to":"1a26ad4f_4a3a1a38","updated":"2015-10-28 13:48:18.000000000","message":"@Sergey, it is not clear for me why this can fail","commit_id":"70eb75d9ca25ff3d35f255e25f354f343303aded"},{"author":{"_account_id":7369,"name":"Sergey Skripnick","email":"sskripnick@mirantis.com","username":"redixin"},"change_message_id":"5d6610b83fe1a1d66fc2e8f558279f43f8c0ef4e","unresolved":false,"context_lines":[{"line_number":26,"context_line":"    \"Where key-1,value-1 and key-2,value-2 are key pairs of template.\")"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"def get_template(template_path):"},{"line_number":30,"context_line":"    import mako.lookup"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"    # NOTE(amaretskiy): \"/\" here allows getting template by absolute path"}],"source_content_type":"text/x-python","patch_set":4,"id":"1a26ad4f_0074be65","line":29,"updated":"2015-10-30 12:49:42.000000000","message":"This method is intended to get mako template by file path.\n\nThis can be done in this way:\n\n def get_template(template_path):\n   return mako.Template(filename\u003dtemplate_path)\n\nI have no idea why current implementation such complicated.\n\nIf we want to use template inheritance we can place all templates in one directory:\n\n lookup_dirs \u003d os.path.dirname(template_path)\n lookup \u003d mako.lookup.TemplateLookup(directories\u003dlookup_dirs)\n return mako.Template(filename\u003dtemplate_path, lookup\u003dlookup)\n\nOr we can even add one more argument with custom_lookup dir:\n\ndef get_template(lookup_dir, template_name):\n  ....\n\nBut currently we can use first option.","commit_id":"90c439185a02e2331c92c53aae993bcf838e767d"},{"author":{"_account_id":10475,"name":"Alexander Maretskiy","email":"amaretskiy@mirantis.com","username":"maretskiy"},"change_message_id":"b1dcd5b2a6606f98cbcca82bfd8ab72632c38eed","unresolved":false,"context_lines":[{"line_number":26,"context_line":"    \"Where key-1,value-1 and key-2,value-2 are key pairs of template.\")"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"def get_template(template_path):"},{"line_number":30,"context_line":"    import mako.lookup"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"    # NOTE(amaretskiy): \"/\" here allows getting template by absolute path"}],"source_content_type":"text/x-python","patch_set":4,"id":"1a26ad4f_bd35ecda","line":29,"in_reply_to":"1a26ad4f_0074be65","updated":"2015-10-30 15:11:04.000000000","message":"Your proposal looks good for Mako, but it will require a bit complicated implementation for Jinja2 (we will swith to Jinja2 soon)","commit_id":"90c439185a02e2331c92c53aae993bcf838e767d"}]}
