bruns (Stefan Brüns)
User

Projects

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Monday

  • Clear sailing ahead.

User Details

User Since
Apr 2 2017, 12:03 PM (132 w, 6 d)
Availability
Available

Recent Activity

Yesterday

bruns requested changes to D21235: Add handling of fuseiso filesystem type.
Fri, Oct 18, 6:03 PM · Frameworks

Thu, Oct 17

bruns added a comment to T7878: Create or find Icon.

The 32px is a little bit fuzzy, but definitely betterthan what we have currently. I would say, go for it!

Thu, Oct 17, 7:36 PM · VDG, Baloo
bruns added a comment to T7878: Create or find Icon.

@astippich - the circle in the background is a very nice improvement!

Thu, Oct 17, 2:36 PM · VDG, Baloo

Sat, Oct 12

bruns committed R293:4f131981493b: Use propertymap to store properties in Baloo::Result (authored by bruns).
Use propertymap to store properties in Baloo::Result
Sat, Oct 12, 10:52 PM
bruns closed D19088: Use propertymap to store properties in Baloo::Result.
Sat, Oct 12, 10:52 PM · Baloo, Frameworks
bruns committed R293:ecebbf85dc91: Add standalone conversion functions for PropertyMap to Json and vice versa (authored by bruns).
Add standalone conversion functions for PropertyMap to Json and vice versa
Sat, Oct 12, 10:43 PM
bruns closed D19087: Add standalone conversion functions for PropertyMap to Json and vice versa.
Sat, Oct 12, 10:43 PM · Baloo, Frameworks

Wed, Oct 9

bruns committed R293:3319b6656043: Replace recursion in FilteredDirIterator with loop iteration (authored by bruns).
Replace recursion in FilteredDirIterator with loop iteration
Wed, Oct 9, 4:49 PM
bruns closed D24502: Replace recursion in FilteredDirIterator with loop iteration.
Wed, Oct 9, 4:49 PM · Baloo, Frameworks
bruns added a comment to D24492: [FilteredDirIterator] Reduce stack pressure.

Let me change that into a proper loop.

Wed, Oct 9, 12:39 PM · Baloo, Frameworks
bruns added reviewers for D24502: Replace recursion in FilteredDirIterator with loop iteration: ngraham, astippich.
Wed, Oct 9, 11:44 AM · Baloo, Frameworks

Tue, Oct 8

bruns requested review of D24502: Replace recursion in FilteredDirIterator with loop iteration.
Tue, Oct 8, 2:14 PM · Baloo, Frameworks

Sun, Oct 6

bruns requested changes to D24422: [dolphin] Improve parsing of Baloo query searchString.

Please do one change at a time.

Sun, Oct 6, 6:48 PM · Dolphin
bruns accepted D23411: Fix crash in writer collection and cleanup.
Sun, Oct 6, 11:45 AM · Baloo, Frameworks

Thu, Sep 26

bruns requested changes to D23787: [baloo_file_extractor] Improve handling of large plain-text files.

Can you please provide an example which:

Thu, Sep 26, 12:21 AM · Baloo, Frameworks

Sun, Sep 22

bruns added a comment to D21235: Add handling of fuseiso filesystem type.

I have been resurrecting this patch again :) and have run into an issue I need some guidance on. To be able to parse the ~/.mtab.fuseiso file I would like to use the KMountPoint class, but this class currently resides in KIO which Solid doesn't depend on. But, KIO actually depends on Solid so would it be an option to move this class from KIO to Solid?

Sun, Sep 22, 1:17 PM · Frameworks

Sep 13 2019

bruns added inline comments to D21204: Ensure mounted nfs filesystems matches their fstab declared counterpart.
Sep 13 2019, 1:51 PM · Frameworks
bruns requested changes to D21204: Ensure mounted nfs filesystems matches their fstab declared counterpart.
Sep 13 2019, 12:40 AM · Frameworks

Sep 10 2019

bruns added a comment to D23420: Use solid to check if a KFileItem is located on a network mount.

Solid should be quite fine. When queried for devices providing 'Solid::DeviceInterface::NetworkShare', it will only use the fstab backend, which does not block and only reparses fstab/mtab when these change.

Sep 10 2019, 6:25 PM · Frameworks

Sep 7 2019

bruns added inline comments to D23411: Fix crash in writer collection and cleanup.
Sep 7 2019, 3:01 PM · Baloo, Frameworks

Aug 29 2019

