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

assets:sync: compare file hash additional to the filemtime #3792

Merged
merged 2 commits into from Jul 5, 2020

Conversation

staabm
Copy link
Member

@staabm staabm commented Jul 4, 2020

closes #3714

Getestet indem man

  • eine datei verändert
  • speichert (damit sich die filemtime ändert)
  • die datei vom inhalt her wieder auf den stand von zuvor zurück verändert
  • erneut speichert

-> jetzt hat man dateien die inhaltlich identisch sind, d.h. mit gleichem content-hash, aber unterschiedlicher filemtime.
-> vor dem PR wurden diese dateien kopiert. Jetzt nach der Änderung nicht mehr

@staabm staabm added this to the REDAXO 5.12 milestone Jul 4, 2020
@gharlan gharlan added the automerge Automatisch PR rebasen und mergen label Jul 5, 2020
@kodiakhq kodiakhq bot merged commit a06f48a into master Jul 5, 2020
@kodiakhq kodiakhq bot deleted the asset-sync-hash branch July 5, 2020 09:10
@staabm
Copy link
Member Author

staabm commented Jan 13, 2022

kurz als info, da wir das gleiche problem an anderen stellen hatten: filemtime liefert bei npm falsche timestamps, wenn man eine alte npm-version verwendet. betroffen ist z.B. npm version 6.14.15.. gefixed ist es spätestens in 8.3.0.

wann es genau gefixed wurde ist nicht bekannt.

relevanter kommentar: yarnpkg/yarn#5577 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Automatisch PR rebasen und mergen
Development

Successfully merging this pull request may close these issues.

assets:sync mit inhalts-check statt alleine filemtime
2 participants