Skip to content

[WIP] tool-cache: in tc.cacheDir() and tc.cacheFile(), allow the user to opt-in to preserving timestamps when copying#1617

Closed
DilumAluthge wants to merge 14 commits intoactions:mainfrom
DilumAluthge-forks:dpa/preservetimestamps-tc-cachedir
Closed

[WIP] tool-cache: in tc.cacheDir() and tc.cacheFile(), allow the user to opt-in to preserving timestamps when copying#1617
DilumAluthge wants to merge 14 commits intoactions:mainfrom
DilumAluthge-forks:dpa/preservetimestamps-tc-cachedir

Conversation

@DilumAluthge
Copy link
Copy Markdown

@DilumAluthge DilumAluthge commented Jan 5, 2024

Replaces #1504 (closes #1504)

The value of preserveTimestamp defaults to false, so by default we preserve the existing behavior.

@DilumAluthge DilumAluthge changed the title [WIP] tool-cache: in tc.cacheDir(), by default, preserve timestamps when copying (but allow the user to override this) [WIP] tool-cache: in tc.cacheDir() and tc.cacheFile(), by default, preserve timestamps when copying (but allow the user to override this) Jan 5, 2024
Copy link
Copy Markdown

@omus omus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good otherwise to me

DilumAluthge and others added 2 commits January 5, 2024 14:27
…amps}`

Co-authored-by: Curtis Vogt <curtis.vogt@gmail.com>
…amps}`

Co-authored-by: Curtis Vogt <curtis.vogt@gmail.com>
@DilumAluthge DilumAluthge changed the title [WIP] tool-cache: in tc.cacheDir() and tc.cacheFile(), by default, preserve timestamps when copying (but allow the user to override this) [WIP] tool-cache: in tc.cacheDir() and tc.cacheFile(), allow the user to opt-in to preserving timestamps when copying Jan 5, 2024
DilumAluthge and others added 9 commits January 5, 2024 15:06
Co-authored-by: Ian Butterworth <i.r.butterworth@gmail.com>
Co-authored-by: Ian Butterworth <i.r.butterworth@gmail.com>
@DilumAluthge DilumAluthge deleted the dpa/preservetimestamps-tc-cachedir branch August 31, 2025 20:05
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.

3 participants