Review Request 122653: Set permissions for links in remote:, necessary for correct visualization

2015-02-21 Thread Stefan Brüns
: https://git.reviewboard.kde.org/r/122653/diff/ Testing --- dolphin remote: -> no lock icon on smb:, mtp:, ... links Thanks, Stefan Brüns

Review Request 122652: Use correct default value when UDS_ACCESS/UDS_FILE_TYPE is not set

2015-02-21 Thread Stefan Brüns
://git.reviewboard.kde.org/r/122652/diff/ Testing --- dolphin remote: -> no lock icon on smb:, mtp:, ... links Thanks, Stefan Brüns

Re: Review Request 122652: Use correct default value when UDS_ACCESS/UDS_FILE_TYPE is not set

2015-02-25 Thread Stefan Brüns
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122652/#review76619 --- Ping! - Stefan Brüns On Feb. 20, 2015, 9:28 p.m., Stefan

Re: Review Request 122652: Use correct default value when UDS_ACCESS/UDS_FILE_TYPE is not set

2015-02-27 Thread Stefan Brüns
KF5 - Stefan --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122652/#review76626 ------- On Feb. 20, 2015, 9:28 p.m., Stefan Brüns

Re: Review Request 122652: Use correct default value when UDS_ACCESS/UDS_FILE_TYPE is not set

