arrowd (Gleb Popov)
User

Projects

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Saturday

  • Clear sailing ahead.

User Details

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

Recent Activity

May 3 2023

arrowd added inline comments to D20217: Remove kwallet4 support.
May 3 2023, 9:03 AM · Plasma

Aug 19 2022

arrowd abandoned D1523: Project Manager View plugin has hardcoded/fixed shortcuts.

Doesn't seem to be relevant anymore.

Aug 19 2022, 8:37 PM · KDevelop
arrowd commandeered D1523: Project Manager View plugin has hardcoded/fixed shortcuts.
Aug 19 2022, 8:36 PM · KDevelop

Aug 29 2021

arrowd abandoned D18758: Make Project::open() method use async KIO methods..
In D18758#678707, @apol wrote:

Let's abandon it, I'd be wary of bringing in such a big dependency into our codebase just like that.

Aug 29 2021, 1:33 PM · KDevelop

Aug 26 2021

arrowd added a comment to D18758: Make Project::open() method use async KIO methods..

So, do we want to get this in?

Aug 26 2021, 4:54 PM · KDevelop

Mar 31 2021

arrowd abandoned D17816: Support for xattrs on kio copy/move.

Mark the revision as closed to reflect reality.

Mar 31 2021, 4:45 AM · Frameworks

Mar 2 2021

arrowd added a comment to D17816: Support for xattrs on kio copy/move.

I was wondering why copying files in Krusader or Dolphin from a vfat-formatted memory card stopped working for me after update. After copying the first file, the transfer stopped progressing and the process ended up consuming 100% CPU forever. Later I discovered that the cause of the breakage was this patch: If fgetxattr() fails with ENOTSUP, the code loops indefinitely calling fgetxattr().

The following hotfix made copying of files in Krusader work again for me:
...

However, the code might need bigger changes to cover all the cases. I do not fully understand the (IMO over-complicated) loops around flistxattr() and fgetxattr(). Note that the fact that one of them uses while (true) { ... } whereas the other one uses do { ... } while (true) does not improve code readability either.

Mar 2 2021, 11:58 AM · Frameworks

Nov 17 2020

arrowd added inline comments to D17816: Support for xattrs on kio copy/move.
Nov 17 2020, 7:08 AM · Frameworks

Oct 30 2020

arrowd added inline comments to D17816: Support for xattrs on kio copy/move.
Oct 30 2020, 5:12 AM · Frameworks

Oct 29 2020

arrowd added inline comments to D17816: Support for xattrs on kio copy/move.
Oct 29 2020, 5:32 PM · Frameworks
arrowd updated the diff for D17816: Support for xattrs on kio copy/move.
  • Put FileProtocol::copyXattrs inside a #if
Oct 29 2020, 5:31 PM · Frameworks

Oct 19 2020

arrowd added a comment to T13779: Implement move operations in ReferencedTopDUContext and std::move referenced contexts when possible.

AFAIK, Phabricator is being retired in favor of GitLab. Why not create an issue there? https://invent.kde.org/kdevelop/kdevelop/-/issues

Oct 19 2020, 7:06 AM · KDevelop

Oct 17 2020

arrowd added a comment to D17816: Support for xattrs on kio copy/move.

Another bump.

Oct 17 2020, 5:10 PM · Frameworks

Sep 21 2020

arrowd updated the diff for D17816: Support for xattrs on kio copy/move.
  • Address comments.
Sep 21 2020, 7:12 AM · Frameworks

Sep 17 2020

arrowd added inline comments to D17816: Support for xattrs on kio copy/move.
Sep 17 2020, 5:26 AM · Frameworks
arrowd updated the diff for D17816: Support for xattrs on kio copy/move.
  • More const QString.
Sep 17 2020, 5:26 AM · Frameworks
arrowd added inline comments to D17816: Support for xattrs on kio copy/move.
Sep 17 2020, 4:44 AM · Frameworks

Sep 10 2020

arrowd updated the diff for D17816: Support for xattrs on kio copy/move.
  • Make checkXattrFsSupport accept directory as a parameter.
  • Consitify QStrings.
Sep 10 2020, 11:18 AM · Frameworks

Sep 6 2020

arrowd updated the diff for D17816: Support for xattrs on kio copy/move.
  • Return "true" if the file doesn't have any xattrs.
Sep 6 2020, 6:28 PM · Frameworks

Aug 25 2020

arrowd added a comment to D17816: Support for xattrs on kio copy/move.

