)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"152f3dee0f01a37f1d040db0ad54a961b0272e2b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"24a6afd7_23665629","updated":"2022-04-12 00:08:01.000000000","message":"since it\u0027s in a separate \"run\" I think in terms of layers it\u0027s the same\n\nbut not having to install wget seems like a win, in terms of keeping the container slim to just what it needs.","commit_id":"1f0948be99aa52a0c01a2da07449105792e2f96b"}],"docker/gerrit/base/Dockerfile":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"35bef6cf475f74509e25ab6b8433cdf7ab83bc6d","unresolved":true,"context_lines":[{"line_number":60,"context_line":"# Download mariadb java client."},{"line_number":61,"context_line":"# Modern gerrit stopped downloading missing libs during init which means we"},{"line_number":62,"context_line":"# need to do the downland and install ourselves."},{"line_number":63,"context_line":"RUN mkdir /var/gerrit/lib \u0026\u0026 \\"},{"line_number":64,"context_line":"  wget https://repo1.maven.org/maven2/org/mariadb/jdbc/mariadb-java-client/2.7.2/mariadb-java-client-2.7.2.jar -O /var/gerrit/lib/mariadb-java-client.jar"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"# Allow incoming traffic"}],"source_content_type":"text/x-dockerfile","patch_set":3,"id":"e8dcc99d_46094546","side":"PARENT","line":63,"updated":"2022-05-04 06:06:47.000000000","message":"The one thing I\u0027m not sure from looking at this is whether the mkdir will be done implicitly or whether we might need to keep this, possibly added to the block above? Might be safer to be more explicit?","commit_id":"0525c5d8965b7add7c3409cfbdd72694726162a2"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"28accdde8a2cdf96e4b1505c8c2f86e7dd9d0e93","unresolved":false,"context_lines":[{"line_number":60,"context_line":"# Download mariadb java client."},{"line_number":61,"context_line":"# Modern gerrit stopped downloading missing libs during init which means we"},{"line_number":62,"context_line":"# need to do the downland and install ourselves."},{"line_number":63,"context_line":"RUN mkdir /var/gerrit/lib \u0026\u0026 \\"},{"line_number":64,"context_line":"  wget https://repo1.maven.org/maven2/org/mariadb/jdbc/mariadb-java-client/2.7.2/mariadb-java-client-2.7.2.jar -O /var/gerrit/lib/mariadb-java-client.jar"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"# Allow incoming traffic"}],"source_content_type":"text/x-dockerfile","patch_set":3,"id":"4d687725_c891fe3f","side":"PARENT","line":63,"in_reply_to":"a58c1b52_71e7fae0","updated":"2022-05-10 05:19:07.000000000","message":"Ack","commit_id":"0525c5d8965b7add7c3409cfbdd72694726162a2"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"3676f0caeeb7a3896162a101b3ecdb325fec7284","unresolved":true,"context_lines":[{"line_number":60,"context_line":"# Download mariadb java client."},{"line_number":61,"context_line":"# Modern gerrit stopped downloading missing libs during init which means we"},{"line_number":62,"context_line":"# need to do the downland and install ourselves."},{"line_number":63,"context_line":"RUN mkdir /var/gerrit/lib \u0026\u0026 \\"},{"line_number":64,"context_line":"  wget https://repo1.maven.org/maven2/org/mariadb/jdbc/mariadb-java-client/2.7.2/mariadb-java-client-2.7.2.jar -O /var/gerrit/lib/mariadb-java-client.jar"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"# Allow incoming traffic"}],"source_content_type":"text/x-dockerfile","patch_set":3,"id":"a58c1b52_71e7fae0","side":"PARENT","line":63,"in_reply_to":"e8dcc99d_46094546","updated":"2022-05-09 22:34:49.000000000","message":"https://docs.docker.com/engine/reference/builder/#add says \"If \u003cdest\u003e doesn’t exist, it is created along with all missing directories in its path.\"","commit_id":"0525c5d8965b7add7c3409cfbdd72694726162a2"}]}