Re: QProcess::startDetached and bash stdout capture

2017-03-25 Thread Thiago Macieira
On sábado, 25 de março de 2017 09:03:28 PDT David Faure wrote: > On samedi 25 mars 2017 16:50:37 CET Thiago Macieira wrote: > > One problem is because the launched process inherits the pipes to stdout > > and stderr > > Really, even with startDetached? I assumed that was

Re: QProcess::startDetached and bash stdout capture

2017-03-25 Thread Thiago Macieira
On sábado, 25 de março de 2017 08:50:37 PDT Thiago Macieira wrote: > One problem is because the launched process inherits the pipes to stdout and > stderr, but the shell sees the process group it launched exiting. It will > probably close the pipe. Normally, this process would

Re: GSoC Project Proposal: Make High-DPI awesome

2017-03-23 Thread Thiago Macieira
since 5.6 ) -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center

Re: GSoC Project Proposal: Make High-DPI awesome

2017-03-23 Thread Thiago Macieira
On quarta-feira, 22 de março de 2017 00:58:20 PDT Jean-Baptiste Kempf wrote: > Hello, > > On Wed, 22 Mar 2017, at 08:03, Thiago Macieira wrote: > > problem with VLC, but I suspect it would be the exact same issue. > > The issue with VLC is that you want to have native

Re: GSoC Project Proposal: Make High-DPI awesome

2017-03-22 Thread Thiago Macieira
s to use configured > settings when starting a KDE workspace (Firefox? Google Chrome? GTK? > Java?) Set GDK_SCALE and CLUTTER_SCALE, that's all. I don't know which of those controls Firefox and Chrome, but they are properly scaled up in my system. I run 3200x1800 in my laptop

Re: We need to enable auto close in github pull requests

2017-03-22 Thread Thiago Macieira
ge them ? That's the ntire point: *no* one is able to merge GitHub Pull Requests in most of the repositories. That's by policy, since the submission should go through Phabricator instead. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center

Re: baloo Qt5.7 -> Qt5.8 oddities

2017-03-09 Thread Thiago Macieira
On segunda-feira, 6 de março de 2017 17:31:31 CET Andreas Müller wrote: > OK - if you followed me till this point: What is the proper fix for > qdbuscpp2xml working properly? Why are we discussing a fix for Qt in the kde-devel mailing list? -- Thiago Macieira - thiago (AT) macieir

Re: What's kde-core-devel for?

2016-12-16 Thread Thiago Macieira
parts maybe to > release-team (like adding new software to releases and such)? > > What do people that read this list think? This list is about discussions about changes that affect all applications. How about killing kde-frameworks-devel and bringing that discussion back here where it

[Differential] [Commented On] D2545: Cleanup KSharedUiServerProxy before qApp exists

2016-12-16 Thread thiago (Thiago Macieira)
thiago added a comment. In https://phabricator.kde.org/D2545#69187, @kfunk wrote: > In https://phabricator.kde.org/D2545#69091, @thiago wrote: > > > There doesn't seem to be a way of doing some clean up before the threads are forcibly killed. > > > > Maybe if I abuse qtmain().

[Differential] [Commented On] D2545: Cleanup KSharedUiServerProxy before qApp exists

2016-12-03 Thread thiago (Thiago Macieira)
thiago added a comment. In https://phabricator.kde.org/D2545#66904, @thiago wrote: > Commit ad66dbe305cff72443f4d3484191872d56e6dfbb in qtbase did try to solve this by disconnecting the senders when the objects were getting deleted. closeConnection() is called before the thread exits

[Differential] [Commented On] D2545: Cleanup KSharedUiServerProxy before qApp exists

2016-12-03 Thread thiago (Thiago Macieira)
thiago added a comment. In https://phabricator.kde.org/D2545#66545, @kfunk wrote: > In https://phabricator.kde.org/D2545#65976, @dfaure wrote: > > > Actually, I think Thiago's still waiting for a backtrace of all threads. > > > I think I've already said this via other channels:

Re: Test your applications on Wayland

2016-10-12 Thread Thiago Macieira
unusable. Should happen in the next couple of weeks, now that 5.6.2 is closed. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center

Re: Review Request 128909: initial, minimal support for OS X

