Skip to content

Commit

Permalink
build, tools: drop leading / from r2dir
Browse files Browse the repository at this point in the history
The script is commented as removing `/home/dist/` part but the `cut`
command is off by one and end up including the `/` character (so that
the resulting string starts with `/`). When this is substituted into
`s3://${cloudflare_bucket}/${r2dir}/${shafile}.asc` we end up with
`//` (one from the template and one from the `r2dir`) which appears
to cause Cloudflare to create an extra top level `/` directory in the
bucket.

PR-URL: #53951
Reviewed-By: Rafael Gonzaga <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: James M Snell <[email protected]>
  • Loading branch information
richardlau authored and marco-ippolito committed Aug 19, 2024
1 parent a38e82e commit 7620e5e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion tools/release.sh
Original file line number Diff line number Diff line change
Expand Up @@ -158,7 +158,7 @@ sign() {
# since the promotion script should take care of uploading them.

# Remove /home/dist/ part
r2dir=$(echo "$shadir" | cut -c 11-)
r2dir=$(echo "$shadir" | cut -c 12-)

# Copy SHASUMS256.txt.asc
# shellcheck disable=SC2086,SC2029
Expand Down

0 comments on commit 7620e5e

Please sign in to comment.