Re: Solid/PowerManagement deprecation

2016-02-10 Thread Jan Grulich
e. [1] - https://quickgit.kde.org/?p=solid-power.git[1] Regards, Jan -- Jan Grulich Software Engineer, Desktop team Red Hat Czech On Wednesday 10 of February 2016 15:55:51 Aleix Pol wrote: > Hi, > I've been looking in the last days what applications and uses make our > soft

Re: Review Request 127825: networkmanager-qt: Don't complain so much about unhandled LastSeen property

2016-05-03 Thread Jan Grulich
that error then it means either LastSeen property was introduced in older NM version or it was backported in your distribution. I'll check that and update that NM_CHECK_VERSION accordingly. - Jan Grulich On Kvě. 3, 2016, 2:04 odp., Allen Wi

Re: Review Request 127825: networkmanager-qt: Don't complain so much about unhandled LastSeen property

2016-05-03 Thread Jan Grulich
> On Kvě. 3, 2016, 2:30 odp., Jan Grulich wrote: > > That doesn't make much sense to me, if you are getting that error then it > > means either LastSeen property was introduced in older NM version or it was > > backported in your distribution. I

Review Request 127843: Properly remove composed characters

2016-05-05 Thread Jan Grulich
r exactly by one composed character so I was trying to find a way to do it similarly. Diffs - src/document/katedocument.cpp 73778a1 src/view/kateview.h 08db0df src/view/kateview.cpp fda6b2d Diff: https://git.reviewboard.kde.org/r/127843/diff/ Testing -

Re: Review Request 127843: Properly remove composed characters

2016-05-05 Thread Jan Grulich
posted them here: https://paste.kde.org/piptzo584 - Jan Grulich On Kvě. 5, 2016, 10:05 dop., Jan Grulich wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.

Re: Review Request 127843: Properly remove composed characters

2016-05-09 Thread Jan Grulich
is is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127843/#review95257 --- On Kvě. 5, 2016, 10:53 dop., Jan Grulich wrote: > > ---

Re: Review Request 127843: Properly remove composed characters

