kirigami unittests broken

/projectroot.home.jenkins.workspace.Frameworks.kirigami.kf5-qt5_SUSEQt512/autotests/tst_actiontoolbar_qml/ [1] https://build.kde.org/job/Frameworks/view/Platform%20-%20SUSEQt5.12/job/kirigami/job/kf5-qt5%20SUSEQt5.12/557/ -- David Faure, fa...@kde.org, http://www.davidfaure.fr Working on KDE Frameworks 5

Re: KDE CI: Frameworks » kcoreaddons » kf5-qt5 FreeBSDQt5.15 - Build # 33 - Fixed!

On jeudi 6 août 2020 10:48:19 CEST CI System wrote: > BUILD SUCCESS > Build > URL https://build.kde.org/job/Frameworks/job/kcoreaddons/job/kf5-qt5%20Free \o/ -- David Faure, fa...@kde.org, http://www.davidfaure.fr Working on KDE Frameworks 5

D26448: Add KRecentFilesMenu to replace KRecentFileAction

dfaure added inline comments. INLINE COMMENTS > nicolasfella wrote in krecentfilesmenu.cpp:150 > I'm wondering how to best make this async without selling my soul to the > mulitthreading devil. QtConcurrent::filtered looks interesting, but depending > on QtConcurrent wouldn't be feasible,

kwayland's testRemoteAccess still flaky

look? -- David Faure, fa...@kde.org, http://www.davidfaure.fr Working on KDE Frameworks 5

Re: xml_mimetypes5 and kcoreaddons

On mercredi 15 juillet 2020 12:24:34 CEST Harald Sitter wrote: > On Wed, Jul 15, 2020 at 12:39 AM David Faure wrote: > > On mardi 14 juillet 2020 19:35:41 CEST Albert Astals Cid wrote: > > > El dimarts, 14 de juliol de 2020, a les 15:14:38 CEST, Jonathan Riddell > >

Re: Frameworks support for Qt 5.12

minimum required version 12 months after Qt 5.15, i.e. on 26 May 2021 * Qt 5.15 LTS will be the minimum required version 18 months after its release, i.e. on 26 Nov 2021 https://community.kde.org/Frameworks/Policies#Frameworks_Qt_requirements -- David Faure, fa...@kde.org, http://www.davidfaure.fr Working on KDE Frameworks 5

Re: xml_mimetypes5 and kcoreaddons

ypes/XmlMessages.sh to rename > it. Fine with me, but don't we have more cases of the same kind, with different names? Any case of translations being "integrated" into some file leads to this. Desktop files, mimetype XML, is there really nothing else? -- David Faure, fa...@kde.org, http://www.davidfaure.fr Working on KDE Frameworks 5

Re: KDE CI: Frameworks » ktexteditor » kf5-qt5 SUSEQt5.15 - Build # 4 - Still Unstable!

On mardi 7 juillet 2020 17:58:26 CEST Christoph Cullmann wrote: > On 2020-07-07 12:16, David Faure wrote: > > Yep :( > > > > You'll tell Simon and/or QTBUG-*? > > I will take a look if I can find some existing bug or open a new. > > Simon left Qt, or? Yes but

Re: KDE CI: Frameworks » ktexteditor » kf5-qt5 SUSEQt5.15 - Build # 4 - Still Unstable!

t; > > > src.variableeditor > > 0% (0/5) > > 0% (0/5) > > 0% (0/579) > > 0% (0/108) > > > > src.view > > 88% (15/17) > >

D14631: Adds a new RenameDialog to KIO with more options for batch renaming

dfaure added a comment. Only if you can find a way to change BatchRenameJob in a binary and behaviour compatible way. And then it will be a dual-headed thing with two modes of operations, awful. All this sounds to me like much more trouble than writing a different job. REPOSITORY R241

Re: ThumbSequenceCreator in KIO not working properly

hin people on kfm-devel about how Dolphin could be changed to use DelegateAnimationHandler, possibly? -- David Faure, fa...@kde.org, http://www.davidfaure.fr Working on KDE Frameworks 5

inotify on FreeBSD

on FreeBSD? [1] https://build.kde.org/job/Frameworks/view/Platform%20-%20FreeBSDQt5.15/job/kcoreaddons/job/kf5-qt5%20FreeBSDQt5.15/9/testReport/projectroot/autotests/kdirwatch_inotify_unittest/ -- David Faure, fa...@kde.org, http://www.davidfaure.fr Working on KDE Frameworks 5

Re: Shift for parts of the CI system to Qt 5.15

.15/ -- David Faure, fa...@kde.org, http://www.davidfaure.fr Working on KDE Frameworks 5

Re: Add loop device interface to Solid framework

scope question, maybe ervin will be willing to provide some input :-) -- David Faure, fa...@kde.org, http://www.davidfaure.fr Working on KDE Frameworks 5

