arrowd (Gleb Popov)
User

Projects

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Friday

  • Clear sailing ahead.

User Details

User Since
Aug 24 2015, 7:47 AM (178 w, 1 d)
Availability
Available

Recent Activity

Sat, Jan 19

arrowd added a comment to D18296: Add support for passing cursor information via URL parameters when running kioclient exec..

Ping? Any chance to get this into 5.15?

Sat, Jan 19, 8:29 AM · Plasma

Wed, Jan 16

arrowd requested review of D18296: Add support for passing cursor information via URL parameters when running kioclient exec..
Wed, Jan 16, 1:59 PM · Plasma

Tue, Jan 15

arrowd abandoned D17938: Add support for passing cursor information to KRun..

Right.

Tue, Jan 15, 4:21 PM · Frameworks
arrowd committed R40:1fa3a0e85e71: Add ability to get cursor data from URL query arguments, like file://foo.txt? (authored by arrowd).
Add ability to get cursor data from URL query arguments, like file://foo.txt?
Tue, Jan 15, 11:17 AM
arrowd closed D18099: Add ability to get cursor data from URL query arguments, like file://foo.txt?line=123&column=456.
Tue, Jan 15, 11:17 AM · Kate
arrowd added a comment to D18099: Add ability to get cursor data from URL query arguments, like file://foo.txt?line=123&column=456.

Can I push this?

Tue, Jan 15, 7:07 AM · Kate

Mon, Jan 14

arrowd added a comment to D5447: Only add auto-detected compilers to model if they actually exist.
In D5447#393144, @apol wrote:

Let's look at this the other way around, what are you trying to fix?

Mon, Jan 14, 6:51 PM
arrowd accepted D18216: Scratchpad: Validate actions when opening context menu.

Works great now, thanks!

Mon, Jan 14, 7:06 AM · KDevelop

Sat, Jan 12

arrowd committed R59:de4ebe3cc162: Fix "unused parameter/variable" warnings. Cleanup whitespace. Original patch by… (authored by arrowd).
Fix "unused parameter/variable" warnings. Cleanup whitespace. Original patch by…
Sat, Jan 12, 6:47 PM
arrowd committed R59:311ac83d93ff: Fix unused parameters warnings (authored by arrowd).
Fix unused parameters warnings
Sat, Jan 12, 6:47 PM
arrowd closed D10115: Fix unused parameters warnings.
Sat, Jan 12, 6:47 PM
arrowd added a comment to D17569: Allow configuring shortcut for Run Scratch action and add to Run menu.

I found the following minor issue:

When I just started KDevelop and open "Scratchpad" toolview, any action except "New Scratch" are greyed out. Even "Remove Scratch", as well as "Run Scratch". When I open any scratch, it turns back to normal, even if I again close all Sratchpad windows.

That's intended. The actions are all associated with the selected scratch in the toolview list. Switching the window to a different scratch also changes this selection, while closing a window keeps the selection.

Sat, Jan 12, 2:17 PM · KDevelop
arrowd added a comment to D17569: Allow configuring shortcut for Run Scratch action and add to Run menu.

But that's not related to your change, so please push it. Not sure where to, 5.3, I guess.

Sat, Jan 12, 8:54 AM · KDevelop
arrowd added a comment to D17569: Allow configuring shortcut for Run Scratch action and add to Run menu.

I found the following minor issue:

Sat, Jan 12, 8:50 AM · KDevelop
arrowd added inline comments to D5447: Only add auto-detected compilers to model if they actually exist.
Sat, Jan 12, 8:27 AM
arrowd updated the diff for D5447: Only add auto-detected compilers to model if they actually exist.

Restore another check.

Sat, Jan 12, 8:26 AM

Fri, Jan 11

arrowd added a comment to D14391: Fix random order of plugin's config pages.

The problem with current revision is that pages added in uicontroller.cpp:530 aren't sorted.

Fri, Jan 11, 1:23 PM · KDevelop

Thu, Jan 10

arrowd updated the diff for D14391: Fix random order of plugin's config pages.

Fix docs, rename append* methods to add*.

Thu, Jan 10, 3:02 PM · KDevelop
arrowd updated the diff for D14391: Fix random order of plugin's config pages.

Remove insert API and use append everywhere.

