[Github-comments] Re: [geany/geany] Fix Mingw CI build (PR #3465)

2023-05-06 Thread Enrico Tröger via Github-comments
Merged #3465 into master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3465#event-9182702234 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Mingw-w64 CI fails (Issue #3461)

2023-05-06 Thread Enrico Tröger via Github-comments
Closed #3461 as completed via #3465. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3461#event-9182702272 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Make Python comment-out mark be `#` without space (Issue #3462)

2023-04-29 Thread Enrico Tröger via Github-comments
I thought about this a couple of days and still don't have a strong opinion on it. I think I've seen both variants (without and with space) on colleagues' screens who are using various other IDEs. In the end, it's just a matter of taste and whatever default we choose, it might not match the

[Github-comments] Re: [geany/geany] Fix building with meson in mingw (PR #3466)

2023-04-23 Thread Enrico Tröger via Github-comments
> I have added a basic mingw64 meson workflow. The CI needs permission to allow > that. Permission granted. Could you rebase on current master, https://github.com/geany/geany/pull/3438 has only just been merged to allow manual start of the workflow :). Thanks for the new workflow, this might

[Github-comments] Re: [geany/geany] Allow Dispatching CI Build Workflow Manually (PR #3438)

2023-04-23 Thread Enrico Tröger via Github-comments
Thanks! -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3438#issuecomment-1519042573 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Allow Dispatching CI Build Workflow Manually (PR #3438)

2023-04-23 Thread Enrico Tröger via Github-comments
Merged #3438 into master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3438#event-9074747806 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Fix building with meson in mingw (PR #3466)

2023-04-22 Thread Enrico Tröger via Github-comments
After a quick shot, the changes look well to me. Tested on native Windows and built fine. > If required, I can try to add the mingw meson build in CI and replace > autotools with meson also. Not sure if the Meson is already ready to be used for CI. Ideally, the CI builds generate full

[Github-comments] Re: [geany/geany] Fix Mingw CI build (PR #3465)

2023-04-20 Thread Enrico Tröger via Github-comments
@eht16 commented on this pull request. > @@ -70,19 +70,12 @@ CFLAGS="\ ARCH="x86_64" MINGW_ARCH="mingw64" HOST="x86_64-w64-mingw32" -export CC="/usr/bin/${HOST}-gcc" -export CPP="/usr/bin/${HOST}-cpp" -export CXX="/usr/bin/${HOST}-g++" -export AR="/usr/bin/${HOST}-ar" -export

[Github-comments] Re: [geany/geany] Fix Mingw CI build (PR #3465)

2023-04-20 Thread Enrico Tröger via Github-comments
@eht16 commented on this pull request. > @@ -153,7 +153,7 @@ geany_private.res: $(top_srcdir)/geany_private.rc libgeany_la_SOURCES += win32.c win32.h libgeany_la_LIBADD += -lole32 -lwsock32 -lcomdlg32 -libgeany_la_LDFLAGS += -Wl,-luuid -mwindows -mms-bitfields -no-undefined `-luuid` is

[Github-comments] Re: [geany/geany] Fix Mingw CI build (PR #3465)

2023-04-20 Thread Enrico Tröger via Github-comments
@eht16 commented on this pull request. > @@ -119,7 +119,6 @@ AC_DEFINE_UNQUOTED([GETTEXT_PACKAGE], ["$PACKAGE"], > [Gettext package.]) AM_COND_IF([MINGW], [pkgdatadir='${prefix}/data'], [pkgdatadir='${datarootdir}/geany']) -AM_COND_IF([MINGW],[LIBS="$LIBS -liconv"]) This came in

[Github-comments] Re: [geany/geany] Fix Mingw CI build (PR #3465)

2023-04-20 Thread Enrico Tröger via Github-comments
@eht16 commented on this pull request. > export NOCONFIGURE=1 export JOBS=${JOBS:-1} +export lt_cv_deplibs_check_method='pass_all' As suggested in https://github.com/geany/geany/issues/3461. This fixes linker errors on `libuuid` with the recent Mingw GTK3 package. -- Reply to this email

[Github-comments] Re: [geany/geany] Fix Mingw CI build (PR #3465)

2023-04-20 Thread Enrico Tröger via Github-comments
@eht16 pushed 1 commit. e465a2b456e7c0aba6aefc1c4b406b8089386d1a Remove unnecessary compiler and linker flags for Mingw builds -- View it on GitHub: https://github.com/geany/geany/pull/3465/files/9a8dd9aea503735ea56040efd66fd5f80b608ff6..e465a2b456e7c0aba6aefc1c4b406b8089386d1a You are

[Github-comments] [geany/geany] Fix Mingw CI build (PR #3465)

2023-04-20 Thread Enrico Tröger via Github-comments
You can view, comment on, or merge this pull request online at: https://github.com/geany/geany/pull/3465 -- Commit Summary -- * Define lt_cv_deplibs_check_method to fix Mingw linker errors -- File Changes -- M scripts/ci_mingw64_geany.sh (1) -- Patch Links --

[Github-comments] Re: [geany/geany] Mingw-w64 CI fails (Issue #3461)

2023-04-17 Thread Enrico Tröger via Github-comments
See https://github.com/msys2/MINGW-packages/issues/16836 -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3461#issuecomment-1512130925 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] [filetypes] Update Kotlin basic types (PR #3456)

2023-04-16 Thread Enrico Tröger via Github-comments
Merged #3456 into master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3456#event-9018351496 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Mingw-w64 CI fails (Issue #3461)

2023-04-16 Thread Enrico Tröger via Github-comments
Not yet. I debugged it a bit and noticed that some change in MSYS2 in the last few weeks causes that no shared libraries (in this case .dlls) are built anymore. Still the `make` succeeds without error. I'm not sure what's going on. https://github.com/geany/geany/pull/3432 was also my first

[Github-comments] Re: [geany/geany] CI: Define JOBS environment variable and use it for make (PR #3432)

2023-04-16 Thread Enrico Tröger via Github-comments
Merged #3432 into master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3432#event-9017623265 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/infrastructure] CI: Add workflow to cleanup old untagged CI container image versions (PR #9)

2023-04-16 Thread Enrico Tröger via Github-comments
Merged #9 into master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/infrastructure/pull/9#event-9017535396 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] MSYS2/MinGW-w64: Missing Header 'alloca.h' (Issue #3437)

2023-04-16 Thread Enrico Tröger via Github-comments
That you are using Meson is an important bit of information. I can reproduce the error when using Meson on Windows/MSYS2. And this is because our Meson setup does *not* `WIN32`. The following patch defines it and the build succeeds: ```diff diff --git a/meson.build b/meson.build index

[Github-comments] Re: [geany/geany] Fold code to certain level only (Issue #3041)

2023-04-16 Thread Enrico Tröger via Github-comments
Not exactly what requested but you can collapse a top-level fold point and expand only the next level when holding the Shift key while expanding a fold point (details at https://www.geany.org/manual/#folding). At least, this makes it easy to expand the first and second level on one fold point

[Github-comments] Re: [geany/geany] [filetypes] Update Kotlin basic types (PR #3456)

2023-04-16 Thread Enrico Tröger via Github-comments
@dolik-rce fine by you? If so, let's merge. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3456#issuecomment-1510377010 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Update lexilla to 5.2.4 (PR #3441)

2023-04-16 Thread Enrico Tröger via Github-comments
I wonder if it is OK to just update Lexilla and not Scintilla as well? But probably better than nothing and we can update Scintilla afterwards anyway. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3441#issuecomment-1510374543 You are receiving this

[Github-comments] Re: [geany/geany] Filetype files documenting lexer property settings (Issue #3442)

2023-04-16 Thread Enrico Tröger via Github-comments
Agreed. And obviously I worked on this already before, as mentioned in #3441. What if we use the code to generate an up2date list in https://github.com/geany/geany/issues/2517 and then copy the generated property definitions into the according filetypes files? -- Reply to this email directly

[Github-comments] Re: [geany/www.geany.org] Add AppImage to Third Party packages (PR #42)

2023-04-16 Thread Enrico Tröger via Github-comments
Merged #42 into master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/www.geany.org/pull/42#event-9017302572 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] [geany/www.geany.org] Replace flake8 by ruff (PR #47)

2023-04-16 Thread Enrico Tröger via Github-comments
You can view, comment on, or merge this pull request online at: https://github.com/geany/www.geany.org/pull/47 -- Commit Summary -- * Replace Flake8 and Isort by Ruff for linting * Move PyLint config to pyproject.toml * Add Python 3.11 for linting jobs, remove Python 3.7 and 3.8 --

[Github-comments] Re: [geany/geany-plugins] Plugin Request: source control gutter indicators (Issue #1244)

2023-03-27 Thread Enrico Tröger via Github-comments
Or just use the existing plugin https://github.com/geany/geany-plugins/tree/master/git-changebar :). This does exactly what you request. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/issues/1244#issuecomment-1485564709 You are receiving this

[Github-comments] Re: [geany/geany] MSYS2/MinGW-w64: Missing Header 'alloca.h' (Issue #3437)

2023-03-23 Thread Enrico Tröger via Github-comments
My local MSYS2 setup on a native Windows system does *not* have `alloca.h` nor does the CI build image has it (https://github.com/geany/infrastructure/pkgs/container/geany-mingw64-ci) and both cleanly build Geany. How do you start your build? Is the `gcc` command you posted above the same

[Github-comments] [geany/infrastructure] CI: Add workflow to cleanup old untagged CI container image versions (PR #9)

2023-03-19 Thread Enrico Tröger via Github-comments
The weekly rebuild of the CI container image for the Mingw64 builds works fine. But the previous image versions are kept as untagged images in the container registry. This workflow should remove all untagged image versions except that last five ones. It is schedules weekly two hours after the

[Github-comments] Re: [geany/geany-plugins] Add mingw-w64 (Windows) CI build (PR #1201)

2023-03-19 Thread Enrico Tröger via Github-comments
@eht16 pushed 4 commits. db7b0d8a03e644977b9187489f23c03c760fd53f CI: Add Windows build ad222a5667f6607213e6ada7c0d58832a57bada2 debug... 58baa1d0e89e1aadec067c857622adc3463983fb CI: Use "JOBS" environment variable for make if set 43f1b202b637c81cb5af6a6bf2f1e4728a8fc6c6 CI: Fix copied Geany

[Github-comments] [geany/geany] CI: Define JOBS environment variable and use it for make (PR #3432)

2023-03-19 Thread Enrico Tröger via Github-comments
I just noticed in the CI workflow for the traditional Autotools based jobs, we did not specify -j for parallel job execution for the main make call to build all the sources. We did however for make check and make distcheck. If there is no reason for not using -j 2, this PR adds it and

[Github-comments] Re: [geany/geany-plugins] Add mingw-w64 (Windows) CI build (PR #1201)

2023-03-16 Thread Enrico Tröger via Github-comments
@eht16 pushed 3 commits. fdbb29d4f8623e3adff69e6668eaab11b04484f3 CI: Add Windows build e0da9a3df7477cb5c878656ee54396ded8604e5f Windows: Update dependency removal cleanup on uninstall 76b11fbbbd7f5ae3f08c325d573e22c6cedbcf75 debug... -- View it on GitHub:

[Github-comments] Re: [geany/geany-plugins] Remove unusable plugins: DevHelp, GeanyPy, Multiterm (PR #1217)

2023-03-16 Thread Enrico Tröger via Github-comments
Merged #1217 into master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/1217#event-8772067041 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany-plugins] Add mingw-w64 (Windows) CI build (PR #1201)

2023-03-12 Thread Enrico Tröger via Github-comments
@eht16 pushed 1 commit. 97c38e6bd818c486f922e564626a3765ca723358 debug... -- View it on GitHub: https://github.com/geany/geany-plugins/pull/1201/files/682bac9d94de36ca08913bb7ae70360162da6ea4..97c38e6bd818c486f922e564626a3765ca723358 You are receiving this because you are subscribed to this

[Github-comments] Re: [geany/geany-plugins] Add mingw-w64 (Windows) CI build (PR #1201)

2023-03-11 Thread Enrico Tröger via Github-comments
@eht16 pushed 1 commit. 682bac9d94de36ca08913bb7ae70360162da6ea4 Windows: Update dependency removal cleanup on uninstall -- View it on GitHub: https://github.com/geany/geany-plugins/pull/1201/files/306f84c4f882e873f1aa68681cc41a3edb9416f0..682bac9d94de36ca08913bb7ae70360162da6ea4 You are

[Github-comments] Re: [geany/geany-plugins] Add mingw-w64 (Windows) CI build (PR #1201)

2023-03-09 Thread Enrico Tröger via Github-comments
@eht16 pushed 1 commit. 306f84c4f882e873f1aa68681cc41a3edb9416f0 CI: Add Windows build -- View it on GitHub: https://github.com/geany/geany-plugins/pull/1201/files/0d36c1d2d37cb909bdf118e538cbdfa28fe5f2cb..306f84c4f882e873f1aa68681cc41a3edb9416f0 You are receiving this because you are

[Github-comments] Re: [geany/geany-plugins] Add mingw-w64 (Windows) CI build (PR #1201)

2023-03-09 Thread Enrico Tröger via Github-comments
@eht16 pushed 1 commit. 2c7bc518b1866be55d6cf9cd489d6e7f97d93379 Merge branch 'master' into ci_add_windows_build -- View it on GitHub: https://github.com/geany/geany-plugins/pull/1201/files/655eeaca7daae67a912c7093fc7fcd28b847c883..2c7bc518b1866be55d6cf9cd489d6e7f97d93379 You are receiving

[Github-comments] Re: [geany/geany-plugins] Remove unusable plugins: DevHelp, GeanyPy, Multiterm (PR #1217)

2023-03-07 Thread Enrico Tröger via Github-comments
@b4n Alright, Webhelper is back in the game. I would like to restart the "merge in about a week counter" to get this done. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/1217#issuecomment-1459624766 You are receiving this because you are

[Github-comments] Re: [geany/geany-plugins] Remove unusable plugins: DevHelp, GeanyPy, Multiterm (PR #1217)

2023-03-07 Thread Enrico Tröger via Github-comments
@eht16 pushed 2 commits. a7c921ddfa31a0a6847d02b2bacf3698429baaa0 Remove unusable plugins: DevHelp, GeanyPy, Multiterm 819efce62614a8a16f8d27a49e057b95cce2aa4c Re-add WebHelper plugin -- View it on GitHub:

[Github-comments] Re: [geany/geany-plugins] Update GitHub workflow to Ubuntu 20.04 (PR #1239)

2023-03-07 Thread Enrico Tröger via Github-comments
@xiota before you are continuing, please note that most of the what you are trying to do is already done in: https://github.com/geany/geany-plugins/pull/1201/files#diff-5c3fa597431eda03ac3339ae6bf7f05e1a50d6fc7333679ec38e21b337cb6721L33 And the PR #1201 is only waiting for #1217 which will remove

[Github-comments] Re: [geany/geany] hi i try to sign in mailing list but not work (Issue #3398)

2023-02-21 Thread Enrico Tröger via Github-comments
> I think the ML approval bot is a human, they have sooo much downtime called > sleep, might take time to approve :-) Actually not. The ML usually do not need manual intervention except for a few moderation requests which are caused by users who used a sender address which is not registered or

[Github-comments] Re: [geany/geany-plugins] Remove unusable plugins: DevHelp, GeanyPy, Multiterm (PR #1217)

2023-02-12 Thread Enrico Tröger via Github-comments
> @eht16 well done for tricking me into giving it another shot ;) Though you > only got bare minimum resurrection from old work, but that doesn't actually > don't look that bad, not sure why I never submitted it. This wasn't completely intended by me but nice that it worked :). Thanks for

[Github-comments] Re: [geany/geany-plugins] Add bracket colors plugin (PR #1221)

2023-02-12 Thread Enrico Tröger via Github-comments
> @eht16 I incorporated your suggested changes. I'm still learning Github, I > apologize if I'm not navigating this pull request correctly sweat_smile (at > my work, I'm used to Gitlab) No worries, it's fine. Thanks for the changes. The PR looks good to me and I would merge it in a few days if

[Github-comments] Re: [geany/geany-plugins] Add bracket colors plugin (PR #1221)

2023-02-12 Thread Enrico Tröger via Github-comments
@eht16 commented on this pull request. > @@ -0,0 +1,35 @@ +# = I don't know if there is a better way :(. Asked primarily for curiosity. -- Reply to this email directly or view it on GitHub:

[Github-comments] Re: [geany/geany] Make cut/copy keybinding select word if no selection (PR #3391)

2023-02-12 Thread Enrico Tröger via Github-comments
I find this also confusing behaviour and don't think we should do this. @ntrel, what about adding a new keybinding which provides the desired behaviour instead? I'd say leave the new keybinding unset but then users can rebind it to Ctrl-C if they like. And all other users won't be confused. --

[Github-comments] Re: [geany/geany] Add recent filename on saving when real_path is not set (PR #3389)

2023-02-12 Thread Enrico Tröger via Github-comments
I agree with the changes. Adding newly opened, unsaved files to recent files does not make much sense. If nobody objects, I'd merge it in a few days. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3389#issuecomment-1427051607 You are receiving this

[Github-comments] Re: [geany/geany] Show modified MRU docs in red (PR #3390)

2023-02-12 Thread Enrico Tröger via Github-comments
Merged #3390 into master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3390#event-8498431551 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Show modified MRU docs in red (PR #3390)

2023-02-12 Thread Enrico Tröger via Github-comments
Nice! -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3390#issuecomment-1427048317 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Save As dialog can save the wrong document (#2599)

2023-02-12 Thread Enrico Tröger via Github-comments
Closed #2599 as completed via #3388. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/2599#event-8498406882 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Fix #2599 - Save As dialog can save the wrong document (PR #3388)

2023-02-12 Thread Enrico Tröger via Github-comments
Merged #3388 into master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3388#event-8498406834 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Fix #2599 - Save As dialog can save the wrong document (PR #3388)

2023-02-12 Thread Enrico Tröger via Github-comments
Tested and works great. I agree that even with #3395, this cannot hurt. There might be more cases where the current doc differs from the doc the Save As dialog were open. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3388#issuecomment-1427044849 You

[Github-comments] Re: [geany/geany] Open CLI docs only once main window is active again (PR #3395)

2023-02-12 Thread Enrico Tröger via Github-comments
When a file is opened from a remote instance while a modal is open, the `handle_input_filename()` is called as fast as the main loop allows. This causes high CPU usage until the main window gets the focus back and the idle function is removed. I'd say this doesn't make it better than before

[Github-comments] Re: [geany/geany-plugins] Add bracket colors plugin (PR #1221)

2023-02-05 Thread Enrico Tröger via Github-comments
@eht16 commented on this pull request. > +// nested bracket +orderStack.push(endPos); +} + +GetOrder(bracket) = orderStack.size() - 1; +} +} + + + +// - +void

[Github-comments] Re: [geany/geany] Feature Request: repeat command (Issue #3382)

2023-02-05 Thread Enrico Tröger via Github-comments
I don't mind if it is added as part of the GeanyMacro plugin or a separate plugin. But I think it doesn't fit too well into Geany core. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3382#issuecomment-1418089147 You are receiving this because you are

[Github-comments] Re: [geany/geany-plugins] Add bracket colors plugin (PR #1221)

2023-02-05 Thread Enrico Tröger via Github-comments
@eht16 requested changes on this pull request. Oh, and could you please add your plugin to the list in the top README? > @@ -0,0 +1,32 @@ +Color brackets, parenthesis, and braces +=== RestructuredText (the text format used here) requires that underlines are exactly as long as

[Github-comments] Re: [geany/geany-plugins] Add bracket colors plugin (PR #1221)

2023-02-05 Thread Enrico Tröger via Github-comments
Looks good! Thanks. > While "rainbow colours" sounds nice "Bracketcolors" is more descriptive, so a > user looking for it is more likely to find it. Full agree. @asifamin13 maybe you could just mention "rainbow colors" in the README, that might already suffice that users can find it also with

[Github-comments] Re: [geany/geany] Help -> Report a bug is a no-op on Ubuntu (Issue #3375)

2023-01-29 Thread Enrico Tröger via Github-comments
@jorisgeer check Preferences->Tools->Browser to see what is configured there. Additionally, you might have a look in Help->Debug Messages to see if there are any error messages after you have clicked one of the Help menu links. Feel free to re-open if it still won't work. -- Reply to this

[Github-comments] Re: [geany/geany] Help -> Report a bug is a no-op on Ubuntu (Issue #3375)

2023-01-29 Thread Enrico Tröger via Github-comments
Closed #3375 as completed. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3375#event-8382202665 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Make ellipsization configurable (#2112)

2023-01-29 Thread Enrico Tröger via Github-comments
Closed #2112 as completed. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/2112#event-8382098597 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Feature Request: Preference option for file display size (Issue #3344)

2023-01-29 Thread Enrico Tröger via Github-comments
Closed #3344 as completed via 4508f77a1150b37d5e16c3e7fc9c7223bc591e88. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3344#event-8382096513 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Make tab label length and window title length configurable (PR #3365)

2023-01-29 Thread Enrico Tröger via Github-comments
Merged #3365 into master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3365#event-8382096346 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] How reduce scrolling speed in Geany ? (#2768)

2023-01-22 Thread Enrico Tröger via Github-comments
The patch was merged in Scintilla so we can close this once the next Scintilla release is integrated in Geany. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/2768#issuecomment-1399525319 You are receiving this because you are subscribed to this

[Github-comments] Re: [geany/geany] Use gtk_show_uri_on_window() in utils_open_browser() by default (PR #3178)

2023-01-20 Thread Enrico Tröger via Github-comments
> @eht16 The mingw-w64 CI seems to recreate the Docker image from scratch. You > implemented a cache didn't you? I noticed that the prebuilt Docker image cannot be pulled from the registry and so it is rebuilt on each CI run. There is no cache itself for the Docker image but it is pushed to the

[Github-comments] Re: [geany/geany] Make tab label length and window title length configurable (PR #3365)

2023-01-20 Thread Enrico Tröger via Github-comments
Thanks for the changes, @00AR. LGTM. I will merge in a few days if there will be no objections. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3365#issuecomment-1398725155 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Add preference for backspace unindenting (PR #3355)

2023-01-20 Thread Enrico Tröger via Github-comments
I agree with @elextr: here it was just one commit, so I just used the big green button and pressed it! Thanks @coalsont! -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3355#issuecomment-1398715833 You are receiving this because you are subscribed to

[Github-comments] Re: [geany/geany] Add preference for backspace unindenting (PR #3355)

2023-01-20 Thread Enrico Tröger via Github-comments
Merged #3355 into master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3355#event-8323647436 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Geany is not save project state by disable linux or by crash or by "killall -9 geany". (#1334)

2023-01-20 Thread Enrico Tröger via Github-comments
Actually we have even more: https://github.com/geany/geany/pull/2114 saves the session when files are opened and closed, merged in 2019 and released with 1.37. This is probably pretty close to what the OP might have requested (it's hard to guess from the description...). -- Reply to this

[Github-comments] Re: [geany/geany-plugins] GeanyLua: Use luaL_register instead of luaL_openlib (PR #1210)

2023-01-15 Thread Enrico Tröger via Github-comments
Thank you! -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/1210#issuecomment-1383212166 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany-plugins] GeanyLua: Use luaL_register instead of luaL_openlib (PR #1210)

2023-01-15 Thread Enrico Tröger via Github-comments
Merged #1210 into master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/1210#event-8230091512 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Use constructor method calltip only if available (PR #3334)

2023-01-15 Thread Enrico Tröger via Github-comments
Merged #3334 into master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3334#event-8230062418 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] CI: Use MSYS2 for Windows build (PR #3315)

2023-01-15 Thread Enrico Tröger via Github-comments
Merged #3315 into master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3315#event-8230023077 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] CI: Use MSYS2 for Windows build (PR #3315)

2023-01-15 Thread Enrico Tröger via Github-comments
@eht16 pushed 1 commit. e2066d4f5f9600b0d8acd0ebeaeddebc93d1f15a Use Ubuntu 20.04 for Linux CI builds -- View it on GitHub: https://github.com/geany/geany/pull/3315/files/f146c2eb70715e94d1e31931cdebb08c86f81799..e2066d4f5f9600b0d8acd0ebeaeddebc93d1f15a You are receiving this because you are

[Github-comments] Re: [geany/infrastructure] Builders: Use the "pacman" package from Debian to avoid building it (PR #8)

2023-01-15 Thread Enrico Tröger via Github-comments
Merged #8 into master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/infrastructure/pull/8#event-8229970482 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany-plugins] GeanyLua: Use luaL_register instead of luaL_openlib (PR #1210)

2023-01-15 Thread Enrico Tröger via Github-comments
Did the changes work for you? With these changes, Geany crashes when using any of the example scripts. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/1210#issuecomment-1383200490 You are receiving this because you are subscribed to this thread.

[Github-comments] Re: [geany/geany-plugins] Remove unusable plugins: DevHelp, GeanyPy, Multiterm (PR #1217)

2023-01-15 Thread Enrico Tröger via Github-comments
It feels weird to delete plugins from @b4n, @kugel- and @codebrainz :upside_down_face: I guess you do not intend to update any of those plugins? -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/1217#issuecomment-1383195689 You are receiving

[Github-comments] Re: [geany/geany] How reduce scrolling speed in Geany ? (#2768)

2023-01-15 Thread Enrico Tröger via Github-comments
I did my best to create https://sourceforge.net/p/scintilla/bugs/2375/, @elextr and @nyamatongwe. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/2768#issuecomment-1383179337 You are receiving this because you are subscribed to this thread. Message

[Github-comments] Re: [geany/geany] How reduce scrolling speed in Geany ? (#2768)

2023-01-15 Thread Enrico Tröger via Github-comments
Reopened #2768. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/2768#event-8229775183 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] preference option for file display size (PR #3365)

2023-01-15 Thread Enrico Tröger via Github-comments
@eht16 requested changes on this pull request. Apart from the few comments, it looks very good. By accident I found https://github.com/geany/geany/issues/2112 which would be solved by this as well. Maybe you could add "closes #2112 and #3344" to the commit message and rephrase it a bit as

[Github-comments] [geany/geany-plugins] Remove unusable plugins: DevHelp, GeanyPy, Multiterm (PR #1217)

2023-01-10 Thread Enrico Tröger via Github-comments
DevHelp depends on unavailable WebKitGtk 1.x. GeanyPy and MultiTerm are GTK 2 only. After we agreed on this, I will take care of updating the website, closing related issues and maybe more references. You can view, comment on, or merge this pull request online at:

[Github-comments] Re: [geany/geany-plugins] Add mingw-w64 (Windows) CI build (PR #1201)

2023-01-10 Thread Enrico Tröger via Github-comments
@eht16 pushed 1 commit. 655eeaca7daae67a912c7093fc7fcd28b847c883 CI: Add Windows build -- View it on GitHub: https://github.com/geany/geany-plugins/pull/1201/files/912b2296837be844348f903855cfff793b79003e..655eeaca7daae67a912c7093fc7fcd28b847c883 You are receiving this because you are

[Github-comments] Re: [geany/geany-plugins] Add mingw-w64 (Windows) CI build (PR #1201)

2023-01-10 Thread Enrico Tröger via Github-comments
> Multiterm would appear to be GTK2 only > > https://github.com/geany/geany-plugins/blob/a8fd23125049d0a0ae8655e2bf95d3f447b3dacd/build/multiterm.m4#L6 > so should not be being CIed. Yes, but even if it is disabled (by `./configure` itself and with `--disable-plugin-multiterm` explicitly),

[Github-comments] Re: [geany/geany-plugins] Add mingw-w64 (Windows) CI build (PR #1201)

2023-01-10 Thread Enrico Tröger via Github-comments
@eht16 commented on this pull request. > @@ -127,8 +127,9 @@ jobs: libxml2-dev # spellcheck libenchant-dev +ä multiterm Yeah, typo on German keyboard with tired eyes :). -- Reply to this email directly or view it on GitHub:

[Github-comments] Re: [geany/geany-plugins] Add mingw-w64 (Windows) CI build (PR #1201)

2023-01-10 Thread Enrico Tröger via Github-comments
@eht16 pushed 1 commit. 912b2296837be844348f903855cfff793b79003e Install valac for multiterm -- View it on GitHub: https://github.com/geany/geany-plugins/pull/1201/files/5139c89911dcd032a30dbd8a4f2134dffcb4aa91..912b2296837be844348f903855cfff793b79003e You are receiving this because you are

[Github-comments] Re: [geany/geany-plugins] Add mingw-w64 (Windows) CI build (PR #1201)

2023-01-10 Thread Enrico Tröger via Github-comments
@eht16 pushed 1 commit. 5139c89911dcd032a30dbd8a4f2134dffcb4aa91 Install valac for multiterm -- View it on GitHub: https://github.com/geany/geany-plugins/pull/1201/files/ece3b2598467472b1bf2654e6fd7123705e79665..5139c89911dcd032a30dbd8a4f2134dffcb4aa91 You are receiving this because you are

[Github-comments] Re: [geany/geany] CI: Use MSYS2 for Windows build (PR #3315)

2023-01-08 Thread Enrico Tröger via Github-comments
I just cleaned up the commit history and removed temporary test settings, so this can be merged. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3315#issuecomment-1374928919 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany-plugins] Add mingw-w64 (Windows) CI build (PR #1201)

2023-01-08 Thread Enrico Tröger via Github-comments
I fully agree. I just cleaned up the commit history and removed temporary test settings. Though we need to merge https://github.com/geany/geany/pull/3315 before. There seems to be a new issue with the Linux build at "make distcheck" which might be related to the updated Ubuntu version (18.04 to

[Github-comments] Re: [geany/geany-plugins] Add mingw-w64 (Windows) CI build (PR #1201)

2023-01-08 Thread Enrico Tröger via Github-comments
@eht16 pushed 1 commit. ece3b2598467472b1bf2654e6fd7123705e79665 CI: Add Windows build -- View it on GitHub: https://github.com/geany/geany-plugins/pull/1201/files/fe58722af22b9fca07f44b05d9bc07c502f0b701..ece3b2598467472b1bf2654e6fd7123705e79665 You are receiving this because you are

[Github-comments] Re: [geany/geany] CI: Use MSYS2 for Windows build (PR #3315)

2023-01-08 Thread Enrico Tröger via Github-comments
@eht16 pushed 1 commit. f146c2eb70715e94d1e31931cdebb08c86f81799 CI: Use MSYS2 for Windows build -- View it on GitHub: https://github.com/geany/geany/pull/3315/files/92d1a4b7267ca87c355ee9438fb190cab21eaf10..f146c2eb70715e94d1e31931cdebb08c86f81799 You are receiving this because you are

[Github-comments] Re: [geany/geany] CI: Use MSYS2 for Windows build (PR #3315)

2023-01-08 Thread Enrico Tröger via Github-comments
@eht16 pushed 1 commit. 6cb951f3510ffb6b4d84b7c5d78010cd8ffe1f8f Remove test settings -- View it on GitHub: https://github.com/geany/geany/pull/3315/files/4bdd15c1aa86f916a8dc362c4d09af570c51b344..6cb951f3510ffb6b4d84b7c5d78010cd8ffe1f8f You are receiving this because you are subscribed to

[Github-comments] Re: [geany/geany-plugins] Add mingw-w64 (Windows) CI build (PR #1201)

2023-01-08 Thread Enrico Tröger via Github-comments
@eht16 pushed 1 commit. 7ee00ef8c67f11404773bc0333f9060a250df44b Remove test settings -- View it on GitHub: https://github.com/geany/geany-plugins/pull/1201/files/8e3ec88fbaad0427faa06f675bf0e62c69a6859c..7ee00ef8c67f11404773bc0333f9060a250df44b You are receiving this because you are

[Github-comments] Re: [geany/geany] Use constructor method calltip only if available (PR #3334)

2023-01-04 Thread Enrico Tröger via Github-comments
If nobody will stop me, I'm going to squash and merge this in a week. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3334#issuecomment-1371163944 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Add preference for backspace unindenting (PR #3355)

2023-01-04 Thread Enrico Tröger via Github-comments
Pong :). Nice try, @elextr but the theme used in the current docs is some Gnome Ubuntu default theme or something, long time ago the screenshots were made with my personal theme but @b4n and me agreed to use something more common and unmodified. @coalsont if you want to update the screenshots,

[Github-comments] Re: [geany/geany] Fix goto popup location (PR #3316)

2023-01-04 Thread Enrico Tröger via Github-comments
Before a final review or ideally merge of https://github.com/geany/geany-plugins/pull/1201 and https://github.com/geany/geany/pull/3315 is necessary to get the pipeline ready for GTK 3.24. -- Reply to this email directly or view it on GitHub:

[Github-comments] Re: [geany/geany] Problem with Dark Menus in MsgWindow Tab Compiler (Issue #3359)

2023-01-01 Thread Enrico Tröger via Github-comments
Closed #3359 as completed. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3359#event-8134504790 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Problem with Dark Menus in MsgWindow Tab Compiler (Issue #3359)

2023-01-01 Thread Enrico Tröger via Github-comments
Additional note: with the next Geany release (though no release date yet), we changed the default colors in #3013. Maybe see the PR details for the colors as inspiration. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3359#issuecomment-1368502256

[Github-comments] Re: [geany/www.geany.org] Document in which languages plugins can be written (PR #46)

2023-01-01 Thread Enrico Tröger via Github-comments
@eht16 pushed 1 commit. 1b18201fab1a2e14ff51cb4cfd38cb13fed9696b Document in which languages plugins can be written -- View it on GitHub: https://github.com/geany/www.geany.org/pull/46/files/026d336b7c0f3f30fe92ca10f9106671aae429da..1b18201fab1a2e14ff51cb4cfd38cb13fed9696b You are receiving

[Github-comments] Re: [geany/www.geany.org] [Documentation] Please specify which programming languages are permissible for geany on https://www.geany.org/support/plugins/ (Issue #45)

2023-01-01 Thread Enrico Tröger via Github-comments
Before the first anniversary of this issue, here we go: #46. -- Reply to this email directly or view it on GitHub: https://github.com/geany/www.geany.org/issues/45#issuecomment-1368487671 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] [geany/www.geany.org] Document in which languages plugins can be written (PR #46)

2023-01-01 Thread Enrico Tröger via Github-comments
Closes #45. You can view, comment on, or merge this pull request online at: https://github.com/geany/www.geany.org/pull/46 -- Commit Summary -- * Document in which languages plugins can be written -- File Changes -- M page_content/support/plugins.md (13) -- Patch Links --

[Github-comments] Re: [geany/geany-plugins] Addons: Fix Wordwrap wrong keybinding use (PR #1212)

2023-01-01 Thread Enrico Tröger via Github-comments
Merged #1212 into master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/1212#event-8134375323 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Use constructor method calltip only if available (PR #3334)

2023-01-01 Thread Enrico Tröger via Github-comments
> Looks good to me apart from the minor comments above. Thanks for the suggestions! -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3334#issuecomment-1368483354 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Use constructor method calltip only if available (PR #3334)

2023-01-01 Thread Enrico Tröger via Github-comments
@eht16 commented on this pull request. > @@ -1843,28 +1843,9 @@ static gint find_start_bracket(ScintillaObject *sci, > gint pos) } -static gchar *find_calltip(const gchar *word, GeanyFiletype *ft) +static GPtrArray *update_tags_for_constructor_calltip(GPtrArray *tags, GeanyFiletype *ft,

<    1   2   3   4   5   6   7   8   9   10   >