2016-09-14 Thread Thiago Macieira
ect `argMax()`? It's all in the same .cpp, it doesn't matter. Inline is correct where it is. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center

Re: Review Request 128790: Call utempter manually

2016-09-04 Thread Thiago Macieira
> On Sept. 4, 2016, 2:13 p.m., Thiago Macieira wrote: > > What happens if one builds on a machine without utempter? Are we still linking to libutempter? - Thiago --- This is an automatically generated e-mail. To reply, vis

Re: License issue with AppImage file in Ark repo

2016-07-27 Thread Thiago Macieira
a KDE-controlled server. But then every dowstream redistributor of Ark would need to do the same. Conclusion: just include the sources. We don't need the actual tarball, much less the pre-generated autotools files. Add a copy of the actual source of GNU Hello, the one that the upstream developers usuall

Re: Generating QtScript bindings for Qt5

2016-07-25 Thread Thiago Macieira
On segunda-feira, 25 de julho de 2016 17:55:25 PDT Thiago Macieira wrote: > > What's the best fix/workaround to this ? Any suggestions ? > > Build it with Qt 4. The library it links to has nothing to do with the > input or output. Well, besides the fact that you shouldn't be

Re: Generating QtScript bindings for Qt5

2016-07-25 Thread Thiago Macieira
ary it links to has nothing to do with the input or output. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center

Re: Error while building mailcommon

2016-07-24 Thread Thiago Macieira
ete your KDE Libs 4 files. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center

Re: Error while building mailcommon

2016-07-24 Thread Thiago Macieira
tGui/QDialog: > No such file or directory QDialog is part of QtWidgets in Qt 5. You're mixing KF5 and kdelibs from KDE 4. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center

Re: [kde-community] Usage of QNetworkAccessManager

2016-07-14 Thread Thiago Macieira
> > Please therefore ensure your application handles redirects > appropriately (the form of the code will depend on the version of Qt > in use) if you decide to use QNAM. You do that by setting the attribute FollowRedirectsAttribute in your QNetworkRequests. -- Thiago Macieira - th

Re: [kde-community] Usage of QNetworkAccessManager

2016-07-14 Thread Thiago Macieira
> > Please therefore ensure your application handles redirects > appropriately (the form of the code will depend on the version of Qt > in use) if you decide to use QNAM. You do that by setting the attribute FollowRedirectsAttribute in your QNetworkRequests. -- Thiago Macieira - th

Re: Using Qt 5.7 as dep for master in build.k.o

2016-07-09 Thread Thiago Macieira
rsions, if they don't use private API. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center

Re: [kdesrc-build]How do I point to the latest Qt

2016-07-03 Thread Thiago Macieira
you need to create a configuration file for that self-built Qt first. For example, for me: $ cat ~/.config/qtchooser/5.conf /home/tjmaciei/obj/qt/qt5/qtbase/bin /home/tjmaciei/obj/qt/qt5/qtbase/lib -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center

Re: Why do we need to run two kwalletd?

2016-05-28 Thread Thiago Macieira
Em sábado, 28 de maio de 2016, às 16:14:23 BRT, Albert Astals Cid escreveu: > El dissabte, 28 de maig de 2016, a les 10:19:59 CEST, Thiago Macieira va > > escriure: > > Commit https://quickgit.kde.org/? > > p=kwallet.git=commit=f7c1586264e029be50f3d9ca15ccad268a8c5601 r

Why do we need to run two kwalletd?

2016-05-28 Thread Thiago Macieira
decision? Why not have *one* wallet for all programs? -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center

Re: Qt 4 Builds

2016-03-27 Thread Thiago Macieira
ollowed suit. Not completely EOL'ed yet. Security fixes are still applied for now, but no other changes. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center

Re: Policy regarding QtWebKit and QtScript

2016-03-07 Thread Thiago Macieira
I noticed something strange with one of my online web mail accounts I only access using rekonq (QtWebKit), so it may be that the engine is already past expiration date. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center

Re: KDE file dialog

2016-03-01 Thread Thiago Macieira
That way > everyone would be happy. Yes. Just volunteer to write that plugin. It may be even accepted as part of the Qt Project. The point is that it's not going to happen without developer effort. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Archite

Re: KDE file dialog

2016-02-29 Thread Thiago Macieira
sumes the desktop you're in actually provides this > functionality. Only if it's a Qt-based desktop. If it's another desktop, then it's Qt's job to integrate with it properly. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center

