HoustonPutman commented on a change in pull request #773: URL: https://github.com/apache/solr/pull/773#discussion_r837913220
########## File path: solr/docker/templates/Dockerfile.official.header.template ########## @@ -34,13 +34,13 @@ ARG SOLR_DOWNLOAD_URL # TODO: see patch in SOLR-15250 for some example ideas on fixing this to be more strict # Override the default solr download location with a prefered mirror, e.g.: -# docker build -t mine --build-arg SOLR_DOWNLOAD_SERVER=http://www-eu.apache.org/dist/lucene/solr . +# docker build -t mine --build-arg SOLR_DOWNLOAD_SERVER=http://www-eu.apache.org/dist/solr/solr . Review comment: Ok cool. Yeah it looked like the structure was already going to be `.../solr/solr/x.y.z.`, which is fine with me, especially with the solr-operator stuff there (and other sub-projects in the future maybe?) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