D26342: Allow overriding to disable auto language detection

dfaure added a comment. In D26342#675180 , @aacid wrote: > In D26342#675164 , @sdepiets wrote: > > > I don't think that's a regression, in the previous behavior you could try to set any language to

D26342: Allow overriding to disable auto language detection

dfaure added a comment. This actually breaks language auto-detection for me in the KMail composer. Testcase: - New Mail - I type "Bonjour," in the body Before: It's detected as French and not underlined as a typo After: The language remains English, and the word is

Re: Opening files by mimetype instead of by file extension

han XXX > because you can't do this" What does "list by mimetype" mean exactly? Is this about a mimetype column in the table/tree view? (missing) Is this about the filter combobox showing mimetypes? (supported) Or is it something else? -- David Faure, fa...@kde.org, http

D29810: Don't use the setenv function after fork

dfaure added a comment. This breaks FreeBSD compilation. Please check: https://build.kde.org/job/Frameworks/job/kcrash/job/kf5-qt5%20FreeBSDQt5.14/17/ REPOSITORY R285 KCrash REVISION DETAIL https://phabricator.kde.org/D29810 To: jpalecek, #frameworks, dfaure Cc: bruns, apol,

D29810: Don't use the setenv function after fork

dfaure closed this revision. REPOSITORY R285 KCrash REVISION DETAIL https://phabricator.kde.org/D29810 To: jpalecek, #frameworks, dfaure Cc: bruns, apol, anthonyfieroni, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham

D29814: Fix segfault on no restart args

dfaure closed this revision. REPOSITORY R285 KCrash REVISION DETAIL https://phabricator.kde.org/D29814 To: jpalecek, #frameworks, dfaure Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29814: Fix segfault on no restart args

dfaure accepted this revision. This revision is now accepted and ready to land. REPOSITORY R285 KCrash BRANCH for-upstream REVISION DETAIL https://phabricator.kde.org/D29814 To: jpalecek, #frameworks, dfaure Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

KRunProxy

/search?_filestring=*.qml&_string=KRun Am I doing this wrong? Are there other file extensions to take into consideration? If not, can I deprecate KRunProxy? -- David Faure, fa...@kde.org, http://www.davidfaure.fr Working on KDE Frameworks 5

D29809: Don't invoke qstring localized stuff in critical section

dfaure added a comment. Done in https://invent.kde.org/frameworks/kcrash/commit/c315685f0e0920e8847afe12c97fbbe4653eb351 REPOSITORY R285 KCrash REVISION DETAIL https://phabricator.kde.org/D29809 To: jpalecek, #frameworks, dfaure Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh,

D29809: Don't invoke qstring localized stuff in critical section

This revision was automatically updated to reflect the committed changes. Closed by commit R285:c315685f0e09: Dont invoke qstring localized stuff in critical section (authored by dfaure). REPOSITORY R285 KCrash CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29809?vs=83231=83233

D29814: Fix segfault on no restart args

dfaure added a comment. This question is still without an answer: "Can you explain how to trigger this crash in the first place? Which application triggers it?" REPOSITORY R285 KCrash REVISION DETAIL https://phabricator.kde.org/D29814 To: jpalecek, #frameworks, dfaure Cc:

D29809: Don't invoke qstring localized stuff in critical section

dfaure accepted this revision. This revision is now accepted and ready to land. REPOSITORY R285 KCrash BRANCH for-upstream REVISION DETAIL https://phabricator.kde.org/D29809 To: jpalecek, #frameworks, dfaure Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29814: Fix segfault on no restart args

dfaure added a comment. This commit seems to include the changes requested in the other review... REPOSITORY R285 KCrash REVISION DETAIL https://phabricator.kde.org/D29814 To: jpalecek, #frameworks, dfaure Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29809: Don't invoke qstring localized stuff in critical section

