elopiothank you veebers and thomi.01:24
=== jono is now known as Guest35005
=== duflu_ is now known as duflu
tsdgeosSaviq: read the description!08:12
Saviqtsdgeos, sooorry :D08:12
tsdgeosSaviq: i tried and failed08:13
tsdgeosthing is that this is executed not in sync with any call you do, so it's even hard to add a qcompare to -108:13
tsdgeosi was hoping i could make it crash or act weird08:14
tsdgeosbut couldn't08:14
tsdgeosmaybe i'm also defending with count() == 0 or isEmpty in other parts of the code08:14
tsdgeosbut still shouldn't be 008:14
tsdgeosSaviq: so did i convince you in https://code.launchpad.net/~aacid/unity8/timeformattertest_LCALL_C/+merge/200991 ?08:18
Saviqtsdgeos, not really, especially since currently 100% of our tests should run under LC_ALL=C08:19
Saviqtsdgeos, as we don't have any place that should be affected by the locale08:20
tsdgeosSaviq: ok, so i can move it to cmake and update the CODING file?08:20
Saviqtsdgeos, gimme a few to think what we want there08:23
Saviqtsdgeos, I've a feeling all our tests should run under LC_ALL=C currently08:24
tsdgeosshould as "they do" or as "we would like to do"08:24
Saviqtsdgeos, can you still run make -C builddir testfoo to run that test?08:24
Saviqtsdgeos, we would like them to08:25
tsdgeosSaviq: no, can't find a way to run it individually08:27
tsdgeosonly make test08:27
tsdgeosor run the binary08:27
Saviqtsdgeos, right, so that's something we should make possible08:27
tsdgeosah no08:27
tsdgeosmake timeformattertest08:27
tsdgeosbut doesn't get autocompleted by the bash make autocompleter08:28
tsdgeosmy bad08:28
Saviqtsdgeos, should've used ninja ;)08:28
tsdgeosi get no autocompletion at all with ninja08:28
Saviqtsdgeos, oh?08:28
tsdgeosand actually it gets autcompleted now :D08:28
tsdgeosdon't know, just woke up08:29
Saviqtsdgeos, that good enough for you?08:29
tsdgeosSaviq: having the LC_ALL in the make step? yeah08:30
Saviqtsdgeos, try and see if we can globally set that for the test env08:30
Saviqtsdgeos, hmm doesn't look like it08:31
Saviqtsdgeos, you can set_test_properties on a per-test basis08:32
Saviqtsdgeos, but not globally08:32
tsdgeosbut don't we have our own "create test" macro?08:34
tsdgeosnow we don't08:35
tsdgeoswe have parts of it scattered around08:35
tsdgeosi see08:35
tsdgeosi.e. set_test_properties08:35
tsdgeosi.e. unity8/tests/plugins/Utils/CMakeLists.txt has run_tests08:36
tsdgeosbut it's used there locally08:36
Saviqtsdgeos, right, we could have a global macro for that indeed08:37
tsdgeosbut then it needs more careful thinking08:38
tsdgeosi.e. in this case we are assuming testname == testfile+.cpp08:38
tsdgeosbut in some other cases we have multiple files08:38
tsdgeosor different libs to link to08:38
Saviqtsdgeos, sure it does :)08:41
Saviqtsdgeos, if you don't want to get into it08:42
Saviqtsdgeos, just move the LC_ALL into CMakeLists.txt for now08:42
Saviqtsdgeos, for that one test08:42
tsdgeosso boils down to maybe just add a warning in CODING and be done with it :D08:42
tsdgeostests are expected to run under the C locale08:42
tsdgeosthen i can blame myself for not following the CODING guidelines :D08:42
Saviqtsdgeos, yup ;)08:43
tsdgeoswill do that08:44
tsdgeosSaviq: ok, comment added to CODING08:47
Saviqtsdgeos, cheers08:48
tsdgeosSaviq: do you want to do https://code.launchpad.net/~aacid/unity8/coding_autopilot_docu/+merge/200974 too or shall i find someone else?08:58
Saviqtsdgeos, looking08:59
Saviqtsdgeos, please add that if you don't `make install`, the tests will run on the system-installed version09:00
tsdgeosSaviq: done09:05
=== om26er is now known as om26erafk
Saviqtsdgeos, https://code.launchpad.net/~saviq/unity8/forgiving-genericscopeview/+merge/201147 please09:28
Saviqmzanetti, can you continue with https://code.launchpad.net/~unity-team/unity8/new-scopes-horizontal-card-layout/+merge/200910 please?09:29
mzanettiyes sir09:29
tsdgeosSaviq: on it09:30
tsdgeosnow i think both my coding file changes are going to conflct...09:30
Saviqtsdgeos, they might indeed09:30
tsdgeoslet's see09:31
Saviqtsdgeos, just merge them together?09:31
Saviqtsdgeos, neither is being -autolanded yet09:31
karniSaviq: thanks for the follow up! :)09:33
Saviqkarni, sure09:33
tsdgeosSaviq: ok, reapprove https://code.launchpad.net/~aacid/unity8/coding_autopilot_docu/+merge/200974 then09:33
tsdgeosi cancelled the other one09:34
karniSaviq: looks like good progress to me :)!09:34
Saviqtsdgeos, thanks09:34
Saviqkarni, glad to hear that09:34
Saviqkarni, I'm flying to Cape Town tomorrow, won't be around much until the 27th, I'll leave you in tsdgeos's capable hands09:35
Saviqkarni, he's the one writing the journals and organic grid now09:35
dednickis anyones indicators not showing up on desktop unity8?09:35
karniSaviq: ACK! Thank you :)09:36
Saviqdednick, seems to work fine09:36
dednickSaviq: hm. must be something funky with mine then09:36
tsdgeosSaviq: there's a lot more previewListView uses that you have not protected, no?09:37
Saviqtsdgeos, those were the only ones that bit me, but you're right09:37
tsdgeosdednick: there's the thing that closing down unity8 will kill them09:37
tsdgeosdednick: so run unity8 and ctrl+c it09:37
tsdgeosSaviq: i think that there are a few more that would break09:38
tsdgeos        forceNoClip: previewListView.onScreen09:38
tsdgeosand the renderloader clicked09:39
dednickSaviq: yeah, but it restarts them when unity8 starts09:39
tsdgeosSaviq: why don't you have a previewListView, is it a test?09:39
Saviqtsdgeos, the scope tool09:39
Saviqtsdgeos, but yeah I'm thinking I should make it differently... just get Dash complete, and not GenericScopeView alone09:40
tsdgeosor DashContent at leat09:40
dednickSaviq: nevermind. old qmenumodel.09:40
tsdgeosthat brings what you need i think09:40
karnimzanetti: Could you please merge this today, so we can get it into PPA ASAP? Horizontal card layout would be very handy for my team. https://code.launchpad.net/~unity-team/unity8/new-scopes-horizontal-card-layout/+merge/20091009:40
sil2100jamesh__: hello!09:40
mzanettikarni: yeah, on it right now. just running tests for a last time09:41
karnimzanetti: sweet! thank you :)09:41
sil2100jamesh__: I wanted to finally deal with the mediascanner-v2 release today, but I just noticed that we might want to modify the source package name09:42
sil2100jamesh__: right now it's mediascanner2.0, but not sure if the 'source' package should have a major and minor version - so maybe mediascanner2?09:42
Saviqtsdgeos, ok then, ignore that branch09:42
* tsdgeos ignores09:43
tsdgeoswhat's up with autopilot again09:43
tsdgeoswhat :D09:44
tsdgeosjust doesn't start :D09:44
tsdgeosit segfaulted like crazy :S09:46
tsdgeos/var/log/syslog: Jan 10 09:16:47 trusty-amd64-20131210-1707 kernel: [2561012.274762] QThread[29589]: segfault at 1cfb6 ip 000000000001cfb6 sp 00007f2584de7228 error 14 in unity8[400000+c000]09:46
tsdgeosdoesn't make any sense09:46
Saviqtsdgeos, yeah, it's been like that for some runs it seems09:47
tsdgeosany clue if it's fauly hardware or something else?09:49
Saviqtsdgeos, we definitely need someone to look at it09:50
tsdgeosmzanetti: can you detach me a autopilot VM?09:52
tsdgeosif so i'll have a look09:52
mzanettitsdgeos: sure09:52
tsdgeosorganic grid is almost done [in my mind :D]09:52
karniHow do I actually work with the unity-scope-tool?09:52
Saviqkarni, right now you dont'09:53
Saviqkarni, I'm fixing09:53
karnioh okay09:53
Saviqkarni, will have a branch for you to try in 3 mins09:53
karniSaviq: wheee \o/ thanks Michał09:54
* karni was wondering how to populate unity-scope-tool with an actual scope09:54
mzanettitsdgeos: ps-trusty-server-amd64-2 is yours once the job 881 is done09:54
mzanettitsdgeos: note that jenkins will revert the VM after the job is finished. so give it a minute before starting to work on it or else jenkins will revert your stuff too09:55
Saviqtsdgeos, https://code.launchpad.net/~saviq/unity8/tool-fulldash/+merge/20115009:59
Saviqtsdgeos, actually need fix10:00
Saviqbut meeting first10:00
tsdgeosmzanetti: even if the machine is doing qmluitests at the moment i can use it for autopilot tests, right?10:02
mzanettitsdgeos: oh... autopilot... we don't run them in VM any more10:03
mzanettitsdgeos: they are executed on a real machine but I don't have access to that one10:03
tsdgeosmzanetti: no?10:03
tsdgeosoh :_/10:03
tsdgeosmzanetti: ok, then put the thing back into the pool10:03
mzanettitsdgeos: do you have autopilot failures in jenkins which you cannot reproduce locally?10:04
tsdgeosmzanetti: unity8 crashing :D10:04
mzanettihmm... but that *should* behave the same on your local machine10:05
tsdgeosmzanetti: see https://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-trusty/1930/consoleFull for the billions of segv10:05
tsdgeosi definitely don't get all these crashes10:07
tsdgeosfginther: ping10:31
Saviqtsdgeos, tool-fulldash fixed10:45
Saviqtsdgeos, going to #ubuntu-ci-eng vanguard to take a look10:47
karniUnity team standup is 2:30 UTC?10:54
tsdgeosSaviq: so no more scope selector and just swipe left/right?10:56
Saviqkarni, +110:56
Saviqtsdgeos, yeah, or select by header10:56
Saviqtsdgeos, felt unnecessary to have the selector now10:57
tsdgeoskarni: i invited you to it10:57
Saviqtsdgeos, not entirely happy that all the scopes are loaded all the time10:57
Saviqtsdgeos, but it works, which is more important10:57
tsdgeosSaviq: but that happened before, no?10:57
tsdgeosSaviq: or you mean the graphical counterpart10:57
Saviqtsdgeos, no, before it only loaded the single dash page, yeah10:57
janimocan unity8 be tested on an x86 workstation? Only GUI needed , no sensors or other fancyness of course10:58
karnitsdgeos: thanks :)10:58
tsdgeosSaviq: why the new mouse area?10:59
Saviqtsdgeos, 'cause otherwise I could click on items behind the controls10:59
tsdgeosjanimo: yes, you can run unity8 on a x86 machine11:00
Saviqjanimo, it will only run in a window for now (i.e. you can't run real applications yet)11:00
janimoSaviq, tsdgeos, any particular setup needed or packages not in the trusty archives?11:00
tsdgeosSaviq: which items behind the controls?¿11:01
tsdgeosSaviq: you mean the right hand side controls?11:01
Saviqtsdgeos, yes11:01
Saviqtsdgeos, you could click items in a dash page to the right of the selected one11:01
tsdgeosi see11:01
Saviqjanimo, nope, apt-get install unity8; unity811:01
janimothe Mir spec doc has a graphic dated May 2013 regarding Mir on the free graphics stack, how valid is that still?11:01
Saviqjanimo, you might want to take that on #ubuntu-mir11:02
janimoSaviq, heh, that indeed worked11:02
janimoSaviq, hey, they sent me here :)11:02
Saviqjanimo, and yeah, you won't be able to run unity8@Mir on x86 yet11:02
Saviqjanimo, not for the free graphics stack and Mir, they didn't ;D11:02
janimoSaviq, for running unity8 I meant11:02
tsdgeosSaviq: does the category combobox (sorry i mean optionselector, wait Oren's gone, can we call it combobox again?) shrink up and down in size for you too when changing scopes?11:03
Saviqtsdgeos, checking11:03
janimofair enough I guess you are mostly testing on ARM/platfrom-api/Android based hw11:03
Saviqtsdgeos, yeah11:03
tsdgeosthat'd be an SDK bug of cours11:03
Saviqtsdgeos, yup11:03
Saviqjanimo, this cycle we'll have it running on x86 for sure11:04
Saviqjanimo, just not yet there11:04
janimoSaviq, do you know if there's a blueprint or some other doc specifically for that effort?11:04
tsdgeosSaviq: want me to report a bug?11:04
janimoSaviq, I'd like to help11:04
Saviqtsdgeos, yes please11:04
Saviqjanimo, https://blueprints.launchpad.net/ubuntu/+spec/client-1404-unity8-on-desktop - bregma is leading that work11:05
janimoSaviq, thanks11:05
tsdgeosSaviq: i guess it changes size because you empty/fill it again?11:08
Saviqtsdgeos, yeah, replace the model11:09
karnitsdgeos: hrm. did you use michal.karnicki@canonical.com for the stand-up invite?11:12
tsdgeosi think so11:12
karnitsdgeos: FTR, no event in my calendar yet11:12
karnitsdgeos: That's okay though, if you guys have it consistently at the same hour11:13
tsdgeosi think i failed to use the google calendar ui11:13
karni^ ^11:13
karnitsdgeos: thanks! :D11:13
tsdgeosyou know all those webui you never know if you have to press apply/save/ok or not11:14
tsdgeostoo much dynamic stuff11:14
mhr3Saviq, looks like category expansion no longer works in the scope tool11:14
karnihahah, true. notably in Gmail "Contacts". Saves on it's own, but somehow never *felt* reliable to me ;)11:14
karnimhr3: in case you're talking about the tool-fulldash branch/MP, it does work for me11:15
mhr3karni, nope, new-scopes trunk11:16
tsdgeosmhr3: damnit, true :D11:16
tsdgeosis this because of the thing i just approved?11:16
tsdgeosor was there already?11:16
=== _salem is now known as salem_
karnimhr3: oh.. then he just fixed it ;D11:16
tsdgeosah was there already11:16
mhr3tsdgeos, worked until now afaict11:16
Saviqmhr3, I fixed it11:16
tsdgeosand then the new thing fixes it11:16
Saviqmhr3, tsdgeos broke it11:16
karnimhr3: https://code.launchpad.net/~saviq/unity8/tool-fulldash/+merge/20115011:16
* tsdgeos read it backwards11:16
Saviqmhr3, and now he's reviewing a fix11:17
tsdgeosSaviq: you don't have tests :-P11:17
Saviqtsdgeos, indeed I don't11:17
tsdgeosSaviq: thing is, the tests we have for genericscope do the expansion thing11:17
tsdgeoshow do they work?11:17
tsdgeosi guess we have a fake list in there?11:18
Saviqtsdgeos, the expansion was only broken in the tool11:18
Saviqtsdgeos, 'cause of the previewListView11:18
Saviqtsdgeos, the MouseArea in GenericScopeView was enabled all the time11:18
tsdgeosyep, the tests have their own previewListView11:18
Saviqtsdgeos, so the expansion works, just you couldn't click it11:18
tsdgeosthat's why it works11:18
Saviqtsdgeos, yup11:18
Saviqtsdgeos, and now the tool does, too11:18
Saviqin a sense11:19
mhr3Saviq, hm, doesn't work for me with tool-fulldash11:20
mhr3file:///home/miso/projects/unity8/qml/ScopeTool.qml:64:5: Type DashContent unavailable11:20
Saviqmhr3, that's based on trunk11:21
Saviqmhr3, merge into new-scopes11:21
mhr3that's what i did11:21
Saviqmhr3, there will be a conflict11:21
karnimhr3: Works here11:21
Saviqmhr3, imports need to look like so:11:21
Saviq import Utils 0.111:21
Saviq import Unity 0.211:21
Saviq import "Components"11:21
Saviq import "Dash"11:21
mhr3got that exactly11:22
Saviqmhr3, full log please11:22
Saviqmhr3, you on saucy, right?11:23
Saviqmhr3, you probably don't have the latest UITK11:23
Saviqmhr3, just upgrade already11:23
mhr3right, this isn't gcc, the first error isn't the issue :)11:23
Saviqmhr3, mzanetti, tsdgeos, karni: bringing Card and CardHeader over to lp:unity811:25
mhr3yey :)11:25
Saviqmzanetti, you can probably review ↑, since you already did :)11:25
mzanettiSaviq: sure11:25
mzanettiupdating the diff takes ages... I'm afraid of the size of that merge :)11:29
mhr3karni, is this desired? http://imgur.com/etQgKSj11:36
mhr3i mean, the first one larger than the rest?11:37
karnimhr3: It's not desired, looks like a bug. Interestingly, can't find that item on my end.11:39
mhr3grooveshark is location dependant iirc11:39
mhr3or maybe that's u1-music11:39
mhr3i guess it can have something to do with the two-line title11:40
karnimhr3: I'll look into it11:40
Saviqmzanetti, you shouldn't be - just 800loc11:45
Saviqmzanetti, not sure what lp is at11:46
Saviqmzanetti, I did overwrite the branch soon after I pushed it, but seems lp got stuck on it11:46
Saviqmzanetti, can reapprove if you want11:46
mzanettino worries11:46
Saviqtsdgeos, think we should force-merge stuff that only fails in otto?11:47
SaviqI'm half-way there11:47
Saviqmzanetti, ↑ ?11:47
Saviqwe're getting queued up again11:47
tsdgeosSaviq: i'd prefer to get it not to black-magicly fail :D11:48
Saviqtsdgeos, of course, except we have no control over that11:48
tsdgeosyeah ;/11:48
Saviqtsdgeos, people are at it11:48
* Saviq runs a suite11:48
tsdgeosSaviq: i mean it's impossible my change to the CODING file or to the hjournal cause those crashes11:49
Saviqtsdgeos, exactly11:49
tsdgeosi would not cry if it gets force merged11:49
* Saviq does11:49
Saviqtsdgeos, and it completed on device, for that matter11:50
karnimhr3: you were right, this line is causing trouble: height: template && template["card-layout"] === "horizontal" ? header.height : width / artShape.aspect11:53
karniThinking of best way to fix it.11:53
karnimhr3: Basically, it's a CardHeader layout bug, and that one is actually still worked on, but I'll try to fix it since I'm already on it.11:55
karniIs there a known mapping of XXS / XXXS font size to QML font pixel/pointSize ?11:58
Saviqkarni, http://developer.ubuntu.com/api/qml/sdk-1.0/Ubuntu.Components.Label/12:01
Saviqkarni, mhr3, actually that "one is bigger than the rest" is, in fact, desired12:01
karniSaviq: haha, thank you :) There we go, design using xxxs that doesn't exist. I'll see what makes best sense12:02
karniSaviq: Is it? Like mhr3 said, I think it's because the title is wrapped.12:02
karnior is it U1MS result? :O12:02
Saviqkarni, mhr3, "Art_Header_horizontal" → "Header and Art are equal height."12:02
karniSaviq: Hrm. Don't you think title overflow should not change CardHeader size?12:03
Saviqkarni, what I *think* doesn't really matter does it ;D12:03
Saviqkarni, that requires a comment on the spec that this happens12:04
karniI'll add the comment12:04
Saviqkarni, and the spec needs to be adjusted, if applicable12:04
Saviqkarni, it might be that we'll need to foresee the largest header height in a configuration12:05
Saviqkarni, and force it to be as high12:05
Saviqkarni, but that's not what the spec says right now12:05
karniI agree12:05
karniSaviq: I would think (which may not matter :D)  text overflow should not cause CardHeader height change12:05
* karni comments on the spec12:06
Saviqkarni, OTOH you don't want to make the header always have empty space at the bottom12:06
Saviqkarni, if there's never overflow12:06
karniSaviq: That I didn't say ;) I'd like to do something smart about that. If you look at Michal's screen, there seems to be unnecessary padding at top (invisible at bottom, because of hidden fields). I think if there was less padding (if that makes any sense), the art would not resize because Header height would not change.12:08
karniJust a thought, I know it's not us to make UI/UX decisions (usually)12:08
Saviqkarni, sure, that padding is an issue in CardHeader, I agree12:08
Saviqkarni, but that won't change the fact that the two will have different size12:08
Saviqkarni, 'cause both will lose padding12:08
karniSaviq: You know I'm relatively new to the subject, but I tend to think 'RelativeLayout' in Android, it's pretty darn flexible, so is Qt/QML :) Anyways, I won't spend too much time on it now, before we get Katie speak her mind.12:10
Saviqkarni, sure, everything is flexible, problem is you can't know the text layout... until you lay it out12:10
Saviqkarni, so you won't know if something wraps or not12:10
=== alan_g is now known as alan_g|walk
Saviqkarni, but even if you know, you'd have to interrogate all results (the not-visible ones, too), to find out whether any of them overflows12:11
Saviqkarni, and adapt the header height12:11
karniSaviq: I found a way yesterday, I thought I needed it for one test. The thing is, it's a hack really (instantiate a QML object, and measure text). I can easily imagine this would have terrible influence on the performance, though.12:11
Saviqkarni, yeah exactly12:11
Saviqkarni, so no, not a *real* way12:12
karniwell, sh!t12:12
karniSaviq: comment added ;)12:12
karnimhr3: we'll have to wait for Katies input, it's a tricky bit.12:12
Saviqkarni, so we need a good-enough solution to be able to foresee what can happen12:12
Saviqkarni, the easiest would probably be to assume the largest height, basically, but that could mean we end up with space that's never used in any header in a given category12:13
karniyeah, that'd sort of suck.12:13
karnieasy and quick, tho.12:13
Saviqtsdgeos, did you notice that suddenly qmluitests pass 100% https://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-trusty/12:14
Saviqwell, almost, but they're pretty green12:14
tsdgeoswell the expand_expand was actually causing more reds than the other one12:14
tsdgeosand i fixed the expand_expand12:14
tsdgeosbut yeah i'd still expect the one for dash scope change on load to fail a bit more reliably12:15
mzanettiSaviq: hey, the only thing I could complain is that make tryCardHeader is not doing anything. do you want to fix that or should I ignore?12:18
Saviqmzanetti, "not doing anything"?12:18
mzanettiSaviq: well... it pops up a window with a blue rectangle.12:18
mzanettino controls to actually try something12:18
Saviqmzanetti, aah try*12:18
Saviqmzanetti, tryCard does that12:18
karnimzanetti: make -C builddir tryCard and change there12:18
Saviqmzanetti, didn't make sense to have it separated out I though12:19
mzanettiok... makes sense12:19
mzanettiSaviq: karni: approved12:22
karnithank you12:22
Saviqmzanetti, thanks12:22
karniSaviq: semi-random question - how heavy would an invisible, extra Label in a layout be? Besides adding one item to the traverse tree when rendering, would it be acceptably lightweight? (You can guess why I'm asking about it - test textWrap)12:29
=== om26erafk is now known as om26er
Saviqkarni, if it'd only be there for testing, don't12:30
karniSaviq: well, that could be used to adjust CardHeader layout (not purely for test purposes, if that's what you're asking). I know it's a wild question, that's why I went ahead and asked ;)12:30
Saviqkarni, we should avoid any unnecessary objects - sure, it wouldn't be even pushed to the scenegraph or the GPU, but it would still be there in the QML tree12:30
* karni nod12:31
Saviqkarni, I really don't think we'll go that route12:31
Saviqkarni, remember you don't even have all the items rendered at once12:31
Saviqkarni, only those that are on screen12:31
Saviqkarni, the rest are destroyed12:31
Saviqkarni, so if at any point you start thinking "look at all the items in the view to..." - stop right there ;)12:32
karniSaviq: Nopes. I was thinking in terms of adding that invisible Label to test wheter text was same as card header title, and base the card header layout on that. I also know that if I wanted to traverse all model items, I wouldn't be doing that in this part of code :) So, not what I thought, no worries :)12:33
Saviqkarni, not following - what would that invisible Label do that the visible one wouldn't?12:34
karniSaviq: no Wrap, single line, possibly elide. if text in that label is !== text in title labe, means we have 2 lines in the card header title labe.12:35
karniSomething along that line, you prolly get the point now :)12:35
karniIt's a hack, but dirty because (if that worked) it'd require an invisible label in the tree, which I don't like either,12:36
karnibut like you said, seems there's no fast way to check if the text has wrapped/there's more than a single line in title label.12:36
karnioh yeah. Text.truncated: bool12:37
Saviqkarni, http://qt-project.org/doc/qt-5.0/qtquick/qml-qtquick2-text.html#lineCount-prop12:37
karnihaha omg. Can I hide now ;)?12:37
Saviqkarni, ;)12:38
karniSaviq: does that in fact return 2 for wrapped label? if so, I probably wasted ~15 of thinking. It was fun, nevertheless ;)12:38
* karni notes to really look better (or sleep longer)12:38
Saviqkarni, yes it does12:39
=== alan_g|walk is now known as alan_g
=== alan_g is now known as alan_g|lunch
Saviqmhr3, https://launchpadlibrarian.net/162016552/buildlog_ubuntu-trusty-armhf.unity-scopes-shell_1%3A0.1.1%2B14.04.20131219.1-0~36~ubuntu14.04.1_FAILEDTOBUILD.txt.gz13:51
Saviqmhr3, that means there's some version mismatch between -api and -shell, doesn't it?13:51
mhr3Saviq, yea, some massive renaming was merged into -api13:52
mhr3yey! :/13:52
* Saviq wonders why it passed on i38613:52
Saviqmhr3, care to comment on bug #1267831 ?13:53
ubot5bug 1267831 in unity-scopes-shell "unity-scopes-shell build problems when building against Qt 5.2" [Critical,New] https://launchpad.net/bugs/126783113:53
Saviqmhr3, maybe -shell needs a dependency version bump?13:53
mhr3Saviq, wait, yea, this is weird13:53
mhr3it's not about the renaming13:54
mhr3Saviq, what is this even?13:54
mhr3it's not trunks13:54
mhr3of neither shell nor api13:54
Saviqmhr3, https://launchpad.net/~canonical-qt5-edgers/+archive/qt5-beta2/+sourcepub/3765701/+listing-archive-extra13:54
Saviqmhr3, might be we need -api there as well, not just -shell?13:55
mhr3pff, week old branches, don't you know we change everything every week? :)13:55
Saviqmhr3, 'cause it builds -shell trunk against -api released?13:55
mhr3Saviq, it also need new unity-api13:55
mhr3but with latest unity-scopes-api, -shell won't build13:56
Saviqmhr3, ok be back in a few13:56
fginthertsdgeos, pong14:02
=== alan_g|lunch is now known as alan_g
tsdgeosfginther: we've been having some weird results in autopiltot runs but people from #ubuntu-ci-eng are already looking into it14:02
tsdgeosso nothing14:02
fginthertsdgeos, ok, I'll check with ci14:03
darklight_Is there a chance of the panel shortcuts finally being customizable in 14.04 ? seriously why was that design ever approved in the first plae14:15
darklight_Not only it's a bad design and can really anny regualr users but it can be a real showstopper for people with disabilities14:18
Saviqfginther, ev registered it at https://app.asana.com/0/9505407366941/950435630488714:23
* Saviq is not sure what to do with that...14:24
fgintherSaviq, I see it, I'm going to rerun an older unity8 binary package to see if something changed on the test system14:24
fgintherSaviq, or possibly a dependency14:25
elopiotsdgeos: this is now ready to land:14:26
darklight_Who could I contact about this? there are bugs open but they are ignored on the target mileston has been postponed for years14:27
tsdgeoselopio: cool14:27
Saviqdarklight_, if it's a design issue, #ubuntu-design is the place to go14:28
=== jamesh__ is now known as jamesh
greybackSaviq: ever seen this before: http://pastebin.ubuntu.com/6726838/14:29
greybackI'm trying to run AP on a pretty clean image. unity8-autopilot package is installed14:29
Saviqgreyback, yeah, your command is wrong14:29
Saviqgreyback, -p -v unity8-autopilot14:29
darklight_Saviq: well design as in feature missing or better badly implemented, would that be the best place where to ask ?14:29
Saviqso it tries to install "-v" and then run "unity8-autopilot" test suite as opposed to "unity8"14:30
greybackSaviq: what? That's ridiculous. -v switch is in the help14:30
Saviqdarklight_, I'm not sure what issue you're talking about, but sounds like yes - this needs to be looked at by designers, only then will code change14:30
Saviqgreyback, ORDER14:30
Saviq-p -v14:30
greybackSaviq: OBJECTION14:31
jameshsil2100: hi.  sorry I missed your ping -- I've been at a conference all week.  Package name change sounds fine.14:31
Saviqgreyback, STANDUP14:31
darklight_Saviq: ok thanks, just to clarify the issue is the hardcoded shortcuts in unity14:31
Saviqdarklight_, got it14:31
sil2100jamesh: hi! No problem, Satoris already commented - the package is preNEWed and soon it will land in the new queue14:31
Saviqgreyback, NOTES14:32
Saviqgreyback, you coming?14:32
jameshwas having drinks with Lennart Potering tonight, and we managed to stay civil the whole way through14:32
Saviqgreyback, can you hear us?14:32
Saviqmhr3, added -scopes-api to the PPA, -api is already there14:35
mhr3Saviq, how high prio is it to make -shell buildable again?14:36
mhr3we're in the process of even more api changes14:36
Saviqmhr3, ah so it's not gonna build anyway?14:37
mhr3not right now, no14:37
Saviqmhr3, whenever you're ready basically14:37
mhr3would build yesterday :)14:37
Saviqright ;)14:37
karniSaviq: Should "card-size" have no influcence on the size of horizontal card size?14:52
Saviqkarni, it does, in vertical14:52
karniSaviq: but not for horizontal cards, right?14:52
Saviqkarni, in horizontal the card width is hardcoded at 40GU14:52
karnisweet ;)14:53
Saviqor 3814:53
fgintherSaviq, have you any clue as to what the unity8 SEGVs are caused by?15:20
tsdgeosfginther: i don't think we do15:23
tsdgeosfginther: as a matter of fact stuff works in our desktops just fine15:23
tsdgeosfginther: and there's no backtrace logging in those machines, is there?15:23
fgintherSaviq, I reran a test that passed before, and now it fails. I also looked at the installed packages between a good and bad run and found no differences. Even checked that the test machine was the same15:23
fginthertsdgeos, there is no reason for crashes to be disabled here, but perhaps they are. I'll check into it15:25
tsdgeosor at least i couldn't find where they were stored :D15:31
Saviqfginther, yeah, I'm completely lost, it just started happening overnight15:36
Saviqfginther, if possible, it would be good to take down a machine to try and reproduce on it to at least get the .crash file15:37
tsdgeosdandrader: https://code.launchpad.net/~aacid/unity8/journal_test_cmake_bugfix/+merge/20120215:39
tsdgeosand i think i found another15:41
tsdgeosyou shouldn't ask for those last minute refactoring things :D15:41
Saviqmhr3, does that FTBFS ring a bell https://launchpad.net/~canonical-qt5-edgers/+archive/qt5-beta2/+sourcepub/3811744/+listing-archive-extra ?15:43
=== alan_g is now known as alan_g|tea
karnimhr3: Just FYI, had this tiny patch for that first item being large (wrapped title) but it fails miserably a potential OEM customization. We'll need to measure it once for the whole category instead as Michal suggested, so that change would be elsewhere :( http://paste.ubuntu.com/6726961/15:45
tsdgeosSaviq: have you manually merged https://code.launchpad.net/~aacid/unity8/journal_test_cmake_bugfix/+merge/201202 alraedy?15:45
Saviqtsdgeos, no15:45
mhr3Saviq, unfortunately, no15:45
Saviqtsdgeos, I wait for the autolanding job to complete15:45
tsdgeosSaviq: ok, because i just added a new fix15:45
mhr3Saviq, opening bug about it15:45
mhr3karni, eeek :P15:46
mhr3karni, i guess we shouldn't use grid if we want it to look good15:47
mhr3needs something more dynamic15:47
karnimhr3: Well, I was assuming you specifically wanted to test the grid with horizontal cards. That is, indeed, a bug. Just needs to be solved elsewhere.15:48
karnimhr3: That's a configuration a 3rd party dev can choose, thus we need to get it right (at some point in time)15:48
mhr3karni, i was just testing the horizontal layout, but i guess that grid implies that each item is same height and therefore this can happen15:49
karnimhr3: ok, I realized you meant that all cards have same height because of the first one. right.15:50
mhr3still, you're right that the cards should try to look similar when possible :)15:51
=== dandrader is now known as dandrader|afk
karnimhr3: I think we'll fix this on the category/card level, so a grid with horizontal cards would look just fine. Nevertheless, it would be interesting to allow horizontal cards of different heights indeed.15:53
Saviqkarni, it's allowed already - just add a summary of different length15:54
Saviqkarni, that's one of the use cases for VJournal15:54
karnid'oh, of course :)15:54
Saviqkarni, but TBH I'm starting to feel like the "mascot is the height of the header" rule is just wrong15:55
mhr3Saviq, that build was with 5.2, right?15:55
karniSaviq: I don't think I have an opinion on that ATM.15:56
=== alan_g|tea is now known as alan_g
=== dandrader|afk is now known as dandrader
sil2100jamesh: are you still around?16:16
fgintherSaviq, tsdgeos, I'm not much closer on the unity8 problem. There is apparently just updated today version of apport that should help to collect the crash, but I've got to put some work into setting this up16:54
fgintherSaviq, tsdgeos, if you need to make progress on unity8 MPs, we may need to disable that job16:54
Saviqfginther, we're fine16:54
Saviqfginther, I've been forcing generic-land16:55
Saviqfginther, so it's not blocking us16:55
Saviqfginther, and it's EOW now16:55
fgintherSaviq, thanks, I'll send an update if I get anywhere today16:56
=== dandrader is now known as dandrader|lunch
=== alan_g is now known as alan_g|EOW
=== dandrader|lunch is now known as dandrader
=== dandrader is now known as dandrader|afk
=== dandrader|afk is now known as dandrader
=== salem_ is now known as _salem

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