Re: KDE file dialog

2016-02-29 Thread Thiago Macieira
esktop. In that case, I'd argue that the Plasma plugin should be loaded, but it needs to be installed too. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center

Re: RCC for icons - update: Re: Icons installed by apps

2016-02-22 Thread Thiago Macieira
he .rcc file. > > We get all the same icons as on Linux neatly in one file Another way to load, instead of using QResource, is to put everything into one .so and simply use QLibrary to open it. Same effect, though this would create an architecture-specific resource, instead of sharing it across mult

Re: Deadlock in kded whith Qt 5.6

2016-01-22 Thread Thiago Macieira
On Friday 22 January 2016 09:21:12 David Faure wrote: > On Thursday 21 January 2016 13:54:32 Thiago Macieira wrote: > > Can I limit the spy hook to method calls? That is, exclude signals. > > Yes, for the purpose of kded/kiod that is fine > A not-loaded-yet-module canno

Re: Deadlock in kded whith Qt 5.6

2016-01-21 Thread Thiago Macieira
we should still be seeing the spy later. > and one log for the second call (kded already started, autoload works) > > http://www.davidfaure.fr/2016/second_call.txt This one shows "invoking message spies" -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org

Re: Deadlock in kded whith Qt 5.6

2016-01-21 Thread Thiago Macieira
On Thursday 21 January 2016 08:37:38 David Faure wrote: > On Wednesday 20 January 2016 08:41:31 Thiago Macieira wrote: > > On Wednesday 20 January 2016 09:08:38 David Faure wrote: > > > > Can you run with QDBUS_DEBUG=1 and tell me the messages it prints? > > > >

Re: Deadlock in kded whith Qt 5.6

2016-01-20 Thread Thiago Macieira
e attached patch so we also get feedback on the queued delivery? -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358; fingerprint: E067 918B B660 DBD1 105C 966C 33F5 F005 6EF

Re: Deadlock in kded whith Qt 5.6

2016-01-20 Thread Thiago Macieira
the spy hooks do get called and in the right (main) thread. And since the hooks are processed before the main dispatcher queue, it shouldn't affect the previous code. Can you run with QDBUS_DEBUG=1 and tell me the messages it prints? -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kd

Re: Deadlock in kded whith Qt 5.6

2016-01-19 Thread Thiago Macieira
nd at https://codereview.qt-project.org/146639 -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358; fingerprint: E067 918B B660 DBD1 105C 966C 33F5 F005 6EF4 5358 >From fafa4b3b89a02e136f

Re: Deadlock in kded whith Qt 5.6

2016-01-19 Thread Thiago Macieira
ve had lots of kded5 issues due to blocking calls over the years > but I don'r remember seeing module loading in any backtraces. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF4

Re: Policy regarding QtWebKit and QtScript

2015-12-26 Thread Thiago Macieira
o the underlying engine. I have no idea if the application > works (well). I'll kick off a build just before I leave for Xmas dinner. Some of those changes may be only to unbundle some libraries and cause chromium to link to system-provided ones. -- Thiago Macieira - thiago (AT) macieira.info -

Re: Policy regarding QtWebKit and QtScript

2015-12-23 Thread Thiago Macieira
re Qt releases and should get fixed in updated qtwebkit source packages. qtquick1 is a whole other story. Its life ends now. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358; fingerprint:

Re: Policy regarding QtWebKit and QtScript

2015-12-23 Thread Thiago Macieira
elease and that is simply too much to ask for, for a module that needs security updates we are hardly getting from upstream. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358; finge

Re: Why is C90 enforced in KDE?

2015-12-07 Thread Thiago Macieira
f. You need MSVC 2013 for that. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358; fingerprint: E067 918B B660 DBD1 105C 966C 33F5 F005 6EF4 5358

Re: Why is C90 enforced in KDE?

2015-12-07 Thread Thiago Macieira
blem is the C99 Library, which MSVC did not support until 2013 and it only did so because C++11 imported it by reference. Microsoft does not care for C. They didn't implement C99 because they didn't want to, not because their compiler team was unable to. So my recommendation: write C++ code. --

Re: KSyCoca, Thread safety, and Cache invalidation

