Re: Problems in KWayland causes by API and ABI compatibility promises

Hello, On Thursday, 16 April 2020 23:38:23 CEST David Edmundson wrote: > On Wed, Apr 8, 2020 at 5:10 PM Kevin Ottens wrote: > > On Wednesday, 1 April 2020 14:04:10 CEST David Edmundson wrote: > > > Here is a list of active uses of the KWayland::Client API. > > > > > > frameworks > > > > > >

D28874: Taiwanese holidays

shrapnel updated this revision to Diff 80346. shrapnel added a comment. Uploaded .diff instead of .patch REPOSITORY R175 KHolidays CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28874?vs=80303=80346 REVISION DETAIL https://phabricator.kde.org/D28874 AFFECTED FILES

KDE CI: Frameworks » plasma-framework » kf5-qt5 FreeBSDQt5.14 - Build # 27 - Fixed!

BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks/job/plasma-framework/job/kf5-qt5%20FreeBSDQt5.14/27/ Project: kf5-qt5 FreeBSDQt5.14 Date of build: Fri, 17 Apr 2020 04:47:27 + Build duration: 3 min 11 sec and counting JUnit Tests

D19913: [plasma-framework] make it compiles without foreach

mlaurent updated this revision to Diff 80344. mlaurent added a comment. Use m_menu->addActions(m_dropActions); REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D19913?vs=80182=80344 BRANCH compile_without_foreach (branched from master)

D19913: [plasma-framework] make it compiles without foreach

This revision was automatically updated to reflect the committed changes. Closed by commit R242:48f60533b922: [plasma-framework] make it compiles without foreach (authored by mlaurent). REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE

D7563: Add privilegeExecution field to file protocol description