2016-05-16 Thread Jan Grulich
pointer is possibly a nullptr? > > > > PS: KateViewInternal.cpp also has a KateWrappingCursor. A proper solution > > would be to expose this class so that proper text navigation is available > > everywhere (e.g. KTextEditor::ViewCursor, just like > > KTextEditor::Do

Re: Review Request 127843: Properly remove composed characters

2016-05-16 Thread Jan Grulich
pointer is possibly a nullptr? > > > > PS: KateViewInternal.cpp also has a KateWrappingCursor. A proper solution > > would be to expose this class so that proper text navigation is available > > everywhere (e.g. KTextEditor::ViewCursor, just like > > KTextEditor::Do

Re: Review Request 127843: Properly remove composed characters

2016-05-16 Thread Jan Grulich
pointer is possibly a nullptr? > > > > PS: KateViewInternal.cpp also has a KateWrappingCursor. A proper solution > > would be to expose this class so that proper text navigation is available > > everywhere (e.g. KTextEditor::ViewCursor, just like > > KTextEditor::Do

Broken ABI in networkmanager-qt

2016-05-17 Thread Jan Grulich
you think is a better option? Change review: https://git.reviewboard.kde.org/r/123642/ I'm really sorry for this problem and will be more careful next time when doing such changes. Thanks. Regards, Jan -- Jan Grulich Software Engineer, Deskto

libmm-qt/libnm-qt as KF5

2014-04-03 Thread Jan Grulich
y to change. I just took inspiration from some Tier1 frameworks and changed it accordingly. NOTE: plasma-nm is not ported to use framework branches, it is still using master branches and I don't want to port it yet. Thanks a lot. Cheers, Jan -- Jan Grulich Red Hat Cz

Re: libmm-qt/libnm-qt as KF5

2014-04-03 Thread Jan Grulich
gt; Plasma NM is already ported to KF5/Plasma2 and it is working perfectly, except a few things that needs to be polished or finished. I meant it is not ported to my latest changes in libmm-qt/libnm-qt (framework branches). Jan > On Thu, Apr 3, 2014 at 11:47 AM, Jan Grulich wrote: > > Hi

Re: libmm-qt/libnm-qt as KF5

2014-04-04 Thread Jan Grulich
reasons to not have those two in KF5, but the one you > > advance > > is clearly the wrong one. > > > > Regards. > > -- > > Kévin Ottens, http://ervin.ipsquad.net > > > > KDAB - proud supporter of KDE, http://www.kdab.com -- Jan Grulich Red Hat Czech, s.r.o jgrul...@redhat.com ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: libmm-qt/libnm-qt as KF5

2014-04-07 Thread Jan Grulich
> > > > You > > > > > can make that dependency optional but do not force it over everybody > > > like > > > you did in NMQt/MMQt's frameworks branches. > > > > > > Besides depending on KF5's cmake modules wha

Re: libmm-qt/libnm-qt as KF5

2014-04-07 Thread Jan Grulich
, Martin Gräßlin wrote: > > On Monday 07 April 2014 10:14:12 David Faure wrote: > > > On Monday 07 April 2014 09:47:43 Jan Grulich wrote: > > > > You are still talking about users, but I'm sure that 99% of them will > > > > install it from distro repositories

Re: libmm-qt/libnm-qt as KF5

2014-04-08 Thread Jan Grulich
ou'd better merge them in plasma-nm. > > Personally I doubt they're useful outside of plasma-nm though. But that's > your job as maintainer to pick a choice. > I'm convinced that they could be useful for others, i.e ModemMana

Review Request 119382: Port kio-mtp to KF5

2014-07-21 Thread Jan Grulich
py my photos from phone to my laptop. Thanks, Jan Grulich ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 119382: Port kio-mtp to KF5

2014-07-21 Thread Jan Grulich
My phone is visible in Dolphin and I was able to finally copy my photos from phone to my laptop. Thanks, Jan Grulich ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 119382: Port kio-mtp to KF5

2014-07-21 Thread Jan Grulich
My phone is visible in Dolphin and I was able to finally copy my photos from phone to my laptop. Thanks, Jan Grulich ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 119382: Port kio-mtp to KF5

2014-07-21 Thread Jan Grulich
119382/diff/ Testing --- My phone is visible in Dolphin and I was able to finally copy my photos from phone to my laptop. Thanks, Jan Grulich ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/l

Re: Review Request 119382: Port kio-mtp to KF5

2014-07-21 Thread Jan Grulich
s://git.reviewboard.kde.org/r/119382/#review62770 ----------- On Čec. 21, 2014, 11:02 dop., Jan Grulich wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/119382/ > -

Re: Review Request 119382: Port kio-mtp to KF5

2014-07-26 Thread Jan Grulich
119382/diff/ Testing --- My phone is visible in Dolphin and I was able to finally copy my photos from phone to my laptop. Thanks, Jan Grulich ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/l

Re: Review Request 119382: Port kio-mtp to KF5

2014-08-05 Thread Jan Grulich
d.kde.org/r/119382/diff/ Testing --- My phone is visible in Dolphin and I was able to finally copy my photos from phone to my laptop. Thanks, Jan Grulich ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.

Review Request 120598: Offer also overwrite option when copying multiple files and there is a conflict

2014-10-15 Thread Jan Grulich
://git.reviewboard.kde.org/r/120598/diff/ Testing --- I tested it in Dolphin and it seems to work. Thanks, Jan Grulich ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Review Request 120599: Properly construct QUrl for a temporary file which will be copied as a new file or edited

2014-10-15 Thread Jan Grulich
/diff/ Testing --- Tested in dolphin and now I'm able to create a new text file or link to an application. Thanks, Jan Grulich ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 120599: Properly construct QUrl for a temporary file which will be copied as a new file or edited

2014-10-15 Thread Jan Grulich
Diffs - src/filewidgets/knewfilemenu.cpp a24ac9c Diff: https://git.reviewboard.kde.org/r/120599/diff/ Testing --- Tested in dolphin and now I'm able to create a new text file or link to an application. Thanks, Jan Grulich ___ Kde-

Re: Review Request 120598: Offer also overwrite option when copying multiple files and there is a conflict

2014-10-15 Thread Jan Grulich
RenameDialog in Kio::CopyJob. Diffs - src/core/copyjob.cpp 9f1826a Diff: https://git.reviewboard.kde.org/r/120598/diff/ Testing --- I tested it in Dolphin and it seems to work. Thanks, Jan Grulich ___ Kde-frameworks-devel mailing list Kde

Re: Review Request 120598: Offer also overwrite option when copying multiple files and there is a conflict

2014-10-16 Thread Jan Grulich
option to RenameDialog in Kio::CopyJob. Diffs (updated) - src/core/copyjob.cpp 9f1826a Diff: https://git.reviewboard.kde.org/r/120598/diff/ Testing --- I tested it in Dolphin and it seems to work. Thanks, Jan Grulich ___ Kde

Re: Review Request 120598: Offer also overwrite option when copying multiple files and there is a conflict

2014-10-16 Thread Jan Grulich
RenameDialog in Kio::CopyJob. Diffs - src/core/copyjob.cpp 9f1826a Diff: https://git.reviewboard.kde.org/r/120598/diff/ Testing --- I tested it in Dolphin and it seems to work. Thanks, Jan Grulich ___ Kde-frameworks-devel mailing

Re: Review Request 120599: Properly construct QUrl for a temporary file which will be copied as a new file or edited

2014-10-16 Thread Jan Grulich
error. Diffs - src/filewidgets/knewfilemenu.cpp a24ac9c Diff: https://git.reviewboard.kde.org/r/120599/diff/ Testing --- Tested in dolphin and now I'm able to create a new text file or link to an application. Thanks, Jan Grulich _

Review Request 120606: Properly parse URL in KUrlNavigator

2014-10-16 Thread Jan Grulich
aded/files/2014/10/16/eda6fb7e-a75e-4c28-ac87-5d16a39a366d__kurlnavigator.png Thanks, Jan Grulich ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 120606: Properly parse URL in KUrlNavigator

2014-10-16 Thread Jan Grulich
e.html - Jan --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120606/#review68553 --- On Říj. 16, 2014, 1:56 odp., Jan

Re: Review Request 120606: Properly parse URL in KUrlNavigator

2014-10-17 Thread Jan Grulich
Url(path))? Also using QUrl::fromUserInput() seems to work correctly and I think it should't get a relative path, otherwise the path wouldn't work in KUrlNavigator if you would like to go back. - Jan --- This is an automaticall

Re: Review Request 120606: Properly parse URL in KUrlNavigator

2014-10-17 Thread Jan Grulich
diff/ Testing --- Tested in dolphin and works fine now. File Attachments Screenshot of the problem https://git.reviewboard.kde.org/media/uploaded/files/2014/10/16/eda6fb7e-a75e-4c28-ac87-5d16a39a366d__kurlnavigator.png Thank

Re: Review Request 120606: Properly parse URL in KUrlNavigator

2014-10-17 Thread Jan Grulich
ile Attachments Screenshot of the problem https://git.reviewboard.kde.org/media/uploaded/files/2014/10/16/eda6fb7e-a75e-4c28-ac87-5d16a39a366d__kurlnavigator.png Thanks, Jan Grulich ___ Kde-frameworks-devel mailing list Kde-frameworks-deve

Review Request 121090: Move kio-mtp to kio-extras

2014-11-10 Thread Jan Grulich
he.h PRE-CREATION mtp/devicecache.cpp PRE-CREATION mtp/filecache.h PRE-CREATION Diff: https://git.reviewboard.kde.org/r/121090/diff/ Testing --- I have tested copying files, creating and removing folders in Dolphin with my Nexus 4 and everything seems to work fine. Thanks, Jan

Re: Review Request 121090: Move kio-mtp to kio-extras

2014-11-13 Thread Jan Grulich
kio-extras with the rest of kioslaves. - Jan --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121090/#review70301 -----------

libnm-qt - a new KF5 framework for Tier 1?

2014-11-13 Thread Jan Grulich
.kde.org. Here is our library http://quickgit.kde.org/?p=libnm-qt.git Thanks a lot. Cheers, Jan -- Jan Grulich Red Hat Czech, s.r.o jgrul...@redhat.com ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailm

Re: Review Request 121090: Move kio-mtp to kio-extras

2014-11-19 Thread Jan Grulich
t kio's as well in kio-extras. Question is, is it > useful outside the workspace? Would a cross-platform application use it? > > It seems to me they would, I can see amarok requiring it even on windows > or gnome. > > Jan Grulich wrote: > To me it makes more sense

Re: libnm-qt - a new KF5 framework for Tier 1?

2014-11-19 Thread Jan Grulich
i understand the maintainers preferred to tie it to the nm > > releases and not to the frameworks releases. Has this changed? > > Oh man, that is what i get for email being weird and getting David answer > before Jan's email, so yes i guess this has changed :D > There is

Re: Where are my systray icons cont'd

2014-11-27 Thread Jan Grulich
ni-qt. I had the same problem on Fedora and installing sni-qt.i686 solved this problem. Cheers, Jan -- Jan Grulich Red Hat Czech, s.r.o jgrul...@redhat.com ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: libnm-qt - a new KF5 framework for Tier 1?

2014-12-02 Thread Jan Grulich
agation. Done, I moved all propertiesChanged() slots to private classes. See https://git.reviewboard.kde.org/r/121304/ Anything else? Jan -- Jan Grulich Red Hat Czech, s.r.o jgrul...@redhat.com ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 121090: Move kio-mtp to kio-extras

2014-12-08 Thread Jan Grulich
ting and removing folders in Dolphin with my Nexus 4 and everything seems to work fine. Thanks, Jan Grulich ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: libnm-qt - a new KF5 framework for Tier 1?

2014-12-08 Thread Jan Grulich
1] - https://community.kde.org/Frameworks/CreationGuidelines [2] - https://bugs.kde.org/describecomponents.cgi?product=frameworks-networkmanager-qt Thanks a lot. Regards, Jan -- Jan Grulich Red Hat Czech, s.r.o jgrul...@redhat.com ___ Kde-frameworks-dev

Re: Review Request 121090: Move kio-mtp to kio-extras

2014-12-10 Thread Jan Grulich
removing folders in Dolphin with my Nexus 4 and everything seems to work fine. Thanks, Jan Grulich ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Review Request 121471: Port KDirSelectDialog to QFileDialog

2014-12-12 Thread Jan Grulich
c Diff: https://git.reviewboard.kde.org/r/121471/diff/ Testing --- Thanks, Jan Grulich ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 121471: Port KDirSelectDialog to QFileDialog

2014-12-15 Thread Jan Grulich
atchextract.cpp 9480e99 kerfuffle/extractiondialog.h 5907403 kerfuffle/extractiondialog.cpp f602861 part/part.cpp 82d6f9c Diff: https://git.reviewboard.kde.org/r/121471/diff/ Testing --- Thanks, Jan Grulich ___ Kde-frameworks-devel m

Re: Review Request 121471: Port KDirSelectDialog to QFileDialog

2014-12-17 Thread Jan Grulich
act.cpp 9480e99 kerfuffle/extractiondialog.h 5907403 kerfuffle/extractiondialog.cpp f602861 part/part.cpp 82d6f9c Diff: https://git.reviewboard.kde.org/r/121471/diff/ Testing --- Thanks, Jan Grulich ___ Kde-frameworks-devel mailing list Kde-f

libnm-qt framework

2014-12-18 Thread Jan Grulich
-networkmanager-qt component in KDE bugzilla. I would like to thank to everyone who made this happen or contributed to libnm-qt in any way. Regards, Jan -- Jan Grulich Red Hat Czech, s.r.o jgrul...@redhat.com ___ Kde-frameworks-devel mailing list Kde-frameworks

Re: Review Request 121471: Port KDirSelectDialog to QFileDialog

2015-01-06 Thread Jan Grulich
il. To reply, visit: https://git.reviewboard.kde.org/r/121471/#review73086 ----------- On Pro. 17, 2014, 11:58 dop., Jan Grulich wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org

New framework: ModemManagerQt

2015-03-12 Thread Jan Grulich
https://projects.kde.org/projects/kde/workspace/libmm-qt Thanks a lot. Regards, Jan -- Jan Grulich Red Hat Czech, s.r.o jgrul...@redhat.com ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinf

Re: New framework: ModemManagerQt

2015-03-16 Thread Jan Grulich
& to params of ip4ConfigChanged, ip6ConfigChanged and > networkTimeZoneChanged? Done. > Cheers, > Albert > Thanks, is there anything else? Regards, Jan -- Jan Grulich Red Hat Czech, s.r.o jgrul...@redhat.com ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

libmm-qt moved to frameworks/modemmanager-qt

2015-03-31 Thread Jan Grulich
ng to keep Plasma 5.2.x you need to keep also libmm-qt in the version released together with it. Regards, Jan -- Jan Grulich Red Hat Czech, s.r.o jgrul...@redhat.com ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kd

Re: libmm-qt moved to frameworks/modemmanager-qt

2015-03-31 Thread Jan Grulich
onathan Riddell wrote: > Congratulations > > Would a recompile do or is the source incompatible with Plasmaßnm 5.2.2 > > Jonathan > > On Tue, Mar 31, 2015 at 01:15:30PM +0200, Jan Grulich wrote: > > Hi, > > > > we have finally moved libmm-qt library to fram

Re: libmm-qt moved to frameworks/modemmanager-qt

2015-03-31 Thread Jan Grulich
On Tuesday 31 of March 2015 21:55 Albert Astals Cid wrote: > El Dimarts, 31 de març de 2015, a les 13:15:30, Jan Grulich va escriure: > > Hi, > > > > we have finally moved libmm-qt library to frameworks and renamed it to > > modemmanager-qt. It's now going

Re: libmm-qt moved to frameworks/modemmanager-qt

2015-04-04 Thread Jan Grulich
On Saturday 04 of April 2015 10:51 David Faure wrote: > On Tuesday 31 March 2015 23:37:22 Albert Astals Cid wrote: > > El Dimarts, 31 de març de 2015, a les 22:32:10, Jan Grulich va escriure: > > > On Tuesday 31 of March 2015 21:55 Albert Astals Cid wrote: > > > > El

Re: Review Request 123349: NetworkManagerQt: comment out unneeded lines from example/CMakeLists.txt

2015-04-13 Thread Jan Grulich
isit: > https://git.reviewboard.kde.org/r/123349/ > --- > > (Updated Dub. 12, 2015, 7:57 odp.) > > > Review request for KDE Frameworks and Jan Grulich. > > > Repository: networkmanager-qt > > > Description >

Re: Review Request 123349: NetworkManagerQt: comment out unneeded lines from example/CMakeLists.txt

2015-04-13 Thread Jan Grulich
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123349/#review78887 --- Ship it! Ship It! - Jan Grulich On Dub. 12, 2015, 7:57

Re: Review Request 123349: NetworkManagerQt: comment out unneeded lines from example/CMakeLists.txt

2015-04-13 Thread Jan Grulich
pp... but ok IIRC it also shows that on api.kde.org > > Jan Grulich wrote: > +1 for removing, it should be same for all frameworks, so there is no > need to mention how to find the library. > > Lamarque Souza wrote: > In quick search I could not find those informatio

Review Request 123477: Add public dependency on NetworkManager

2015-04-23 Thread Jan Grulich
needed for ModemManagerQt. Diffs - CMakeLists.txt c9e3274 KF5NetworkManagerQtConfig.cmake.in cdabe8e Diff: https://git.reviewboard.kde.org/r/123477/diff/ Testing --- Thanks, Jan Grulich ___ Kde-frameworks-devel mailing list Kde-framew

Re: Review Request 123477: Add public dependency on NetworkManager

2015-04-23 Thread Jan Grulich
> On Dub. 23, 2015, 1:35 odp., Aleix Pol Gonzalez wrote: > > KF5NetworkManagerQtConfig.cmake.in, line 4 > > <https://git.reviewboard.kde.org/r/123477/diff/1/?file=362685#file362685line4> > > > > nitpick. weird 1 space indentation. Already spotted. On Dub.

Re: Review Request 123477: Add public dependency on NetworkManager

2015-04-23 Thread Jan Grulich
On Dub. 23, 2015, 1:35 odp., Jan Grulich wrote: > > And yes, this is required as long as NetworkManager headers are included by > > NetworkManagerQt headers. > > Jan Grulich wrote: > I just found out that installing FindNetworkManager.cmake into CMake > modules i

Re: Review Request 123477: Add public dependency on NetworkManager

2015-04-27 Thread Jan Grulich
xtra-cmake-modules. If this is how it should be done, then similar patch would be needed for ModemManagerQt. Diffs - CMakeLists.txt c9e3274 KF5NetworkManagerQtConfig.cmake.in cdabe8e Diff: https://git.reviewboard.kde.org/r/123477/diff/ Testing --- Thanks, J

Review Request 123532: Add find modules for ModemManager and NetworkManager

2015-04-27 Thread Jan Grulich
://git.reviewboard.kde.org/r/123532/diff/ Testing --- Thanks, Jan Grulich ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Review Request 123533: Add public dependency on NetworkManager

2015-04-27 Thread Jan Grulich
/FindNetworkManager.cmake ece8835 src/CMakeLists.txt e2be7d8 src/fakenetwork/CMakeLists.txt 97c2867 Diff: https://git.reviewboard.kde.org/r/123533/diff/ Testing --- Thanks, Jan Grulich ___ Kde-frameworks-devel mailing list Kde-frameworks

Review Request 123534: Add public dependency on ModemManager

2015-04-27 Thread Jan Grulich
/FindModemManager.cmake d522e93 src/CMakeLists.txt 29a7a63 src/fakemodem/CMakeLists.txt b6d4a90 Diff: https://git.reviewboard.kde.org/r/123534/diff/ Testing --- Thanks, Jan Grulich ___ Kde-frameworks-devel mailing list Kde-frameworks-devel

Re: Review Request 123532: Add find modules for ModemManager and NetworkManager

2015-04-29 Thread Jan Grulich
://git.reviewboard.kde.org/r/123532/#review79633 --- On Dub. 27, 2015, 2:38 odp., Jan Grulich wrote: > > --- > This is an automatically generated e-mail. To reply, visit

Re: Review Request 123532: Add find modules for ModemManager and NetworkManager

2015-04-29 Thread Jan Grulich
> On Dub. 28, 2015, 4:09 odp., Lamarque Souza wrote: > > It worked here, so it's a +1 from me. I just would like the > > extra-cmake-module maintainer to give the final "ship it". > > Jan Grulich wrote: > Hmm, I realized that this would break b

Re: Review Request 123532: Add find modules for ModemManager and NetworkManager

2015-04-30 Thread Jan Grulich
ledge of CMake is quite limited, so proper review would be needed :). Diffs (updated) - find-modules/FindModemManager.cmake PRE-CREATION find-modules/FindNetworkManager.cmake PRE-CREATION Diff: https://git.reviewboard.kde.org/r/123532/diff/ Testing --- Thanks, Jan Gr

