Skip to content

maketgz: fix RELEASE-TOOLS.md for daily tarballs - #14820

Closed
dfandrich wants to merge 1 commit into
masterfrom
dfandrich/release-tools
Closed

dfandrich wants to merge 1 commit into
masterfrom
dfandrich/release-tools

Conversation

@dfandrich

Copy link
Copy Markdown
Contributor

The daily snapshots have no associated git tag, so provide a commit has
instead in these cases. Fix the dpkg detection since the shell would
exit immediately without showing an error message if it weren't found.
Also, use "which" instead of "command" since the latter is bash-specific
and this script specifies a plain Bourne shell.

Closes #14820

@github-actions github-actions Bot added the script label Sep 7, 2024
The daily snapshots have no associated git tag, so provide a commit has
instead in these cases. Fix the dpkg detection since the shell would
exit immediately without showing an error message if it weren't found.
Also, use "which" instead of "command" since the latter is bash-specific
and this script specifies a plain Bourne shell.

Closes #14820
@dfandrich
dfandrich force-pushed the dfandrich/release-tools branch from d8a23d7 to 70efbcb Compare September 7, 2024 02:23
@dfandrich

Copy link
Copy Markdown
Contributor Author

shellcheck tells me command is more portable than which, so I switched that back.

@dfandrich

Copy link
Copy Markdown
Contributor Author

Analysis of PR #14820 at 70efbcba:

Test 721 failed, which has NOT been flaky recently, so there could be a real issue in the PR.

Test 2055 failed, which has NOT been flaky recently, so there could be a real issue in the PR.

Generated by Testclutch

@dfandrich dfandrich closed this in c72dd0b Sep 7, 2024
@dfandrich
dfandrich deleted the dfandrich/release-tools branch September 7, 2024 16:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants