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

Support --features trusted on macOS #334

Merged
merged 2 commits into from
Jun 14, 2023
Merged

Conversation

eeeebbbbrrrr
Copy link
Contributor

@eeeebbbbrrrr eeeebbbbrrrr commented Jun 12, 2023

With this PR, PL/Rust now supports installation as a Trusted Procedural Language on macOS.

It also supports cross compiling between x86_64 and aarch64 on macOS.

We do NOT support cross-compiling between operating systems.

With this PR, PL/Rust now supports installation
as a Trusted Procedural Language on macOS.

It also supports cross compiling between x86_64 and aarch64 on macOS.

We do **NOT** support cross-compiling between operating systems.
@eeeebbbbrrrr eeeebbbbrrrr requested a review from thomcc June 12, 2023 18:53
Copy link
Contributor

@thomcc thomcc left a comment

Choose a reason for hiding this comment

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

Looks great. No objections -- the weird bits are non-public and this is much simpler than the alternative.

"postgres"
cfg_if::cfg_if! {
if #[cfg(target_vendor = "apple")] {
"apple-darwin"
Copy link
Contributor

Choose a reason for hiding this comment

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

Not exactly the vendor (more like vendor-os) but eh, doesn't matter much.

}}
cfg_if::cfg_if! {
if #[cfg(all(feature = "trusted", target_os = "macos"))] {
const OS: &str = "postgres";
Copy link
Contributor

Choose a reason for hiding this comment

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

Definitely not the OS. This one is a little silly, but these aren't pub so whatever.

@eeeebbbbrrrr
Copy link
Contributor Author

Thanks for making CI work on macOS, @BradyBonnette

@eeeebbbbrrrr eeeebbbbrrrr merged commit 1bc752d into develop Jun 14, 2023
@eeeebbbbrrrr eeeebbbbrrrr mentioned this pull request Jun 14, 2023
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.

3 participants