Thu, Jan 10, 2:35 PM · KDevelop
arrowd commandeered D14391: Fix random order of plugin's config pages.
Thu, Jan 10, 2:31 PM · KDevelop
arrowd added inline comments to D5447: Only add auto-detected compilers to model if they actually exist.
Thu, Jan 10, 1:23 PM

Tue, Jan 8

arrowd added a comment to D17858: clang: Also detect Clang builtin dirs at runtime on Unix.

Sorry for being late to the party. Checked this on FreeBSD - no regressions.

Tue, Jan 8, 5:08 PM · KDevelop
arrowd committed R32:16c8d3c661a7: GrepView: Extend default file extensions to search (authored by arrowd).
GrepView: Extend default file extensions to search
Tue, Jan 8, 4:57 PM
arrowd closed D17892: GrepView: Extend default file extensions to search.
Tue, Jan 8, 4:57 PM · KDevelop
arrowd committed R32:b53ea9aa452e: Meson plugin: First working version (authored by dmensinger).
Meson plugin: First working version
Tue, Jan 8, 4:52 PM
arrowd closed D17278: Meson plugin: First working version.
Tue, Jan 8, 4:52 PM · Documentation, KDevelop
arrowd updated the diff for D18099: Add ability to get cursor data from URL query arguments, like file://foo.txt?line=123&column=456.

Change the type from uint to int.

Tue, Jan 8, 4:34 PM · Kate
arrowd added inline comments to D5447: Only add auto-detected compilers to model if they actually exist.
Tue, Jan 8, 4:31 PM
arrowd updated the diff for D5447: Only add auto-detected compilers to model if they actually exist.

Restore a check.

Tue, Jan 8, 4:30 PM
arrowd added a comment to D18099: Add ability to get cursor data from URL query arguments, like file://foo.txt?line=123&column=456.

+1, but it would be good if we could share this code...

Tue, Jan 8, 3:29 PM · Kate
arrowd requested review of D18099: Add ability to get cursor data from URL query arguments, like file://foo.txt?line=123&column=456.
Tue, Jan 8, 3:25 PM · Kate

Sat, Jan 5

arrowd added a comment to D17938: Add support for passing cursor information to KRun..

I don't know, "cursorRow" and "cursorColumn" are pretty editor-specific. I'm not sure they make sense in the KRun API.

Sat, Jan 5, 5:26 PM · Frameworks
arrowd accepted D17525: Show "Move into Source" action in code menu.

Yep, sorry, everything's alright.

Sat, Jan 5, 9:33 AM · KDevelop
arrowd added inline comments to D5447: Only add auto-detected compilers to model if they actually exist.
Sat, Jan 5, 9:11 AM
arrowd accepted D17370: Add assistant to generate header guards.

Work great, good job!

Sat, Jan 5, 8:56 AM · KDevelop
arrowd accepted D17892: GrepView: Extend default file extensions to search.
Sat, Jan 5, 8:52 AM · KDevelop

Fri, Jan 4

arrowd accepted D17278: Meson plugin: First working version.

I also don't know why the icons don't show up. I have oriented myself on the code from the CMake plugin, and I don't know what I am doing different/wrong. Maybe KDevelop has to be installed in a system directory (/usr) for this to work?

Fri, Jan 4, 3:30 PM · Documentation, KDevelop
arrowd added a comment to D17278: Meson plugin: First working version.

Yep, that fixed the issue.

Fri, Jan 4, 1:22 PM · Documentation, KDevelop
arrowd committed R32:263a57379ef6: QuickOpen: Trim whitespace from input (authored by arrowd).
QuickOpen: Trim whitespace from input
Fri, Jan 4, 10:44 AM
arrowd closed D17885: QuickOpen: Trim whitespace from input.
Fri, Jan 4, 10:44 AM · KDevelop

Thu, Jan 3

arrowd added a comment to D17370: Add assistant to generate header guards.

It now compiles and unit test passes, however my own quick test revealed following problem.

Thu, Jan 3, 8:15 PM · KDevelop
arrowd added a comment to D17525: Show "Move into Source" action in code menu.

Strange. I don't have this menu item even with your patch.

Thu, Jan 3, 8:03 PM · KDevelop
arrowd accepted D17839: Update kdevelop app icon to latest breeze-icons version.

