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

[BUG] - Makefile will not build due to reliance on bash in scripts/fmtcheck.sh #378

Open
shawnmchugh opened this issue Jul 10, 2023 · 1 comment
Assignees
Labels

Comments

@shawnmchugh
Copy link

shawnmchugh commented Jul 10, 2023

Describe the bug
The makefile to build the terraform-provider-vultr will not build due to scripts/fmtcheck.sh having an implicit reliance on bash being present on the system.

To Reproduce
Steps to reproduce the behavior:

  1. cd $GOPATH/src/github.com/vultr/terraform-provider-vultr
  2. gmake build

Expected behavior
Expected makefile and associated scripts to use POSIX compliant shell for portability to systems that use a different default shell, such as OpenBSD, MacOS, etc.

Screenshots/Output
env: bash: No such file or directory
gmake: *** [GNUmakefile:32: fmtcheck] Error 127

Desktop (please complete the following information where applicable:

  • OS: OpenBSD 7.3
  • Language Version: Go 1.20.1, latest terraform-provider-vultr release
  • Shell: OpenBSD ksh

Additional context

Not sure if this is a bug or feature request, but seems more of a bug, since any other system not using bash as a default will have the same issue. I already have gmake on the system, I'd rather not have to install bash as well. I am able to work around the issue by using gofmt and go install manually or from another script/makefile, of course, but not ideal.

@shawnmchugh shawnmchugh changed the title [BUG] - Makefile will not build due to reliance on bash [BUG] - Makefile will not build due to reliance on bash in scripts/fmtcheck.sh Jul 10, 2023
@optik-aper optik-aper self-assigned this Jul 25, 2023
@optik-aper
Copy link
Member

Thanks for the report. We should support POSIX compliance and will attempt to do so.

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

No branches or pull requests

2 participants