| mterry | Unity failed to build on arm. https://launchpadlibrarian.net/131652234/buildlog_ubuntu-raring-armhf.unity_6.12.0daily13.02.19-0ubuntu1_FAILEDTOBUILD.txt.gz | 09:21 |
|---|---|---|
| mterry | Seems to be "/build/buildd/unity-6.12.0daily13.02.19/plugins/unityshell/src/inputremover.cpp:426:73: error: cast from 'unsigned char*' to 'long unsigned int*' increases required alignment of target type [-Werror=cast-align]" | 09:21 |
| mterry | And on line 461 | 09:21 |
| mterry | sil2100, bregma ^ | 09:21 |
| sil2100 | Some new commit, hm? | 09:21 |
| mterry | Yeah, it built yesterday | 09:22 |
| sil2100 | mterry: let my try and fix that | 09:26 |
| sil2100 | mterry: I fixed the build failure probably, but a bit hackishly - trying to build the whole unity trunk now on ARM for testing | 10:16 |
| mterry | sil2100, thanks | 10:24 |
| didrocks | sil2100: if you look at intel/ati results, we are back to 25/26 failures per config | 10:35 |
| didrocks | waiting nvidia | 10:35 |
| didrocks | sil2100: urgh, 38 failures on it :/ | 10:38 |
| didrocks | mterry: ^ | 10:39 |
| sil2100 | huh? | 10:48 |
| sil2100 | didrocks: anthy tests for ibus failed as well this time, geh :| | 10:49 |
| didrocks | sil2100: do you mind looking as well? | 10:49 |
| MCR_ | Since the weekend I have no indicators and no global menus anymore - maybe this is related ? | 10:49 |
| didrocks | MCR_: using the daily release? | 10:50 |
| MCR_ | yes | 10:50 |
| didrocks | yeah, was an issue on the ppa | 10:50 |
| didrocks | but should be fixed by now | 10:50 |
| MCR_ | unfortunately not here... | 10:50 |
| didrocks | MCR_: apt-cache policy ? | 10:51 |
| didrocks | sil2100: is it what we are seeing in the video for nvidia? ^ | 10:51 |
| MCR_ | didrocks, 500 http://ppa.launchpad.net/unity-team/staging/ubuntu/ raring/main amd64 Packages | 10:53 |
| MCR_ | release v=13.04,o=LP-PPA-unity-team-staging,a=raring,n=raring,l=Unity staging ppa,c=main | 10:53 |
| MCR_ | origin ppa.launchpad.net | 10:53 |
| MCR_ | ? | 10:53 |
| didrocks | MCR_: doesn't tell me the version :) | 10:53 |
| MCR_ | unity: | 10:54 |
| MCR_ | Installed: 6.12.0daily13.02.08bzr3162pkg0raring0 | 10:54 |
| MCR_ | Candidate: 6.12.0daily13.02.08bzr3162pkg0raring0 | 10:54 |
| didrocks | MCR_: ah, you are using staging | 10:57 |
| didrocks | MCR_: this is not the daily build ppa | 10:57 |
| MCR_ | didrocks, oh, there is a new PPA ? I did not know that. I'll search... | 10:58 |
| didrocks | MCR_: you should just use raring | 10:58 |
| didrocks | MCR_: it's getting the latest and greatest daily | 10:59 |
| didrocks | which pass tests | 10:59 |
| MCR_ | ok, thx - I will purge the PPA then... | 10:59 |
| didrocks | sounds like a good plan :) | 10:59 |
| MCR_ | :) | 11:00 |
| === mmrazik is now known as mmrazik|lunch | ||
| MCR_ | didrocks, purged the PPA, now using Installed: 6.12.0daily13.02.08-0ubuntu1 - still no indicators and no global menus :( | 11:18 |
| didrocks | MCR_: what version of libindicator do you have? | 11:30 |
| MCR_ | didrocks, libindicator7: | 11:33 |
| MCR_ | Installed: 12.10.2daily13.02.15-0ubuntu1 | 11:33 |
| MCR_ | (thanks 4 your help, btw) | 11:33 |
| didrocks | MCR_: this version is from the daily-release ppa | 11:33 |
| didrocks | isn't it? | 11:34 |
| MCR_ | didrocks - hmm, I purged the PPA, but maybe something went wrong with that - thanks for the help again - I'll investigate | 11:36 |
| didrocks | ok :) | 11:39 |
| sil2100 | grrr | 11:41 |
| === _salem is now known as salem_ | ||
| smspillaz | mterry: ah bugger, sorry about that, will fix | 11:56 |
| smspillaz | mterry: suprised we don't have ci jobs for unity on arm | 11:57 |
| smspillaz | sil2100: were you going to fix that or are you busy atm? | 11:57 |
| smspillaz | its easy I can fix it now | 11:57 |
| sil2100 | smspillaz: I'm testing my fix now, armhf package is still building though | 12:00 |
| sil2100 | On unity-team/ppa | 12:00 |
| smspillaz | sil2100: ah okay. Do you have a branch somewhere ? | 12:01 |
| sil2100 | smspillaz: https://code.launchpad.net/~sil2100/unity/fix_ftbfs_arm_alignment_hack | 12:01 |
| sil2100 | It's a hack | 12:01 |
| bregma | smspillaz, unity-on-arm ci was disabled due to speedness issues | 12:02 |
| sil2100 | I could have changed unsigned char to unsigned long as well and just cast it to unsigned char instead, but this one works too | 12:02 |
| smspillaz | bregma: isn't there an autolander job for it? | 12:02 |
| smspillaz | bregma: we have that for compiz | 12:02 |
| bregma | autolanding shouldn;t have to take up to half a day per commit | 12:02 |
| smspillaz | bregma: trunk also shouldn't get broken :) | 12:03 |
| bregma | if there is more or better hardware, it should get reenabled asap, that's up to QA | 12:03 |
| bregma | I suspect there are other, more urgent uses for arm builders right now | 12:04 |
| smspillaz | that's true | 12:04 |
| smspillaz | bregma: I wonder why we can't cross compile yet | 12:04 |
| bregma | cross building with dpkg is not really an option, it's still way too broken despite multi-arch starting to trickle in | 12:04 |
| smspillaz | sil2100: yikes, that is hacky, I wonder why reinterpret_cast from unsigned char * to unsigned long * doesn't work on arm | 12:04 |
| smspillaz | bregma: yeah I know, I've tried it myself it sucks | 12:05 |
| sil2100 | smspillaz: well, ARM and its misalignment issues does not allow casting from a smaller pointer type to a bigger one | 12:05 |
| sil2100 | The other-way around is fine though, also, void * is safe here | 12:06 |
| smspillaz | huh | 12:06 |
| sil2100 | That's hm, a really bitchy thing anyway | 12:06 |
| MCR_ | didrocks, just FYI: I solved the no-indicators problems now by adding the ppa:indicator-applet-developers/indicator-staging-ppa | 12:06 |
| sil2100 | Usually __aligned__ should be used to fix such problems, but here we're getting the array from X, so I didn't know if we could do it in this way | 12:07 |
| didrocks | sil2100: did you see my ping/question about the videos on nvidia? | 12:10 |
| sil2100 | didrocks: ah, yes, I thought that I answered that: the videos show that something is wrong with detecting ibus enabled - since it seems as if anthy is enabled (I think?) but the test fails noticing - but it might be an AP issue, not a regression | 12:11 |
| didrocks | sil2100: ok, do you want us to relaunch the tests to see if it's transiant? | 12:13 |
| didrocks | (that doesn't prevent to continue looking at the old run) | 12:14 |
| sil2100 | didrocks: yes, please, this way we won't waste time | 12:14 |
| didrocks | ok, let's do that o/o\o/ | 12:15 |
| sil2100 | I checked and nothing changed in trunk that could have broken ibus for one engine | 12:15 |
| didrocks | (running) | 12:15 |
| didrocks | sil2100: please, let us know when the armhf fix is merged so that we rebuild unity | 12:16 |
| smspillaz | didrocks: sorry about that one :( | 12:34 |
| didrocks | smspillaz: no worry dude, I wished that we had those running as part of the merger :) | 12:35 |
| sil2100 | smspillaz: ok, unity built fine with that hack of mine - you had a nicer idea for fixing it, or can we try merging it in as it is ;) ? | 12:39 |
| smspillaz | sil2100: merge as is | 12:39 |
| sil2100 | smspillaz: if something better comes up, I'm sure we can re-do this - I'll just add a comment that it's a hack, so that no one starts getting all WTF? after a while ;) | 12:40 |
| smspillaz | sil2100: well, reinterpret_cast'ing from unsigned char * to unsigned long * is a hack too | 12:40 |
| smspillaz | it only works because libX11 and the server align the property data with the specified property type | 12:41 |
| sil2100 | smspillaz: can you approve it then :) ? Thanks! | 12:43 |
| smspillaz | sil2100: can you link me the mp? I don't get email notifications for unity | 12:44 |
| smspillaz | (cause I don't work for canonical anymore) | 12:44 |
| smspillaz | sil2100: done, you will need to change the global branch status yourself (I don't have permission to do that) | 12:46 |
| smspillaz | howdy kamstrup | 12:46 |
| sil2100 | smspillaz: \o/ | 12:58 |
| MCR_ | smspillaz, you do not need to work for Canonical to get those emails, should I subscribe you ? | 12:59 |
| sil2100 | MCR_: not sure if anyone would willingly like to recieve those ;p | 13:00 |
| sil2100 | ;) | 13:00 |
| MCR_ | just freaks like MCR ;) | 13:00 |
| MCR_ | smspillaz, https://code.launchpad.net/~unity-team/unity/trunk/+addsubscriber | 13:00 |
| MCR_ | smspillaz, a mnior fix: https://code.launchpad.net/~mc-return/compiz/compiz.merge-type-is-unsigned-fix/+merge/149267 | 13:07 |
| MCR_ | *minor | 13:08 |
| MCR_ | sil2100, got one for Unity as well: https://code.launchpad.net/~mc-return/unity/unity.merge-minor-fixes/+merge/149269 | 13:14 |
| smspillaz | MCR_: saw that, acked, thanks | 13:17 |
| MCR_ | smspillaz, it is *very* minor, but still ;) | 13:18 |
| MCR_ | sil2100, smspillaz: Thanx 4 the fast approvals :) | 13:20 |
| * smspillaz is still waiting on the builder to check the xorg-gtest stuff ... again :( | 13:22 | |
| smspillaz | this time it failed because I merged something new in which didn't quite follow the dso linking rule | 13:22 |
| smspillaz | which the pbuilders are strict about | 13:22 |
| === mmrazik|lunch is now known as mmrazik | ||
| MCR_ | smspillaz, urgh - btw, bug 1094650 is still valid here | 13:25 |
| ubot5 | bug 1094650 in compiz (Ubuntu) "Compilation of lp:compiz with gcc 4.8.0 fails with -WERROR because of warnings" [Medium,New] https://launchpad.net/bugs/1094650 | 13:25 |
| MCR_ | smspillaz, I have to turn off WERROR or compilation fails here... | 13:25 |
| smspillaz | MCR_: its in-progress on the gnome-end | 13:27 |
| MCR_ | ah, ok - thanks | 13:27 |
| smspillaz | MCR_: there is not much we can do about it | 13:27 |
| MCR_ | sure, it's not a big problem | 13:27 |
| smspillaz | MCR_: well, the problem is that its a non-standard compiler option, and passing that option to clang makes it error out | 13:28 |
| MCR_ | the error is about gcc though, but I've seen your report and their answer now... | 13:29 |
| smspillaz | MCR_: this is just why C++ can be a pain sometimes | 13:36 |
| MCR_ | smspillaz, btw - we have to party :) -> today MCR branch No. 100 will be merged \o/ | 13:42 |
| smspillaz | heh | 13:43 |
| smspillaz | MCR_: well, I will be 21 soon :) So maybe we can roll it into that | 13:44 |
| MCR_ | Sam, it is still incredible and a miracle for me, how someone can be such an C++ expert with this young age ;) | 13:45 |
| smspillaz | pfft, I know nothing | 13:45 |
| MCR_ | Congratulation to the 21 of course... Hopefully someone gets you a faster machine, so you do not have to fight with disk space and compiling times anymore... | 13:47 |
| smspillaz | MCR_: well, I'm waiting until machines with broadwell come out | 13:48 |
| smspillaz | of course, last year I told myself I was waiting until machines with IVB came out | 13:48 |
| smspillaz | ... and the year before sandybridge .... | 13:48 |
| MCR_ | The open source community should buy you such a machine each year ;) | 13:49 |
| smspillaz | heh | 13:49 |
| === dandrader is now known as dandrader|afk | ||
| mterry | didrocks, the rm /var/crash hack didn't fix it | 14:19 |
| mterry | didrocks, but on the plus side, tests are under threshold now | 14:19 |
| mterry | didrocks, any objections if I make that percent->number change in the thresholds to their current values? (33 failed tests and 14 regressions allowed) | 14:20 |
| didrocks | mterry: yeah, those are high enough values, aren't they above the threshold even? | 14:20 |
| mterry | didrocks, those are the threshold. I think 481 tests. 7% and 3% respectively | 14:21 |
| didrocks | mterry: ok, perfect, I understood "current values" as, "current failure on nvidia" :) | 14:21 |
| mterry | didrocks, why do we allow so many regressions? darn tests should be reliable | 14:22 |
| didrocks | mterry: well, I agree, sil2100 did his best to fix as many as possible, (remember that we went from 130+ failures to where we are now) | 14:22 |
| didrocks | mterry: oh, btw, rebuilding unity alone now? :) | 14:23 |
| mterry | didrocks, I suppose... Maybe it won't crash this time. It looks... like maybe we crashed in between suites of autopilot (if the Date file inside the apport crash is reliable) | 14:24 |
| didrocks | mterry: yeah, so a valid case for it | 14:24 |
| * mterry loads up his trusty jenkins web interface | 14:24 | |
| didrocks | tsss :p | 14:24 |
| * didrocks is so saddddd | 14:24 | |
| === dandrader|afk is now known as dandrader | ||
| mterry | didrocks, so the fact remains that compiz is crashing sometimes and we don't know why and can't get traces from it for some reason (why?) | 14:31 |
| === dandrader is now known as dandrader|afk | ||
| smspillaz | sil2100: andyrock: Trevinho: https://code.launchpad.net/~compiz-team/compiz/compiz.fix_1110138/+merge/147548 free karma | 15:08 |
| mterry | smspillaz, in reviewing the debian/ changes to compiz, I noticed and filed bug 1130160 | 15:11 |
| ubot5 | bug 1130160 in compiz (Ubuntu) "Add copytex to default/unity profiles" [Undecided,New] https://launchpad.net/bugs/1130160 | 15:11 |
| smspillaz | mterry: ah, I knew I had missed it somewhere | 15:11 |
| smspillaz | mterry: thanks | 15:11 |
| mterry | smspillaz, really it was thanks to didrocks who was over my shoulder :) | 15:12 |
| smspillaz | mterry: are you running with the default settings ? | 15:12 |
| smspillaz | mterry: (+ upgrade) | 15:12 |
| mterry | smspillaz, um, I think so | 15:13 |
| smspillaz | mterry: can you post the output of | 15:14 |
| smspillaz | gsettings get org.compiz.core:/org/compiz/profiles/unity/core/ active-plugins | 15:14 |
| smspillaz | I haven't got unity installed at the moment and need to get the default plugin order | 15:14 |
| mterry | smspillaz, ['core', 'composite', 'opengl', 'compiztoolbox', 'decor', 'vpswitch', 'snap', 'mousepoll', 'resize', 'place', 'move', 'wall', 'grid', 'regex', 'imgpng', 'session', 'gnomecompat', 'animation', 'fade', 'unitymtgrabhandles', 'workarounds', 'scale', 'expo', 'ezoom', 'unityshell'] | 15:15 |
| smspillaz | hmm | 15:15 |
| smspillaz | copytex isn't there | 15:16 |
| smspillaz | mterry: cat ~/.config/compiz-1/compizconfig/done_upgrades ? | 15:16 |
| sil2100 | smspillaz: could that be the reason for why popey's decorator was crashing when using the patch you prepared for VB issues? | 15:17 |
| mterry | smspillaz, empty... maybe I'm not on the latest daily-build version yet | 15:17 |
| smspillaz | sil2100: yeah probably | 15:17 |
| mterry | smspillaz, I have the latest compiz from daily-build | 15:18 |
| smspillaz | mterry: just to double check, ls /etc/compizconfig/upgrades ? | 15:19 |
| smspillaz | hmm wait | 15:19 |
| smspillaz | that doesn't seem right | 15:19 |
| mterry | smspillaz, 01, 02, and 03 | 15:19 |
| smspillaz | hmm | 15:20 |
| smspillaz | I think I changed the install dir to $datadir/compizconfig/upgrades on didrocks request but the .install file didn't get updated | 15:21 |
| smspillaz | mterry: okay, branch coming your way | 15:23 |
| smspillaz | mterry: https://code.launchpad.net/~compiz-team/compiz/compiz.fix_1130160/+merge/149308 | 15:24 |
| mterry | smspillaz, thanks | 15:29 |
| andyrock | smspillaz, sure | 15:35 |
| smspillaz | andyrock: cheers | 15:35 |
| andyrock | smspillaz, quick question | 15:36 |
| andyrock | ing *Window::glPaint (during occlusion test) windows are passed from top to bottom right? | 15:37 |
| smspillaz | andyrock: yep | 15:40 |
| andyrock | here the first passed window is nautilus | 15:42 |
| andyrock | smspillaz, maybe the desktop window? | 15:43 |
| smspillaz | andyrock: yep | 15:43 |
| smspillaz | andyrock: what are you trying to do ? | 15:43 |
| andyrock | smspillaz, fixing bug 748539 | 15:44 |
| ubot5 | bug 748539 in unity (Ubuntu) "Multi-monitor - Panel and launcher visible on top of multimonitor non-focused fullscreen apps" [High,In progress] https://launchpad.net/bugs/748539 | 15:44 |
| andyrock | just debugging for the moment | 15:45 |
| smspillaz | andyrock: heh, have fun with that one | 15:52 |
| smspillaz | andyrock: I'm pretty sure panels and such are meant to go above unfocused fullscreen applications | 15:53 |
| smspillaz | thats how compiz' stacking policy works | 15:53 |
| smspillaz | though maybe only if there's a window that needs to go above that fullscreen window | 15:54 |
| smspillaz | ... which is how it works if there's another one on another monitor | 15:54 |
| smspillaz | eg normal window > docks > fullscreens > other docks > other normals | 15:55 |
| smspillaz | maybe we could make it so that this policy only applies where they are on the same monitor, though that feels tricky | 15:55 |
| andyrock | we just need to don't draw unity on monitor A if the first window (in the stack) on monitor A is a fullscreen one | 15:58 |
| andyrock | maybe there are corner case | 15:58 |
| andyrock | i'm testing this solution now | 15:58 |
| andyrock | of course during alt+tab unity should be drawn | 15:59 |
| smspillaz | andyrock: that's true, but "don't draw unity on monitor B" is not really a concept that nux understands | 15:59 |
| smspillaz | maybe you can get away with it in the multimonitor case because we have to do clipping | 16:00 |
| smspillaz | but selectively painting BaseWindow objects ... that's something I spent about a month hacking into nux | 16:00 |
| === dandrader|afk is now known as dandrader | ||
| smspillaz | bschaefer: if you have time for reviews, feel free to look at these two: | 16:38 |
| smspillaz | erm | 16:38 |
| === francisco is now known as Guest89363 | ||
| mterry | smspillaz, the latest compiz autolanding run seems to have hit a merge conflict with debian/changelog entries. Note that if your branch is linked with a bug, you don't need to edit the changelog at all. I'll fix the merge conflict, just heads up | 17:30 |
| mterry | didrocks, https://code.launchpad.net/~mterry/compiz/latestsnapshot/+merge/149357 | 18:02 |
| === salem_ is now known as _salem | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!