Re: Review Request 123768: port to new ecm_add_test PROPERTIES argument

2015-05-13 Thread Jan Grulich
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123768/#review80283 --- Ship it! Ship It! - Jan Grulich On Kvě. 13, 2015, 9 dop

Re: Review Request 121471: Port KDirSelectDialog to QFileDialog

2015-05-17 Thread Jan Grulich
act.cpp 9480e99 kerfuffle/extractiondialog.h 5907403 kerfuffle/extractiondialog.cpp f602861 part/part.cpp 82d6f9c Diff: https://git.reviewboard.kde.org/r/121471/diff/ Testing --- Thanks, Jan Grulich ___ Kde-frameworks-devel mailing list

Re: Review Request 124208: Fallback for componentFriendlyForAction name resolving

2015-08-10 Thread Jan Grulich
but after it gets saved then everything is merged into one component called "System settings". This was working fine before this change. - Jan Grulich On Čec. 1, 2015, 7:49 dop., Martin Gräßlin wrote: > > --- > This is an

Re: network manager qt - how to get the passwords

2015-08-27 Thread Jan Grulich
ping me anytime on IRC on #solid channel. [1] - https://quickgit.kde.org/?p=plasma-nm.git[1] > Thanks > Heiko Regards, Jan -- Jan Grulich Software Engineer, Desktop team Red Hat Czech [1] https://quickgit.kde.org/?p=plasma-nm.git ___ K

