Skip to content
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

mkfs-util: propagate SOURCE_DATE_EPOCH to mcopy #29000

Merged
merged 1 commit into from
Aug 29, 2023
Merged

mkfs-util: propagate SOURCE_DATE_EPOCH to mcopy #29000

merged 1 commit into from
Aug 29, 2023

Conversation

malt3
Copy link
Contributor

@malt3 malt3 commented Aug 28, 2023

If set, mcopy will respect SOURCE_DATE_EPOCH and deterministically sets ctime, mtime and atime when creating folders / files in FAT partitions.
Since mcopy is invoked with execve, SOURCE_DATE_EPOCH has to be propagated explicitly.

src/shared/mkfs-util.c Outdated Show resolved Hide resolved
src/shared/mkfs-util.c Outdated Show resolved Hide resolved
src/shared/mkfs-util.c Outdated Show resolved Hide resolved
src/shared/mkfs-util.c Outdated Show resolved Hide resolved
@poettering poettering added the reviewed/needs-rework 🔨 PR has been reviewed and needs another round of reworks label Aug 29, 2023
Co-authored-by: Paul Meyer <49727155+katexochen@users.noreply.github.com>
@poettering
Copy link
Member

lgtm. any reason you still set this to draft?

@malt3
Copy link
Contributor Author

malt3 commented Aug 29, 2023

lgtm. any reason you still set this to draft?

I didn't have time to check if all reproducibility issues are resolved when it comes to FAT.
But let's merge and I'll create a follow up PR if needed.

@malt3 malt3 marked this pull request as ready for review August 29, 2023 08:50
@github-actions github-actions bot added please-review PR is ready for (re-)review by a maintainer and removed reviewed/needs-rework 🔨 PR has been reviewed and needs another round of reworks labels Aug 29, 2023
@DaanDeMeyer DaanDeMeyer added good-to-merge/waiting-for-ci 👍 PR is good to merge, but CI hasn't passed at time of review. Please merge if you see CI has passed and removed please-review PR is ready for (re-)review by a maintainer labels Aug 29, 2023
@bluca bluca merged commit 4947de2 into systemd:main Aug 29, 2023
@github-actions github-actions bot removed the good-to-merge/waiting-for-ci 👍 PR is good to merge, but CI hasn't passed at time of review. Please merge if you see CI has passed label Aug 29, 2023
@malt3 malt3 deleted the mcopy-source-date-epoch branch August 29, 2023 14:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

4 participants