ngraham added a comment. [insert I-have-no-idea-what-I'm-doing dog meme here] When trying to create items in root-owned locations, I'm getting an errors saying "The process for the file protocol died unexpectedly." or else Dolphin simply crashes with a totally unhelpful backtrace.

D7563: Add privilegeExecution field to file protocol description

chinmoyr requested changes to this revision. chinmoyr added inline comments. This revision now requires changes to proceed. INLINE COMMENTS > file_unix.cpp:1331 > +KAuth::Action::DetailsMap details; > +details.insert(KAuth::Action::AuthDetail::DetailMessage, >

D7563: Add privilegeExecution field to file protocol description

ngraham updated this revision to Diff 80343. ngraham added a comment. Set the patch base REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D7563?vs=80342=80343 BRANCH arcpatch-D7563 REVISION DETAIL https://phabricator.kde.org/D7563 AFFECTED FILES

D7563: Add privilegeExecution field to file protocol description

ngraham updated this revision to Diff 80342. ngraham added a comment. - Fix build failure - Add more stuff (thanks @chinmoyr) REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D7563?vs=80341=80342 BRANCH arcpatch-D7563 REVISION DETAIL

D7563: Add privilegeExecution field to file protocol description

ngraham commandeered this revision. ngraham added a reviewer: chinmoyr. ngraham added a comment. Yoink! REPOSITORY R241 KIO BRANCH master REVISION DETAIL https://phabricator.kde.org/D7563 To: ngraham, #frameworks, dfaure, chinmoyr Cc: kde-frameworks-devel, feverfew, mreeves, mati865,

D7563: Add privilegeExecution field to file protocol description

ngraham added a comment. Thanks @chinmoyr! REPOSITORY R241 KIO BRANCH master REVISION DETAIL https://phabricator.kde.org/D7563 To: chinmoyr, #frameworks, dfaure Cc: kde-frameworks-devel, feverfew, mreeves, mati865, ngraham, elvisangelaccio, LeGast00n, cblack, michaelh, bruns

D7563: Add privilegeExecution field to file protocol description

chinmoyr updated this revision to Diff 80341. chinmoyr marked an inline comment as done. chinmoyr added a comment. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. Rebased on master. REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE

D17816: Support for xattrs on kio copy/move

bruns requested changes to this revision. bruns added inline comments. This revision now requires changes to proceed. INLINE COMMENTS > file_unix.cpp:141 > +// Get the list of keys > +ssize_t listlen = 512; > +QByteArray keylist(listlen, Qt::Uninitialized); The idea is almost right,

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

BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kio/job/kf5-qt5%20SUSEQt5.14/54/ Project: kf5-qt5 SUSEQt5.14 Date of build: Thu, 16 Apr 2020 23:43:51 + Build duration: 11 min and counting BUILD ARTIFACTS

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

BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kio/job/kf5-qt5%20FreeBSDQt5.14/62/ Project: kf5-qt5 FreeBSDQt5.14 Date of build: Thu, 16 Apr 2020 23:43:51 + Build duration: 7 min 51 sec and counting JUnit Tests Name:

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

BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kio/job/kf5-qt5%20SUSEQt5.12/553/ Project: kf5-qt5 SUSEQt5.12 Date of build: Thu, 16 Apr 2020 23:43:51 + Build duration: 5 min 44 sec and counting BUILD ARTIFACTS

D28902: CopyJob: when stat'ing the dest, use StatBasic.

dfaure created this revision. dfaure added a reviewer: meven. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. dfaure requested review of this revision. REVISION SUMMARY However the StatDetail enum doesn't specify which enum value will get us UDS_LOCAL_PATH

D28901: Add KIO::StatRecursiveSize detail value so kio_trash only does this on demand

dfaure created this revision. dfaure added reviewers: meven, ngraham. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. dfaure requested review of this revision. REVISION SUMMARY This indirectly fixes the testtrash unittest which noticed that the directory

D17816: Support for xattrs on kio copy/move

D24773: kio_trash: Add size, modification, access and create date for trash:/

dfaure added a comment. Yep, that's it. Renaming trash:/A to trash:/B calls KIO::moveAs which stats trash:/ [useful to resolve desktop:/ to a local dir for instance], which now updates the on-disk cache... with 'A', before it gets renamed to 'B'. I think I know what the best solution

D19913: [plasma-framework] make it compiles without foreach

apol accepted this revision. apol added inline comments. INLINE COMMENTS > dropmenu.cpp:80 > } else if (m_menu) { > -foreach (QAction *action, m_dropActions) { > +for (QAction *action : qAsConst(m_dropActions)) { > m_menu->addAction(action);

D28900: Fix wayland scanner warnings

davidedmundson accepted this revision. This revision is now accepted and ready to land. REPOSITORY R240 Extra CMake Modules BRANCH master REVISION DETAIL https://phabricator.kde.org/D28900 To: apol, #build_system, #kwin, #frameworks, davidedmundson Cc: kde-frameworks-devel,

D28900: Fix wayland scanner warnings

apol created this revision. apol added reviewers: Build System, KWin, Frameworks. Herald added projects: Frameworks, Build System. Herald added subscribers: kde-buildsystem, kde-frameworks-devel. apol requested review of this revision. REVISION SUMMARY Tells cmake not to automoc certain files

D26888: work around to fully support the windows backend

albertvaka added a comment. @vonreth @broulik Can you folks give this a final review? We have been building KDE Connect for Windows off a custom branch with this patch, but I would like to use official builds of KNotifications so it would be great if we can merge it. REPOSITORY R289

D26888: work around to fully support the windows backend

brute4s99 marked 2 inline comments as done. REPOSITORY R289 KNotifications REVISION DETAIL https://phabricator.kde.org/D26888 To: brute4s99, vonreth, broulik, #kde_connect Cc: albertvaka, meven, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D26888: work around to fully support the windows backend

brute4s99 updated this revision to Diff 80333. brute4s99 added a comment. work around to fully support the windows backend REPOSITORY R289 KNotifications CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D26888?vs=74291=80333 BRANCH arcpatch-D26888 REVISION DETAIL

D28891: Nicaragua Holidays

ngraham added a comment. You're very welcome. Thank you for the patch! REPOSITORY R175 KHolidays REVISION DETAIL https://phabricator.kde.org/D28891 To: carguello, winterz, #frameworks Cc: ngraham, kde-frameworks-devel, #vdg, LeGast00n, cblack, michaelh, bruns

D28891: Nicaragua Holidays

carguello added a comment. Thank you for your time and support.  REPOSITORY R175 KHolidays REVISION DETAIL https://phabricator.kde.org/D28891 To: carguello, winterz, #frameworks Cc: ngraham, kde-frameworks-devel, #vdg, LeGast00n, cblack, michaelh, bruns

D28891: Nicaragua Holidays

ngraham added a comment. Great job! I'll hand it over to @winterz and other reviewers now. REPOSITORY R175 KHolidays REVISION DETAIL https://phabricator.kde.org/D28891 To: carguello, winterz, #frameworks Cc: ngraham, kde-frameworks-devel, #vdg, LeGast00n, cblack, michaelh, bruns

D28891: Nicaragua Holidays

carguello added a comment. Okay, I'm confident it went through this time REPOSITORY R175 KHolidays REVISION DETAIL https://phabricator.kde.org/D28891 To: carguello, winterz, #frameworks Cc: ngraham, kde-frameworks-devel, #vdg, LeGast00n, cblack, michaelh, bruns

D28891: Nicaragua Holidays

carguello updated this revision to Diff 80322. carguello added a comment. Adding Nicaragua Holidays file back REPOSITORY R175 KHolidays CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28891?vs=80319=80322 BRANCH master REVISION DETAIL https://phabricator.kde.org/D28891

D28891: Nicaragua Holidays

ngraham added a comment. git add [the path to the new file you want to add]` arc diff --update D28891 REPOSITORY R175 KHolidays REVISION DETAIL https://phabricator.kde.org/D28891 To: carguello, winterz, #frameworks Cc: ngraham, kde-frameworks-devel, #vdg, LeGast00n, cblack,

D28891: Nicaragua Holidays

carguello added a comment.  How can I add it back? Sorry this is my first time using git and Phabricator REPOSITORY R175 KHolidays REVISION DETAIL https://phabricator.kde.org/D28891 To: carguello, winterz, #frameworks Cc: ngraham, kde-frameworks-devel, #vdg, LeGast00n, cblack,

D28891: Nicaragua Holidays

ngraham added a comment. Now the new file you wanted to add is gone. :) With Phabricator, you need to include all commits in your patch, not just the latest one. REPOSITORY R175 KHolidays REVISION DETAIL https://phabricator.kde.org/D28891 To: carguello, winterz, #frameworks Cc:

D28891: Nicaragua Holidays

carguello updated this revision to Diff 80319. carguello added a comment. Previously messed up holidays/holidays.qrc I have fixed it now REPOSITORY R175 KHolidays CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28891?vs=80318=80319 BRANCH master REVISION DETAIL

D28891: Nicaragua Holidays

carguello added a comment. I think I messed up  REPOSITORY R175 KHolidays REVISION DETAIL https://phabricator.kde.org/D28891 To: carguello, winterz, #frameworks Cc: ngraham, kde-frameworks-devel, #vdg, LeGast00n, cblack, michaelh, bruns

Re: Problems in KWayland causes by API and ABI compatibility promises

On Wed, Apr 8, 2020 at 5:10 PM Kevin Ottens wrote: > > Hello, > > On Wednesday, 1 April 2020 14:04:10 CEST David Edmundson wrote: > > Here is a list of active uses of the KWayland::Client API. > > > > frameworks > > plasma-framework (for window positioning) > > > > apps: > > spectacle

D28891: Nicaragua Holidays

carguello updated this revision to Diff 80318. carguello added a comment. Remove file extension .qrc per instructions REPOSITORY R175 KHolidays CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28891?vs=80309=80318 BRANCH master REVISION DETAIL

Re: Problems in KWayland causes by API and ABI compatibility promises

> Given Doodle's take forever and everyone's probably quarantined > anyway, I propose we do it just after the Monday Plasma meeting. If > someone is eager to share an opinion and can't make that, message here > and we'll choose something more formally. That didn't really work out. Doodle it is!

D28891: Nicaragua Holidays

ngraham removed a reviewer: VDG. REPOSITORY R175 KHolidays REVISION DETAIL https://phabricator.kde.org/D28891 To: carguello, winterz, #frameworks Cc: ngraham, kde-frameworks-devel, #vdg, LeGast00n, cblack, michaelh, bruns

D28891: Nicaragua Holidays

ngraham added reviewers: winterz, Frameworks. ngraham added a comment. Thanks for the patch! You need to update the `holidays/holidays.qrc` file too, to mention this new file that you're adding. Also I don't think you need to have the `.qrc` extension on the file you're adding. REPOSITORY

D28891: Nicaragua Holidays

carguello added a reviewer: VDG. REPOSITORY R175 KHolidays REVISION DETAIL https://phabricator.kde.org/D28891 To: carguello, #vdg Cc: kde-frameworks-devel, #vdg, LeGast00n, cblack, michaelh, ngraham, bruns

D28892: [autotests] Optimistic attempt to fix RemoteAccessTest reilability

davidedmundson updated this revision to Diff 80312. davidedmundson added a comment. update REPOSITORY R127 KWayland CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28892?vs=80311=80312 BRANCH master REVISION DETAIL https://phabricator.kde.org/D28892 AFFECTED FILES

D28892: [autotests] Optimistic attempt to fix RemoteAccessTest reilability

davidedmundson updated this revision to Diff 80311. davidedmundson added a comment. spelling REPOSITORY R127 KWayland CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28892?vs=80310=80311 BRANCH master REVISION DETAIL https://phabricator.kde.org/D28892 AFFECTED FILES

D28892: [autotests] Optimistic attempt to fix RemoteAccessTest reilability

davidedmundson created this revision. davidedmundson added a reviewer: KWin. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. davidedmundson requested review of this revision. REVISION SUMMARY In this test we are waiting on 4 events. 2 things via 2 threads.

D28891: Nicaragua Holidays

carguello created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. carguello requested review of this revision. REVISION SUMMARY Added file containing Nicaragua holidays in holidays/plan2/holiday_ni_es REPOSITORY R175 KHolidays BRANCH

D28808: [ExpandableListItem] make it touch-friendly

ngraham added a comment. Good question. I'm not sure it's possible here; the list views that use this component as a delegate might have to learn how to unset the selection with their own `TapHandler`. I'm not 100% sure on that though. REPOSITORY R242 Plasma Framework (Library) REVISION

Re: KDE CI: Administration » Dependency Build Extragear stable-kf5-qt5 AndroidQt5.14 - Build # 10 - Still Failing!

Hi, this looks like a simple prison build failure to me. However, I cannot reprocuce this with the binary factory container and the current Prison Android CI builds is blue. There is a single red Android build in the Prison CI history (https://build.kde.org/job/Frameworks/

D28808: [ExpandableListItem] make it touch-friendly

niccolove added a comment. This generically works, except for: would it be possible to make it so that clicking on an empty point will drop the list and button hover status? otherwise, you kind of get stuck to this situation: F8242311: Screenshot_20200416_224831.png

D28808: [ExpandableListItem] make it touch-friendly

niccolove added a reviewer: niccolove. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D28808 To: ngraham, #plasma, apol, #vdg, niccolove Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D28811: Add drawer on mobile and clean code

ochurlaud accepted this revision. ochurlaud added a comment. In D28811#649209 , @ognarb wrote: > I now generate the CSS from aether-sass. The code can be found https://invent.kde.org/websites/aether-sass/-/blob/master/css/aether-sidebar.scss.

Re: KDE CI: Administration » Dependency Build Extragear stable-kf5-qt5 AndroidQt5.14 - Build # 10 - Still Failing!

On Fri, Apr 17, 2020 at 6:37 AM Johan Ouwerkerk wrote: > > > On Thu, Apr 16, 2020 at 10:51 AM Ben Cooksley wrote: >> >> Hi all, >> >> Does anyone know why the below would have suddenly started failing a little >> while back? >> >> Thanks, >> Ben > > > Based on the error message I would expect

D19913: [plasma-framework] make it compiles without foreach

mlaurent added a comment. Ping ?:) REPOSITORY R242 Plasma Framework (Library) BRANCH compile_without_foreach (branched from master) REVISION DETAIL https://phabricator.kde.org/D19913 To: mlaurent, dfaure Cc: ahmadsamir, nicolasfella, broulik, apol, kde-frameworks-devel, LeGast00n,

Re: KDE CI: Administration » Dependency Build Extragear stable-kf5-qt5 AndroidQt5.14 - Build # 10 - Still Failing!

On Thu, Apr 16, 2020 at 10:51 AM Ben Cooksley wrote: > Hi all, > > Does anyone know why the below would have suddenly started failing a > little while back? > > Thanks, > Ben > Based on the error message I would expect this to be related to the effort to fix Android pkgs not being generated for

D27840: Introduce SettingState* elements to ease KCM writing

ervin updated this revision to Diff 80305. ervin added a comment. Fix issues found with Cyril's patches on various KCMs. REPOSITORY R296 KDeclarative CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27840?vs=78698=80305 REVISION DETAIL https://phabricator.kde.org/D27840

D28874: Taiwanese holidays

Zren added a comment. Phab only understands `.diff`. Phab does not recognize `.patch`. git diff --staged > D28874.diff Or: git diff origin/master > D28874.diff REPOSITORY R175 KHolidays REVISION DETAIL https://phabricator.kde.org/D28874 To: shrapnel, #vdg, Zren,

D28819: [KRichTextEdit] Always treat key press as single modification in undo stack

poboiko added a comment. @dfaure There seem to be a regression caused by this patch: if I reach the end of current view, and then press Enter so it has to scroll down, it scrolls to the very beginning of the document instead. This is the same issue as in ancient bug 195828

D28874: Taiwanese holidays

shrapnel updated this revision to Diff 80303. shrapnel added a comment. Fixed the typos REPOSITORY R175 KHolidays CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28874?vs=80288=80303 REVISION DETAIL https://phabricator.kde.org/D28874 AFFECTED FILES

D28865: [PhraseAndIterator] Remove temporary arrays when checking matches

This revision was automatically updated to reflect the committed changes. Closed by commit R293:a27a2282da3f: [PhraseAndIterator] Remove temporary arrays when checking matches (authored by bruns). REPOSITORY R293 Baloo CHANGES SINCE LAST UPDATE

D28864: [Extractor] Better balance idle and busy mode

This revision was automatically updated to reflect the committed changes. Closed by commit R293:ef3535630914: [Extractor] Better balance idle and busy mode (authored by bruns). REPOSITORY R293 Baloo CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28864?vs=80297=80300 REVISION DETAIL

D28883: Add wrapper for wl_global_remove

apol accepted this revision. This revision is now accepted and ready to land. REPOSITORY R127 KWayland BRANCH master REVISION DETAIL https://phabricator.kde.org/D28883 To: davidedmundson, #plasma, apol Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D28864: [Extractor] Better balance idle and busy mode

ngraham accepted this revision. ngraham added a comment. This revision is now accepted and ready to land. Thanks, makes sense to me! REPOSITORY R293 Baloo BRANCH submit REVISION DETAIL https://phabricator.kde.org/D28864 To: bruns, #baloo, ngraham Cc: kde-frameworks-devel, hurikhan77,

D28863: [Extractor] Fix idle monitoring

This revision was automatically updated to reflect the committed changes. Closed by commit R293:0d56f65f03c3: [Extractor] Fix idle monitoring (authored by bruns). REPOSITORY R293 Baloo CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28863?vs=80237=80299 REVISION DETAIL

D28862: [Extractor] Remove IdleStateMonitor wrapper class

This revision was automatically updated to reflect the committed changes. Closed by commit R293:bd502bf0af46: [Extractor] Remove IdleStateMonitor wrapper class (authored by bruns). REPOSITORY R293 Baloo CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28862?vs=80295=80298 REVISION

D28865: [PhraseAndIterator] Remove temporary arrays when checking matches

ngraham accepted this revision. This revision is now accepted and ready to land. REPOSITORY R293 Baloo BRANCH submit REVISION DETAIL https://phabricator.kde.org/D28865 To: bruns, #baloo, ngraham Cc: kde-frameworks-devel, hurikhan77, lots0logs, LeGast00n, cblack, fbampaloukas, domson,

D28864: [Extractor] Better balance idle and busy mode

bruns updated this revision to Diff 80297. bruns edited the summary of this revision. bruns added a comment. extend summary REPOSITORY R293 Baloo CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28864?vs=80296=80297 BRANCH submit REVISION DETAIL

D28864: [Extractor] Better balance idle and busy mode

ngraham added a comment. Can you explain the conceptual reasoning behind this change? REPOSITORY R293 Baloo REVISION DETAIL https://phabricator.kde.org/D28864 To: bruns, #baloo, ngraham Cc: kde-frameworks-devel, hurikhan77, lots0logs, LeGast00n, cblack, fbampaloukas, domson,

D28863: [Extractor] Fix idle monitoring

ngraham accepted this revision. This revision is now accepted and ready to land. REPOSITORY R293 Baloo BRANCH submit REVISION DETAIL https://phabricator.kde.org/D28863 To: bruns, #baloo, ngraham Cc: kde-frameworks-devel, hurikhan77, lots0logs, LeGast00n, cblack, fbampaloukas, domson,

D28864: [Extractor] Better balance idle and busy mode

bruns updated this revision to Diff 80296. bruns added a comment. Extend commit message, reduce idle timeout REPOSITORY R293 Baloo CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28864?vs=80238=80296 REVISION DETAIL https://phabricator.kde.org/D28864 AFFECTED FILES

D28862: [Extractor] Remove IdleStateMonitor wrapper class

ngraham accepted this revision. This revision is now accepted and ready to land. REPOSITORY R293 Baloo BRANCH master REVISION DETAIL https://phabricator.kde.org/D28862 To: bruns, #baloo, ngraham Cc: broulik, kde-frameworks-devel, hurikhan77, lots0logs, LeGast00n, cblack, fbampaloukas,

D28862: [Extractor] Remove IdleStateMonitor wrapper class

bruns updated this revision to Diff 80295. bruns added a comment. add line break before closing brace in lambda, use 2 minutes REPOSITORY R293 Baloo CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28862?vs=80236=80295 BRANCH master REVISION DETAIL

D28862: [Extractor] Remove IdleStateMonitor wrapper class

bruns added inline comments. INLINE COMMENTS > broulik wrote in app.cpp:63 > Coding style Side note - both the Frameworks and the reference Qt coding style say nothing about lambdas. If you read the `connect(...);` as a single statement, it should stay as is. If you read the lambda as a

D28874: Taiwanese holidays

Zren added a comment. There's a few extra spaces at the end of `"植樹節 "` and `"中秋節 補假 "` too. Not sure how well you know `git`, but make sure to run `git add holidays/plan2/holiday_tw_zh` again after making your changes, before running `git diff --staged` to upload the modified diff.

D28874: Taiwanese holidays

winterz added a comment. problems: - "religous" should be "religious" - missing ':' at the start of Constitution Day - missing ':' at the start of Christmas Day REPOSITORY R175 KHolidays REVISION DETAIL https://phabricator.kde.org/D28874 To: shrapnel, #vdg, Zren, winterz Cc:

D28874: Taiwanese holidays

winterz requested changes to this revision. winterz added a comment. This revision now requires changes to proceed. testing fails. don't know why yet. REPOSITORY R175 KHolidays REVISION DETAIL https://phabricator.kde.org/D28874 To: shrapnel, #vdg, Zren, winterz Cc: winterz,

D28701: Add KPackage support to KNewStuffCore

leinir edited the summary of this revision. REPOSITORY R304 KNewStuff REVISION DETAIL https://phabricator.kde.org/D28701 To: leinir, #plasma, #knewstuff, #frameworks, ngraham, mart, davidedmundson Cc: ngraham, kde-frameworks-devel, LeGast00n, cblack, michaelh, bruns

D28701: Add KPackage support to KNewStuffCore

leinir updated this revision to Diff 80289. leinir added a comment. Address @ngraham's (and my own) worry about the synchronous behaviour exhibited by KPackage... Something a bit like this probably wants to go into KPackage itself, perhaps we can consider this after we've done a bit of

D28874: Taiwanese holidays

shrapnel updated this revision to Diff 80288. shrapnel added a comment. Hopefully the changes to both files are there now REPOSITORY R175 KHolidays CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28874?vs=80277=80288 REVISION DETAIL https://phabricator.kde.org/D28874 AFFECTED

D28882: Create protocol to manage video feeds

apol updated this revision to Diff 80284. apol added a subscriber: zzag. apol added a comment. renamed protocol as per @zzag's suggestion REPOSITORY R127 KWayland CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28882?vs=80278=80284 BRANCH master REVISION DETAIL

D28874: Taiwanese holidays

Zren added a comment. Now you're missing `holidays/plan2/holiday_tw_zh` which can still be seen here: https://phabricator.kde.org/D28874?id=80260 git add holidays/holidays.qrc git add holidays/plan2/holiday_tw_zh git diff --staged > D28874.diff REPOSITORY R175 KHolidays

D28745: Skip caching thumbnails on encrypted filesystems

marcingu added a comment. Ok, Thanks! I'll check it this weekend. REPOSITORY R320 KIO Extras REVISION DETAIL https://phabricator.kde.org/D28745 To: marcingu, ivan, broulik, #dolphin, ngraham, meven, bruns Cc: bruns, meven, ngraham, kde-frameworks-devel, kfm-devel, azyx, nikolaik,

KDE CI: Frameworks » kirigami » kf5-qt5 SUSEQt5.14 - Build # 20 - Unstable!

BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kirigami/job/kf5-qt5%20SUSEQt5.14/20/ Project: kf5-qt5 SUSEQt5.14 Date of build: Thu, 16 Apr 2020 14:48:35 + Build duration: 2 min 47 sec and counting BUILD ARTIFACTS

KDE CI: Frameworks » kirigami » kf5-qt5 FreeBSDQt5.14 - Build # 24 - Unstable!

BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kirigami/job/kf5-qt5%20FreeBSDQt5.14/24/ Project: kf5-qt5 FreeBSDQt5.14 Date of build: Thu, 16 Apr 2020 14:48:36 + Build duration: 1 min 16 sec and counting JUnit Tests Name:

KDE CI: Frameworks » kirigami » kf5-qt5 SUSEQt5.12 - Build # 417 - Unstable!

BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kirigami/job/kf5-qt5%20SUSEQt5.12/417/ Project: kf5-qt5 SUSEQt5.12 Date of build: Thu, 16 Apr 2020 14:48:35 + Build duration: 1 min 53 sec and counting BUILD ARTIFACTS

D28882: Create protocol to manage video feeds

apol added a dependent revision: D28884: screencasting: Adoption of the org_kde_plasma_video_streaming protocol. REPOSITORY R127 KWayland REVISION DETAIL https://phabricator.kde.org/D28882 To: apol, #kwin, jgrulich Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D28883: Add wrapper for wl_global_remove

davidedmundson created this revision. davidedmundson added a reviewer: Plasma. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. davidedmundson requested review of this revision. REVISION SUMMARY Removes the Global from the registry, but does not delete the

D28882: Create protocol to manage video feeds

apol created this revision. apol added reviewers: KWin, jgrulich. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. apol requested review of this revision. REVISION SUMMARY Creates a protocol based on Pipewire that allows us to request feeds to be set up by

D28874: Taiwanese holidays

shrapnel updated this revision to Diff 80277. shrapnel edited the summary of this revision. shrapnel added a comment. Added holiday_tw_zh to holidays.qrc-file (I'm new to git/phabricator so hopefully it makes sense) REPOSITORY R175 KHolidays CHANGES SINCE LAST UPDATE

D28874: Taiwanese holidays

winterz added a comment. you need to add the change to holidays.qrc to this patch REPOSITORY R175 KHolidays REVISION DETAIL https://phabricator.kde.org/D28874 To: shrapnel, #vdg, Zren Cc: winterz, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D28854: [KRichTextWidget] Add support for headings

poboiko marked 3 inline comments as done. poboiko added a comment. In D28854#649189 , @abstractdevelop wrote: > Hey, thanks for this @poboiko I used to have to implement this myself, so this will be very useful in my app, O20.Word. ;)

D28745: Skip caching thumbnails on encrypted filesystems

bruns added inline comments. INLINE COMMENTS > thumbnail.cpp:726 > > +const auto mountsList = KMountPoint::currentMountPoints(); > +const auto mount = mountsList.findByPath(filePath); Calling currentMountPoints() for every created file is definitely a bad idea ...

D28745: Skip caching thumbnails on encrypted filesystems

bruns requested changes to this revision. bruns added inline comments. This revision now requires changes to proceed. INLINE COMMENTS > thumbnail.cpp:729 > +const auto thumbRootMount = > mountsList.findByPath(m_thumbBasePath); > +//If file is on the same filesystem as

D28880: [KWallet] Port last usage of QRegExp to QRegularExpression

ahmadsamir created this revision. ahmadsamir added reviewers: Frameworks, dfaure, blaze. Herald added a project: Frameworks. ahmadsamir requested review of this revision. REVISION SUMMARY QRegularExpression::wildcardToRegularExpression() mainly handles file pattern globbing (e.g. "*.txt")

D28745: Skip caching thumbnails on encrypted filesystems

meven accepted this revision. meven added a comment. In D28745#649011 , @marcingu wrote: > In D28745#648036 , @meven wrote: > > > This is gonna have an hefty toll on perf as it will add a `getmntent`

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

BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kio/job/kf5-qt5%20FreeBSDQt5.14/61/ Project: kf5-qt5 FreeBSDQt5.14 Date of build: Thu, 16 Apr 2020 10:25:10 + Build duration: 44 min and counting JUnit Tests Name: projectroot

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

BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kio/job/kf5-qt5%20SUSEQt5.14/53/ Project: kf5-qt5 SUSEQt5.14 Date of build: Thu, 16 Apr 2020 10:25:09 + Build duration: 18 min and counting BUILD ARTIFACTS

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

BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kio/job/kf5-qt5%20SUSEQt5.12/552/ Project: kf5-qt5 SUSEQt5.12 Date of build: Thu, 16 Apr 2020 10:25:09 + Build duration: 9 min 23 sec and counting BUILD ARTIFACTS

D28647: Fix KIO::Scheduler::emitReparseSlaveConfiguration() to work if called twice in same process

marten added a comment. I'm happy to work on the refactoring if you think it's the right thing to do. Do you mean splitting SchedulerPrivate::slotReparseSlaveConfiguration() up into two halves, the first part (KProtocolManager::reparseConfiguration through to NetRC::self()->reload -

D28862: [Extractor] Remove IdleStateMonitor wrapper class

bruns marked an inline comment as done. REPOSITORY R293 Baloo REVISION DETAIL https://phabricator.kde.org/D28862 To: bruns, #baloo, ngraham Cc: broulik, kde-frameworks-devel, hurikhan77, lots0logs, LeGast00n, cblack, fbampaloukas, domson, ashaposhnikov, michaelh, astippich, spoorun,

D28862: [Extractor] Remove IdleStateMonitor wrapper class

bruns added a comment. In D28862#649411 , @broulik wrote: > It used to be 2 minutes, now it's 1? It used to be infinity ... 1 minute is completely sufficient. INLINE COMMENTS > broulik wrote in app.cpp:59 > Why the member variable? It's

  1   2   >