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.

Sign up for GitHub

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

Jump to bottom

Unpin skopeo during mac acceptance tests #176

Merged
merged 1 commit into from
Sep 18, 2020
Merged

Unpin skopeo during mac acceptance tests #176

merged 1 commit into from
Sep 18, 2020

Conversation

wagoodman
Copy link
Contributor

Fixes acceptance test failures by unpinning skopeo:

./test/acceptance/mac.sh: line 33: skopeo: command not found
+ brew install https://raw.githubusercontent.com/Homebrew/homebrew-core/75e8d7a40af77b48cc91f4bdb7d669f891a6de60/Formula/skopeo.rb
Error: Calling Installation of skopeo from a GitHub commit URL is disabled! Use 'brew extract skopeo' to stable tap on GitHub instead.

From https://github.com/anchore/syft/runs/1127902909?check_suite_focus=true

@wagoodman
unpin skopeo during mac acceptance tests
635aaed 
Signed-off-by: Alex Goodman <alex.goodman@anchore.com>
@wagoodman wagoodman added the bug Something isn't working label Sep 18, 2020
@wagoodman wagoodman merged commit 45b5cab into main Sep 18, 2020
@wagoodman wagoodman deleted the unpin-skopeo branch September 18, 2020 17:54
GijsCalis pushed a commit to GijsCalis/syft that referenced this pull request Feb 19, 2024
@wagoodman
unpin skopeo during mac acceptance tests (anchore#176)
3cc5074 
Signed-off-by: Alex Goodman <alex.goodman@anchore.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@luhring luhring luhring approved these changes

@alfredodeza alfredodeza Awaiting requested review from alfredodeza

@dakaneye dakaneye Awaiting requested review from dakaneye

Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@wagoodman @luhring