[Github-comments] Re: [geany/geany-plugins] updatechecker: Port to GIO and small other fixes (PR #1340)

2024-04-27 Thread elextr via Github-comments
@b4n: > Yet, does it currently matter? Well if you are willing to maintain them in sync its fine. But who knows if a security alert will require them to change libsoup3.1 API, its a web library after all! @xiota: > You're too far ahead. Takes a while to catch up. Just around longer, but

[Github-comments] Re: [geany/geany] Cannot install (Issue #3852)

2024-04-27 Thread elextr via Github-comments
The desktop specification requires desktop managers (like Cinnamon) to search the list of directories in `$XDG_DATA_DIRS` for desktop files and use the first found only to add to menus, two applications with the same name cannot show in any desktop spec compliant menu. Geany installs to

[Github-comments] [geany/geany] color priority/precedence (Issue #3860)

2024-04-27 Thread Sergio Abreu A. via Github-comments
An identifier inside a string should be colored as identifier, not as string. Probably it's related to the execution order applying the colors. Now the string scheme has more importance than identifier, and it should be the opposite. An example: `"Here is a $variable is inside a string"`

[Github-comments] Re: [geany/geany] Cannot install (Issue #3852)

2024-04-27 Thread ineuw via Github-comments
Installed Linux Mint 21.3 on a spare drive and after updating the iso installation, installed @rdipardo's script, and followed with the latest Geany installation script of 'Yage' (or 2.1.) This may be a repeat installation because Linux Mint Cinnamon menu did not display the results of

[Github-comments] Re: [geany/geany] Sync our ctags to the latest tag version (p6.1.20240421.0) (PR #3859)

2024-04-27 Thread Colomban Wendling via Github-comments
I'll look into this Soon™, but the JS changes might have an issue (might be what you see, might be something else), see [this](https://github.com/universal-ctags/ctags/commit/6d85089456ed215ce6b6a673744ae42ccc5e0e99#diff-eb186936271438240da764e7ccf36ffa04ba8f0a6e1eabd1c69400164ae9f608R1) which

[Github-comments] Re: [geany/geany] Sync our ctags to the latest tag version (p6.1.20240421.0) (PR #3859)

2024-04-27 Thread Jiří Techet via Github-comments
I also added a commit updating the matlab parser to the upstream regex-based one. I'd like to avoid the situation that we have to maintain our (non-regex) version of the parser like in: https://github.com/geany/geany/pull/3358 https://github.com/geany/geany/pull/3563 Also, the kinds generated

[Github-comments] Re: [geany/geany] Sync our ctags to the latest tag version (p6.1.20240421.0) (PR #3859)

2024-04-27 Thread Jiří Techet via Github-comments
@techee pushed 2 commits. aa0c796b5c1e6a1b6a4050086309a5864819eca3 Switch to uctags regex-based matlab parser d515bce2167fd8623c83dc51e38562a4b822d019 Use repoinfo.h with the used tag version -- View it on GitHub:

[Github-comments] Re: [geany/geany] Sync our ctags to the latest tag version (p6.1.20240421.0) (PR #3859)

2024-04-27 Thread Jiří Techet via Github-comments
@techee pushed 8 commits. 7943b7d9ec47805caac6b071ca13da4a73c14326 Add ldscript parser f21e1de08482a73fdc41694e7dbc81c9735a29dc Update parser kind mappings e00a2684d34446ae7426ee8e00349f8d3595e2fc Map title/subtitle of the rst parser cdf1aa1fbc7ba8d93dc3373a0c8600a80a4c03be Map defines to

[Github-comments] [geany/geany] Sync our ctags to the latest tag version (p6.1.20240421.0) (PR #3859)

2024-04-27 Thread Jiří Techet via Github-comments
There were mostly expected problems like new source files that had to be added, rename of `perl6.c` to `raku.c`, update of kind mappings, etc. The only unexpected thing is the behavior of the javascript parser - this is from the javascript commit message: There are lots of differences because

[Github-comments] Re: [geany/geany-plugins] codenav/goto_file: Fix use-after-free crash (PR #1339)

2024-04-27 Thread Colomban Wendling via Github-comments
@b4n approved this pull request. LGBI, makes sense. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/1339#pullrequestreview-2026699270 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany-plugins] Various small WebHelper improvements (PR #1295)

2024-04-27 Thread Colomban Wendling via Github-comments
Merged #1295 into master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/1295#event-12632694851 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany-plugins] updatechecker: Port to GIO and small other fixes (PR #1340)

2024-04-27 Thread Colomban Wendling via Github-comments
> Yeah, but then its back to four plugins to synchronise on > libsoup3/webkit2gtk-4.1. Yet, does it currently matter? I mean, the code is ready (but for more testing maybe) for all four now, porting isn't what is holding back anymore. Sure the question might arise again for a future libsoup4

[Github-comments] Re: [geany/geany-plugins] geniuspaste: Port to libsoup3 (and avoid GTK3 deprecations) (PR #1342)

2024-04-27 Thread Colomban Wendling via Github-comments
> It didn't appear to do either. No popup or message in status or msgwin on > success. I don't recall seeing anything but some usual gtk errors in terminal. Weird… unless you have a libsoup2 plugin loaded? IIUC nothing will stop you from doing so, but things will just perform bizarrely (or

[Github-comments] Re: [geany/geany-plugins] geniuspaste: Port to libsoup3 (and avoid GTK3 deprecations) (PR #1342)

2024-04-27 Thread xiota via Github-comments
> it uses Geany's own `utils_open_browser()`, so that suggests this doesn't > work for you either? I had seen the browser setting, but don't know what actually uses it. Same with the terminal command. What actually uses it? I also often have ... "send selection to" not working until I

[Github-comments] Re: [geany/geany-plugins] updatechecker: Port to GIO and small other fixes (PR #1340)

2024-04-27 Thread xiota via Github-comments
> has again pointed out the point I made a while back about this I realized later that you'd already said it. You're too far ahead. Takes a while to catch up. > four plugins to synchronise This number could be reduced... -- Reply to this email directly or view it on GitHub:

[Github-comments] Re: [geany/geany-plugins] projectorganizer: Add popup panel for navigation (PR #1341)

2024-04-27 Thread Colomban Wendling via Github-comments
@b4n requested changes on this pull request. LGTM, see inline comments. Apart from that, I was kind of disappointed that the "goto file" only lists Geany open documents rather than the project's documents -- maybe you'd like to add that? Also, I hoped to be able to go to a specific file's

[Github-comments] Re: [geany/geany-plugins] Consider to add a plugin to support Asciidoctor (Issue #1282)

2024-04-27 Thread elextr via Github-comments
Lets move the build tool discussion elsewhere rather than polluting this FR which has pointed to your plugin, but users have to compile it themselves. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/issues/1282#issuecomment-2080655894 You are

[Github-comments] Re: [geany/geany-plugins] updatechecker: Port to GIO and small other fixes (PR #1340)

2024-04-27 Thread elextr via Github-comments
> just keep using the libsoup implementation Yeah, but then its back to four plugins to synchronise on libsoup3/webkit2gtk-4.1. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/1340#issuecomment-2080645295 You are receiving this because you

[Github-comments] Re: [geany/geany-plugins] updatechecker: Port to GIO and small other fixes (PR #1340)

2024-04-27 Thread Colomban Wendling via Github-comments
@elextr hence checking if it does :) But if it's a concern, just keep using the libsoup implementation, it *ought* to work with HTTP as that's the whole point of that library :smiley: -- Reply to this email directly or view it on GitHub:

[Github-comments] Re: [geany/geany-plugins] geniuspaste: Port to libsoup3 (and avoid GTK3 deprecations) (PR #1342)

2024-04-27 Thread Colomban Wendling via Github-comments
> Do Debian and Ubuntu even bother rebuilding packages for old releases? No, the dependency only matter for users of older releases that would like to build a newer Geany by themselves. And Debian was just an example, sometimes we check what even more slow-paced distro do, like RHEL or so.

[Github-comments] Re: [geany/geany-plugins] geniuspaste: Port to libsoup3 (and avoid GTK3 deprecations) (PR #1342)

2024-04-27 Thread Colomban Wendling via Github-comments
> After testing the plugin, I think it should be dropped. Most of the pastebins > don't work. Indeed only 3 did in my tests (Geany's, Debian's and Sprunge) > There's no confirmation of what it's sending. […] Once data is sent, there's > no way to delete it, which is a security risk if

[Github-comments] Re: [geany/geany-plugins] geniuspaste: Port to libsoup3 (and avoid GTK3 deprecations) (PR #1342)

2024-04-27 Thread elextr via Github-comments
> Do Debian and Ubuntu even bother rebuilding packages for old releases? Bugfixes, security fixes and those sorts of things, but rarely new versions. To be honest @xiota has a point, if the plugin causes constraints on others we need to look at what value it adds, and if it only works for one

[Github-comments] Re: [geany/geany-plugins] updatechecker: Port to GIO and small other fixes (PR #1340)

2024-04-27 Thread elextr via Github-comments
@xiota: > (win/mac – the platforms most likely to need update checkers). has again pointed out the point I made a while back about this, its useful for Windows and Macos, but for Linux it lies, just because the project releases a new Geany does not put it in the distro they are using. And for

[Github-comments] Re: [geany/geany-plugins] LaTeX: Switch handling of insert (list) environments over to snippets workflow (PR #1330)

2024-04-27 Thread Frank Lanitz via Github-comments
@frlan pushed 1 commit. 5fb3c82fa0c7024cfa78586afa4da0f3b279f019 Propper NULL-terminate strings and fix an trailing space inside a template -- View it on GitHub:

[Github-comments] Re: [geany/geany-plugins] geniuspaste: Port to libsoup3 (and avoid GTK3 deprecations) (PR #1342)

2024-04-27 Thread xiota via Github-comments
> not available in Debian before Bookworm (12, current stable) Do Debian and Ubuntu even bother rebuilding packages for old releases? After testing the plugin, I think it should be dropped. Most of the pastebins don't work. There's no confirmation of what it's sending. Half the time,

[Github-comments] Re: [geany/geany-plugins] updatechecker: Port to GIO and small other fixes (PR #1340)

2024-04-27 Thread Colomban Wendling via Github-comments
@techee any chance you could check whether gvfs works with HTTP(S) under macos? -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/1340#issuecomment-2080478237 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany-plugins] geniuspaste: Port to libsoup3 (and avoid GTK3 deprecations) (PR #1342)

2024-04-27 Thread Colomban Wendling via Github-comments
Thanks for review & testing! BTW, a few things things to note: * libsoup3 is fairly recent (2021-09-18), and e.g. not available in Debian before Bookworm (12, current stable) * I probably won't make it possible to support both libsoup2 and libsoup3, because the changes are too complex which