Re: regressions because of the new tank handling

On 16 September 2017 at 22:12, Stefan Fuchs wrote: > Hi Lubomir, > > Am 16.09.2017 um 20:20 schrieb Lubomir I. Ivanov: > > On 16 September 2017 at 20:42, Stefan Fuchs wrote: > > Lubomir was faster and confirms what I already typed into the mail: > No, I didn't fix

Re: regressions because of the new tank handling

Hi Lubomir, Am 16.09.2017 um 20:20 schrieb Lubomir I. Ivanov: > On 16 September 2017 at 20:42, Stefan Fuchs wrote: >> Lubomir was faster and confirms what I already typed into the mail: >> No, I didn't fix this. What is the case is that the missing private header >> file

Re: regressions because of the new tank handling

On 16 September 2017 at 20:42, Stefan Fuchs wrote: > > Lubomir was faster and confirms what I already typed into the mail: > No, I didn't fix this. What is the case is that the missing private header > file "qgeofiletilecache_p.h" is simply not existing when checking out > version

Re: regressions because of the new tank handling

Am 16.09.2017 um 18:28 schrieb Dirk Hohndel: > >> On Sep 16, 2017, at 2:41 AM, Stefan Fuchs > > wrote: >> What I see/saw is the following: >> - First of all downloading the private headers for QtLocation and >> QtPositioning works but building the googlemaps

Re: regressions because of the new tank handling

> On Sep 16, 2017, at 10:39 AM, Lubomir I. Ivanov wrote: > > On 16 September 2017 at 12:41, Stefan Fuchs wrote: >> QtPositioning works but building the googlemaps plugin fails. Is there also >> a minimum Qt version > 5.5.1 required for this? > > ok, i saw

Re: regressions because of the new tank handling

On 16 September 2017 at 12:41, Stefan Fuchs wrote: > QtPositioning works but building the googlemaps plugin fails. Is there also > a minimum Qt version > 5.5.1 required for this? ok, i saw the log file. one private header which was added in QtLocation 5.6 seems to be required.

Re: regressions because of the new tank handling

> On Sep 16, 2017, at 2:41 AM, Stefan Fuchs wrote: > What I see/saw is the following: > - First of all downloading the private headers for QtLocation and > QtPositioning works but building the googlemaps plugin fails. Is there also a > minimum Qt version > 5.5.1 required for

Re: regressions because of the new tank handling

On 16 September 2017 at 13:21, Lubomir I. Ivanov wrote: > On 16 September 2017 at 12:41, Stefan Fuchs wrote: > >> - And now I still have this error when trying to run the resulting exe >> although I installed qtdeclarative5-qtquick2-plugin: >>

Re: regressions because of the new tank handling

On 16 September 2017 at 12:41, Stefan Fuchs wrote: > Am 15.09.2017 um 03:45 schrieb Dirk Hohndel: > > On Sep 14, 2017, at 11:39 AM, Willem Ferguson > wrote: > I include four dives I did last month, as well as the profile info on > Subsurface 4.6.4.

Re: regressions because of the new tank handling

Am 15.09.2017 um 03:45 schrieb Dirk Hohndel: >> On Sep 14, 2017, at 11:39 AM, Willem Ferguson >> wrote: >> I include four dives I did last month, as well as the profile info on >> Subsurface 4.6.4. Unfortunately I cannot build the latest master due to Qt >> 5.9

Re: regressions because of the new tank handling

>From phone. .so version problems I could not solve. Lots of. I am departing on dive trip. Back around Oct. 5th. Will look at it intensively when I get back. Gave up last time, around 2 months ago. Thanks for your help above. I will keep contact. Kind regards, willem On 15 Sep 2017 16:44,

Re: regressions because of the new tank handling

On Fri, Sep 15, 2017 at 12:04:23PM +0300, Lubomir I. Ivanov wrote: > > On Sep 14, 2017, at 11:39 AM, Willem Ferguson > > wrote: > > I include four dives I did last month, as well as the profile info on > > Subsurface 4.6.4. Unfortunately I cannot build the

Re: regressions because of the new tank handling

> On Sep 14, 2017, at 11:39 AM, Willem Ferguson > wrote: > I include four dives I did last month, as well as the profile info on > Subsurface 4.6.4. Unfortunately I cannot build the latest master due to Qt > 5.9 problems on Ubuntu. > i cannot confirm any

Re: regressions because of the new tank handling

> On Sep 14, 2017, at 11:39 AM, Willem Ferguson > wrote: > I include four dives I did last month, as well as the profile info on > Subsurface 4.6.4. Unfortunately I cannot build the latest master due to Qt > 5.9 problems on Ubuntu. Can you say more about

Re: regressions because of the new tank handling

Thank you, Linus, I will keep a check on it with new releases. Kind regards, Willem On 14 Sep 2017 9:23 PM, "Linus Torvalds" wrote: > On Thu, Sep 14, 2017 at 11:39 AM, Willem Ferguson > wrote: > > > > I include four dives I did

Re: regressions because of the new tank handling

On Thu, Sep 14, 2017 at 11:39 AM, Willem Ferguson wrote: > > I include four dives I did last month, as well as the profile info on > Subsurface 4.6.4. Unfortunately I cannot build the latest master due to Qt > 5.9 problems on Ubuntu. These are sidemount dives

Re: regressions because of the new tank handling

On Thu, Sep 14, 2017 at 10:55 AM, Linus Torvalds wrote: > > However, while playing around with an alternate fix (just stating the > sensor explicitly in the xml file), I note that we seem to get that > case wrong too, and that might in fact be a bug in the new code.

Re: regressions because of the new tank handling

On Thu, Sep 14, 2017 at 1:36 AM, Stefan Fuchs wrote: > > First of all I have to admit that maybe this "spike in the pressure graph" > item doesn't really fit together with the other things I reported. I don't > believe that this item is also linked to the change Linus did. I even

Re: regressions because of the new tank handling

Hi Dirk, hi Linus, Am 13.09.2017 um 23:00 schrieb Dirk Hohndel: >> Hmm. The infobox pressure thing I will look into. > Thanks, just saw the patch. Thanks for fixing this so quickly! >> The spikes in the data I would need a test file for to see what's up. > Stefan, can you provide that? First of

Re: regressions because of the new tank handling

On Wed, Sep 13, 2017 at 02:00:49PM -0700, Dirk Hohndel wrote: > > > On Sep 13, 2017, at 1:00 PM, Linus Torvalds > > wrote: > > > > The tankbar issue was not an "unintended side effect". I very much > > talked about it, and you said you'd fix it up because it was

Re: regressions because of the new tank handling

> On Sep 13, 2017, at 1:00 PM, Linus Torvalds > wrote: > > On Wed, Sep 13, 2017 at 11:20 AM, Dirk Hohndel wrote: >> Given that I have been rather distracted for the last few weeks, I haven't >> followed up on making sure that the changes to the

Re: regressions because of the new tank handling

On Wed, Sep 13, 2017 at 1:00 PM, Linus Torvalds wrote: > > Hmm. The infobox pressure thing I will look into. This should fix it. Linus From a2741d5cd59ce1cd16c97d46f6e548f763f5c4ef Mon Sep 17 00:00:00 2001 From: Linus Torvalds

regressions because of the new tank handling

Given that I have been rather distracted for the last few weeks, I haven't followed up on making sure that the changes to the tank handling that Linus made a while ago didn't have any other unintended side effects. It appears that Stefan found a couple which he reported here: