)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":2394,"name":"Adam Spiers","email":"aspiers@suse.com","username":"adam.spiers"},"change_message_id":"5c22e703c2e9cb7a6690756f1ee4d1e4f7832499","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Note that while this is the complete auto page generation of the"},{"line_number":14,"context_line":"devstack.org site it does not yet push the content back to GitHub."},{"line_number":15,"context_line":"The -p options works, this is a permissions/repo issue."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I61dc3d56e4a4832a9ddd1904dd8af65c15a17e50"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAATH%2F%2FSEo%3D","line":15,"updated":"2013-10-18 13:30:52.000000000","message":"Is this supposed to say \"The -p option doesn\u0027t work yet; this is ...\" ?","commit_id":"3f3ebc9816209ad193a6f8d5bf1e6dc81c848ddf"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"6a520560c29f165264d12f11f5a34cfd4312c309","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Note that while this is the complete auto page generation of the"},{"line_number":14,"context_line":"devstack.org site it does not yet push the content back to GitHub."},{"line_number":15,"context_line":"The -p options works, this is a permissions/repo issue."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I61dc3d56e4a4832a9ddd1904dd8af65c15a17e50"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAATH%2F%2FRkg%3D","line":15,"in_reply_to":"AAAATH%2F%2FSEo%3D","updated":"2013-10-18 15:53:44.000000000","message":"The -p option works for me because I have push access to the cloudbuilders/devstack repo.  The real fix is to host devstack.org in something foundation controlled.  That should also get the static pages into the usual review process.","commit_id":"3f3ebc9816209ad193a6f8d5bf1e6dc81c848ddf"}],"tools/build_docs.sh":[{"author":{"_account_id":2394,"name":"Adam Spiers","email":"aspiers@suse.com","username":"adam.spiers"},"change_message_id":"5c22e703c2e9cb7a6690756f1ee4d1e4f7832499","unresolved":false,"context_lines":[{"line_number":98,"context_line":"echo \"$FILES\" \u003edocs-files"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"cd docs"},{"line_number":101,"context_line":"git add `find . -name \\*.html`"},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"# Push our changes back up to the docs branch"},{"line_number":104,"context_line":"if ! git diff-index HEAD --quiet \u0026\u0026 [[ -n $PUSH ]]; then"}],"source_content_type":"text/x-sh","patch_set":1,"id":"AAAATH%2F%2FSDk%3D","line":101,"updated":"2013-10-18 13:30:52.000000000","message":"Here personally I\u0027d prefer: find . -name \\*.html -print0 | xargs -0 git add\n\nbut it\u0027s probably not worth worrying about in this case :)","commit_id":"3f3ebc9816209ad193a6f8d5bf1e6dc81c848ddf"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"6a520560c29f165264d12f11f5a34cfd4312c309","unresolved":false,"context_lines":[{"line_number":98,"context_line":"echo \"$FILES\" \u003edocs-files"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"cd docs"},{"line_number":101,"context_line":"git add `find . -name \\*.html`"},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"# Push our changes back up to the docs branch"},{"line_number":104,"context_line":"if ! git diff-index HEAD --quiet \u0026\u0026 [[ -n $PUSH ]]; then"}],"source_content_type":"text/x-sh","patch_set":1,"id":"AAAATH%2F%2FRkA%3D","line":101,"in_reply_to":"AAAATH%2F%2FSDk%3D","updated":"2013-10-18 15:53:44.000000000","message":"While I don\u0027t think we\u0027ll hit the length limit in DevStack that is a better example to set for general use...","commit_id":"3f3ebc9816209ad193a6f8d5bf1e6dc81c848ddf"},{"author":{"_account_id":2394,"name":"Adam Spiers","email":"aspiers@suse.com","username":"adam.spiers"},"change_message_id":"d366f0663e127a2082c6214704beccf986453476","unresolved":false,"context_lines":[{"line_number":96,"context_line":""},{"line_number":97,"context_line":"# Build list of scripts to process"},{"line_number":98,"context_line":"FILES\u003d\"\""},{"line_number":99,"context_line":"for f in `find . -type f -name \\*.sh -not -path \\*shocco/\\*`; do"},{"line_number":100,"context_line":"    echo $f"},{"line_number":101,"context_line":"    FILES+\u003d\"$f \""},{"line_number":102,"context_line":"    mkdir -p docs/`dirname $f`;"}],"source_content_type":"text/x-sh","patch_set":2,"id":"AAAATH%2F%2FIm4%3D","line":99,"updated":"2013-10-22 17:28:25.000000000","message":"I just found a bug: this find(1) accidentally includes files within .git/\n\nIt works if you change it to:\n\nfor f in `find . -name .git -prune -o \\( -type f -name \\*.sh -not -path \\*shocco/\\* -print \\)`; do","commit_id":"4e94d380aeab982b16a6ebd523be734dab0e76da"}]}