Bump.

Aug 25 2020, 1:51 PM · Frameworks

Jul 30 2020

arrowd updated the diff for D17816: Support for xattrs on kio copy/move.

Rebase on master.

Jul 30 2020, 9:00 AM · Frameworks

Jun 30 2020

arrowd added inline comments to D17816: Support for xattrs on kio copy/move.
Jun 30 2020, 8:08 AM · Frameworks
arrowd updated the diff for D17816: Support for xattrs on kio copy/move.
  • Remove extraneous debugging output.
Jun 30 2020, 8:07 AM · Frameworks

Jun 29 2020

arrowd updated the diff for D17816: Support for xattrs on kio copy/move.
  • Handle attrs with empty values.
  • Add test for it.
  • Fix syscalls for FreeBSD case.
Jun 29 2020, 6:09 PM · Frameworks

Jun 23 2020

arrowd added inline comments to D17816: Support for xattrs on kio copy/move.
Jun 23 2020, 5:23 PM · Frameworks
arrowd updated the diff for D17816: Support for xattrs on kio copy/move.
  • Improve comment.
Jun 23 2020, 4:44 PM · Frameworks

Jun 20 2020

arrowd updated the diff for D17816: Support for xattrs on kio copy/move.
  • Rebase on master.
Jun 20 2020, 4:30 PM · Frameworks

Jun 3 2020

arrowd added a comment to D17816: Support for xattrs on kio copy/move.

I'd appreciate if someone test this on Linux and MacOS. I got FreeBSD covered.

Jun 3 2020, 5:56 PM · Frameworks
arrowd updated the diff for D17816: Support for xattrs on kio copy/move.
  • Address comments.
Jun 3 2020, 5:55 PM · Frameworks

Jun 2 2020

arrowd added a comment to D17816: Support for xattrs on kio copy/move.

Almost every comment have been addressed. Please, give this another look.

Jun 2 2020, 7:43 PM · Frameworks
arrowd updated the diff for D17816: Support for xattrs on kio copy/move.
  • Address comments.
Jun 2 2020, 7:42 PM · Frameworks

Jun 1 2020

arrowd updated the diff for D17816: Support for xattrs on kio copy/move.
  • Change the algorithm in FileProtocol::copyXattrs()
Jun 1 2020, 6:03 PM · Frameworks

May 27 2020

arrowd added a comment to D17816: Support for xattrs on kio copy/move.

Mark stale command as done.

May 27 2020, 8:59 PM · Frameworks
arrowd updated the diff for D17816: Support for xattrs on kio copy/move.
  • Use std::function to store a generator function for m_setXattrCmd's arguments.
May 27 2020, 8:57 PM · Frameworks

May 24 2020

arrowd updated the diff for D17816: Support for xattrs on kio copy/move.
  • Refactor common code from compareXattr() into readXattr() as requested in comments.
May 24 2020, 3:05 PM · Frameworks

May 19 2020

arrowd updated the diff for D17816: Support for xattrs on kio copy/move.
  • Use full paths to command line utilities and pass xattr args correctly.
  • Mark some stale comments as done.
May 19 2020, 7:28 PM · Frameworks

May 17 2020

arrowd updated the diff for D17816: Support for xattrs on kio copy/move.
  • Fix tests on FreeBSD.
  • Fix bug: Move keylist.squeeze() call before acquiring an interator.
May 17 2020, 10:27 AM · Frameworks

May 15 2020

arrowd updated the diff for D17816: Support for xattrs on kio copy/move.

I decided to help with this a bit.

May 15 2020, 4:06 PM · Frameworks
arrowd commandeered D17816: Support for xattrs on kio copy/move.
May 15 2020, 4:04 PM · Frameworks
arrowd committed R877:0670398778d3: gpgme: Handle FreeBSD like Linux. (authored by arrowd).
gpgme: Handle FreeBSD like Linux.
May 15 2020, 12:25 PM
arrowd committed R877:73673f2bfbd3: Point qtbase to ICU headers in the Craft root, otherwise system ICU headers may… (authored by arrowd).
Point qtbase to ICU headers in the Craft root, otherwise system ICU headers may…
May 15 2020, 9:41 AM
arrowd committed R877:6f99aec61eff: Do not build iconv on MacOS to restore previous behavior. (authored by arrowd).
Do not build iconv on MacOS to restore previous behavior.
May 15 2020, 8:54 AM
arrowd committed R877:8046f3563818: Move iconv blueprint from _linux to _unix. (authored by arrowd).
Move iconv blueprint from _linux to _unix.
May 15 2020, 8:54 AM

