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

Fix pg16 CI #1326

Merged
merged 1 commit into from
Oct 10, 2023
Merged

Conversation

eeeebbbbrrrr
Copy link
Contributor

Taking what we learned from PL/Rust and pg16 and CI, this does essentially the same two things:

  • removes all the old postgres/clang/llvm-related packages that github might have installed
  • sets up clang14 only

The "problem" seems to be having multiple versions of clang installed at once. There may be some way to say, "dear bindgen, only use this particular clang", but I think this is more expedient. Also, it's probably best CI doesn't have a bunch of clutter we don't need.

Taking what we learned from PL/Rust and pg16 and CI, this does
essentially the same two things:

- removes all the old postgres/clang/llvm-related packages that github
  might have installed
- sets up clang14 only

The "problem" seems to be having multiple versions of clang installed at
once.  There may be some way to say, "dear bindgen, only use this
particular clang", but I think this is more expedient.  Also, it's
probably best CI doesn't have a bunch of clutter we don't need.
@eeeebbbbrrrr
Copy link
Contributor Author

merging w/o review since it's "just CI", and we need it to work and someone already got confused by this in #1320

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.

1 participant