| tsdgeos | taaaaaaaaaaags | 07:13 |
|---|---|---|
| tsdgeos | imported | 07:13 |
| * tsdgeos kills them | 07:13 | |
| Saviq | tsdgeos, okies, I described everything I know about the crash in https://trello.com/c/lAudAZVp/40-13-shell-rotation-mp-review-iteration-on-feature-and-autopilot-tests | 08:57 |
| Saviq | tsdgeos, will you please try and see what you can do about it | 08:58 |
| tsdgeos | yes | 08:58 |
| tsdgeos | finishing a bug report and patch to qt upstream | 08:58 |
| tsdgeos | and will hop into that | 08:58 |
| Saviq | tx | 09:01 |
| tsdgeos | well, let's see what they think of https://codereview.qt-project.org/#/c/110348/ | 09:09 |
| tsdgeos | fixes the testcase but may not be the best thing ever :D | 09:09 |
| tsdgeos | Saviq: so those crashes only happen on the device? or desktop too? | 09:26 |
| Saviq | tsdgeos, didnae try on desktop | 09:27 |
| tsdgeos | k | 09:27 |
| tsdgeos | let me try it | 09:27 |
| tsdgeos | because it is defenitely nicer to debug in the desktop | 09:27 |
| Saviq | indeed | 09:28 |
| === Malsasa_ is now known as Malsasa | ||
| === marcusto_ is now known as marcustomlinson | ||
| tsdgeos | not much success trying to make it crash in the pc | 10:06 |
| === marcusto_ is now known as marcustomlinson_ | ||
| === MacSlow is now known as MacSlow|lunch | ||
| === marcusto_ is now known as marcustomlinson_ | ||
| === MacSlow|lunch is now known as MacSlow | ||
| tsdgeos | Saviq: http://paste.ubuntu.com/10821832/ | 13:31 |
| tsdgeos | QV4_MM_AGGRESSIVE_GC=1 qmlscene it | 13:31 |
| tsdgeos | kaboom | 13:31 |
| tsdgeos | :D | 13:31 |
| tsdgeos | MacSlow: ↑↑ | 13:32 |
| Saviq | tsdgeos, nice | 13:33 |
| Saviq | tsdgeos, I did see the regex bits in the trace, and remember we've had issues with that before | 13:33 |
| MacSlow | tsdgeos, oh ha | 13:42 |
| MacSlow | tsdgeos, Saviq: so since we now know what it is... looking for a home-grown fix against Qt 5.4 or try 5.5 first? | 13:42 |
| tsdgeos | 5.5 doesn't crash with that example | 13:43 |
| MacSlow | tsdgeos, so 5.5 it is then?! | 13:43 |
| tsdgeos | well 5.5. crashes with lots of other things :D | 13:43 |
| MacSlow | tsdgeos, *sigh* | 13:43 |
| tsdgeos | i mean it's just alpha | 13:43 |
| MacSlow | tsdgeos, hm... I'm using RegExp in notifications for sure.. but it sofar never blew up like that.... iirc | 13:44 |
| tsdgeos | well it all depends on how/when the garbage collector kicks in | 13:45 |
| MacSlow | tsdgeos, I can give it a go to fix it... unless you've already an idea about the what and how for a fix. | 13:45 |
| tsdgeos | MacSlow: fix 1-> remove the regexp :D fix 2->fix the js stuff in QtDeclarative | 13:46 |
| tsdgeos | that's all i can think of for the moment | 13:48 |
| MacSlow | tsdgeos, I'll try to see if I can eliminate all RegExp from my unity8-shellRotation branch first | 13:49 |
| MacSlow | tsdgeos, just three uses of RegExp in all our qml-code... should not be too hard | 13:50 |
| MacSlow | tsdgeos, and the relevant are of course just in snap-decision notifications :/ | 13:51 |
| MacSlow | tsdgeos, Saviq: I'll try to come up with RegExp-less work-around for my unity8-shellRotation branch, test it some more and then MP it... while waiting on review I'll turn to a fix for QtDeclaraitve. Ok? | 13:54 |
| tsdgeos | i'm having a look at qt at the moment | 13:54 |
| MacSlow | tsdgeos, ok | 13:54 |
| MacSlow | tsdgeos, on the side... is there perhaps a RegExp-fix in Qt 5.5 we could backport to 5.4 ourselves? | 13:55 |
| tsdgeos | maybe | 13:56 |
| * tsdgeos tips hat to MacSlow for the suggestion | 14:02 | |
| MacSlow | tsdgeos, you found a fix in Qt 5.5? | 14:03 |
| tsdgeos | yes | 14:03 |
| MacSlow | tsdgeos, sweet! | 14:03 |
| MacSlow | tsdgeos, btw... http://pastebin.ubuntu.com/10821942 | 14:03 |
| * MacSlow just bricked his n4 *sigh* | 14:04 | |
| MacSlow | well not reall bricked... just re-flashing required | 14:04 |
| tsdgeos | so patch accepted by upstream :) | 14:06 |
| tsdgeos | https://codereview.qt-project.org/#/c/110411/ | 14:06 |
| tsdgeos | Mirv: can we get ↑ ? | 14:07 |
| tsdgeos | Saviq: ↑↑ | 14:07 |
| MacSlow | tsdgeos, nice :) | 14:07 |
| Saviq | \o/ | 14:07 |
| MacSlow | Saviq, tsdgeos is the hero of the day :) | 14:08 |
| Saviq | that was quick! | 14:08 |
| MacSlow | indeed | 14:08 |
| tsdgeos | cheers to MacSlow that convinced me to look at the diff | 14:09 |
| tsdgeos | i was going all crazy and thinking of fixing it myself | 14:09 |
| greyback | nice one | 14:10 |
| MacSlow | tsdgeos, well by now I've learned to take shortcuts as early as possible ;) | 14:10 |
| MacSlow | tsdgeos, on the other hand... I'm just fed up with that branch and want to get it in :) | 14:10 |
| tsdgeos | now this is hoping | 14:11 |
| * MacSlow high5's tsdgeos anyway | 14:11 | |
| tsdgeos | that this is actually that is making it crash in the run without QV4_MM_AGGRESSIVE_GC=1 scenario too | 14:11 |
| * tsdgeos hi5 back | 14:11 | |
| MacSlow | *clap* | 14:11 |
| tsdgeos | and makes one wonder if we should run all our tests with QV4_MM_AGGRESSIVE_GC to find this stuff | 14:11 |
| MacSlow | tsdgeos, sounds to be worth a try... maybe we'll catch more non-determinstic issues | 14:12 |
| Saviq | tsdgeos, MacSlow, I'm building qtdeclarative packages with this patch, will report asap | 14:13 |
| tsdgeos | Saviq: awesomeness | 14:13 |
| MacSlow | Saviq, ok | 14:13 |
| dandrader | Saviq, did trunk get bad tags again or is the script outdated? http://paste.ubuntu.com/10822008/ | 14:14 |
| Saviq | dandrader, the script doesn't get outdated any more | 14:14 |
| Saviq | dandrader, and yeah, we did bring those in the landing yesterday, but should be gone now | 14:14 |
| MacSlow | Saviq, what's the tag-"threshold"? | 14:15 |
| tsdgeos | yeah i killed them this morning | 14:15 |
| tsdgeos | we forgot about running pot_file again :D | 14:16 |
| tsdgeos | https://code.launchpad.net/~aacid/unity8/pot_file/+merge/256151 | 14:16 |
| tsdgeos | anyone? ↑ | 14:16 |
| MacSlow | tsdgeos, on it | 14:16 |
| Saviq | MacSlow, wdym "threshold"? | 14:19 |
| Saviq | MacSlow, if a tag does not point at a revision, it's bad | 14:19 |
| Mirv | tsdgeos: like, in vivid/vivid-rtm regardless of the qtdbus patches? ok, if you can please file a LP bug I can refer to | 14:20 |
| tsdgeos | Mirv: yes, this is a "proper/clear" 5.4 bugfix | 14:20 |
| tsdgeos | Mirv: sure. opening the bugfix now | 14:21 |
| Mirv | tsdgeos: wow! you mean that'd affect the 1421009? | 14:21 |
| MacSlow | Saviq, I see... but I think to remember we wanted tags clearned in the past, which were just "too old" (a few months) | 14:21 |
| tsdgeos | Mirv: hmmm, what is 1421009 ? | 14:21 |
| Saviq | Mirv, no | 14:21 |
| tsdgeos | Mirv: ah no, sorry no, it's just a different bugfix | 14:22 |
| Saviq | MacSlow, no, it was always a case of "they're wrong" | 14:22 |
| Mirv | oh, no, ok. I thought some magic so that the million dbus patches wouldn't be needed :) | 14:22 |
| Mirv | ok, I'll have a different landing for this one then | 14:22 |
| tsdgeos | Mirv: unfortunately no :D | 14:22 |
| Mirv | I guess it'll need to go to vivid-rtm though with final freeze on Thu | 14:22 |
| Saviq | Mirv, OTOH this could cause relatively random crashes anywhere that's using RegExes in QML | 14:24 |
| tsdgeos | yeah this makes sense for vivid proper imho | 14:25 |
| tsdgeos | Mirv: https://bugs.launchpad.net/ubuntu/+source/qtdeclarative-opensource-src/+bug/1443981 | 14:25 |
| ubot5 | Ubuntu bug 1443981 in qtdeclarative-opensource-src (Ubuntu) "Protect the RegExp against early garbage collection" [Undecided,New] | 14:25 |
| Saviq | ooh there's a W-Series | 14:26 |
| tsdgeos | upgrade! | 14:26 |
| Saviq | don't think there's an archive yet ;) | 14:26 |
| Saviq | there's only a placeholder in LP | 14:26 |
| MacSlow | tsdgeos, approved your pot_file branch... will top-approve it when jenkins is happy too | 14:27 |
| tsdgeos | oki | 14:27 |
| Saviq | https://launchpad.net/ubuntu/w-series | 14:28 |
| Mirv | tsdgeos: thanks | 14:28 |
| MacSlow | but we don't have a funky animal yet... or did I miss that? | 14:28 |
| Saviq | nope | 14:29 |
| MacSlow | with w it's going to be interesting... | 14:29 |
| MacSlow | we could get back to :) | 14:30 |
| MacSlow | warthog | 14:30 |
| davmor2 | MacSlow: Wascling Wabbit </elma_fudd_impression> | 14:30 |
| MacSlow | :) | 14:30 |
| davmor2 | MacSlow: Wonderful Whippet | 14:31 |
| MacSlow | davmor2, "whippet"? | 14:37 |
| davmor2 | MacSlow: http://en.wikipedia.org/wiki/Whippet think smaller, slower greyhound | 14:38 |
| Saviq | wombat FTW! | 14:39 |
| Saviq | heh, it takes LP 1h50 to build qtdeclarative... looks like some 25m on arale ;) | 14:46 |
| mzanetti | haha | 14:47 |
| tsdgeos | Saviq: really? | 14:52 |
| tsdgeos | we need a arale powered ci | 14:52 |
| tsdgeos | with someone continously clicking the screen | 14:52 |
| Saviq | tsdgeos, it's not -j9 on LP I think | 14:52 |
| tsdgeos | Saviq: does the hardware even has that many cores? | 14:52 |
| Saviq | tsdgeos, arale has 8 | 14:53 |
| Saviq | tsdgeos, 2x4 | 14:53 |
| tsdgeos | Saviq: sure i mean the CI | 14:53 |
| tsdgeos | Saviq: it's actually 4+4 no? 4 big 4 small | 14:53 |
| Saviq | tsdgeos, oh no, that's part of why they're not -j9 ;) | 14:53 |
| Saviq | yeah 4+4 | 14:53 |
| Saviq | tsdgeos, but I mean I don't think they parallelize at all | 14:53 |
| * Saviq checks to see he's not full of sh*t | 14:54 | |
| === dandrader is now known as dandrader|afk | ||
| Saviq | greyback, should there not be a 1:1 dependency between qtdeclarative5-qtmir-plugin and qtmir-{android,desktop}? | 14:59 |
| * Saviq sees the plugin updated but not qtmir-android | 14:59 | |
| greyback | Saviq: think there should be yes | 15:00 |
| Saviq | greyback, dandrader|afk, how about the shell-rotation-aware qtubuntu, should anything pull it in? | 15:01 |
| mzanetti | MacSlow, Wollmilchsau | 15:01 |
| MacSlow | mzanetti, well... I would like that :) | 15:01 |
| mzanetti | +1 :D | 15:01 |
| MacSlow | mzanetti, to be honest Linux is that almost... there's hardly anything it cannot do or become :) | 15:02 |
| greyback | Saviq: usual thing which pulls it in is ubuntu-touch-session - can't have unity8 depend on it explicitly as unnecessary for desktop | 15:02 |
| mzanetti | MacSlow, which is why that name came to my mind | 15:02 |
| Saviq | greyback, well, we could have -android | -desktop | 15:02 |
| greyback | Saviq: true | 15:02 |
| MacSlow | mzanetti, how could we convince sabdfl of that ;) | 15:03 |
| Saviq | greyback, I'm just asking because I can install rotation-aware unity8 and only qtmir plugin gets pulled in with, right now | 15:03 |
| mzanetti | MacSlow, with this image perhaps: http://www.forexfactory.com/attachment.php?attachmentid=255957&stc=1&d=1244150501 | 15:03 |
| MacSlow | mzanetti, there have to be better examples :) | 15:03 |
| mzanetti | :D | 15:04 |
| greyback | Saviq: a stronger package dependency chain would do no harm. Thought it kept relatively weak to stop androidy stuff installing on desktop | 15:08 |
| greyback | no other reason | 15:09 |
| Saviq | tsdgeos, MacSlow, https://owncloud.sawicz.net/index.php/s/0afe9e8a599ef8fef2f2a105e9db357d# | 15:17 |
| MacSlow | Saviq, grabbing... | 15:18 |
| === dandrader|afk is now known as dandrader | ||
| tsdgeos | MacSlow: Saviq: looks good here, have you had any issue? | 15:47 |
| Saviq | tsdgeos, no crash in sight | 15:48 |
| tsdgeos | \o/ | 15:49 |
| MacSlow | tsdgeos, just about to finish the setup (re-flashing, pulling packages etc) | 15:49 |
| Saviq | we really need to fix the on-exit crash... | 15:50 |
| tsdgeos | do we have an on exit crash? | 15:50 |
| Saviq | tsdgeos, it's an abort in mir | 15:51 |
| tsdgeos | oh | 15:51 |
| Saviq | MacSlow, 6 subsequent successful runs of the two rotation tests | 15:54 |
| Saviq | and counting | 15:54 |
| Saviq | got it to fail now, no crash | 15:58 |
| MacSlow | Saviq, what kind of failure? | 16:00 |
| Saviq | MacSlow, http://pastebin.ubuntu.com/10822593/ | 16:00 |
| MacSlow | Saviq, ah... that's new | 16:00 |
| MacSlow | Saviq, compilation almost done here | 16:01 |
| Saviq | MacSlow, kk | 16:01 |
| === Malsasa_ is now known as Malsasa | ||
| MacSlow | Saviq, hm... first run and it fails with http://pastebin.ubuntu.com/10822702 | 16:27 |
| MacSlow | Saviq, might be I overwritten some of the debs out of your zip with other apt-gets | 16:33 |
| MacSlow | Saviq, currently it ran fine a couple of attempts | 16:33 |
| MacSlow | Saviq, 10 in a row without failure yet. | 16:34 |
| MacSlow | Saviq, hm... I still get crashes | 16:37 |
| MacSlow | Saviq, got crash files written for malit, unity8-dash and webbrowser-app | 16:38 |
| MacSlow | Saviq, at least no unity8, but then it seems the qtdeclaritive fix isn't enough | 16:39 |
| MacSlow | Saviq, I'll look into the crashes and send you the gathered info. | 16:42 |
| === alan_g is now known as alan_g|EOD | ||
| ZehCnaS34 | Is this where I go to talk about deb packages? | 17:05 |
| ZehCnaS34 | talk about *improving* dep packaging | 17:06 |
| ZehCnaS34 | *deb | 17:06 |
| === dandrader is now known as dandrader|lunch | ||
| === Malsasa_ is now known as Malsasa | ||
| === dandrader|lunch is now known as dandrader | ||
| === dandrader is now known as dandrader|afk | ||
| === dandrader|afk is now known as dandrader | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!