Subham-KRLX commented on PR #319:
URL: https://github.com/apache/iceberg-cpp/pull/319#issuecomment-3562654228
@wgtmac Thank you for the feedback I have addressed both of your comments
Documented all ICEBERG_*_URL environment variables at the top of
IcebergThirdpartyToolchain.cmake (L24-40)
Removed BUILD_CHINA.md
Updated README.md with generic guidance on customizing dependency URLs
The PR now only adds optional environment variables with documentation
keeping everything flexible without region-specific docs.
--
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]
