Re: Triggering rebuild after changing a *.json file

2014-11-23 Thread Volker Krause
On Sunday 23 November 2014 04:36:30 Milian Wolff wrote: in my quest for better *.json support in KF5 based applications, I noticed that we currently do not rebuild properly on changes to the *.desktop or *.json files. indeed, turns out I have the same problem in GammaRay. For KDevelop, I'm

Re: Triggering rebuild after changing a *.json file

2014-11-23 Thread Andreas Pakulat
Hi, On Sun, Nov 23, 2014 at 4:36 AM, Milian Wolff m...@milianw.de wrote: Hey all, in my quest for better *.json support in KF5 based applications, I noticed that we currently do not rebuild properly on changes to the *.desktop or *.json files. For KDevelop, I'm thus playing around with

Re: Review Request 120154: Equal entries implies existing directory

2014-11-23 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120154/#review70819 --- Ship it! Ship It! - David Faure On Sept. 12, 2014, 7:25

Re: Review Request 120154: Equal entries implies existing directory

2014-11-23 Thread Jan-Marek Glogowski
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120154/ --- (Updated Nov. 23, 2014, 4:36 p.m.) Status -- This change has been

Re: Review Request 120154: Equal entries implies existing directory

2014-11-23 Thread Luigi Toscano
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120154/#review70820 --- Which branch has the patch been submitted to? - Luigi

Re: Triggering rebuild after changing a *.json file

2014-11-23 Thread Milian Wolff
On Sunday 23 November 2014 10:39:02 Andreas Pakulat wrote: Hi, On Sun, Nov 23, 2014 at 4:36 AM, Milian Wolff m...@milianw.de wrote: Hey all, in my quest for better *.json support in KF5 based applications, I noticed that we currently do not rebuild properly on changes to the *.desktop

Re: Review Request 120154: Equal entries implies existing directory

2014-11-23 Thread David Faure
On Nov. 23, 2014, 4:46 p.m., Luigi Toscano wrote: Which branch has the patch been submitted to? I was just cleaning up something that had already been committed. dd160dee7c3f68aa49ad1868d87db61d69d10334 in kio master bf0a8b445afe6caed0e89f7fcf5cda7d63d2ccf8 in kdelibs KDE/4.14 - David

Re: Triggering rebuild after changing a *.json file

2014-11-23 Thread Kevin Funk
On Sunday 23 November 2014 17:54:21 Milian Wolff wrote: On Sunday 23 November 2014 10:39:02 Andreas Pakulat wrote: Hi, On Sun, Nov 23, 2014 at 4:36 AM, Milian Wolff m...@milianw.de wrote: Hey all, in my quest for better *.json support in KF5 based applications, I noticed

Re: Triggering rebuild after changing a *.json file

2014-11-23 Thread Milian Wolff
On Sunday 23 November 2014 23:37:45 Kevin Funk wrote: On Sunday 23 November 2014 17:54:21 Milian Wolff wrote: On Sunday 23 November 2014 10:39:02 Andreas Pakulat wrote: Hi, On Sun, Nov 23, 2014 at 4:36 AM, Milian Wolff m...@milianw.de wrote: Hey all, in my quest for

Re: Triggering rebuild after changing a *.json file

2014-11-23 Thread Kevin Funk
On Sunday 23 November 2014 23:37:45 Kevin Funk wrote: On Sunday 23 November 2014 17:54:21 Milian Wolff wrote: On Sunday 23 November 2014 10:39:02 Andreas Pakulat wrote: Hi, On Sun, Nov 23, 2014 at 4:36 AM, Milian Wolff m...@milianw.de wrote: Hey all, in my quest for

ThreadWeaver: using QObject's as Jobs

2014-11-23 Thread Milian Wolff
Hey All, esp. Mirko. Could you please document, e.g. by adding another example, on how one is supposed to use Job classes that are QObjects, i.e. have signals/slots? There is this strange QObjectDecorator thingy which I don't quite get, it looks like its supposed to add some signals to another

Re: Review Request 121083: Replace manual export files with CMake's generate_export_header

2014-11-23 Thread Hugo Pereira Da Costa
On Nov. 22, 2014, 12:18 a.m., Harald Sitter wrote: This breaks the kde4 build as ECM is not being used there and generate_export_header is not available without ECM. Andrius da Costa Ribas wrote: generate_export_header isn't in ECM, but in cmake itself