Re: set passwords only if changed

2015-09-04 Thread Jan Grulich
hen NM loads them from it's storage and secret agent is not used at all. > Thank in advance for your help. > > Best Regards > Heiko I hope that helps. Regards, Jan -- Jan Grulich Software Engineer, Desktop team Red Hat Czech ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Review Request 125289: Use Q_SLOTS/Q_SIGNALS instead of slots/signals in all headers from include dir

2015-09-17 Thread Jan Grulich
clude/QtCrypto/qca_safetimer.h 68dde5c Diff: https://git.reviewboard.kde.org/r/125289/diff/ Testing --- Thanks, Jan Grulich ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 125289: Use Q_SLOTS/Q_SIGNALS instead of slots/signals in all headers from include dir

2015-09-17 Thread Jan Grulich
marked as submitted. Review request for KDE Frameworks and Ivan Romanov. Changes --- Submitted with commit 66b9754170759d9333d5fc1e348642814d0310dd by Jan Grulich to branch master. Repository: qca Description --- This is used also in other header files from include folder. I&#

Re: qca-qt5 (qt5 branch) merge into qca (master branch)

2015-09-24 Thread Jan Grulich
27;s qt5 branch into master and > replacing the qt5 tarball with a new release that supports both qt4 > and qt5? > + 1, we added qca as dependency for plasma-nm and to be able to build it you need the latest fix I pushed to qca thus new release of qca before Plasma 5.5.0 rel

