KDE CI: Frameworks » kcoreaddons » kf5-qt5 SUSEQt5.13 - Build # 22 - Fixed!

2019-08-27 Thread CI System
BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks/job/kcoreaddons/job/kf5-qt5%20SUSEQt5.13/22/ Project: kf5-qt5 SUSEQt5.13 Date of build: Wed, 28 Aug 2019 05:49:19 + Build duration: 9 min 15 sec and counting BUILD ARTIFACTS

D22143: Add proper logging using ECMQtDeclareLoggingCategory

2019-08-27 Thread Méven Car
meven updated this revision to Diff 64787. meven added a comment. Rebase on master REPOSITORY R159 KActivities Statistics CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D22143?vs=64784=64787 BRANCH arcpatch-D22143 REVISION DETAIL https://phabricator.kde.org/D22143 AFFECTED

D23500: Support for native Matlab strings

2019-08-27 Thread Dominik Haumann
dhaumann added a comment. If you use kateversion="5.0" then you can use the new text styles we introduced with KDE 5: https://kate-editor.org/2014/03/07/kate-part-kf5-new-default-styles-for-better-color-schemes/ This allows to remove many of the hardcoded colors so that the

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

2019-08-27 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kcoreaddons/job/kf5-qt5%20FreeBSDQt5.13/20/ Project: kf5-qt5 FreeBSDQt5.13 Date of build: Wed, 28 Aug 2019 05:49:19 + Build duration: 1 min 46 sec and counting JUnit Tests Name:

D23497: Keep 'lib' as default libdir on Arch Linux based systems

2019-08-27 Thread Antonio Rojas
This revision was automatically updated to reflect the committed changes. Closed by commit R240:c806bd48a29f: Keep lib as default LIBDIR on Arch Linux based systems (authored by arojas). REPOSITORY R240 Extra CMake Modules CHANGES SINCE LAST UPDATE

D22144: Add kio recentlyused:/ to access KActivityStats data

2019-08-27 Thread Méven Car
meven added a comment. @dfaure A missing piece for this kio slave is to be able to filter to the current app, meaning the app using the slave. I am not sure there is a way to achieve this yet, please correct me if I am wrong, I've looked around but I could have missed something.

D22144: Add kio recentlyused:/ to access KActivityStats data

2019-08-27 Thread Méven Car
meven marked 7 inline comments as done. meven added inline comments. INLINE COMMENTS > dfaure wrote in recentlyused.cpp:91 > I'm confused. Which code from ForwardingSlaveBase is this class benefiting > from? > With internalRewriteUrl returning false all SlaveBase reimplementations in > that

D22144: Add kio recentlyused:/ to access KActivityStats data

2019-08-27 Thread Méven Car
meven marked 2 inline comments as done. REPOSITORY R320 KIO Extras REVISION DETAIL https://phabricator.kde.org/D22144 To: meven, ivan, #frameworks, ngraham, dfaure Cc: dhaumann, elvisangelaccio, kde-frameworks-devel, kfm-devel, aprcela, vmarinescu, fprice, LeGast00n, MrPepe, fbampaloukas,

D22143: Add proper logging using ECMQtDeclareLoggingCategory

2019-08-27 Thread Méven Car
meven updated this revision to Diff 64784. meven added a comment. kactivities-stat-logsettings.h -> kactivities-stats-logsettings.h, KACTIVITY_STAT_LOG -> KACTIVITIE_STATS_LOG REPOSITORY R159 KActivities Statistics CHANGES SINCE LAST UPDATE

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

2019-08-27 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kio/job/kf5-qt5%20FreeBSDQt5.13/39/ Project: kf5-qt5 FreeBSDQt5.13 Date of build: Wed, 28 Aug 2019 03:10:54 + Build duration: 15 min and counting JUnit Tests Name: projectroot

KDE CI: Frameworks » purpose » kf5-qt5 SUSEQt5.12 - Build # 104 - Unstable!

2019-08-27 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/purpose/job/kf5-qt5%20SUSEQt5.12/104/ Project: kf5-qt5 SUSEQt5.12 Date of build: Wed, 28 Aug 2019 03:11:39 + Build duration: 8 min 57 sec and counting BUILD ARTIFACTS

D23512: Avoid overloading KCModule::changed

2019-08-27 Thread Aleix Pol Gonzalez
apol added a comment. +1 INLINE COMMENTS > kcmodule.h:382 > */ > void changed(); > Use KCONFIGWIDGETS_DEPRECATED? REPOSITORY R265 KConfigWidgets REVISION DETAIL https://phabricator.kde.org/D23512 To: davidedmundson Cc: apol, kde-frameworks-devel, LeGast00n, GB_2,

