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

Refactor remaining core extension build scripts #845

Merged
merged 1 commit into from
Dec 13, 2024
Merged

Conversation

theory
Copy link
Collaborator

@theory theory commented Dec 13, 2024

As in 8d2a1e7 (#838) make a shallow clone of the Git repo and use USE_PGXS=1 to build and install against the system Postgres instead of configuring the cloned source, eliminating the need to move files in the install_command.

Also, run the install_command with bash instead of sh, for better shell semantics. Required for lines like this to work:

rm "$(pg_config --sharedir)"/extension/plperlu{--*,.*}

Make minor edits to the jsonb_plperl, plperl, and pltcl build scripts so they fire again and properly remove unwanted files.

@theory theory self-assigned this Dec 13, 2024
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-845.d2qgksbj4k37j8.amplifyapp.com

theory added a commit that referenced this pull request Dec 13, 2024
As in 8d2a1e7 (#838) make a shallow clone of the Git repo and use
`USE_PGXS=1` to build and install against the system Postgres instead of
configuring the cloned source, eliminating the need to move files in the
`install_command`.
@theory theory marked this pull request as ready for review December 13, 2024 19:38
theory added a commit that referenced this pull request Dec 13, 2024
As in 8d2a1e7 (#838) make a shallow clone of the Git repo and use
`USE_PGXS=1` to build and install against the system Postgres instead of
configuring the cloned source, eliminating the need to move files in the
`install_command`.

Also, run the `install_command` with `bash` instead of `sh`, for better
shell semantics. Required for lines like this to work:

```sh
rm "$(pg_config --sharedir)"/extension/plperlu{--*,.*}
```

Make minor edits to the jsonb_plperl, plperl, and pltcl build scripts so
they fire again and properly remove unwanted files.
theory added a commit that referenced this pull request Dec 13, 2024
As in 8d2a1e7 (#838) make a shallow clone of the Git repo and use
`USE_PGXS=1` to build and install against the system Postgres instead of
configuring the cloned source, eliminating the need to move files in the
`install_command`.

Also, run the `install_command` with `bash` instead of `sh`, for better
shell semantics. Required for lines like this to work:

```sh
rm "$(pg_config --sharedir)"/extension/plperlu{--*,.*}
```

Make minor edits to the jsonb_plperl, plperl, and pltcl build scripts so
they fire again and properly remove unwanted files.
As in 8d2a1e7 (#838) make a shallow clone of the Git repo and use
`USE_PGXS=1` to build and install against the system Postgres instead of
configuring the cloned source, eliminating the need to move files in the
`install_command`.

Also, run the `install_command` with `bash` instead of `sh`, for better
shell semantics. Required for lines like this to work:

```sh
rm "$(pg_config --sharedir)"/extension/plperlu{--*,.*}
```

Make minor edits to the jsonb_plperl, plperl, and pltcl build scripts so
they fire again and properly remove unwanted files.
@theory theory merged commit 9a888ad into main Dec 13, 2024
165 checks passed
@theory theory deleted the pg17-more-core branch December 13, 2024 20:54
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