Re: [Github-comments] [geany/geany] Extending the function of "Open selected file" (#1959)
If its a directory I would have thought you would want to open the Geany file chooser, not an external application. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/1959#issuecomment-423716474
Re: [Github-comments] [geany/geany] Extending the function of "Open selected file" (#1959)
What about variant A. I suggested? Isn't it too basic and simple to require separate plugin? For example, to open a folder: - "Open selected file" already can tell if the when the string selected/under cursor is a directory - in Preferences-> Tools -> Tools paths , add another field for the user to specify the path to "File manager" application - all is left is to modify the code of "Open selected file" to make it invoke that path -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/1959#issuecomment-423713449
Re: [Github-comments] [geany/geany-plugins] GeanyVC: support subversion authenticated commit (#777)
ok, well, might as well just configure `--username` and `--passwd` on the commands, oh wait, its not configurable :( -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/issues/777#issuecomment-423687221
Re: [Github-comments] [geany/geany-plugins] GeanyVC: support subversion authenticated commit (#777)
@elextr the command is not run in a TTY, there's no way for the user to interact -- or that would be on the terminal from which Geany was launched at best, if stdin was inherited. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/issues/777#issuecomment-423679117
Re: [Github-comments] [geany/geany-plugins] GeanyVC: support subversion authenticated commit (#777)
Can't you just remove the `--non-interactive` to have svn prompt for credentials? By the nature of an IDE Geany is totally insecure, better to have handle usernames and passwords. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/issues/777#issuecomment-423669635
Re: [Github-comments] [geany/geany-plugins] GeanyVC: support subversion authenticated commit (#777)
Not in the SVN working directory, but `svn` can save user credentials under several files in the `~/.subversion/auth` directory tree. However I guess it might be tricky to parse them all and get the right credentials to use for the current working directory commit. Moreover, to save subversion passwords in that way is not very safe. I think it should be simpler to ask for credentials each time we need them, or make the user store them in Geany preferences. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/issues/777#issuecomment-423580656
Re: [Github-comments] [geany/geany-plugins] GeanyVC: support subversion authenticated commit (#777)
Can't username and password be set somewhere in a SVN configuration file? -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/issues/777#issuecomment-423577652
Re: [Github-comments] [geany/geany-plugins] GeanyVC: support subversion authenticated commit (#777)
I guess that in order to achieve this, the plugin should start managing interactive usage... or it would be solved by getting the error code which is returned if unauthorized (1) and try again by asking username/password to the user and issuing the command `svn commit --user USERNAME --passwd PASSWORD --non-interactive -m "MESSAGE" FILE_LIST ` `svn` exits with error code "1" in that case, and outputs on stderr this string: > svn: E215004: Authentication failed and interactive prompting is disabled; > see the --force-interactive option > svn: E215004: Commit failed (details follow): > svn: E215004: No more credentials or we tried too many times. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/issues/777#issuecomment-423567158
Re: [Github-comments] [geany/geany-plugins] GeanyLua: update for new Scintilla 3.10 (#776)
@Skif-off thanks for the review, and sorry not to have given you the time to submit yours ^^ -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/776#issuecomment-423538642
Re: [Github-comments] [geany/geany-plugins] GeanyLua: update for new Scintilla 3.10 (#776)
Merged #776 into master. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/776#event-1860240801
Re: [Github-comments] [geany/geany-plugins] GeanyVC: support subversion authenticated commit (#777)
GeanyVC uses the real `svn` commands under the hood, so it should be possible to get this to work. I am not knowledgeable about SVN authenticated commits; what would you do to get this to work on the command line? GeanyVC uses `svn commit --non-interactive -m "MESSAGE" FILE_LIST` -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/issues/777#issuecomment-423538017
Re: [Github-comments] [geany/geany] Extending the function of "Open selected file" (#1959)
Why not use GeanPy, Peasy or GeanyLua? And OS will do everything itself... -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/1959#issuecomment-423521325
Re: [Github-comments] [geany/geany-plugins] GeanyLua: update for new Scintilla 3.10 (#776)
@b4n Thank you for PR! I also did it two-three of weeks ago and got the same changes. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/776#issuecomment-423516667
[Github-comments] [geany/geany-plugins] GeanyVC: support subversion authenticated commit (#777)
I just started to use Geany, coming from Bluefish... it's really a great tool and I wish to tank everyone contributing! I only noticed a problem when I try commit to a Subversion repository which needs the user to be authenticated. No problem wen I commit to a repository which does not require authentication. Here is the screenshot of a packet capture between client and SVN server which shows the issue:  I think to be unable to fix without help from someone. Thanks! -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/issues/777
Re: [Github-comments] [geany/geany] Update link to GTK+3 bundle for test builds to self-hosted version (#1931)
I restarted PR builds that were failing, they should be back we relevant results soon. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/1931#issuecomment-423471721
Re: [Github-comments] [geany/geany] Build: Could not resolve host: win32builder.gnome.org (#1930)
Closed #1930 via #1931. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/1930#event-1859686127
Re: [Github-comments] [geany/geany] Update link to GTK+3 bundle for test builds to self-hosted version (#1931)
Merged #1931 into master. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/1931#event-1859686116
[Github-comments] [geany/geany-plugins] GeanyLua: update for new Scintilla 3.10 (#776)
@Skif-off could you take a look, as you seem to be interested in GeanyLua? This change is required for GeanyLua to build against current Geany master. I'm not sure of the implication of the changes in messages arguments, but this is a GeanyLua API break as well so it might be alright anyway. You can view, comment on, or merge this pull request online at: https://github.com/geany/geany-plugins/pull/776 -- Commit Summary -- * geanylua: Update Scintilla symbols for Geany 1.34 -- File Changes -- M geanylua/glspi_sci.h (13) M geanylua/glspi_ver.h (2) -- Patch Links -- https://github.com/geany/geany-plugins/pull/776.patch https://github.com/geany/geany-plugins/pull/776.diff -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/776
Re: [Github-comments] [geany/geany-plugins] GeanyLua: glspi_keycmd.h: added "Strip Trailing Spaces" (#759)
Merged #759 into master. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/759#event-1859655846