dfaure added a comment. The changes you mention don't appear in phabricator. I suggest amending your commit locally (so there's only one) and then arc diff HEAD~ REPOSITORY R285 KCrash REVISION DETAIL https://phabricator.kde.org/D29809 To: jpalecek, #frameworks, dfaure Cc:

D18915: Fix batchrename changing extension to lower case

dfaure added a comment. The fix will be in 5.15.1. Can you add Qt version checks if you intend to keep the KIO workaround? Or we just say it's fixed in Qt and we discard this. Your choice. But the unittest would make sense to keep (with Qt version checks, if there's no workaround).

D18915: Fix batchrename changing extension to lower case

dfaure added a comment. The fix landed in dev. I now uploaded a backport to 5.15 for review. https://codereview.qt-project.org/c/qt/qtbase/+/302532 REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D18915 To: cfoster, #dolphin, #frameworks, abalaji, dfaure Cc:

D26113: Places: Use Solid::Device::DisplayName for DisplayRole

dfaure accepted this revision. dfaure added a comment. This revision is now accepted and ready to land. Thanks! REPOSITORY R241 KIO BRANCH arcpatch-D26113 REVISION DETAIL https://phabricator.kde.org/D26113 To: meven, #frameworks, ngraham, dfaure Cc: dfaure, feverfew, bruns, broulik,

D29445: [KOpenWithDialog] When pointing at a non-executable file print more meaningful error

dfaure requested changes to this revision. dfaure added a comment. This revision now requires changes to proceed. I have modified DesktopExecParser in kio commit d1e9325fba37eddb9cb44173edfc1fee122a0c57 to return an

D26113: Places: Use Solid::Device::DisplayName for DisplayRole

