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 the static build #2328

Merged
merged 1 commit into from
Jun 15, 2022
Merged

fix the static build #2328

merged 1 commit into from
Jun 15, 2022

Conversation

robx
Copy link
Contributor

@robx robx commented Jun 15, 2022

Fixes #2327.

  • the isExecutable patch was broken, remove it
  • the ncurses fix was broken, we still need enableStatic
  • the original error was caused by the new ghc-bignum
    package, which isn't a "proper" Haskell package; we filter
    this out explicitly now

- the isExecutable patch was broken, remove it
- the ncurses fix was broken, we still need `enableStatic`
- the original error was caused by the new ghc-bignum
  package, which isn't a "proper" Haskell package; we filter
  this out explicitly now
@robx robx changed the title fixing static build fix the static build Jun 15, 2022
@robx robx marked this pull request as ready for review June 15, 2022 21:13
@robx
Copy link
Contributor Author

robx commented Jun 15, 2022

Works 🎉,

$ ldd ./postgrest
	not a dynamic executable

on the CI artifact.

@robx robx mentioned this pull request Jun 15, 2022
Copy link
Member

@steve-chavez steve-chavez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! The pg 14 error in CI seems like a temporary one. We got the same on #2311 and when re-running the job it was cleared.

@robx robx merged commit 734f8e6 into PostgREST:main Jun 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

static build is broken
2 participants