Skip to content

GH-46895: [CI][Dev] Fix shellcheck errors in the ci/scripts/install_minio.sh #46896

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 25, 2025

Conversation

hiroyuki-sato
Copy link
Collaborator

@hiroyuki-sato hiroyuki-sato commented Jun 24, 2025

Rationale for this change

This is the sub issue #44748.

  • SC2046: Quote this to prevent word splitting.
  • SC2086: Double quote to prevent globbing and word splitting.
shellcheck ci/scripts/install_minio.sh

In ci/scripts/install_minio.sh line 37:
if [ -z ${archs[$arch]} ]; then
        ^-------------^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
if [ -z "${archs[$arch]}" ]; then


In ci/scripts/install_minio.sh line 74:
  mkdir -p $(dirname ${output})
           ^------------------^ SC2046 (warning): Quote this to prevent word splitting.
                     ^-------^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
  mkdir -p $(dirname "${output}")


In ci/scripts/install_minio.sh line 76:
    wget -nv --output-document ${output} ${url}
                               ^-------^ SC2086 (info): Double quote to prevent globbing and word splitting.
                                         ^----^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
    wget -nv --output-document "${output}" "${url}"


In ci/scripts/install_minio.sh line 78:
    curl --fail --location --output ${output} ${url}
                                    ^-------^ SC2086 (info): Double quote to prevent globbing and word splitting.
                                              ^----^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
    curl --fail --location --output "${output}" "${url}"


In ci/scripts/install_minio.sh line 85:
  download ${prefix}/bin/minio ${url}
           ^-------^ SC2086 (info): Double quote to prevent globbing and word splitting.
                               ^----^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
  download "${prefix}"/bin/minio "${url}"


In ci/scripts/install_minio.sh line 86:
  chmod +x ${prefix}/bin/minio
           ^-------^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
  chmod +x "${prefix}"/bin/minio


In ci/scripts/install_minio.sh line 91:
  download ${prefix}/bin/mc ${url}
           ^-------^ SC2086 (info): Double quote to prevent globbing and word splitting.
                            ^----^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
  download "${prefix}"/bin/mc "${url}"


In ci/scripts/install_minio.sh line 92:
  chmod +x ${prefix}/bin/mc
           ^-------^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
  chmod +x "${prefix}"/bin/mc

For more information:
  https://www.shellcheck.net/wiki/SC2046 -- Quote this to prevent word splitt...
  https://www.shellcheck.net/wiki/SC2086 -- Double quote to prevent globbing ...

What changes are included in this PR?

  • Quote variables.

Are these changes tested?

Yes.

Are there any user-facing changes?

No.

Copy link

⚠️ GitHub issue #46895 has been automatically assigned in GitHub to PR creator.

@github-actions github-actions bot added the awaiting review Awaiting review label Jun 24, 2025
@hiroyuki-sato
Copy link
Collaborator Author

The cpp.yml is using install_minio.sh.
And CI of C++ related jobs passed.

Do we need additional tests?

ci/scripts/install_minio.sh latest ${ARROW_HOME}

Copy link
Member

@kou kou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@kou kou merged commit f5e868e into apache:main Jun 25, 2025
28 of 29 checks passed
@kou kou removed the awaiting review Awaiting review label Jun 25, 2025
@github-actions github-actions bot added the awaiting merge Awaiting merge label Jun 25, 2025
@hiroyuki-sato hiroyuki-sato deleted the topic/shellcheck-install_minio branch June 25, 2025 01:09
Copy link

After merging your PR, Conbench analyzed the 3 benchmarking runs that have been run so far on merge-commit f5e868e.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting merge Awaiting merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants