-
Notifications
You must be signed in to change notification settings - Fork 113
BE-419: Fix cargo publish failures for darwin-kperf crates
#8471
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
base: main
Are you sure you want to change the base?
Changes from all commits
1d3af06
23049c0
954078f
6374031
c57b1ad
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,6 +1,6 @@ | ||
| { | ||
| "name": "@rust/darwin-kperf-codegen", | ||
| "version": "0.1.0-private", | ||
| "version": "0.1.1-private", | ||
| "private": true, | ||
| "license": "MIT OR Apache-2.0" | ||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,10 +1,11 @@ | ||
| [package] | ||
| name = "darwin-kperf-criterion" | ||
| version = "0.1.0" | ||
| version = "0.1.1" | ||
| authors.workspace = true | ||
| edition.workspace = true | ||
| license = "MIT OR Apache-2.0" | ||
| publish = true | ||
| readme = "../README.md" | ||
| description = "Criterion measurement plugin using Apple's hardware performance counters" | ||
| documentation = "https://docs.rs/darwin-kperf-criterion" | ||
| repository = "https://github.com/hashintel/hash/tree/main/libs/darwin-kperf/criterion" | ||
|
|
@@ -13,7 +14,7 @@ categories = ["development-tools::profiling"] | |
|
|
||
| [dependencies] | ||
| # Public workspace dependencies | ||
| darwin-kperf-events = { version = "0.1.0", path = "../events", public = true } | ||
| darwin-kperf-events = { workspace = true, public = true } | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This dependency specification causes publishing to fail because cargo tries to resolve darwin-kperf-events ^0.1.1 from crates.io during publish, but only 0.1.0 exists there. The publishing order needs to ensure darwin-kperf-events 0.1.1 is published first, or the dependency should be specified with an explicit version that exists on crates.io. Spotted by Graphite Agent (based on CI logs) |
||
|
|
||
| # Public third-party dependencies | ||
| codspeed-criterion-compat-walltime = { workspace = true, optional = true, public = true, default-features = false } | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These workspace dependency declarations specify version 0.1.1 for darwin-kperf packages, but when publishing to crates.io, the dependencies must be published before dependents. The CI pipeline needs to publish darwin-kperf-events and darwin-kperf-sys first (which have no dependencies), then darwin-kperf-criterion, and finally darwin-kperf.
Spotted by Graphite Agent (based on CI logs)

Is this helpful? React π or π to let us know.