Looks good to me.

Thu, Jan 3, 7:02 PM · KDevelop
arrowd added a comment to D17525: Show "Move into Source" action in code menu.

Sorry for silly question, but where this change is reflected in KDevelop UI?

Thu, Jan 3, 6:57 PM · KDevelop
arrowd added a comment to D17278: Meson plugin: First working version.

Tried this out on FreeBSD with KDevelop master.

Thu, Jan 3, 6:46 PM · Documentation, KDevelop
arrowd added a comment to D15565: Fix bug 389060 (Heaptrack analysis keeps firing /usr/bin/plasmoidviewer).

Since 5.3.1 was released, can we get this in?

Thu, Jan 3, 6:11 PM · KDevelop
arrowd accepted D14391: Fix random order of plugin's config pages.

I have tried this out on master and it works for me. If no one objects in next few days, please push this.

Thu, Jan 3, 5:32 PM · KDevelop
arrowd updated the diff for D10115: Fix unused parameters warnings.

Refresh diff and address comments.

Thu, Jan 3, 4:34 PM
arrowd commandeered D10115: Fix unused parameters warnings.
Thu, Jan 3, 4:30 PM
arrowd updated the diff for D5447: Only add auto-detected compilers to model if they actually exist.

Pass absolute path instead of the compiler name to createCompiler(). Remove strange check that was breaking tests.

Thu, Jan 3, 2:26 PM
arrowd commandeered D5447: Only add auto-detected compilers to model if they actually exist.
Thu, Jan 3, 2:25 PM
arrowd closed T6685: Fix debugger unit test failures on CI as Resolved.

Judging from current CI output, this has been fixed.

Thu, Jan 3, 12:45 PM · KDevelop
arrowd added inline comments to D17794: Pkg-config plugin.
Thu, Jan 3, 12:38 PM · KDevelop
arrowd added inline comments to D17370: Add assistant to generate header guards.
Thu, Jan 3, 12:03 PM · KDevelop
arrowd added a reviewer for D17370: Add assistant to generate header guards: KDevelop.
Thu, Jan 3, 11:40 AM · KDevelop
arrowd requested review of D17938: Add support for passing cursor information to KRun..
Thu, Jan 3, 9:35 AM · Frameworks

Tue, Dec 25

arrowd added inline comments to D17794: Pkg-config plugin.
Tue, Dec 25, 7:33 AM · KDevelop

Dec 3 2018

arrowd added a comment to D17255: KDevelop: support for installing into a non-standard parallel prefix.

If the problem this patch solves is to have CMAKE_INSTALL_PREFIX to be set for newly-created projects, why not just add a configuration option to the CMake plugin?

Dec 3 2018, 4:45 AM · KDevelop

Oct 15 2018

arrowd added a comment to D16218: [KDevelop/Core]: safe signal-handler implementation.

Oh, misunderstanding! I simply don't have a full KF5 build environment on FreeBSD yet. I'm running TrueOS in a VM on a system that isn't exactly powerful, so I'm waiting until "Project Trident" is released with a full KF5 desktop option.

Oct 15 2018, 10:26 AM · KDevelop
arrowd added inline comments to D16218: [KDevelop/Core]: safe signal-handler implementation.
Oct 15 2018, 10:04 AM · KDevelop
arrowd added a comment to D16218: [KDevelop/Core]: safe signal-handler implementation.

Re: the testing I did on FreeBSD: that's with a dedicated demonstrator; I cannot currently build KDevelop5 there.

Oct 15 2018, 10:04 AM · KDevelop
arrowd added inline comments to D16218: [KDevelop/Core]: safe signal-handler implementation.
Oct 15 2018, 10:02 AM · KDevelop

Oct 6 2018

arrowd added inline comments to D15976: Fix KDevelop's detection for Visual Studio 2017 compiler and tools..
Oct 6 2018, 8:22 AM · KDevelop

Oct 4 2018

arrowd added a comment to D15895: Use CLANG_INCLUDE_DIRS for clang include dir.

FreeBSD+Clang 6 also works fine with this patch.

Oct 4 2018, 9:24 AM · KDevelop

Oct 2 2018