bruns added a comment to D23475: [baloo-widgets] Apply condensed date to file metadata properties.

I think it would be preferable to do this in KFM::PropertyInfo

Aug 29 2019, 11:53 AM · Dolphin, Baloo

Aug 24 2019

bruns added a comment to D23394: Add xattrfetchjob.

I think you end up with less code and duplication when you just merge the two existing jobs into one, and handle the fallback there.

Aug 24 2019, 11:42 AM · Baloo

Aug 23 2019

bruns added inline comments to D17737: [CopyJob] Create clones in btrf/xfs mount.
Aug 23 2019, 6:23 PM · Frameworks

Aug 17 2019

bruns added inline comments to D19774: Use ellipsis in search field.
Aug 17 2019, 1:24 PM · KDE Edu
bruns added inline comments to D17595: Upstream Dolphin's file rename dialog.
Aug 17 2019, 3:22 AM · Frameworks
bruns added inline comments to D23112: Add a event Spy for GtkFileChooser recent files.
Aug 17 2019, 1:22 AM · Plasma

Aug 16 2019

bruns accepted D23200: Fixes a crash in Peruse triggered by baloo.
Aug 16 2019, 4:00 PM · Baloo, Frameworks
bruns requested changes to D23200: Fixes a crash in Peruse triggered by baloo.

Please just return without message, and also change the one below.

Aug 16 2019, 2:20 PM · Baloo, Frameworks

Aug 8 2019

bruns added a comment to D23008: [baloo_file_extractor] Be more resistant to multiple QSocketNotifier events.

@poboiko - is the problem you describe purely theoretical, or did you actually see it happen?

Aug 8 2019, 12:37 PM · Baloo, Frameworks

Aug 7 2019

bruns requested changes to D23008: [baloo_file_extractor] Be more resistant to multiple QSocketNotifier events.
Aug 7 2019, 10:35 PM · Baloo, Frameworks

Jul 19 2019

bruns added a comment to D22557: [IndexCleaner] ignore non-existent entries inside config.

Shouldn't we make sure that entries get removed from the config file properly?

Jul 19 2019, 3:27 PM · Baloo, Frameworks

Jul 17 2019

bruns added a comment to D21002: Remove kde4 migration agent completely.

It tried to install the no longer existing DBus service file. Fixup pushed.

Jul 17 2019, 11:52 PM · Frameworks
bruns committed R311:a52d3139e655: Do not try to install no longer existing file (authored by bruns).
Do not try to install no longer existing file
Jul 17 2019, 11:51 PM
bruns committed R311:dc5c7229bc4f: Remove kde4 migration agent completely (authored by bruns).
Remove kde4 migration agent completely
Jul 17 2019, 10:34 PM
bruns closed D21002: Remove kde4 migration agent completely.
Jul 17 2019, 10:34 PM · Frameworks
bruns added a comment to D22333: Move Solid::Device::listFromQuery calls to a separate thread.

Kinda, you're summary missing a key part.
(the commit description is a bit poor)

