/srv/irclogs.ubuntu.com/2014/01/30/#ubuntu-unity.txt

=== Saviq_ is now known as Saviq
=== duflu_ is now known as duflu
veebersMirv: ping, can you let me know when you're online please, I have a couple of questions re: the qt5.2 && autopilot issue06:31
Mirvveebers: I've been here for 2h already, ask away07:15
veebersMirv: Hmm, I've been having connectivity issues today :-\07:15
veebersMirv: Re: the Qt 5.2 bug from yesterday. First question: How would I get autopilot-qt trunk re-built into that ppa? I found an issue with a recent change in trunk on the devicse07:16
veebersdevices*07:16
veebersI think that backing them out will either fix the issue or at least get us another step closer07:16
Mirvveebers: autopilot-qt is already rebuilt in that PPA07:16
Mirvand is automatically rebuilt whenever lp:autopilot-qt changges07:17
veebersMirv: Sorry, I meant "rebuilt into it after I've merged my change"07:17
Mirvthe "~77" in the version number indicates the bzr version07:17
veebersMirv: ah cool, so once I merge this change it'll get in?07:17
veebersright07:17
Mirvveebers: yes, although it's "daily" ie triggered once a day, but I can (or anyone in  ~canonical-qt5-edgers) also trigger it manually at https://code.launchpad.net/~canonical-qt5-edgers/+recipe/daily-qt52-autopilot-qt07:18
veebersMirv: Once this MR is merged and built into the ppa, would you be able to run your tests again? (I'm EOD here unfortunately) So that we can have more details tomorrow?07:26
veeberserr, this MR is: https://code.launchpad.net/~veebers/autopilot-qt/backout-latest-change/+merge/20388007:26
Mirvveebers: yes, keeping it in a tab and once merged I'll launch a rebuild07:30
Mirvthanks for working on it!07:31
veebersMirv: nw, I'm hoping this change should have us sorted.07:31
veebersMirv: Thanks for taking care of that, hopefully i'll have a goodnews email from you tomorrow :-)07:32
veebersRight, I'm off for the night. o/07:32
veebersMirv: oh I forgot, if that branch doesn't merge itself, you may have to hit-up CI (sorry to put this on your shoulders, auto-merge was going to be removed at some stage, not sure if it has or not) :-\07:34
veebersMirv: Ugh, I spoke to soon, already merged :-) Right it seems I really need to EOD then07:34
Mirvok :)07:35
tsdgeosSaviq: maybe add those little more steps you had to do in https://bugs.launchpad.net/bugs/1274289 ?08:23
ubot5Ubuntu bug 1274329 in unity8 (Ubuntu) "duplicate for #1274289 /usr/bin/unity8:11:g_shell_quote:upstart_app_launch_start_application:TaskController::start:Application::respawn:Application::setState" [Undecided,Confirmed]08:23
Saviqtsdgeos, https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1274329/comments/208:23
ubot5Ubuntu bug 1274329 in unity8 (Ubuntu) "/usr/bin/unity8:11:g_shell_quote:upstart_app_launch_start_application:TaskController::start:Application::respawn:Application::setState" [Undecided,Confirmed]08:23
tsdgeossomehow i didn't get that e-mail ^_^08:24
Saviqtsdgeos, it's a new bug that the other's a duplicate of08:24
Saviqtsdgeos, I created that from errors.u.c08:24
Saviqwonder why it didn't retrace well, though08:25
=== rsalveti_ is now known as rsalveti
tsdgeosah08:25
* tsdgeos hates testGenericScopeView, each time you add a new test it breaks for some unknown weird reason :(08:26
Saviq:/08:27
Saviqoh08:31
SaviqGoogle's selling Moto to Lenovo08:31
Saviqor, sold already08:32
Saviqhttps://news.google.com/news/rtc?ncl=dvNGXcwM6Di8mNMAt-C-70tYiRfrM&authuser=0&topic=tc&siidp=e20b3d5289f2842606b2b078d76dfee0fc7608:33
tsdgeosyeah08:33
tsdgeosyesterday's news :D08:34
tsdgeoslenovo's got too much money08:34
tsdgeosbought servers from ibm and phones from google08:34
Saviqtsdgeos, if they do with both what they did with the ThinkPads...08:38
Saviqthey'd have used their money well ;)08:38
tsdgeosSaviq: think so? i've been told by some ex-thinkpad users that quality went down but not price08:46
Saviqtsdgeos, oh well I meant well for Lenovo08:47
tsdgeosok :D08:47
Saviq;)08:47
Saviqdidrocks, hey, there seems to be something wrong with unity8 publishing - ports don't have any of the new releases http://pastebin.ubuntu.com/6842730/ :/09:11
didrocksSaviq: wait a little bit more, there has been an upgrade on the publisher machine09:14
didrocksthe cache was empty and other minor issues, which is catching up09:14
Saviqdidrocks, ok, bad timing, then09:15
didrocksSaviq: well, the whole archive was affected since yesterday09:16
tsdgeosshould we be able to run tests under xvfb-run?09:16
didrocksso not only you :)09:16
tsdgeosxvfb-run make testGenericScopeView09:16
tsdgeosfails09:16
tsdgeosi guess not :/09:17
Saviqmzanetti, one for you https://code.launchpad.net/~saviq/unity8/lazyimage-crop/+merge/20389709:22
* mzanetti is drowning in mps :D09:22
mzanettiwill do them today09:22
mzanettijust waiting for ci to pass again09:23
Saviqmzanetti, might be a long wait :/09:23
mzanettireally?09:24
Saviqmzanetti, we let https://code.launchpad.net/~unity-team/unity8/unity8.test_nested_mir/+merge/203088/comments/474588 through, which had two actual failures09:25
Saviqbut now it seems even worse http://s-jenkins.ubuntu-ci:8080/job/autopilot-testrunner-otto-trusty/2393/testReport/junit/unity8.shell.tests.test_hud/TestHud/test_hide_hud_dragging_Desktop_Nexus_4_/09:26
mzanettihmpf09:27
SaviqI'm worried there's a new ap release that changes the private bits09:27
mzanettiare we using them in our tests?09:28
Saviqhmm 20140123, not that new09:28
Saviqmzanetti, yeah, we have some helpers for dragging and such09:28
mzanettihmm... I don't remember the self.touch._device.drag() to require the "_device" part when I last used it09:29
mzanettiso yeah... that would confirm your suspicion09:29
Saviqotoh it passes locally09:30
Saviqand the last autopilot release is from 012309:30
Saviqso we'd know already09:30
tsdgeosuhoh09:35
tsdgeosunity8.shell.tests.test_system_integration.SystemIntegrationTests.test_networkmanager_integration(Desktop Nexus 4) failed in CI ?¿09:36
tsdgeosis that a new test or?09:36
* tsdgeos watches the videos09:36
tsdgeosvideo is 1s long09:36
tsdgeos:D09:36
tsdgeos/var/local/autopilot/autopilot.log: 09:04:53.139 DEBUG __init__:296 - Unity loaded and ready.09:37
tsdgeos/var/local/autopilot/autopilot.log: Not authorized.09:37
tsdgeosnot authorized?09:37
tsdgeosknowing what outputed that text would be amazing :D09:38
tsdgeoscan't find it in unity8 nor unity-mir09:39
tsdgeosah09:40
tsdgeos/var/local/autopilot/autopilot.log: CalledProcessError: Command 'pkcheck --action-id org.freedesktop.NetworkManager.enable-disable-network --process 883809:40
tsdgeos/var/local/autopilot/autopilot.log: ' returned non-zero exit status 109:40
seb128Saviq, what is this "contained" for OptionSelector? (wasn't ValueSelector already deprecated btw?)09:40
tsdgeosor09:40
tsdgeos/home/ubuntu/.xsession-errors: init: unity8 main process (8838) killed by TERM signal09:40
tsdgeosi guess this is the bad one09:40
Saviqtsdgeos, actually no, you've got mail09:41
Saviqseb128, no idea TBH09:41
tsdgeosSaviq: ah09:42
seb128Saviq, k09:42
tsdgeosok :/09:42
tsdgeosSaviq: have we told the ci guys? fginther is in deep sleep mode, no?09:46
Saviqtsdgeos, shouldn't be, he's in London this week I think09:46
tsdgeosah09:46
fginthertsdgeos, I'm awake09:49
fginthertsdgeos, Saviq, what's up?09:52
tsdgeosfginther: you've got a mail from mterry09:52
fginthertsdgeos, thanks, looking09:55
karniSaviq: Do you know what is "icon aspect ratio"? That term just showed up in the future dash spec in the new mascot section, so I had to immediately ask Katie to well define it :(10:02
mhr3sil2100, you at the office?10:02
Saviqkarni, we have an aspect ration for art already, that might be it, or maybe they've added aspect for mascot as well?10:03
karniSaviq: yes, for the mascot10:03
karniI asked to specify the mascot further.10:03
karniSaviq: I don't mean JSON. I mean what it means to be in "icon aspect ratio" :)10:04
karniMaybe she just meant 1.010:04
Saviqkarni, no, it's actually different10:04
karniSo it was a good call to ask her for clarification.10:04
Saviqkarni, it's 8x7.5 GU for the icons10:06
karniThank you!10:06
mhr3sil2100, done?10:24
=== _salem is now known as salem_
sil2100mhr3: yes10:31
mhr3Saviq, what does 12 ÷ 38 mean exactly? any value between 12 and 38?10:44
Saviqmhr3, yes10:44
mhr3Saviq, greyback would tell you that you use incorrect operators :P10:44
greybackdamn straight10:45
Saviqmhr3, |[12, 38]|, that better? ;P10:45
mhr3<12, 38>10:45
mhr3;)10:45
Saviqsome would write <12, 38>10:45
tsdgeos(12, 38) here10:49
tsdgeosSaviq: i don't think https://bugs.launchpad.net/unity8/+bug/1249255 is the bug i fixed the other day, see the video10:52
ubot5Ubuntu bug 1249255 in Unity 8 "click apps in Home scope disappear on flaky network connections" [Critical,Triaged]10:52
tsdgeosthat video says scope bug to me10:53
Saviqtsdgeos, yeah10:53
Saviqtsdgeos, I saw it yesterday again, too10:53
Cimichildrenrect is not affected by visibility of children?11:58
=== MacSlow is now known as MacSlow|lunch
karniCimi: I'm quite sure it is. when you make a child invisible, it still takes up space12:01
karniyou need to size the component down, for instance. which, I think, is a QML shortcoming12:01
Cimikarni, indeed12:01
tsdgeosMirv: ping12:01
Cimikarni, I need to calculate by hand12:01
karniinvisible and gone values of visibility, like on Android, would be useful12:01
Cimi:(12:01
Mirvtsdgeos: spong12:08
tsdgeosMirv: did you distro-patch https://codereview.qt-project.org/#change,76374 ?12:08
mhr3sil2100, yey, i see the merger works, it still runs the tests and everything, right?12:08
tsdgeosomg, someone wrote combo button in a spec!12:10
tsdgeos:D12:10
Mirvtsdgeos: yes12:10
tsdgeosMirv: you may want to replace it with https://codereview.qt-project.org/#change,76937 that seems what will be actually merged upstream12:10
tsdgeosMirv: or wait for 5.2.1 and then just drop mine12:11
tsdgeossince it seems the second will end up in 5.2.112:11
Saviqfginther, thanks for the update12:12
sil2100mhr3: yes, at least it should ;p12:13
Saviqkarni, well, it really depends, when you anchor something to an invisible item, is your anchor supposed to stop working?12:14
Saviqkarni, if you use any of the positioners, they will actually ignore invisible (and 0-width/height) items12:14
karniSaviq: That I don't know, I was only talking of visibility.12:14
karniSaviq: will they ignore invisible items of non 0-size?12:14
Saviqkarni, yes12:14
Saviqkarni, yeah, but "takes up space" is a vague term12:14
karniCimi: FYI ↑12:14
karniSaviq: I shouldn't have generalized.12:15
Saviqkarni, in 5.2 we'll get more automagic positioners like HBox, VBox12:15
Saviqkarni, so when we have that, you can use less and less anchors12:16
karniSaviq: unrelated -> http://paste.ubuntu.com/6843465/12:16
karniSaviq: great :)12:16
Saviqkarni, it's just that outside of those positioners, there's no automagic layouting, so "gone" would have undefined behaviors, really12:16
karniSaviq: yes, that's why Android has something like "anchor to parent when the anchor target missing" :)12:17
karniThat would be lovely.12:17
karniso when you make an item "gone" (invisible and not taking space), the other anchored component would then span the parent, for instance.12:17
Saviqkarni, so: anchor.left: foo.visible ? foo.left : parent.left ;)12:18
karniSaviq: Does that mascot fix make sense to you? that 4x size of mascot was "wyssane z palca" :D12:18
mhr3sil2100, awesome, thx12:18
karniSaviq: heheheh, right ;)12:18
Saviqkarni, yeah, quite arbitrary - Crop is needed, of course12:18
karniSaviq: at the same time, full image source is mem consuming12:18
Saviqkarni, but the *4 shouldn't be needed12:18
karnibut better than this totally blurish image12:19
Saviqkarni, try with Math.max(width, height)12:19
karniSaviq: that'll still blur. in this case (the screenshot), width == height == 8GU12:19
karniblur == boo :(12:19
Saviqkarni, yeah, thing is that blur is a bug in UbuntuShape, basically12:20
karni:O12:20
Mirvtsdgeos: it seems to work well enough as is however, nothing seriously different? I'll rather wait until 5.2.1 at least initially, enough stuff to do everywhere anyhow12:20
karniw00t. that's unrelated to the Image component?12:20
karniarrrgh12:20
Saviqkarni, please check if you get the same without the Shape12:20
karniok12:20
Mirvbut, I'll note it up anyhow that the patch needs to go12:20
tsdgeosMirv: sure, it's an equivalent fix, but if you don't drop it after 5.2.1 weird things may happen12:21
Saviqkarni, aaactually https://bugs.launchpad.net/unity-lens-applications/+bug/127115812:21
ubot5Ubuntu bug 1271158 in qtbase-opensource-src (Ubuntu) "Application Lens Icons are blurry, compared with home screen" [Undecided,In progress]12:21
Saviqtsdgeos, we don't have ↑ yet in our qt do we?12:22
tsdgeosSaviq: i think not12:22
* Saviq would like to get it to not have to work around it... but then do we care about 5.0 enough still...12:23
SaviqMirv, would you accept a backported patch for 5.0 still?12:24
SaviqMirv, or do we not touch 5.0 packages any more?12:24
karniSaviq: just read the bug reports. whoaa12:24
karniSaviq: *any* idea if we'll get that tsdgeos fix before mid Feb in ubuntu image?12:25
Saviqkarni, if we switch to Qt 5.2, yes, if we don't - maybe - that's what my question above is about12:26
karnisorry, didn't notice12:26
tsdgeosSaviq: honestly i don't even remember if i was too late for 5.2.1 or not12:28
* tsdgeos checks12:28
Saviqtsdgeos, says "stable"12:29
tsdgeosSaviq: yeah but stable may be 5.2.1 or not depending on when it got merged :D12:29
Saviqtsdgeos, 5.2.1 is "release" already?12:29
tsdgeosit is now12:29
tsdgeosbut not sure if it was when that got merged12:29
tsdgeosnope12:30
tsdgeos5.2.2 only ^_^12:30
tsdgeosso we'd still need to distropatch it12:30
Cimihttps://code.launchpad.net/~unity-team/unity8/unity8.previews_audioPlayer/+merge/20338712:37
Cimiready for review now12:37
Cimiwho knows my commits are 1 hour behind?12:37
MirvSaviq: sure, I could prepare it but according to normal test plan all AP tests should be run on device with that patch added and I really don't have time for that at the moment. but yes I could get the patch to the packaging branch and prepare package in PPA, for validation.12:39
tsdgeosSaviq: so in the preview image list, what will be my input in "sources"? a js array? a qt model?12:39
karnimhr3: that non-dconf-dash-pages -- the env var should list scope *names* concatenated by ; symbol, right?12:39
MirvSaviq: lp:~kubuntu-packagers/kubuntu-packaging/qtdeclarative-opensource-src_5.0.2 + lp:~kubuntu-packagers/kubuntu-packaging/qtbase-opensource-src_5.0.212:39
Saviqtsdgeos, array12:39
mhr3karni, right12:39
karnimhr3: tnx12:40
tsdgeosok12:40
mhr3karni, ie UNITY_SCOPES_LIST=scope-A;scope-onlinemusic12:40
karnimhr3:  :))12:40
MirvSaviq: I think that while we prepare everything for 5.2 we should also keep 5.0.2 good, so that it works as a panic button backup plan if we don't get 5.2 functional enough12:40
SaviqMirv, yeah, ok, I'll backport the patch, then12:40
karni\o/12:40
SaviqMirv, we'll need it for 5.2 as well12:41
karniSaviq: if I can help in any way, please let me know12:41
Saviqkarni, no need, it's a simple thing to do12:41
karni:)12:41
Saviqkarni, but still, Math.max(width, height) for both sourceSize.width and .height12:43
karniack12:43
MirvSaviq: tsdgeos: first real Unity8 AP results bug #127448412:43
ubot5bug 1274484 in Unity 8 "5 unity8 AP tests failing with Qt 5.2" [Critical,New] https://launchpad.net/bugs/127448412:43
Mirvthe previous bug was autopilot-qt problem12:43
SaviqMirv, \o/12:43
Saviqkarni, this way the in-mem image will always be not-smaller than the display12:44
karniCorrect! :)12:44
tsdgeosdamn, i should stop suggsting features to design and make my life harder D:12:44
karnitsdgeos: :D12:44
Saviqkarni, that's for cropping, for fitting it's correct to use actual image size12:44
Saviqtsdgeos, lol ;D12:44
* karni nods12:44
karniSaviq: and along the spec, mascot should be cropped. I'll get it ready in a moment.12:45
karni:)12:45
Saviqmhr3, can you guys please stop spamming us with -api and -shell MPs? I get like 20-30 emails every hour! :P12:46
SaviqCimi, s/artist/subtitle/12:47
CimiSaviq, subtitle???12:47
mhr3Saviq, i take it as an official request for us to stop working? ;)12:47
SaviqCimi, it won't always be artist12:47
karnimhr3: trololol :"D12:47
CimiSaviq, ok12:47
CimiSaviq, also fixing a test12:47
SaviqCimi, please group anchors, too12:47
CimiSaviq, just realised after testing12:47
CimiSaviq, ok12:48
=== MacSlow is now known as MacSlow|lunch
SaviqCimi, if you have if () in js, either wrap in { } or put it on one line12:49
SaviqCimi, 187+ if (sec.length == 1)12:49
Saviq188+ sec = "0" + sec12:49
Cimiok12:49
karniSaviq: what do we do with mascot "icon aspect ratio" :/ I saw you comment on that.12:50
karniround it ;D?12:51
karnij/k. we'd have to round resulting pixel dimentions12:51
karniI take it wouldn't look great when size has +0.25px12:51
Saviqkarni, I doubt you'll notice, QML is good at that12:52
karniSaviq: That's even though you said we should avoid these kind of values?12:52
karniWell, my sight aint perfect :D12:52
Saviqkarni, so let's do what they want us to do, but let's revisit12:52
karniok12:52
* karni will leave a comment12:52
Saviqkarni, +TODO please12:52
Saviqyup12:52
karni:)12:53
karniSaviq: any comments before I whip up a branch with MP? http://paste.ubuntu.com/6843643/13:01
karniuhhh.. I messed up one digit13:01
karniheight: units.gu(5.625) :|13:01
karnifixed: http://paste.ubuntu.com/6843648/13:02
Saviqkarni, yeah, looks good13:02
karnithanks13:02
Saviqkarni, with cropping, you might also want to set alignment13:03
Saviqkarni, http://qt-project.org/doc/qt-5.0/qtquick/qml-qtquick2-image.html#horizontalAlignment-prop13:03
karnigood catch!13:03
=== alan_g is now known as alan_g|lunch
CimiSaviq, ok, better now13:07
Cimifixed tests and coding style13:08
Cimitsdgeos, as reference, for your gallery13:10
Cimihttps://code.launchpad.net/~unity-team/unity8/unity8.previews_audioPlayer/+merge/20338713:10
=== MacSlow|lunch is now known as MacSlow
=== dandrader is now known as dandrader|afk
=== dandrader|afk is now known as dandrader
karniSaviq: awaiting +1 from Katie (pinged her on IRC), proposed here: https://code.launchpad.net/~unity-team/unity8/unity8-fix-mascot/+merge/20396713:49
=== alan_g|lunch is now known as alan_g
Saviqkarni, k thanks13:50
Saviqtsdgeos, how do I find out which git rev was your fix merged in?13:52
Saviqah 8cf981113:53
SaviqMirv, hum, quilt push -a fails in qtbase_5.0.213:55
Saviqand in lp:~kubuntu-packagers/kubuntu-packaging/qtbase-opensource-src, too? wth?13:57
tsdgeosCimi: yep, was already having a look at that MR14:07
karniSaviq: Katie +1'ed the change, updating MP description.14:17
karnidone14:17
kgunn_Saviq just letting u know, still in dr office, gonna miss stand up14:20
Saviqkgunn_ k14:22
=== karni is now known as Guest17108
Saviq@unity: standup14:30
mterryyay, it worked!14:31
mterrykgunn, dandrader: in case you didn't switch ^14:31
Guest17108tsdgeos: Cimi: I just wanted to mention we need (Phone delivery, for MWC in Feb) as basic previews as picture + single Buy button :)14:40
Guest17108lol wtf :D /me is karni14:41
=== Guest17108 is now known as karni
Saviqkarni, tst_CardHeader.qml: bad whitespace in line 9814:46
Saviqkarni, in the mascot fix14:46
karnid'uh :)14:46
karniSaviq: is there a tool you guys use to avoid these kind of things :D?14:47
Saviqkarni, yeah, "make test" ;)14:47
karniSaviq: oh, wasn't aware that checks white space as well. I only ran testCard and testCardHeader for this branch. my bad.14:48
karnioh yeah :D there it is14:48
Saviqkarni, it will probably complain about stuff in your working dir14:48
Saviqkarni, like in builddir/14:48
Saviqkarni, you can obviously ignore that14:49
karninope, it was all good. pushed14:49
karniWill make sure to run full suite next time.14:49
Saviqmterry, can you please skip the test instead:15:08
Saviqhttp://docs.python.org/2/library/unittest.html#skipping-tests-and-expected-failures15:08
mterrySaviq, sure15:09
mterrySaviq, done15:11
Saviqmterry, thanks15:11
Saviqsil2100, unity8 ready for publishing15:17
Saviqdidrocks, ↑15:17
sil2100Saviq: looking15:17
didrocksSaviq: acked15:19
Saviqdidrocks, thanks15:19
sil2100Saviq: published, be ready to clean and merge once it gets out of proposed15:20
Saviqsil2100, yup15:20
dednickSaviq: is the bfb to apps just changing the icon to open app scope rather than home scope?15:21
Saviqdednick, yeah15:21
Saviqdednick, and please do that against lp:~unity-team/unity8/new-scopes15:22
Saviqdednick, or well, you probably can't do it yet15:22
Saviqdednick, 'cause there isn't an apps scope yet...15:22
dednickSaviq: ah, i c.15:22
Saviqdednick, I'll let you know when it's ready15:22
dednickSaviq: ok15:22
Saviqmterry, kgunn, infographic can already be mocked per-user can't it?15:25
mterrySaviq, yup.  You use the usermetricsinput command.  We may need some guidance from pete-woods on how to drive it though15:25
mterrypete-woods, can I use usermetricsinput for users that don't exist?  We're looking to create some dummy data for a demo15:26
pete-woodsmterry: no you can't15:26
pete-woodsit verifies everything using apparmor15:27
kgunnpete-woods: your such a downer :)15:27
mterrypete-woods, ok.  So we'll have to make users for these names15:27
pete-woods:)15:27
mterrykgunn, ^ easy enough15:27
kgunnsounds good15:27
pete-woodsthe command has help, but basically it's something like15:27
kgunnmore than they asked for, so it'll be a nice touch15:28
pete-woodsusermetricsinput data-source-id "%1 bananas today" "no bananas today" <number> <number> ….15:28
pete-woodswhere the data-source-id has to be unique15:29
karnimhr3: ssweeny asked when we can expect your fix-jsong-merging and non-dconf-dash-pages land in archive15:29
cwaynepete-woods: usermetricsinput is in the build by default?15:30
cwayneif so, we can drop the shenanigans we used int he custom image to fake data15:30
pete-woodscwayne: probably not, it's in the libusermetrics-tools package15:31
pete-woodscwayne: just confirmed that it's not15:32
cwaynemakes sense15:33
cwaynethanks pete-woods15:33
pete-woodsnp :)15:33
dandraderSaviq, 1- how/where is "copy&paste" supposed to work. 2- did it ever work?15:38
Saviqdandrader, it worked with surfaceflinger for sure, it's in qtubuntu15:39
Saviqricmm, can you point dandrader further ↑?15:39
dandraderthe display of my nexus 10 turns off and does go back on if I press the power button or "powerd-cli display on bright"15:40
ricmmdandrader: qtubuntu has an implementation that makes use of the platform-api15:41
ricmmthe platform-api backend used to go over binder, through the android app manager, to the other end15:41
ricmmnow, it should go either through an IPC impl that unity8 owns (dbus, socket) or through Mir at a lower level15:41
dandraders/does go/does not go15:43
mhr3karni, when C28 in the landing sheet says "landed" :)15:43
ricmmdandrader: is any of what I said relatively clear/familiar to you?15:44
karnimhr3: you have a row :D thanks15:44
dandraderricmm, enough to get started, I think.15:45
dandraderricmm, and how is the UI for copy and paste? should long pressing a selection show a pop up with a "copy" option?15:45
ricmmthat exists right now15:45
ricmmthe qtubuntu and up components are existing15:46
ricmmwhat needs re-implementing is the plumbing that sits behind scenes in p-api backend15:46
dandraderricmm, but it behaves as I explained above?15:46
dandraderthat was my question15:46
ricmmyes, give it a try15:46
ricmmopen browser and see for yourself15:46
ricmmtheres a quick list that pops up15:46
dandraderricmm, I only see "select word" "select all"15:46
ricmmwell you first need to do the selection15:47
ricmmtry on a picture15:47
dandraderricmm, btw, is there a sure-fire way to turn on the display15:47
dandrader?15:47
dandraderI started having troubles with it since yesterday...15:47
ricmmpressing the button should do it15:47
ricmmif it fails, file a bug15:47
ricmmheh15:47
* dandrader reboots his N1015:47
dandraderweird...15:48
dandraderok, something is very wrong. I'm getting some black flicker on my screen when playing with unity8....15:50
ricmmlow battery15:51
ricmmn10 flickers to black when running out of battery15:51
dandraderso the indicator is buggy, it's showing full battery, let's check via command line15:51
dandrader"upower --show-info /org/freedesktop/UPower/devices/battery" shows 100% full15:52
ricmmthats weird then15:52
dandraderok, and after the display goes off due to inactivity it doesn't come back unless I reboot15:53
dandradersigh, better go for lunch now15:53
=== dandrader is now known as dandrader|lunch
tsdgeosahhhhhhhhhhh, mako tests failing now?¿?¿15:58
tsdgeoswhen did that start?¿15:58
* tsdgeos cries in his corner15:59
tsdgeosSaviq: any idea ↑↑↑16:03
Saviqtsdgeos, https://code.launchpad.net/~saviq/unity8/bring-back-unity-mir-dep/+merge/203963 probably16:04
tsdgeosah, makes sense16:04
Saviqtsdgeos, just merging16:04
tsdgeos\o/16:04
Saviqtsdgeos, restart, should be good now16:05
Saviqkarni, first stab at CardTool: https://code.launchpad.net/~saviq/unity8/card-tool/+merge/204019, it's not completely good yet and not enough tests, but would like an early proofread if you have some time16:19
karnibrb otp16:19
mhr3sil2100, /me wants a silo16:24
sil2100mhr3: hi! So, the silo is not assigned for now since we first want to check the last image16:27
sil2100mhr3: no worries, I'll try to assign one, just we need consult the current situation first16:27
sil2100mhr3: (all under control)16:28
karniSaviq: sure, looking :)16:28
mhr3sil2100, ok, also for some reason one branch approved for devel is not merging, i wonder why16:28
=== greyback is now known as greyback|food
sil2100mhr3: uuuuhh... shit's going down I see16:30
sil2100mhr3: ok, let me look at that in a moment with fginther16:31
tsdgeostedg: pete-woods: any idea when the hud that supports quit is going to be in trusty?16:31
sil2100seb128: merge conflict in your branches in the CITrain it seems?16:31
thostr_tsdgeos: working on it...16:31
pete-woodstsdgeos: I'm adding the stuff to the landing sheet imminently16:31
seb128sil2100, yeah, cf #ubuntu-desktop, being resolved16:31
tsdgeospete-woods: thostr_: ok, tx16:31
seb128sil2100, thanks for the warning though ;-)16:31
seb128warning -> ping16:32
tintouhi all, I need libunity to use libgee-0.8 to have it working on my project (see https://code.launchpad.net/~tintou/libunity/fix-gee/+merge/203152 ) do you know if it is possible to have it merged any soon ?16:33
sil2100seb128: np ;)16:35
mhr3tintou, and other people would need libunity to use gee-1.0, the only sensible fix is to remove all usage of gee from libunity16:35
tintoumhr3: why would they prefer gee-1.0 over 0.8 ?16:36
mhr3tintou, because it's 0.2 better? :)16:37
tintoumhr3: it's not, they have packaged it wrong16:37
mhr3tintou, my point stands, the sensible fix is to remove gee16:37
mhr3completely16:38
seb128mhr3, you g hater!16:38
seb128;-)16:38
mhr3seb128, yea, let's get rid of glib too, it breaks things too often :P16:38
seb128lol16:39
seb128mhr3, are you at the office next week?16:39
mhr3seb128, yea, i'll come at least twice :)16:39
Saviqmhr3, why did you decide staging branch in the end?16:39
seb128mhr3, great, desktop is going to be there, looking forward seeing you ;-)16:40
mhr3Saviq, >16:40
mhr3One extra branch I created is lp:unity-scopes-api/staging which will be used for pushing changes from the devel branch to trunk (if we tried to do the merge directly from devel to trunk, we'd have to freeze devel for the entire time that landing to distro is happening, which is currently just too long).16:40
Saviqmhr3, ah so you had a devel branch already16:40
Saviqmhr3, we decided to go straight into trunk16:41
mhr3Saviq, yes, devel is our new trunk, staging is devel-about-to-land16:41
seb128you guys are crazy16:41
seb128landing doesn't take that long16:41
mhr3right :P16:41
mhr3seb128, i requested landing 2hours ago, didn't even get silo yet.. build will take about 2 hours, testing + publish another ~5hours16:43
tedgmhr3, Thinking we'll do the same thing for other projects.  Only thing that makes sense really.16:45
tedgmhr3, Did you make a "devel" series?16:45
tedgmhr3, Thinkings we should make the names consistent.16:45
seb128mhr3, build is taking 2 hours ?!16:45
Saviqmhr3, and testing 5 hrs? are you running the whole suite for all projects?16:45
tedgseb128, It builds the source package, and then goes to PPA.16:45
seb128tedg, right, for u-s-s the roundtip for everything is like 1.5h, that includes build, ppa, publish to archive, migrate to release and merge back to trunk16:46
tedgTook UAL about 20 hours.16:46
seb128I'm doing my 3rd landing of the day16:46
* Saviq just went from MP to release in 2.5h, and wasn't really paying attention16:46
seb128tedg, what step was slow? getting a silo because CI guys were sleeping when you made the request?16:47
mhr3tedg, yep, devel and staging16:47
Saviqseb128, I see where you're going ;D16:47
tedgseb128, A bunch of things, but yes, the multiple requests were an issue.  And the builders all died in the middle.16:47
mhr3Saviq, includes migration from proposed16:48
Saviqmhr3, yes16:48
seb128tedg, ok, publisher being down because of the box being upgraded to precise is not something you can associate to the workflow16:48
tedgseb128, Remember we can't "do" anything.  We can just get to a step and ask someone else to do it.  You have permissions to do things.16:48
mhr3Saviq, no idea why it takes that long, but that's how it is16:48
seb128*can't*16:48
seb128mhr3, how long does it take to build your source on an archive builder?16:48
seb128mhr3, unity-scopes-api takes 7 min16:49
Saviqmhr3, there's definitely steps to improve (automated ap runs, "get-silo-on-my-phone" app and such), but I've done three unity8 landings in three days and have been quite happy16:49
seb128mhr3, a complete roundtrip should be around 1h16:49
mhr3seb128, the build itself is fast, getting all the builds on all archs, waiting for available builders... waiting for migration from proposed, two days ago all that took 7hours16:50
Saviqmhr3, really took maybe .5h of *my* time (that includes going through a fairly extensive test plan for unity8)16:50
Saviqwe should gather stats16:50
Saviqdidrocks, ↑16:51
seb128mhr3, you got unlucky it seems, maybe builders were DoSed by a chromium security update or something16:51
seb128mhr3, I've done some ~10 landings now and my experience doesn't match yours16:51
didrocksSaviq: right, that's part of the Airline plan16:51
didrocksI have no time for the train16:51
didrocksalready many requests :p16:51
Saviqdidrocks, 'stood16:51
=== greyback|food is now known as greyback
tedgI think the stat that is the most significant is "how long from a simple patch to in development archive" -- seems that it's still easier to get an SRU.16:53
Saviqtedg, 2.5h for me today - and could've been quicker if I paid attention16:54
seb128tedg, 1.5h for me on u-s-s (which is not too slow to build)16:54
seb128that's a lot better than SRU16:54
* Saviq afk's, will be back later to have some quiet time with the laptop... when all of you are asleep :P16:55
seb128Saviq, have fun!16:55
tedgseb128, Saviq, hope that happens, doubt it, but we'll see.16:57
seb128tedg, that what happens?16:58
tedgseb128, That it takes that long to land an MR.16:58
seb128tedg, the 1.5h roundtrip for u-s-s is the current situation, not something futur16:58
seb128the main issue is that somebody from CI needs to be around to give you a silo, but they know about that issue/have plans to do better16:59
tedgseb128, Apparently no new silos are being allocated right now.17:00
tedgseb128, I'll wait.17:00
seb128tedg, yeah, that's a policy issue rather than one with the technical part of the system17:01
seb128they decided to put things on hold for some reason17:01
tedgThe system and the policies it enforces are indistinguishable from each other.17:01
seb128they sort of are17:03
seb128it's like saying that things are slow to land in Ubuntu because during beta freeze there is a freeze17:04
* tedg misses commonly agreed upon and discussed freezes17:04
asactedg: go through your lander17:06
seb128yeah, me too, but apparently we didn't find a better way to prevent things to get out of control17:06
tedgReally?  So you think 12.04 was "out of control" ?17:06
seb128asac, the issue is not the lander, is that something the line stop because you guys want to get back to full control before allowing changes again17:07
seb128tedg, no, but we didn't have the same standards about the devel release than the one currently enforced on the touch images17:07
asacyes, thats normal business of maintaining a continuously green baseline.17:08
asacget over it17:08
asacin turn we have all upstream engineering working on trunk/devel release and we dont end up having painful integration mess in the end17:08
tedgYes, we maintained a continually green baseline on 11.10 while we worked on 12.04.17:08
tedgasac, Yes, we delay the painful integration mess.17:09
ricotzmhr3, hi, do you have a moment17:17
mhr3ricotz, sup17:18
ricotzmhr3, since the session-indicator isnt really smart about checking if unity is actually running, is it feasible to add some envar-support to enable libunity?17:19
ricotzor even add a check for plank ;)17:19
mhr3ricotz, libunity has api to check for unity actually17:19
mhr3inspector iirc17:20
ricotzmhr3, yeah, it simply looks for the presence of com.canonical.Unity17:20
ricotzwhich plank provides to enable it17:20
mhr3ricotz, ok, then i didn't get what you're after17:20
ricotzbut indicator session is doing so too for using the unity-session-dialogues17:21
ricotzi was thinking about something like http://paste.debian.net/plain/79279 to avoid the need to mimic the unity dbus17:22
mhr3ricotz, so you're running indicators in non-unity environment, but the unity dbus name is provided by someone because of... something and that breaks something else? :)17:22
ricotzhehe, kind of ;)17:22
ricotz(even http://paste.debian.net/plain/79280 )17:23
mhr3ricotz, i suppose the primary reason is that plank wants to provide progress indicators and numbers on the icons?17:24
ricotzthis is regarding the count/progress/menu support for dock-items17:24
ricotzright17:24
ricotzso having a session-variable to overrule the dbus-check would be nice17:25
mhr3wondering what would be a nicer solution17:26
ricotzor could this lead to anything else breaking17:27
mhr3owning a separate name for the "notifications" would be nicer17:28
ricotzhmm, this would do it too17:28
ricotzif you prefer a dbus solution17:29
mhr3ricotz, ok, so the launcher items do not emit any notifications if unity is not running, i'd say that it should check for unity *or* X and work if either is present17:34
ricotzmhr3, do you want to look into it while you are looking for a nice name, or do you prefer a patch proposal17:34
mhr3ricotz, patch pls :)17:34
ricotzmhr3, exactly, that is what i mean to look for plank ;)17:34
ricotzwhat name would you look for "com.canonical.Unity.Notifications"? is this taken?17:35
mhr3ricotz, yes, would be better to use something neutral (non-canonical)17:36
ricotzor "net.launchpad.plank"17:36
mhr3doesn't seem too neutral17:37
mhr3the last part anyway, the first two are fine17:37
ricotzright, but isn't it suppose to be an owned domain17:37
mhr3it might be linking to blueprint on lp17:38
mhr3or something like that17:38
ricotzthis would target a specific project too17:39
mhr3ubuntu?17:39
ricotzhmm, no idea yet17:41
karniSaviq: hey, got side tracked, sorry. I'll grab lunch (~1h) and will gladly review your branch, would that work?17:57
Saviqkarni, no worries, it's not ready for real review yet anyway17:59
Saviqkarni, lunch, btw? where are you working from these days/17:59
Saviq?17:59
karniSaviq: Warsaw. I tend to start work 10-11 AM. And I have a screwed up eating rhythm :D18:00
karniSaviq: that way, I have bit better cross over with the US guys on my team18:00
Saviqkarni, well, yeah... but lunch at 7pm ;)18:00
karnialso, when I occasionally EOD between 2-5am, it'd be hard to start the time you guys do ;D18:00
Saviqkarni, I understand breakfast at 4pm (happens to me all the time) ;)18:00
karnihahahh18:00
karniI take that my lunch is quite early haha18:01
=== karni is now known as karni-food
=== Wig is now known as Wug
=== alan_g is now known as alan_g|EOD
=== dandrader|lunch is now known as dandrader
=== salem_ is now known as _salem
=== _salem is now known as salem_
mterrySaviq, description of that disable-nm-test updated, whoops18:36
ricotzmhr3, maybe like that http://paste.debian.net/plain/7931218:38
Saviqmterry, nw18:40
ricotzmhr3, regarding the gee-0.8/gee-1.0 problem, why is gee-1.0 mentioned in pkg-config files while it isnt used in the public api?18:41
mhr3ricotz, would be awesome if you factored out a DBusNameWatcher class and make the Inspector just use new DBusNameWatcher("com.canonical.Unity"), and LauncherEntry would then check both Inspector and it's own DBusNameWatcher("net.launchpad...")18:42
mhr3ricotz, forgot to remove i guess...18:42
ricotzhmm, i see18:43
ricotzi wasnt going for a rewrite of this part though ;)18:44
mhr3ricotz, sorry, i like clean solutions :P18:46
mhr3anyway.. heading home18:47
ricotzmhr3, ttyl18:48
dandraderricmm, for the record: a full device reset did solve my "black flicker" + "display doesn't go back on again" issue18:49
=== karni-food is now known as karni
dandradergreyback, is the side stage work still pretty much a WIP? I'm getting that vertical shadow separator in front of fullscreen apps even though there's nothing on the side stage. is that expected at this point?18:59
greybackdandrader: there are quite a bugs yeah18:59
greybackdandrader: can I ask, what version of upstart-app-launch have you installed?19:00
dandradergreyback, 0.3+14.04.20  just flashed my device19:01
greybackdandrader: ok, then it has the latest changes I was waiting for (some side stage parts were broken until it lands)19:01
greybackkgunn: if you want to start logging side stage bugs, the latest image (155) is one to choose.19:04
kgunngreyback: ok...getting my n10 sorted now19:04
=== boiko_ is now known as boiko
=== salem_ is now known as _salem
kgunnmterry: greyback ...so i don't see how you guys want displayConfigOutput.orientation19:52
mterrykgunn, lp:mir/devel gained a new parameter19:52
kgunnat least i thot the signature came from display_configuration.h from mir ?19:52
kgunnmterry: right...but we need to land 0.1.4 first19:53
mterrykgunn, I thought 0.1.4 was mir/devel still19:53
kgunnmterry: so you're not wrong...just early :)19:53
mterrykgunn, OK you know what's up then  :)19:53
kgunnmterry: yeah...we really have to snapshot for history and such, so 0.1.4 is the dev-branch equiv of r133019:54
mterrykgunn, aw bummer.  I want r135319:54
kgunnmterry: dude you and me both....19:55
mterryso that means next release will also be a SONAME bump and take a while19:55
kgunnmterry: first xserver broke, then platform api, now mir won't pass unit test ....but it was yesterday...and this is all on the new ci train19:55
kgunnmterry: well...as soon as i can get 0.1.4 in ...we'll do 0.1.5 right after19:56
kgunnin fact i may just ask duflu to queue it up tonight19:56
mterrycool19:56
dandraderany idea what piece of software does the work of holding the clipboard contents?19:57
dandraderin ubuntu19:57
dandraderis it the xserver itself?20:00
* mterry reboots20:02
=== dandrader is now known as dandrader|afk
Saviqkarni, https://code.launchpad.net/~saviq/unity8/improve-card-cardheader-layouts/+merge/20409220:10
* karni looks20:10
Saviqdandrader|afk, in real ubuntu, yes20:10
karniSaviq: just branched your card-tool for a spin20:10
Saviqkarni, wait for it, got an update20:10
karniSaviq: 82-85 - adding semicolons only inflated diff size. was this intentional?20:15
Saviqkarni, yeah, semicolons in JS === good20:15
karniSaviq: gotcha20:16
Saviqkarni, didn't want a separate MP20:16
* karni will remember20:16
Saviqkarni, resubmitted with prerequisite, btw https://code.launchpad.net/~saviq/unity8/card-tool/+merge/20409320:17
karniSaviq: would you like to add 'MP Submission Checklist Template' :)?20:18
Saviqkarni, indeed20:18
karnion improve-card-cardheader-layouts20:18
karnihrm. ↑ that one is taking a while to update the diff.20:21
karniSaviq: Just FYI, I know this is WIP. testCardTool Totals: 4 passed, 4 failed, 0 skipped20:23
=== dandrader|afk is now known as dandrader
karniSaviq: FYI recent comment of Katie on the spec doc VS your tryCardTool last test case: "without art a horizontal card doesn't exist"20:25
karniSaviq: I don't see a reason why, but that's what she said.20:26
karniI guess she meant if there's no art, there should be mascot. *BUT* if you have news feed, and only want text?20:26
Saviqkarni, very simple why20:26
Saviqkarni, horizontal and no art == vertical and no art ;)20:26
Saviqkarni, h vs. v is only about header moving around art20:26
Saviqkarni, so if no art20:26
Saviqkarni, on tests, yeah, still to-do20:27
karnipfff of course :))20:27
Saviqkarni, pushed a few visual tweaks to the improve layouts branch20:27
dandraderSaviq, any preference/opinion on what you would like to see holding the clipboard contents in the brave new mir+unity8 world?20:27
karniSaviq: ack20:28
Saviqdandrader, Mir is supposed to have a minimal default implementation, but we might extend on it through ContentHub20:28
dandraderSaviq, hmm, I thought mir wouldn't care about it20:29
Saviqdandrader, Mir is supposed to be a tool to build shells, if every shell needs to reimplement something as simple as a copy/paste buffer...20:30
Saviqdandrader, then we have Wayland ;)20:30
Saviqkarni, put checklist in20:30
karniok20:30
Saviqkarni, sorry, --overwritten card-tool20:31
karniok20:32
karnibrb otp20:32
karni(ubuntu support lol ;) )20:32
kgunndandrader: Saviq ...mir would provide the mechanism, but shell would need to arbitrate through content hub right?20:33
kgunni mean eventually...not for mwc demo20:33
kgunnbut some app could conceivably say ...don't copy my text right ?20:33
Saviqkgunn, it's not the apps' text, it's the user's20:34
kgunnso that's a bridge too far20:34
Saviq;)20:34
Saviqkgunn, but anyway, with contenthub it'll be unity8/unity-mir that will extend the basic c/p buffer that should have a default implementation in Mir, I'd say20:35
Saviqkgunn, contenthub is ubuntu-specific, Mir is not meant to be20:35
Saviqkgunn, so yeah, Mir won't know about CH20:35
kgunnSaviq: but to the point of text, i guess we're just counting on secure input keeping someone naughty from doing a text highlight & screen scrape?20:36
dandraderkgunn, so now I'm supposed to just hack away "something that works"?20:36
Saviqkgunn, oh yeah, nothing without user interaction20:36
Saviqdandrader, yeah, did ricmm point you at what worked in qtubuntu?20:37
dandraderSaviq, qtubuntu calls platform-api which used to use android's clipboard service20:37
Saviqdandrader, can we not use it still? is it bound to surfaceflinger? or did we drop that service?20:38
dandraderSaviq, now it's about replacing that android clipboard service usage in platform api with something else20:38
Saviqdandrader, now it's about getting it to work again, I'd say, with as little effort as possible?20:38
dandraderricmm, so, we can no longer use android's ClipboardService, right?20:48
Saviqdandrader, he's in London, doubt he'll answer20:51
dandraderah20:51
dandraderSaviq,  well, from what I understood, his suggestion is to make platform-api now talk via some IPC (new socket, dbus, mir) to unity8 and have unity8 being the guy that stores the clipboard value for now20:54
Saviqdandrader, sure, we even have the 'mir-opaque' comms channel for that20:59
Saviqdandrader, that's meant to allow things talk through the mir socket without it knowing what the conversation is about21:00
Saviqdandrader, if it's impossible to use the old approach, then yeah, we need to build something new that's as little work as possible21:00
karniSaviq: these fonts are suspiciously small I'm afraid. title font is smaller than summary..21:03
Saviqkarni, ah well, summary should probably be even smaller21:03
karnishit. stuff will get really small.21:03
Saviqkarni, https://docs.google.com/a/canonical.com/file/d/0B-a_7E3tDxOgOHAwNEt1S2Zfa2s/edit21:04
karniSaviq: I'm right there21:04
karniHeader Text Styles21:04
karniXXS maps to "small"?21:04
Saviqkarni, xx-s21:04
karnititle should be XXS. your code has 'small' for titleLabel21:05
Saviqkarni, I was trying to match the visuals as close as possible, wasn't looking at the spec at the moment...21:05
karniaha21:05
Saviqkarni, I'd say folks don't know what xx-small means, then...21:05
karniSaviq: man.. I really have mixed feelings about those font sizes :( Can we either make summary the size of title font, or not change them for now.. ?21:06
Saviqkarni, sure, I didn't look at summary, sorry about that21:06
karniI almost need a magnifying glass to read these21:06
karniSaviq: thank you21:06
Saviqkarni, well, on the phone?21:06
karniSaviq: At least we'll have consisten look21:06
Saviqkarni, GRID_UNIT_PX=12 and you'll be set ;)21:06
karni:D21:06
Saviqkarni, stuff's smaller on desktop by default21:06
Saviqthan on phone21:06
Saviqespecially if you have a high-density display21:07
karnihrm :D that's much better21:07
karniIn that case, can you just fix the summary font size?21:07
karniI still think subtitle will be ridiculously small :D but that's easy to change, if need be.21:08
Saviqkarni, looking at https://docs.google.com/a/canonical.com/file/d/0B-a_7E3tDxOgQ002UmQxbjJmMHc/edit summary is to be same size as title, so will make it "small"21:10
karniSaviq: last question. tryCard 2nd test case "ARt, header, summary - vertical, small" - out of "foo" title, only 'f' is displayed. While unrelated to your MP at all, I think that makes so little sense.21:10
karniSaviq: yes, thank you21:10
Saviqkarni, sure, mascot will probably be forbidden in small21:10
karniack21:10
karnimakes sense21:10
Saviqkarni, we'll build those kind of rules in later21:10
* karni nods21:10
Saviqkarni, as in the scope tool needs to tell you that a certain construct is disallowed21:12
karniSaviq: I had an initial look at your card-tool. I like it, certainly good direction. I'd like to read it more thoroughly though, and I'll have to leave soon, I'm sorry. I'm happy to do that first thing tomorrow :)21:13
karniSaviq: yeah21:13
Saviqkarni, and we'll try and degrade gracefully21:13
Saviqkarni, no worries, hopefully will be complete by the time you're back tomorrow21:13
karniSaviq: that + the API needs to console.warn if such construct is used anyway, and degrade gracefully, like you said21:13
karniSaviq: neat. do you ever sleep? ;P21:13
Saviqkarni, well, console.warn is kinda useless... it's client-side, so no one will see it21:14
karniSaviq: don't even tell me if you have a girl/wife haha!21:14
Saviqkarni, sometimes, and yes, 6 years married ;)21:14
karniSaviq: devs will! (or should be able to)21:14
karniSaviq: :DD21:14
Saviqkarni, we want to pass those as errors up to errors.ubuntu.com or somewhere21:15
Saviqkarni, so that scope authors will get an aggregate view21:15
Saviqkarni, but yeah, there are plans21:15
karniniiiiiiiice :>21:15
karnilike those plans!21:15
karniSaviq: I'll disappear for a moment. If you push this comment font size fix, I'll approve before I leave :)21:17
Saviqkarni, didn't I already?21:17
* karni looks21:17
Saviqkarni, yeah, it's there, sorry didn't ping21:18
Saviqkarni, wrong21:18
Saviqsec21:18
karnirev 67521:18
Saviqkarni, now21:18
karniok21:18
karniSaviq: I assume that jenkins failure is known issue?21:20
Saviqkarni, already fixed in another branch21:20
karniSaviq: thank you, +1 :)21:21
karniCatch you tomorrow, and get some sleep as well! :)21:21
Saviqkarni, o/21:21
karni\o21:21
karniSaviq: sounds like I overdid it. I shall remove those tests from unity8-fix-mascot21:45
Saviqkarni, weren't you AFK?21:46
karniSaviq: let's say I'm back for a moment before I leave (or not, might change plans)21:46
Saviq;)21:46
karniSaviq: In any case, although I added them *after* code, I felt that was the TDD way heh.21:46
karnieven if these were visual properties, but you're right21:47
Saviqkarni, problem with that then is that you need to update those value in the test and in the code, but that's not meaningful, really21:47
karniSaviq: yeah, I could get into lengthy discussion about it (tests are documentation, blah blah :D), but yea. I'll remove em.21:48
Saviqkarni, when you come up with a way to measure coverage for QML, we can talk again ;P21:49
karnihaha21:49
* karni notes for 10% time project xDDD21:49
karnibut seriously, I have a better thing in mind. wireframing in qmlscene. would be so useful *not* to use transparent rectangles in code to figure out what's wrong in the UI.21:50
Saviqkarni, well... http://doc.qt.digia.com/qtcreator/creator-debugging-qml.html21:53
Saviqkarni, unfortunately it's been pretty unstable recently21:53
karniSaviq: Bookmarked to Read :)21:53
karniI would sure do my research first, that was just an idea :)21:54
karniSaviq: FYI updated https://code.launchpad.net/~unity-team/unity8/unity8-fix-mascot/+merge/20396722:04
Saviqkarni, cheers22:04
karnicheers22:04
Saviqkarni, Cannot assign to non-existent property "verticalAlignmnent"22:06
SaviqETOOMANYNS22:06
karniSaviq: sorry. I'll get back to you tomorrow then.22:06
Saviqkarni, just a typo22:06
Saviqkarni, anyway, have a good night!22:06
karniyou too!22:07
karnithanks22:07
karniPushed. For gods sake, mascot, go away :D22:10
Saviqmzanetti, go away22:50
kgunnyou go away22:53
mzanetti:D22:53

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