2015-11-14 Thread Thiago Macieira
imeZone and we could read the system file, bypassing localtime_r and tzset. John would know more. > This tzset() issue is really awful, we'll never manage to make sure that > 100% of the code that needs to be threadsafe uses UTC everywhere. Hence the mutex. -- Thiago Macieira - thiago (AT) maciei

Re: Review Request 119011: KInit: call setgroups(0, 0) before calling setgid()

2015-11-10 Thread Thiago Macieira
(usermod), for groups like vboxusers and systemd-journal. Now, start_kdeinit unconditionally drops all groups and that's wrong. It should call getgrouplist(3) and set those groups on the user. Besides, I'm not convinced the rpmlint warning was correct. - Thiago Macieira On Julho 1, 2014, 10:21

Re: Debug ThreadWeaver::IdDecorator

2015-10-26 Thread Thiago Macieira
eadWeaver library in debug mode so we can see what the ?? are. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358; fingerprint: E067 918B B660 DBD1 105C 966C 33F5 F005 6EF4 5358 >

Re: DBus service for screenshots.

2015-09-24 Thread Thiago Macieira
ll likely not map to what you expose > right now. Emmanuele is right: please agree on the x...@lists.freedesktop.org what the interface will be before you release software depending on it. Proof of concepts are fine, but until you reach a stable interface, no one can depend on it. -- Thia

Re: Review Request 124856: Call newInstance from the child on first invocation

2015-09-23 Thread Thiago Macieira
, not 5.7. If you want to discuss suspending the execution of the thread, please start that discussion. I have the beginnings of a hackish patch for that... - Thiago Macieira On Set. 23, 2015, 7:44 p.m., Albert Astals Cid wrote

Re: [Development] Please help me get my pending review count down

2015-09-23 Thread Thiago Macieira
On Thursday 20 August 2015 17:16:40 Thiago Macieira wrote: > On Friday 21 August 2015 01:45:12 Albert Astals Cid wrote: > > So with your new patches and https://git.reviewboard.kde.org/r/124856/ i > > can run konsole fine both with --nofork and in the normal mode. > >

Re: Review Request 125220: KService: use a real global static for QThreadStorage rather than Q_GLOBAL_STATIC.

2015-09-13 Thread Thiago Macieira
in a Q_GLOBAL_STATIC. I don't understand what you meant by id: QThreadStorageData::get can only be called once the construction has finished, in which case there aren't multiple threads racing. - Thiago Macieira On Set. 13, 2015, 10:10 p.m., David Faure wrote

Re: Best-practise currently for testing internal parts of libs? *_TEST_EXPORT macro?

2015-08-31 Thread Thiago Macieira
tected visibility instead of default, this will be required. > # if defined(calligrasheetsodf_EXPORTS) > # define CALLIGRA_SHEETS_ODF_TEST_EXPORT KDE_EXPORT > # else > # define CALLIGRA_SHEETS_ODF_TEST_EXPORT KDE_IMPORT > # endif -- Thiago Macieira - thiago (AT) m

Re: Review Request 124856: Call newInstance from the child on first invocation

2015-08-21 Thread Thiago Macieira
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124856/#review84149 --- Makes sense! - Thiago Macieira On Ago. 21, 2015, 4:06 p.m

Re: Review Request 124856: Call newInstance from the child on first invocation

2015-08-21 Thread Thiago Macieira
://git.reviewboard.kde.org/r/124856/ --- (Updated Ago. 21, 2015, 4:06 p.m.) Review request for KDE Frameworks and Thiago Macieira. Repository: kdelibs4support Description --- For the first invocation, call newInstance

Re: [Development] Please help me get my pending review count down

2015-08-20 Thread Thiago Macieira
On Sunday 02 August 2015 10:32:21 Thiago Macieira wrote: ... which makes me wonder: did I fix the problem but never gave Albert the patches that fixed them? And the answer is... yes-kinda. I had fixed the issue, but never applied it to QtDBus. By replacing QThread with QDaemonThread

Re: Using nullptr instead of Q_NULLPTR

2015-08-18 Thread Thiago Macieira
people with old compilers by making the code SC. Back to anything that is not nullptr: 0, NULL, __null, Q_NULLPTR, whatever. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358; fingerprint

Re: Using nullptr instead of Q_NULLPTR