May 9 2020

arrowd committed R877:926e8bafb12f: Enable qtx11extras for FreeBSD too. (authored by arrowd).
Enable qtx11extras for FreeBSD too.
May 9 2020, 10:38 AM

Apr 15 2020

arrowd added a watcher for FreeBSD: arrowd.
Apr 15 2020, 9:26 AM
arrowd added a comment to T12977: FreeBSD Plasma Integration.

If I mount the CD (e.g. by "make accessible from other applications") and then press the physical eject button (no applications are open on that mount), it does not eject. The disk stays mounted. I honestly don't know what behavior I expect, though.

Apr 15 2020, 9:23 AM · FreeBSD

Mar 16 2020

arrowd committed R877:658841a8e003: Add harfbuzz patch for qtbase. (authored by arrowd).
Add harfbuzz patch for qtbase.
Mar 16 2020, 11:04 AM

Mar 13 2020

arrowd committed R138:0964d0713c5a: Use CraftCore.compiler instead of OsDetection. (authored by arrowd).
Use CraftCore.compiler instead of OsDetection.
Mar 13 2020, 10:38 AM
arrowd committed R138:56cc8c0eed95: Always set LD_LIBRARY_PATH, not for Linux only. (authored by arrowd).
Always set LD_LIBRARY_PATH, not for Linux only.
Mar 13 2020, 10:32 AM
arrowd committed R138:bf26676f2bfd: AutoToolsBuildSystem: Enforce "make" only on Windows, otherwise use super() to… (authored by arrowd).
AutoToolsBuildSystem: Enforce "make" only on Windows, otherwise use super() to…
Mar 13 2020, 10:32 AM
arrowd committed R877:5b303a48d00c: dev-utils/perl: On FreeBSD disable multijob, as perl has "make" hardcoded in… (authored by arrowd).
dev-utils/perl: On FreeBSD disable multijob, as perl has "make" hardcoded in…
Mar 13 2020, 8:29 AM

Feb 26 2020

arrowd committed R877:4fb3cc701509: Add FreeBSD to libs/_unix group. (authored by arrowd).
Add FreeBSD to libs/_unix group.
Feb 26 2020, 9:24 AM
arrowd committed R877:9167c5a98926: libs/glib: Pass --enable-dtrace=0 to fix build on FreeBSD. (authored by arrowd).
libs/glib: Pass --enable-dtrace=0 to fix build on FreeBSD.
Feb 26 2020, 9:23 AM

Feb 23 2020

arrowd committed R138:a041b85e708f: Fix typo in CraftSettings.ini template. (authored by arrowd).
Fix typo in CraftSettings.ini template.
Feb 23 2020, 8:26 PM

Feb 12 2020

arrowd committed R138:01a7081dc180: FreeBSD compat. (authored by arrowd).
FreeBSD compat.
Feb 12 2020, 6:30 PM

Oct 2 2019

arrowd committed R126:f6dcff90718f: Correctly handle relative paths with line information, like `../foo.txt:123`. (authored by arrowd).
Correctly handle relative paths with line information, like `../foo.txt:123`.
Oct 2 2019, 7:20 AM

Sep 27 2019

arrowd committed R126:37cac45cc1fc: Fix a bug in `kioclient exec foo.txt:123` handling. (authored by arrowd).
Fix a bug in `kioclient exec foo.txt:123` handling.
Sep 27 2019, 6:09 PM

Sep 26 2019

arrowd retitled D18758: Make Project::open() method use async KIO methods. from Fix ASan error in test_cmakemanager by catching all signals before objects they are delivered to are deleted. to Make Project::open() method use async KIO methods..
Sep 26 2019, 10:27 AM · KDevelop
arrowd committed R32:d1b13f5217be: Add IRuntime::findExecutable method. Use it in `custom-definesandincludes`… (authored by arrowd).
Add IRuntime::findExecutable method. Use it in `custom-definesandincludes`…
Sep 26 2019, 10:26 AM
arrowd closed D5447: Only add auto-detected compilers to model if they actually exist.
Sep 26 2019, 10:26 AM

Jul 20 2019

arrowd added a comment to D22525: kioclient: Don't convert `:x:y` to `?line=x&column=y` for URLs starting with remote schemes..

I'm not familiar with KDE release engineering, so I'm asking to do merging someone else.

