BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20ktexteditor%20kf5-qt5%20FreeBSDQt5.9/23/
Project:
Frameworks ktexteditor kf5-qt5 FreeBSDQt5.9
Date of build:
Sun, 04 Feb 2018 04:15:55 +
Build duration:
39 min and counting
JUnit Tests
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20plasma-framework%20kf5-qt5%20FreeBSDQt5.9/34/
Project:
Frameworks plasma-framework kf5-qt5 FreeBSDQt5.9
Date of build:
Sun, 04 Feb 2018 04:16:09 +
Build duration:
36 min and counting
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kio%20kf5-qt5%20SUSEQt5.7/111/
Project:
Frameworks kio kf5-qt5 SUSEQt5.7
Date of build:
Sun, 04 Feb 2018 04:15:36 +
Build duration:
35 min and counting
JUnit Tests
Name: (root)
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20ktexteditor%20kf5-qt5%20SUSEQt5.10/37/
Project:
Frameworks ktexteditor kf5-qt5 SUSEQt5.10
Date of build:
Sun, 04 Feb 2018 04:15:55 +
Build duration:
29 min and counting
JUnit Tests
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20krunner%20kf5-qt5%20SUSEQt5.7/15/
Project:
Frameworks krunner kf5-qt5 SUSEQt5.7
Date of build:
Sun, 04 Feb 2018 04:15:46 +
Build duration:
23 min and counting
JUnit Tests
Name:
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20krunner%20kf5-qt5%20FreeBSDQt5.9/7/
Project:
Frameworks krunner kf5-qt5 FreeBSDQt5.9
Date of build:
Sun, 04 Feb 2018 04:15:46 +
Build duration:
23 min and counting
JUnit Tests
Name:
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20plasma-framework%20kf5-qt5%20SUSEQt5.7/61/
Project:
Frameworks plasma-framework kf5-qt5 SUSEQt5.7
Date of build:
Sun, 04 Feb 2018 04:16:09 +
Build duration:
21 min and counting
JUnit
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kio%20kf5-qt5%20FreeBSDQt5.9/92/
Project:
Frameworks kio kf5-qt5 FreeBSDQt5.9
Date of build:
Sun, 04 Feb 2018 04:15:36 +
Build duration:
20 min and counting
JUnit Tests
Name: (root)
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20ktexteditor%20kf5-qt5%20SUSEQt5.7/38/
Project:
Frameworks ktexteditor kf5-qt5 SUSEQt5.7
Date of build:
Sun, 04 Feb 2018 04:15:55 +
Build duration:
18 min and counting
JUnit Tests
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20plasma-framework%20kf5-qt5%20SUSEQt5.10/58/
Project:
Frameworks plasma-framework kf5-qt5 SUSEQt5.10
Date of build:
Sun, 04 Feb 2018 04:16:09 +
Build duration:
12 min and counting
JUnit
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20krunner%20kf5-qt5%20SUSEQt5.10/15/
Project:
Frameworks krunner kf5-qt5 SUSEQt5.10
Date of build:
Sun, 04 Feb 2018 04:15:46 +
Build duration:
3 min 42 sec and counting
JUnit Tests
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kcoreaddons%20kf5-qt5%20SUSEQt5.10/41/
Project:
Frameworks kcoreaddons kf5-qt5 SUSEQt5.10
Date of build:
Sun, 04 Feb 2018 02:18:50 +
Build duration:
5 min 15 sec and counting
JUnit Tests
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kcoreaddons%20kf5-qt5%20SUSEQt5.7/36/
Project:
Frameworks kcoreaddons kf5-qt5 SUSEQt5.7
Date of build:
Sun, 04 Feb 2018 02:18:50 +
Build duration:
4 min 25 sec and counting
JUnit Tests
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kcoreaddons%20kf5-qt5%20FreeBSDQt5.9/25/
Project:
Frameworks kcoreaddons kf5-qt5 FreeBSDQt5.9
Date of build:
Sun, 04 Feb 2018 02:18:51 +
Build duration:
3 min 51 sec and counting
JUnit
wesdawg edited the summary of this revision.
REPOSITORY
R216 Syntax Highlighting
REVISION DETAIL
https://phabricator.kde.org/D10286
To: wesdawg, #framework_syntax_highlighting
Cc: #frameworks, #framework_syntax_highlighting, michaelh, genethomas, ngraham,
cullmann, vkrause, dhaumann
wesdawg created this revision.
wesdawg added a reviewer: Framework: Syntax Highlighting.
wesdawg created this object with visibility "All Users".
wesdawg added a project: Framework: Syntax Highlighting.
Restricted Application added a project: Frameworks.
Restricted Application added a subscriber:
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kio%20kf5-qt5%20FreeBSDQt5.9/91/
Project:
Frameworks kio kf5-qt5 FreeBSDQt5.9
Date of build:
Sat, 03 Feb 2018 23:37:15 +
Build duration:
11 min and counting
JUnit Tests
Name: (root)
This revision was automatically updated to reflect the committed changes.
Closed by commit R241:68a832c9e2aa: Use the destination KFileItem object to
figure out of its writable instead of… (authored by markg).
REPOSITORY
R241 KIO
CHANGES SINCE LAST UPDATE
ngraham accepted this revision.
This revision is now accepted and ready to land.
REPOSITORY
R241 KIO
BRANCH
kfileitemproperties_optimization
REVISION DETAIL
https://phabricator.kde.org/D10282
To: markg, dfaure, ngraham
Cc: ngraham, #frameworks, michaelh
markg created this revision.
markg added a reviewer: dfaure.
Restricted Application added a project: Frameworks.
Restricted Application added a subscriber: Frameworks.
markg requested review of this revision.
REVISION SUMMARY
Use KFileItem::isWritable instead of the detour with
What refactoring? Just add a separate window for editiong annotations
and a small "Edit" button to the bottom row, that opens this separate
window. I had some work in progress in this direction, but currently
lacking time to finish it...
2018-02-03 23:15 GMT+03:00 Damir Porobic
I will get in touch with the kde e.v.
I just want to make clear that I wouldn't implement such system without
discussing it beforehand with the community.
This is just an open discussion I wanted to have to get feedback.
The platform I'm working on so far has nothing to be sorry about, legally.
El dissabte, 3 de febrer de 2018, a les 21:26:24 CET, Nicolás Alvarez va
escriure:
> 2018-02-03 17:07 GMT-03:00 Albert Astals Cid :
> > El dissabte, 3 de febrer de 2018, a les 18:07:27 CET, Camilo Higuita
> > Rodriguez>
> > va escriure:
> >> Hi,everyone
> >>
> >> I'd like to
Not only android, but for plasma mobile, and there's a lot more than just
take a screenshot of the whole application, for example:
Add text over a screenshot or stickers to...
Select a rectangle section instead of the whole screen
If a screenshot of a selected part is text in a image, save
Camilo Higuita Rodriguez ha scritto:
> Excuse mentioned jump into the conversation out of the blue.
>
> Something that would be really cool is to refactor spectacle or ksnip to make
> use of qqc2 with Kirigami pointing to plans mobile too.
> If that were the case and you feel interested I could
Excuse mentioned jump into the conversation out of the blue.
Something that would be really cool is to refactor spectacle or ksnip to
make use of qqc2 with Kirigami pointing to plans mobile too.
If that were the case and you feel interested I could jump in and help.
Camilo
On Feb 3, 2018 3:23
On February 3, 2018 at 1:19:33 PM, Camilo Higuita Rodriguez (
chigui...@unal.edu.co) wrote:
My case is the following, let's say that there's a feature for streaming
audio, and the users stream the song they are listening to, what would go
down in that case? Who's in legal troubles?
Also if I
Thanks so much for understanding! Spectacle currently has no formal
maintainer, but myself and a few others are serving this role in a de
facto capacity. The Spectacle Phabricator project
(https://phabricator.kde.org/project/view/78/) would be a good place to
coordinate. We would welcome a
My case is the following, let's say that there's a feature for streaming
audio, and the users stream the song they are listening to, what would go
down in that case? Who's in legal troubles?
Also if I were to send an mp3 over email to a few friends, where's the
illegal stand point there? The
On February 3, 2018 at 1:08:06 PM, Albert Astals Cid (aa...@kde.org) wrote:
El dissabte, 3 de febrer de 2018, a les 18:07:27 CET, Camilo Higuita
Rodriguez
va escriure:
> Hi,everyone
>
> I'd like to discuss something with the community, and maybe get some
legal
> input:
>
> As some of you might
I do see your point here and it indeed makes sense. I could invest some time
and add the annotation features (and eventually other stuff) to Spectacle. The
lack of those features is actually the reason why we're having this discussion,
so it would make sense to resolve them.
Does Spectacle have
Reproducing, publicly performing or redistributing a copyrighted work without a
license is illegal almost everywhere (China/Russia/etc. may be exceptions).
RIP The Pirate Pay, Megaupload, etc. Not that some folks haven't made a ton of
money doing it anyway until a crack-down.
Andreas
Hello Camilo,
Not a lawyer either and you would need confirmation from a legal expert, but
you questions are not hard:
> 1- I would like to either have the option to *stream live* the music an
> user is currently listening to to a group of friends. here the music file
> isn't being storaged in
El dissabte, 3 de febrer de 2018, a les 18:07:27 CET, Camilo Higuita Rodriguez
va escriure:
> Hi,everyone
>
> I'd like to discuss something with the community, and maybe get some legal
> input:
>
> As some of you might already know I'm working on a open online platform to
> share music
ngraham added reviewers: VDG, andreaska.
REPOSITORY
R266 Breeze Icons
REVISION DETAIL
https://phabricator.kde.org/D10279
To: astippich, #breeze, #vdg, andreaska
Cc: ngraham, #frameworks, michaelh
astippich added a comment.
It just adds a missing link to an already existing icon, is a screenshot
needed?
REPOSITORY
R266 Breeze Icons
REVISION DETAIL
https://phabricator.kde.org/D10279
To: astippich, #breeze
Cc: ngraham, #frameworks, michaelh
Am 2018-02-03 20:36, schrieb Nate Graham:
I know, my answer sucks. I really do hate to have to bring it up. And
I don't mean to throw cold water on your project or your enthusiasm!
KSnip looks cool.
The thing is, single-developer projects tend to have a predictable
lifecycle: lots of energy and
ngraham accepted this revision as: Baloo, ngraham.
This revision is now accepted and ready to land.
REPOSITORY
R293 Baloo
BRANCH
search (branched from master)
REVISION DETAIL
https://phabricator.kde.org/D10205
To: michaelh, #frameworks, #dolphin, #baloo, mgallien, ngraham
Cc:
ngraham added a comment.
Before-and-after screenshots?
REPOSITORY
R266 Breeze Icons
REVISION DETAIL
https://phabricator.kde.org/D10279
To: astippich, #breeze
Cc: ngraham, #frameworks, michaelh
astippich created this revision.
astippich added a reviewer: Breeze.
Restricted Application added a project: Frameworks.
Restricted Application added a subscriber: Frameworks.
astippich requested review of this revision.
REVISION SUMMARY
media-album-cover icon is missing for the 24px size,
Am 2018-02-03 18:07, schrieb Camilo Higuita Rodriguez:
Hi,everyone
I'd like to discuss something with the community, and maybe get some
legal input:
As some of you might already know I'm working on a open online
platform to share music information between users, such as public
playlists,
ngraham accepted this revision.
REPOSITORY
R824 Baloo Widgets
BRANCH
signalavailable (branched from master)
REVISION DETAIL
https://phabricator.kde.org/D10113
To: michaelh, elvisangelaccio, smithjd, vhanda, ngraham, #dolphin, #frameworks
Cc: dhaumann
I know, my answer sucks. I really do hate to have to bring it up. And I
don't mean to throw cold water on your project or your enthusiasm! KSnip
looks cool.
The thing is, single-developer projects tend to have a predictable
lifecycle: lots of energy and progress in the beginning, then around
wcpan updated this revision to Diff 26460.
wcpan added a comment.
Don't emit error signal before we tried all list commands.
Use a special error code to skip error reporting.
REPOSITORY
R241 KIO
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D9672?vs=24747=26460
BRANCH
wcpan created this revision.
Restricted Application added a project: Frameworks.
Restricted Application added a subscriber: Frameworks.
wcpan requested review of this revision.
REPOSITORY
R241 KIO
BRANCH
fix-ftp-list
REVISION DETAIL
https://phabricator.kde.org/D10278
AFFECTED FILES
Hey Camilo,
not an expert but I recommend you to ask a lawyer. Just think of Youtube:
There are different viewing restrictions depending on the country you are
living in. Maybe streaming audio is equal... You could also ask the
streaming service providers how to proceed...
Cheers,
Christian
Am
chinmoyr added a comment.
In https://phabricator.kde.org/D10273#200254, @thiago wrote:
> That doesn't make sense. There's QFile::encodeName in the code
I was talking talking about FdSender which is used only in kauth helper.
REPOSITORY
R241 KIO
REVISION DETAIL
thiago added a comment.
That doesn't make sense. There's QFile::encodeName in the code.
Is there a section of the library that is used in elevated privilege code,
but not all of it?
REPOSITORY
R241 KIO
REVISION DETAIL
https://phabricator.kde.org/D10273
To: chinmoyr, #frameworks
Yeah, I was kind of afraid that you would say that.
When I started to work on the project I hadn't thought about extending any
other projects, mostly due to my limited knowledge of Qt at that time. Now I
have invested a lot of time and effort into this project and it seems to be
working fine
elvisangelaccio accepted this revision as: elvisangelaccio.
This revision is now accepted and ready to land.
REPOSITORY
R824 Baloo Widgets
BRANCH
signalavailable (branched from master)
REVISION DETAIL
https://phabricator.kde.org/D10113
To: michaelh, elvisangelaccio, smithjd, vhanda,
chinmoyr added inline comments.
INLINE COMMENTS
> thiago wrote in fdsender.cpp:24
> The problem here is the API. Why is it using std::string in the first place?
The idea was to use std c++ and avoid qt throughout the class because the code
will be executed with elevated privileges.
REPOSITORY
thiago added a comment.
Looks good.
INLINE COMMENTS
> fdsender.cpp:24
>
> FdSender::FdSender(const std::string )
> : m_socketDes(-1)
The problem here is the API. Why is it using std::string in the first place?
REPOSITORY
R241 KIO
REVISION DETAIL
chinmoyr added inline comments.
INLINE COMMENTS
> thiago wrote in fdreceiver.cpp:88
> __APPLE__ is there. My question is about OpenBSD, NetBSD and DragonflyBSD.
> Are you breaking them? You may not know the answer, but you need to document
> the issue if you intentionally cause a Failure To
thiago added a comment.
The patch that I can see accomplishes what the description says it should do.
And I agree with the idea of the patch.
INLINE COMMENTS
> dfaure wrote in fdreceiver.cpp:88
> This is UNIX-only, but indeed OSX is missing. Does that support getpeereid?
__APPLE__ is
ngraham added a comment.
Needs a rebase on top of master, FWIW. Can't wait to have this!
REPOSITORY
R241 KIO
BRANCH
openwithdialog-filter-app-tree
REVISION DETAIL
https://phabricator.kde.org/D8056
To: simgunz, dfaure, #frameworks, #vdg, ngraham, rkflx
Cc: rkflx, romangg, fabianr,
chinmoyr updated this revision to Diff 26451.
chinmoyr added a comment.
Fixed buffer overflow.
REPOSITORY
R241 KIO
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D10273?vs=26445=26451
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D10273
AFFECTED FILES
I love the idea of annotations built into a screenshot tool! Our users
have been asking for this.
However, I hate to be that guy, but have you considered extending
Spectacle instead? Spectacle is mature, well-integrated into KDE Plasma,
has all the features highlighted on
ossi added inline comments.
INLINE COMMENTS
> sharefd_p.h:61
> +const size_t pathSize = finalPath.size();
> +if (pathSize > 5 && pathSize < sizeof(a.sun_path) - 1) {
> #ifdef __linux__
you now have a buffer overflow on linux. you need to split the conditional.
REPOSITORY
Hi,everyone
I'd like to discuss something with the community, and maybe get some legal
input:
As some of you might already know I'm working on a open online platform to
share music information between users, such as public playlists, comments
on tracks and on the playback progress like
chinmoyr edited the summary of this revision.
REPOSITORY
R241 KIO
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D9966
To: chinmoyr, #frameworks, thiago, dfaure, ossi
Cc: ngraham, fvogt, lbeltrame, dfaure, michaelh
chinmoyr updated this revision to Diff 26446.
chinmoyr added a comment.
Moved changes related to SocketAddress to https://phabricator.kde.org/D10273.
Changed #error to #warning. So even if an OS does not have getpeereid or
getsockopt compilation will still succeed but copying files will
chinmoyr updated this revision to Diff 26445.
chinmoyr added a comment.
Check for null byte at index 1 in case of returning address length on linux.
Use size of finalPath as pathSize.
REPOSITORY
R241 KIO
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D10273?vs=26443=26445
chinmoyr updated this revision to Diff 26443.
chinmoyr added a comment.
Fixed typos
REPOSITORY
R241 KIO
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D10273?vs=26441=26443
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D10273
AFFECTED FILES
chinmoyr updated this revision to Diff 26441.
chinmoyr added a comment.
Used QByteArray in FdSender
Undoed behavioral changes.
REPOSITORY
R241 KIO
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D10273?vs=26438=26441
BRANCH
master
REVISION DETAIL
ossi added inline comments.
INLINE COMMENTS
> fdsender.cpp:27
> {
> +SocketAddress addr(path.c_str());
> +if (!addr.address()) {
this actually constructs a qbytearray. that should be probably explicit.
a better approach would be moving the class' interface to qbytearray (or
actually
chinmoyr updated this revision to Diff 26438.
chinmoyr added a comment.
Corrected number of bytes to be copied.
Added unlink().
REPOSITORY
R241 KIO
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D10273?vs=26436=26438
BRANCH
master
REVISION DETAIL
ossi added inline comments.
INLINE COMMENTS
> chinmoyr wrote in sharefd_p.h:60
> I just feel like the job of SocketAddress should be to create the address
> structure and not perform any file operation. Removal of the socket file
> should be handled by file ioslave or FdReceiver.
that's a
chinmoyr marked 2 inline comments as not done.
chinmoyr added inline comments.
INLINE COMMENTS
> ossi wrote in sharefd_p.h:60
> you still need to address that *somehow* ;)
I just feel like the job of SocketAddress should be to create the address
structure and not perform any file operation.
ossi added inline comments.
INLINE COMMENTS
> sharefd_p.h:60
> -::strcpy(a.sun_path, finalPath.c_str());
> -::unlink(finalPath.c_str());
> -#endif
you still need to address that *somehow* ;)
> sharefd_p.h:61
> +if (pathSize > 0 && pathSize < sizeof(a.sun_path) - 1) {
>
chinmoyr updated this revision to Diff 26436.
chinmoyr marked 3 inline comments as done.
chinmoyr added a comment.
Added space on correct side of & and on either sides of binary operators.
Used memcpy.
Restored original position of data member.
REPOSITORY
R241 KIO
CHANGES SINCE LAST
ossi added inline comments.
INLINE COMMENTS
> fdreceiver.h:45
> +QString m_path;
> +QSocketNotifier *m_readNotifier;
> };
why are you moving the member?
it doesn't matter in this case, but generally it's better to have the bigger
members first, concentrating in particular on equal
chinmoyr created this revision.
chinmoyr added a reviewer: Frameworks.
Restricted Application added a project: Frameworks.
chinmoyr requested review of this revision.
REVISION SUMMARY
This patch changes SocketAddress class to create an address structure for
a pathname socket (on all
rkflx added a comment.
In https://phabricator.kde.org/D8056#200013, @simgunz wrote:
> I cannot push so someone needs to do it for me.
No problem, I can do it for you.
REPOSITORY
R241 KIO
BRANCH
openwithdialog-filter-app-tree
REVISION DETAIL
rkflx added a comment.
@spoorun You make it sound like the sidebar is the only way to access those
folders. That's not true: The file dialog opens in "Home" by default, where you
can directly click on Pictures and the other folders anyway.
Regarding not adapting the file dialog to
BUILD SUCCESS
Build URL
https://build.kde.org/job/Frameworks%20kwidgetsaddons%20kf5-qt5%20SUSEQt5.7/23/
Project:
Frameworks kwidgetsaddons kf5-qt5 SUSEQt5.7
Date of build:
Sat, 03 Feb 2018 13:06:20 +
Build duration:
3 min 45 sec and counting
JUnit
BUILD SUCCESS
Build URL
https://build.kde.org/job/Frameworks%20kwidgetsaddons%20kf5-qt5%20FreeBSDQt5.9/16/
Project:
Frameworks kwidgetsaddons kf5-qt5 FreeBSDQt5.9
Date of build:
Sat, 03 Feb 2018 13:04:43 +
Build duration:
3 min 23 sec and counting
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kwidgetsaddons%20kf5-qt5%20SUSEQt5.7/22/
Project:
Frameworks kwidgetsaddons kf5-qt5 SUSEQt5.7
Date of build:
Sat, 03 Feb 2018 12:59:07 +
Build duration:
7 min 11 sec and counting
JUnit
BUILD SUCCESS
Build URL
https://build.kde.org/job/Frameworks%20kwidgetsaddons%20kf5-qt5%20SUSEQt5.10/24/
Project:
Frameworks kwidgetsaddons kf5-qt5 SUSEQt5.10
Date of build:
Sat, 03 Feb 2018 13:03:07 +
Build duration:
2 min 23 sec and counting
JUnit
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kwidgetsaddons%20kf5-qt5%20FreeBSDQt5.9/15/
Project:
Frameworks kwidgetsaddons kf5-qt5 FreeBSDQt5.9
Date of build:
Sat, 03 Feb 2018 12:59:07 +
Build duration:
5 min 31 sec and counting
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kwidgetsaddons%20kf5-qt5%20SUSEQt5.10/23/
Project:
Frameworks kwidgetsaddons kf5-qt5 SUSEQt5.10
Date of build:
Sat, 03 Feb 2018 12:59:07 +
Build duration:
3 min 59 sec and counting
JUnit
dfaure added a comment.
Right, the port to QByteArray could be splitted out of this socket security
fix commit.
REPOSITORY
R241 KIO
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D9966
To: chinmoyr, #frameworks, thiago, dfaure, ossi
Cc: ngraham, fvogt, lbeltrame, dfaure,
The link that gives a hint on what needs to be done is under the "Incubated
Projects" section, I've expected it under "Incubation Process".
I've created a page based on the template and added a link to "Incubated
Projects" under "Candidates", don't know if I was supposed to do that or
someone
This revision was automatically updated to reflect the committed changes.
Closed by commit R241:32b3bbcd16fd: Use the KFileItemListProperties copy
constructor instead of the conversion from… (authored by markg).
REPOSITORY
R241 KIO
CHANGES SINCE LAST UPDATE
jtamate accepted this revision.
This revision is now accepted and ready to land.
REPOSITORY
R241 KIO
BRANCH
KFileItemListProperties_stage2
REVISION DETAIL
https://phabricator.kde.org/D10261
To: markg, dfaure, jtamate
Cc: #frameworks, michaelh, ngraham
jtamate accepted this revision.
jtamate added a comment.
Looks good (and symmetric).
Is it difficult to create a new test?
REPOSITORY
R244 KCoreAddons
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D10257
To: dfaure, mwolff, jtamate, markg
Cc: markg, ngraham,
ossi added a comment.
In https://phabricator.kde.org/D9966#199979, @dfaure wrote:
> Thiago: now it's ready for your review ;)
if he wants to look past the already pointed out non-atomicity. i for one
just refuse to look at this mess any further.
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kwidgetsaddons%20kf5-qt5%20SUSEQt5.7/21/
Project:
Frameworks kwidgetsaddons kf5-qt5 SUSEQt5.7
Date of build:
Sat, 03 Feb 2018 11:22:14 +
Build duration:
20 min and counting
JUnit Tests
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kdelibs4support%20kf5-qt5%20SUSEQt5.10/25/
Project:
Frameworks kdelibs4support kf5-qt5 SUSEQt5.10
Date of build:
Sat, 03 Feb 2018 11:12:59 +
Build duration:
28 min and counting
JUnit
simgunz added a comment.
Good!
I cannot push so someone needs to do it for me.
Soon, I'll have more free time, and I'll try to polish the code (remove fetch
more, use Qt 5.10 recursive filtering).
REPOSITORY
R241 KIO
BRANCH
openwithdialog-filter-app-tree
REVISION DETAIL
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kdelibs4support%20kf5-qt5%20SUSEQt5.7/26/
Project:
Frameworks kdelibs4support kf5-qt5 SUSEQt5.7
Date of build:
Sat, 03 Feb 2018 11:12:59 +
Build duration:
18 min and counting
JUnit Tests
dfaure accepted this revision.
dfaure added a comment.
This revision is now accepted and ready to land.
Thanks, looks ok to me now.
Well, all that code about incremental fetching (canFetchMore/fetchMore) only
to then fetch everything upfront, could be simplified to just create the full
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kwidgetsaddons%20kf5-qt5%20SUSEQt5.10/22/
Project:
Frameworks kwidgetsaddons kf5-qt5 SUSEQt5.10
Date of build:
Sat, 03 Feb 2018 11:22:14 +
Build duration:
3 min 8 sec and counting
JUnit
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kwidgetsaddons%20kf5-qt5%20FreeBSDQt5.9/14/
Project:
Frameworks kwidgetsaddons kf5-qt5 FreeBSDQt5.9
Date of build:
Sat, 03 Feb 2018 11:22:14 +
Build duration:
2 min 56 sec and counting
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20kdelibs4support%20kf5-qt5%20FreeBSDQt5.9/20/
Project:
Frameworks kdelibs4support kf5-qt5 FreeBSDQt5.9
Date of build:
Sat, 03 Feb 2018 11:12:59 +
Build duration:
8 min 34 sec and counting
dfaure added a comment.
Thanks, looks ok to me now.
Thiago: now it's ready for your review ;)
REPOSITORY
R241 KIO
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D9966
To: chinmoyr, #frameworks, thiago, dfaure, ossi
Cc: ngraham, fvogt, lbeltrame, dfaure, michaelh
chinmoyr updated this revision to Diff 26421.
chinmoyr added a comment.
1. Used QByteArray in SocketAddress.
2. Rearranged members of FdReceiver.
3. Removed changes not directly related to these fixes.
REPOSITORY
R241 KIO
CHANGES SINCE LAST UPDATE
chinmoyr reopened this revision.
This revision is now accepted and ready to land.
REPOSITORY
R241 KIO
REVISION DETAIL
https://phabricator.kde.org/D9966
To: chinmoyr, #frameworks, thiago, dfaure, ossi
Cc: ngraham, fvogt, lbeltrame, dfaure, michaelh
BUILD SUCCESS
Build URL
https://build.kde.org/job/Frameworks%20kio%20kf5-qt5%20FreeBSDQt5.9/89/
Project:
Frameworks kio kf5-qt5 FreeBSDQt5.9
Date of build:
Sat, 03 Feb 2018 10:37:13 +
Build duration:
12 min and counting
JUnit Tests
Name: (root)
BUILD SUCCESS
Build URL
https://build.kde.org/job/Frameworks%20kio%20kf5-qt5%20SUSEQt5.7/107/
Project:
Frameworks kio kf5-qt5 SUSEQt5.7
Date of build:
Sat, 03 Feb 2018 10:18:43 +
Build duration:
20 min and counting
JUnit Tests
Name: (root)
This revision was automatically updated to reflect the committed changes.
Closed by commit R241:27aa3764c0bf: Improve error handling in file ioslave
(authored by chinmoyr).
REPOSITORY
R241 KIO
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D10266?vs=26417=26420
REVISION DETAIL
1 - 100 of 118 matches
Mail list logo