D23513: Gettext: Add "Translated String" style and spellChecking attribute

2019-08-27 Thread jonathan poelen
This revision was automatically updated to reflect the committed changes. Closed by commit R216:0f150cafac4b: Gettext: Add Translated String style and spellChecking attribute (authored by jpoelen). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D23513?vs=64782=64783#toc REPOSITORY R216

D23497: Keep 'lib' as default libdir on Arch Linux based systems

2019-08-27 Thread Aleix Pol Gonzalez
apol accepted this revision. This revision is now accepted and ready to land. REPOSITORY R240 Extra CMake Modules REVISION DETAIL https://phabricator.kde.org/D23497 To: arojas, apol Cc: kde-frameworks-devel, kde-buildsystem, LeGast00n, GB_2, bencreasy, michaelh, ngraham, bruns

D23513: Gettext: Add "Translated String" style and spellChecking attribute

2019-08-27 Thread jonathan poelen
jpoelen updated this revision to Diff 64782. jpoelen added a comment. - increase version REPOSITORY R216 Syntax Highlighting CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D23513?vs=64775=64782 BRANCH po_spellcheck (branched from master) REVISION DETAIL

D23515: Format class: add functions to know if XML files set style attributes

2019-08-27 Thread Nibaldo González
nibags added a dependent revision: D23516: Fix for all themes: allow turn off attributes in XML highlighting files. REPOSITORY R216 Syntax Highlighting REVISION DETAIL https://phabricator.kde.org/D23515 To: nibags, #framework_syntax_highlighting, dhaumann, cullmann, vkrause Cc:

D23516: Fix for all themes: allow turn off attributes in XML highlighting files

2019-08-27 Thread Nibaldo González
nibags created this revision. nibags added reviewers: KTextEditor, cullmann, vkrause, dhaumann. Herald added projects: Kate, Frameworks. Herald added subscribers: kde-frameworks-devel, kwrite-devel. nibags requested review of this revision. REVISION SUMMARY Depends on D23515

D23515: Format class: add functions to know if XML files set style attributes

2019-08-27 Thread Nibaldo González
nibags created this revision. nibags added reviewers: Framework: Syntax Highlighting, dhaumann, cullmann, vkrause. Herald added projects: Kate, Frameworks. Herald added subscribers: kde-frameworks-devel, kwrite-devel. nibags requested review of this revision. REVISION SUMMARY The functions

D23145: [Exe Thumbnailer] Match 16 bit executables and don't request icon types wrestool cannot handle

2019-08-27 Thread Kai Uwe Broulik
broulik updated this revision to Diff 64777. broulik added a comment. - Minimize unrelated changes... REPOSITORY R320 KIO Extras CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D23145?vs=63697=64777 REVISION DETAIL https://phabricator.kde.org/D23145 AFFECTED FILES

D23513: Gettext: Add "Translated String" style and spellChecking attribute

2019-08-27 Thread Christoph Cullmann
cullmann accepted this revision. cullmann added a comment. This revision is now accepted and ready to land. Nice! Please inc version and push. REPOSITORY R216 Syntax Highlighting BRANCH po_spellcheck (branched from master) REVISION DETAIL https://phabricator.kde.org/D23513 To:

D23513: Gettext: Add "Translated String" style and spellChecking attribute

2019-08-27 Thread jonathan poelen
jpoelen edited the summary of this revision. REPOSITORY R216 Syntax Highlighting REVISION DETAIL https://phabricator.kde.org/D23513 To: jpoelen, #framework_syntax_highlighting, dhaumann, cullmann Cc: kwrite-devel, kde-frameworks-devel, LeGast00n, GB_2, domson, michaelh, ngraham, bruns,

D23513: Gettext: Add "Translated String" style and spellChecking attribute

2019-08-27 Thread jonathan poelen
jpoelen edited the summary of this revision. REPOSITORY R216 Syntax Highlighting REVISION DETAIL https://phabricator.kde.org/D23513 To: jpoelen, #framework_syntax_highlighting, dhaumann, cullmann Cc: kwrite-devel, kde-frameworks-devel, LeGast00n, GB_2, domson, michaelh, ngraham, bruns,

D23513: Gettext: Add "Translated String" style and spellChecking attribute

