/srv/irclogs.ubuntu.com/2015/08/25/#ubuntu-unity.txt

=== Malsasa is now known as Guest12573
=== Malsasa_ is now known as Malsasa
pstolowskitsdgeos, hey07:11
tsdgeospstolowski: hi07:12
pstolowskitsdgeos, silo 44 has all the scopes and unity api stuff for wily to sync it with api changes in vivid. we need to land it somehow with unity807:13
pstolowskitsdgeos, it's not dual landing, so i guess we need a binary sync of unity8?07:13
pstolowskitsdgeos, would be good to check if unity8 actually compiles against that silo07:14
tsdgeoswe do you need to land it with unity8?07:14
pstolowskitsdgeos, yes, because of the activation api change07:14
tsdgeossure but the current unity8 already doesn't work/build07:15
tsdgeosso it's not a need to but a nice to, no?07:15
pstolowskitsdgeos, what do you mean? to just land silo 44?07:16
tsdgeoswon't be worse than what we have today, no?07:17
pstolowskitsdgeos, i see. yes07:17
tsdgeosi think it's a good way to unblock you07:17
tsdgeosand then we can decide what to do07:18
pstolowskitsdgeos, would be good to check if unity8 compiles with this silo anyway to make sure i didn't screw up versions etc. or do you have problems with other unity8 dependencies too?07:20
tsdgeospstolowski: that makes sense07:20
tsdgeoslet me check07:20
tsdgeosi think that's the onyl dewpendency issue we had07:20
pstolowskitsdgeos, so perhaps i should add unity8 to the silo and check if it builds?07:22
tsdgeospstolowski: i can do it in my chroot07:22
pstolowskitsdgeos, okay, thanks07:22
tsdgeosif you add it it'll land it and change the changelog and stuff07:22
tsdgeosand will be weird, no?07:22
pstolowskitsdgeos, no, i'd remove it afterwards07:23
tsdgeosah, well let me just try it here07:23
tsdgeoshmmm07:28
tsdgeosor maybe we need to land it07:28
tsdgeossince it will remove unity8 on dist-upgade07:28
tsdgeosdon't want to end up with a phone image without a phone shell :D07:29
tsdgeospstolowski: it builds fine07:36
pstolowskitsdgeos, \o/ thanks07:37
tsdgeospstolowski: so now the question is what to do07:37
tsdgeospstolowski: because if we do nothing as i suggested, it removes unity8 -> bad, a binary sync is in theory binary incompatible because of gcc stuff -> bad, and building will mess up our changelog -> bad07:39
tsdgeos:D07:39
pstolowskiawesome :D07:40
tsdgeosi guess wait for sil or mzanetti to show up?07:40
pstolowskitsdgeos, ok07:41
=== mzanetti_ is now known as mzanetti
tsdgeosmzanetti: yo07:49
mzanettihey07:49
tsdgeosmzanetti: pstolowski is landing the new unity-api for wily our options are07:49
tsdgeos we do nothing, it removes unity8 -> bad, a binary sync from vivid-overlay is in theory binary incompatible because of gcc stuff -> bad, and building will mess up our changelog -> bad07:50
pstolowskitsdgeos, mzanetti not only unity-api, also unity-scopes-shell and unity-scopes-api in same silo07:50
tsdgeosor that's how i understand it :D07:50
mzanettimeh07:50
mzanettiok... will see what I can do07:51
mzanettitsdgeos, ok... so seems like we need a branch now... I suggest we move development towards the overlay branch and then we can sync the code over to trunk whenever pstolowski forces us to08:03
mzanettiand release a wily package from there08:03
tsdgeosmzanetti: so we do want to use master as overlay branch and then have a wily/ for wily? or use overlay branch for overlay and master for wily?08:04
mzanettioverlay -> overlay ,  trunk -> wily08:05
mzanettitsdgeos, otherwise it'll confuse people08:05
tsdgeosk08:05
tsdgeosmzanetti: we already have that overlay branch btw08:06
mzanettitsdgeos, says the branches have diverged. afaik you created the overlay branch08:06
mzanettitsdgeos, is there anything to lose if I overwrite it?08:06
tsdgeosno08:07
tsdgeosit's just what master had when wily was created08:07
mzanettiah... LP pushed translations08:07
tsdgeosright08:08
tsdgeosthat's what i was going to mention08:08
mzanettiok. updated08:08
tsdgeoswe need to make sure that this change doesn't break translations08:08
tsdgeosor we just keep both overlay and master totally in sync08:08
tsdgeosso it doesn't matter08:08
mzanettitsdgeos, yeah... I would try to keep them in sync. just have an additional patch that fixes packaging or so08:11
tsdgeosk08:12
pstolowskimzanetti, tsdgeos fyi, Michi has been working on changes to unity-scopes-api packaging that will make it possible for us to maintain a single source tree for both releases and merge the two trunks we have right now. once this is finished we will be able to apply similar solution to the rest of our projects, but for now we need to have a split to unblock landings08:48
mzanettipstolowski, ok. I'm preparing a release package for wily08:49
pstolowskitsdgeos, hmm09:26
tsdgeospstolowski: yes?09:26
pstolowskitsdgeos, i was just trying to install silo 4 (inline music playback) on a freshly flashed krillin09:26
pstolowskitsdgeos, it says unity8 would downgrade09:26
pstolowskitsdgeos, but citrain doesn't say the silo is dirty09:26
tsdgeos:S09:26
pstolowskitsdgeos, have you landed anything recently in vivid-overlay?09:27
tsdgeostoday09:29
tsdgeosyes09:29
tsdgeosright silo needs rebuilding09:29
pstolowskitsdgeos, ok, i guess i'll rebuild then :/09:29
mzanettipstolowski, so, unity-api trunk has unity-shell-scopes=6 still10:02
cimitsdgeos, on the preview rating, how do you plan making a listview?10:02
cimitsdgeos, do we have any design on that?10:03
tsdgeosmzanetti: right, silo 4410:03
mzanettiack10:03
mzanettipstolowski, tsdgeos. so I'll make a branch that builds with that, and add it to that silo?10:03
tsdgeoscimi: just make it be like the repeater but with lazily loaded children10:03
tsdgeosmzanetti: it "just builds", or at least that's what my chroot says10:04
tsdgeoschroot+silo44 of coruse10:04
cimitsdgeos, maybe I need a coffee but how we can do that? we want the listview not to be full height?10:05
cimitsdgeos, or we want it to scroll and hide reviews?10:05
mzanettitsdgeos, but we still want it as part of that silo, no?10:05
tsdgeosmzanetti: i guess so, i convinved pstolowski that we didn't but then i guess we do if we're branching off10:06
pstolowskimzanetti, yes10:06
mzanettipstolowski, add this one then: https://code.launchpad.net/~mzanetti/unity8/build-for-wily/+merge/26901810:06
tsdgeoscimi: we do not want the list to scroll, the page already scrolls10:06
cimitsdgeos, indeed, so how can it be lazy loading?10:06
pstolowskimzanetti, hmm it's targeted against existing trunk?10:06
tsdgeoscimi: because it's a listview and that's what listviews do :D10:06
cimitsdgeos, if we have a listview that doesn't scroll and is as long as all the delegates, how is different from a repeater?10:06
pstolowskimzanetti, ah, you want to create another trunk for vivid, right?10:07
mzanettipstolowski, yes. we'll be using the overlay branch for vivid+overlay10:07
pstolowski+110:07
tsdgeoscimi: because you set displayMarginBeginning/end accordingly10:07
mzanettiwe need to resubmit the existing mps but that's ok10:07
tsdgeosand then only the delegates on view are created10:07
cimitsdgeos, ok never used that property, just reviewed when you did10:08
mzanettipstolowski, I see you're building silo410:09
mzanettipstolowski, those MP's need to be redirected towards the overlay branch too10:09
mzanettithe audioCardSupport one10:09
pstolowskimzanetti, ah, good point10:10
mzanettiI'll do10:10
mzanettipstolowski, hmm... can we reconfigure silo 4? I can't find it in the new "spreadsheet"10:11
mzanettifound it... it was hiding behind an "...older..." link10:12
mzanettipstolowski, ok. reconfigured10:14
pstolowskimzanetti, thanks10:15
=== pstolowski is now known as pstolowski|lunch
mzanettipstolowski|lunch, hey, please let me know before you land that silo to wily10:44
mzanettineed to make sure OTA-6 is out of the way before that10:45
tsdgeosmzanetti: almost nice, we almost got a vivid CI for the overlay branch :D10:46
mzanettitsdgeos, ?10:46
tsdgeosmzanetti: https://code.launchpad.net/~unity-team/unity8/audioCardSupport/+merge/26902910:47
tsdgeoshttps://code.launchpad.net/~unity-team/unity8/audioCardSupport/+merge/269029/comments/67650010:47
tsdgeosgot run on vivid10:47
tsdgeosbut since it's vivid only, not with the overlay ppa10:47
tsdgeosalso fails to build10:47
tsdgeos:D10:47
tsdgeosoh wait i see some ppa in there10:48
mzanettithe ppa is there, yes10:48
tsdgeosnice, let's propse another branch maybe we get a green light :D10:48
tsdgeosthat'd be so weird10:48
tsdgeos:D10:48
tsdgeosafter all that time10:49
mzanettilol10:49
mzanettiI didn't know there's a vivid ci configured on that branch10:49
mzanettitsdgeos, do you know what happened with this branch? https://code.launchpad.net/~mterry/unity8/tutorial-bottom-gap-rtm/+merge/25001010:50
mzanettiwhy is it in the list of approved ones?10:50
tsdgeosmzanetti: i just proposed https://code.launchpad.net/~aacid/unity8/cropped_image_image_urls_silent/+merge/269040 against overlaty, let's see what CI says10:51
mzanettiyep10:51
tsdgeosmzanetti: no idea really what happened with the branch10:52
tsdgeosi guess it was just approved when we abandoned rtm?10:52
mzanettitsdgeos, yeah... but why does it show up today? it hasn't been around before10:53
mzanettiand I can't see any manual change10:53
tsdgeosmzanetti: show up where?10:53
mzanettitsdgeos, https://code.launchpad.net/unity8/+activereviews10:53
tsdgeosit always shows there?10:53
tsdgeosmaybe before you were visiting https://code.launchpad.net/~unity-team/unity8/trunk/+activereviews like i was?10:54
mzanettiah... that might be true10:54
tsdgeosbut if we're going ot use overlay10:54
tsdgeoswe probably need to move to https://code.launchpad.net/unity8/+activereviews10:54
mzanettiyep10:56
tsdgeosit's nicer doesn't have the WiP ones :D10:57
cimitsdgeos, how do I use those displayMargin?11:13
cimitsdgeos, it looks like the property is used to show more delegates rather than showing less11:14
tsdgeoscimi: you set it so that the listview only creates delegates for the really visible part of the listview11:14
tsdgeoscimi: you set it negative11:14
cimiah, cruel11:15
ltinklmzanetti, https://code.launchpad.net/~lukas-kde/unity8/clickAwayNotifications/+merge/268170 updated in case you get some time to review, thx :)11:21
cimitsdgeos, setting those without hacks seems hard at first glance11:25
tsdgeoscimi: define "hacks"11:26
cimitsdgeos, chaining back using parent or adding extra properties?11:27
cimitsdgeos, this is a preview Widget, doesn't know where it is11:27
mzanettiltinkl, ack, thanks11:27
cimitsdgeos, you might have to add special properties to set this display margins from outside11:27
cimitsdgeos, or in the preview widget look for parent and see where those parent are11:28
tsdgeoscimi: yep11:28
tsdgeosyou use properties, looking at your parents is a bad idea :D11:28
cimitsdgeos, so I will need to add those alias only for this preview widget, right?11:29
cimitsdgeos, and through the loader of PreviewWidgetFactory too11:29
cimidoesn't sound nice11:30
cimiis there an alternative?11:30
tsdgeoscimi: i don't think so, you need to pass the info down, so all of the intermediate stuff need it too11:32
tsdgeoscimi: or you can just try looking at your parent :D11:33
tsdgeosyou don't really need to look at your parent11:33
cimitsdgeos, padding a displayMarginBeginning/end from preview widget factory loader doesn't sound nice either11:33
cimipassing11:33
ciminot padding11:33
tsdgeoswell you don't pass the display margin11:33
tsdgeosyou pass the visibility range11:33
cimimmmm11:34
tsdgeosanyhow11:34
tsdgeosyou can try using mapToItem with null parent11:34
tsdgeosthat gives you the pos against the root view11:34
tsdgeosthat should be ok-ish too i guess11:34
tsdgeosfood!11:34
=== MacSlow is now known as MacSlow|lunch
=== pstolowski|lunch is now known as pstolowski
tsdgeosmzanetti: https://code.launchpad.net/~aacid/unity8/make_pot_file/+merge/26905712:54
mzanettilol12:55
mzanettiI started to fix this too12:55
mzanettiand seems ltinkl too12:55
tsdgeos:D12:59
tsdgeosmy other branch against overlay has still not come back13:00
tsdgeoswhich is good news13:00
tsdgeosmay even be running on CI13:00
mzanettiyeah13:01
ltinklmzanetti, go ahead :)13:03
mzanettiltinkl, too late... tsdgeos beat us13:03
mzanettidandrader, ping13:04
dandradermzanetti, pong13:04
mzanettidandrader, now I remember why I was so struggling with the focus stuff... because there's a bug that I fixed but now it's back after your changes13:04
mzanettidandrader, go to make tryShell, set all to desktop mode13:04
mzanettiand hit the "start all apps" button13:04
mzanettiCPU will loop13:05
mzanetti100%13:05
dandradermzanetti, and this is the only way to reproduce this issue?13:05
mzanettidandrader, well, happens if 2 items are added to the model in the same event loop cycle13:06
mzanettiso could happen in real usage too13:06
mzanettiwe're not really using it yet in production so hard to tell13:06
mzanettitsdgeos, your branch returned13:08
tsdgeoslook13:09
tsdgeosit almost worked :D13:09
mzanettiyeah13:09
mzanettiqmltests config seems borked13:10
tsdgeosactually it's pretty good13:10
tsdgeosthe 2 autopilots that fail13:10
tsdgeosare the 2 that i've patches waiting for13:10
tsdgeosqmluitests is weird13:10
mzanettitsdgeos, qmltests has issues with gcc13:10
tsdgeosis it because it misses the ppa?13:11
mzanettiwhich one? overlay?13:11
mzanettiseems it's there13:11
tsdgeoshmmm13:11
mzanettiat least there's a hook called D09add_ppa~ci-train-ppa-service~stable-phone-overlay13:11
tsdgeosyeah D09add_ppa~ci-train-ppa-service~stable-phone-overlay13:12
mzanettimaybe it lacks a dist-upgrade13:12
tsdgeosyeah13:12
tsdgeos gcc-4.9-base is pure vivid13:13
tsdgeosas well as  cpp-4.913:13
tsdgeosso the old versions probably come from a stale thing?13:13
tsdgeosbut there's an apt-get update there13:13
tsdgeosalmost nice13:14
tsdgeos:D13:14
tsdgeosget us shell access and we'll fix /debug it13:14
* tsdgeos hides13:14
mzanettitsdgeos, heh, yeah... I can't do that any more :(13:17
mzanettitsdgeos, need to ping someone from ci13:17
tsdgeos:D13:17
mzanettitsdgeos, weirdest thing is, they want to provide jenkins as a service, where we'd be responsible to configure our jobs ourselves. That sounds good, but when I asked for login information to ssh into the vms they said that won't happen13:19
tsdgeosyeah13:19
tsdgeosyou get to debug by looking at the thin air13:19
mzanettiso I'm really not sure how we should set up those jobs tvh13:20
mzanettitbh13:20
mzanettibut we'll see13:20
mzanettimaybe there's something I'm just not aware of13:20
dandradermzanetti, can't reproduce the busy CPU issue you described13:25
mzanettiwhat should I say13:26
mzanettidandrader, let me create a video for you then :)13:27
=== dandrader is now known as dandrader|afk
mzanettidandrader|afk, http://notyetthere.org/data/focus-loop.ogv13:35
mzanettitsdgeos, vm upgraded, job re-kicked13:59
mzanettilet's see13:59
mzanettitsdgeos, nope, still failing with the same one14:06
tsdgeosmzanetti: :/14:07
=== dandrader|afk is now known as dandrader
dandraderoh, right. I forgot to change to desktop mode14:17
* dandrader tries14:17
dandradermzanetti, yeah, I can reproduce it14:18
mzanettiok :)14:19
guest42315balloons, why aren't you guys on #ubuntuonair?15:17
guest42315balloons, i can't see you there, there are only 6 people o_O15:18
=== attente is now known as willcooke
=== MacSlow|lunch is now known as MacSlow
=== willcooke is now known as attente
=== cyphase_ is now known as cyphase

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!