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

File monitor doesn't work with SMB #280

Closed
tsujan opened this issue Sep 22, 2018 · 2 comments · Fixed by #388
Closed

File monitor doesn't work with SMB #280

tsujan opened this issue Sep 22, 2018 · 2 comments · Fixed by #388

Comments

@tsujan
Copy link
Member

tsujan commented Sep 22, 2018

I thought it shouldn't work with SMB until @agaida said that it works in pcmanfm (GTK based).

@PCMan
Copy link
Member

PCMan commented Sep 23, 2018

Unfortunately, gvfs does not support file monitor for remote filesystems.
In gtk+ version of pcmanfm, we tried to do some simulation by emitting the signals during file operations (does not always work as expected), but this is not implemented in the Qt version yet.

@tsujan
Copy link
Member Author

tsujan commented Sep 23, 2018

Thank you very much for the explanation! So, there's no bug to fix but a feature to add.

tsujan added a commit that referenced this issue Apr 9, 2019
Closes lxqt/pcmanfm-qt#933 and closes #280

After a file transfer job is finished inside a directory, if it is the path of an open folder that lacks file monitoring, this patch reloads its corresponding folder. In this way, the lack of file monitoring is partially compensated for.

Please note that this doesn't work with `search://` because the files inside `search://` don't belong to it.

WARNING: All libfm-qt based apps should be recompiled after this.
agaida pushed a commit that referenced this issue Apr 10, 2019
Closes lxqt/pcmanfm-qt#933 and closes #280

After a file transfer job is finished inside a directory, if it is the path of an open folder that lacks file monitoring, this patch reloads its corresponding folder. In this way, the lack of file monitoring is partially compensated for.

Please note that this doesn't work with `search://` because the files inside `search://` don't belong to it.

WARNING: All libfm-qt based apps should be recompiled after this.
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 a pull request may close this issue.

2 participants