)]}'
{"doc/source/configuration.rst":[{"author":{"_account_id":4220,"name":"patrick-crews","email":"gleebix@gmail.com","username":"patrick-crews"},"change_message_id":"ed55548ebc7ad26914312d272b20b14115065580","unresolved":false,"context_lines":[{"line_number":213,"context_line":"    ::"},{"line_number":214,"context_line":""},{"line_number":215,"context_line":"        LOGDIR\u003d$DEST/logs"},{"line_number":216,"context_line":""},{"line_number":217,"context_line":"    *Note the use of ``DEST`` to locate the main install directory; this"},{"line_number":218,"context_line":"    is why we suggest setting it in ``local.conf``.*"},{"line_number":219,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"1a930d6b_ced11acf","line":216,"updated":"2015-01-21 20:26:04.000000000","message":"I feel there should be something here describing behavior when $SCREEN_LOGDIR is set and perhaps some \u0027why\u0027 this might be set (infra purposes, etc).\n\nThis, of course, depends on just how SCREEN_LOGDIR will live (if it will purged from infra configs, etc in a short time after this lands).\n\nOtherwise, would be helpful for those who are delving into the wonderful world of infra.","commit_id":"dde41d0797f0b085099cd5c9ff1e0110d5ae3fbd"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"9f797b512914c52bed8413e9f8cc4e6078ea051a","unresolved":false,"context_lines":[{"line_number":213,"context_line":"    ::"},{"line_number":214,"context_line":""},{"line_number":215,"context_line":"        LOGDIR\u003d$DEST/logs"},{"line_number":216,"context_line":""},{"line_number":217,"context_line":"    *Note the use of ``DEST`` to locate the main install directory; this"},{"line_number":218,"context_line":"    is why we suggest setting it in ``local.conf``.*"},{"line_number":219,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"1a930d6b_f695c0e4","line":216,"in_reply_to":"1a930d6b_ced11acf","updated":"2015-01-22 06:15:47.000000000","message":"This is the second in a series of reviews doing this migration...d-g comes next to transition away from using SCREEN_LOGDIR.\n\nI purposely removed SCREEN_LOGDIR from the docs to indicate that it should not be used.  Its current usage should not change, if it does I did something wrong.","commit_id":"dde41d0797f0b085099cd5c9ff1e0110d5ae3fbd"}],"functions-common":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"1c02f892b250df46fbe7b4de43ac9750f5fae98e","unresolved":false,"context_lines":[{"line_number":1230,"context_line":"    exec 6\u003e\u0026-"},{"line_number":1231,"context_line":""},{"line_number":1232,"context_line":"    local real_logfile\u003d\"${LOGDIR}/${service}.log.${CURRENT_LOG_TIME}\""},{"line_number":1233,"context_line":"    echo \"LOGDIR: $LOGDIR\""},{"line_number":1234,"context_line":"    echo \"SCREEN_LOGDIR: $SCREEN_LOGDIR\""},{"line_number":1235,"context_line":"    echo \"log: $real_logfile\""},{"line_number":1236,"context_line":"    if [[ -n ${LOGDIR} ]]; then"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"3a961159_8d503d0c","line":1233,"updated":"2014-12-14 22:39:15.000000000","message":"i don\u0027t think we need this output for every _run_process call?","commit_id":"59d9556b034e01fdebfae156682ac774635931f0"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"8017c7960c9763bc34ed524fbaa432e30bfc1fd3","unresolved":false,"context_lines":[{"line_number":1230,"context_line":"    exec 6\u003e\u0026-"},{"line_number":1231,"context_line":""},{"line_number":1232,"context_line":"    local real_logfile\u003d\"${LOGDIR}/${service}.log.${CURRENT_LOG_TIME}\""},{"line_number":1233,"context_line":"    echo \"LOGDIR: $LOGDIR\""},{"line_number":1234,"context_line":"    echo \"SCREEN_LOGDIR: $SCREEN_LOGDIR\""},{"line_number":1235,"context_line":"    echo \"log: $real_logfile\""},{"line_number":1236,"context_line":"    if [[ -n ${LOGDIR} ]]; then"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"3a961159_3578528f","line":1233,"in_reply_to":"3a961159_8d503d0c","updated":"2014-12-22 20:18:47.000000000","message":"correct, it needs to come out after I\u0027m done testing things...","commit_id":"59d9556b034e01fdebfae156682ac774635931f0"}]}