Skip to content

Create a new maint tool to prevent common publishing mistakes (#265)#266

Merged
Byron merged 5 commits intorust-lang:mainfrom
Byron:publish-safety
Apr 5, 2026
Merged

Create a new maint tool to prevent common publishing mistakes (#265)#266
Byron merged 5 commits intorust-lang:mainfrom
Byron:publish-safety

Conversation

@Byron
Copy link
Copy Markdown
Member

@Byron Byron commented Apr 5, 2026

Tasks

  • try the maint publish tool
  • review and refactor
  • test it to ensure a future publish would work: create a new release with the newly found patch

The zlib-ng vendored sources provide `src/zlib-ng/arch/arm/crc32_armv8.c`,
but the cc-based build path still referenced `crc32_acle.c`.

That breaks packaging and `cargo publish --dry-run` verification for
`--no-default-features --features zlib-ng-no-cmake-experimental-community-maintained`
on ARM targets.

Switch the cc source list to `crc32_armv8` so the packaged crate matches
the vendored source tree and the ARM release verification path succeeds.

Co-authored-by: Sebastian Thiel <sebastian.thiel@icloud.com>
…-lang#265)

Co-authored-by: Sebastian Thiel <sebastian.thiel@icloud.com>
@Byron Byron marked this pull request as ready for review April 5, 2026 01:26
@Byron Byron merged commit 6ce001e into rust-lang:main Apr 5, 2026
48 checks passed
@Byron Byron deleted the publish-safety branch April 5, 2026 06:00
@weihanglo
Copy link
Copy Markdown
Member

The other solution that might be simpler is setting the NO_FOO_PKG_CONFIG environment variables, so it guarantee to run the vendoring branch, and missing submodule would fail: https://docs.rs/pkg-config/0.3.32/pkg_config/#environment-variables

@weihanglo
Copy link
Copy Markdown
Member

And thanks for the fix!

@Byron
Copy link
Copy Markdown
Member Author

Byron commented Apr 6, 2026

That would have been easier indeed 😁!
Nonetheless, I am glad maint exists now.

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.

v1.1.26 missing src/zlib/ sources in published crate (Windows build broken)

3 participants