dfaure added a comment. Please check that kfileplacesmodeltest and kfileplacesviewtest still pass. (they fail here with baloosearch: stuff for some reason, I didn't investigate; but it passes on CI) REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D26113 To: meven,

D29814: Fix segfault on no restart args

dfaure added a comment. Can you explain how to trigger this crash in the first place? Which application triggers it? INLINE COMMENTS > jpalecek wrote in kcrash.cpp:272 > Well I was thinking about `resize()`, then found there wasn't any. However, > if we want to be clear, maybe `args = {

D29814: Fix segfault on no restart args

dfaure requested changes to this revision. dfaure added a comment. This revision now requires changes to proceed. I'm a bit confused by the bug this is fixing. Surely this doesn't happen to all cases of crashes without autorestart enabled?? Also, it sounds like a null check might be

D29810: Don't use the setenv function after fork

dfaure accepted this revision. This revision is now accepted and ready to land. REPOSITORY R285 KCrash BRANCH for-upstream REVISION DETAIL https://phabricator.kde.org/D29810 To: jpalecek, #frameworks, dfaure Cc: bruns, apol, anthonyfieroni, kde-frameworks-devel, LeGast00n, cblack,

D29809: Don't invoke qstring localized stuff in critical section

dfaure added a comment. Makes sense; just two minor things. INLINE COMMENTS > kcrash.cpp:95 > +#ifdef Q_OS_LINUX > +QByteArray socketpath; > +#endif prepend `static`, it's only used in this file. > kcrash.cpp:662 > > -// Create socket path to transfer ptrace scope and open

D29668: Do not reject icon theme dir with invalid context/type.

dfaure requested changes to this revision. dfaure added inline comments. This revision now requires changes to proceed. INLINE COMMENTS > kicontheme.cpp:761 > } else if (tmp.isEmpty()) { > // do nothing. key not required > } else { (OK, if this key is not required then surely

Re: Recent breakage in kwallet

Thanks for the pointer. Pretty sure it's wrong, but I'll reply there. On mardi 26 mai 2020 09:31:26 CEST Marco Martin wrote: > would the recent patch > https://invent.kde.org/frameworks/kwallet/-/merge_requests/1 > fix anything? > > On Sat, May 16, 2020 at 10:40 AM Dav

Re: KDE CI: Frameworks » ktexteditor » kf5-qt5 SUSEQt5.14 - Build # 19 - Still Unstable!

gt; > 88% (15/17) > > 88% (15/17) > > 57% (3782/6599) > > 43% (1661/3880) > > > > src.vimode > > 100% (30/30) > > 100% (30/30) > > 81% (1898/2332) > > 61% (979/1599) > > > > src.vimode.config > > 0% (0/1) > > 0% (0/1) > > 0% (0/120) > > 0% (0/76) > > > > src.vimode.emulatedcommandbar > > 100% (12/12) > > 100% (12/12) > > 98% (896/916) > > 90% (588/652) > > > > src.vimode.modes > > 100% (10/10) > > 100% (10/10) > > 87% (3266/3761) > > 79% (1884/2374) > > > > Links: > > -- > > [1] > > https://build.kde.org/job/Frameworks/job/ktexteditor/job/kf5-qt5%20SUSEQt5 > > .14/19/artifact/abi-compatibility-results.yaml [2] > > https://build.kde.org/job/Frameworks/job/ktexteditor/job/kf5-qt5%20SUSEQt5 > > .14/19/artifact/acc/KF5TextEditor-5.71.0.xml [3] > > https://build.kde.org/job/Frameworks/job/ktexteditor/job/kf5-qt5%20SUSEQt5 > > .14/19/artifact/compat_reports/KF5TextEditor_compat_report.html [4] > > https://build.kde.org/job/Frameworks/job/ktexteditor/job/kf5-qt5%20SUSEQt5 > > .14/19/artifact/logs/KF5TextEditor/5.71.0/log.txt -- David Faure, fa...@kde.org, http://www.davidfaure.fr Working on KDE Frameworks 5

D28590: Add a QString Solid::Device::displayName, used in Fstab Device for network mounts

dfaure added a comment. @meven you're confusing me with my clone @ervin. REPOSITORY R245 Solid REVISION DETAIL https://phabricator.kde.org/D28590 To: meven, #frameworks, bruns, sitter, dfaure Cc: dfaure, broulik, ervin, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D28590: Add a QString Solid::Device::displayName, used in Fstab Device for network mounts

dfaure requested changes to this revision. dfaure added inline comments. This revision now requires changes to proceed. INLINE COMMENTS > fstabdevice.cpp:73 > +if (m_displayName.isEmpty()) { > +QStringList currentMountPoints = > FstabHandling::currentMountPoints(m_device); > +

D28765: KSettings::Dialog: add support for KPluginInfos without a KService

dfaure added a comment. https://invent.kde.org/frameworks/kcmutils/-/merge_requests/2 REPOSITORY R295 KCMUtils REVISION DETAIL https://phabricator.kde.org/D28765 To: dfaure, pino, broulik, mart, davidedmundson, ngraham, svuorela Cc: ahmadsamir, rikmills, wbauer, kossebau, svuorela,

D29738: Fix service file specifying 'Run in terminal' giving an error code 100

dfaure added a comment. (Intrusive) fix is up at https://invent.kde.org/frameworks/kio/-/merge_requests/3 REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D29738 To: marten, #frameworks, dfaure Cc: meven, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29738: Fix service file specifying 'Run in terminal' giving an error code 100

dfaure added a comment. Well spotted. Indeed, when using kioexec (because of the DeleteTemporaryFiles option) we no longer detect non-existing executables. I'll look into fixing this. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D29738 To: marten, #frameworks,

D29558: Add KIO::OpenUrlJob::setShowOpenWithDialog as replacement for KRun::displayOpenWithDialog

dfaure added a comment. Yeah, the full story is that I pushed to git.kde.org, reverted because I changed my mind, then the sysadmins told me I wasn't supposed to push to git.kde.org at all (it was blocked for everyone, but I have sysadmin privileges for historical and bus-factor reasons).

D29558: Add KIO::OpenUrlJob::setShowOpenWithDialog as replacement for KRun::displayOpenWithDialog

dfaure reopened this revision. dfaure added a comment. This revision is now accepted and ready to land. Not committed after all. REPOSITORY R241 KIO BRANCH 2020_05_displayOpenWithDialog REVISION DETAIL https://phabricator.kde.org/D29558 To: dfaure, ahmadsamir, broulik, svuorela Cc:

D29800: Fix URL being passed as argument when launching a .desktop file

dfaure accepted this revision. dfaure added a comment. This revision is now accepted and ready to land. Thanks for the unittest and fix! Oversight on my side indeed. INLINE COMMENTS > openurljobtest.h:58 > void takeOverAfterMimeTypeFound(); > +void runDeskopFileDirectly(); >

D29782: [StatJob] Make mostLocalUrl ignore remote (ftp, http...etc) URLs

dfaure added a comment. OK, I picked testtrash because kio_trash *is* a :local protocol. If it doesn't use UDS_LOCAL_PATH, at least it means the job will actually go through (no early filtering out). That's at least interesting to test too. Good that http will give no error. That's

D29558: Add KIO::OpenUrlJob::setShowOpenWithDialog as replacement for KRun::displayOpenWithDialog

dfaure added a comment. OK I'm having second thoughts about this. Because of Windows, and because of the case of multiple URLs. There's "displaying an open with dialog because we couldn't find any app, after a left-click on a file" and there's "displaying an open with dialog because

D29558: Add KIO::OpenUrlJob::setShowOpenWithDialog as replacement for KRun::displayOpenWithDialog

dfaure closed this revision. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D29558 To: dfaure, ahmadsamir, broulik, svuorela Cc: kossebau, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29558: Add KIO::OpenUrlJob::setShowOpenWithDialog as replacement for KRun::displayOpenWithDialog

dfaure updated this revision to Diff 82992. dfaure marked 2 inline comments as done. dfaure added a comment. API docs: remove trailing dot, mention 2 default values REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29558?vs=82358=82992 BRANCH

D29782: [StatJob] Make mostLocalUrl ignore remote (ftp, http...etc) URLs

dfaure added a comment. Can you add a unittest for a KIO::mostLocalUrl() in testtrash.cpp (which is :local, so it should work) and another one in jobtest.cpp for a http URL (e.g. to test which error code we're getting, if any?) + call mostLocalUrl() on a "normal" StatJob in

Re: Request for ktexteditor patch release

gnored after session reopening (Kate, KDevelop) (bug 421375) -- David Faure, fa...@kde.org, http://www.davidfaure.fr Working on KDE Frameworks 5

Re: Recent breakage in kwallet

age Any neon-specific patches to kwallet? -- David Faure, fa...@kde.org, http://www.davidfaure.fr Working on KDE Frameworks 5

D29610: [kio_file] Handle renaming file 'A' to 'a' on FAT32 filesystems

dfaure accepted this revision. dfaure added a comment. This revision is now accepted and ready to land. OK, at least we'll have moved the code to the right place :-) INLINE COMMENTS > ahmadsamir wrote in file_unix.cpp:1052 > const QByteArray dest1 = "/mnt/fat32/A"; > const QByteArray

D29787: Fix krununittest

dfaure accepted this revision. dfaure added a comment. This revision is now accepted and ready to land. Let's hope the CI has xterm installed I doubt it. Maybe we'll need to pick "ls" instead, even if that will look weird. REPOSITORY R241 KIO BRANCH l-terminal (branched from master)

D29385: Introduce KIO::OpenUrlJob, a rewrite and replacement for KRun

dfaure added a comment. That's the point, a NFS mount *is* a local URL, so we do use QFile for it. And then it takes forever because the kernel has to talk over a socket to answer us. Yes this is horrendous. I hate network mounts :-) REPOSITORY R241 KIO REVISION DETAIL

D29385: Introduce KIO::OpenUrlJob, a rewrite and replacement for KRun

dfaure added a comment. It's 3 times faster on my local SSD. Now think of a NFS mount on a server from another country REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D29385 To: dfaure, ahmadsamir, broulik, meven, kossebau, davidedmundson, nicolasfella,

Re: Request for ktexteditor patch release

5a36e21473c00a8d30cbea0f150a13f c7b568e75c147161992f8875fe36fb46885bccddb63c22edaf81071583f4204c sources/ktexteditor-5.70.1.tar.xz Please add a description of the bug in www/info/kde-frameworks-5.70.0.php (or give me a patch if you can't push). Also, please make sure to write a unittest for this regression so we catch it next time. -- D

D29767: CopyJob: Check if destination dir is a symlink

dfaure accepted this revision. dfaure added a comment. This revision is now accepted and ready to land. Thanks for the quick fix and for the test! Just minor issues in comments, feel free to push after fixing. INLINE COMMENTS > jobtest.cpp:626 > +// just the same as

D29738: Fix service file specifying 'Run in terminal' giving an error code 100

dfaure accepted this revision. dfaure added a comment. This revision is now accepted and ready to land. Nice! In addition to the bugfix, calling resultingArguments() only once is a definite improvement, given everything that happens in there. I didn't realize we were calling it twice.

Re: Request for plasma framework patch release

On jeudi 14 mai 2020 15:24:57 CEST David Faure wrote: > On jeudi 14 mai 2020 13:56:50 CEST David Edmundson wrote: > > We seem to have a common crasher newly introduced in plasma-framework. A > > dozen reports in a few days. > > > > Can I ask fo

Re: Request for plasma framework patch release

; Sorry. Here you go. plasma-framework v5.70.1 bfea43379bc364bc1bb0719d9a1a73a57284214d e28884da2e3389513d96794f71fdfa22a8b4ba62396b7d061b4f18d9b5549d19 sources/plasma-framework-5.70.1.tar.xz -- David Faure, fa...@kde.org, http://www.davidfaure.fr Working on KDE Frameworks 5

D29610: [kio_file] Handle renaming file 'A' to 'a' on FAT32 filesystems

dfaure added inline comments. INLINE COMMENTS > file_unix.cpp:1074 > > if (::rename(_src.data(), _dest.data())) { > if (auto err = execWithElevatedPrivilege(RENAME, {_src, _dest}, > errno)) { Wouldn't it be enough to just call QFile::rename here? The whole idea is: if

D29610: [kio_file] Handle renaming file 'A' to 'a' on FAT32 filesystems

dfaure requested changes to this revision. This revision now requires changes to proceed. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D29610 To: ahmadsamir, #frameworks, dfaure Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29610: [kio_file] Handle renaming file 'A' to 'a' on FAT32 filesystems

dfaure added inline comments. INLINE COMMENTS > ahmadsamir wrote in copyjob.cpp:1965 > Good point, I missed that the original code was checking the file names and > the paths with the same compare() call. > > About /dir/file and /DIR/FILE, the parent dirs is actually one dir since > "dir" and

D29708: Introduce EXCLUDE_DEPRECATED_BEFORE_AND_AT

dfaure added a comment. > there are lots of KRun::run* usages in non-deprecated API sadly in KIO :( OK, I'll fix that. REPOSITORY R241 KIO BRANCH excludedeprecated REVISION DETAIL https://phabricator.kde.org/D29708 To: kossebau, #frameworks, dfaure, meven, ahmadsamir Cc:

Re: kcoreaddons_add_plugin executable/plugin directory conflict

path, it has less priority than QT_PLUGIN_PATH, so it was picking up my installed plugins instead of the locally built ones. So now modules/ECMAddTests.cmake sets the QT_PLUGIN_PATH env var for when running tests (https://phabricator.kde.org/D8660). Therefore we could indeed move all plugins to a subdir. T

D27203: Don't try to open files we can't figure out where they are

dfaure requested changes to this revision. dfaure added inline comments. This revision now requires changes to proceed. INLINE COMMENTS > kdeplatformfiledialoghelper.cpp:226 > +//passing non-local files as the working directory is not supported. > +//See

D26066: [KProcessInfo] Add parent PID

dfaure added a comment. Yep, seems fine to me. REPOSITORY R244 KCoreAddons REVISION DETAIL https://phabricator.kde.org/D26066 To: broulik, #frameworks, davidedmundson Cc: meven, dfaure, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29668: Do not reject icon theme dir with invalid context/type.

dfaure added a comment. What are the values of Context and Type? "Legacy" and "UI" ? I can't see anything in index.theme https://ftp.gnome.org/pub/GNOME/sources/adwaita-icon-theme/3.36/ REPOSITORY R302 KIconThemes REVISION DETAIL https://phabricator.kde.org/D29668 To:

Re: Fwd: KDE CI: Administration » Dependency Build Calligra stable-kf5-qt5 WindowsMSVCQt5.14 - Build # 24 - Still Failing!

On Sunday, May 10, 2020 11:53:31 AM CEST Ben Cooksley wrote: > Can someone investigate the below please? Fixed by Ahmad today in https://phabricator.kde.org/D29599 -- David Faure, fa...@kde.org, http://www.davidfaure.fr Working on KDE Frameworks 5

D29610: [CopyJob] Use stricter conditions when using QFile::rename in slotResultRenaming

dfaure accepted this revision. dfaure added a comment. This revision is now accepted and ready to land. OK for now, to fix the unittests. The *real* fix however is to use QFile::rename in kio_file so that this failure to rename doesn't even happen in the first place. In this commit can

D28760: KSettings::Dialog: avoid duplicate entries due cascading $XDG_DATA_DIRS

dfaure added inline comments. INLINE COMMENTS > svuorela wrote in dialog.cpp:317 > I think I read once that whenever you used a ordered map over an unordered > map, you need to justify it by talking to your manager about it. But that's > also a bit from the bucket of nitpickery unless we are

D28760: KSettings::Dialog: avoid duplicate entries due cascading $XDG_DATA_DIRS

dfaure closed this revision. REPOSITORY R295 KCMUtils REVISION DETAIL https://phabricator.kde.org/D28760 To: dfaure, apol, broulik, davidedmundson, kossebau, svuorela Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29547: KRun: deprecate all static 'run*' methods, with full porting instructions

dfaure closed this revision. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D29547 To: dfaure, ahmadsamir, broulik, svuorela Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29547: KRun: deprecate all static 'run*' methods, with full porting instructions

dfaure updated this revision to Diff 82453. dfaure marked 2 inline comments as done. dfaure added a comment. take review into account REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29547?vs=82325=82453 BRANCH 2020_05_deprecate_KRun_run_methods REVISION

D29547: KRun: deprecate all static 'run*' methods, with full porting instructions

dfaure marked 2 inline comments as done. dfaure added inline comments. INLINE COMMENTS > ahmadsamir wrote in krun.h:216 > I don't think you want both @deprecated? I did, but Friedrich had a less confusing suggestion: @deprecated since 5.6. Since 5.71 use ApplicationLauncherJob, otherwise

D23875: KCoreDirLister: fix crash when creating new folders from kfilewidget

dfaure added a subscriber: chinmoyr. dfaure added a comment. In D23875#532045 , @dhaumann wrote: > @dfaure: D21197 can be closed / abandoned? Yes, but @chinmoyr has disappeared, it seems? REPOSITORY

D28760: KSettings::Dialog: avoid duplicate entries due cascading $XDG_DATA_DIRS

dfaure added a comment. ping? REPOSITORY R295 KCMUtils REVISION DETAIL https://phabricator.kde.org/D28760 To: dfaure, apol, broulik, davidedmundson, kossebau, svuorela Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29547: KRun: deprecate all static 'run*' methods, with full porting instructions

dfaure added a reviewer: svuorela. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D29547 To: dfaure, ahmadsamir, broulik, svuorela Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29558: Add KIO::OpenUrlJob::setShowOpenWithDialog as replacement for KRun::displayOpenWithDialog

dfaure added a reviewer: svuorela. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D29558 To: dfaure, ahmadsamir, broulik, svuorela Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29599: [CopyJob] Try to fix windows build

dfaure accepted this revision. dfaure added a comment. This revision is now accepted and ready to land. Bonus points for keeping an eye on CI, I wish everyone did that ;-) REPOSITORY R241 KIO BRANCH l-fix-win-build (branched from master) REVISION DETAIL

D29577: Stop using deprecated methods

dfaure accepted this revision. This revision is now accepted and ready to land. REPOSITORY R320 KIO Extras BRANCH master REVISION DETAIL https://phabricator.kde.org/D29577 To: aacid, dfaure Cc: kossebau, kde-frameworks-devel, kfm-devel, dfaure, waitquietly, azyx, nikolaik, pberestov,

D29361: KCrash: remove debug output which breaks unittests from using ~/.qttest/config for categorized logging

This revision was automatically updated to reflect the committed changes. Closed by commit R285:58afbdf29a83: KCrash: remove debug output which breaks unittests from using ~/.qttest/config… (authored by dfaure). REPOSITORY R285 KCrash CHANGES SINCE LAST UPDATE

D29574: Use KSERVICE_DEPRECATED_VERSION_BELATED

dfaure accepted this revision. This revision is now accepted and ready to land. REPOSITORY R309 KService BRANCH usebelated REVISION DETAIL https://phabricator.kde.org/D29574 To: kossebau, #frameworks, dfaure Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29573: ECMGenerateExportHeader: add generation of *_DEPRECATED_VERSION_BELATED()

dfaure accepted this revision. dfaure added a comment. This revision is now accepted and ready to land. So many things to take care of :-) REPOSITORY R240 Extra CMake Modules BRANCH addbelated REVISION DETAIL https://phabricator.kde.org/D29573 To: kossebau, #frameworks,

D29572: Build with EXCLUDE_DEPRECATED_BEFORE_AND_AT=CURRENT

dfaure accepted this revision. dfaure added a comment. This revision is now accepted and ready to land. Thanks! REPOSITORY R309 KService BRANCH buildwithoutdeprecated REVISION DETAIL https://phabricator.kde.org/D29572 To: kossebau, #frameworks, dfaure Cc: kde-frameworks-devel,

D29561: Add missing compiler deprecation tag for 5-args KServiceAction constructor

dfaure accepted this revision. This revision is now accepted and ready to land. REPOSITORY R309 KService BRANCH addmissingdeprecationtag REVISION DETAIL https://phabricator.kde.org/D29561 To: kossebau, #frameworks, dfaure Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham,

D29485: [CopyJob] Check free space for remote urls before copying and other improvements

dfaure added inline comments. INLINE COMMENTS > ahmadsamir wrote in copyjob.cpp:444 > Excellent points; I was scratching my head trying to figure out what use > UDS_LOCAL_PATH is when, while testing with qDebug(), it was _always_ empty, > the only time it wasn't empty was in testtrash unit

D29558: Add KIO::OpenUrlJob::setShowOpenWithDialog as replacement for KRun::displayOpenWithDialog

dfaure created this revision. dfaure added reviewers: ahmadsamir, broulik. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. dfaure requested review of this revision. REVISION SUMMARY This will not look up the preferred app or run scripts/executables, it

D29547: KRun: deprecate all static 'run*' methods, with full porting instructions

dfaure added a dependent revision: D29557: Add KIO::OpenUrlJob::setShowOpenWithDialog as replacement for KRun::displayOpenWithDialog. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D29547 To: dfaure, ahmadsamir, broulik Cc: kde-frameworks-devel, LeGast00n, cblack,

D29485: [CopyJob] Check free space for remote urls before copying and other improvements

dfaure accepted this revision. dfaure added inline comments. This revision is now accepted and ready to land. INLINE COMMENTS > ahmadsamir wrote in copyjob.cpp:477 > I meant connection to the remote ulr/host; however e.g Dolphin already > reports those connection errors in the status bar when

D29547: KRun: deprecate all static 'run*' methods, with full porting instructions

dfaure created this revision. dfaure added reviewers: ahmadsamir, broulik. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. dfaure requested review of this revision. REVISION SUMMARY [Next up, KRun::displayOpenWithDialog] REPOSITORY R241 KIO BRANCH

D29541: KBookmarkMenuTest: extend unittest to cover undeprecated API

dfaure accepted this revision. This revision is now accepted and ready to land. REPOSITORY R294 KBookmarks BRANCH extendunittestfornudeprecated REVISION DETAIL https://phabricator.kde.org/D29541 To: kossebau, #frameworks, nicolasfella, dfaure, ahmadsamir Cc: kde-frameworks-devel,

D29485: [CopyJob] Check free space for remote urls before copying and other improvements

dfaure requested changes to this revision. dfaure added a comment. This revision now requires changes to proceed. Sorry, I have one last comment about a comment :) INLINE COMMENTS > copyjob.cpp:477 > +// Check available free space for remote urls > +// TODO: find a

D29537: [CopyJob] Get rid of an old TODO and use QFile::rename()

dfaure accepted this revision. dfaure added a comment. This revision is now accepted and ready to land. Nice! REPOSITORY R241 KIO BRANCH l-qfile-rename (branched from master) REVISION DETAIL https://phabricator.kde.org/D29537 To: ahmadsamir, #frameworks, dfaure Cc:

D29528: [OpenUrlJob] Improve comments/docs

dfaure accepted this revision. This revision is now accepted and ready to land. REPOSITORY R241 KIO BRANCH l-late (branched from master) REVISION DETAIL https://phabricator.kde.org/D29528 To: ahmadsamir, #frameworks, dfaure Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham,

D29485: [CopyJob] Check free space for remote urls before copying and other improvements

dfaure added inline comments. INLINE COMMENTS > copyjob.cpp:430 > +if (!m_privilegeExecutionEnabled && !isWritable) { > +const QUrl dest = m_asMethod ? > m_dest.adjusted(QUrl::RemoveFilename) : m_dest; > +q->setError(ERR_WRITE_ACCESS_DENIED); This is

<    1   2   3   4   5   6   7   8   9   10   >