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

Artifact builds upon release #302

Merged
merged 36 commits into from
May 9, 2023
Merged

Artifact builds upon release #302

merged 36 commits into from
May 9, 2023

Conversation

BradyBonnette
Copy link
Contributor

@BradyBonnette BradyBonnette commented May 4, 2023

The main story of these changes is that Debian packages of PL/Rust are created and uploaded to Github whenever a new release of PL/Rust is created. This includes the aarch64/arm64 packages that required some cloud infrastructure not included in this repo.

Additionally in these changes are some dependency updates, since pgrx 0.8.3 was needed.

EDIT: Note that the Github Actions changes for producing the Debian packages will not go in effect until the changes are merged into tcdi:main

Copy link
Contributor

@eeeebbbbrrrr eeeebbbbrrrr left a comment

Choose a reason for hiding this comment

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

Excellent work, @BradyBonnette! This is great
image

@eeeebbbbrrrr
Copy link
Contributor

As we discussed offline, a mention of Debian artifacts in the top-level project README would be great, but that's no reason to hold an approval on this.

* Updates all commands to use sudo instead of implying root
* Adds a warning about where to store the downloaded file on the target
system, since apt has an issue installing a package if it doesnt have
write access to the directory where the file resides
* Other various updates
@BradyBonnette BradyBonnette merged commit b821159 into tcdi:develop May 9, 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.

2 participants