Jul 20 2019, 6:41 PM · Plasma
arrowd committed R126:6d86fd453417: kioclient: Don't convert `:x:y` to `?line=x&column=y` for URLs starting with… (authored by arrowd).
kioclient: Don't convert `:x:y` to `?line=x&column=y` for URLs starting with…
Jul 20 2019, 3:20 PM
arrowd closed D22525: kioclient: Don't convert `:x:y` to `?line=x&column=y` for URLs starting with remote schemes..
Jul 20 2019, 3:20 PM · Plasma
arrowd added a comment to D22525: kioclient: Don't convert `:x:y` to `?line=x&column=y` for URLs starting with remote schemes..

I suppose the kate developers like the fact that this currently works over FTP, SFTP, FISH, SMB, etc.
So maybe only HTTP[S]/WEBDAV should be blacklisted (because there queries have a different meaning, one that we can't know client-side).

Jul 20 2019, 10:49 AM · Plasma

Jul 19 2019

arrowd added a reviewer for D22525: kioclient: Don't convert `:x:y` to `?line=x&column=y` for URLs starting with remote schemes.: Frameworks.
Jul 19 2019, 3:03 PM · Plasma
arrowd added a comment to D22525: kioclient: Don't convert `:x:y` to `?line=x&column=y` for URLs starting with remote schemes..

Oh, if the latter syntax also works, then you are right.

Jul 19 2019, 3:02 PM · Plasma

Jul 18 2019

arrowd added a comment to D22525: kioclient: Don't convert `:x:y` to `?line=x&column=y` for URLs starting with remote schemes..

Could we only apply the :xx check on the filename part, not on the server part? Someone might expect that http://path.to/file.txt:99 also works for remote files.

Jul 18 2019, 5:00 PM · Plasma
arrowd requested review of D22525: kioclient: Don't convert `:x:y` to `?line=x&column=y` for URLs starting with remote schemes..
Jul 18 2019, 9:03 AM · Plasma

Jul 10 2019

arrowd committed R32:01e62e6cdcbb: Remove invalid check from test_projectload test. (authored by arrowd).
Remove invalid check from test_projectload test.
Jul 10 2019, 7:15 AM
arrowd closed D22350: Remove invalid check from test_projectload test..
Jul 10 2019, 7:15 AM · KDevelop

Jul 9 2019

arrowd requested review of D22350: Remove invalid check from test_projectload test..
Jul 9 2019, 6:33 PM · KDevelop

Jun 18 2019

arrowd added a comment to D18758: Make Project::open() method use async KIO methods..

so, I've now committed an alternative fix (or so I hope...) see:

commit bd048e67f056b5be25ed57fb2be947444f68c24e
Author: Milian Wolff <mail@milianw.de>
Date:   Mon Jun 17 22:26:32 2019 +0200

    Guard against crashes when IStatus object gets destroyed at bad times
Jun 18 2019, 8:48 AM · KDevelop

Jun 10 2019

arrowd added a comment to D20967: [UserMetaData] Add method to query which attributes are set.

Now the test passes!

Jun 10 2019, 11:52 AM · Baloo, Frameworks
arrowd committed R286:346918916474: Enable usermetadatawritertest for all UNIXes, not only Linux. (authored by arrowd).
Enable usermetadatawritertest for all UNIXes, not only Linux.
Jun 10 2019, 11:26 AM
arrowd closed D21720: Enable usermetadatawritertest for all UNIXes, not only Linux..
Jun 10 2019, 11:26 AM · Baloo, Frameworks
arrowd requested review of D21720: Enable usermetadatawritertest for all UNIXes, not only Linux..
Jun 10 2019, 11:25 AM · Baloo, Frameworks
arrowd added a comment to D20967: [UserMetaData] Add method to query which attributes are set.

Apparently, the usermetadatawritertest was not executed on *BSD before, can you check if the test succeeds on the old code?

Jun 10 2019, 11:22 AM · Baloo, Frameworks
arrowd added a comment to D20967: [UserMetaData] Add method to query which attributes are set.

It compiles now, but the test fails:

Jun 10 2019, 11:02 AM · Baloo, Frameworks
arrowd added inline comments to D20967: [UserMetaData] Add method to query which attributes are set.
Jun 10 2019, 7:40 AM · Baloo, Frameworks

May 28 2019

arrowd added a comment to D18758: Make Project::open() method use async KIO methods..
In D18758#469623, @apol wrote:

Quite possibly, it doesn't happen outside of tests after all, right?

May 28 2019, 8:22 AM · KDevelop

May 24 2019

arrowd added a comment to D18758: Make Project::open() method use async KIO methods..

I'm more interested in fixing the intial issue. With KIO jobs in openProject turned async, the crash now happens right in qWait(1) of KDevelop::TestCore::shutdown(). And what's strange is that adding qWait(1) at the end of testReload() test still fixes issue.

May 24 2019, 4:14 PM · KDevelop
arrowd accepted D21380: show static word wrap marker always if requested.
May 24 2019, 1:45 PM · Frameworks, Kate

May 21 2019

arrowd updated the diff for D18758: Make Project::open() method use async KIO methods..

Changes:

  • Import QtPromise 0.5.0 into kdevplatform/3dparty/ directory.
  • Remove synchronious KIO calls from Project::open() and use QPromises there instead.
May 21 2019, 10:12 AM · KDevelop

May 7 2019

arrowd added inline comments to D5447: Only add auto-detected compilers to model if they actually exist.
May 7 2019, 3:12 PM
arrowd updated the diff for D5447: Only add auto-detected compilers to model if they actually exist.
  • Use compiler executable name instead of absolute path.
May 7 2019, 3:12 PM

May 6 2019

arrowd added inline comments to D5447: Only add auto-detected compilers to model if they actually exist.
May 6 2019, 6:26 PM
arrowd updated the diff for D5447: Only add auto-detected compilers to model if they actually exist.
  • Address some comments.
May 6 2019, 6:24 PM
arrowd committed R32:4c947c58f85d: Fix deadlock exception in FileManagerListJob. (authored by arrowd).
Fix deadlock exception in FileManagerListJob.
May 6 2019, 12:19 PM

May 2 2019

arrowd added a comment to R32:a0fd2014489c: Don't crash when background listing outlasts file manager list job.

This commit introduced a crash in some tests on FreeBSD. The m_listing mutex gets locked twice by the same thread causing deadlock exception.

May 2 2019, 3:10 PM

Apr 13 2019

arrowd added a comment to D18758: Make Project::open() method use async KIO methods..

Oh, really? Hmm! I wouldn't be opposed to enable compilation with exceptions myself, what do the others say? We don't need to use them excessively, but for error handling in async promise chains, that would be quite useful I think?

Apr 13 2019, 6:55 PM · KDevelop

Apr 11 2019

arrowd committed R126:8072a6acf221: Add support for passing cursor information via URL parameters when running… (authored by arrowd).
Add support for passing cursor information via URL parameters when running…
Apr 11 2019, 7:46 AM
arrowd closed D18296: Add support for passing cursor information via URL parameters when running kioclient exec..
Apr 11 2019, 7:46 AM · Plasma

Apr 10 2019

arrowd added inline comments to D18296: Add support for passing cursor information via URL parameters when running kioclient exec..
Apr 10 2019, 6:53 AM · Plasma
arrowd updated the diff for D18296: Add support for passing cursor information via URL parameters when running kioclient exec..

Address comments.

Apr 10 2019, 6:52 AM · Plasma

Apr 9 2019

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

Is the makeURL function still used, or should it be removed now?

Apr 9 2019, 5:21 AM · Plasma

Apr 6 2019

arrowd added inline comments to D18296: Add support for passing cursor information via URL parameters when running kioclient exec..
Apr 6 2019, 5:33 PM · Plasma
arrowd added a comment to D18296: Add support for passing cursor information via URL parameters when running kioclient exec..

Is it really necessary to copy urlinfo.h here? Wouldn't be enough to just create a static function and put the URL parsing logic there?

Apr 6 2019, 5:32 PM · Plasma
arrowd added a comment to D18567: Make testActiveDocumentsGetBestPriority() clean up after itself.

@thomassc it is because I commandeered your revision to be able to update it. If you want, you do the same. First select "Commandeer Revision" at the bottom of this page, and then use arc diff (possibly with --update D18567) to update it.

Apr 6 2019, 12:33 PM · KDevelop

Apr 5 2019

arrowd updated the diff for D18567: Make testActiveDocumentsGetBestPriority() clean up after itself.

Simplify code.

Apr 5 2019, 12:56 PM · KDevelop
arrowd commandeered D18567: Make testActiveDocumentsGetBestPriority() clean up after itself.
Apr 5 2019, 12:56 PM · KDevelop