confluentinc / terraform-provider-confluent

Terraform Provider for Confluent
Apache License 2.0
31 stars 64 forks source link

Fix "syntax error: package statement must be first" #475

Closed linouk23 closed 1 month ago

linouk23 commented 1 month ago

Context

This PR should solve the error in goreleaser-darwin-fips job.

The root cause is our previous CI builds used gpatch--2.7.6.arm64_ventura.bottle.tar.gz, while the most recent (failed) one used gpatch--2.7.6.arm64_ventura.bottle.1.tar.gz. This indicates that there’s a slight difference in the bottles (likely a minor patch or metadata update).

Based on the output, the key differences between the two installations of gpatch seem to be:

Different Bottle Versions:

In the second installation, there’s an additional message in the output:

######################################################################## 100.0%
==> Pouring gpatch--2.7.6.arm64_ventura.bottle.1.tar.gz
==> Caveats
GNU "patch" has been installed as "gpatch".
If you need to use it as "patch", you can add a "gnubin" directory
to your PATH from your bashrc like:

    PATH="/opt/homebrew/opt/gpatch/libexec/gnubin:$PATH"

about setting up gpatch as the default patch.

This suggests that the installed gpatch version is not symlinked as patch by default. If this is missing, scripts expecting patch might fail if they cannot find gpatch.

Testing

confluent-cla-assistant[bot] commented 1 month ago

:tada: All Contributor License Agreements have been signed. Ready to merge.
Please push an empty commit if you would like to re-run the checks to verify CLA status for all contributors.