)]}'
{"docker/base/Dockerfile.j2":[{"author":{"_account_id":13039,"name":"Martin André","email":"m.andre@redhat.com","username":"mandre"},"change_message_id":"ca6ba9031b6e965dd0464b2dab72a563768cc46f","unresolved":false,"context_lines":[{"line_number":43,"context_line":"{% block base_yum_conf %}"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"COPY yum.conf /etc/yum.conf"},{"line_number":46,"context_line":"{% endblock %}"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"{% if base_distro in [\u0027centos\u0027] %}"},{"line_number":49,"context_line":"{% set centos_contentdir \u003d \u0027centos\u0027 %}"}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"5f7c97a3_81e67d4a","line":46,"updated":"2018-06-04 08:54:56.000000000","message":"We could have the /etc/yum/vars/contentdir setup in the \u0027base_yum_conf\u0027 block, WDYT? This would retain the possibility to customize it. In other words, I\u0027m suggesting we move the line 46 to 57.","commit_id":"9723832192f81b040823dbe802ebd1c199abc269"},{"author":{"_account_id":1955,"name":"Alan Pevec","email":"alan.pevec@redhat.com","username":"apevec"},"change_message_id":"44746d15ebdcd38a0c24bf938d858ae013dd19aa","unresolved":false,"context_lines":[{"line_number":43,"context_line":"{% block base_yum_conf %}"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"COPY yum.conf /etc/yum.conf"},{"line_number":46,"context_line":"{% endblock %}"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"{% if base_distro in [\u0027centos\u0027] %}"},{"line_number":49,"context_line":"{% set centos_contentdir \u003d \u0027centos\u0027 %}"}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"5f7c97a3_c1a9d57c","line":46,"in_reply_to":"5f7c97a3_81e67d4a","updated":"2018-06-04 09:10:04.000000000","message":"What would you customize in contentdir, it can only be either \u0027centos\u0027 or \u0027altarch\u0027 ?","commit_id":"9723832192f81b040823dbe802ebd1c199abc269"},{"author":{"_account_id":13039,"name":"Martin André","email":"m.andre@redhat.com","username":"mandre"},"change_message_id":"dcb33eb189c5e3d5daa8fa4c05e81fcd5c44146c","unresolved":false,"context_lines":[{"line_number":43,"context_line":"{% block base_yum_conf %}"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"COPY yum.conf /etc/yum.conf"},{"line_number":46,"context_line":"{% endblock %}"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"{% if base_distro in [\u0027centos\u0027] %}"},{"line_number":49,"context_line":"{% set centos_contentdir \u003d \u0027centos\u0027 %}"}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"5f7c97a3_41e705ae","line":46,"in_reply_to":"5f7c97a3_c1a9d57c","updated":"2018-06-04 09:29:31.000000000","message":"I was thinking it could save us some trouble if the /etc/yum/vars/contentdir file ever evolves, but yeah like you said it\u0027s not really necessary today.","commit_id":"9723832192f81b040823dbe802ebd1c199abc269"}]}