2019-08-27 Thread jonathan poelen
jpoelen created this revision. jpoelen added reviewers: Framework: Syntax Highlighting, dhaumann, cullmann. jpoelen added projects: Kate, Frameworks. jpoelen requested review of this revision. REVISION SUMMARY New style ("Translated String") for a string associated with msgstr and

D23512: Avoid overloading KCModule::changed

2019-08-27 Thread David Edmundson
davidedmundson created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. davidedmundson requested review of this revision. REVISION SUMMARY KCModule contains a signal and a protected slot with the same name. With the new connect syntax this

D22143: Add proper logging using ECMQtDeclareLoggingCategory

2019-08-27 Thread Ivan Čukić
ivan requested changes to this revision. ivan added a comment. This revision now requires changes to proceed. Change everything to plural. I agree with @kossebau, though I don't think it will lead to problems in future, so I could say it is bearable dirtyness. INLINE COMMENTS >

D22143: Add proper logging using ECMQtDeclareLoggingCategory

2019-08-27 Thread Méven Car
meven added a comment. @kossebau what about my last changes ? @ivan I feel this is ok to have `${CMAKE_BINARY_DIR}/src/kactivities-stat-logsettings.cpp` in the test CMakeLists.txt file with the documentation associated with ECMQtDeclareLoggingCategory and my comment, no-one can miss

D23498: Highlighting of keywords, variables and operators as such for Matlab source files

2019-08-27 Thread Christoph Cullmann
cullmann requested changes to this revision. cullmann added a comment. This revision now requires changes to proceed. Change looks ok modulo missing version update, for more input please take a look at the reply in D23500 Support for native Matlab strings

D23484: gcc naming conflict

2019-08-27 Thread Christoph Cullmann
This revision was automatically updated to reflect the committed changes. Closed by commit R216:3ff7f78ebcb7: gcc naming conflict (authored by mickaelbo, committed by cullmann). REPOSITORY R216 Syntax Highlighting CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D23484?vs=64727=64757

D23472: Mimic QInputControl::isAcceptableInput() when filtering typed characters

2019-08-27 Thread Christoph Cullmann
This revision was automatically updated to reflect the committed changes. Closed by commit R39:0bf552243404: Mimic QInputControl::isAcceptableInput() when filtering typed characters (authored by ahmadsamir, committed by cullmann). REPOSITORY R39 KTextEditor CHANGES SINCE LAST UPDATE

D23472: Mimic QInputControl::isAcceptableInput() when filtering typed characters

2019-08-27 Thread Christoph Cullmann
cullmann accepted this revision. cullmann added a comment. This revision is now accepted and ready to land. Actually, I can take care of the changes after applying this, is already a good start, thanks! REPOSITORY R39 KTextEditor BRANCH ahmad/soft-hyphen (branched from master) REVISION

D23500: Support for native Matlab strings

2019-08-27 Thread Christoph Cullmann
cullmann requested changes to this revision. cullmann added a comment. This revision now requires changes to proceed. Hi, I think it is very nice that you pickup the matlab file! Three things I would like to have changed: 1. please move the example from the comment in the XML to

D23483: Fix section hierachy in distributing.md

2019-08-27 Thread Antonio Rojas
This revision was automatically updated to reflect the committed changes. Closed by commit R293:33e042970075: Fix section hierachy in distributing.md (authored by arojas). REPOSITORY R293 Baloo CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D23483?vs=64721=64754 REVISION DETAIL

D23500: Support for native Matlab strings

2019-08-27 Thread Benjamin Buch
bbuch added a reviewer: cullmann. REPOSITORY R216 Syntax Highlighting REVISION DETAIL https://phabricator.kde.org/D23500 To: bbuch, cullmann Cc: kwrite-devel, kde-frameworks-devel, LeGast00n, GB_2, domson, michaelh, ngraham, bruns, demsking, cullmann, sars, dhaumann

D23500: Support for native Matlab strings

2019-08-27 Thread Benjamin Buch
bbuch created this revision. Herald added projects: Kate, Frameworks. Herald added subscribers: kde-frameworks-devel, kwrite-devel. bbuch requested review of this revision. REVISION SUMMARY As of Matlab 2016b, a separate string data type and literal was introduced in Matlab.

D23483: Fix section hierachy in distributing.md

2019-08-27 Thread Friedrich W. H. Kossebau
kossebau accepted this revision. kossebau added a comment. This revision is now accepted and ready to land. Seems to me more a bug of doxygen. Is it not allowed to put level-3 titles below level-1 titles? At least as long as one does not do numbering, there should not be a problem, in

