-
Notifications
You must be signed in to change notification settings - Fork 668
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
Ignore the desktop.ini file in every directory, not only in top dir. #8299
Conversation
The comment |
The desktop.ini file only concerns the local Explorer. If you sync it and the "other" computer does have a different screen resolution for example, things mit go wrong. OneDrive also does not sync |
Imo, it should be always synced, but renamed to a safe name. E.g. desktop.ini._RENAMED |
src/csync/csync_exclude.cpp
Outdated
@@ -201,8 +201,8 @@ static CSYNC_EXCLUDE_TYPE _csync_excluded_common(const QString &path, bool exclu | |||
#endif | |||
|
|||
/* We create a Desktop.ini on Windows for the sidebar icon, make sure we don't sync it. */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The comment is now misleading.
Internal ci issues are fixed on 2.7 and master |
57a11b4
to
7c20000
Compare
Kudos, SonarCloud Quality Gate passed! |
See #8298 for reasons.