Re: Building on OSX

2016-12-30 Thread Thiago Macieira
gt; it. You can build it from source in 5.7 too: http://download.qt.io/community_releases/5.7/5.7.1/ 5.8 will be the same. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Techn

Re: [PATCH] Warn user about wrong characters in date/time format strings

2016-10-23 Thread Thiago Macieira
date format to dd" de "MM" de " -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center ___ subsurface mailing list subsurface@subsurface-divelog.org htt

Re: Qt theme ignored in AppImage

2016-09-26 Thread Thiago Macieira
On segunda-feira, 26 de setembro de 2016 15:08:11 PDT Thiago Macieira wrote: > Is it Gtk3? The qgtk3 plugin would need to be supplied in the AppImage, > which could trigger including of the Gtk libraries in the AppImage too. Actually, Dirk could ship the plugin without the Gtk lib

Re: BT on Mac

2016-09-26 Thread Thiago Macieira
e me the backtrace of that QObject::connect warning? (this requires that Qt print no other warnings before this one) My guess is that this is the emission of the QBluetoothServiceDiscoveryAgent::serviceDiscovered signal starting at QBluetoothServiceDiscoveryAgentPrivate::performMinimalService

Re: thank you, Yahoo

2016-07-12 Thread Thiago Macieira
mailing lists I subscribe to. I will ask someone and let you know what it is called. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center ___ subsurface mailing list subsur

Re: [PATCH] Time zones (again)

2016-07-01 Thread Thiago Macieira
ted patch. Everything looks right, thanks Robert. The only minor thing is the indentation mistake in the last section: - diveplan.when = startTime.toTime_t(); + diveplan.when = startTime.toTime_t(); -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Softwar

Re: [PATCH] Set -std=c++11 flag for g++

2016-06-23 Thread Thiago Macieira
iables with the correct type, with a few exceptions. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center ___ subsurface mailing list subsurface@subsurface-divelog.org ht

Re: [PATCH] Re: Time zones (again)

2016-06-22 Thread Thiago Macieira
t::UTC); I've just realised that my recent changes in Qt 5.8 pessimises the default- constructed QDateTime on 32-bit systems, compared to 5.7... gotta fix it. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center __

Re: [PATCH] Set -std=c++11 flag for g++

2016-06-22 Thread Thiago Macieira
On quarta-feira, 22 de junho de 2016 14:59:49 PDT Dirk Hohndel wrote: > > On Jun 22, 2016, at 2:49 PM, Thiago Macieira <thi...@macieira.org> wrote: > > > > On quarta-feira, 22 de junho de 2016 22:47:23 PDT Anton Lundin wrote: > >> @@ -73,6 +73,7 @@ elseif (&q

Re: [PATCH] Set -std=c++11 flag for g++

2016-06-22 Thread Thiago Macieira
nches too. Do we support building with MSVC at all? Also note that GCC 6 defaults to C++14, so effectively this change "downgrades" the default support. But that might be a good idea, to make people using the latest GCC not add features that older compilers can't grok. -- Thiago

Re: Time zones (again)

2016-06-22 Thread Thiago Macieira
On terça-feira, 21 de junho de 2016 14:34:38 PDT Thiago Macieira wrote: > Anyway, Qt API updated for 5.8: > https://codereview.qt-project.org/163231 It has just passed CI testing and is confirmed now for 5.8. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org So

Re: Time zones (again)

2016-06-21 Thread Thiago Macieira
subtract > gettimezoneoffset() which is the difference between some date 5pm UTC and > 5pm system time zone. QDateTime::offsetFromUtc should give you that information too. It was added in Qt 5.2, which is acceptable for use in Subsurface now (it wasn't when we started). -- Thiago Ma

Re: Time zones (again)

2016-06-21 Thread Thiago Macieira
stamp limits (that may not be exactly 32-bit). Right, good point. At least x32 fixed it, but the IAMCU ABI didn't (not that it's likely to be extensively used with Linux). Anyway, Qt API updated for 5.8: https://codereview.qt-project.org/163231 -- Thiago Macieira - thiago (AT) macieira.i

Re: Time zones (again)

2016-06-21 Thread Thiago Macieira
On terça-feira, 21 de junho de 2016 12:03:41 PDT Linus Torvalds wrote: > On Tue, Jun 21, 2016 at 11:22 AM, Thiago Macieira <thi...@macieira.org> wrote: > >>fromTime_t(qint64, const QTimeZone); > > > > I can add that, but I can't add a qint64 toTime_t(), as yo

Re: Time zones (again)

2016-06-21 Thread Thiago Macieira
On terça-feira, 21 de junho de 2016 10:26:50 PDT Linus Torvalds wrote: > On Tue, Jun 21, 2016 at 9:08 AM, Thiago Macieira <thi...@macieira.org> wrote: > > That function is totally busted, as you've noticed. Not fixing is a > > discouragement, forcing people to find better alt

Re: Time zones (again)

2016-06-21 Thread Thiago Macieira
ut there may be similar gotchas elsewhere. That function is totally busted, as you've noticed. Not fixing is a discouragement, forcing people to find better alternatives. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Archit

Re: Qt 5.7.0 iOS issue

2016-06-17 Thread Thiago Macieira
nt, right? It's not like the entire module was removed. Sounds like a packaging issue then. Jani, can you take a look? > I filed a bug about this: > > https://bugreports.qt.io/browse/QTBUG-54192 > <https://bugreports.qt.io/browse/QTBUG-54192> > > Thiago, any idea whom I should pok

Re: so apparently Qt 5.7 was released

2016-06-16 Thread Thiago Macieira
7 and we've made sure it builds, but that's about it. It might not be available in pre-built packages, but you can download the source and build (note: you probably want 8 GB of RAM or more to build, but I do build it on two machines with 4, so I know it works). -- Thiago Macieira - thiago (AT) m

Re: Build problems (qtTranslator)

2016-05-01 Thread Thiago Macieira
On domingo, 1 de maio de 2016 21:11:20 PDT Dirk Hohndel wrote: > > On May 1, 2016, at 8:24 PM, Thiago Macieira <thi...@macieira.org> wrote: > > > > On domingo, 1 de maio de 2016 15:54:55 PDT Robert Helling wrote: > >> "_qtTranslator&quo

[PATCH] Move the qtTranslator and ssrfTranslator pointers to qt-init.cpp

2016-05-01 Thread Thiago Macieira
No point in having it defined in each main program's .cpp. Especially since the unit tests don't define them. Signed-off-by: Thiago Macieira <thi...@macieira.org> --- core/qt-init.cpp| 1 + export-html.cpp | 3 +-- subsurface-desktop-main.cpp | 2 -- subsurface-

Re: Build problems (qtTranslator)

2016-05-01 Thread Thiago Macieira
bsurface-desktop-main.cpp:QTranslator *qtTranslator, *ssrfTranslator; subsurface-mobile-main.cpp:QTranslator *qtTranslator, *ssrfTranslator; That means the tests will not link. The error appears to be legitimate. Dirk, did you have tests enabled in your build? -- Thiago Macieira - thiago (AT) maciei

Re: Latest iOS release

2016-05-01 Thread Thiago Macieira
> } > } Try this: p *(QQuickItemPrivate *)d_ptr.d > Any idea how to get more information here? Or should I just hardcode a loop > detection? > -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center _

Re: [PATCH] Extend time parsing to before 1970

2016-04-28 Thread Thiago Macieira
t;QDateTime::fromMSecsSinceEpoch(when*1000, Qt::UTC) > > or something. I have *not* tested whether that works. That's what I recommend. The range is ± ~292 million years . It's also 64-bit even on 32-bit systems with time_t definitions unsafe after 2038. Which means you've got +

Re: QML question

2016-04-27 Thread Thiago Macieira
) is there anything special about the content of those strings? 3) can you show the code after that? -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center ___ subsurface mailing li

Re: Better "make clean"? Or easier configuration?

2016-04-20 Thread Thiago Macieira
make rebuild_cache -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center ___ subsurface mailing list subsurface@subsurface-divelog.org http://lists.subsurface-divelog.

Re: Meeting at QtCon?

2016-04-04 Thread Thiago Macieira
orship and maybe land a keynote on Subsurface > and Subsurface-mobile... :-) > > Which obviously means that I'm hoping to be there. Ditto here, I'd like to be there. I need to sit down and think about which sessions to submit. -- Thiago Macieira - thiago (AT) macieira.info - t

Re: signed vs unsigned

2016-03-23 Thread Thiago Macieira
ed well-behaved modulo-2 overflows. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center ___ subsurface mailing list subsurface@subsurface-divelog.org http://lists.subsurface-

