Skip to content

Commit

Permalink
remove PATH hack in tools/build
Browse files Browse the repository at this point in the history
We added this in order for #408 to pass the CI tests. Our CI image had
pgx 0.2 on its PATH, but since we were upgrading to 0.4, we wanted to
use the 0.4 series pgx when running our CI tests (which use
`tools/build`).

This allowed us to pass our CI tests, but now that we've merged the
upgrade, we don't need this, and it surely isn't useful to anyone
running tests with `tools/build` locally.
  • Loading branch information
rtwalker committed May 31, 2022
1 parent 4e6eed2 commit 7d303cc
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions tools/build
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,6 @@

set -ex

# TODO After we upgrade to 0.4 make this change in docker/ci/Dockerfile and then remove this line.
export PATH=/home/postgres/pgx/0.4/bin:$PATH

print() {
printf '%s\n' "$*"
}
Expand All @@ -19,7 +16,7 @@ die() {
}

usage() {
die 'build [ -n -pg1[234] ] ( test-crates | test-extension | test-post-install | test-doc | test-updates | clippy)'
die 'build [ -n -pg1[234] ] ( test-crates | test-extension | test-updates | test-post-install | test-doc | clippy)'
}

require_pg_version() {
Expand Down

0 comments on commit 7d303cc

Please sign in to comment.