D24582: Replace usage of SlaveBase::config() by SlaveBase::mapConfig()

2019-10-15 Thread David Faure
dfaure accepted this revision. This revision is now accepted and ready to land. REPOSITORY R241 KIO BRANCH arcpatch-D24582 REVISION DETAIL https://phabricator.kde.org/D24582 To: meven, #frameworks, dfaure Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D24682: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-15 Thread Volker Krause
vkrause accepted this revision. This revision is now accepted and ready to land. BRANCH useECMGenerateExportHeader REVISION DETAIL https://phabricator.kde.org/D24682 To: kossebau, #frameworks, vkrause

D24537: Icons for windy weather

2019-10-15 Thread Noah Davis
ndavis accepted this revision. ndavis added a comment. This revision is now accepted and ready to land. I'm fine with this blue. REPOSITORY R266 Breeze Icons BRANCH windy-weather-icons (branched from master) REVISION DETAIL https://phabricator.kde.org/D24537 To: cblack, #vdg, ndavis C

D24665: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-15 Thread Ben Cooksley
bcooksley added a comment. I've now removed the Staging Area URI for the KArchive repository, so you should now be able to use Arcanist with it again. REPOSITORY R243 KArchive REVISION DETAIL https://phabricator.kde.org/D24665 To: kossebau, #frameworks, dfaure Cc: bcooksley, kde-framewo

D24497: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-15 Thread Laurent Montel
mlaurent accepted this revision. mlaurent added a comment. This revision is now accepted and ready to land. Seems ok for me REPOSITORY R265 KConfigWidgets BRANCH useECMGenerateExportHeader REVISION DETAIL https://phabricator.kde.org/D24497 To: kossebau, #frameworks, mlaurent Cc: mlaur

D24681: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-15 Thread Laurent Montel
mlaurent accepted this revision. mlaurent added a comment. This revision is now accepted and ready to land. Seems ok for me REPOSITORY R304 KNewStuff BRANCH useECMGenerateExportHeader REVISION DETAIL https://phabricator.kde.org/D24681 To: kossebau, #frameworks, leinir, mlaurent Cc: ml

D24497: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-15 Thread Friedrich W. H. Kossebau
kossebau updated this revision to Diff 68010. kossebau added a comment. update to latest master with new deprecated method REPOSITORY R265 KConfigWidgets CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D24497?vs=67736&id=68010 BRANCH useECMGenerateExportHeader REVISION DETAIL

D24537: Icons for windy weather

2019-10-15 Thread Carson Black
cblack updated this revision to Diff 68009. cblack added a comment. Remove excess icon REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D24537?vs=68008&id=68009 BRANCH windy-weather-icons (branched from master) REVISION DETAIL https://phabricator.kd

D24537: Icons for windy weather

2019-10-15 Thread Carson Black
cblack updated this revision to Diff 68008. cblack added a comment. Colour changes REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D24537?vs=67874&id=68008 BRANCH windy-weather-icons (branched from master) REVISION DETAIL https://phabricator.kde.or

D24537: Icons for windy weather

2019-10-15 Thread Nathaniel Graham
ngraham added a comment. I don't mind the cyan myself, though I'll admit my wife (who is an artist) commented on it negatively when I showed it to her. Perhaps make it a bit bluer and then we can get these nice icons in. I'd rather not bikeshed for too long on them. :) REPOSITORY R266 Br

D24682: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-15 Thread Friedrich W. H. Kossebau
kossebau added inline comments. INLINE COMMENTS > recurrence.h:149 > > +#if KCALENDARCORE_BUILD_DEPRECATED_SINCE(5, 64) > /** Only using `KCALENDARCORE_BUILD_DEPRECATED_SINCE` here and not `KCALENDARCORE_ENABLE_DEPRECATED_SINCE`, because we have no idea yet how all compilers would deal

D24682: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-15 Thread Friedrich W. H. Kossebau
kossebau created this revision. kossebau added reviewers: Frameworks, vkrause. kossebau requested review of this revision. REVISION SUMMARY Allows - projects linking to KCalendarCore to hide deprecated API up to a given version or silence deprecation warnings after a given version, using