arrowd added a comment to D15895: Use CLANG_INCLUDE_DIRS for clang include dir.

Does it work with clang 6.0?

Oct 2 2018, 2:03 PM · KDevelop

Sep 28 2018

arrowd added a comment to D9344: [KDevelop] : [fixed] consistent use of the project name allowing to create multiple projects in a single source tree.
A proper solution would be to delete just the settings file, if overwriting is to be done.
Sep 28 2018, 8:30 AM · KDevelop
arrowd added a comment to D9344: [KDevelop] : [fixed] consistent use of the project name allowing to create multiple projects in a single source tree.

I don't quite get it. In which situation do we want to override the project (delete .kdev file), but do not clear .kdev dir?

Sep 28 2018, 7:04 AM · KDevelop
arrowd added a comment to T6262: Overriding an existing project should wipe .kdev4 directory.

(In any case I don't see how it could *not* be recreated given that it will hold the new settings file).

Sep 28 2018, 7:01 AM · KDevelop

Aug 22 2018

arrowd added a comment to D14432: Appwizard plugin. Add more repository information in the project file. .

IMHO we should change the code then to allow the respective vcs plugin used for the new project to write any additional data it wants/needs to store in the config, instead of this generic writing which is only currently needed for one system. No idea yet how this could be done, not looked at the code, just speaking in principle.

Aug 22 2018, 10:38 AM · KDevelop

Aug 14 2018

arrowd committed R32:bbcf0cfa29c6: Use "gmake" command in custommake plugin on FreeBSD. Fixes unit test. (authored by arrowd).
Use "gmake" command in custommake plugin on FreeBSD. Fixes unit test.
Aug 14 2018, 9:47 AM
arrowd closed D14731: Use "gmake" command in custommake plugin on FreeBSD. Fixes unit test..
Aug 14 2018, 9:47 AM · KDevelop

Aug 13 2018

arrowd added a comment to T9249: KDevelop's GDB pretty printer units tests fail due to non-UTF-8 default codec.

Yes, I think the issue is in the test itself. Thanks for poking with CI.

Aug 13 2018, 11:59 AM · FreeBSD, KDevelop, build.kde.org
arrowd added a comment to T9249: KDevelop's GDB pretty printer units tests fail due to non-UTF-8 default codec.

KDevelop tests on FreeBSD are disabled now due to hangs, but the last test run shows that python support is working OK. The test fails somewhere in the middle, though, but python gets initialized OK.

Aug 13 2018, 11:34 AM · FreeBSD, KDevelop, build.kde.org

Aug 11 2018

arrowd added a comment to D13905: Fix KTextEditor::MainWindow to outlive the KTextEditor::View objects.

Same for Linux: https://build.kde.org/view/KDevelop/job/KDevelop%20kdevelop%20kf5-qt5%20SUSEQt5.9/330/

Aug 11 2018, 10:24 AM · KDevelop
arrowd added a comment to D13905: Fix KTextEditor::MainWindow to outlive the KTextEditor::View objects.

It seems, after this change the test ktexteditorpluginintegration started to fail: https://build.kde.org/view/KDevelop/job/KDevelop%20kdevelop%20kf5-qt5%20FreeBSDQt5.10/57/testReport/junit/(root)/TestSuite/test_ktexteditorpluginintegration/

Aug 11 2018, 10:23 AM · KDevelop
arrowd committed R32:db57eb31c392: Fix `test_clang-duchain` by making the test expectedly fail. (authored by arrowd).
Fix `test_clang-duchain` by making the test expectedly fail.
Aug 11 2018, 9:56 AM
arrowd closed D14315: Fix `test_clang-duchain` by making the test expectedly fail..
Aug 11 2018, 9:55 AM · KDevelop
arrowd added a comment to D14731: Use "gmake" command in custommake plugin on FreeBSD. Fixes unit test..

It turned out that BSD make just don't need that flag, so add it conditionally.

Aug 11 2018, 9:49 AM · KDevelop
arrowd updated the diff for D14731: Use "gmake" command in custommake plugin on FreeBSD. Fixes unit test..

Address comments.

Aug 11 2018, 9:48 AM · KDevelop
arrowd updated the diff for D14731: Use "gmake" command in custommake plugin on FreeBSD. Fixes unit test..

Fix the test another way.

Aug 11 2018, 9:44 AM · KDevelop

Aug 10 2018

arrowd added a comment to D14731: Use "gmake" command in custommake plugin on FreeBSD. Fixes unit test..

Yay for more unit test fixes :)

