)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"f89a8aac33a00ad7fea5acf7eef3a6f9cd21cb72","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Don\u0027t reinstall package just after installing."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Boolean logic used in istall_package was wrong: on nominal flow packages where actually installed twice. This should fix it."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: Ia465414936b272d04523a11f83d6ded378fe1daf"},{"line_number":12,"context_line":"Closes-Bug: #1518544"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"fa69d971_e95e63e8","line":9,"updated":"2016-01-03 14:12:37.000000000","message":"\"install_package\"\n\ns/where/were/\n\nAlso line breaks needed.","commit_id":"f4bbda26c17435305a6c1bad9aeb2fbeed4d2799"}],"functions-common":[{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"f89a8aac33a00ad7fea5acf7eef3a6f9cd21cb72","unresolved":false,"context_lines":[{"line_number":1215,"context_line":"# install_package package [package ...]"},{"line_number":1216,"context_line":"function install_package {"},{"line_number":1217,"context_line":"    update_package_repo"},{"line_number":1218,"context_line":"    real_install_package $@ || RETRY_UPDATE\u003dTrue update_package_repo \u0026\u0026 real_install_package $@"},{"line_number":1219,"context_line":"}"},{"line_number":1220,"context_line":""},{"line_number":1221,"context_line":"# Distro-agnostic function to tell if a package is installed"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"fa69d971_0962d7ad","side":"PARENT","line":1218,"updated":"2016-01-03 14:12:37.000000000","message":"I\u0027d usually use ( ) to group the boolean instead of using an if statement. Not sure what the preferred style is here though.","commit_id":"e8788c0129e0634bbefe0382ad82b70a38f38bc6"},{"author":{"_account_id":17377,"name":"Federico Ressi","email":"federico.ressi@intel.com","username":"fressi"},"change_message_id":"72468895a8c38baf3f78404227d2ff28678949b4","unresolved":false,"context_lines":[{"line_number":1215,"context_line":"# install_package package [package ...]"},{"line_number":1216,"context_line":"function install_package {"},{"line_number":1217,"context_line":"    update_package_repo"},{"line_number":1218,"context_line":"    real_install_package $@ || RETRY_UPDATE\u003dTrue update_package_repo \u0026\u0026 real_install_package $@"},{"line_number":1219,"context_line":"}"},{"line_number":1220,"context_line":""},{"line_number":1221,"context_line":"# Distro-agnostic function to tell if a package is installed"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"fa69d971_d44d3673","side":"PARENT","line":1218,"in_reply_to":"fa69d971_0962d7ad","updated":"2016-01-03 15:55:48.000000000","message":"I used \u0027if\u0027 to split a long line in two. I don\u0027t know which is the preferred style for this project.","commit_id":"e8788c0129e0634bbefe0382ad82b70a38f38bc6"}]}