D24681: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-15 Thread Friedrich W. H. Kossebau
kossebau added a comment. No support for EXCLUDE_DEPRECATED_BEFORE_AND_AT, as some deprecated API is still used internally, so this needs more clean-up by someone(tm) first. REPOSITORY R304 KNewStuff REVISION DETAIL https://phabricator.kde.org/D24681 To: kossebau, #frameworks, leinir Cc

D24681: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-15 Thread Friedrich W. H. Kossebau
kossebau created this revision. kossebau added reviewers: Frameworks, leinir. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. kossebau requested review of this revision. REVISION SUMMARY Allows - projects linking to KNewStuff to hide deprecated API up to

D24621: [RFC] New Konsole and Yakuake icons

2019-10-15 Thread Nathaniel Graham
ngraham added a comment. I really like the Yakuake icon that has a subtle "Y" in it. If everyone else does too, maybe that should be moved into a separate patch so we can get it in quickly and then concentrate on the Konsole icon. Personally I'm not 100% sold on integrating the bottom ta

D24477: Add PATH/LD_LIBRARY_PATH to qrcAlias invocation

2019-10-15 Thread Luigi Toscano
ltoscano added a comment. In D24477#547779 , @ndavis wrote: > For some reason, Arc made me the committer and the author instead of just the committer. Maybe because patches sent through the phabricator web interface have no authorship in

D24477: Add PATH/LD_LIBRARY_PATH to qrcAlias invocation

2019-10-15 Thread Noah Davis
ndavis added a comment. For some reason, Arc made me the committer and the author instead of just the committer. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D24477 To: masonm, #vdg, ndavis Cc: kde-frameworks-devel, #vdg, LeGast00n, GB_2, michaelh, ngraham, b

D24672: GIT_SILENT Run uncrustify-kf5 on the whole tree

2019-10-15 Thread Christoph Cullmann
cullmann added a comment. As an non-framework module, there is no need to follow the frameworks style, if not wanted. REPOSITORY R374 KolourPaint REVISION DETAIL https://phabricator.kde.org/D24672 To: ahmadsamir, kde-frameworks-devel, mkoller Cc: cullmann, ognarb, kde-frameworks-devel

D24672: GIT_SILENT Run uncrustify-kf5 on the whole tree

2019-10-15 Thread Ahmad Samir
ahmadsamir added a comment. In D24672#547744 , @mkoller wrote: > why ? > I don't like lots of these changes. Because it doesn't follow the KDE coding style, AFAICS (and changing it to use that common style will make it easier for whoev

D24672: GIT_SILENT Run uncrustify-kf5 on the whole tree

2019-10-15 Thread Martin Koller
mkoller requested changes to this revision. mkoller added a comment. This revision now requires changes to proceed. why ? I don't like lots of these changes. REPOSITORY R374 KolourPaint REVISION DETAIL https://phabricator.kde.org/D24672 To: ahmadsamir, kde-frameworks-devel, mkoller Cc:

D24665: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-15 Thread Friedrich W. H. Kossebau
kossebau added a comment. In D24665#547727 , @bcooksley wrote: > As part of the transition to Gitlab, Sysadmin removed the staging repositories which a few Phabricator repositories had been configured to use. > > That error can be ignored.

D24678: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-15 Thread Friedrich W. H. Kossebau
kossebau added a comment. EXCLUDE_DEPRECATED_BEFORE_AND_AT needs to be implemented by someone who has overview of internals, if wanted. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D24678 To: kossebau, #plasma, mart, davidedmundson, apol Cc: kde

D24678: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-15 Thread Friedrich W. H. Kossebau
kossebau created this revision. kossebau added reviewers: Plasma, mart, davidedmundson, apol. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. kossebau requested review of this revision. REVISION SUMMARY Allows - projects linking to Plasma to hide depreca

D24665: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-15 Thread Ben Cooksley
bcooksley added a comment. As part of the transition to Gitlab, Sysadmin removed the staging repositories which a few Phabricator repositories had been configured to use. That error can be ignored. REPOSITORY R243 KArchive REVISION DETAIL https://phabricator.kde.org/D24665 To: koss

D24672: GIT_SILENT Run uncrustify-kf5 on the whole tree