2015-08-14 Thread Thiago Macieira
is dangerous... My recommendation: it's broken now so leave it. Fix it if someone complains, but otherwise just leave it. In one year's time, the discussion will be moot. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center

Re: Using nullptr instead of Q_NULLPTR

2015-08-14 Thread Thiago Macieira
it. PS: A Krazy check checking for usage of NULLs and Q_NULLPTR would be lovely. Checking the use of 0's isn't easy, right? -Wzero-as-null-pointer-constant Qt headers should pass that starting with 5.6. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect

Re: Using nullptr instead of Q_NULLPTR

2015-08-14 Thread Thiago Macieira
tested and untested things there. I would have said the docs or the wiki somewhere, but I've just discovered that the docs are wrong... :-) -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358

Re: Using nullptr instead of Q_NULLPTR

2015-08-13 Thread Thiago Macieira
at that time. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358; fingerprint: E067 918B B660 DBD1 105C 966C 33F5 F005 6EF4 5358

Re: Using nullptr instead of Q_NULLPTR

2015-08-13 Thread Thiago Macieira
. The page is just plainly wrong. That's the list of platforms the Qt CI tests on. It's not the list of platforms we are supposed to compile with and support. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP

Re: [Development] Please help me get my pending review count down

2015-08-05 Thread Thiago Macieira
On Tuesday 28 July 2015 19:06:59 Thiago Macieira wrote: Deadline: 6 weeks before Qt 5.7 feature freeze. If we reach that date and the patches are still not integrated, I will have to make a decision as Maintainer as to the future of them. It will be to either apply them despite any

Re: Replacement for KDateTime

2015-08-04 Thread Thiago Macieira
that allowed the GUI platform plugin to set some details. Obviously this would not apply to non-GUI apps. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358; fingerprint: E067 918B B660 DBD1

Re: Replacement for KDateTime

2015-08-04 Thread Thiago Macieira
anything to show. It's the same problem and requires the same solution as the language itself. There are mechanisms, but they're neither very well-used and they cause thundering herd. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source

Re: Replacement for KDateTime

2015-08-04 Thread Thiago Macieira
to make /bin/ls work then. What's your suggestion to make this work for a KDE environment and applications, John? -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358; fingerprint: E067

Re: Replacement for KDateTime

2015-08-03 Thread Thiago Macieira
I'm going to accept is one that also applies to /bin/ls. So far, the idea of setting LC_TIME to a file containing the rules is the only solution I've seen to accomplish that. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology

Re: Replacement for KDateTime

2015-08-03 Thread Thiago Macieira
to the C API if the locale is a file or one not found in the database it carries. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358; fingerprint: E067 918B B660 DBD1 105C 966C 33F5 F005

Re: Replacement for KDateTime

2015-08-03 Thread Thiago Macieira
QDateTime somehow gets converted to time_t (out of range). -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358; fingerprint: E067 918B B660 DBD1 105C 966C 33F5 F005 6EF4 5358

Re: Replacement for KDateTime

2015-08-03 Thread Thiago Macieira
too. The roadmap currently stands as follows: Does this mean there is no roadmap? :) It means the roadmap is empty. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358; fingerprint

Re: Replacement for KDateTime

2015-08-02 Thread Thiago Macieira
for requires support from translators, please add a new class to QtCore. Alternatively, do you John have any roadmap about QLocale? Perhaps we could help with filling the missing bits into QLocale directly too. The roadmap currently stands as follows: -- Thiago Macieira - thiago

Re: [Development] Please help me get my pending review count down

2015-08-02 Thread Thiago Macieira
but never gave Albert the patches that fixed them? -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358; fingerprint: E067 918B B660 DBD1 105C 966C 33F5 F005 6EF4 5358

Re: [Development] Please help me get my pending review count down

2015-07-28 Thread Thiago Macieira
[cross-posting to k-c-d to get KDE dev attention] On Tuesday 28 July 2015 14:51:49 Thiago Macieira wrote: - QtDBus changes: https://codereview.qt-project.org/101967 https://codereview.qt-project.org/102762 https://codereview.qt-project.org/103731 https://codereview.qt-project.org/103732

Re: KSyCoca, Thread safety, and Cache invalidation

