)]}'
{"stack.sh":[{"author":{"_account_id":7175,"name":"Robbie Harwood","email":"rharwood@redhat.com","username":"frozencemetery"},"change_message_id":"22111390eaaae1bcf960f3a66e2f9dbc8e30cc41","unresolved":false,"context_lines":[{"line_number":34,"context_line":"# Make sure umask is sane"},{"line_number":35,"context_line":"umask 022"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"# Not all distros have sbin in PATH for regular users."},{"line_number":38,"context_line":"PATH\u003d$PATH:/usr/local/sbin:/usr/sbin:/sbin"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"# Keep track of the devstack directory"}],"source_content_type":"text/x-sh","patch_set":1,"id":"1abeadc6_044ee03a","line":37,"updated":"2014-08-22 21:16:55.000000000","message":"Checking in path will more than triple the length of the code for no gain that I can see.  Please do not do that.","commit_id":"7df9d1be17162feabeaba35faa87baf09debe590"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"c6c0ab80fe7a96a66041852fbf3fa864fa490c98","unresolved":false,"context_lines":[{"line_number":34,"context_line":"# Make sure umask is sane"},{"line_number":35,"context_line":"umask 022"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"# Not all distros have sbin in PATH for regular users."},{"line_number":38,"context_line":"PATH\u003d$PATH:/usr/local/sbin:/usr/sbin:/sbin"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"# Keep track of the devstack directory"}],"source_content_type":"text/x-sh","patch_set":1,"id":"baada198_fd571bff","line":37,"updated":"2014-07-24 16:30:28.000000000","message":"Would you mind checking that these new paths are not already in $PATH before adding them ?\n\nI know it doesn\u0027t hurt to have the same path several times in $PATH but it\"s not really nice.","commit_id":"7df9d1be17162feabeaba35faa87baf09debe590"},{"author":{"_account_id":11279,"name":"Angus Lees","email":"gus@inodes.org","username":"gus"},"change_message_id":"09bc41317292bfe4f0124b38d4e781a9e6863adc","unresolved":false,"context_lines":[{"line_number":34,"context_line":"# Make sure umask is sane"},{"line_number":35,"context_line":"umask 022"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"# Not all distros have sbin in PATH for regular users."},{"line_number":38,"context_line":"PATH\u003d$PATH:/usr/local/sbin:/usr/sbin:/sbin"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"# Keep track of the devstack directory"}],"source_content_type":"text/x-sh","patch_set":1,"id":"baada198_6fcd035d","line":37,"in_reply_to":"baada198_fd571bff","updated":"2014-07-25 00:29:38.000000000","message":"Checking before adding to PATH isn\u0027t the usual shell idiom I see elsewhere (in devstack and outside openstack), so I\u0027d rather not :/\nIs there somewhere that PATH is displayed that makes it unusually important for it to be \"pretty\"?","commit_id":"7df9d1be17162feabeaba35faa87baf09debe590"}]}