)]}'
{"lib/cinder":[{"author":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"change_message_id":"2ae449f9b357911d7ace591ad941880af43fcfac","unresolved":false,"context_lines":[{"line_number":468,"context_line":"function install_cinderclient() {"},{"line_number":469,"context_line":"    git_clone $CINDERCLIENT_REPO $CINDERCLIENT_DIR $CINDERCLIENT_BRANCH"},{"line_number":470,"context_line":"    setup_develop $CINDERCLIENT_DIR"},{"line_number":471,"context_line":"    sudo install -D -m 0644 -o $STACK_USER {$CINDERCLIENT_DIR/tools/,/etc/bash_completion.d/}cinder.bash_completion"},{"line_number":472,"context_line":"}"},{"line_number":473,"context_line":""},{"line_number":474,"context_line":"# apply config.d approach for cinder volumes directory"}],"source_content_type":"application/octet-stream","patch_set":8,"id":"AAAARn%2F%2Fw7U%3D","line":471,"updated":"2013-08-22 05:30:18.000000000","message":"why not :\n\n{$CINDERCLIENT_DIR/tools,/etc/bash_completion.d}/cinder.","commit_id":"fac533e38db871631cee33d0e3c94884035851b8"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"655b42f50a5e856acf294c4ddb8b92ed1e8e234e","unresolved":false,"context_lines":[{"line_number":468,"context_line":"function install_cinderclient() {"},{"line_number":469,"context_line":"    git_clone $CINDERCLIENT_REPO $CINDERCLIENT_DIR $CINDERCLIENT_BRANCH"},{"line_number":470,"context_line":"    setup_develop $CINDERCLIENT_DIR"},{"line_number":471,"context_line":"    sudo install -D -m 0644 -o $STACK_USER {$CINDERCLIENT_DIR/tools/,/etc/bash_completion.d/}cinder.bash_completion"},{"line_number":472,"context_line":"}"},{"line_number":473,"context_line":""},{"line_number":474,"context_line":"# apply config.d approach for cinder volumes directory"}],"source_content_type":"application/octet-stream","patch_set":8,"id":"AAAASX%2F%2F8a4%3D","line":471,"in_reply_to":"AAAARn%2F%2Fuu8%3D","updated":"2013-08-23 12:17:26.000000000","message":"The -d would mean just create a directory (no source).\nThe -D means the last path element is a file.\nThe -t does not creates the directory.","commit_id":"fac533e38db871631cee33d0e3c94884035851b8"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"bf1a05da3d907ed1e78117962c6d68b191253bfa","unresolved":false,"context_lines":[{"line_number":468,"context_line":"function install_cinderclient() {"},{"line_number":469,"context_line":"    git_clone $CINDERCLIENT_REPO $CINDERCLIENT_DIR $CINDERCLIENT_BRANCH"},{"line_number":470,"context_line":"    setup_develop $CINDERCLIENT_DIR"},{"line_number":471,"context_line":"    sudo install -D -m 0644 -o $STACK_USER {$CINDERCLIENT_DIR/tools/,/etc/bash_completion.d/}cinder.bash_completion"},{"line_number":472,"context_line":"}"},{"line_number":473,"context_line":""},{"line_number":474,"context_line":"# apply config.d approach for cinder volumes directory"}],"source_content_type":"application/octet-stream","patch_set":8,"id":"AAAARn%2F%2Fuu8%3D","line":471,"in_reply_to":"AAAARn%2F%2Fw7U%3D","updated":"2013-08-22 16:05:05.000000000","message":"Why use {} at all?  The last arg to \u0027install -D\u0027 is a dir so you don\u0027t need to re-type the filename if it does not change.","commit_id":"fac533e38db871631cee33d0e3c94884035851b8"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"d3140d0c5cda10f821fba4dbde8eeb5e8c2f66d5","unresolved":false,"context_lines":[{"line_number":468,"context_line":"function install_cinderclient() {"},{"line_number":469,"context_line":"    git_clone $CINDERCLIENT_REPO $CINDERCLIENT_DIR $CINDERCLIENT_BRANCH"},{"line_number":470,"context_line":"    setup_develop $CINDERCLIENT_DIR"},{"line_number":471,"context_line":"    sudo install -D -m 0644 -o $STACK_USER {$CINDERCLIENT_DIR/tools/,/etc/bash_completion.d/}cinder.bash_completion"},{"line_number":472,"context_line":"}"},{"line_number":473,"context_line":""},{"line_number":474,"context_line":"# apply config.d approach for cinder volumes directory"}],"source_content_type":"application/octet-stream","patch_set":8,"id":"AAAASX%2F%2F7lw%3D","line":471,"in_reply_to":"AAAASX%2F%2F8a4%3D","updated":"2013-08-23 15:32:42.000000000","message":"Damn...I\u0027ve always stayed with the BSD options for portability...-D -t _should_ create the dir path IMnsHO...\n\nSo I still don\u0027t think this scans well but it it is fine.","commit_id":"fac533e38db871631cee33d0e3c94884035851b8"}]}