-
Notifications
You must be signed in to change notification settings - Fork 281
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
add nautilus extension support #575
base: master
Are you sure you want to change the base?
Conversation
60cfd42
to
dcfb29d
Compare
Most part of it should work now. Please visit https://github.com/haiwen/seafile-client/tree/nautilus_extension/nautilus-seafile for how to install on your system! |
Hey @Chilledheart, does this mean there's not going to be a context menu to create a share link? I thought this was the point of the extension. ;) |
@dakira I will do this part once I have enough time. |
Might I suggest making this also available to Caja and other Nautilus forks. Caja is the file manager for MATE and extensions work exactly the same as Nautilus for 99% of all cases. |
@bertvandepoel There is no need. The Nautilus forks (and all other file managers) can just use the available code as a reference to build their own. That's how it worked with Dropbox. You should contact the MATE devs. |
@dakira I'll contact them then ;) |
b0bd47d
to
d804779
Compare
eb3335a
to
3488a80
Compare
Closes haiwen/seafile#1233. - ci: add nautilus build - improve client auotconnect handle - autorefresh overlay icons - cmake learns how to install and uninstall nautilus-seafile now - support readonly state - support locked/locked-by-me state
3488a80
to
7f68ef0
Compare
Will this be merged soon? |
cc @lins05 |
cc @killing |
The code is not completely stable yet. We still need some time to improve it. Unfortunately we don't have much time to work on this recently. |
I've got errors while trying to compile it on fedora 23 : is it the right place to check this out? `./build.sh
|
Are there plans to finish this? I got this error trying to install it. Information about my system: Fedora 23 (Linux fedora 4.4.4-301.fc23.x86_64 #1 SMP Fri Mar 4 17:42:42 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux)
|
@jhasse thanks, works here. |
Are there any plans to continue development? |
Installing this still works here. |
I meant progress on actually merging the pull request, although it's good to know that the current version works. |
Afaik the dev submitting the PR is not working on Seafile anymore. As long as no one from the community makes it production ready I don't think something is going to happen. |
I've picked up development of the nautilus extension and added some fixes. You can find the latest version here: https://github.com/benruijl/seafile-client/tree/nautilus_extension I am considering to make seafile-nautilus into a separate package, just like the dropbox-nautilus package. Let me know what you think is best. |
Hi @benruijl, Thanks for that. We're very interested in your continued development on nautilus extension. We'll soon test your code and see whether we can make it into official client too. |
Hi @killing , I look forward to hearing from you. In the meantime I have made a package for Arch Linux, so that people can install the extension easily: |
@killing any progress on the integration into official client? |
@benruijl can you create a new pull request from your branch, so that this one can be closed? We can continue the discussion there, then! |
@dakira the following problem has not been resolved yet, which causes the plugin to crash: haiwen/seafile#1891 Somehow calling |
This addition looks so good for Ubuntu and other Linux users. Is it in priority? |
I just complied this on Fedora 28. The extension causes nautilus to crash. I'll post more information when I get a chance to debug. I suspect it is a change in Nautilus 3.28. Here is the stack trace that is getting dumped into journald: |
The extension has been broken for a very long time. You can see my last post for a minimal failing example. The problem lies outside my code. |
What is the status nowadays? |
Maybe @freeplant or @lins05 could have a look? The old comment by @benruijl seems to be a good hint. |
WIP:
Closes haiwen/seafile#1233.