Though... why does this change fix the unit test exactly? Without knowing more details, I am surprised that one cannot use the normal "make".

Aug 10 2018, 2:44 PM · KDevelop
arrowd requested review of D14731: Use "gmake" command in custommake plugin on FreeBSD. Fixes unit test..
Aug 10 2018, 1:51 PM · KDevelop
arrowd added a comment to D14315: Fix `test_clang-duchain` by making the test expectedly fail..

Another ping. Anything wrong with this change?

Aug 10 2018, 10:30 AM · KDevelop

Aug 6 2018

arrowd committed R32:a221c66cce12: Skip attach tests on FreeBSD due to bug in GDB. (authored by arrowd).
Skip attach tests on FreeBSD due to bug in GDB.
Aug 6 2018, 11:30 AM
arrowd closed D14484: Skip attach tests on FreeBSD due to bug in GDB..
Aug 6 2018, 11:30 AM · KDevelop

Jul 30 2018

arrowd updated the diff for D14484: Skip attach tests on FreeBSD due to bug in GDB..

Address comment.

Jul 30 2018, 2:10 PM · KDevelop
arrowd added inline comments to D14484: Skip attach tests on FreeBSD due to bug in GDB..
Jul 30 2018, 1:37 PM · KDevelop
arrowd updated the diff for D14484: Skip attach tests on FreeBSD due to bug in GDB..

Address comments.

Jul 30 2018, 1:36 PM · KDevelop
arrowd requested review of D14484: Skip attach tests on FreeBSD due to bug in GDB..
Jul 30 2018, 12:54 PM · KDevelop
arrowd committed R32:cfdaa12723cb: Fix two tests in test_gdb. (authored by arrowd).
Fix two tests in test_gdb.
Jul 30 2018, 8:14 AM
arrowd closed D14470: Fix two tests in test_gdb..
Jul 30 2018, 8:14 AM · KDevelop

Jul 29 2018

arrowd added a comment to D14470: Fix two tests in test_gdb..

RE line 960.

Jul 29 2018, 7:54 PM · KDevelop
arrowd requested review of D14470: Fix two tests in test_gdb..
Jul 29 2018, 7:50 PM · KDevelop
arrowd added a comment to T9249: KDevelop's GDB pretty printer units tests fail due to non-UTF-8 default codec.

Now the test says

Jul 29 2018, 10:06 AM · FreeBSD, KDevelop, build.kde.org
arrowd added a comment to D14315: Fix `test_clang-duchain` by making the test expectedly fail..

Ping?

Jul 29 2018, 8:47 AM · KDevelop

Jul 28 2018

arrowd added a comment to T9249: KDevelop's GDB pretty printer units tests fail due to non-UTF-8 default codec.

devel/gdb package has been updated to support UTF-8 in FreeBSD. With gdb-8.1_5 the test passes.

Jul 28 2018, 5:52 PM · FreeBSD, KDevelop, build.kde.org

Jul 26 2018

arrowd abandoned D13613: RFC: Fix libclang default include paths on FreeBSD..

Without @mwolff approval I can't close this, only abandon.

Jul 26 2018, 4:52 AM · KDevelop

Jul 25 2018

arrowd added a comment to T9249: KDevelop's GDB pretty printer units tests fail due to non-UTF-8 default codec.

For the rest seems this needs some FreeBSD experts to continue here, so I added the respective tag for a start while stepping back for now myself.

Jul 25 2018, 11:12 AM · FreeBSD, KDevelop, build.kde.org
arrowd added a comment to T9249: KDevelop's GDB pretty printer units tests fail due to non-UTF-8 default codec.

I think the problem is how FreeBSD packages GDB.

Jul 25 2018, 9:55 AM · FreeBSD, KDevelop, build.kde.org
arrowd added a comment to T9249: KDevelop's GDB pretty printer units tests fail due to non-UTF-8 default codec.

Aha, that indeed gives a clue:

Jul 25 2018, 6:38 AM · FreeBSD, KDevelop, build.kde.org