timsaucer commented on code in PR #26119: URL: https://github.com/apache/datafusion/pull/26119#discussion_r4210337975
########## dev/release/README.md: ########## @@ -423,22 +442,25 @@ svn ls https://dist.apache.org/repos/dist/dev/datafusion To delete a release candidate: ```shell -svn delete -m "delete old DataFusion RC" https://dist.apache.org/repos/dist/dev/datafusion/apache-datafusion-50.0.0-rc1/ +svn delete -m "delete old DataFusion RC" https://dist.apache.org/repos/dist/dev/datafusion/apache-datafusion-55.2.0-rc1/ ``` #### Delete old releases from `release` SVN -Only the latest release should be available. Delete old releases after -publishing the new release. +Keep the last major release (`X.0.0`) and the releases after it. Delete older releases +after publishing the new release. For example, after publishing 55.2.0 keep `datafusion-55.0.0`, +`datafusion-55.1.0` and `datafusion-55.2.0`, and delete every 54.x release. To get a list of DataFusion releases: ```shell svn ls https://dist.apache.org/repos/dist/release/datafusion ``` -To delete a release: +To delete releases: ```shell -svn delete -m "delete old DataFusion release" https://dist.apache.org/repos/dist/release/datafusion/datafusion-50.0.0 +svn delete -m "delete old DataFusion release" \ + https://dist.apache.org/repos/dist/release/datafusion/datafusion-54.0.0 \ + https://dist.apache.org/repos/dist/release/datafusion/datafusion-54.1.0 Review Comment: The PR is changing the line ``` svn delete -m "delete old DataFusion release" https://dist.apache.org/repos/dist/release/datafusion/datafusion-50.0.0 ``` to ``` svn delete -m "delete old DataFusion release" \ https://dist.apache.org/repos/dist/release/datafusion/datafusion-54.0.0 \ https://dist.apache.org/repos/dist/release/datafusion/datafusion-54.1.0 ``` When I've run this command in the past it was helpful to copy & paste this line and edit the version number. I'm guessing when you went through this you needed to delete both 54.0.0 and 54.1.0? I was just suggesting this be ``` svn delete -m "delete old DataFusion release" https://dist.apache.org/repos/dist/release/datafusion/datafusion-54.1.0 ``` -- 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]