2019-10-15 Thread Christoph Feck
cfeck added a reviewer: mkoller. REPOSITORY R374 KolourPaint REVISION DETAIL https://phabricator.kde.org/D24672 To: ahmadsamir, kde-frameworks-devel, mkoller Cc: ognarb, kde-frameworks-devel

D24675: Deprecate the KTcpSocket-based variant of SslUi::askIgnoreSslErrors

2019-10-15 Thread Ahmad Samir
ahmadsamir added a reviewer: dfaure. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D24675 To: vkrause, dfaure Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D24593: Modified breeze shadows

2019-10-15 Thread Niccolò Venerandi
niccolove added a comment. In D24593#546586 , @filipf wrote: > > b) radial gradients instead of linear on the four sides to make the center darker than the angles. > > Other than this, they look great to me. I think they should be uniformly

D24675: Deprecate the KTcpSocket-based variant of SslUi::askIgnoreSslErrors

2019-10-15 Thread Volker Krause
vkrause added a task: T11544: Remove KTcpSocket. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D24675 To: vkrause Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D24676: Port the KSSLD D-Bus interface from KSslError to QSslError

2019-10-15 Thread Volker Krause
vkrause added a task: T11620: Port from KSslError to QSslError. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D24676 To: vkrause Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D24676: Port the KSSLD D-Bus interface from KSslError to QSslError

2019-10-15 Thread Volker Krause
vkrause created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. vkrause requested review of this revision. REPOSITORY R241 KIO BRANCH master REVISION DETAIL https://phabricator.kde.org/D24676 AFFECTED FILES src/core/ksslcertificatemana

D24675: Deprecate the KTcpSocket-based variant of SslUi::askIgnoreSslErrors

2019-10-15 Thread Volker Krause
vkrause created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. vkrause requested review of this revision. REVISION SUMMARY The KSslErrorUiData variant below covers KTcpSocket too as a source for errors, next to QSslSocket/QNAM. Also, ad

D24672: GIT_SILENT Run uncrustify-kf5 on the whole tree

2019-10-15 Thread Ahmad Samir
ahmadsamir added a comment. T11214 is still being discussed, and we can always run clang-format later on. And uncrustify-kf5 has been used before on some of the KDE repos, so it's sort of have been fine-tuned a bit to KDE. Anyway, I don't mind ei

D24672: GIT_SILENT Run uncrustify-kf5 on the whole tree

2019-10-15 Thread Carl Schwan
ognarb added a comment. Instead of using uncrustify-kf5, shouldn't we use the clang-format file from T11214 ? REPOSITORY R374 KolourPaint REVISION DETAIL https://phabricator.kde.org/D24672 To: ahmadsamir, kde-frameworks-devel Cc: ognarb, kde-framework

D24673: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-15 Thread Friedrich W. H. Kossebau
kossebau created this revision. kossebau added reviewers: Plasma, mart, broulik. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. kossebau requested review of this revision. REVISION SUMMARY Allows - projects linking to KDeclarative to hide deprecated API

D24672: GIT_SILENT Run uncrustify-kf5 on the whole tree

2019-10-15 Thread Ahmad Samir
ahmadsamir added a task: T11576: Port kolourpaint away from kdelibs4support. REPOSITORY R374 KolourPaint REVISION DETAIL https://phabricator.kde.org/D24672 To: ahmadsamir, kde-frameworks-devel Cc: kde-frameworks-devel

D24672: GIT_SILENT Run uncrustify-kf5 on the whole tree

2019-10-15 Thread Ahmad Samir
ahmadsamir added a reviewer: kde-frameworks-devel. ahmadsamir added a subscriber: kde-frameworks-devel. REPOSITORY R374 KolourPaint REVISION DETAIL https://phabricator.kde.org/D24672 To: ahmadsamir, kde-frameworks-devel Cc: kde-frameworks-devel

D24671: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-15 Thread Friedrich W. H. Kossebau
kossebau updated this revision to Diff 67991. kossebau added a comment. keep the current filename of the export header file REPOSITORY R290 KPackage CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D24671?vs=67990&id=67991 BRANCH useECMGenerateExportHeader REVISION DETAIL https

D24667: Activate when both associatedWidget and menu are null