2015-07-20 Thread Thiago Macieira
On Monday 20 July 2015 16:05:06 David Faure wrote: Or maybe QDateTime::currentDateTime() could avoid calling the awful tzset()? Thiago, any input? QDateTime::currentDateTimeUtc() does not call tzset. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect

Re: KSyCoca, Thread safety, and Cache invalidation

2015-07-14 Thread Thiago Macieira
something to the user. There's no other valid reason. (writing to a log counts as showing to the user) -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358; fingerprint: E067 918B B660 DBD1 105C

Re: Frameworks compiler and Qt requirements after Qt 5.7?

2015-06-28 Thread Thiago Macieira
that the minimum Qt requires. If the minimum Qt is 5.6, then KDE's current requirements are higher than Qt's. So no change is necessary. If the minimum Qt is 5.7, then Qt's requirements are higher. Effectively, Qt's requirements become KDE's. -- Thiago Macieira - thiago (AT) macieira.info - thiago

Re: Frameworks compiler and Qt requirements after Qt 5.7?

2015-06-26 Thread Thiago Macieira
for C++14 support and MSVC 2015 does not follow it. Therefore, for Qt's purposes, MSVC 2015 supports *no* C++14. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358; fingerprint: E067 918B

Re: Review Request 123874: Fix build with Qt = 5.4.2

2015-05-21 Thread Thiago Macieira
083c9269ed73e8771e1dbe10812696b45b7389f3 and report back if it worked. If there are problems, let's fix them in Qt itself. If there are issues that can't be fixed in Qt itself, then kde-multimedia would be the wrong mailing list to discuss it. -- Thiago Macieira - thiago (AT) macieira.info - thiago

Re: Weird linkage failure on OSX CI

2015-05-20 Thread Thiago Macieira
here, since we would get similar failures everwhere we use std::string, right? Any idea what could be wrong? This obviously compiles just fine on Linux. Compiler bug. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology

Re: Is KF5::KCodecs::base64Encode now including a trailing newline?

2015-05-19 Thread Thiago Macieira
of the byte string, where the kdelibs4 version didn't. There's QByteArray::toBase64. You may want to use that instead. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358; fingerprint: E067 918B B660

Re: Is KF5::KCodecs::base64Encode now including a trailing newline?

2015-05-19 Thread Thiago Macieira
suggestion works perfectly. There's also .trimmed() and .simplified(), which would strip the whitespaces (including newlines). -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358; fingerprint

Re: Solid::AcAdapter gone w/o replacement, ie. all is Battery now?

2015-05-17 Thread Thiago Macieira
probably enough. You may be powered by something other than an AC adapter (USB ports, for example). -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358; fingerprint: E067 918B B660 DBD1 105C

Re: Alternative to QDateTime::isDateOnly ?

2015-05-01 Thread Thiago Macieira
. a discriminated union) Why wouldn't QVariant be enough? -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358; fingerprint: E067 918B B660 DBD1 105C 966C 33F5 F005 6EF4 5358

Re: Exporting symbols to access them from tests

2015-04-24 Thread Thiago Macieira
otherwise. What is Qt doing in its magic build system - does anyone know? In the .pro file requres(contains(QT_CONFIG, private_tests)) And/or in the .cpp file: #ifdef QT_BUILD_INTERNAL The private_tests config key is the one that sets QT_BUILD_INTERNAL macro. -- Thiago Macieira - thiago

Re: Distros and QtWebEngine

2015-04-21 Thread Thiago Macieira
evil stuff into their browser than Apple. In fact, Chromium and Chrome already have the reputation of hiding spyware (mis)features in their code. I'm not sure about spyware, but they do have a history of inserting things of theirs, like the HTTP-over-QUIC they recently talked about. -- Thiago

Re: CVS dump

2015-03-15 Thread Thiago Macieira
. Anything you'd need from the CVS repository should also be found in Subversion and Git servers. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358; fingerprint: E067 918B B660 DBD1 105C 966C

Re: kdelibs and dependencies

2015-02-19 Thread Thiago Macieira
maybe a handful of standard Linux libraries. Tier 2 components depend on other Tier 1 components. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358; fingerprint: E067 918B B660 DBD1 105C

Re: Review request: QBluez

2015-02-17 Thread Thiago Macieira
. QEXPECT_FAIL is when you know it's wrong but either can't solve it or can't do it now. If it's an expected situation, handle the situation. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358

Re: Review request: QBluez

2015-02-16 Thread Thiago Macieira
, of course, you want to contribute the code to the Qt Project. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358; fingerprint: E067 918B B660 DBD1 105C 966C 33F5 F005 6EF4 5358

Re: Review Request 122320: use xcb-screen count instead of qguiapplication.screens

2015-02-04 Thread Thiago Macieira
count is the correct one? In the commit message. - Thiago Macieira On Fev. 4, 2015, 12:31 a.m., Nick Shaforostoff wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122320

Re: Another proposal for modernization of our infrastructure

2015-02-01 Thread Thiago Macieira
need to do is tell it which commits you want to push, like: git gpush HEAD +thiago +ossi +tronical It would have been ready if I had had the time to review it. You can find it in his dashboard and mine (in qt/qtrepotools). -- Thiago Macieira - thiago (AT) macieira.info - thiago

Re: Another proposal for modernization of our infrastructure

2015-02-01 Thread Thiago Macieira
do -- but I don't feel the pressing need to for someone to work on it. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358; fingerprint: E067 918B B660 DBD1 105C 966C 33F5 F005 6EF4 5358

Re: Another proposal for modernization of our infrastructure

2015-02-01 Thread Thiago Macieira
for anyone who is already using Git. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358; fingerprint: E067 918B B660 DBD1 105C 966C 33F5 F005 6EF4 5358

Re: Another proposal for modernization of our infrastructure

2015-01-30 Thread Thiago Macieira
, 2f6d90dc0ba7853d32a902dbaf92e69ba133c2b1, d124a6d287ef836b3290d92baa2f1e14b18a36bd and 66522d326988614cbe544103f99be14e415e44bd. Ismo, what's the status on upstreaming the one-page-review feature? What did upstream say last time this was attempted? -- Thiago Macieira - thiago

Re: Another proposal for modernization of our infrastructure

2015-01-30 Thread Thiago Macieira
on the installation, so check the settings and try to turn it on. Note that there's no patch to make an single-page all-diff for the new UI. It only exists for the old UI, which is why the new UI is disabled for the Qt Gerrit. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org

Re: Changes to our Git infrastructure

2015-01-04 Thread Thiago Macieira
and widespread languages in the world. For command-line applications on Linux? -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358; fingerprint: E067 918B B660 DBD1 105C 966C 33F5 F005 6EF4 5358

Re: Changes to our Git infrastructure

2015-01-03 Thread Thiago Macieira
infrastructure need to be able to handle the demands we place on them without consuming resources unnecessarily. I understand it is, but you'll have to speak to other sysadmins. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center

Re: Changes to our Git infrastructure

2015-01-03 Thread Thiago Macieira
On Saturday 03 January 2015 12:39:42 Lydia Pintscher wrote: On Sat, Jan 3, 2015 at 12:07 PM, Thiago Macieira thi...@kde.org wrote: For Gerrit: I think before checking Ben's list against specific implementations we should make sure the list is actually correct and complete. It's the basis

systemd unit files (was: kded5)

2014-12-30 Thread Thiago Macieira
to be linked (or installed/copied) to standard D-Bus directories. It does, if XDG_DATA_DIRS was set to point to it. But we need to start supplying systemd unit files, as kdbus will not support activation. We'll need to fall back to the system's way of starting services. -- Thiago Macieira - thiago

Re: Changes to branch management

2014-12-23 Thread Thiago Macieira
push bad tags. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358; fingerprint: E067 918B B660 DBD1 105C 966C 33F5 F005 6EF4 5358

Re: kde : core : c++ 11/14 support : a road-map?

2014-11-26 Thread Thiago Macieira
-initialisation in constructors. If KDE wants to push the envelope, applications could begin requiring C++11 as of next year. KF5 maybe in 2017. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358

Re: ThreadWeaver: using QObject's as Jobs

2014-11-25 Thread Thiago Macieira
On Tuesday 25 November 2014 12:07:27 Milian Wolff wrote: On Monday 24 November 2014 19:28:57 Thiago Macieira wrote: On Monday 24 November 2014 23:00:45 Mirko Boehm wrote: Can I inherit from QObjectDecorator and pass this as the JobInterface *decoratee? The decorator wraps

<    1   2   3   4   5   6   >