Re: SV: stack trace for the crash

2016-03-03 Thread Thiago Macieira
On segunda-feira, 25 de janeiro de 2016 09:33:58 PST Saether Jan-Arve wrote: > I will try to reproduce it here and create a issue for it. Hello Jan Arve Did you manage to reproduce it? It's easily reproducible with Subsurface in 5.6.0rc, but it's not dissected to a testcase. -- Thi

Re: New Bug Reports/Feature Requests

2016-02-23 Thread Thiago Macieira
000 psi / 1 atm = 204.13x expansion in volume 11.1 L * 204.13 = 2265,9 L = 80.02 ft³ This is including the 11.1L of air at 1 atm that was left inside the bottle. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Te

Re: [PATCH] Divelogs.de import: encode + character on password

2016-02-22 Thread Thiago Macieira
b")); Correct, as the documentation says: \note This method does not treat spaces (ASCII 0x20) and plus ("+") signs as the same, like HTML forms do. If you need spaces to be represented as plus signs, use actual plus signs. -- Thiago Macieira - thiago (AT) macieira.info - thia

Re: C++ cout within Subsurface

2016-02-09 Thread Thiago Macieira
On terça-feira, 9 de fevereiro de 2016 13:04:24 PST Tomaz Canabrava wrote: > Willem, > > Also do not include iostream on c code. ;) > And do what sebas said, plenty of debug examples on the code. qDebug() also requires C++ :-) -- Thiago Macieira - thiago (AT) macieira.i

Fwd: Now Accepting GSoC 2016 Mentor Organization Applications

2016-02-08 Thread Thiago Macieira
ps.com. Visit this group at https://groups.google.com/group/google-summer-of-code-announce. For more options, visit https://groups.google.com/d/optout. ----- -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - I

Re: stack trace for the crash

2016-01-22 Thread Thiago Macieira
e: foreach (QQuickItem *invalid, d->m_invalidateAfterRearrange) invalidate(invalid); When invalidate() invalidates a child item, it may call setImplicitSize again. The code in 5.6 has changed a little, but the principles are still the same. Jan-Arve, JP, any hint you could offer? How is t

Re: mobile ui todo

2015-12-03 Thread Thiago Macieira
next week, fingers crossed. As usual, stuff is delayed. Don't expect the final 5.6 until January. -- 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: [PATH] fix broken mac build on old clangs.

2015-11-09 Thread Thiago Macieira
t; But what this might mean is that Subsurface will try to stay on Qt 5.6 for > quite a while. Given that it's the long term release that may not be > entirely unreasonable either. And that's the point of a long-term release. Since we're going to increase the compiler requirements, ther

Re: [PATH] fix broken mac build on old clangs.

2015-11-09 Thread Thiago Macieira
a subset of the C++11 core language features and almost none of the C++11 Standard Library features, corresponding to the intersection of the GCC 4.7, Clang 3.3 and Microsoft Visual Studio 2012 support. Which is equivalent to the VS2012 support. Thanks Microsoft. -- Thiago Macieira - thiago (AT)

Re: [PATH] fix broken mac build on old clangs.

2015-11-09 Thread Thiago Macieira
On Sunday 08 November 2015 22:57:03 Dirk Hohndel wrote: > > On Nov 8, 2015, at 10:41 PM, Thiago Macieira <thi...@macieira.org> wrote: > > > > Can you try adding the -stdlib option in the AppleClang branch to see what > > happens? > > I have an odd ques

Re: [PATH] fix broken mac build on old clangs.

2015-11-09 Thread Thiago Macieira
you try Qt's tst_compiler test for constexpr, it fails. -- 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 F0

Re: [PATH] fix broken mac build on old clangs.

2015-11-09 Thread Thiago Macieira
eted function ‘Foo& Foo::operator=(const Foo&)’ :1:44: note: declared here Another tiny but relevant difference is that GCC reports the first error on the function declared private, which is very confusing to people. Clang reports both on the point of use, like GCC did for the deleted func

Re: [PATH] fix broken mac build on old clangs.

2015-11-08 Thread Thiago Macieira
gt; + set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -std=gnu99 ") Why? Anyone using Clang has a recent enough version of it that it supports C++11. GCC 5.0 and Clang 3.7 switched C mode to C11 by default (your patch reverts that to C99); GCC 6.0 is switching to C++14 by default. -- Thiago Maci

