Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@matsuo
Copy link
Contributor

@matsuo matsuo commented Aug 5, 2023

Checklist
  • make test-all (UNIX) passes. CI will also test this

Description of change

This pull request fixes release-build.sh to run "make build" on macOS.

On macOS, "make build" failed because there is no sha256sum command in the default environment.

% sw_vers 
ProductName:		macOS
ProductVersion:		13.5
BuildVersion:		22G74
% make build    
./release-build.sh alpha-darwin-amd64
./release-build.sh: line 39: sha256sum: command not found
make: *** [release/goss-alpha-darwin-amd64] Error 127

@matsuo matsuo requested a review from aelsabbahy as a code owner August 5, 2023 19:55
@aelsabbahy
Copy link
Member

Great catch, didn't notice that MacOS was missing from the RC.

Thank you for the contribution!

@aelsabbahy aelsabbahy merged commit 592be23 into goss-org:master Aug 5, 2023
@matsuo
Copy link
Contributor Author

matsuo commented Aug 5, 2023

Thank you for your quick confirmation!

@matsuo matsuo deleted the modify-release-build-sh branch August 5, 2023 23:47
@aelsabbahy
Copy link
Member

Hmm, seems like there are still no osx binaries in the latest release.

@petemounce I think you were involved in some of those scripts. Any idea why osx might be failing?

@matsuo
Copy link
Contributor Author

matsuo commented Aug 7, 2023

@aelsabbahy
I believe I may have found the cause. Please check #829.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants