Re: Review Request 130013: Make PlainTextExtractor match "text/plain" mimetype

2017-03-17 Thread Matthieu Gallien
> On mars 15, 2017, 9:55 matin, Matthieu Gallien wrote: > > Ship It! > > Matthieu Gallien wrote: > KFileMetaData being in use by itself, IMHO Baloo problematics should not > block this correction. > > Matthieu Gallien wrote: > I almost forgot to ask an automatic test for your fix. Plea

Re: Review Request 130013: Make PlainTextExtractor match "text/plain" mimetype

2017-03-17 Thread Igor Poboiko
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/130013/ --- (Updated March 17, 2017, 8:23 a.m.) Status -- This change has been m

D5079: debug when disabling kcrash due to env vars

2017-03-17 Thread Harald Sitter
sitter created this revision. Restricted Application added a project: Frameworks. Restricted Application added a subscriber: Frameworks. REVISION SUMMARY otherwise it's fairly inobvious why drkonqi doesn't kick in REPOSITORY R285 KCrash BRANCH master REVISION DETAIL https://phabricator.

D5079: debug when disabling kcrash due to env vars

2017-03-17 Thread Harald Sitter
sitter added a reviewer: dfaure. REPOSITORY R285 KCrash REVISION DETAIL https://phabricator.kde.org/D5079 To: sitter, dfaure Cc: #frameworks

D5087: Introduce ecm_generate_qmltypes

2017-03-17 Thread Aleix Pol Gonzalez
apol created this revision. Restricted Application added projects: Frameworks, Build System. REVISION SUMMARY Introduces a function that allows to generate plugins.qmltypes files for our qml plugins using qmlplugindump much like Qt does. TEST PLAN Introduced it locally on Purpose and it wor

D4975: Prevent misdetection of EOF on Linux

2017-03-17 Thread Christoph Feck
cfeck added a reviewer: ossi. REPOSITORY R291 KPty REVISION DETAIL https://phabricator.kde.org/D4975 To: Lekensteyn, #konsole, hindenburg, ossi Cc: #frameworks

D5088: Generate plugins.qmltypes files for the plugins we install

2017-03-17 Thread Aleix Pol Gonzalez
apol created this revision. Restricted Application added projects: Plasma, Frameworks. Restricted Application added a subscriber: plasma-devel. REPOSITORY R242 Plasma Framework (Library) BRANCH master REVISION DETAIL https://phabricator.kde.org/D5088 AFFECTED FILES CMakeLists.txt src/

D5087: Introduce ecm_generate_qmltypes

2017-03-17 Thread Aleix Pol Gonzalez
apol updated this revision to Diff 12570. apol added a comment. Some adaptions so it works for Plasma Framework too REPOSITORY R240 Extra CMake Modules CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D5087?vs=12565&id=12570 BRANCH master REVISION DETAIL https://phabricator.kde

D5089: KDE compiler settings fail to consider AppleClang

2017-03-17 Thread René J . V . Bertin
rjvbb created this revision. rjvbb added projects: Build System, Frameworks. REVISION SUMMARY KDECompilerSettings.cmake and KDEFrameworksCompilerSettings.cmake fail to consider the fact that cmake considers Apple's clang compiler to be different (differences that exist but are largely irreleva

D4975: Prevent misdetection of EOF on Linux

2017-03-17 Thread Oswald Buddenhagen
ossi added a comment. please verify that this is in fact a kernel bug (include ml references), not something that should be expected for some weird reasons (compare the solaris path). if this is a bug, does it actually affect released versions which are not being "upgraded away" on short n

D4975: Prevent misdetection of EOF on Linux

2017-03-17 Thread Peter Wu
Lekensteyn added a comment. In https://phabricator.kde.org/D4975#95727, @ossi wrote: > please verify that this is in fact a kernel bug (include ml references), not something that should be expected for some weird reasons (compare the solaris path). I suspect it is a kernel bug, b

D5089: KDE compiler settings fail to consider AppleClang

2017-03-17 Thread Aleix Pol Gonzalez
apol added a comment. Maybe it would make sense to use MATCHES? are there any other *Clang*? REPOSITORY R240 Extra CMake Modules REVISION DETAIL https://phabricator.kde.org/D5089 To: rjvbb, #build_system, #frameworks Cc: apol, kde-mac, #frameworks, #build_system

Jenkins-kde-ci: kmediaplayer master kf5-qt5 » Linux,gcc - Build # 475 - Failure!