Re: [PATH] fix broken mac build on old clangs.

2015-11-08 Thread Thiago Macieira
On Sunday 08 November 2015 21:57:54 Tomaz Canabrava wrote: > They don't like c++11 They who? What's the error? Is that a build with the old GCC 4.2 instead of Clang? -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Cen

Re: [PATH] fix broken mac build on old clangs.

2015-11-08 Thread Thiago Macieira
ption in the AppleClang branch to see what happens? -- 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: [PATH] fix broken mac build on old clangs.

2015-11-08 Thread Thiago Macieira
anch of LLVM and that should have enough C++11 support for us, according thttp://clang.llvm.org/cxx_status.html. What were the errors? -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358; fingerprint

Re: cloud storage on Android

2015-11-07 Thread Thiago Macieira
o Android development. -- 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 ___ s

Re: trying to stop calling into the MainWindow

2015-11-06 Thread Thiago Macieira
On Friday 06 November 2015 15:34:21 Lubomir I. Ivanov wrote: > > connect( sender, ::method, receiver, ::method); > > method referencing in Qt, yey! connect(sender, ::signal, receiver, <:=]{}); -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Ar

Re: trying to stop calling into the MainWindow

2015-11-06 Thread Thiago Macieira
On Friday 06 November 2015 11:14:05 Tomaz Canabrava wrote: > KGammaRay No K in GammaRay. http://www.kdab.com/gammaray/ https://github.com/KDAB/GammaRay -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/

Re: [RFC PATCH]: honor LC_TIME variable (somewhat)

2015-10-26 Thread Thiago Macieira
. or replacing it with loc = QLocale()... ... except it was introduced in 2a8f32b8 "Fix date and time l10n", so there may be a reason why. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF4535

Re: crash on planning a dive with LANG=fi

