Skip to content

Conversation

@AsherGlick
Copy link
Owner

If a file is moved into a folder then the current cache invalidation detection wont see it if its modified time is not updated as well. This adds a check on mtime (modified) as well as ctime (change) to see if either of these values are newer then the current cached files.

@AsherGlick
Copy link
Owner Author

@vreelb to see if this works on windows as well.

@vreelb
Copy link
Contributor

vreelb commented Dec 18, 2022

Unfortunately it does not appear to work on Windows.
image
image

@vreelb
Copy link
Contributor

vreelb commented Dec 18, 2022

FWIW it is still likely useful to merge this if it solves this on Linux, as Windows is no worse off with this change than before.

@AsherGlick
Copy link
Owner Author

That's unfortunate to hear. Some research into the windows file APIs make it seem like this is impossible on windows because that type of data is not tracked by the OS. I wonder how other systems handle it.

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