damccorm commented on PR #35962: URL: https://github.com/apache/beam/pull/35962#issuecomment-3761713843
IIRC, the old test relied on distutils which was removed in Python 3.12, so I think the example needed to change, and that's why we were seeing failures. In retrospect, I think the way we did it/I recommended was a mistake. Originally, I saw we were just using the setup file to install dependencies, which was an odd usage, but I now see that we're explicit about this being to demonstrate a custom command. I also didn't realize it was our only setup_file coverage. I think we should mostly back out this change and update to just not use distutils instead -- 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]
