)]}'
{"docker/base/Dockerfile.j2":[{"author":{"_account_id":13039,"name":"Martin André","email":"m.andre@redhat.com","username":"mandre"},"change_message_id":"9292f52971127c271b5e750d38e5c866ac7100fd","unresolved":false,"context_lines":[{"line_number":42,"context_line":""},{"line_number":43,"context_line":"{% set centos_contentdir \u003d \u0027centos\u0027 %}"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"{% if base_arch in [\u0027aarch64\u0027, \u0027ppc64le\u0027] %}"},{"line_number":46,"context_line":"    {% set centos_contentdir \u003d \u0027altarch\u0027 %}"},{"line_number":47,"context_line":"{% endif %}"},{"line_number":48,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"5f7c97a3_3972907c","line":45,"updated":"2018-05-14 07:57:58.000000000","message":"I would have preferred the check to be more future proof, something like:\n\n{% if base_arch \u003d\u003d x86_64 %}\n    {% set centos_contentdir \u003d \u0027centos\u0027 %}\n{% else %}\n    {% set centos_contentdir \u003d \u0027altarch\u0027 %}\n{% endif %}\n\nbut I see you already gathered the necessary votes so not a big deal :)","commit_id":"ec5daaaa634351d24706e605c2c6726be402d35b"},{"author":{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},"change_message_id":"9aa83144290530e8c653dcc5f9f22f25f3b760fe","unresolved":false,"context_lines":[{"line_number":42,"context_line":""},{"line_number":43,"context_line":"{% set centos_contentdir \u003d \u0027centos\u0027 %}"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"{% if base_arch in [\u0027aarch64\u0027, \u0027ppc64le\u0027] %}"},{"line_number":46,"context_line":"    {% set centos_contentdir \u003d \u0027altarch\u0027 %}"},{"line_number":47,"context_line":"{% endif %}"},{"line_number":48,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"5f7c97a3_59966c40","line":45,"in_reply_to":"5f7c97a3_3972907c","updated":"2018-05-14 08:06:20.000000000","message":"When someone with s/390 appears then this block will be one of their smallest problem ;D","commit_id":"ec5daaaa634351d24706e605c2c6726be402d35b"},{"author":{"_account_id":13039,"name":"Martin André","email":"m.andre@redhat.com","username":"mandre"},"change_message_id":"106e738b3bb5530c6a83bb9031e100aacded86ec","unresolved":false,"context_lines":[{"line_number":42,"context_line":""},{"line_number":43,"context_line":"{% set centos_contentdir \u003d \u0027centos\u0027 %}"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"{% if base_arch in [\u0027aarch64\u0027, \u0027ppc64le\u0027] %}"},{"line_number":46,"context_line":"    {% set centos_contentdir \u003d \u0027altarch\u0027 %}"},{"line_number":47,"context_line":"{% endif %}"},{"line_number":48,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"5f7c97a3_b4d9c757","line":45,"in_reply_to":"5f7c97a3_59966c40","updated":"2018-05-14 08:52:03.000000000","message":"Haha, true -- that\u0027s why it\u0027s important to show solidarity and ease their pain where we can :)","commit_id":"ec5daaaa634351d24706e605c2c6726be402d35b"}]}