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

Pass RUSTFLAGS to rustc on schema generate #513

Merged
merged 1 commit into from
Mar 30, 2022

Conversation

Hoverbear
Copy link
Contributor

@Hoverbear Hoverbear commented Mar 28, 2022

On some operating systems, such as Alpine, rustc requires special arguments to be able to generate a cdylib. (Eg: https://github.com/zombodb/zombodb/blob/f5848c652786e18dc2d1029f2a0747b4b1da0e3b/docker-build-system/zombodb-build-alpine-3.12/Dockerfile#L28)

This patch changes cargo-pgx schema to use any existing RUSTFLAGS in the environment.

Please note it does not read .cargo/config.

@Hoverbear Hoverbear self-assigned this Mar 28, 2022
@eeeebbbbrrrr eeeebbbbrrrr merged commit fc2fb8c into develop Mar 30, 2022
@eeeebbbbrrrr
Copy link
Contributor

LGTM. Merged

eeeebbbbrrrr added a commit that referenced this pull request Mar 30, 2022
* update copyright headers (#511)

* Add pg_operator.h (#512)

* Add include for pg_operator.h

* Regenerate bindings

* Add Copy and Clone to the time_stamp_* types (#509)

* Add from impls for pg_sys -> pgx timestamp* (#510)

* Remove dead --workspace flag on cargo pgx test (#520)

* Pass RUSTFLAGS to rustc on schema generate (#513)

* Prepare 0.4.2 (#522)

* upgrade dependencies

* bump version to v0.4.2

Co-authored-by: Einar <[email protected]>
Co-authored-by: James Sewell <[email protected]>
Co-authored-by: Ana Hobden <[email protected]>
@eeeebbbbrrrr eeeebbbbrrrr mentioned this pull request Apr 19, 2022
@eeeebbbbrrrr eeeebbbbrrrr deleted the pass-rustflags-to-rustc branch June 20, 2023 17:59
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