Skip to content

Commit d9618fe

Browse files
authored
MINOR: [Release] Make post-08-remove-old-artifacts.sh crossplatform (apache#47250)
### Rationale for this change `dev/release/post-08-remove-old-artifacts.sh` uses [`tac`](https://man7.org/linux/man-pages/man1/tac.1.html) which may not be available on non-Linuxes. We should be able to get its same behavior with more common utilities. ### What changes are included in this PR? Swaps the call to `tac` with `sort --version-sort` which should be equivalent. ### Are these changes tested? Yes. This shouldn't impact which artifacts are removed, just the order in which they are. ### Are there any user-facing changes? No. Authored-by: Bryce Mecum <[email protected]> Signed-off-by: Bryce Mecum <[email protected]>
1 parent 22676fe commit d9618fe

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

dev/release/post-08-remove-old-artifacts.sh

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -42,8 +42,7 @@ old_releases=$(
4242
svn ls ${release_base_url}/ | \
4343
grep -E '^arrow-[0-9]' | \
4444
sort --version-sort --reverse | \
45-
tail -n +2 | \
46-
tac
45+
tail -n +2
4746
)
4847
for old_release_version in $old_releases; do
4948
echo "Remove old release: ${old_release_version}"

0 commit comments

Comments
 (0)