)]}'
{"stack.sh":[{"author":{"_account_id":112,"name":"Jesse Andrews","email":"anotherjesse@gmail.com","username":"anotherjesse"},"change_message_id":"279b9b729973f4d434c0874a64071e331002527a","unresolved":false,"context_lines":[{"line_number":466,"context_line":""},{"line_number":467,"context_line":"if [[ \"$OFFLINE\" \u003d \"False\" ]]; then"},{"line_number":468,"context_line":"    # install apt requirements"},{"line_number":469,"context_line":"    apt_get update"},{"line_number":470,"context_line":"    apt_get install $(get_packages)"},{"line_number":471,"context_line":""},{"line_number":472,"context_line":"    # install python requirements"}],"source_content_type":"text/x-sh","patch_set":2,"id":"AAAAC3%2F%2F7lY%3D","line":469,"updated":"2011-12-20 21:16:18.000000000","message":"perhaps update the apt_get method to check and return if offline?","commit_id":"39d132f0bb267f9f97fc2d5e12d22e7df7f30ccc"},{"author":{"_account_id":112,"name":"Jesse Andrews","email":"anotherjesse@gmail.com","username":"anotherjesse"},"change_message_id":"279b9b729973f4d434c0874a64071e331002527a","unresolved":false,"context_lines":[{"line_number":470,"context_line":"    apt_get install $(get_packages)"},{"line_number":471,"context_line":""},{"line_number":472,"context_line":"    # install python requirements"},{"line_number":473,"context_line":"    sudo PIP_DOWNLOAD_CACHE\u003d/var/cache/pip pip install --use-mirrors `cat $FILES/pips/*`"},{"line_number":474,"context_line":"fi"},{"line_number":475,"context_line":""},{"line_number":476,"context_line":"# git clone only if directory doesn\u0027t exist already.  Since ``DEST`` might not"}],"source_content_type":"text/x-sh","patch_set":2,"id":"AAAAC3%2F%2F7lU%3D","line":473,"updated":"2011-12-20 21:16:18.000000000","message":"perhaps a pip_install that abstracts this and does the return if online - since eventually pip install shouldn\u0027t do everything - only what is needed for the enabled services","commit_id":"39d132f0bb267f9f97fc2d5e12d22e7df7f30ccc"},{"author":{"_account_id":112,"name":"Jesse Andrews","email":"anotherjesse@gmail.com","username":"anotherjesse"},"change_message_id":"279b9b729973f4d434c0874a64071e331002527a","unresolved":false,"context_lines":[{"line_number":1322,"context_line":"    for image_url in ${IMAGE_URLS//,/ }; do"},{"line_number":1323,"context_line":"        # Downloads the image (uec ami+aki style), then extracts it."},{"line_number":1324,"context_line":"        IMAGE_FNAME\u003d`basename \"$image_url\"`"},{"line_number":1325,"context_line":"        if [ ! -f $FILES/$IMAGE_FNAME -a \"$OFFLINE\" \u003d \"False\" ]; then"},{"line_number":1326,"context_line":"            wget -c $image_url -O $FILES/$IMAGE_FNAME"},{"line_number":1327,"context_line":"        fi"},{"line_number":1328,"context_line":""}],"source_content_type":"text/x-sh","patch_set":2,"id":"AAAAC3%2F%2F7lk%3D","line":1325,"updated":"2011-12-20 21:16:18.000000000","message":"might not need this check if the image already has been downloaded?","commit_id":"39d132f0bb267f9f97fc2d5e12d22e7df7f30ccc"},{"author":{"_account_id":1155,"name":"sleepsonthefloor","username":"sleepsonthefloor"},"change_message_id":"f3c8c1ff0e96bc13d771f19598e0e88b42db788a","unresolved":false,"context_lines":[{"line_number":160,"context_line":"    echo \"$default\""},{"line_number":161,"context_line":"}"},{"line_number":162,"context_line":""},{"line_number":163,"context_line":"# Configure stach.sh behaviour"},{"line_number":164,"context_line":"OFFLINE\u003d`trueorfalse False $OFFLINE`"},{"line_number":165,"context_line":""},{"line_number":166,"context_line":"# Set the destination directories for openstack projects"}],"source_content_type":"text/x-sh","patch_set":3,"id":"AAAAC3%2F%2F7NU%3D","line":163,"updated":"2011-12-21 23:07:12.000000000","message":"sp of stack.sh.  Also, should improve this comment to better describe the intent/usage of the var","commit_id":"96eb592ee57a1c6d86d7d499e677d7c3494c59af"},{"author":{"_account_id":1155,"name":"sleepsonthefloor","username":"sleepsonthefloor"},"change_message_id":"f3c8c1ff0e96bc13d771f19598e0e88b42db788a","unresolved":false,"context_lines":[{"line_number":466,"context_line":""},{"line_number":467,"context_line":"function pip_install {"},{"line_number":468,"context_line":"    [[ \"$OFFLINE\" \u003d \"True\" ]] \u0026\u0026 return"},{"line_number":469,"context_line":"    sudo PIP_DOWNLOAD_CACHE\u003d/var/cache/pip pip install --use-mirrors $@"},{"line_number":470,"context_line":"}"},{"line_number":471,"context_line":""},{"line_number":472,"context_line":"# install apt requirements"}],"source_content_type":"text/x-sh","patch_set":3,"id":"AAAAC3%2F%2F7NA%3D","line":469,"updated":"2011-12-21 23:07:12.000000000","message":"probably outside the scope of this request, but should consider using uniq, and only installing pips for ENABLED_SERVICES","commit_id":"96eb592ee57a1c6d86d7d499e677d7c3494c59af"}]}