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

feat: Add support for NTFS extra field #279

Merged
merged 1 commit into from
Feb 26, 2025

Conversation

sorairolake
Copy link
Contributor

@sorairolake sorairolake commented Jan 14, 2025

Add support for NTFS extra field as described in PKWARE's APPNOTE.TXT v6.3.9.

A ZIP file containing this field can be created using 7-Zip:

7z a archive.zip README.md

You can check the timestamp by running:

7z l -slt archive.zip

Also, I added an optional dependency on nt-time crate. This is to make it easier to treat the obtained timestamps as time.

@sorairolake sorairolake force-pushed the feature/ntfs-extra-field branch 2 times, most recently from 8b64b64 to 9674c04 Compare January 14, 2025 17:54
@sorairolake sorairolake changed the title feat: Add support for NTFS Extra Field feat: Add support for NTFS extra field Jan 14, 2025
@Pr0methean Pr0methean added this pull request to the merge queue Feb 25, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 25, 2025
@Pr0methean Pr0methean added this pull request to the merge queue Feb 25, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 25, 2025
@sorairolake sorairolake force-pushed the feature/ntfs-extra-field branch from 9674c04 to f6fa8e0 Compare February 26, 2025 07:27
@Pr0methean Pr0methean added this pull request to the merge queue Feb 26, 2025
Merged via the queue into zip-rs:master with commit d5de364 Feb 26, 2025
39 checks passed
@sorairolake sorairolake deleted the feature/ntfs-extra-field branch February 26, 2025 21:44
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.

2 participants