2015-10-11 Thread Thiago Macieira
is use of `l' was introduced in Amendment 1 to ISO C90. So %S is looking for a null-terminated string of wchar_t. Since the input string isn't wchar_t, it crashes on wcslen(). -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Tech

Re: Retina screen support

2015-10-09 Thread Thiago Macieira
entation hasn't > quite come to the rescue, yet. There was something when we used qmake, but wasn't completely necessary and we don't use qmake anymore. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center

Re: [PATCH] Fix 32-bit overflow in Divesoft Freedom time handling

2015-10-03 Thread Thiago Macieira
u * u; // this is fine -- 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: [PATCH 2/2] qtserialbluetooth: use QIODevice::Unbuffered io

2015-09-29 Thread Thiago Macieira
, that really shows the bug is probably in the receiving firmware, that it can't deal with block sends of differing sizes. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358; fingerprint: E06

Re: ostc BT download failure

2015-09-29 Thread Thiago Macieira
On Tuesday 29 September 2015 17:25:34 Miika Turkia wrote: > Or it could be too old qt (5.4.1). Is there any easy way to upgrade to > 5.5 with poor Internet connection? Hi Miika, Sorry, no easy way to do that. The libraries are big... -- Thiago Macieira - thiago (AT) macieira.info -

Re: [PATCH 2/2] qtserialbluetooth: use QIODevice::Unbuffered io

2015-09-29 Thread Thiago Macieira
mode should not have made a difference. If it does, it's probably a bug somewhere (the firmware receiving the update or QtBluetooth). -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358; fingerprint

Re: Bluetooth issue on Mac

2015-09-28 Thread Thiago Macieira
BluetoothSocket::connectToService. But I don't see anything evident that would explain how the signal gets emitted from a different thread that the QBluetoothSocket would be running on... -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open

Re: Location management

2015-09-25 Thread Thiago Macieira
e::allocateTimerId() (in > /usr/lib/x86_64-linux-gnu/libQt5Core.so.5.4.1) > ==32235==by 0x6DB5328: Is this a crash on exit? I don't see how the timer ID freelist could be null at any other point in time. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Archi

Re: Windows binary saga

2015-09-22 Thread Thiago Macieira
1: http://code.qt.io/cgit/qt/qtbase.git/commit/?id=8046a574d27b71ac9540b6cd3c029c2a83dd731e. -- 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 1

Re: Windows binary saga

2015-09-22 Thread Thiago Macieira
On Tuesday 22 September 2015 21:16:17 Dirk Hohndel wrote: > > On Sep 22, 2015, at 20:56, Thiago Macieira <thi...@macieira.org> wrote: > >> On Tuesday 22 September 2015 20:52:09 you wrote: > >> I notice that even though this is using Qt5.5 it still identifies itself

Re: QtWebKit is deprecated [was: Re: detecting Windows 10 [was: Re: [PATCH] Fix Bluetooth address truncation]]

2015-09-20 Thread Thiago Macieira
On Sunday 20 September 2015 23:08:46 Lubomir I. Ivanov wrote: > On 20 September 2015 at 23:03, Thiago Macieira <thi...@macieira.org> wrote: > > On Sunday 20 September 2015 12:57:28 Dirk Hohndel wrote: > >> A major pain for Linux. > >> > >> Ubuntu 14.0

Re: QtWebKit is deprecated [was: Re: detecting Windows 10 [was: Re: [PATCH] Fix Bluetooth address truncation]]

2015-09-20 Thread Thiago Macieira
ng WebKit) on OS X, Windows and Linux in the past week. Note that webkit requires Perl, Python and Ruby interpreters to build. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358; fingerprint:

Re: QtWebKit is deprecated [was: Re: detecting Windows 10 [was: Re: [PATCH] Fix Bluetooth address truncation]]

2015-09-20 Thread Thiago Macieira
and the fact that it bundles a lot of libraries). Qt developers have been working with Google developers to unbundle chromium/qtwebengine, but I don't know how far along that has gone. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Techn

Re: [PATCH] Fix Bluetooth address truncation

2015-09-19 Thread Thiago Macieira
Patches look good. On Saturday 19 September 2015 19:40:05 Claudiu Olteanu wrote: > Hi there, > > There was an issue reported by Steve regarding the Bluetooth address > on Windows 10 platforms. The first patch should fix the problem. > > Claudiu -- Thiago Macieira - thiago

Re: Beta Testing

2015-09-09 Thread Thiago Macieira
rts can give a quick answer to that. Yes, it can. http://doc.qt.io/qt-5/qsettings.html#QSettings-4 It's the second argument to that constructor. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG

Re: smoke testing Beta 1

2015-09-04 Thread Thiago Macieira
is). The trick will be to teach Qt and cmake to use that toolchain... Maybe if you somehow move/symlink/whatever /Applications/XCode.app/Resources/Developer/XCodeToolchain.Default (or a similar name, see the full path to clang in Makefiles that qmake generates). -- Thiago Macieira - thiago (AT

Re: smoke testing Beta 1

2015-09-03 Thread Thiago Macieira
ks/libGrantlee_Templates.5.dylib Something in that metadata told QtCore that this isn't a suitable plugin. It's probably the version number. > So my summary so far - it finds the shared library and the plugins, but it > doesn't feel like using them for some reason? Yep. -- Thiago Maciei

Re: smoke testing Beta 1

2015-09-03 Thread Thiago Macieira
On Thursday 03 September 2015 13:45:58 Thiago Macieira wrote: > On Thursday 03 September 2015 10:52:54 Dirk Hohndel wrote: > > And while looking at this and being confused, I noticed this... > > > > (this is grepping for libGrantlee_Templates.5.dylib in the dtruss output)

Re: smoke testing Beta 1

2015-09-03 Thread Thiago Macieira
es.5.dylib\n > Referenced from: > /Applications/Subsurface.app/Contents/PlugIns/grantlee/5.0/grantlee_default > tags.so\n Reason: image not found)" Looks like grantlee_defaulttags.so could not be loaded because /Users/hohndel/src/install-root/lib/libGrantlee_Templates.5.dylib was not fo

Re: Re. subsurface: FTBFS in experimental

2015-08-27 Thread Thiago Macieira
, the same way that we do for Windows and OS X: divecomputer, marble, Qt, ICU. -- 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: at some point we need to release Subsurface 4.5

2015-08-27 Thread Thiago Macieira
, libQt5Bluetooth.dylib, Qt5Bluetooth.dll, QtBluetooth.framework, as well as do #include QtBluetooth/Xxxx qtconnectivity (always written in lowercase) is the name of the Git repository and tarball that contains QtBluetooth. It also contains QtNfc. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org

Re: Re. subsurface: FTBFS in experimental

2015-08-27 Thread Thiago Macieira
that around six months ago ) And why is that? Because Qt bundles a lot of libraries (statically built into Qt libs and plugins). Example: libxcb. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG

Re: [PATCH] Add native Bluetooth support for Windows platforms

2015-08-20 Thread Thiago Macieira
too long, you could do everything in one line. -- 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: [PATH] Add native Bluetooth support for Windows platforms

2015-08-18 Thread Thiago Macieira
(remoteDeviceInfo.address().toString(), Qt::MatchContains); Please merge as much of the code as you can. The first two (or more) lines of the two branches of the #if are identical, so it's a good idea to keep them together to avoid code duplication. Ditto on patch 15. -- Thiago Macieira

Re: [PATCH] Add native Bluetooth support for Windows platforms

2015-08-18 Thread Thiago Macieira
or toLatin1, depending on context. I will take a look at the code when I review and suggest how to do 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

Re: [PATH] Add native Bluetooth support for Windows platforms

2015-08-18 Thread Thiago Macieira
Please do this only if you can do it quickly. Your code is correct, just a little inefficient. If it isn't straightforward, please keep it as-is and provide a patch later to fix it. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source

Re: Testing Native Bluetooth support on Windows platforms

2015-08-11 Thread Thiago Macieira
On Tuesday 11 August 2015 23:36:05 Claudiu Olteanu wrote: I totally forgot about the existence of Windows Vista :) We wish we had that option too. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG

Re: Grantlee problems

2015-07-30 Thread Thiago Macieira
a backtrace of that warning complaining threading issue. Since we're talking about threading issues, an all-thread backtrace would be adviseable (gdb command: t a a bt) -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center

Re: Grantlee problems

2015-07-30 Thread Thiago Macieira
the current dive site: 756762654 This is also a debug statement, not warning. -- 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

Re: Ctrl-+ and Qt5.5

2015-07-20 Thread Thiago Macieira
On Monday 20 July 2015 08:40:24 Thiago Macieira wrote: I have an external USB with numeric keypad in the office. I'll see if I can test that. Ctrl+KP_Plus works for me even without the patch. I can't tell what the patch is fixing then. -- Thiago Macieira - thiago (AT) macieira.info - thiago

Re: Ctrl-+ and Qt5.5

2015-07-20 Thread Thiago Macieira
On Monday 20 July 2015 05:54:17 Dirk Hohndel wrote: On Sun, Jul 19, 2015 at 07:43:19PM -0700, Thiago Macieira wrote: On Sunday 19 July 2015 19:38:25 Thiago Macieira wrote: On Sunday 19 July 2015 06:28:37 Gaetan Bisson wrote: [2015-07-19 17:06:48 +0300] Claudiu Olteanu: I have

Re: [PATCH] Bluetooth improvements

2015-07-20 Thread Thiago Macieira
, but I'd say anything besides 5.4 and 5.5 is not a good idea, as no one is testing 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

Re: [GSoC] Week 8 (Native Bluetooth support)

2015-07-20 Thread Thiago Macieira
On Tuesday 21 July 2015 00:46:07 Claudiu Olteanu wrote: What I know is that programming with Win32/Winsock2 API is quite tricky (at least for me..:) ). Not just you. And it only gets worse with WinRT, which lacks even Winsock2... Fortunately, that's not a target for Subsurface. -- Thiago

Re: Ctrl-+ and Qt5.5

2015-07-19 Thread Thiago Macieira
://projects.archlinux.org/svntogit/packages.git/tree/trunk/keypad-shor tcuts.patch?h=packages/qt5 I'll ask our Qt maintainer. Testing it here. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358; fingerprint

Re: [PATCH] Bluetooth improvements

2015-07-19 Thread Thiago Macieira
version :). [1] - http://doc.qt.io/qt-5.5/qtglobal.html#Q_OS_MAC That's what I call misleading information... That means it doesn't include OpenDarwin. Fix: https://codereview.qt-project.org/121841 -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect

Re: Ctrl-+ and Qt5.5

2015-07-19 Thread Thiago Macieira
On Sunday 19 July 2015 19:38:25 Thiago Macieira wrote: On Sunday 19 July 2015 06:28:37 Gaetan Bisson wrote: [2015-07-19 17:06:48 +0300] Claudiu Olteanu: I have a Fedora 22 with Qt 5.5.0 and the shortcut is working. Subsurface version: 4.4.2-1040-g5cbbff008411c322. It might be related

Re: Ctrl-+ and Qt5.5

2015-07-19 Thread Thiago Macieira
an upstream bug. All Ctrl+Shift shortcuts stopped working in 5.5, such as Qt Creator's Ctrl+. I tried to debug this during the Qt Contributor Summit but didn't get anywhere. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology

Re: [PATCH] Bluetooth improvements

2015-07-19 Thread Thiago Macieira
: Q_OS_DARWIN and Q_OS_MAC also mean Q_OS_IOS || Q_OS_OSX. -- 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: [PATCH] Bluetooth improvements

2015-07-19 Thread Thiago Macieira
(TARGET_OS_MAC) TARGET_OS_MAC # define Q_OS_OSX # define Q_OS_MACX // compatibility synonym # endif #endif -- 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: [PATCH] Bluetooth support for Android

2015-07-15 Thread Thiago Macieira
week to see if he's back. -- 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: [PATCH] Bluetooth support for Android

2015-07-14 Thread Thiago Macieira
())); + return; + } +#endif Please add a simple comment to the source code why this is done, in addition to the commit. Something like // on Android, the device gets stuck in if we try to use it before discovery is done -- Thiago Macieira - thiago (AT) macieira.info - thiago

Re: different segfault with lastest master

2015-07-11 Thread Thiago Macieira
corrupted inside an internal impelmentation of QArrayData Thiago? It crashed inside malloc, so the problem is not in the stack trace. Something corrupted the malloc structures. Valgrind would have been the best, but didn't work... -- Thiago Macieira - thiago (AT) macieira.info - thiago

Re: Make marble compile with Qt 5.5 on OSX

2015-07-09 Thread Thiago Macieira
On Thursday 09 July 2015 16:22:54 Tomaz Canabrava wrote: it seems that QDataStream was removed from some includes Sorry about that; that was my change. It was removed from qstringlist.h, which means that QIODevice is not present from qstringlist.h and that brought qobject.h. -- Thiago

Re: [GSoC] Week 6 (Native Bluetooth support)

2015-07-08 Thread Thiago Macieira
already got access to OS X in real hardware anyway... -- 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: [GSoC] Week 6 (Native Bluetooth support)

2015-07-07 Thread Thiago Macieira
... It should be libqt5bluetooth5, according to packages.ubuntu.com. -- 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: [GSoC] Week 6 (Native Bluetooth support)

2015-07-06 Thread Thiago Macieira
... /me goes back to reviewing IPv6 support for the IoT software -- 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: [PULL REQUEST] Subsurface branch - QtBluetooth serial implementation

2015-07-06 Thread Thiago Macieira
inquiry. Therefore I choose to use !(mode == QBluetoothLocalDevice::HostPoweredOff)) instead of (mode == QBluetoothLocalDevice::HostDiscoverable || mode == QBluetoothLocalDevice::HostConnectable || ..) I meant doing bool on = !(x == y); instead of bool on = x != y; -- Thiago

Re: [GSoC] Week 6 (Native Bluetooth support)

2015-07-06 Thread Thiago Macieira
and is implemented in qtserialbluetooth.cpp source. I don't know what changed from Qt 5.4.1 version to Qt 5.5.0 because it worked before the Qt update. nm -C qtserialbluetooth.o Does it appear? Does it appear with () at the end? -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software

Re: great progress on dive site handling

2015-06-29 Thread Thiago Macieira
I can tell: I don't know if m_tile is correctly supposed to be NULL there and the code should be checking for validity before dereferencing; or if it wasn't supposed to be NULL and loading of the tile was required. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software

Re: great progress on dive site handling

2015-06-29 Thread Thiago Macieira
/2015/n4474.pdf -- 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: Warning. Danger. Dragons ahead

2015-06-19 Thread Thiago Macieira
similar and seems to be legal on all the file systems. Why have separators at all? MMDDHHmmss should be enough until Dec 31st, . -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358

Re: Current git no compilee

2015-06-17 Thread Thiago Macieira
is that of Qt itself. qmake would set QT_DEBUG if you compiled in debug mode, but CMake may not be doing that, so you inherited the Qt default (release mode). -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP

Fwd: [Development] Bluetooth support for Windows (RT/10) in Qt 5.6

2015-06-16 Thread Thiago Macieira
-project.org http://lists.qt-project.org/mailman/listinfo/development - -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center PGP/GPG: 0x6EF45358; fingerprint: E067 918B B660

<    1   2   3   >