This patch does the following:

  • It does the search in the other thread. That creates and iterates every possible device. This is expensive as there are lots of potential devices. We fetch pretty much everything udisks knows.
  • From the given list we recreate the select few devices that are relevant. This is relatively cheap. It does still block as you point out, but now for a much much smaller set of objects compared to before.

    (on my laptop that's 56 devices created when searching, resulting in 8 useful devices).
Jul 17 2019, 4:41 PM · Plasma
bruns requested changes to D22502: [FileIndexerConfig] skip invalid entries from included/excludedFolders.
Jul 17 2019, 2:49 PM · Baloo, Frameworks
bruns added a comment to D22502: [FileIndexerConfig] skip invalid entries from included/excludedFolders.

The correct fix is to check the returned/calculated ID in the IndexCleaner, otherwise its racy.

Jul 17 2019, 2:49 PM · Baloo, Frameworks

Jul 16 2019

bruns added a comment to D22333: Move Solid::Device::listFromQuery calls to a separate thread.
Jul 16 2019, 2:28 PM · Plasma
bruns added a comment to D22339: Make sure solid backends are reentrant.

I would have preferred this to only land when D22333 had been addressed fully, I assumed this required no further notice.

Jul 16 2019, 2:13 PM · Frameworks
bruns added a reverting change for R293:d6d86cb86cb6: Revert "[PowerStateMonitor] Be conservative when determining power state": R293:397b4a041f2e: Revert "Revert "[PowerStateMonitor] Be conservative when determining power….
Jul 16 2019, 11:55 AM
bruns committed R293:397b4a041f2e: Revert "Revert "[PowerStateMonitor] Be conservative when determining power… (authored by bruns).
Revert "Revert "[PowerStateMonitor] Be conservative when determining power…
Jul 16 2019, 11:55 AM
bruns added a comment to D21672: [PowerStateMonitor] Be conservative when determining power state.

Why don't you notify the relevant people first

He kinda did by writing a bug report, but let's not argue but investigate and fix why this seemingly innocent patch caused the behavior described.

Jul 16 2019, 10:22 AM · Baloo, Frameworks
bruns added a comment to D21672: [PowerStateMonitor] Be conservative when determining power state.

Reverted by d6d86cb86c on master. See https://bugs.kde.org/show_bug.cgi?id=409405 for more info. Possibly a critical problem in released 5.60 @bruns.

Jul 16 2019, 10:15 AM · Baloo, Frameworks

Jul 13 2019

bruns added a comment to D22339: Make sure solid backends are reentrant.

This is inacceptable!

Jul 13 2019, 1:16 AM · Frameworks
bruns added a comment to D22333: Move Solid::Device::listFromQuery calls to a separate thread.
In D22333#494810, @apol wrote:

Also, the code is calling non-threadsafe code from multiple threads now (e.g. once from each the two dataengines helper threads). Each one will call the udisks2 Manager::deviceCache() method.

There's a Backend per-thread and deviceCache is not static so each thread will call its own.

Jul 13 2019, 1:02 AM · Plasma

Jul 12 2019

bruns added a comment to D22333: Move Solid::Device::listFromQuery calls to a separate thread.
In D22333#494415, @apol wrote:

Again, where is it blocking? Which backend?

udisks2 mainly, but every backend can block by its virtue.

Jul 12 2019, 9:03 PM · Plasma

Jul 11 2019

bruns added a comment to D22333: Move Solid::Device::listFromQuery calls to a separate thread.
In D22333#494384, @apol wrote:
In D22333#494152, @apol wrote:

Why not just a singleshot timer from the constructor? Avoids any initial blocking ...

Initial, but doesn't fix the problem. We could potentially delay this few seconds instead, but we'd still be getting an odd ~500ms freeze randomly
Or we can just decide to move blocking or heavy algorithms to separate threads and enjoy our multicore computers and a fluid experience.

Where does the blocking happen? How do you guarantee none of the later call block? Large parts of the code are executed in the main thread anyway, only the initial list creation happens in a worker thread.

listFromQuery is the big blocking call we are moving into a separate thread.

Jul 11 2019, 7:12 PM · Plasma
bruns added a comment to D22333: Move Solid::Device::listFromQuery calls to a separate thread.
In D22333#494152, @apol wrote:

Why not just a singleshot timer from the constructor? Avoids any initial blocking ...

Initial, but doesn't fix the problem. We could potentially delay this few seconds instead, but we'd still be getting an odd ~500ms freeze randomly
Or we can just decide to move blocking or heavy algorithms to separate threads and enjoy our multicore computers and a fluid experience.

Jul 11 2019, 2:29 PM · Plasma
bruns added a comment to D22339: Make sure solid backends are reentrant.
In D22339#494149, @apol wrote:

How do you guarantee *each thread local* copy now is connected to the right signals, i.e. the socket notifier for the mtab and inotify for fstab?

each backend will create things separately now, so it should work like it used to so far when there was just one thread involved.

Jul 11 2019, 2:15 PM · Frameworks
bruns added a comment to D21427: Always skip trailing slashes in FilderedDirIterator.

Ping!

Apparently, it does fix bug 409257, which is pretty serious one (db corruption, after all).

Jul 11 2019, 12:48 PM · Baloo, Frameworks
bruns requested changes to D22333: Move Solid::Device::listFromQuery calls to a separate thread.

Why not just a singleshot timer from the constructor? Avoids any initial blocking ...

Jul 11 2019, 11:39 AM · Plasma
bruns requested changes to D22339: Make sure solid backends are reentrant.

How do you guarantee *each thread local* copy now is connected to the right signals, i.e. the socket notifier for the mtab and inotify for fstab?

Jul 11 2019, 11:37 AM · Frameworks

Jul 10 2019

bruns committed R120:7fa46fa74987: [ksmserver] Remove obsolete KDELibs4Support dependency (authored by bruns).
[ksmserver] Remove obsolete KDELibs4Support dependency
Jul 10 2019, 12:26 AM
bruns closed D22296: [ksmserver] Remove obsolete KDELibs4Support dependency.
Jul 10 2019, 12:26 AM · Plasma
bruns updated subscribers of D22296: [ksmserver] Remove obsolete KDELibs4Support dependency.
Jul 10 2019, 12:22 AM · Plasma
bruns updated the diff for D22296: [ksmserver] Remove obsolete KDELibs4Support dependency.

remove inhibition

Jul 10 2019, 12:20 AM · Plasma

Jul 8 2019

bruns added inline comments to D22296: [ksmserver] Remove obsolete KDELibs4Support dependency.
Jul 8 2019, 6:35 PM · Plasma
bruns added a comment to D22296: [ksmserver] Remove obsolete KDELibs4Support dependency.
Jul 8 2019, 4:55 PM · Plasma

Jul 7 2019

bruns committed R120:bf811e448c76: [DeviceNotifications] Keep last message for each device (authored by bruns).
[DeviceNotifications] Keep last message for each device
Jul 7 2019, 7:13 PM
bruns closed D22312: [DeviceNotifications] Keep last message for each device.
Jul 7 2019, 7:13 PM · Plasma
bruns requested review of D22312: [DeviceNotifications] Keep last message for each device.
Jul 7 2019, 6:02 PM · Plasma
bruns accepted D22192: remove foreach.
Jul 7 2019, 4:56 PM · Baloo
bruns added inline comments to D22192: remove foreach.
Jul 7 2019, 4:22 PM · Baloo

Jul 6 2019

bruns added a comment to D22296: [ksmserver] Remove obsolete KDELibs4Support dependency.

After having looked into the callchain again, I am not sure what the best action here is:

Jul 6 2019, 2:22 PM · Plasma
bruns requested review of D22296: [ksmserver] Remove obsolete KDELibs4Support dependency.
Jul 6 2019, 11:02 AM · Plasma

Jul 5 2019

bruns added inline comments to D22192: remove foreach.
Jul 5 2019, 8:08 PM · Baloo
bruns added a comment to D21002: Remove kde4 migration agent completely.

Can I have a "go"?

Jul 5 2019, 7:59 PM · Frameworks
bruns accepted D20526: Fix extracting of some properties to match what was written.
Jul 5 2019, 7:56 PM · Baloo, Frameworks
bruns added a comment to D22276: Add an action to insert a non-indented newline.

Another way to avoid indented empty line is to clear the trailing spaces/indentation when pressing enter while the cursor is at the end of a line containing only spaces or \t. This can lead to a nightmare though if someone want to keep trailing spaces...

Jul 5 2019, 10:34 AM · Frameworks, Kate

Jul 1 2019

bruns added inline comments to D22147: Better use of Qt APIs in Plasma::Theme.
Jul 1 2019, 2:38 PM · Frameworks

Jun 30 2019

bruns accepted D22159: Check for DQT_NO_CAST_FROM_BYTEARRAY.
Jun 30 2019, 7:27 PM · Baloo
bruns added inline comments to D22159: Check for DQT_NO_CAST_FROM_BYTEARRAY.
Jun 30 2019, 10:57 AM · Baloo

Jun 29 2019

bruns accepted D22157: Use debugging category in taglib extractor/writer.
Jun 29 2019, 6:40 PM · Baloo, Frameworks
bruns accepted D22096: Check for DQT_NO_CAST_FROM_ASCII.
Jun 29 2019, 3:51 PM · Baloo
bruns added inline comments to D22157: Use debugging category in taglib extractor/writer.
Jun 29 2019, 3:50 PM · Baloo, Frameworks
bruns added inline comments to D20526: Fix extracting of some properties to match what was written.
Jun 29 2019, 3:35 PM · Baloo, Frameworks

Jun 27 2019

bruns added inline comments to D22096: Check for DQT_NO_CAST_FROM_ASCII.
Jun 27 2019, 9:51 PM · Baloo

Jun 26 2019

bruns added inline comments to D22096: Check for DQT_NO_CAST_FROM_ASCII.
Jun 26 2019, 2:58 PM · Baloo

Jun 25 2019

bruns committed R245:541a3d3a0573: [Fstab] Select appropriate icon for home or root directory (authored by bruns).
[Fstab] Select appropriate icon for home or root directory
Jun 25 2019, 9:02 AM
bruns closed D22081: [Fstab] Select appropriate icon for home or root directory.
Jun 25 2019, 9:02 AM · Frameworks
bruns committed R245:54a501f00bbe: [Fstab] Show mounted "overlay" filesystems (authored by bruns).
[Fstab] Show mounted "overlay" filesystems
Jun 25 2019, 9:00 AM
bruns closed D22080: [Fstab] Show mounted "overlay" filesystems.
Jun 25 2019, 9:00 AM · Frameworks

Jun 24 2019

bruns committed R623:6cff71f9dab0: Port away from KDELibs4Support, use Solid::Power interface (authored by bruns).
Port away from KDELibs4Support, use Solid::Power interface
Jun 24 2019, 5:55 PM
bruns closed D21976: Port away from KDELibs4Support, use Solid::Power interface.
Jun 24 2019, 5:55 PM
bruns committed R623:04cb8277574b: Remove explicit initialization of default constructed members (authored by bruns).
Remove explicit initialization of default constructed members
Jun 24 2019, 5:54 PM
bruns closed D21975: Remove explicit initialization of default constructed members.
Jun 24 2019, 5:54 PM
bruns closed D21974: Fix warning, remove unsigned int < 0 check.
Jun 24 2019, 5:53 PM
bruns committed R623:6214113c8a29: Fix warning, remove unsigned int < 0 check (authored by bruns).
Fix warning, remove unsigned int < 0 check
Jun 24 2019, 5:53 PM
bruns added a comment to D15739: [Places panel] Don't show Root by default.

If adding such a special case makes sense, yes. I'd even argue about 'mntent.mnt_dir == "/" && !isKnownFilesystem(mntent)' or something like that to ensure that an entry for / is always provided.

Jun 24 2019, 4:51 PM · Frameworks
bruns added a dependent revision for D22080: [Fstab] Show mounted "overlay" filesystems: D22081: [Fstab] Select appropriate icon for home or root directory.
Jun 24 2019, 4:43 PM · Frameworks
bruns requested review of D22081: [Fstab] Select appropriate icon for home or root directory.
Jun 24 2019, 4:43 PM · Frameworks
bruns requested review of D22080: [Fstab] Show mounted "overlay" filesystems.
Jun 24 2019, 4:42 PM · Frameworks
bruns added a comment to D15739: [Places panel] Don't show Root by default.

It actually already does, via the fstab backend (currently network (SMB/NFS) and various fuse mounts). Fstab in this case also includes everything in MTAB, should be quite easy to extend this.

What is the output of cat /proc/self/mounts (fell free to remove anything irrelevant, like cgroups)?

The entries involved in / are these:

/dev/sr0 /run/initramfs/live iso9660 ro,relatime 0 0
/dev/loop0 /run/initramfs/squashfs_container squashfs ro,relatime 0 0
/dev/loop1 /run/rootfsbase ext4 ro,relatime,data=ordered 0 0
LiveOS_rootfs / overlay rw,relatime,lowerdir=/run/rootfsbase,upperdir=/run/overlayfs/rw,workdir=/run/overlayfs/work 0 0
Jun 24 2019, 3:24 PM · Frameworks
bruns added a comment to D15739: [Places panel] Don't show Root by default.

Even if all (block) devices and their mountpoints were shown in the devices view, there would be no equivalent of "/". One loop device provides the read-only base for /, but that's actually more confusing than the current state as it looks like /, but is only a read-only view of "the past".

I guess solid needs to gain support for mountpoints not backed by devices?

Jun 24 2019, 2:34 PM · Frameworks

Jun 23 2019

bruns committed R623:50383c74098b: Make the notifications less obtrusive (authored by bruns).
Make the notifications less obtrusive
Jun 23 2019, 9:42 PM
bruns closed D22027: Make the notifications less obtrusive.
Jun 23 2019, 9:42 PM
bruns closed D22026: Use own eventIds and ComponentName instead of generic plasma workspace ones.
Jun 23 2019, 9:41 PM
bruns committed R623:7268a2da05f8: Use own eventIds and ComponentName instead of generic plasma workspace ones (authored by bruns).
Use own eventIds and ComponentName instead of generic plasma workspace ones
Jun 23 2019, 9:41 PM
bruns committed R623:27bcd6a78c29: Fix usage of 0 for null pointer constants (authored by bruns).
Fix usage of 0 for null pointer constants
Jun 23 2019, 9:40 PM