[Github-comments] [geany] Towards gvfs-fuse independence (#963)

Jiří Techet notifications at xxxxx
Sun Mar 13 22:56:27 UTC 2016


Based on the discussion here https://github.com/geany/geany/issues/921 I tried what is still missing to support opening remote files without gvfs-fuse. As I suspected, there wasn't much missing - mostly some g_file_test() calls which didn't work remotely, file name validation which didn't allow URIs, getting filenames instead of URIs from file dialogs etc.

What works after these patches:
* opening links like https://developer.gnome.org/gio/stable/GFile.html both from command-line and file open dialog
* browsing remote locations in file browser
* opening/saving ftp/ssh/etc. locations when mounted - I guess they need to be mounted to work - even gedit needs them mounted (it would probably be possible to use g_file_mount_enclosing_volume () to pop up dialog to make user mount them, not sure if we want it or not)
* opening/saving remote projects

What doesn't work:
* TM parsing - the only reason here's the g_stat() in tm_source_file_init(), after commenting it out it works but TM at the moment doesn't know about Geany's file operation method used so it's not possible to implement this properly (I plan moving the TM - without ctags - into src making it effectively part of Geany after which it will have access to Geany's definitions)
* File->Properties - I simply have been lazy to implement it
* of course things like find in files, build commands etc. which we cannot run remotely
You can view, comment on, or merge this pull request online at:

  https://github.com/geany/geany/pull/963

-- Commit Summary --

  * Add wrappers around g_file_test() and use them everywhere
  * Update utils_get_path_from_uri() to return URI when using GIO
  * Fix warning in utils_tidy_path()
  * Use g_file_new_for_uri() when needed
  * Preserve URIs in tm_get_real_path()
  * Merge implementations of write_data_to_disk() and utils_write_file()
  * Use gtk_file_chooser_get_uris() in file opening/saving dialogs
  * Add GIO version of document_rename_file()
  * Add GIO implementation to utils_get_file_list_full()
  * Add file reading utils function
  * Load project config files using the new utility function
  * Allow opening projects using URI

-- File Changes --

    M plugins/export.c (2)
    M plugins/filebrowser.c (6)
    M plugins/htmlchars.c (2)
    M plugins/saveactions.c (6)
    M src/build.c (4)
    M src/callbacks.c (8)
    M src/dialogs.c (8)
    M src/document.c (131)
    M src/editor.c (2)
    M src/highlighting.c (6)
    M src/keybindings.c (4)
    M src/keyfile.c (4)
    M src/libmain.c (24)
    M src/msgwindow.c (6)
    M src/plugins.c (2)
    M src/project.c (57)
    M src/symbols.c (2)
    M src/templates.c (2)
    M src/ui_utils.c (6)
    M src/utils.c (322)
    M src/utils.h (16)
    M src/vte.c (2)
    M src/win32.c (4)
    M tagmanager/src/tm_source_file.c (16)

-- Patch Links --

https://github.com/geany/geany/pull/963.patch
https://github.com/geany/geany/pull/963.diff

---
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/963
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20160313/d8e84cb5/attachment.html>


More information about the Github-comments mailing list