Skip to content

Conversation

@weihanglo
Copy link
Member

What does this PR try to resolve?

With this commit, all tar entries' timestamp is the same,
all set to DETERMINISTIC_TIMESTAMP in tar-rs.

This fixes the first half of #16237
that cargo package should tar everything in the same timestamp.

How to test and review this PR?

Test passes.

We only set deterministic timestamp for actual file we copy cover during
`cargo package`ing. Generated files should actually follow.
With this commit, all tar entries' timestamp is the same,
all set to DETERMINISTIC_TIMESTAMP in tar-rs.

This fixes the first half of rust-lang#16237
that `cargo package` should tar everything in the same timestamp.
@rustbot rustbot added Command-package S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Nov 11, 2025
@rustbot
Copy link
Collaborator

rustbot commented Nov 11, 2025

r? @ehuss

rustbot has assigned @ehuss.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@epage epage added this pull request to the merge queue Nov 11, 2025
Merged via the queue into rust-lang:master with commit e7246c6 Nov 11, 2025
25 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Nov 11, 2025
@weihanglo weihanglo deleted the deterministic-mtime branch November 11, 2025 21:18
bors added a commit to rust-lang/rust that referenced this pull request Nov 12, 2025
Update cargo

10 commits in 445fe4a68f469bf936b2fd81de2c503b233a7f4f..2d4fa139552ebdd5f091a1401ed03f7dc62cb43f
2025-11-07 18:08:19 +0000 to 2025-11-12 15:56:06 +0000
- feat: Add unstable rustc-unicode flag (rust-lang/cargo#16243)
- fix(package): all tar entries timestamp be the same (rust-lang/cargo#16242)
- feat: emit help messages for github pull request url in dependency (rust-lang/cargo#16207)
- docs: fix comments for alternative registry fns (rust-lang/cargo#16235)
- add into_value utility function for inheritableField (rust-lang/cargo#16234)
- fix(command-vendor): strip_prefix panic in cp_sources method (rust-lang/cargo#16214)
- fix(lock): Be moore direct in the error message (rust-lang/cargo#16233)
- fix(lock): In error, differentiate between creating and updating lockfile (rust-lang/cargo#16227)
- fix(cli): Refer to commands, not subcommands (rust-lang/cargo#16226)
- fix(run): Help teach about argument escaping (rust-lang/cargo#16225)
@rustbot rustbot added this to the 1.93.0 milestone Nov 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants