Skip to content
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

Install script should use something other than shasum #537

Open
TJM opened this issue Nov 4, 2021 · 1 comment
Open

Install script should use something other than shasum #537

TJM opened this issue Nov 4, 2021 · 1 comment
Labels
carvel accepted This issue should be considered for future work and that the triage process has been completed enhancement This issue is a feature request priority/unprioritized-backlog Higher priority than priority/awaiting-more-evidence but not planned. Contributions are welcome.

Comments

@TJM
Copy link

TJM commented Nov 4, 2021

Describe the problem/challenge you have
I was trying to create a docker container to use ytt during a ci/cd operation, and found that the install script wants 'shasum` to be installed, which is apparently part of perl-utils in alpine, which of course also requires perl.

/ # apk add shasum
fetch https://dl-cdn.alpinelinux.org/alpine/v3.14/main/x86_64/APKINDEX.tar.gz
fetch https://dl-cdn.alpinelinux.org/alpine/v3.14/community/x86_64/APKINDEX.tar.gz
ERROR: unable to select packages:
  shasum (no such package):
    required by: world[shasum]
/ # apk add perl-utils
(1/3) Installing libbz2 (1.0.8-r1)
(2/3) Installing perl (5.32.1-r0)
(3/3) Installing perl-utils (5.32.1-r0)
Executing busybox-1.33.1-r3.trigger
OK: 43 MiB in 17 packages

As it shows above, this results in 43MiB larger docker image, unnecessarily.

Describe the solution you'd like

I love that the install script is checking SHA sums, but maybe we could use something other than "shasum" executable?

Maybe sha256sum directly?

Anything else you would like to add:

I know there are other workarounds... just a suggestion/recommendation.


Vote on this request

This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.

👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"

We are also happy to receive and review Pull Requests if you want to help working on this issue.

@TJM TJM added carvel triage This issue has not yet been triaged for relevance enhancement This issue is a feature request labels Nov 4, 2021
@cppforlife
Copy link
Contributor

agreed @TJM. we need to improve this.

@cppforlife cppforlife added carvel accepted This issue should be considered for future work and that the triage process has been completed and removed carvel triage This issue has not yet been triaged for relevance labels Nov 4, 2021
@pivotaljohn pivotaljohn added the priority/unprioritized-backlog Higher priority than priority/awaiting-more-evidence but not planned. Contributions are welcome. label Nov 16, 2021
@aaronshurley aaronshurley moved this to To Triage in Carvel Aug 18, 2022
@pivotaljohn pivotaljohn moved this from To Triage to Unprioritized in Carvel Sep 27, 2022
@github-project-automation github-project-automation bot moved this to To Triage in Carvel Feb 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
carvel accepted This issue should be considered for future work and that the triage process has been completed enhancement This issue is a feature request priority/unprioritized-backlog Higher priority than priority/awaiting-more-evidence but not planned. Contributions are welcome.
Projects
Status: To Triage
Development

No branches or pull requests

3 participants