)]}'
{"jenkins/scripts/common_translation_update.sh":[{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"c438fe4713db64cede835dd803d91bec06b40b96","unresolved":false,"context_lines":[{"line_number":236,"context_line":"        -f zanata.xml"},{"line_number":237,"context_line":"}"},{"line_number":238,"context_line":""},{"line_number":239,"context_line":"# Setup a tripleo-ui project for Zanata"},{"line_number":240,"context_line":"function setup_tripleo_ui {"},{"line_number":241,"context_line":"    local project\u003dtripleo_ui"},{"line_number":242,"context_line":"    local version\u003d${1:-master}"},{"line_number":243,"context_line":""},{"line_number":244,"context_line":"    # Extract messages"}],"source_content_type":"text/x-sh","patch_set":1,"id":"5a3905b3_f69c965f","line":241,"range":{"start_line":239,"start_character":10,"end_line":241,"end_character":28},"updated":"2017-01-24 18:25:42.000000000","message":"can we generalize that like setup_npm ?  I hope that other ReactOS projects can use the same approach, can\u0027t they?","commit_id":"e1962b8590429ec884481c9516082aff3c460899"},{"author":{"_account_id":4978,"name":"Julie Pichon","email":"jpichon@redhat.com","username":"jpichon"},"change_message_id":"a5dc5056c1971a0ba8557f22126bc49ced6de3ad","unresolved":false,"context_lines":[{"line_number":246,"context_line":"    # Transform them into .pot files"},{"line_number":247,"context_line":"    npm run json2pot"},{"line_number":248,"context_line":""},{"line_number":249,"context_line":"    /usr/local/jenkins/slave_scripts/create-zanata-xml.py \\"},{"line_number":250,"context_line":"        -p $project -v $version --srcdir . --txdir . \\"},{"line_number":251,"context_line":"        -r \u0027**/*.pot\u0027 -f zanata.xml"},{"line_number":252,"context_line":"}"},{"line_number":253,"context_line":""},{"line_number":254,"context_line":"# Setup project so that git review works, sets global variable"}],"source_content_type":"text/x-sh","patch_set":2,"id":"3a461143_679fd450","line":251,"range":{"start_line":249,"start_character":0,"end_line":251,"end_character":35},"updated":"2017-01-26 15:55:41.000000000","message":"Found a number of issues with this, working on a new patchset.","commit_id":"88ff3416098547cf32f7e29e536ab1052d30a21e"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"fb26a47924e2b6e77fe26f3f33ca9ff6e96a820e","unresolved":false,"context_lines":[{"line_number":112,"context_line":"    $VENV/bin/pip install -U os-testr"},{"line_number":113,"context_line":"}"},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"# Setup nodejs within a python venv. Match version with the NodeJS"},{"line_number":116,"context_line":"# version used in the nodejs6-npm gate."},{"line_number":117,"context_line":"function setup_nodeenv {"},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"    VENV\u003d$(mktemp -d)"},{"line_number":120,"context_line":"    virtualenv $VENV"},{"line_number":121,"context_line":"    $VENV/bin/pip install -U nodeenv"},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"    NODE_VENV\u003d$VENV/node_venv"},{"line_number":124,"context_line":"    $VENV/bin/nodeenv --node 6.9.4 $NODE_VENV"},{"line_number":125,"context_line":"    source $NODE_VENV/bin/activate"},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"}"},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"# Setup a project for Zanata. This is used by both Python and Django projects."},{"line_number":130,"context_line":"# syntax: setup_project \u003cproject\u003e \u003czanata_version\u003e \u003cmodulename\u003e [\u003cmodulename\u003e ...]"}],"source_content_type":"text/x-sh","patch_set":5,"id":"3a461143_f4cdcc31","line":127,"range":{"start_line":115,"start_character":0,"end_line":127,"end_character":1},"updated":"2017-01-30 11:47:11.000000000","message":"Could you install this in the existing venv done by setup_venv?\n\nWe need os-testr installed as well (for health reporting) - and one environment should be enough.\n\nNOte that setup_venv is called at the beginning of our scripts, so no need to recreate one here.","commit_id":"520797b511a7d0a91a6995bcd7bdfaf6bef72df1"},{"author":{"_account_id":4978,"name":"Julie Pichon","email":"jpichon@redhat.com","username":"jpichon"},"change_message_id":"3a12d781b1540040cdfd4ab9d6b063e22fa1c9ae","unresolved":false,"context_lines":[{"line_number":112,"context_line":"    $VENV/bin/pip install -U os-testr"},{"line_number":113,"context_line":"}"},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"# Setup nodejs within a python venv. Match version with the NodeJS"},{"line_number":116,"context_line":"# version used in the nodejs6-npm gate."},{"line_number":117,"context_line":"function setup_nodeenv {"},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"    VENV\u003d$(mktemp -d)"},{"line_number":120,"context_line":"    virtualenv $VENV"},{"line_number":121,"context_line":"    $VENV/bin/pip install -U nodeenv"},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"    NODE_VENV\u003d$VENV/node_venv"},{"line_number":124,"context_line":"    $VENV/bin/nodeenv --node 6.9.4 $NODE_VENV"},{"line_number":125,"context_line":"    source $NODE_VENV/bin/activate"},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"}"},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"# Setup a project for Zanata. This is used by both Python and Django projects."},{"line_number":130,"context_line":"# syntax: setup_project \u003cproject\u003e \u003czanata_version\u003e \u003cmodulename\u003e [\u003cmodulename\u003e ...]"}],"source_content_type":"text/x-sh","patch_set":5,"id":"3a461143_cfd11d59","line":127,"range":{"start_line":115,"start_character":0,"end_line":127,"end_character":1},"in_reply_to":"3a461143_f4cdcc31","updated":"2017-01-30 12:05:49.000000000","message":"I\u0027m only noticing now that $VENV appears to be a global variable referenced directly in other functions as well, my bad for not noticing earlier. I\u0027ll update the patch.","commit_id":"520797b511a7d0a91a6995bcd7bdfaf6bef72df1"}],"jenkins/scripts/propose_translation_update.sh":[{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"c438fe4713db64cede835dd803d91bec06b40b96","unresolved":false,"context_lines":[{"line_number":210,"context_line":"}"},{"line_number":211,"context_line":""},{"line_number":212,"context_line":""},{"line_number":213,"context_line":"function propose_tripleo_ui {"},{"line_number":214,"context_line":"    pull_from_zanata tripleo-ui"},{"line_number":215,"context_line":""},{"line_number":216,"context_line":"    # Update .pot files"},{"line_number":217,"context_line":"    npm run build"}],"source_content_type":"text/x-sh","patch_set":1,"id":"5a3905b3_f6ead6ab","line":214,"range":{"start_line":213,"start_character":16,"end_line":214,"end_character":31},"updated":"2017-01-24 18:25:42.000000000","message":"can we generalize this for npm?","commit_id":"e1962b8590429ec884481c9516082aff3c460899"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"c438fe4713db64cede835dd803d91bec06b40b96","unresolved":false,"context_lines":[{"line_number":250,"context_line":"        setup_training_guides \"$ZANATA_VERSION\""},{"line_number":251,"context_line":"        propose_training_guides"},{"line_number":252,"context_line":"        ;;"},{"line_number":253,"context_line":"    tripleo-ui)"},{"line_number":254,"context_line":"        setup_tripleo_ui \"$ZANATA_VERSION\""},{"line_number":255,"context_line":"        propose_tripleo_ui"},{"line_number":256,"context_line":"        ;;"}],"source_content_type":"text/x-sh","patch_set":1,"id":"5a3905b3_56f1ea94","line":253,"range":{"start_line":253,"start_character":3,"end_line":253,"end_character":15},"updated":"2017-01-24 18:25:42.000000000","message":"this case is fine - but what if more projects want to go this route?","commit_id":"e1962b8590429ec884481c9516082aff3c460899"},{"author":{"_account_id":4978,"name":"Julie Pichon","email":"jpichon@redhat.com","username":"jpichon"},"change_message_id":"7a23730f7d08ab7c876c71735539359ff4265dcd","unresolved":false,"context_lines":[{"line_number":213,"context_line":"function propose_reactjs {"},{"line_number":214,"context_line":"    pull_from_zanata \"$PROJECT\""},{"line_number":215,"context_line":""},{"line_number":216,"context_line":"    # Update .pot files"},{"line_number":217,"context_line":"    npm run build"},{"line_number":218,"context_line":"    npm run json2pot"},{"line_number":219,"context_line":""},{"line_number":220,"context_line":"    # Clean up files (removes incomplete translations and untranslated strings)"},{"line_number":221,"context_line":"    cleanup_module \"i18n\""}],"source_content_type":"text/x-sh","patch_set":3,"id":"3a461143_375796cf","line":218,"range":{"start_line":216,"start_character":0,"end_line":218,"end_character":20},"updated":"2017-01-27 10:21:54.000000000","message":"This is done as part of setup_reactjs_project just before so I will remove it.","commit_id":"9c7b416934d6353420b941fbdf8c58907ae62e0b"},{"author":{"_account_id":4978,"name":"Julie Pichon","email":"jpichon@redhat.com","username":"jpichon"},"change_message_id":"7a23730f7d08ab7c876c71735539359ff4265dcd","unresolved":false,"context_lines":[{"line_number":221,"context_line":"    cleanup_module \"i18n\""},{"line_number":222,"context_line":""},{"line_number":223,"context_line":"    # Convert po files to ReactJS i18n JSON format"},{"line_number":224,"context_line":"    for lang in `ls *.po | sed \u0027s/\\.po$//\u0027`; do"},{"line_number":225,"context_line":"        npm run po2json -- ./i18n/$lang.po -o ./i18n/$lang.json"},{"line_number":226,"context_line":"        # The files are created as a one-line JSON file - expand them"},{"line_number":227,"context_line":"        python -m json.tool ./i18n/$lang.json ./i18n/locales/$lang.json"}],"source_content_type":"text/x-sh","patch_set":3,"id":"3a461143_d75c92ef","line":224,"range":{"start_line":224,"start_character":17,"end_line":224,"end_character":24},"updated":"2017-01-27 10:21:54.000000000","message":"I forgot to update that bit now that files are downloaded under ./i18n","commit_id":"9c7b416934d6353420b941fbdf8c58907ae62e0b"}],"jenkins/scripts/upstream_translation_update.sh":[{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"c438fe4713db64cede835dd803d91bec06b40b96","unresolved":false,"context_lines":[{"line_number":71,"context_line":"    tripleo-ui)"},{"line_number":72,"context_line":"        setup_tripleo_ui \"$ZANATA_VERSION\""},{"line_number":73,"context_line":"        # The pot file is generated in the top directory"},{"line_number":74,"context_line":"        ALL_MODULES\u003d\".\""},{"line_number":75,"context_line":"        ;;"},{"line_number":76,"context_line":"    *)"},{"line_number":77,"context_line":"        # Common setup for python and django repositories"}],"source_content_type":"text/x-sh","patch_set":1,"id":"5a3905b3_d66b1af0","line":74,"range":{"start_line":74,"start_character":8,"end_line":74,"end_character":23},"updated":"2017-01-24 18:25:42.000000000","message":"we call later copy_pot ALL_MODULES - I don\u0027t think that that step will work with \".\" as argument. It would otherwise decent into \".tox\" as well and that\u0027s what we have to avoid.\n\nI think you want \"i18n\" here as directory.\n\nPlease test this locally.","commit_id":"e1962b8590429ec884481c9516082aff3c460899"}]}