2019-10-15 Thread Konrad Materka
kmaterka added inline comments. INLINE COMMENTS > kstatusnotifieritem.cpp:619 > > -if (d->associatedWidget == d->menu) { > +if (d->associatedWidget && d->associatedWidget == d->menu) { > d->statusNotifierItemDBus->ContextMenu(pos.x(), pos.y()); This should fix a situation when

D24667: Activate when both associatedWidget and menu are null

2019-10-15 Thread Konrad Materka
kmaterka added a comment. This fixes issue from first comment made by @davidedmundson . Second is still not fixed. Let me copy it here: > We have one menu for the whole tray instance > W

D24671: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-15 Thread Friedrich W. H. Kossebau
kossebau added a comment. Given all the `KPACKAGE_*` macros are public, they should be consistently in the namespace, so for that purpose I also simply changed the BASE_NAME of the export macros. REPOSITORY R290 KPackage REVISION DETAIL https://phabricator.kde.org/D24671 To: kossebau,

D24671: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-15 Thread Friedrich W. H. Kossebau
kossebau created this revision. kossebau added reviewers: Frameworks, Plasma, mart, apol. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. kossebau requested review of this revision. REVISION SUMMARY Allows - projects linking to KPackage to hide deprecate

D24667: Activate when both associatedWidget and menu are null

2019-10-15 Thread Konrad Materka
kmaterka added reviewers: broulik, nicolasfella. REPOSITORY R289 KNotifications REVISION DETAIL https://phabricator.kde.org/D24667 To: kmaterka, davidedmundson, broulik, nicolasfella Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

KDE CI: Frameworks » kpackage » kf5-qt5 FreeBSDQt5.13 - Build # 20 - Still Unstable!

2019-10-15 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kpackage/job/kf5-qt5%20FreeBSDQt5.13/20/ Project: kf5-qt5 FreeBSDQt5.13 Date of build: Tue, 15 Oct 2019 14:33:12 + Build duration: 1 min 7 sec and counting JUnit Tests Name: projec

D24667: Activate when both associatedWidget and menu are null

2019-10-15 Thread Konrad Materka
kmaterka created this revision. kmaterka added a reviewer: davidedmundson. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. kmaterka requested review of this revision. REVISION SUMMARY If associatedWidget and menu are the same then instead of "activate" acti

D23760: Create new "Recent folders" icon

2019-10-15 Thread Nathaniel Graham
ngraham added a comment. In D23760#547560 , @meven wrote: > Any plans on updating this icon ? > I think we talked about it but I can't remember the conclusion. > F7600623: Screenshot_20191015_161452-2.png

D24468: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-15 Thread Friedrich W. H. Kossebau
This revision was automatically updated to reflect the committed changes. Closed by commit R236:b935a737eda7: Use ECMGenerateExportHeader to manage deprecated API better (authored by kossebau). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D24468?vs=67737&id=67985#toc REPOSITORY R236 K

D23760: Create new "Recent folders" icon

2019-10-15 Thread Méven Car
meven added a comment. Any plans on updating this icon ? I think we talked about it but I can't remember the conclusion. F7600623: Screenshot_20191015_161452-2.png REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D23760

D24665: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-15 Thread Friedrich W. H. Kossebau
kossebau added a comment. Created patch manually here, arc claims: stag...@git.kde.org: Permission denied (publickey). not yet reached sysadmin to tell what could be the issue. REPOSITORY R243 KArchive REVISION DETAIL https://phabricator.kde.org/D24665 To: kossebau, #framewor

D21204: Ensure mounted nfs filesystems matches their fstab declared counterpart

2019-10-15 Thread Méven Car
meven updated this revision to Diff 67981. meven marked an inline comment as done. meven added a comment. Avoid else after continue; REPOSITORY R245 Solid CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D21204?vs=65955&id=67981 BRANCH arcpatch-D21204 REVISION DETAIL https://ph

D24665: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-15 Thread Friedrich W. H. Kossebau
kossebau added reviewers: Frameworks, dfaure. REPOSITORY R243 KArchive REVISION DETAIL https://phabricator.kde.org/D24665 To: kossebau, #frameworks, dfaure Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D24665: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-15 Thread Friedrich W. H. Kossebau
kossebau created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. kossebau requested review of this revision. REVISION SUMMARY Allows - projects linking to KArchive to hide deprecated API up to a given version or silence deprecation warnin

D24657: Add support for _GTK_FRAME_EXTENTS

2019-10-15 Thread Nathaniel Graham
ngraham added a dependent revision: D24660: [x11] Add support for _GTK_FRAME_EXTENTS. REPOSITORY R278 KWindowSystem REVISION DETAIL https://phabricator.kde.org/D24657 To: zzag, #kwin Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D21204: Ensure mounted nfs filesystems matches their fstab declared counterpart

2019-10-15 Thread Méven Car
meven edited the summary of this revision. REPOSITORY R245 Solid REVISION DETAIL https://phabricator.kde.org/D21204 To: meven, bruns, #frameworks, ngraham Cc: broulik, dhaumann, anthonyfieroni, ngraham, bruns, apol, kde-frameworks-devel, LeGast00n, GB_2, michaelh

D24431: Restore cursor thumbnailer

2019-10-15 Thread Kai Uwe Broulik
broulik planned changes to this revision. REPOSITORY R320 KIO Extras REVISION DETAIL https://phabricator.kde.org/D24431 To: broulik, #plasma, fredrik, ngraham Cc: adridg, ngraham, kde-frameworks-devel, kfm-devel, iasensio, fprice, LeGast00n, MrPepe, fbampaloukas, alexde, GB_2, Codezela, fev

KDE CI: Frameworks » kwayland » kf5-qt5 SUSEQt5.12 - Build # 71 - Unstable!

2019-10-15 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kwayland/job/kf5-qt5%20SUSEQt5.12/71/ Project: kf5-qt5 SUSEQt5.12 Date of build: Tue, 15 Oct 2019 13:04:43 + Build duration: 20 min and counting BUILD ARTIFACTS abi-compatibility-r

KDE CI: Frameworks » kwayland » kf5-qt5 FreeBSDQt5.13 - Build # 25 - Still Unstable!

2019-10-15 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kwayland/job/kf5-qt5%20FreeBSDQt5.13/25/ Project: kf5-qt5 FreeBSDQt5.13 Date of build: Tue, 15 Oct 2019 13:04:42 + Build duration: 18 min and counting JUnit Tests Name: projectroot

D24662: Fix non-integer scale copy on creation of OutputChangeSet

2019-10-15 Thread Friedrich W. H. Kossebau
This revision was automatically updated to reflect the committed changes. Closed by commit R127:264e94052c5c: Fix non-integer scale copy on creation of OutputChangeSet (authored by kossebau). REPOSITORY R127 KWayland CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D24662?vs=67972&id=67

D24662: Fix non-integer scale copy on creation of OutputChangeSet

2019-10-15 Thread Friedrich W. H. Kossebau
kossebau added a comment. Found by D24663 when trying to implement EXCLUDE_DEPRECATED_BEFORE_AND_AT and doing test builds :) REPOSITORY R127 KWayland BRANCH fixOutputChangeSetScaleCopy REVISION DETAIL https://phabricator.kde.org/D24662 To: kosseba

D24663: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-15 Thread Friedrich W. H. Kossebau
kossebau created this revision. kossebau added a reviewer: Plasma. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. kossebau requested review of this revision. REVISION SUMMARY Allows - projects linking to KWayland to hide deprecated API up to a given ver

D24662: Fix non-integer scale copy on creation of OutputChangeSet

2019-10-15 Thread David Edmundson
davidedmundson accepted this revision. This revision is now accepted and ready to land. REPOSITORY R127 KWayland BRANCH fixOutputChangeSetScaleCopy REVISION DETAIL https://phabricator.kde.org/D24662 To: kossebau, #plasma, davidedmundson Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh,

D24662: Fix non-integer scale copy on creation of OutputChangeSet

2019-10-15 Thread Friedrich W. H. Kossebau
kossebau created this revision. kossebau added a reviewer: Plasma. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. kossebau requested review of this revision. REPOSITORY R127 KWayland BRANCH fixOutputChangeSetScaleCopy REVISION DETAIL https://phabricato

D24630: Replace all qWarning and related calls with categorised logging

2019-10-15 Thread Arjen Hiemstra
This revision was automatically updated to reflect the committed changes. Closed by commit R249:b45c3cea521e: Replace all qWarning and related calls with categorised logging (authored by ahiemstra). REPOSITORY R249 KI18n CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D24630?vs=67893&i

D24657: Add support for _GTK_FRAME_EXTENTS

2019-10-15 Thread Vlad Zahorodnii
zzag removed a dependent revision: D24660: [x11] Add support for _GTK_FRAME_EXTENTS. REPOSITORY R278 KWindowSystem REVISION DETAIL https://phabricator.kde.org/D24657 To: zzag, #kwin Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D24657: Add support for _GTK_FRAME_EXTENTS

2019-10-15 Thread Vlad Zahorodnii
zzag added a dependent revision: D24660: [x11] Add support for _GTK_FRAME_EXTENTS. REPOSITORY R278 KWindowSystem REVISION DETAIL https://phabricator.kde.org/D24657 To: zzag, #kwin Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D24657: Add support for _GTK_FRAME_EXTENTS

2019-10-15 Thread Vlad Zahorodnii
zzag created this revision. zzag added a reviewer: KWin. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. zzag requested review of this revision. REVISION SUMMARY _GTK_FRAME_EXTENTS is a property set by the client that specifies extents of the client-side dr

D23851: Avoid sending KDirNotify::emitFilesAdded when the emptytrashjob finishes

2019-10-15 Thread Méven Car
meven added a comment. If there are no objections I will merge this soonish REPOSITORY R241 KIO BRANCH arcpatch-D23851 REVISION DETAIL https://phabricator.kde.org/D23851 To: meven, broulik, ngraham Cc: elvisangelaccio, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D21783: [WIP]Show more details in warning dialog shown before starting a privileged operation

2019-10-15 Thread Chinmoy Ranjan Pradhan
chinmoyr added inline comments. INLINE COMMENTS > dfaure wrote in slavebase.h:964 > Missing "m" prefix like the other members, but wait isn't this BIC? > Adding a new member to an exported class certainly is. This needs to go into > the SlaveBasePrivate class instead. Since we need details

D21783: [WIP]Show more details in warning dialog shown before starting a privileged operation

2019-10-15 Thread Chinmoy Ranjan Pradhan
chinmoyr updated this revision to Diff 67955. chinmoyr marked 3 inline comments as done. chinmoyr added a comment. Addressed the comments. REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D21783?vs=59738&id=67955 BRANCH arcpatch-D21783 REVISION DETAIL https:

KDE CI: Frameworks » kio » kf5-qt5 SUSEQt5.12 - Build # 269 - Successful!

2019-10-15 Thread CI System
BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks/job/kio/job/kf5-qt5%20SUSEQt5.12/269/ Project: kf5-qt5 SUSEQt5.12 Date of build: Tue, 15 Oct 2019 09:16:38 + Build duration: 24 min and counting BUILD ARTIFACTS acc/KF5KIO-5.64.0.xmllog

KDE CI: Frameworks » kio » kf5-qt5 FreeBSDQt5.13 - Build # 138 - Still unstable!

2019-10-15 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kio/job/kf5-qt5%20FreeBSDQt5.13/138/ Project: kf5-qt5 FreeBSDQt5.13 Date of build: Tue, 15 Oct 2019 09:16:22 + Build duration: 9 min 47 sec and counting JUnit Tests Name: projectro

KDE CI: Frameworks » kio » kf5-qt5 SUSEQt5.13 - Build # 151 - Successful!

2019-10-15 Thread CI System
BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks/job/kio/job/kf5-qt5%20SUSEQt5.13/151/ Project: kf5-qt5 SUSEQt5.13 Date of build: Tue, 15 Oct 2019 09:16:57 + Build duration: 8 min 30 sec and counting BUILD ARTIFACTS acc/KF5KIO-5.64.0.

D24655: give all pointers in slavebase a default init to nullptr

2019-10-15 Thread Méven Car
meven added a comment. Thanks @sitter REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D24655 To: sitter, dfaure, meven, vkrause Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D24656: [SlaveBase] Prevent crash by properly initializing Kconfig objects

2019-10-15 Thread Méven Car
meven abandoned this revision. meven added a comment. Superseeded by https://phabricator.kde.org/D24655 REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D24656 To: meven, #frameworks, vkrause, dfaure Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D24656: [SlaveBase] Prevent crash by properly initializing Kconfig objects

2019-10-15 Thread Méven Car
meven created this revision. meven added reviewers: Frameworks, vkrause, dfaure. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. meven requested review of this revision. REVISION SUMMARY As suggested at https://phabricator.kde.org/R241:a497d47c45a5#inline-16

D24655: give all pointers in slavebase a default init to nullptr

2019-10-15 Thread Harald Sitter
This revision was automatically updated to reflect the committed changes. Closed by commit R241:c2bd18f17e61: give all pointers in slavebase a default init to nullptr (authored by sitter). REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D24655?vs=67944&id=67947 REV

D24655: give all pointers in slavebase a default init to nullptr

2019-10-15 Thread Volker Krause
vkrause accepted this revision. This revision is now accepted and ready to land. REPOSITORY R241 KIO BRANCH master REVISION DETAIL https://phabricator.kde.org/D24655 To: sitter, dfaure, meven, vkrause Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D24655: give all pointers in slavebase a default init to nullptr

2019-10-15 Thread Harald Sitter
sitter created this revision. sitter added reviewers: dfaure, meven. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. sitter requested review of this revision. REVISION SUMMARY lest they don't get initialized and then end up crashing should get get deleted (

D24582: Replace usage of SlaveBase::config() by SlaveBase::mapConfig()

2019-10-15 Thread Méven Car
meven added a comment. In D24582#547267 , @dfaure wrote: > Looks fine, but a bit verbose. Maybe we could have configValue() overloads (in SlaveBase) for bool, int and QString, to cover the most common use cases? > > - mapConfig().value(QStr

D24641: Collect more information from version control systems

2019-10-15 Thread Harald Sitter
sitter added a comment. Do we have a request for this outside kbibtex? My attitude towards adding things to ECM is always "is there more than one user". ECM_SOURCE_VERSION_CONTROL_COMMIT_COUNT seems very specific and opinionated (what's origin? does it count commits in merges? also the q

D24582: Replace usage of SlaveBase::config() by SlaveBase::mapConfig()

2019-10-15 Thread Méven Car
meven updated this revision to Diff 67943. meven added a comment. Add configValue overloads to simplify code REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D24582?vs=67772&id=67943 BRANCH arcpatch-D24582 REVISION DETAIL https://phabricator.kde.org/D24582

KDE CI: Frameworks » kio » kf5-qt5 FreeBSDQt5.13 - Build # 136 - Still Unstable!

2019-10-15 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kio/job/kf5-qt5%20FreeBSDQt5.13/136/ Project: kf5-qt5 FreeBSDQt5.13 Date of build: Tue, 15 Oct 2019 07:14:46 + Build duration: 34 min and counting JUnit Tests Name: projectroot Fai

KDE CI: Frameworks » kio » kf5-qt5 SUSEQt5.12 - Build # 267 - Still Unstable!

2019-10-15 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kio/job/kf5-qt5%20SUSEQt5.12/267/ Project: kf5-qt5 SUSEQt5.12 Date of build: Tue, 15 Oct 2019 07:14:46 + Build duration: 33 min and counting BUILD ARTIFACTS acc/KF5KIO-5.64.0.xmllo

D23523: [SlaveBase] Use QMap instead of KConfig to store ioslave config

2019-10-15 Thread Méven Car
This revision was automatically updated to reflect the committed changes. Closed by commit R241:a497d47c45a5: [SlaveBase] Use QMap instead of KConfig to store ioslave config (authored by meven). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D23523?vs=67763&id=67940#toc REPOSITORY R241

D23523: [SlaveBase] Use QMap instead of KConfig to store ioslave config

2019-10-15 Thread Méven Car
meven added a comment. In D23523#547265 , @dfaure wrote: > Looks fine, but it seems that the uploaded diff is only the delta compared to the previous diff, not the overall change. Anyhow, feel free to push both. It was on purpose to have