2017-03-17 Thread no-reply
GENERAL INFO BUILD FAILURE Build URL: https://build.kde.org/job/kmediaplayer%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/475/ Project: PLATFORM=Linux,compiler=gcc Date of build: Fri, 17 Mar 2017 17:57:41 + Build duration: 1 min 20 sec CHANGE SET No changes

D5089: KDE compiler settings fail to consider AppleClang

2017-03-17 Thread René J . V . Bertin
rjvbb added a comment. In https://phabricator.kde.org/D5089#95732, @apol wrote: > Maybe it would make sense to use MATCHES? are there any other *Clang*? That was my first idea, use `MATCHES "*Clang"`, but then I realised I have no idea if there are other SomethingClang. Searching

Jenkins-kde-ci: kmediaplayer master kf5-qt5 » Linux,gcc - Build # 476 - Fixed!

2017-03-17 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/kmediaplayer%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/476/ Project: PLATFORM=Linux,compiler=gcc Date of build: Fri, 17 Mar 2017 18:10:12 + Build duration: 57 sec CHANGE SET No changes JUNIT RESULTS Name: (root)

Jenkins-kde-ci: kmediaplayer master kf5-qt5 » Linux,gcc - Build # 476 - Fixed!

2017-03-17 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/kmediaplayer%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/476/ Project: PLATFORM=Linux,compiler=gcc Date of build: Fri, 17 Mar 2017 18:10:12 + Build duration: 57 sec CHANGE SET No changes JUNIT RESULTS Name: (root)

D5089: KDE compiler settings fail to consider AppleClang

2017-03-17 Thread Kevin Funk
kfunk added a comment. +1 to apol's recommendation. `... MATCHES "Clang"` is the expression everyone uses. Even in cmake.git: Modules/CMakeDetermineCCompiler.cmake 138: if(CMAKE_C_COMPILER_ID MATCHES "GNU" OR CMAKE_C_COMPILER_ID MATCHES "Clang") Also see: http://stackov

D5089: KDE compiler settings fail to consider AppleClang

2017-03-17 Thread Kevin Funk
kfunk requested changes to this revision. This revision now requires changes to proceed. REPOSITORY R240 Extra CMake Modules REVISION DETAIL https://phabricator.kde.org/D5089 To: rjvbb, #build_system, #frameworks, kfunk Cc: kfunk, apol, kde-mac, #frameworks, #build_system

D4975: Prevent misdetection of EOF on Linux

2017-03-17 Thread Oswald Buddenhagen
ossi added a comment. first try googling it; maybe there is even a response on stackoverflow (as so often happens). then try posting it yourself to stackoverflow. dunno whether such questions are welcome on the linux-kernel list, but if not, there might be other lists where it would fit.

D4975: Prevent misdetection of EOF on Linux

2017-03-17 Thread Peter Wu
Lekensteyn added a comment. In https://phabricator.kde.org/D4975#95754, @ossi wrote: > first try googling it; maybe there is even a response on stackoverflow (as so often happens). Was one of the first things to try, that only pointed to an issue in Emacs and another that clarifi

D4975: Prevent misdetection of EOF on Linux

2017-03-17 Thread Oswald Buddenhagen
ossi added a comment. yeah, but without knowing the root cause, your fix may be just wrong, or at least the comment/commit message may be misleading. also, https://community.kde.org/Policies/Commit_Policy#Don.27t_commit_code_you_don.27t_understand REPOSITORY R291 KPty REVISION DETAIL

D5090: More focus on KIO than Konqueror

2017-03-17 Thread Luigi Toscano
ltoscano created this revision. Restricted Application added a project: Frameworks. Restricted Application added a subscriber: Frameworks. REVISION SUMMARY - more highlight on "applications that use a certain KIO", Konqueror is an example; - use kio_http and not Konqueror in the user agent;

D5091: Updated user agents (Firefox 52 ESR, Chromium 52)

2017-03-17 Thread Luigi Toscano
ltoscano created this revision. Restricted Application added a project: Frameworks. Restricted Application added a subscriber: Frameworks. REVISION SUMMARY New user agents, as the available ones are a bit old. TEST PLAN Compiles, and the new user agents can be selected for example in Konque

D5092: [KMessageWidget] Fix wrong size hint when animatedShow() is called with a hidden parentWidget()

2017-03-17 Thread Christoph Feck
cfeck created this revision. Restricted Application added a project: Frameworks. Restricted Application added a subscriber: Frameworks. REVISION SUMMARY If the parent widget is not visible, then immediately show() the message widget to compute proper size hint. Otherwise, when the widget is lat