)]}'
{"nodepool/elements/cache-bindep/install.d/50-cache-pkgs":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"fff2e8e51c3ec5402873329ebaf9d7038ace7875","unresolved":false,"context_lines":[{"line_number":26,"context_line":"FALLBACKFILE\u003djenkins/data/bindep-fallback.txt"},{"line_number":27,"context_line":"PACKAGES\u003d`git --git-dir\u003d$FALLBACKREPO/.git show master:$FALLBACKFILE`"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"# NOTE(pabelanger): We currently don\u0027t gate on other-requirements.txt"},{"line_number":30,"context_line":"# for centos and fedora nodes. As such, we have no way to ensure this will"},{"line_number":31,"context_line":"# not break DIB process. So we only run this for ubuntu-trusty."},{"line_number":32,"context_line":"if [ -f /usr/bin/apt-get ] ; then"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"7af24918_b822c2ff","line":29,"updated":"2016-03-07 00:37:08.000000000","message":"so as mentioned in IRC, we probably handle missing packages better than apt-get does, since with \"-y\" i\u0027m pretty sure missing packages will not cause an error.  that assertion needs testing...\n\nbut it bears some thinking about, because it\u0027s hard to tell the difference between a really missing package and a typo.  it seems like there\u0027s room for discussion around this","commit_id":"7158aa01f864faa498b30e8cdd4a8d2053774c94"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"fff2e8e51c3ec5402873329ebaf9d7038ace7875","unresolved":false,"context_lines":[{"line_number":55,"context_line":"    # --ignore-missing conflicts with set -e, so force it to be ok"},{"line_number":56,"context_line":"    apt-get -y --ignore-missing -d install $PACKAGELIST || true"},{"line_number":57,"context_line":"else"},{"line_number":58,"context_line":"    ${YUM} install -y --downloadonly $PACKAGELIST"},{"line_number":59,"context_line":"fi"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"7af24918_f8456ab3","line":58,"updated":"2016-03-07 00:37:08.000000000","message":"for true idempotence (is that a word?) we usually use ${YUM:-yum} but i think this ok","commit_id":"7158aa01f864faa498b30e8cdd4a8d2053774c94"}]}