2015-03-16 Thread Stefan Brüns
fixes a > bug, it can go in. > Of course it should be forward-ported to KIO Framework if it is still > relevant there. > > Stefan Brüns wrote: > So shall I commit this (and https://git.reviewboard.kde.org/r/122653/ as > well)? > > AFAIK next Application

Re: Review Request 122652: Use correct default value when UDS_ACCESS/UDS_FILE_TYPE is not set

2015-03-18 Thread Stefan Brüns
marked as submitted. Review request for KDE Frameworks, kdelibs and David Faure. Changes --- Submitted with commit 6215e878706dc6e79d1b6ac5df33f5751d6c7ec0 by Stefan Brüns to branch master. Bugs: 339193 http://bugs.kde.org/show_bug.cgi?id=339193 Repository: kdelibs Description

Re: Review Request 122653: Set permissions for links in remote:, necessary for correct visualization

2017-02-08 Thread Stefan Brüns
. CCBUG: 339193 Diffs - kioslave/remote/remoteimpl.cpp 5d973c6c1b6c31b7f3107d0d15805ef04bfdd661 Diff: https://git.reviewboard.kde.org/r/122653/diff/ Testing --- dolphin remote: -> no lock icon on smb:, mtp:, ... links Thanks, Stefan Brüns

Re: Review Request 113587: Add support for x-gvfs style options in fstab

2017-03-13 Thread Stefan Brüns
/backends/fstab/fstabstorageaccess.cpp 5482d48 Diff: https://git.reviewboard.kde.org/r/113587/diff/ Testing --- Add "x-gvfs-name=Share%20on%20Laptop" to CIFS mount, open dolphin -> Name has changed Thanks, Stefan Brüns

Review Request 112294: Implement multi-seat support in KDM

2013-08-26 Thread Stefan Brüns
/112294/diff/ Testing --- Single seat system, several multiseat systems Thanks, Stefan Brüns

Re: Review Request 112294: Implement multi-seat support in KDM

2013-08-27 Thread Stefan Brüns
On Aug. 27, 2013, 2:08 p.m., Stefan Brüns wrote: > > looks fine to me, just that tiny minor nitpick on variable naming. I assume this applies to systemd_monitor{,_fd} in dm.c as well. Will fix, waiting a few days for more comments ... -

Re: Review Request 112294: Implement multi-seat support in KDM

2013-09-02 Thread Stefan Brüns
d8dd6f3 kdm/backend/session.c 0e7901c Diff: http://git.reviewboard.kde.org/r/112294/diff/ Testing --- Single seat system, several multiseat systems Thanks, Stefan Brüns

Re: Review Request 112294: Implement multi-seat support in KDM

2013-09-02 Thread Stefan Brüns
/session.c 0e7901c Diff: http://git.reviewboard.kde.org/r/112294/diff/ Testing --- Single seat system, several multiseat systems Thanks, Stefan Brüns

Re: Review Request 112294: Implement multi-seat support in KDM

2013-09-04 Thread Stefan Brüns
generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112294/#review39302 --- On Sept. 2, 2013, 11:34 p.m., Stefan Brüns wrote: > > --- > This is an automaticall

Re: Review Request 113866: kio bookmarks: Only iterate over element nodes in findToolbar()

2013-11-15 Thread Stefan Brüns
crash. Possible fix for 320006 Diffs - kio/bookmarks/kbookmark.cc 8459a02 Diff: http://git.reviewboard.kde.org/r/113866/diff/ Testing --- None Thanks, Stefan Brüns

Re: Review Request 113866: kio bookmarks: Only iterate over element nodes in findToolbar()

2013-12-16 Thread Stefan Brüns
QDomElement. toElement will return NULL and accessing it will crash. Possible fix for 320006 Diffs (updated) - kio/bookmarks/kbookmark.cc 8459a02 Diff: http://git.reviewboard.kde.org/r/113866/diff/ Testing --- None Thanks, Stefan Brüns

Re: Review Request 113866: kio bookmarks: Only iterate over element nodes in findToolbar()

2013-12-16 Thread Stefan Brüns
viewboard.kde.org/r/113866/#review44510 --- On Dec. 16, 2013, 6:33 p.m., Stefan Brüns wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/113866/ > ---

Review Request 114506: Create testcase for bookmark toolbar

2013-12-16 Thread Stefan Brüns
/kbookmarktest.cpp 5ab002b Diff: http://git.reviewboard.kde.org/r/114506/diff/ Testing --- Thanks, Stefan Brüns

Re: Review Request 113866: kio bookmarks: Only iterate over element nodes in findToolbar()

2014-01-11 Thread Stefan Brüns
QDomElement. toElement will return NULL and accessing it will crash. Possible fix for 320006 Diffs - kio/bookmarks/kbookmark.cc 8459a02 Diff: https://git.reviewboard.kde.org/r/113866/diff/ Testing --- None Thanks, Stefan Brüns

Re: Review Request 112294: Implement multi-seat support in KDM

2014-03-27 Thread Stefan Brüns
> On Sept. 3, 2013, 10:20 p.m., Oswald Buddenhagen wrote: > > given that there is no intention to make further feature releases of the > > kde workspace which will include kdm, i wonder why we'd go through the > > (potentially tedious) process of upstreaming this no

Re: Review Request 112294: Implement multi-seat support in KDM

2014-03-29 Thread Stefan Brüns
/backend/server.c d8dd6f3 kdm/backend/session.c 0e7901c Diff: https://git.reviewboard.kde.org/r/112294/diff/ Testing --- Single seat system, several multiseat systems Thanks, Stefan Brüns

Re: Review Request 112294: Implement multi-seat support in KDM

2014-03-29 Thread Stefan Brüns
/backend/client.c 26bb0b4 kdm/backend/dm.h b2f8c61 kdm/backend/dm.c 77a2ef7 kdm/backend/server.c d8dd6f3 kdm/backend/session.c 0e7901c Diff: https://git.reviewboard.kde.org/r/112294/diff/ Testing --- Single seat system, several multiseat systems Thanks, Stefan Brüns

Re: Review Request 112294: Implement multi-seat support in KDM

2014-03-29 Thread Stefan Brüns
taking the "seat" into account, so the only possibility to apply a specific config is to use "layout". - Stefan --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.or

Re: Review Request 112294: Implement multi-seat support in KDM

2014-03-29 Thread Stefan Brüns
------- On March 29, 2014, 5:38 p.m., Stefan Brüns wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/112294/ > --

Re: Review Request 112294: Implement multi-seat support in KDM

2014-03-30 Thread Stefan Brüns
> On March 28, 2014, 10:59 a.m., Oswald Buddenhagen wrote: > > kdm/backend/server.c, line 79 > > <https://git.reviewboard.kde.org/r/112294/diff/2/?file=186613#file186613line79> > > > > why the redundant supply of the seat as a layout? > > Stefan

Re: Review Request 112294: Implement multi-seat support in KDM

2014-04-06 Thread Stefan Brüns
/server.c d8dd6f3 kdm/backend/session.c 0e7901c Diff: https://git.reviewboard.kde.org/r/112294/diff/ Testing --- Single seat system, several multiseat systems Thanks, Stefan Brüns

Re: Review Request 112294: Implement multi-seat support in KDM

2014-04-06 Thread Stefan Brüns
ly generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/112294/#review54418 --- On April 6, 2014, 8:16 p.m., Stefan Brüns wrote: > > --- > This is an auto

Re: Review Request 112294: Implement multi-seat support in KDM

2014-04-06 Thread Stefan Brüns
/plasma-wallpaper-image.desktop 6484545 Diff: https://git.reviewboard.kde.org/r/112294/diff/ Testing --- Single seat system, several multiseat systems Thanks, Stefan Brüns

Re: Review Request 112294: Implement multi-seat support in KDM

2014-04-07 Thread Stefan Brüns
nable to me. why are you re-defining the display to > > be permanent? when the seat goes away, kdm won't know what to do with it. > > Stefan Brüns wrote: > When a seat goes away, rStopDisplay(d, DS_RESERVE) is called - maybe a > "d->displayType = dLoca

Re: Review Request 112294: Implement multi-seat support in KDM

2014-05-18 Thread Stefan Brüns
38a8c70 kdm/config.def 751c0ed kdm/kfrontend/kdm_config.c 368c8d1 Diff: https://git.reviewboard.kde.org/r/112294/diff/ Testing --- Single seat system, several multiseat systems Thanks, Stefan Brüns

Re: Review Request 112294: Implement multi-seat support in KDM

2014-05-18 Thread Stefan Brüns
; > be permanent? when the seat goes away, kdm won't know what to do with it. > > Stefan Brüns wrote: > When a seat goes away, rStopDisplay(d, DS_RESERVE) is called - maybe a > "d->displayType = dLocal | dReserve" is missing for these cases. > But as

Re: Review Request 112294: Implement multi-seat support in KDM

2014-05-18 Thread Stefan Brüns
--------- On May 18, 2014, 7:44 p.m., Stefan Brüns wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/112294/ >

Re: Review Request 112294: Implement multi-seat support in KDM

2014-05-24 Thread Stefan Brüns
On May 24, 2014, 11:46 a.m., Stefan Brüns wrote: > > regarding DynamicDisplays and your magic backwards compat handling ... i > > think the seat-encoded config stuff i insisted on makes that redundant, > > after all. suppose we do it like that: > > > > Static

Re: Review Request 112294: Implement multi-seat support in KDM

2014-05-25 Thread Stefan Brüns
://git.reviewboard.kde.org/r/112294/diff/ Testing --- Single seat system, several multiseat systems Thanks, Stefan Brüns

Re: Review Request 112294: Implement multi-seat support in KDM

2014-05-25 Thread Stefan Brüns
g/r/112294/#comment40630> This is actually needed to supply the seat to logind session registration via pam_systemd. This could be placed earlier, but I chose this place because it is near the old CK session registration, and it only has to be before pam_open_session(...) - Stefan Brün

Re: Review Request 112294: Implement multi-seat support in KDM

2014-05-26 Thread Stefan Brüns
esource.c <https://git.reviewboard.kde.org/r/112294/#comment40688> They are: http://domainkeys.sourceforge.net/underscore.html kdm/backend/resource.c <https://git.reviewboard.kde.org/r/112294/#comment40689> see above kdm/backend/session.c <https://git.reviewboard.kde.org/

Re: Review Request 112294: Implement multi-seat support in KDM

2014-05-29 Thread Stefan Brüns
cular pam module needs this setting, it should be done > > much earlier than here, and the next line would be pointless (we use > > pam_getenvlist). that would also mean that pam is a hard dependency, which > > should be expressed by the configure code and appropriate ifdefs. >

Re: Review Request 112294: Implement multi-seat support in KDM

2014-05-29 Thread Stefan Brüns
> On May 26, 2014, 2:50 p.m., Stefan Brüns wrote: > > kdm/backend/dm.c, line 1430 > > <https://git.reviewboard.kde.org/r/112294/diff/7/?file=274916#file274916line1430> > > > > Hm, I prefer this notation, in particular as this is the same as for > &

Re: Review Request 112294: Implement multi-seat support in KDM

2014-05-29 Thread Stefan Brüns
: https://git.reviewboard.kde.org/r/112294/#review58394 ------- On May 26, 2014, 12:06 a.m., Stefan Brüns wrote: > > --- > This is an automatically generated e-m

Re: Review Request 112294: Implement multi-seat support in KDM

2014-05-29 Thread Stefan Brüns
sounds strange IMHO - Stefan --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/112294/#review58442 --- On May 26, 2014, 12:06

Re: Review Request 112294: Implement multi-seat support in KDM

2014-05-29 Thread Stefan Brüns
/resource.c 38a8c70 kdm/backend/server.c d8dd6f3 kdm/config-kdm.h.cmake 3e8912d kdm/kfrontend/kdm_config.c 368c8d1 Diff: https://git.reviewboard.kde.org/r/112294/diff/ Testing --- Single seat system, several multiseat systems Thanks, Stefan Brüns

Re: Review Request 112294: Implement multi-seat support in KDM

2014-05-31 Thread Stefan Brüns
automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/112294/#review58845 ------- On May 29, 2014, 7:03 p.m., Stefan Brüns wrote: > > --- > This

Re: Review Request 112294: Implement multi-seat support in KDM

2014-05-31 Thread Stefan Brüns
> On May 26, 2014, 8:32 a.m., Oswald Buddenhagen wrote: > > kdm/backend/dm.c, line 1436 > > <https://git.reviewboard.kde.org/r/112294/diff/7/?file=274916#file274916line1436> > > > > "no matching unused" > > Stefan Brüns wrote: > "

Re: Review Request 112294: Implement multi-seat support in KDM

2014-05-31 Thread Stefan Brüns
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/112294/#review58845 ------- On May 29, 2014, 7:03 p.m., Stefan Brüns wrote: > > --

Re: Review Request 112294: Implement multi-seat support in KDM

2014-06-01 Thread Stefan Brüns
t; > seat changed? i don't think you are handling the transition (which would > > involve the 'raiser' state, iirc), so the display would come up again only > > after the next hup or replug. > > > > it's probably not worth it to handle the com

Re: Review Request 113587: Add support for x-gvfs style options in fstab

2014-06-21 Thread Stefan Brüns
solid/solid/backends/fstab/fstabdevice.h bac409e Diff: https://git.reviewboard.kde.org/r/113587/diff/ Testing --- Add "x-gvfs-name=Share%20on%20Laptop" to CIFS mount, open dolphin -> Name has changed Thanks, Stefan Brüns

Review Request: Avoid creating an empty .tbcache for bookmarks

2012-09-25 Thread Stefan Brüns
tbcache file is created when the toolbar bookmarks are the full bookmarks, i.e. no entries/folders with toolbar attribute. Diffs - kio/bookmarks/kbookmarkmanager.cc d8a9cb7 Diff: http://git.reviewboard.kde.org/r/106563/diff/ Testing --- Thanks, Stefan Brüns

Re: Review Request: Avoid creating an empty .tbcache for bookmarks

2012-09-28 Thread Stefan Brüns
r/106563/#review19523 --- On Sept. 25, 2012, 11:01 a.m., Stefan Brüns wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/106563/ > -

Re: Review Request: Avoid creating an empty .tbcache for bookmarks

2012-09-28 Thread Stefan Brüns
--- Thanks, Stefan Brüns

Re: Review Request: Avoid creating an empty .tbcache for bookmarks

2012-09-30 Thread Stefan Brüns
/kbookmarkmanager.cc d8a9cb7 Diff: http://git.reviewboard.kde.org/r/106563/diff/ Testing --- Thanks, Stefan Brüns