mhall119can I build unity8 on utopic, or does it require vivid?02:21
mhall119unity8-build-deps is trying to install newer versions of packages than I have02:21
RAOFmhall119: I believe you *should* be able to build on utopic (as long as you also have the various deps-from-source, such as Mir), but vivid's a better bet.02:46
mhall119RAOF: well I'm certainly getting an error on a clean branch03:02
=== charles_ is now known as charles
tsdgeospstolowski: did you get time to produce the fields i need to implement the review modification stuff?08:39
pstolowskitsdgeos, not yet, sorry, will get them soonish08:48
tsdgeoscimi: can you do https://code.launchpad.net/~aacid/unity8/noComponentsInGeneratedCardCreatorCode/+merge/258252 ?09:23
tsdgeosmzanetti: can you do https://code.launchpad.net/~aacid/unity8/fixtestMainNavigation/+merge/258173 ?09:23
tsdgeosand i'll also need a volunteer for https://code.launchpad.net/~aacid/unity8/fallbackImage/+merge/25839909:23
cimitsdgeos, you 100% we don't need components in the cards?09:24
cimicarousel and such... (just asking weird cases)09:24
tsdgeoscimi: the component is per category, so no, the card should never need it, we can generate the code based on the component contents09:25
tsdgeossame for template btw09:25
tsdgeosgenerated code should not depend on template because we can know that on compile time09:25
tsdgeosMacSlow: https://code.launchpad.net/~fboucault/unity8/notifications_icon_respect_ratio/+merge/258069 for you!09:43
MacSlowtsdgeos, thx for pointing that out... I've been looking up and down my inbox and don't see any email notification regarding this09:48
mzanettitsdgeos, running make testDashContent gives me this: http://paste.ubuntu.com/11007513/10:17
mzanettitsdgeos, make xvfbtestDashContent passes fine though10:18
mzanettiI don't think it's related to that branch10:18
mzanettijust as a FXI10:18
tsdgeosmzanetti: hmmm, really? damnit, i had that whole test running for 2 hours here :/10:18
tsdgeosmzanetti: that's with or without my branch?10:18
mzanettion that above branch10:18
mzanettibut the failing line doesn't seem to be the one you touched10:19
tsdgeossure, but the thing is making it all more stable :/10:19
tsdgeosnot just one10:19
tsdgeosmzanetti: you have it failing all the time?10:19
mzanetti2 out of 2 failing with testDashContent10:19
mzanetti2 out of 2 passing with xvfbtestDashContent10:19
mzanettitsdgeos, tried a third time, failed again10:21
tsdgeosnot here10:21
tsdgeosdoes runnig it standalone makes it fail too?10:22
* mzanetti tries10:22
mzanettiyep. still failing10:22
mzanettimake testDashContent FUNCTION="DashContent::test_show_header_on_list_movement"10:22
tsdgeoslet me give you a patch to try to see what's happening10:23
mzanettinow that's interesting... it's passing with GRID_UNIT_PX=16 here10:24
mzanettibut with the default of 8 it fails10:24
mzanettiprobably still just timing10:24
tsdgeosmzanetti: do you actually see some animation happening?10:26
tsdgeosmzanetti: can you try http://paste.ubuntu.com/11007606/ and paste the output10:26
tsdgeosi wonder if the animation is simply not happening for you10:27
mzanettitsdgeos, qml: 120 11210:28
mzanettinot sure what animation I should see10:28
tsdgeosmzanetti: the header coming down10:28
tsdgeosi guess you don't :D10:28
tsdgeosdoes replacing the units.gu(2) by 4 in the touchFlick help?10:30
mzanettitsdgeos, yeah. not animating10:30
tsdgeosprobably not but let's make sure10:31
mzanettitsdgeos, yes, passing with units.gu(4)10:31
tsdgeosi guess we can increase that10:31
tsdgeosit's just to trigger a "horizontal move"10:31
tsdgeosso it doesn't matter the actual number10:31
tsdgeosjust weird you need it and not me10:32
tsdgeospushed anyawy10:32
* mzanetti reverts and pulla10:32
mzanettiack. passing now10:34
* tsdgeos shurgs10:37
tsdgeostx for the review :)10:38
=== MacSlow is now known as MacSlow|lunch
dandrader_mzanetti, are wily repos already open?11:25
dandrader_ie, can we release stuff? :)11:25
mzanettidandrader_, not that I know of11:25
Saviqdandrader_, mzanetti, there are silos targeting wily at least http://people.canonical.com/~platform/citrain_dashboard/#?q=ubuntu%2Flanding-01411:30
mzanettiyeah, just saw that too11:31
mzanettiI just asked sil, he said the system is ready but the higher-ups didn't formally open it yet, as it usually seems to happen11:31
mzanettiso I guess we can start preparing silos targettting it11:31
cimitsdgeos, I need to start using Ubuntu Components 1.3, but this spits out dozens of new deprecated debug messages11:32
cimitsdgeos, shall I first fix those, then in a later branch do the feature?11:33
Saviqpete-woods, you might want to look at lp:unity8 for a refactored QmlPlugins.cmake11:33
cimimore than dozen, fundreds :D11:33
pete-woodsSaviq: that's where I stole it from11:33
tsdgeoscimi: can we just introduce it in the file you need it?11:34
Saviqpete-woods, yeah, but it doesn't look current?11:34
Saviqpete-woods, ah wait, it is11:34
Saviqpete-woods, but11:34
Saviqpete-woods, qmake foo will break cross-building11:34
pete-woodsSaviq: you're right, it's not quite current11:34
pete-woodswill pull updates11:34
pete-woodsSaviq: from the looks of it I just need to delete a few buts11:36
cimitsdgeos, yeah but I think it complains for others...11:38
cimitsdgeos, I receive ton of debug messages, but the debug messages aren't clear11:38
cimitsdgeos, <Unknown File>: QML UCDeprecatedTheme: Theme.palette is deprecated. Use ThemeSettings instead11:39
cimiunknown file :/11:39
Saviqcimi, grep -r Theme.palette ;)11:40
cimiSaviq, did, MANY files :)11:40
pete-woodsSaviq: I take it that QMLPLUGIN_DESTINATION is magically set by Qt now?11:42
Saviqpete-woods, no11:43
Saviqpete-woods, but you shouldn't use qmake either, that won't work in cross-building11:43
pete-woodsSaviq: well I just stole your macro :p11:44
Saviqpete-woods, my macro never called qmake ;)11:44
Saviqpete-woods, because we don't install in the public dir11:44
pete-woodsSaviq: ah, that explains that then11:44
pete-woodsI can just set it to usr/lib/ARCH/…11:45
Saviqpete-woods, that's the only way right now11:45
pete-woodsworks for me!11:45
Saviqpete-woods, just a note, those macros (should actually be functions...) are meant to be wrapped11:46
Saviqso they're generic11:46
Saviqpete-woods, http://bazaar.launchpad.net/~unity-team/unity8/trunk/view/head:/plugins/CMakeLists.txt11:46
pete-woodsthere is a wrapper function right at the bottom11:46
pete-woodsSaviq: yeah, I stole that function11:46
Saviqpete-woods, good, I just saw that export_* was installing to QT_INSTALL_QML by default, don't do that :)11:47
Saviqpete-woods, on that note, you might want to have a look at https://code.launchpad.net/~unity-team/unity8/refactor-qmltests/+merge/25783511:48
pete-woodsSaviq: will do :)11:50
Saviqpete-woods, we need tests in cmake-extras btw..11:53
Saviqpete-woods, and re: QT_INSTALL_QML, QTBUG: https://bugreports.qt.io/browse/QTBUG-29987 - but seems they just punted it11:53
cimitsdgeos, some warning might not be ours, since I grep for some of those warnings and they are not in unity811:59
pete-woodsSaviq: yeah, I'm using a landing of indicator-network which uses most of the macros as a sort of test at the moment11:59
cimitsdgeos, might be sdk broken somewhere11:59
pete-woodsSaviq: but yeah, need to come up with a proper testing structure11:59
pete-woodswow, marked as invalid12:00
=== alan_g is now known as alan_g|lunch
pete-woodsit's amazing how difficult we are able to make each other's lives12:01
Saviqpete-woods, yeah... [testing] totally, I'm quite scared to rely on this for all our builds with no tests... refactoring this thing was a mess12:02
Saviqpete-woods, sure, unity8 is a good testing ground for it, but would rather... you know ;)12:03
pete-woodswell it looks like your refactoring only significantly touches the QmlTest.cmake12:04
Saviqpete-woods, oh yeah, that's just for this12:04
Saviqpete-woods, but QmlTest.cmake should make it into cmake-extras at some poitn12:05
pete-woodsSaviq: sure, but that file is big and scary!12:06
pete-woodspatches welcome! :p12:06
Saviqonce it actually gets ACKed, we'll see how it goes12:06
pete-woodsSaviq: do you remember the cmake variable for arch triplet, btw?12:07
pete-woodsI always forget the thing, and have to scour all my projects for it12:07
Saviqpete-woods, is there one? :D /me asked gcc -dumpmachine ;)12:07
* Saviq fixes qmltest12:13
Saviqpete-woods, hum hum, isn't that going to be arm for x-building?12:15
Saviqthat's probably good for this, not good for my use...12:16
SaviqOTOH we're not running those tests when x-building...12:16
pete-woodsSaviq: tbh I thought it pretty much did what you said, i.e. parse gcc -somethingorother12:17
Saviqpete-woods, yeah, but when cross, gcc is not gcc ;)12:17
Saviqyou don't want to run tests on build arch if it != host arch anyway12:18
pete-woodstrue, dat!12:18
ChrisTownsend 12:27
tsdgeoscimi: well not broken but updated no12:34
cimitsdgeos, yeah, but if I write a patch that deprecates something, I make sure I don't have deprecates in my own code :)12:35
tsdgeoscimi: oh well12:42
=== MacSlow|lunch is now known as MacSlow
=== alan_g|lunch is now known as alan_g
=== dandrader_ is now known as dandrader|afk
=== dandrader|afk is now known as dandrader
=== dandrader is now known as dandrader|lunch
=== alan_g is now known as alan_g|EOD
=== dandrader|lunch is now known as dandrader
=== dandrader is now known as dandrader|afk
tedggreyback, Apparently it's already done for bug 141609618:29
ubot5bug 1416096 in ubuntu-system-settings (Ubuntu RTM) "expose battery charging state on the bus" [Medium,Fix released] https://launchpad.net/bugs/141609618:29
tedggreyback, You should be able to use the same value18:29
* tedg is sad that he didn't find an action item for charles18:29
greybacktedg: sweet18:29
charlescurrently it's a string; you could test for "plugged_in = (state=='charging') || (state=='fully-charged')"18:30
=== dandrader|afk is now known as dandrader
josharensonelopio_: do you want to review this, or should I get someone from unity? https://code.launchpad.net/~josharenson/unity8/qa_helpers/+merge/25843519:12
josharensonelopio_: re qa_helper... How do you feel about making _is_greeter_active() public so that process_helpers can assert that the greeter was visible before calling unlock? I'm not sure its necessary, but that's how it was before.22:47
josharensonelopio_: I guess the wait_for_greeter call would fail actually so there is no reason... I like the logic all contained in the module better anyway22:53
elopio_josharenson: yes, I thougth that wait will raise the exception. But as you prefer, both sound good to me.22:54

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