D23207: Fixing implementation of FileJob interface in smb/sftp slaves

2019-08-27 Thread Harald Sitter
sitter added a comment. SlaveBase (which the slaves derive from) runs a command loop, out of this command loop come the actual calls to the API functions. So, in said command loop we can verify which state the slave is in before and more importantly after any API call. Specifically about

D23498: Highlighting of keywords, variables and operators as such for Matlab source files

2019-08-27 Thread Benjamin Buch
bbuch added a reviewer: cullmann. REPOSITORY R216 Syntax Highlighting REVISION DETAIL https://phabricator.kde.org/D23498 To: bbuch, cullmann Cc: kwrite-devel, kde-frameworks-devel, LeGast00n, GB_2, domson, michaelh, ngraham, bruns, demsking, cullmann, sars, dhaumann

D23498: Highlighting of keywords, variables and operators as such for Matlab source files

2019-08-27 Thread Benjamin Buch
bbuch created this revision. Herald added projects: Kate, Frameworks. Herald added subscribers: kde-frameworks-devel, kwrite-devel. bbuch requested review of this revision. REVISION SUMMARY Currently, Matlab source files display keywords, variables, and operators as normal text. So they don't

D23490: [KUrlNavigator] Add MIME types supported by krarc to isCompressedPath

2019-08-27 Thread Nathaniel Graham
ngraham edited the summary of this revision. ngraham edited reviewers, added: Frameworks, cfeck, pino, dfaure; removed: kde-frameworks-devel. Herald added a subscriber: kde-frameworks-devel. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D23490 To: nhiga, #frameworks,

D23207: Fixing implementation of FileJob interface in smb/sftp slaves

2019-08-27 Thread Alexander Saoutkin
feverfew added a comment. In D23207#520233 , @sitter wrote: > Not knowing the background here at a glance I would argue that SlaveBase in KIO should be getting state verification on all of this,. Sorry, I'm not sure what you mean by

D23497: Keep 'lib' as default libdir on Arch Linux based systems

2019-08-27 Thread Antonio Rojas
arojas created this revision. arojas added a reviewer: apol. Herald added projects: Frameworks, Build System. Herald added subscribers: kde-buildsystem, kde-frameworks-devel. arojas requested review of this revision. REVISION SUMMARY Port of upstream commit

D23207: Fixing implementation of FileJob interface in smb/sftp slaves

2019-08-27 Thread Harald Sitter
sitter added a comment. Not knowing the background here at a glance I would argue that SlaveBase in KIO should be getting state verification on all of this,. REPOSITORY R320 KIO Extras REVISION DETAIL https://phabricator.kde.org/D23207 To: feverfew, chinmoyr, fvogt, sitter, dfaure Cc:

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

2019-08-27 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kio/job/kf5-qt5%20FreeBSDQt5.13/38/ Project: kf5-qt5 FreeBSDQt5.13 Date of build: Tue, 27 Aug 2019 14:55:04 + Build duration: 8 min 22 sec and counting JUnit Tests Name:

D21721: Bring KNewStuffQuick to feature parity with KNewStuff(Widgets)

2019-08-27 Thread Dan Leinir Turthra Jensen
leinir edited the summary of this revision. REPOSITORY R304 KNewStuff REVISION DETAIL https://phabricator.kde.org/D21721 To: leinir, #knewstuff, #vdg, #frameworks Cc: anthonyfieroni, pino, ngraham, kde-frameworks-devel, LeGast00n, GB_2, michaelh, bruns

D23344: assert slave command finality

2019-08-27 Thread Harald Sitter
sitter updated this revision to Diff 64744. sitter added a comment. - wrap in custom assert defines that either assert or qwarn based on a cmake option - new cmake option KIO_ASSERT_SLAVE_STATES enables the asserts. the option is only on by default when run on jenkins - fix a bunch of

D23493: [server] Cache current mode

2019-08-27 Thread Roman Gilg
romangg added a dependent revision: D23495: Get output pixel size from output device. REPOSITORY R127 KWayland REVISION DETAIL https://phabricator.kde.org/D23493 To: romangg, #kwin Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D23103: Add decibel power units (dBW and multiples)

2019-08-27 Thread Nathaniel Graham
This revision was automatically updated to reflect the committed changes. Closed by commit R292:edca5ae10796: Add decibel power units (dBW and multiples) (authored by iasensio, committed by ngraham). REPOSITORY R292 KUnitConversion CHANGES SINCE LAST UPDATE

