bneradt commented on PR #13889: URL: https://github.com/apache/trafficserver/pull/13889#issuecomment-6391991613
Addressed the latest review in 88b2f1a05c (migration commit 22c217958d): - Legacy ENABLE_AUTEST/ENABLE_AUTEST_UDS, AUTEST_SANDBOX, and AUTEST_OPTIONS now migrate once and are removed from the cache. Deprecated values override cached Uranium defaults for that migration; subsequent configuration uses the Uranium names. Nonempty legacy options produce a reminder to use pytest arguments. - Restored the autest, autest_no_install, and autest-uds build targets as aliases. - Added an actual ATS configure regression seeded with the old cache entries, checking successful repeat configuration and target discovery. This dependency-heavy test is opt-in through ATS_CMAKE_COMPATIBILITY_BUILD; it was executed locally. - Removed the unsupported ConnectingEntry coverage claim from both the PR description and migration commit message. Validation: 15 compatibility checks passed, including actual configure; framework suite 128 passed with the opt-in test skipped in that separate run. Formatting, incremental build, installation, and Sphinx docs passed. Thank you for testing the existing-build upgrade path. -- 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]