Re: Review Request 127843: Properly remove composed characters

2016-05-25 Thread Jan Grulich
nately time for that. - Jan --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127843/#review95764 --- On Kvě. 5, 2016,

Re: Review Request 127843: Properly remove composed characters

2016-05-26 Thread Jan Grulich
s/src/katedocument_test.h 8abbad9 autotests/src/katedocument_test.cpp dd41e0f src/document/katedocument.cpp 73778a1 src/view/kateview.h 08db0df src/view/kateview.cpp fda6b2d Diff: https://git.reviewboard.kde.org/r/127843/diff/ Testing -

Re: Review Request 127843: Properly remove composed characters

2016-05-26 Thread Jan Grulich
p dd41e0f src/document/katedocument.cpp 73778a1 src/view/kateview.h 08db0df src/view/kateview.cpp fda6b2d Diff: https://git.reviewboard.kde.org/r/127843/diff/ Testing --- Thanks, Jan Grulich ___ Kde-frameworks-devel mailing list Kde-frame

Re: Review Request 127843: Properly remove composed characters

2016-05-26 Thread Jan Grulich
marked as submitted. Review request for Kate, KDE Frameworks, Christoph Cullmann, and Dominik Haumann. Changes --- Submitted with commit c7096f60bbe094ba92456fd39ff3102435d9c677 by Jan Grulich to branch master. Repository: ktexteditor Description --- When using Indic locales