D23103: Add decibel power units (dBW and multiples)

2019-08-27 Thread Kai Uwe Broulik
broulik accepted this revision. REPOSITORY R292 KUnitConversion BRANCH log_power_units REVISION DETAIL https://phabricator.kde.org/D23103 To: iasensio, broulik, ngraham Cc: ngraham, kde-frameworks-devel, LeGast00n, GB_2, michaelh, bruns

D23103: Add decibel power units (dBW and multiples)

2019-08-27 Thread Ismael Asensio
iasensio added a comment. Friendly ping REPOSITORY R292 KUnitConversion BRANCH log_power_units REVISION DETAIL https://phabricator.kde.org/D23103 To: iasensio, broulik, ngraham Cc: ngraham, kde-frameworks-devel, LeGast00n, GB_2, michaelh, bruns

D23493: [server] Cache current mode

2019-08-27 Thread Roman Gilg
romangg added a task: T11459: AbstractWaylandOutput and output device consolidation. REPOSITORY R127 KWayland REVISION DETAIL https://phabricator.kde.org/D23493 To: romangg, #kwin Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D23493: [server] Cache current mode

2019-08-27 Thread Roman Gilg
romangg created this revision. romangg added a reviewer: KWin. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. romangg requested review of this revision. REVISION SUMMARY Instead of looping over all modes for retrieving current mode data copy it to a

D23490: [KUrlNavigator] Add MIME types supported by krarc to isCompressedPath

2019-08-27 Thread N. Higa
nhiga added a comment. This patch may be related to Bug 411330 . REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D23490 To: nhiga, kde-frameworks-devel Cc: LeGast00n, GB_2, michaelh, ngraham, bruns

D23490: [KUrlNavigator] Add MIME types supported by krarc to isCompressedPath

2019-08-27 Thread N. Higa
nhiga created this revision. nhiga added a reviewer: kde-frameworks-devel. nhiga added a project: Frameworks. nhiga requested review of this revision. REVISION SUMMARY With D23476 , this patch should let users who have Krusader installed to view files inside

D22144: Add kio recentlyused:/ to access KActivityStats data

2019-08-27 Thread Méven Car
meven edited the test plan for this revision. REPOSITORY R320 KIO Extras REVISION DETAIL https://phabricator.kde.org/D22144 To: meven, ivan, #frameworks, ngraham, dfaure Cc: dhaumann, elvisangelaccio, kde-frameworks-devel, kfm-devel, aprcela, vmarinescu, fprice, LeGast00n, MrPepe,

D22144: Add kio recentlyused:/ to access KActivityStats data

2019-08-27 Thread Méven Car
meven updated this revision to Diff 64736. meven added a comment. Inherit from SlaveBase instead of ForwardingSlaveBase, rename url parameter to path parameter REPOSITORY R320 KIO Extras CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D22144?vs=64504=64736 BRANCH

D21721: Bring KNewStuffQuick to feature parity with KNewStuff(Widgets)

2019-08-27 Thread Dan Leinir Turthra Jensen
leinir edited the summary of this revision. REPOSITORY R304 KNewStuff REVISION DETAIL https://phabricator.kde.org/D21721 To: leinir, #knewstuff, #vdg, #frameworks Cc: anthonyfieroni, pino, ngraham, kde-frameworks-devel, LeGast00n, GB_2, michaelh, bruns

D21721: Bring KNewStuffQuick to feature parity with KNewStuff(Widgets)

2019-08-27 Thread Dan Leinir Turthra Jensen
leinir edited the summary of this revision. REPOSITORY R304 KNewStuff REVISION DETAIL https://phabricator.kde.org/D21721 To: leinir, #knewstuff, #vdg, #frameworks Cc: anthonyfieroni, pino, ngraham, kde-frameworks-devel, LeGast00n, GB_2, michaelh, bruns

D21721: Bring KNewStuffQuick to feature parity with KNewStuff(Widgets)

2019-08-27 Thread Dan Leinir Turthra Jensen
leinir retitled this revision from "[WIP] Bring KNewStuffQuick to feature parity with KNewStuff(Widgets)" to "Bring KNewStuffQuick to feature parity with KNewStuff(Widgets)". leinir edited the summary of this revision. leinir edited the test plan for this revision. leinir added reviewers:

D23484: gcc naming conflict

2019-08-27 Thread Christoph Cullmann
cullmann accepted this revision. cullmann added a comment. This revision is now accepted and ready to land. ok, thanks REVISION DETAIL https://phabricator.kde.org/D23484 To: mickaelbo, #framework_syntax_highlighting, cullmann Cc: jriddell, cullmann, kde-frameworks-devel, kwrite-devel,

