Skip to content

Commit 10f12e2

Browse files
committed
build fails due to missing libpq (#213)
1 parent 4e31515 commit 10f12e2

File tree

4 files changed

+19
-15
lines changed

4 files changed

+19
-15
lines changed

.github/workflows/rust.yml

+2-4
Original file line numberDiff line numberDiff line change
@@ -73,10 +73,8 @@ jobs:
7373
fail-fast: false
7474
matrix:
7575
os: [ ubuntu-18.04, macos-10.15 ]
76-
# NOTE: Although we'd like to use stable, we cannot with a newer version
77-
# of Propolis as a dependency (which only builds with nightly).
78-
# toolchain: [ nightly-2021-04-24, stable ]
79-
toolchain: [ nightly-2021-04-24 ]
76+
# See rust-toolchain for why we're using nightly here.
77+
toolchain: [ nightly ]
8078
steps:
8179
# actions/checkout@v2
8280
- uses: actions/checkout@28c7f3d2b5162b5ddd3dfd9a45aa55eaf396478b

Cargo.lock

+1-2
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Cargo.toml

+9
Original file line numberDiff line numberDiff line change
@@ -32,3 +32,12 @@ panic = "abort"
3232
#dropshot = { path = "../dropshot/dropshot" }
3333
#[patch."https://github.com/oxidecomputer/steno"]
3434
#steno = { path = "../steno" }
35+
36+
#
37+
# We maintain a fork of pq-sys to address
38+
# https://github.com/sgrif/pq-sys/issues/36. If the fix for that (PR#37) lands
39+
# upstream, we can remove our fork.
40+
#
41+
[patch.crates-io.pq-sys]
42+
git = 'https://github.com/oxidecomputer/pq-sys'
43+
branch = "oxide"

rust-toolchain.toml

+7-9
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,15 @@
1-
# We use nightly to enable the "asm" feature for dtrace probes.
1+
# We use nightly for two reasons:
22
#
3-
# Additionally, we specify this specific verion of nightly to
4-
# workaround known bugs for illumos on more recent versions.
5-
# Context: https://github.com/rust-lang/rust/pull/85772
3+
# 1. To enable the "asm" feature for dtrace probes.
4+
# 2. To make use of our workaround for issue sgrif/pq-sys#36. This relies on a
5+
# Cargo feature that will not be available on stable until 1.56.
66
#
7-
# Short-term: Once that bug is fixed, we should feel free
8-
# to update to migrate to a more recent nightly.
7+
# When both of those are part of stable Rust, we should update this file to
8+
# point to "stable" instead.
99
#
10-
# Long-term: Once the ASM feature is stabilized, we should
11-
# consider using stable instead of nightly.
1210

1311
[toolchain]
1412
# NOTE: This toolchain is also specified within .github/workflows/rust.yml
1513
# If you update it here, update that file too.
16-
channel = "nightly-2021-04-24"
14+
channel = "nightly"
1715
profile = "default"

0 commit comments

Comments
 (0)