Review Request 128843: Use standard org.freedesktop.DBus.Properties interfaces for monitoring PropertiesChanged signal for NM 1.4.0+

2016-09-06 Thread Jan Grulich
src/vlandevice.cpp 415dec9 src/vpnconnection.cpp d2d17e4 src/vpnconnection_p.h 25ba589 src/wimaxdevice.cpp daec0a6 src/wireddevice.cpp af147b1 src/wirelessdevice.cpp dad918d Diff: https://git.reviewboard.kde.org/r/128843/diff/ Testing --- Thanks, Jan Grulich

Re: Review Request 128843: Use standard org.freedesktop.DBus.Properties interfaces for monitoring PropertiesChanged signal for NM 1.4.0+

2016-09-06 Thread Jan Grulich
src/vpnconnection_p.h 25ba589 src/wimaxdevice.cpp daec0a6 src/wireddevice.cpp af147b1 src/wirelessdevice.cpp dad918d Diff: https://git.reviewboard.kde.org/r/128843/diff/ Testing --- Thanks, Jan Grulich

Re: Review Request 128843: Use standard org.freedesktop.DBus.Properties interfaces for monitoring PropertiesChanged signal for NM 1.4.0+

2016-09-06 Thread Jan Grulich
these changes locally. I was messing with them when trying to find out why my active connections don't get notified about changed state. - Jan Grulich On Zář. 6, 2016, 11:56 dop., Jan Grulich wrote: > > --- > This is an