D23484: gcc naming conflict

2019-08-27 Thread Mickael Bosch
mickaelbo updated this revision to Diff 64727. mickaelbo added a comment. It makes sense. CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D23484?vs=64719=64727 REVISION DETAIL https://phabricator.kde.org/D23484 AFFECTED FILES src/indexer/katehighlightingindexer.cpp To:

D23484: gcc naming conflict

2019-08-27 Thread Christoph Cullmann
cullmann requested changes to this revision. cullmann added a comment. This revision now requires changes to proceed. I would prefer that we just rename the vars. REPOSITORY R216 Syntax Highlighting REVISION DETAIL https://phabricator.kde.org/D23484 To: mickaelbo,

D23484: gcc naming conflict

2019-08-27 Thread Mickael Bosch
mickaelbo created this revision. mickaelbo added a reviewer: Framework: Syntax Highlighting. Herald added projects: Kate, Frameworks. Herald added a subscriber: kde-frameworks-devel. mickaelbo requested review of this revision. REVISION SUMMARY With gcc 7.4.0 (and maybe some other versions),

D23483: Fix section hierachy in distributing.md

2019-08-27 Thread Antonio Rojas
arojas created this revision. arojas added reviewers: bruns, kossebau. Herald added projects: Frameworks, Baloo. Herald added subscribers: Baloo, kde-frameworks-devel. arojas requested review of this revision. REVISION SUMMARY Fixes QCH build with doxygen 1.8.16, which seems to be more strict

D23420: Use solid to check if a KFileItem is located on a network mount

2019-08-27 Thread Kai Uwe Broulik
broulik added a comment. > isSlow() didn't check linkDest before. Even more reason to do it properly now :P `KFileSystemType` uses `statfs`. I didn't find any documentation on whether it follows symlinks, but it is documented to return an `ELOOP` error code "Too many symbolic links

D23420: Use solid to check if a KFileItem is located on a network mount

2019-08-27 Thread Méven Car
meven marked an inline comment as done. meven added a comment. In D23420#519949 , @broulik wrote: > Now we still need the `linkDest` check :) isSlow() didn't check linkDest before. REPOSITORY R241 KIO BRANCH arcpatch-D23420

D23420: Use solid to check if a KFileItem is located on a network mount

2019-08-27 Thread Méven Car
meven updated this revision to Diff 64716. meven added a comment. Add a const& clean unused variable REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D23420?vs=64715=64716 BRANCH arcpatch-D23420 REVISION DETAIL https://phabricator.kde.org/D23420 AFFECTED

D23420: Use solid to check if a KFileItem is located on a network mount

2019-08-27 Thread Anthony Fieroni
anthonyfieroni added inline comments. INLINE COMMENTS > anthonyfieroni wrote in kfileitem.cpp:764 > Get it as `const Solid::StorageAccess`, you can use `auto storageAccess = ...` Let clarify what David mean for (const Solid::Device& device : devices) { auto storageAccess = device.as();

D23420: Use solid to check if a KFileItem is located on a network mount

2019-08-27 Thread Kai Uwe Broulik
broulik added a comment. Now we still need the `linkDest` check :) REPOSITORY R241 KIO BRANCH arcpatch-D23420 REVISION DETAIL https://phabricator.kde.org/D23420 To: meven, dfaure, #frameworks Cc: broulik, anthonyfieroni, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham,

D23420: Use solid to check if a KFileItem is located on a network mount

2019-08-27 Thread Méven Car
meven updated this revision to Diff 64715. meven added a comment. better check file path REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D23420?vs=64645=64715 BRANCH arcpatch-D23420 REVISION DETAIL https://phabricator.kde.org/D23420 AFFECTED FILES

D23420: Use solid to check if a KFileItem is located on a network mount

2019-08-27 Thread Méven Car
meven marked an inline comment as done. REPOSITORY R241 KIO BRANCH arcpatch-D23420 REVISION DETAIL https://phabricator.kde.org/D23420 To: meven, dfaure, #frameworks Cc: broulik, anthonyfieroni, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

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

2019-08-27 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kcoreaddons/job/kf5-qt5%20FreeBSDQt5.13/19/ Project: kf5-qt5 FreeBSDQt5.13 Date of build: Tue, 27 Aug 2019 05:48:04 + Build duration: 47 min and counting JUnit Tests Name: