dfaure added a comment.
That's trivial, isn't it? Rename networking.cpp to networking_unix.cpp, and
move to networking.cpp the methods that are platform-independent.
REPOSITORY
R239 KDELibs4Support
REVISION DETAIL
https://phabricator.kde.org/D9386
To: habacker, dfaure, ltoscano,
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kservice%20kf5-qt5%20SUSEQt5.10/9/
Project:
Frameworks kservice kf5-qt5 SUSEQt5.10
Date of build:
Wed, 20 Dec 2017 07:44:48 +
Build duration:
4 min 18 sec and counting
JUnit Tests
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kdelibs4support%20kf5-qt5%20SUSEQt5.7/14/
Project:
Frameworks kdelibs4support kf5-qt5 SUSEQt5.7
Date of build:
Wed, 20 Dec 2017 06:29:00 +
Build duration:
12 min and counting
JUnit Tests
BUILD FAILURE
Build URL
https://build.kde.org/job/Frameworks%20kdelibs4support%20kf5-qt5%20SUSEQt5.7/13/
Project:
Frameworks kdelibs4support kf5-qt5 SUSEQt5.7
Date of build:
Wed, 20 Dec 2017 06:04:00 +
Build duration:
22 min and counting
JUnit Tests
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kdelibs4support%20kf5-qt5%20SUSEQt5.10/13/
Project:
Frameworks kdelibs4support kf5-qt5 SUSEQt5.10
Date of build:
Wed, 20 Dec 2017 06:04:00 +
Build duration:
21 min and counting
JUnit
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kdelibs4support%20kf5-qt5%20FreeBSDQt5.9/8/
Project:
Frameworks kdelibs4support kf5-qt5 FreeBSDQt5.9
Date of build:
Wed, 20 Dec 2017 06:04:00 +
Build duration:
15 min and counting
JUnit
bcooksley added a comment.
I still question how these installs conflict - as all the scenarios where
they could conflict should already be taken into account:
1. Linux distros, FreeBSD, etc have no co-installability issues here
2. Windows installs should be one prefix per application,
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kdelibs4support%20kf5-qt5%20FreeBSDQt5.9/7/
Project:
Frameworks kdelibs4support kf5-qt5 FreeBSDQt5.9
Date of build:
Wed, 20 Dec 2017 01:16:31 +
Build duration:
17 min and counting
JUnit
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kdelibs4support%20kf5-qt5%20SUSEQt5.7/12/
Project:
Frameworks kdelibs4support kf5-qt5 SUSEQt5.7
Date of build:
Wed, 20 Dec 2017 01:16:32 +
Build duration:
11 min and counting
JUnit Tests
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kdelibs4support%20kf5-qt5%20SUSEQt5.10/12/
Project:
Frameworks kdelibs4support kf5-qt5 SUSEQt5.10
Date of build:
Wed, 20 Dec 2017 01:16:32 +
Build duration:
6 min 50 sec and counting
habacker closed this revision.
REPOSITORY
R239 KDELibs4Support
REVISION DETAIL
https://phabricator.kde.org/D9386
To: habacker, dfaure, ltoscano, bcooksley
Cc: #frameworks
This revision was automatically updated to reflect the committed changes.
Closed by commit R311:32745503b519: Use cmake variable kwallet_xml for
specifing org.kde.KWallet.xml (authored by habacker).
CHANGED PRIOR TO COMMIT
https://phabricator.kde.org/D9381?vs=24036=24135#toc
REPOSITORY
R311
This revision was automatically updated to reflect the committed changes.
Closed by commit R311:74ce261240ae: Use correct service executable path for
installing kwalletd dbus service on… (authored by habacker).
REPOSITORY
R311 KWallet
CHANGES SINCE LAST UPDATE
habacker added a comment.
In https://phabricator.kde.org/D9404#181209, @dfaure wrote:
> So the if() is needed,
yes,
> the AFTER isn't enough?
I tried that and it does not help - AFTER|BEFORE affects only the order of
directories added with include_directories
> I
habacker added a comment.
In https://phabricator.kde.org/D9386#181227, @dfaure wrote:
> If this wasn't kdelibs4support, I'd insist for sharing the implementation
rather than duplicating it.
Would be nice to share, but networking.cpp is only be used on non win32
platform, so how
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kio%20kf5-qt5%20FreeBSDQt5.9/18/
Project:
Frameworks kio kf5-qt5 FreeBSDQt5.9
Date of build:
Tue, 19 Dec 2017 23:07:17 +
Build duration:
24 min and counting
JUnit Tests
Name: (root)
dfaure accepted this revision.
This revision is now accepted and ready to land.
REPOSITORY
R311 KWallet
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D9381
To: habacker, dfaure
Cc: #frameworks
dfaure added a comment.
Well, the kstars hack shouldn't exist and makes me cringe.
But indeed the kate mac-packaging script is a valid case, please get in touch
with Christoph Cullmann about it.
The rekonq use case looks valid too but that code doesn't seem really
maintained
dfaure accepted this revision.
dfaure added a comment.
This revision is now accepted and ready to land.
If this wasn't kdelibs4support, I'd insist for sharing the implementation
rather than duplicating it. But oh well :-)
REPOSITORY
R239 KDELibs4Support
BRANCH
master
REVISION DETAIL
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kio%20kf5-qt5%20SUSEQt5.10/38/
Project:
Frameworks kio kf5-qt5 SUSEQt5.10
Date of build:
Tue, 19 Dec 2017 23:00:46 +
Build duration:
7 min 31 sec and counting
JUnit Tests
Name:
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kio%20kf5-qt5%20SUSEQt5.7/39/
Project:
Frameworks kio kf5-qt5 SUSEQt5.7
Date of build:
Tue, 19 Dec 2017 23:00:46 +
Build duration:
7 min 28 sec and counting
JUnit Tests
Name: (root)
dfaure added a comment.
QStandardPaths::InstallLocation cannot ever exist, if you mean by that "the
install location for KF5" or "the install location for a KF5-based
application". How would that work? There could be (at least) 3 different
install locations, at least on Unix: one for Qt,
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kio%20kf5-qt5%20FreeBSDQt5.9/17/
Project:
Frameworks kio kf5-qt5 FreeBSDQt5.9
Date of build:
Tue, 19 Dec 2017 22:48:01 +
Build duration:
18 min and counting
JUnit Tests
Name: (root)
This revision was automatically updated to reflect the committed changes.
Closed by commit R241:d9c5673c20ea: TransferJob: fix for when the
readChannelFinished has already been emitted… (authored by aacid).
REPOSITORY
R241 KIO
CHANGES SINCE LAST UPDATE
aacid closed this revision.
REPOSITORY
R241 KIO
REVISION DETAIL
https://phabricator.kde.org/D9190
To: aacid, dfaure, apol, albertvaka
Cc: ngraham, nicolasfella, anthonyfieroni, #frameworks
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kio%20kf5-qt5%20SUSEQt5.7/38/
Project:
Frameworks kio kf5-qt5 SUSEQt5.7
Date of build:
Tue, 19 Dec 2017 22:48:01 +
Build duration:
10 min and counting
JUnit Tests
Name: (root)
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kio%20kf5-qt5%20SUSEQt5.10/37/
Project:
Frameworks kio kf5-qt5 SUSEQt5.10
Date of build:
Tue, 19 Dec 2017 22:48:01 +
Build duration:
6 min 21 sec and counting
JUnit Tests
Name:
This revision was automatically updated to reflect the committed changes.
Closed by commit R241:ca2364d9c203: Fix crash, presumably since Qt 5.10?
(authored by apol).
REPOSITORY
R241 KIO
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D9399?vs=24103=24130
REVISION DETAIL
apol accepted this revision.
This revision is now accepted and ready to land.
REPOSITORY
R246 Sonnet
REVISION DETAIL
https://phabricator.kde.org/D9018
To: leinir, #frameworks, apol
Cc: vonreth, cgiboudeaux, alexeymin, apol, #frameworks
dfaure added a comment.
So the if() is needed, the AFTER isn't enough?
I wonder how AFTER works in combination with linking to targets that provide
include dirs.
This looks all a bit hackish, but I don't have a better suggestion other than
this possible simplification.
REPOSITORY
dfaure accepted this revision.
This revision is now accepted and ready to land.
REPOSITORY
R241 KIO
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D9399
To: apol, #frameworks, dfaure
Cc: anthonyfieroni, ngraham
dfaure added a comment.
Well, if this QSet can be huge, then these conversions from QList and to
QList are going to take a lot of CPU.
Are you sure it wouldn't be much faster to do a linear search at append time,
in exchange for saving all those temporary-containers conversions?
This
dfaure accepted this revision.
REPOSITORY
R311 KWallet
REVISION DETAIL
https://phabricator.kde.org/D9383
To: habacker, dfaure, ltoscano, bcooksley
Cc: #frameworks
dfaure accepted this revision.
dfaure added a comment.
This revision is now accepted and ready to land.
Oops :-)
REPOSITORY
R241 KIO
REVISION DETAIL
https://phabricator.kde.org/D9405
To: broulik, dfaure
Cc: #frameworks
leinir added a comment.
Would it be possible to get an accept here, or is there something people
fundamentally disagree with about it?
REPOSITORY
R246 Sonnet
REVISION DETAIL
https://phabricator.kde.org/D9018
To: leinir, #frameworks
Cc: vonreth, cgiboudeaux, alexeymin, apol, #frameworks
vonreth added a comment.
+1
REPOSITORY
R246 Sonnet
REVISION DETAIL
https://phabricator.kde.org/D9018
To: leinir, #frameworks
Cc: vonreth, cgiboudeaux, alexeymin, apol, #frameworks
BUILD SUCCESS
Build URL
https://build.kde.org/job/Frameworks%20kwayland%20kf5-qt5%20SUSEQt5.7/11/
Project:
Frameworks kwayland kf5-qt5 SUSEQt5.7
Date of build:
Tue, 19 Dec 2017 15:56:35 +
Build duration:
8 min 58 sec and counting
JUnit Tests
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kwayland%20kf5-qt5%20FreeBSDQt5.9/9/
Project:
Frameworks kwayland kf5-qt5 FreeBSDQt5.9
Date of build:
Tue, 19 Dec 2017 15:56:36 +
Build duration:
6 min 44 sec and counting
JUnit Tests
BUILD SUCCESS
Build URL
https://build.kde.org/job/Frameworks%20kwayland%20kf5-qt5%20SUSEQt5.10/11/
Project:
Frameworks kwayland kf5-qt5 SUSEQt5.10
Date of build:
Tue, 19 Dec 2017 15:56:36 +
Build duration:
5 min 30 sec and counting
JUnit Tests
This revision was automatically updated to reflect the committed changes.
Closed by commit R127:b13560ceac59: Fix ASAN warning on TestXdgForeign
(authored by davidedmundson).
Restricted Application edited projects, added Plasma on Wayland; removed Plasma.
REPOSITORY
R127 KWayland
CHANGES
This revision was automatically updated to reflect the committed changes.
Closed by commit R127:9121e6897254: Scope exported/imported objects to the test
(authored by davidedmundson).
Restricted Application edited projects, added Plasma; removed Plasma on Wayland.
REPOSITORY
R127 KWayland
graesslin accepted this revision.
This revision is now accepted and ready to land.
Restricted Application edited projects, added Plasma on Wayland; removed Plasma.
REPOSITORY
R127 KWayland
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D9407
To: davidedmundson, #plasma,
graesslin accepted this revision.
This revision is now accepted and ready to land.
Restricted Application edited projects, added Plasma; removed Plasma on Wayland.
REPOSITORY
R127 KWayland
REVISION DETAIL
https://phabricator.kde.org/D9406
To: davidedmundson, #plasma, graesslin
Cc:
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kirigami%20kf5-qt5%20SUSEQt5.10/18/
Project:
Frameworks kirigami kf5-qt5 SUSEQt5.10
Date of build:
Tue, 19 Dec 2017 15:13:40 +
Build duration:
4 min 56 sec and counting
JUnit Tests
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kirigami%20kf5-qt5%20FreeBSDQt5.9/9/
Project:
Frameworks kirigami kf5-qt5 FreeBSDQt5.9
Date of build:
Tue, 19 Dec 2017 15:13:40 +
Build duration:
1 min 9 sec and counting
JUnit Tests
broulik added a comment.
> What should be the move here?
What do you mean?
REPOSITORY
R242 Plasma Framework (Library)
REVISION DETAIL
https://phabricator.kde.org/D9324
To: broulik, #plasma, #vdg, ngraham
Cc: abetts, plasma-devel, #frameworks, ZrenBot, progwolff, lesliezhai,
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kservice%20kf5-qt5%20SUSEQt5.7/8/
Project:
Frameworks kservice kf5-qt5 SUSEQt5.7
Date of build:
Tue, 19 Dec 2017 12:58:15 +
Build duration:
12 min and counting
JUnit Tests
Name:
BUILD FAILURE
Build URL
https://build.kde.org/job/Frameworks%20kservice%20kf5-qt5%20SUSEQt5.10/8/
Project:
Frameworks kservice kf5-qt5 SUSEQt5.10
Date of build:
Tue, 19 Dec 2017 12:58:15 +
Build duration:
12 min and counting
JUnit Tests
Name:
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kservice%20kf5-qt5%20FreeBSDQt5.9/5/
Project:
Frameworks kservice kf5-qt5 FreeBSDQt5.9
Date of build:
Tue, 19 Dec 2017 12:58:15 +
Build duration:
3 min 23 sec and counting
JUnit Tests
This revision was automatically updated to reflect the committed changes.
Closed by commit R320:5e6c6fe0c7a1: Fix TagLib detection and build on Windows
(authored by kfunk).
REPOSITORY
R320 KIO Extras
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D9398?vs=24075=24104
REVISION DETAIL
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20ktexteditor%20kf5-qt5%20WindowsMSVCQt5.9/66/
Project:
Frameworks ktexteditor kf5-qt5 WindowsMSVCQt5.9
Date of build:
Tue, 19 Dec 2017 11:26:55 +
Build duration:
1 hr 3 min and counting
apol updated this revision to Diff 24103.
apol added a comment.
One change at a time
REPOSITORY
R241 KIO
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D9399?vs=24102=24103
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D9399
AFFECTED FILES
apol updated this revision to Diff 24102.
apol added a comment.
Use QIODevice::ReadWrite
REPOSITORY
R241 KIO
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D9399?vs=24100=24102
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D9399
AFFECTED FILES
kfunk created this revision.
Restricted Application added a project: Frameworks.
Restricted Application added a subscriber: Frameworks.
REVISION SUMMARY
Also prefer kde_target_enable_exceptions() over the function enabling
exceptions globally.
REPOSITORY
R286 KFileMetaData
BRANCH
master
dfaure added a comment.
I wonder if the bug is inside QIODevice::skip, which shouldn't CHECK_READABLE
in all cases (only when it's going to fall back to reading).
Alternatively we could open the device in ReadWrite mode, no? That might make
more sense, for a device that already has some
dfaure added a comment.
How does that remove the warning from QIODevice::skip, which we're now
calling directly instead of indirectly? I don't get it.
REPOSITORY
R241 KIO
REVISION DETAIL
https://phabricator.kde.org/D9399
To: apol, #frameworks, dfaure
Cc: anthonyfieroni, ngraham
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kirigami%20kf5-qt5%20FreeBSDQt5.9/8/
Project:
Frameworks kirigami kf5-qt5 FreeBSDQt5.9
Date of build:
Tue, 19 Dec 2017 11:40:45 +
Build duration:
3 min 46 sec and counting
JUnit Tests
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kirigami%20kf5-qt5%20SUSEQt5.10/17/
Project:
Frameworks kirigami kf5-qt5 SUSEQt5.10
Date of build:
Tue, 19 Dec 2017 11:40:45 +
Build duration:
2 min 31 sec and counting
JUnit Tests
BUILD SUCCESS
Build URL
https://build.kde.org/job/Frameworks%20ktexteditor%20kf5-qt5%20FreeBSDQt5.9/8/
Project:
Frameworks ktexteditor kf5-qt5 FreeBSDQt5.9
Date of build:
Tue, 19 Dec 2017 11:26:55 +
Build duration:
14 min and counting
JUnit Tests
dfaure accepted this revision.
dfaure added a comment.
This indeed seems more logical than the previous version of the patch. Thanks.
REPOSITORY
R241 KIO
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D9190
To: aacid, dfaure, apol, albertvaka
Cc: ngraham, nicolasfella,
apol updated this revision to Diff 24100.
apol added a comment.
Use QIODevice::skip instead
REPOSITORY
R241 KIO
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D9399?vs=24077=24100
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D9399
AFFECTED FILES
apol accepted this revision.
apol added a comment.
This revision is now accepted and ready to land.
LGTM
REPOSITORY
R241 KIO
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D9190
To: aacid, dfaure, apol, albertvaka
Cc: ngraham, nicolasfella, anthonyfieroni, #frameworks
davidedmundson created this revision.
davidedmundson added a reviewer: Plasma.
Restricted Application added projects: Plasma on Wayland, Frameworks.
Restricted Application added subscribers: Frameworks, plasma-devel.
REVISION SUMMARY
Internally XdgForeign has some delete laters
If we clean
davidedmundson created this revision.
Restricted Application added subscribers: Frameworks, plasma-devel.
Restricted Application added projects: Plasma on Wayland, Frameworks.
REVISION SUMMARY
m_imported was parented to the TestForeign class, which lasts for all
the tests. This caused an
davidedmundson added a reviewer: Plasma.
Restricted Application edited projects, added Plasma; removed Plasma on Wayland.
REPOSITORY
R127 KWayland
REVISION DETAIL
https://phabricator.kde.org/D9407
To: davidedmundson, #plasma
Cc: plasma-devel, #frameworks, ZrenBot, progwolff, lesliezhai,
broulik created this revision.
broulik added a reviewer: dfaure.
Restricted Application added a project: Frameworks.
Restricted Application added a subscriber: Frameworks.
REVISION SUMMARY
In https://phabricator.kde.org/R241:6246cc48067845208cf5acd8b798abd68349cf18
this code was ported away
BUILD SUCCESS
Build URL
https://build.kde.org/job/Frameworks%20kauth%20kf5-qt5%20WindowsMSVCQt5.9/17/
Project:
Frameworks kauth kf5-qt5 WindowsMSVCQt5.9
Date of build:
Tue, 19 Dec 2017 10:14:55 +
Build duration:
3 min 59 sec and counting
JUnit Tests
mlaurent added a comment.
Not commited ?
REPOSITORY
R241 KIO
REVISION DETAIL
https://phabricator.kde.org/D9332
To: renatoo, mwolff, mlaurent
Cc: mlaurent, aacid, ngraham, #frameworks
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kwayland%20kf5-qt5%20FreeBSDQt5.9/8/
Project:
Frameworks kwayland kf5-qt5 FreeBSDQt5.9
Date of build:
Tue, 19 Dec 2017 08:10:35 +
Build duration:
16 min and counting
JUnit Tests
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kwayland%20kf5-qt5%20SUSEQt5.10/10/
Project:
Frameworks kwayland kf5-qt5 SUSEQt5.10
Date of build:
Tue, 19 Dec 2017 08:10:35 +
Build duration:
14 min and counting
JUnit Tests
Name:
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kwayland%20kf5-qt5%20SUSEQt5.7/10/
Project:
Frameworks kwayland kf5-qt5 SUSEQt5.7
Date of build:
Tue, 19 Dec 2017 08:10:35 +
Build duration:
6 min 3 sec and counting
JUnit Tests
71 matches
Mail list logo