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

Avoid useless rescan : Detect if local lib files have changed after sync #177

Merged
merged 1 commit into from
Apr 5, 2024

Conversation

ospring
Copy link
Contributor

@ospring ospring commented Mar 29, 2024

Rescan trigger after each sync seems to be the origin to issue #172. The PR add a comparaison of the content of the lib directory before and after the sync. The rescan will be launch only if the content has changed. Comparaison is done with the result of a detailed LS command (so, file name, size and dates).
To avoid the date of synced files to change after each sync, adding a --remote-time to the CURL command will allow to use the remote date of the file so if files are not changed remotely, dates on the local lib will stay the same and allow comparaison of the lib content to work as expected.

…an when unneeded. make curl to use remote file date to allow detect file change with dates
@ospring ospring changed the title Avoid useless rescan : Detect if local lib file has changed after sync Avoid useless rescan : Detect if local lib files have changed after sync Mar 29, 2024
@fsantini fsantini merged commit b571607 into fsantini:master Apr 5, 2024
1 check passed
@ospring ospring deleted the fixBackToHome branch April 6, 2024 06:45
@geephile
Copy link

geephile commented Apr 8, 2024

I still experience the issue (but i might be to lazy in the way i upgrade, dropping the new KoboRoot.tgz in the .kobo folder instead of uninstalling KoboCloud and after that installing it). If my upgrade method is not correct, please point me to the correct way :)

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