GENERAL INFO
BUILD UNSTABLE
Build URL:
https://build.kde.org/job/kio%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/397/
Project: PLATFORM=Linux,compiler=gcc
Date of build: Sun, 22 Jan 2017 22:14:19 +
Build duration: 15 min
CHANGE SET
Revision 97ba474388b3e3e98dec72c7cc5cdb0903f536a0 by
adridg added a comment.
Also, kactivities-stats could use an .arcconfig
REPOSITORY
R159 KActivities Statistics
REVISION DETAIL
https://phabricator.kde.org/D4248
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: adridg, #frameworks, ivan, #kactivities
adridg added reviewers: Frameworks, ivan, KActivities.
REPOSITORY
R159 KActivities Statistics
REVISION DETAIL
https://phabricator.kde.org/D4248
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: adridg, #frameworks, ivan, #kactivities
Cc: #frameworks
adridg created this revision.
Restricted Application added a project: Frameworks.
Restricted Application added a subscriber: Frameworks.
REVISION SUMMARY
The autotests/CMakeLists.txt doesn't add Boost_INCLUDE_DIR to the
include search path (src/CMakeLists.txt does) so the autotests fail
to
adridg marked 5 inline comments as done.
adridg added inline comments.
INLINE COMMENTS
> kfunk wrote in FindInotify.cmake:51
> Style: Indent off
Assuming that means "comments indent 0" and not "comments indentation should
match surrounding indent, which is 4". It's a bit ambiguous.
> kfunk
adridg updated this revision to Diff 10439.
adridg added a comment.
Restricted Application added projects: Frameworks, Buildsystem.
- Follow up on review
REPOSITORY
R240 Extra CMake Modules
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D3826?vs=9401=10439
BRANCH
inotify
GENERAL INFO
BUILD UNSTABLE
Build URL:
https://build.kde.org/job/kio%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/396/
Project: PLATFORM=Linux,compiler=gcc
Date of build: Sun, 22 Jan 2017 19:51:07 +
Build duration: 16 min
CHANGE SET
Revision d252f7958633f3374e77e1cc52f11c534ca55219 by
> On Jan. 22, 2017, 7:22 p.m., Milian Wolff wrote:
> > lgtm, and I doubt that emoticons play a role for kate/kdevelop/krita or
> > anything else on windows where dbus is usually disabled?
If it's a problem, we could make the dbus stuff optional, i.e. it would all
work for a standalone app,
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129870/#review102189
---
Ship it!
lgtm, and I doubt that emoticons play a role
drosca updated this revision to Diff 10438.
drosca added a comment.
Add autotest
REPOSITORY
R302 KIconThemes
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D4247?vs=10437=10438
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D4247
AFFECTED FILES
cactus added a comment.
In https://phabricator.kde.org/D4234#79253, @brauch wrote:
> I'm a bit skeptical, this means I can't add closing braces when they are
missing ...
Yes, I agree with you. If you are in this situation: (((|)) with | for the
cursor, you must type 3 ')' to
drosca created this revision.
drosca added a reviewer: Frameworks.
Restricted Application added a project: Frameworks.
REVISION SUMMARY
Match the behavior of Qt's internal icon engines.
TEST PLAN
Rendering is the same for unaffected icons + highdpi still works.
Example case: Render icon
brauch added a comment.
I'm a bit skeptical, this means I can't add closing braces when they are
missing ...
REPOSITORY
R39 KTextEditor
REVISION DETAIL
https://phabricator.kde.org/D4234
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: cactus,
cullmann added a comment.
I am not sure if "path" is enough here, but also the filename is important.
Could we not guard it with "filename changed" instead for local files?
REPOSITORY
R39 KTextEditor
REVISION DETAIL
https://phabricator.kde.org/D4190
EMAIL PREFERENCES
cullmann added a comment.
Given it eliminates some members, this looks nice, thought I don't use that
feature enough to judge if the behavior change is wanted.
REPOSITORY
R39 KTextEditor
REVISION DETAIL
https://phabricator.kde.org/D4234
EMAIL PREFERENCES
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129870/
---
Review request for KDE Frameworks and Milian Wolff.
Repository:
16 matches
Mail list logo