Skip to content

ci: pass --locked to cargo install --path cargo-pgrx#2292

Merged
eeeebbbbrrrr merged 1 commit into
pgcentralfoundation:developfrom
philippemnoel:ci-cargo-install-locked
May 31, 2026
Merged

ci: pass --locked to cargo install --path cargo-pgrx#2292
eeeebbbbrrrr merged 1 commit into
pgcentralfoundation:developfrom
philippemnoel:ci-cargo-install-locked

Conversation

@philippemnoel

@philippemnoel philippemnoel commented May 15, 2026

Copy link
Copy Markdown
Contributor

Summary

Fix the CI

Test plan

  • `grep -rn "cargo install" .github/workflows/` confirms every `--path cargo-pgrx` invocation now passes `--locked`
  • CI on this PR builds green (will verify once it runs)

`cargo install --path` ignores `Cargo.lock` unless `--locked` is given,
so every CI run re-resolved transitive deps fresh from crates.io. This
recently started failing on the `verify package can build` job: the
fresh resolution pulled in `rustix 0.37.28`, which uses `#[rustc_attrs]`
attributes that the current nightly compiler rejects with:

    error: attributes starting with `rustc` are reserved for use by the
           `rustc` compiler
      --> rustix-0.37.28/src/backend/linux_raw/io/errno.rs:28:25

`Cargo.lock` on `develop` already pins `rustix 1.1.2`, which builds
fine; the CI just wasn't honoring it.

Add `--locked` to every `cargo install --path cargo-pgrx` invocation
across the three workflow files (5 in tests.yml, 1 in runas.yml, 1 in
package-test.yaml) so CI uses the same resolution as `cargo build` and
becomes reproducible.
@eeeebbbbrrrr eeeebbbbrrrr merged commit 1a26e15 into pgcentralfoundation:develop May 31, 2026
15 checks passed
eeeebbbbrrrr added a commit that referenced this pull request May 31, 2026
Welcome to pgrx v0.18.1. This is a minor release but fixes important
build problems on aarch64 Linux introduced in v0.18.0.

As always, install the cli tool with `cargo install cargo-pgrx --version
0.18.1 --locked` and make sure to update your extension's to depend on
`=0.18.1`.


## What's Changed
* upgrade dependencies, including moving rust to 1.96.0 by @eeeebbbbrrrr
in #2296
* cargo-pgrx: inject `-Wl,--no-gc-sections` to keep `.pgrxsc` sections
on aarch64 Linux by @eeeebbbbrrrr in
#2280
* feat: include `rewrite/rewriteManip.h` by @isdaniel in
#2295
* Use `zip` instead of deprecated `zip-extract` by @cbandy in
#2294 (also by
@Natureshadow)
* cargo-pgrx: add `--valgrind` flag to `cargo pgrx regress` by
@philippemnoel in #2290
* cargo-pgrx: build Postgres in parallel during `cargo pgrx init` by
@philippemnoel in #2291
* ci: pass `--locked` to `cargo install --path cargo-pgrx` by
@philippemnoel in #2292
* Fix README version range inconsistency (17 → 18) by @hobostay in
#2287
* fix: don't allocate for static strings in `ereport!` by @gruuya in
#2282
* can `cargo clippy --allow-dirty --fix -- -Wclippy::use_self` by
@adamnemecek in #2283
* feature: teach `cargo pgrx schema` how to emit just the SQL for named
entities by @eeeebbbbrrrr in
#2293
* Fix "PostgresSQL" typo to "PostgreSQL" in test code by @hobostay in
#2286

## New Contributors
* @hobostay made their first contribution in
#2287
* @adamnemecek made their first contribution in
#2283
* @Natureshadow 

**Full Changelog**:
v0.18.0...v0.18.1
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