KDE CI: Frameworks purpose kf5-qt5 FreeBSDQt5.9 - Build # 14 - Fixed!

2018-01-29 Thread CI System
BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks%20purpose%20kf5-qt5%20FreeBSDQt5.9/14/ Project: Frameworks purpose kf5-qt5 FreeBSDQt5.9 Date of build: Tue, 30 Jan 2018 04:00:11 + Build duration: 3 min 35 sec and counting JUnit Tests

D10113: baloo-widgets: Emit metaDataRequestFinished once per request

2018-01-29 Thread Elvis Angelaccio
elvisangelaccio added inline comments. INLINE COMMENTS > michaelh wrote in filemetadataprovider.cpp:323 > In that case it might be safer to do > > IndexedDataRetriever *ret = new IndexedDataRetriever(filePath, this); > ​connect(ret, SIGNAL(finished(KJob*)), this, > SLOT(slotLoadi

D10172: KRun: allow executing "add network folder" without confirmation prompt

2018-01-29 Thread Elvis Angelaccio
elvisangelaccio added a comment. +1 REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D10172 To: dfaure, mwolff, dhaumann Cc: elvisangelaccio, ngraham, #dolphin, #frameworks, michaelh

Re: KDirListerCache destroy comment in kcoredirlister.cpp (about pre-2013 code)

2018-01-29 Thread René J . V . Bertin
On Monday January 29 2018 17:09:20 Kevin Funk wrote: Somebody had too little precious time, apparently. Apologies for not having added an indication in the subject that this concerns almost fossile code and is thus aimed at readers with a certain maturity O:-)

D10113: baloo-widgets: Emit metaDataRequestFinished once per request

2018-01-29 Thread Michael Heidelbach
michaelh updated this revision to Diff 26183. michaelh added a comment. Merge branch 'master' of git://anongit.kde.org/baloo-widgets Change execution order in setFileItems REPOSITORY R824 Baloo Widgets CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D10113?vs=26149&id=26183 BRANC

Re: KDirListerCache destroy comment in kcoredirlister.cpp

2018-01-29 Thread Kevin Funk
On Monday, 29 January 2018 11:32:56 CET René J.V. Bertin wrote: > Hi, > > Researching an issue I'm seeing (hang/crash on exit) I found the following > vestigial out-commented code the KCoreDirListerCache ctor in > kcoredirlister.cpp: FYI: Note to the mailing list, before more precious developer t

D8532: [WIP] Restrict file extractor with Seccomp

2018-01-29 Thread David Kahles
davidk edited the summary of this revision. REPOSITORY R293 Baloo REVISION DETAIL https://phabricator.kde.org/D8532 To: davidk, apol, ossi Cc: ngraham, nicolasfella, #frameworks, michaelh

D10105: baloo-widgets: Refactor filemetadataprovider for better readability

2018-01-29 Thread Michael Heidelbach
This revision was automatically updated to reflect the committed changes. Closed by commit R824:a81ceaeaa963: baloo-widgets: Refactor filemetadataprovider for better readability (authored by michaelh). REPOSITORY R824 Baloo Widgets CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D10105

D10105: baloo-widgets: Refactor filemetadataprovider for better readability

2018-01-29 Thread Nathaniel Graham
ngraham added a comment. I would have deleted the existing `arcpatch-D10105` branch so ad to avoid `arc` creating a `arcpatch-D10105_1` branch, but yeah, that looks sane. REPOSITORY R824 Baloo Widgets BRANCH requestfinished (branched from master) REVISION DETAIL https://phabricator.kd

D10105: baloo-widgets: Refactor filemetadataprovider for better readability

2018-01-29 Thread Michael Heidelbach
michaelh added a comment. Just to be sure: $ git checkout -b otto origin/master $ arc patch D10105 now in branch arcpatch-D10105_1 $ arc checkout dataavail $ arc rebase arcpatch-D10105_1 $ arc checkout unittest $ arc rebase dataavail and so on? REPOSITOR

D10105: baloo-widgets: Refactor filemetadataprovider for better readability

2018-01-29 Thread Nathaniel Graham
ngraham added a comment. Yes, Arc tracks dependencies on the revision level, not the branch level. If you're nervous, you don't even need to delete the local branch, either. Just switch to master and pull down this copy of it with `arc patch D10105` REPOSITORY R824 Baloo Widgets BRANC

D10105: baloo-widgets: Refactor filemetadataprovider for better readability

2018-01-29 Thread Michael Heidelbach
michaelh added a comment. Are you aware of the 3 branches dependent on this one? REPOSITORY R824 Baloo Widgets BRANCH requestfinished (branched from master) REVISION DETAIL https://phabricator.kde.org/D10105 To: michaelh, elvisangelaccio, ngraham, vhanda, smithjd, #dolphin, #framework

D10105: baloo-widgets: Refactor filemetadataprovider for better readability

2018-01-29 Thread Nathaniel Graham
ngraham added a comment. In cases like that, I usually delete the local branch and re-download the patch using Arc's version: `arc patch D10105`. Then go onto the `arcpatch-D10105` branch and `arc land --preview` from there. REPOSITORY R824 Baloo Widgets BRANCH requestfinished (branched

D10105: baloo-widgets: Refactor filemetadataprovider for better readability

2018-01-29 Thread Michael Heidelbach
michaelh added a comment. Can't land this Usage Exception: arc can not identify which revision exists on branch 'refactor'. $ arc branch arcpatch-D10105 Accepted D10105: baloo-widgets: Refactor filemetadataprovider for better readability master

D10113: baloo-widgets: Emit metaDataRequestFinished once per request

2018-01-29 Thread Michael Heidelbach
michaelh planned changes to this revision. michaelh added a comment. Needs rebasing REPOSITORY R824 Baloo Widgets REVISION DETAIL https://phabricator.kde.org/D10113 To: michaelh, elvisangelaccio, smithjd, vhanda, ngraham, #dolphin, #frameworks Cc: dhaumann

D8532: [WIP] Restrict file extractor with Seccomp

2018-01-29 Thread David Kahles
davidk added a comment. So, are there any more opinions on the whitelist vs. blacklist topic? Personally I still prefer the blacklist as I fear regressions in the future, especially because baloo is unmaintained. REPOSITORY R293 Baloo REVISION DETAIL https://phabricator.kde.org/D8532

D8532: [WIP] Restrict file extractor with Seccomp

2018-01-29 Thread David Kahles
davidk edited the summary of this revision. davidk edited the test plan for this revision. REPOSITORY R293 Baloo REVISION DETAIL https://phabricator.kde.org/D8532 To: davidk, apol, ossi Cc: ngraham, nicolasfella, #frameworks, michaelh

KDE CI: Frameworks kio kf5-qt5 FreeBSDQt5.9 - Build # 82 - Still Unstable!

2018-01-29 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks%20kio%20kf5-qt5%20FreeBSDQt5.9/82/ Project: Frameworks kio kf5-qt5 FreeBSDQt5.9 Date of build: Mon, 29 Jan 2018 12:31:52 + Build duration: 37 min and counting JUnit Tests Name: (root)

KDE CI: Frameworks purpose kf5-qt5 FreeBSDQt5.9 - Build # 13 - Unstable!

2018-01-29 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks%20purpose%20kf5-qt5%20FreeBSDQt5.9/13/ Project: Frameworks purpose kf5-qt5 FreeBSDQt5.9 Date of build: Mon, 29 Jan 2018 12:33:47 + Build duration: 24 min and counting JUnit Tests Name:

Re: KDirListerCache destroy comment in kcoredirlister.cpp

2018-01-29 Thread René J . V . Bertin
On Monday January 29 2018 11:32:56 René J.V. Bertin wrote: Hi, > The difference must be in the order in which certain things are deleted > during the global destruction phase, but which and how to restore the proper > order. One obvious difference is the fact that those KF5 applications link

Re: save and restore the geometry of KMainWindow

2018-01-29 Thread Alexander Semke
resending this to kde-devel... On 28.01.2018 18:45, Alexander Semke wrote: Hi, KMainWindow takes care of saving/restoring the state of the main window, menus and toolbars. Also, the size of the main window is correctly stored and restored. However, I don't see how to save and restore the pos

D10174: Add syntax highlighting for SELinux CIL Policies & File Contexts

2018-01-29 Thread Nibaldo González
nibags added a project: Framework: Syntax Highlighting. REPOSITORY R216 Syntax Highlighting REVISION DETAIL https://phabricator.kde.org/D10174 To: nibags, #framework_syntax_highlighting Cc: #framework_syntax_highlighting, #frameworks, michaelh, genethomas, cullmann, vkrause, dhaumann

D10174: Add syntax highlighting for SELinux CIL Policies & File Contexts

2018-01-29 Thread Nibaldo González
nibags edited the summary of this revision. nibags added a reviewer: Framework: Syntax Highlighting. nibags added a subscriber: Framework: Syntax Highlighting. REPOSITORY R216 Syntax Highlighting REVISION DETAIL https://phabricator.kde.org/D10174 To: nibags, #framework_syntax_highlighting Cc

D10174: Add syntax highlighting for SELinux CIL Policies & File Contexts

2018-01-29 Thread Nibaldo González
nibags created this revision. Restricted Application added a project: Frameworks. Restricted Application added a subscriber: Frameworks. nibags requested review of this revision. REVISION SUMMARY SELinux CIL Policies: The SELinux Common Intermediate Language (CIL) is a language for creating

D10173: windowthumbnail: Fix the GLXFBConfig selection code

2018-01-29 Thread Fredrik Höglund
fredrik created this revision. fredrik added a reviewer: Plasma. Restricted Application added projects: Plasma, Frameworks. Restricted Application added subscribers: Frameworks, plasma-devel. fredrik requested review of this revision. REVISION SUMMARY Check that the sizes of the color channels i

KDirListerCache destroy comment in kcoredirlister.cpp

2018-01-29 Thread René J . V . Bertin
Hi, Researching an issue I'm seeing (hang/crash on exit) I found the following vestigial out-commented code the KCoreDirListerCache ctor in kcoredirlister.cpp: // Probably not needed in KF5 anymore: // The use of KUrl::url() in ~DirItem (sendSignal) crashes if the static for QRegExpEngi

D10045: Don't assert on empty names

2018-01-29 Thread David Faure
dfaure added a comment. See https://phabricator.kde.org/D10172 for the fix for the first usability issue. However I can't reproduce the assert... Can you add this to your kio_remote to find out more? There must be one strange .desktop file around somewhere. diff --git a/src/ios

D10172: KRun: allow executing "add network folder" without confirmation prompt

2018-01-29 Thread David Faure
dfaure created this revision. dfaure added reviewers: mwolff, dhaumann. Restricted Application added a project: Frameworks. dfaure requested review of this revision. REVISION SUMMARY Dolphin enables the KRun feature of asking before executing shell scripts and desktop files (not really for sec

D10166: Add -Wlogical-op -Wzero-as-null-pointer-constant to KF5 warnings

2018-01-29 Thread Christophe Giboudeaux
cgiboudeaux added a comment. These flags are not in the Clang warnings (list for each version here: https://github.com/Barro/compiler-warnings) REPOSITORY R240 Extra CMake Modules REVISION DETAIL https://phabricator.kde.org/D10166 To: aacid Cc: cgiboudeaux, dhaumann, #frameworks, #build