Re: Review Request 128843: Use standard org.freedesktop.DBus.Properties interfaces for monitoring PropertiesChanged signal for NM 1.4.0+

2016-09-06 Thread Jan Grulich
rtiesChanged signal. - Jan --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128843/#review98930 --- On Zář. 6, 2016, 11:56 dop.

Re: Review Request 128843: Use standard org.freedesktop.DBus.Properties interfaces for monitoring PropertiesChanged signal for NM 1.4.0+

2016-09-06 Thread Jan Grulich
> On Zář. 6, 2016, 12:01 odp., Lamarque Souza wrote: > > > > Jan Grulich wrote: > Every DeviceType inherits from Device, same as DeviceTypePrivate inherits > from DevicePrivate so when DeviceType is connected to PropertiesChanged > signal and doesn't imple

Re: Review Request 128843: Use standard org.freedesktop.DBus.Properties interfaces for monitoring PropertiesChanged signal for NM 1.4.0+

2016-09-06 Thread Jan Grulich
marked as submitted. Review request for KDE Frameworks, Network Management and Lamarque Souza. Changes --- Submitted with commit 7671424efb02e7194a18a4913da16bc2044a4cb9 by Jan Grulich to branch master. Repository: networkmanager-qt Description --- Since NM 1.2.x they switched to

Re: Review Request 129042: Replace all blocking calls when initializing properties by just one blocking call to get them all

2016-09-27 Thread Jan Grulich
device.cpp 6118f2f src/wireddevice.cpp 66bb649 src/wirelessdevice.cpp 3f2972b Diff: https://git.reviewboard.kde.org/r/129042/diff/ Testing --- Thanks, Jan Grulich

Re: Review Request 129042: Replace all blocking calls when initializing properties by just one blocking call to get them all

2016-09-27 Thread Jan Grulich
297 src/vpnconnection.cpp 5dee6fc src/vpnconnection_p.h a8195ad src/wimaxdevice.cpp 6118f2f src/wireddevice.cpp 66bb649 src/wirelessdevice.cpp 3f2972b Diff: https://git.reviewboard.kde.org/r/129042/diff/ Testing --- Thanks, Jan Grulich

Re: Review Request 129042: Replace all blocking calls when initializing properties by just one blocking call to get them all

2016-09-29 Thread Jan Grulich
device.cpp fdfa297 src/vpnconnection.cpp 5dee6fc src/vpnconnection_p.h a8195ad src/wimaxdevice.cpp 6118f2f src/wireddevice.cpp 66bb649 src/wirelessdevice.cpp 3f2972b Diff: https://git.reviewboard.kde.org/r/129042/diff/ Testing --- Thanks, Jan Grulich

Re: Review Request 129042: Replace all blocking calls when initializing properties by just one blocking call to get them all

2016-09-29 Thread Jan Grulich
t; &, const QVariant &) Unhandled property "S390Subchannels" > > ``` You can ignore that, that was there even before. For "AllDevices" and "Devices" properties we ignore them intentionally, because there is also "DeviceAdded" and "

Re: Review Request 129042: Replace all blocking calls when initializing properties by just one blocking call to get them all

2016-09-30 Thread Jan Grulich
marked as submitted. Review request for KDE Frameworks, Network Management, Aleix Pol Gonzalez, and Lamarque Souza. Changes --- Submitted with commit 00acf17a2071d86e218d3d49a0e53157eaa265dd by Jan Grulich to branch master. Repository: networkmanager-qt Description --- Uses

Re: Review Request 123533: Add public dependency on NetworkManager

2016-10-20 Thread Jan Grulich
t that properly. - Jan --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123533/#review100170 --- On Dub. 27, 2015, 2:42

Re: Review Request 129302: Fix include dir in pri file

2016-11-01 Thread Jan Grulich
, they are used in many places. - Jan Grulich On Lis. 1, 2016, 8:24 dop., David Rosca wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.

Re: Review Request 129303: Fix include dir in pri file

2016-11-01 Thread Jan Grulich
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129303/#review100460 --- Ship it! Ship It! - Jan Grulich On Lis. 1, 2016, 8:21

Review Request 129395: Add option to run an app on a discrete graphics card to KPropertiesDialog

2016-11-14 Thread Jan Grulich
/kpropertiesdialog.cpp 18c8479 src/widgets/krun.cpp d93b1f2 Diff: https://git.reviewboard.kde.org/r/129395/diff/ Testing --- Thanks, Jan Grulich

Review Request 129396: Allow to read X-KDE-RunOnDiscreteGpu property from desktop file + take this option into account when starting service using KToolInvocation

2016-11-14 Thread Jan Grulich
into account and set required variable accordingly. Diffs - src/kdeinit/ktoolinvocation.cpp c9eb922 src/services/kservice.h 66525b4 src/services/kservice.cpp 6574367 Diff: https://git.reviewboard.kde.org/r/129396/diff/ Testing --- Thanks, Jan Grulich

Review Request 129397: Set environment variables from KLaunchRequest when starting new process

2016-11-14 Thread Jan Grulich
- src/klauncher/klauncher.cpp 828038b Diff: https://git.reviewboard.kde.org/r/129397/diff/ Testing --- Thanks, Jan Grulich

Re: Review Request 129395: Add option to run an app on a discrete graphics card to KPropertiesDialog

2016-11-14 Thread Jan Grulich
https://git.reviewboard.kde.org/media/uploaded/files/2016/11/14/89203f95-9267-47db-b2ad-0eeb42357a1c__kpropertiesdialog.png Thanks, Jan Grulich

Review Request 129399: Take X-KDE-RunOnDiscreteGpu property into account when starting app using KRun

2016-11-14 Thread Jan Grulich
--- Thanks, Jan Grulich

  1   2   3   4   >