/srv/irclogs.ubuntu.com/2013/06/25/#ubuntu-unity.txt

=== _salem is now known as salem_
=== salem_ is now known as _salem
jibelsil2100, I'm reverting the upstart job added yesterday, I'm wondering if that's what prevent lightdm from starting07:38
sil2100jibel: looks like that's the only change we made, so ACK - but not sure why that's happening ;/07:39
sil2100jibel: on the other hand...07:39
sil2100jibel: yeah, let's do that07:40
sil2100jibel: too bad we can't really check if the new upstart job ran or not - maybe there's some error in the  config file?07:40
jibelsil2100, the job didn't even start07:41
jibelsil2100, I'll move the addgroup to the test setup and logout/in before running the tests07:42
sil2100jibel: ok, thanks07:42
sil2100jibel: btw. in this case, is there a way of getting the container of the test when it times out? Since I don't see the link to the archive07:43
jibelsil2100, the archive is here for ATI http://10.97.4.138/otto/saucy-i386-20130624-0916/archive/07:45
jibelfor example the tarball for the latest platform test is http://10.97.4.138/otto/saucy-i386-20130624-0916/archive/ubuntu_13.10_saucy_salamander_alpha_i386_20130624.1372137898.otto07:45
sil2100jibel: how do I extract the link id? Since the date of the link I can guess, but then it's -0916 <- are these the hours?07:47
sil2100jibel: normally otto says at the end where to look for the container07:47
jibelsil2100, right, and that's where I found it. Which job doesn't have the link?07:48
didrockssil2100: hours, yeah ;)07:48
sil2100jibel: http://10.97.0.1:8080/job/autopilot-saucy-daily_release/203/label=autopilot-ati/console <- here, but I see a mention to "Container 'saucy-i386-20130624-0916' stopped", so I guess I know what to use now ;)07:49
jibelsil2100, interesting, it looks like a bug that the link is not displayed on timeout :)07:51
tsdgeosgreyback: did you have time to have some look at the LVWPH thing?07:53
greybacktsdgeos: I didn't, but I will today07:56
greybacktsdgeos: let me do a quick read to get familiar with the code, then we can chat so you can explain to me how it actually works07:59
tsdgeosoka07:59
sil2100jibel: give me a sign once it's reverted, ok? :)08:07
sil2100I'll re-run the stacks then08:07
jibelsil2100, it's reverted. I'm trying to do the same differently but currently facing gtk and gobject segfaults when the session restarts08:09
sil2100huh08:09
mzanettikatie: good morning. the lockscreen tweaks you suggested are landed08:12
katiemzanetti, good morning08:13
katiemzanetti, great news08:13
sil2100jibel: any way I can help?08:14
dpmmorning mhr3, pstolowski, for the scopes tutorial, we're installing the binary on /usr/lib/x86_64-linux-gnu/unity-scope-openclipart/openclipart - that makes the tutorial a bit more complex, as we then require a build system to determine the x86_64-linux-gnu part in the path and replace it in the Exec= line of the DBUS .service file. Would it be ok to install to /usr/bin instead?08:15
dpmI'd like to make the tutorial simple, but at the same time, I'd like to make sure we install to the standard location for scopes08:15
jibelsil2100, that's fine, I'm testing a fix and should have something working within 15min08:15
mhr3dpm, once the scope loader branch lands, you'll be installing a .so, and that should definitely go somewhere in /usr/lib/...08:16
mhr3+arch_triplet of course08:17
dpmmhr3, gotcha. But as we're not yet creating a .so, I guess for the purposes of the first tutorial iteration, it should be ok to install to /usr/bin, and then update the tutorial and code when the scope loader branch lands?08:18
mhr3dpm, why complicate it later, when it can remain the same? :)08:21
mhr3ie be complicated from the start :P08:21
dpmmhr3, if I understood it correctly, the .service file will go away, so if that's the case, we won't need to do the .service.in file Exec= line replacement in the tutorial's makefile, so things should even be easier. Is this assumption correct?08:23
mhr3dpm, probably not in the first iteration, might happen later08:23
mhr3for now we'll still have the .service file even once it's a module08:24
mhr3and it'll say something like Exec=scope-dbus-runner /path/to/the.so08:25
dpmmhr3, ok, thanks, I'll bring the Exec line replacement bits back into the tutorial, then08:25
mhr3or maybe scope-dbus-runner /path/to/the.scope08:25
dpmok, that makes it clearer, thanks!08:25
mhr3dpm, well the latter would allow you to get rid of the arch triplet08:26
mhr3as .scope files are not arch-specific08:26
mhr3although ultimately that only means that the .scope file will need to include path to the .so together with the triplet, so you just can't get rid of it :)08:28
dpmbummer08:28
mhr3well, unless you convince platform that scopes don't need to be tripeltted08:29
seb128mhr3, well, if those are not arch specific, they can go to /usr/share?08:30
mhr3fwiw compiz's plugins aren't08:30
mhr3seb128, they're native binaries08:31
seb128mhr3, binary = arch specific (e.g compiled code)?08:31
nic-doffaymzanetti, looks like the one branch didn't land.08:31
nic-doffayReapproving now08:31
jibelsil2100, I added the notion of setup-hook to otto, place any executable in ./usr/local/share/otto/setup-hooks/ and it will be executed at the end of otto-setup08:31
mzanettinic-doffay: wait08:31
nic-doffayJenkins.08:31
mhr3seb128, yes08:31
nic-doffaymzanetti, ?08:31
mhr3"native binary"08:31
mzanettinic-doffay: Saviq wanted to give it a look too08:31
jibelsil2100, I added a hook that adds ubuntu to group autopilot08:31
mzanettinic-doffay: or which one are you talking about?08:32
jibelsil2100, you can re-run the stacks08:32
sil2100jibel: \o/ Good idea :)08:32
nic-doffaymzanetti, launcher folding.08:32
sil2100jibel: excellent! Awesomeness08:32
seb128mhr3, right, I misread your ".scope files are not arch-specific"08:32
seb128mhr3, ignore me ;-)08:32
Saviqnic-doffay, mzanetti yeah, I'll have a quick look soon08:33
sil2100jibel: seems to work! I ran the QA stack and the tests started running correctly :)08:38
jibelsil2100, great, no segfault from compiz or a libg(tk|object) ?08:40
sil2100jibel: hmmm, no segfault, but I got a UInputError('"/dev/uinput" does not exist or is not a character device file - verify that the uinput module is loaded',) again on autopilot autopilot tests, let me check the container08:42
sil2100jibel: i.e. as if the ubuntu user was not added to the autopilot group again08:43
sil2100Will know in a moment08:44
nic-doffaySaviq, I'm writing a small test to compare the TV with the current ListItems (since even I'm unfamiliar with exactly what they look like once implemented) are there any off the top of your head that you think I can just skip?08:44
sil2100jibel: although I see /var/log/upstart/otto-setup.log: I: Adding user ubuntu to group autopilot08:44
nic-doffayBy TV I mean the sidebar..08:44
Saviqnic-doffay, not sure what you mean?08:44
Saviqnic-doffay, they're really completely different components, only thing to compare is the behaviour08:45
Saviqnic-doffay, which is kind of visual08:45
nic-doffaySaviq, yeah that's why I'm doing this test.08:45
nic-doffaySo design can see the behaviour side by side.08:45
Saviqnic-doffay, mhm08:46
sil2100Strrrange08:46
sil2100jibel: it seems that the ubuntu user is correctly added to autopilot08:46
Saviqnic-doffay, biggest issue with the sidebar is it doesn't do mouse input (on purpose)08:46
nic-doffaySaviq, so in your opinion should just that be worked on and then shown?08:46
Saviqnic-doffay, so comparing side-by-side is gonna be difficult08:46
sil2100jibel: I'll look into why, but on your side all is ok ;) Diving in!08:47
Saviqnic-doffay, just compare what the doc describes, and the SDK ListItems08:47
Saviqnic-doffay, I don't think the UTV implementation has anything to do here - it's just something we could start working off of when we decide what needs to be worked on08:47
vesarI'm getting module "Unity.Notifications" is not installed". Any pointers what to install to  get around it?08:47
Saviqvesar, qtdeclarative5-unity-notifications-plugin08:48
vesarSaviq, thanks08:48
Saviqvesar, should have gotten installed through ./build -s, did it not?08:48
jibelsil2100, the device is probably protected by the container. It should by allowed in the config and bindmounted from the host, let me try something08:49
vesarSaviq, no it didn't for me at least. I ran rm -rf ../unity_build, ./build -s, ./build -c08:49
sil2100jibel: that makes sense and would rationally explain what's going on08:50
Saviqvesar, are you on trunk? from lp:unity/8.0?08:51
vesarSaviq, no. That is one of mzanettis launcher branches08:51
dednickSaviq: ping. would you happen to have anymore review of indicators-client branch yet?08:52
Saviqdednick, never got to it yet, but it's at the top of my TODO, so something'll be there for sure08:52
dednickSaviq: :) ok. no prob08:53
Saviqdednick, there's one thing that you replied to, though, the setSource() with properties08:53
Saviqdednick, you wrote that they're dynamic, but what's the problem there?08:53
Saviqdednick, I didn't mean that you should just write js objects {prop: "value"} in QML08:53
Saviqdednick, but export a compatible map from C++08:54
Saviqdednick, and just pass it to setSource instead of doing the thing in onLoaded08:54
jibelsil2100, http://paste.ubuntu.com/5797934/ in r22608:55
jibelapplied and deployed08:55
sil2100uh :)08:55
sil2100I wonder how those tests passed in the past?08:56
dednickSaviq: ok. were you meaning doing a:08:58
dednickLoader.setSource("x.qml", { "indicatorProperties": indicatorProperties } )    OR,08:58
dednickLoader.setSource("x.qml", { "busName" : indicatorProperties["busName"]; ... })08:58
dednickexcuse the bad qml.08:58
sil2100jibel: let me re-run QA then, thanks!08:58
Saviqdednick, the latter, but the { } object built in C++08:59
jibelsil2100, if it fails again, I'm out of idea :)08:59
Saviqdednick, so, Loader.setSource("x.qml", indicatorProperties), ideally08:59
sil2100jibel: ouch!09:01
sil2100http://10.97.0.1:8080/job/autopilot-saucy-daily_release/215/label=autopilot-ati/console09:01
sil2100jibel: 2013-06-25 09:00:39,573 ERROR Can't start lxc container09:01
jibelbah09:02
dednickSaviq: I'm not exactly sure what you mean by the c++ map. Is it a specific type of object which does the "property transfer"?09:08
Cimimzanetti, doing your review, sorry yesterday afternoon got headache and mostly stop working...09:09
Saviqdednick, I *think* a QVariantMap should work09:10
dednickSaviq: ok. i'll give it a try09:11
dednickSaviq: i actually click what you mean now. I didnt realise you can just pass a map and it would set all the individual props for you.09:12
Saviqdednick, yeah, I felt there was some misunderstanding09:12
jibelsil2100, I re-ran QA http://10.97.0.1:8080/job/autopilot-saucy-daily_release/label=autopilot-ati/216/console09:15
jibelbut it failed, can you check why09:16
Saviqdednick, I was trying to read through the code to find out whether it would work http://qt.gitorious.org/qt/qtdeclarative/blobs/5f1b7bf39298dafdd07e576eec2a6a367e80b264/src/quick/items/qquickloader.cpp#line46309:17
Saviqdednick, and as long as http://qt.gitorious.org/qt/qtdeclarative/blobs/5f1b7bf39298dafdd07e576eec2a6a367e80b264/src/quick/items/qquickloader.cpp#line899 can deal with it09:18
Saviqdednick, then it should09:18
dednickSaviq: yeah, i think it should be ok09:18
dednickwill know in a few minutes09:18
Saviqdednick, cool, thanks09:18
nic-doffaySaviq, just added the ExpandingDropDown item to a test. Thing is it references the dropDown component which is null. Do you have an idea why this is the case?09:22
Saviqnic-doffay, it needs to talk to the Sidebar, to be able to find out how far it can / has to expand09:23
Saviqnic-doffay, actually09:23
SaviqdropDown in ExpandingDropDown is just an internal object (AbstractExpandingDropDown)09:24
Saviqnic-doffay, you need to make sure it's available, too09:24
jibelsil2100, hm, inside the container the device uinput simply doesn't exits09:24
nic-doffaySaviq, so the dropDown properties are set by the parent sidebar item?09:25
nic-doffayBecause atm it has all the aliases which reference the dropDown.09:26
Saviqnic-doffay, by whatever uses ExpandingDropDown, yes09:26
Saviqnic-doffay, EDD is just a wrapper around the Abstract one09:27
Saviqnic-doffay, so it's not necessarily the sidebar, but just something that sets it up correctly09:27
nic-doffaySaviq, got it.09:27
dednickSaviq: needs to be an object :(09:29
Saviqdednick, yeah, that's what I was afraid of, we'd need to cast ourselves09:30
Saviqdednick, not worth it, probably09:30
sil2100jibel: hmm09:35
sil2100jibel: I thought that every Ubuntu system has that device, as the module should be built in?09:35
greybacktsdgeos: found 1 bug, can you check it: https://code.launchpad.net/~aacid/unity/UseC++LVWPH/+merge/168073/comments/38162809:37
tsdgeosgreyback: is it the one opening the app?09:37
tsdgeosjust found that myself09:37
tsdgeosok, so no :D09:37
greybacktsdgeos: I resize the LVWPH, but it doesn't redraw09:37
tsdgeostrue, that's easy to fix09:38
greybackyep, should be09:38
* tsdgeos investigates the open bug he just found first09:38
greybackI'm doing more functional testing, and have read about 30% of the code so far09:38
greybackI'm gathering questions, so I'll be bothering you later09:39
tsdgeosoki09:42
sil2100jibel: actually... hmmm!09:43
nic-doffaySaviq, here's what my test looks like at the moment. I'm getting a label at the top but nothing's being appended to the dropDown.09:46
nic-doffayhttps://pastebin.canonical.com/93345/09:46
sil2100jibel: I'm investigating that, since maybe we shouldn't even run those tests09:47
Saviqnic-doffay, unless you have an isolated issue, I'm afraid I can't help, just dig through it - put some console.log()s in ther eto see what's happening09:47
jibelsil2100, ok, but I don't understand why this device is not there while it's on the host. what we do is pretty straightforward. it's a bindmount udev, cp the content, umount. Unless it's removed afterwards it should be there09:50
sil2100jibel: we could probably work-around it by creating the device with mknod by ourselves, but I wonder why it's not being copied09:53
sil2100jibel: since I confirmed with the guys and having /dev/uinput seems essential09:53
jibelsil2100, that's what I did to unblock the situation, now I get /var/local/autopilot/autopilot.log: UInput: UInputError('"/dev/uinput" cannot be opened for writing',)10:09
jibelpushing another fix10:09
jibelat some point we'll succeed10:09
jibelsil2100, this run should be good http://10.97.0.1:8080/job/autopilot-saucy-daily_release/label=autopilot-ati/222/console10:14
jibelat least it'll get rid of that uinput error10:14
jibelit's like if the udev rule from python-autopilot didn't apply actually10:15
sil2100\o/10:24
sil2100jibel: oh?10:25
sil2100Maybe because we're doing that copy of the container10:25
sil2100mlankhorst: hi!10:28
sil2100mlankhorst: do you know how the testing of the new X-stack is proceeding?10:28
mlankhorstafaik it's ready10:29
sil2100\o/10:29
sil2100seb128: ^ ? Do you know anything about that?10:30
seb128sil2100, did you read the call from testing email from mlankhorst on ubuntu-devel@?10:30
sil2100seb128: uuh, no, but let me make sure I'm on that list then? I thought I was10:31
mlankhorstbut it seems people are more interested in retiring pandaboard than x-stack, so I have all the test results I need ;)10:31
seb128sil2100, https://lists.ubuntu.com/archives/ubuntu-devel/2013-June/037386.html10:31
seb128mlankhorst, well, people are more interested in making sure that the xorg upgrade doesn't break unity or Mir or slow down work from other teams by breaking the system of the engineers working there10:32
mlankhorstmir's not really affected, it is just a small patch on top of X10:32
sil2100seb128: thanks, it seems I was not subscribed to that mailing list sadly...10:33
seb128sil2100, you should subscribe ;-)10:33
sil2100seb128: I did just now ;)10:33
mlankhorstand their ddx's would need another rebuild too, but that should be about it..10:34
seb128mlankhorst, ok, let's wait to hear back from olli today and give them a few extra days of testing then we can copy the xorg update to saucy10:34
sil2100mlankhorst: so, when do you think we could get things pushed to -proposed ?10:34
mlankhorstwhen seb128 thinks it's ok really, I looked at the mir xorg parts before and it seems that the ddx's would just need another rebuild bump, and the xserver patch rebased :/10:35
sil2100jibel: I see that the uinput error is no more on the -ati machine - did you also do the same for -intel? Since in the latest QA test result -intel was still failing because of that10:37
sil2100jibel: thanks for dealing with this issue btw.!10:37
mlankhorstbrb, food10:38
jibelsil2100, done, deployed everywhere10:42
sil2100\o/10:42
=== tvoss is now known as tvoss|quick_lunc
sil2100Re-running stuff then!10:43
mhr3didrocks, gentle reminder about master scopes virtual pkg :)10:45
didrocksmhr3: do you mind gently reminding me that in ~2 hours? :)10:46
didrocksI think I'll handle it then!10:46
sil2100mhr3: were you able to find out something regarding the DBus issues with unity? ;)10:47
didrockssil2100: did you gently reminding him? :)10:47
mhr3didrocks, reminder set up ;)10:48
didrocks;)10:49
mhr3sil2100, nope, didn't see you guys ping me about a live system that can be diagnosed10:50
sil2100Didn't know you were waiting for that! Ok, let me remind you next time we run and run into that10:51
CimiMacSlow_, file:///home/cimi/Development/unity/8-launcher-new-folding/Shell.qml:31:1: module "Unity.Notifications" is not installed10:55
Cimi     import Unity.Notifications 1.0 as NotificationBackend10:55
CimiMacSlow_, which is the package? :)10:55
MacSlow_Cimi, it's...10:56
MacSlow_Cimi, qtdeclarative5-unity-notifications-plugin10:56
CimiMacSlow_, we should add to ./build -s10:56
MacSlow_Cimi, well it is part of the build-script10:57
CimiMacSlow_, it didn't install here10:57
MacSlow_Cimi, when did you pull from trunk last time?10:57
CimiMacSlow_, mmm I used this branch10:57
=== MacSlow_ is now known as MacSlow
Cimi8-launcher-new-folding10:57
MacSlowCimi, don't know that one :)10:57
sil2100jibel: all passed \o/ Awesome!10:58
sil2100jibel: for the qa stack, re-running all the others now!10:58
Cimiin any case doesn't start here :\10:58
MacSlowCimi, perhaps it does not have the fixes regarding the build-script10:58
MacSlowCimi, did you install it?10:58
MacSlowCimi, manually I mean10:58
jibelsil2100, Good news, now trying to find why the udev rule doesn't create that device.10:59
CimiI did apt-get install10:59
MacSlowCimi, do you have /usr/lib/x86_64-linux-gnu/qt5/qml/Unity/Notifications.1 ?11:00
MacSlowCimi, well path of course depends on you machine architecture11:00
CimiI'm trying with trunk11:01
Cimiwho codes in a vm?11:10
CimiI run ./run and nothing happens11:10
Cimioh no, it does now11:10
* Cimi shuts up11:11
Cimimzanetti, ping11:11
MacSlowCimi, you have you dependency-issue sorted out now?11:13
CimiMacSlow, yep11:13
mzanettiCimi: hey!11:13
Cimihey boss11:13
mzanettiboss? there's a misunderstanding somewhere11:14
Cimilol11:14
mzanetti:D11:14
Cimianyway slave11:14
mzanettibetter now :D11:14
CimiI am not sure the thin divider asset is correct11:14
=== MacSlow is now known as MacSlow|lunch
Cimiis it?11:14
Cimilooks opposite to the other dividers we have, white on top and black at bottom11:15
mzanettiCimi: wow! good catch!11:15
* mzanetti fixes11:15
mzanettiCimi: this happens because the launcher can be inverted11:15
Cimiok11:15
mzanettiwhich in fact rotates the whole thing by 180 degrees and I forgot swapping the divider in that case11:16
Trevinhofginther: this should fix your issues https://code.launchpad.net/~3v1n0/bamf/ignore-autostart-desktop-files/+merge/17127411:21
Trevinhoseb128: yours as well ^^ ;)11:22
seb128Trevinho, great! do you need testing for that fix?11:22
Saviqtsdgeos, mzanetti, looking at QQuickLoader::setSource() http://qt.gitorious.org/qt/qtdeclarative/blobs/5f1b7bf39298dafdd07e576eec2a6a367e80b264/src/quick/items/qquickloader.cpp#line52511:24
Saviqcan you think of something we could build in C++ to pass as the second argument?11:25
Saviqother than v8::Object, that is11:25
tsdgeosSaviq: nope, why you need that?11:28
Saviqtsdgeos, we need to set some properties on the item when it's Loaded11:28
Saviqtsdgeos, but that's fine, we're just doing it in a loop11:28
tsdgeosfor my LVWPH tests i needed to excercise one of those functions that took a v8thing and i ended up going the C++ -> metaCall to JS -> function11:28
tsdgeos:D11:28
Saviqwas hoping there would be a better way with Loader.setSource()11:28
Saviqtsdgeos, nice...11:29
sil2100didrocks: https://code.launchpad.net/~sil2100/cupstream2distro-config/fix_sdk_graphicaleffects/+merge/171276 <- quick-fix11:31
didrockssil2100: approved, please redeploy :)11:32
sil2100didrocks: thanks :)11:32
didrockssil2100: FYI, I'm changing the versioning scheme as per #ubuntu-release, more on that soon11:32
mzanettiSaviq: sorry. was focused on something. is the question still valid?11:34
Saviqmzanetti, if you can come up with something, sure11:34
mzanettiSaviq: I don't understand the use case yet11:35
Saviqmzanetti, you have a Loader11:35
Roxastryhello all. Who know, how set icon in system tray without save in file system form application if this icon storage in pixbuf in application.11:35
Roxastrysorry for my bad English =)11:35
Saviqmzanetti, a source url, and a set of properties to set on Loader.item when it's constructed11:35
Saviqmzanetti, that's exactly what Loader.setSource(url, properties) is for11:35
mzanettiSaviq: something like pageStack.push("MyPage.qml", {foo: "bar}) ?11:36
Saviqmzanetti, yeah11:36
Saviqmzanetti, but in our case we'd need to provide the properties map from C++11:36
Saviqmzanetti, and AFAICS that's impossible short of building a v8::Object in our C++, which I'd rather not do :)11:36
Saviqmzanetti, and we have a "workaround" by the means of a loop, so it's mostly an academic problem11:37
mzanettiSaviq: can I see the workaround?11:37
Saviqmzanetti, sure11:40
Saviqmzanetti, https://code.launchpad.net/~nick-dedekind/unity/8.indicators-client/+merge/168022/comments/37768511:40
Saviqmzanetti, line 55 of the diff mentioned11:41
mzanettiSaviq: didn't you say in C++?11:42
Saviqmzanetti, the workaround is in QML ;)11:42
Saviqmzanetti, but "indicatorProperties" come from C++11:43
=== tvoss|quick_lunc is now known as tvoss
=== MacSlow|lunch is now known as MacSlow
didrocksmhr3: it's already on its own binary package in fact12:20
didrocks(the -common one)12:21
didrocksit just needs to provide a virtual package to be changeable I guess12:21
didrocksmhr3: so, this should be enough: https://code.launchpad.net/~didrocks/libunity/provides-untity-scopes-json-def/+merge/17128812:24
didrockssil2100: I've deployed the new versionning scheme on all stack, but unity/head12:24
didrockssil2100: if you want to build a new package from this stack, you'll have to redeploy it (from latest cupstream2distro-config trunk) first12:26
didrockssil2100: ah, unity stopped, so deploying12:28
jibelsil2100, FYI the problem with uinput is that udev doesn't create input devices in the container (other input devices under /dev/input are bind mounted from the host).12:32
jibelSo our workaround with mknod is good enough, I'll just move it to autopilot setup instead of otto.12:32
sil2100didrocks: ACK, thanks!12:44
sil2100jibel: excellent, thanks for the heads-up, I guess this workaround is good enough for us for now12:45
sil2100didrocks: is a re-run for every stack required, or can I publish those with the old versioning still?12:47
dednickdandrader: ping12:47
dandraderdednick, pong12:47
dednickdandrader: hey. how's the dda work going?12:47
dandraderdednick, it's done. but I can't propose it as it depends on https://code.launchpad.net/~dandrader/unity/8_dragHandle/+merge/17017212:48
dandraderwhich no one reviewed yet12:48
dednickdandrader: i can take a look if you want12:49
dandraderdednick, that would be great12:49
mhr3didrocks, nooooo12:52
mhr3didrocks, should have provided more details, sorry, i was talking about the home scope12:53
mhr3didrocks, there we have a bunch of .scope files, and want to split them up into a separate bin pkg12:53
mhr3didrocks, but since you already did that for the json definition as well, i think that's useful, afterall there should be different client-scopes for desktop and phone :)12:54
=== mmrazik is now known as mmrazik|afk
dednickdandrader: one quick one. there is a conflict with trunk.12:57
Saviqmzanetti, re: snapping... shouldn't the preferredHighlightBegin/End do exactly that? i.e. you need to set it to values that make sure that it's flat initially?13:02
mzanettiSaviq: yeah... but the preferredhighlight should be in the center13:02
mzanettithe design requirement is to distribute space equally at top/bottom13:03
mzanettiactually I think this is a bug in the ListView13:03
Saviqmzanetti, ah, so currentItem is always in the middle?13:03
mzanettibecause if you drag it manually you can well make it stick unfolded at the end13:03
mzanettibut setting contentY does not do the trick13:04
mzanettiSaviq: yes. currentItem is in the middle - but I don't use the currentItem/Index notation here13:04
mzanettiSaviq: hmm... wait a sec... I have an idea...13:05
Saviqmzanetti, you could always try flick() - it seems to do the trick when tapping on bottom/top13:05
mzanettiSaviq: yeah... just not a fan of such stuff... feels like a bad hack. but yeah. that would be my last-resort thing I guess13:05
dandraderdednick, fixed13:06
Saviqmzanetti, perfectly agree13:06
tvoss_didrocks, ping13:10
mzanettiSaviq: found it: ListView.ApplyRange - the view attempts to maintain the highlight within the range. However, the highlight can move outside of the range at the ends of the list or due to mouse interaction.13:12
Saviqmzanetti, yup13:12
mzanettiSaviq: thats why it only works when dragging/flicking13:12
=== _salem is now known as salem_
fginthersil2100, thanks for fixing the uinput permissions issue, what was the problem?13:20
=== mmrazik|afk is now known as mmrazik
sil2100fginther: hi! jibel did most of the fixing, since adding ubuntu user autopilot permissions was not enough, as it seemed that the otto container did not have /dev/uinput at all13:22
mzanettiSaviq: ok... giving up. If you don't mind I'll go with the flick() in onCompleted. seems to work prefectly fine13:22
sil2100fginther: we worked around that by creating it with mknod during otto autopilot setup13:22
sil2100And it works!13:22
fginthersil2100, that is strange. I thought uinput was one of the default udev rules. hmmm13:23
sil2100fginther: since /dev/uinput is on the machine, but gets lost during the copy13:23
Saviqmzanetti, yeah, just add a comment what it does13:23
tedgsil2100, Did the dbusmenu stuff pass last night?13:23
fginthersil2100, oh, so is that a consequence of the fs overlay?13:23
sil2100tedg: yes! I think it did, as I saw build was OK for indicators - how were you able to fix that flacky problem?13:24
sil2100fginther: most probably13:24
tedgsil2100, One of the flaky ones were timeouts, one test we decided to disable until we can figure out more info.13:24
Saviqjeez django-users... "how can I do something in a function after return"...13:25
Saviqit's like the worst place to be subscribed to13:25
sil2100tedg: that's a good choice I guess, since it wouldn't make sense to block the indicator stack just because of this one test13:26
mterrykgunn, Saviq: I have to skip standup, will fill in minutes later13:31
Saviqmterry, k13:31
kgunnmterry: later13:32
Saviqpaulliu, standup13:32
mterry(i moved yesterday, am waiting for internet hookup to arrive, but it is late)13:33
mzanettiMacSlow: bottomline: sync with veebers13:42
didrockssil2100: yes :)13:43
MacSlowmzanetti, ok... funny that there's an stand-up entry for my from yesterday although I had a day off :)13:44
didrocksmhr3: hum, that json file is used for what in the upstream side?13:44
mzanettiMacSlow: hehe13:44
MacSlowmzanetti, thanks for the heads up!13:44
mzanettiMacSlow: yeah... mostly because we were syncing with chris on autopilot and you're on that too currently13:44
didrockstvoss_: pong13:44
mmrazikdidrocks: shall we add mc-return to "trusted" users? Otherwise (unless somebody re-approves) the following will stay where it is:13:45
mmrazikhttps://code.launchpad.net/~ivenvd/compiz/compiz.fix_1193792/+merge/17097513:45
mhr3didrocks, to know which scopes are installed by default13:45
didrocksmhr3: oh right, then, we have the diff13:45
didrocksmmrazik: I think he can be trusted for compiz13:46
didrocksmmrazik: is it a global thing or per project?13:46
MacSlowmzanetti, ah... just saw that branch... that's my python regressions-tests put there13:46
mmrazikdidrocks: its a global thing. It essentially says who is allowed to run code on the private jenkins13:46
mmrazikonce there it is not a big difference against which project you create the MP13:47
didrocksmmrazik: ok, I think it's fine in that case, as long as this is not linked to merge approval rights :)13:47
mzanettiMacSlow: not sure about what branch you are talking, but yes, such a regression-tests branch has been mentioned in the meeting yesterday13:47
mmrazikdidrocks: nope. You still need to be part of the right launchpad team to do that13:47
didrocksmhr3: you want one binary package per master scope file?13:47
mmrazikfginther: FYI -- I'm adding mc-return to the jenkins config as trusted13:48
mmrazik(see above)13:48
mhr3didrocks, nah, keep them all in one pkg13:48
MacSlowmzanetti, it's the notification-backend branch from the api-team, where I put my python-based regression tests... I guess they'll be used as a starting point for the autopilot-tests13:48
MacSlowmzanetti, ah... veebers is Chris from NZ13:50
didrocksmhr3: ok, then creating unity-scopes-master virtual package?13:51
didrocksmhr3: sorry, btw, but with /usr/share/unity/scopes/info.scope, shouldn't I have an info master scope?13:51
mhr3didrocks, perhaps unity-scopes-master-default ?13:52
mzanettiMacSlow: it might make sense if you just ping him in the morning when you join the channel. usually he's around for half an hour or so13:52
mhr3didrocks, cause there can be more master scopes in different pkgs13:52
MacSlowmzanetti, yeah... also just writing him an eMail.13:53
mzanetticool13:53
didrocksmhr3: but each master scope doesn't mean we will see them in the dash, right?13:53
didrocksmhr3: I don't have an "info" icon at the bottom13:53
mhr3didrocks, no, but you have it in the Categories filter in home13:54
didrocksin my mind, there were a 1 to 1 mapping13:54
mhr3time to change your mind :)13:55
didrocksmhr3: so, it's because of no "Icon/SearchHint/ContentHint"?13:55
mhr3didrocks, we have lots of scopes, so ones that are actually visible in the dash bar is a separate setting these days13:57
fginthermmrazik, ack13:57
mhr3didrocks, (in dconf)13:57
didrocksmhr3: ah, but the choice of what to display is made within the master scope list only, right?13:57
mhr3didrocks, no, it's not limited just to master scopes, you can put "subscopes" there as well13:59
didrocksmhr3: so master scopes are just defining the categories (for deduplication) in a view (being home scope or a particular view, like the video ones)14:00
didrocksone*14:00
mhr3didrocks, yep, but all scopes expose everything that's needed for them to be displayed in the view14:01
didrocksmhr3: excellent, I think I got it then :)14:01
didrockslet me split quickly the unity-scope-home14:01
didrocks(I prefer to get it right in my mind first ;))14:01
mhr3sure, it's good when you know what you're doing :)14:02
didrocksmhr3: isn't it?14:02
=== davidcalle_ is now known as davidcalle
=== Zhenech_ is now known as Zhenech
didrocksmhr3: https://code.launchpad.net/~didrocks/unity-scope-home/ship-master-scopes-def/+merge/17131114:14
mhr3didrocks, shouldn't there be a hard dep?14:16
mhr3didrocks, i mean, the master scopes are now just recommended, right?14:17
mhr3i think we can easily do hard dep, if there's a virtual pkg, no?14:17
didrocksmhr3: right, with the alternative, it can be dep, but I think recommends would be more appropriate?14:17
didrocksmhr3: I don't really want to do a circular dep, so that would mean the definition won't dep on the service14:18
didrocksif you are fine with that, I can change it14:18
mhr3didrocks, hmmm, well without home scope the definitions are pretty useless14:18
didrocksmhr3: right, that's why there is the second dep14:19
didrocksmhr3: but they are not harmful either :)14:19
mhr3didrocks, the primary reason i don't like recommends is upgrades14:19
didrocksmhr3: ah, well, in that case, both will be delivered at the same time14:20
mhr3but if you tell me it's fine, i'll be fine :)14:20
didrocksmhr3: no, I can remove the deps the other way around14:20
didrockslet me push this14:20
didrocksmhr3: rev 12814:20
mhr3ok, i think i'm happy with that14:21
mhr3didrocks, thx a lot14:22
didrocksmhr3: yw ;)14:22
mhr3didrocks, btw i expected there's more magic associated with virtual pkgs... :)14:23
didrocksmhr3: there is some implicit :)14:25
didrocksmhr3: like for the iso to build, you need to have the real package first14:26
didrocksand no dep versionning on a virtual package :p14:26
Saviqdednick, you made me almost go down in the review progress with the indicators-client move :P14:34
Saviqs/down/back/14:34
dednickSaviq: :)14:34
=== mmrazik is now known as mmrazik|afk
dandradermzanetti, ping15:13
mzanettidandrader: pong15:13
dandradermzanetti, is it by design that dragging the launcher beyond the middle of the screen causes it to hide itself on release?15:14
mzanettidandrader: hmm... probably not. don't have anything that would specify that.15:14
mzanettidandrader: well, it should do that when minimizing an app... but I know, it does it without an app too15:16
mzanettiwhich is probably not the really good15:16
mhr3Saviq, should be a quick one - https://code.launchpad.net/~mhr3/unity/category_content_type/+merge/17134315:43
Saviqmhr3, we're wrapping at 120 columns, can you please un-wrap line 33 of the diff?15:45
mhr3are you saying i have to resize my terminal? :-O15:47
mhr3Saviq, pushed15:47
Saviqmhr3, you need to stop living in the '80s ;P15:48
mhr3Saviq, lol, good one15:52
=== jhodapp is now known as jhodapp|lunch
Saviqmhr3, I hate to be doing this to you... but we're doing 4 space indents...15:56
Saviq/we need astyle config around15:57
mhr3Saviq, eh, my bad, modelines would be nice though15:57
Saviqmhr3, indeed15:57
tsdgeospre-commit hook, not sure if possible in bzr, gstreamer had it afair15:58
tsdgeosin git15:58
Saviqtsdgeos, not possible in bzr (unless you install it in your ~/.bzr)15:59
Saviqtsdgeos, I could think of a generic hook for your ~/.bzr that would use some config file checked into your repository16:00
Saviqtsdgeos, so possible in that sense16:00
mhr3Saviq, hope the newline between the case and the { is fine16:01
tsdgeossure, would still need "manual intervention"16:01
mhr3Saviq, "case Foo: {" just looks wrong :P16:01
tsdgeosso people that are not "the usual hackers" still would have the problem16:01
Saviqmhr3, +116:01
seb128tedg, mhr3: jbicha just pinged us about lp #1155968 ... is the new version fine for unity's stack?16:01
ubot5Launchpad bug 1155968 in zeitgeist (Ubuntu) "Upgrade zeitgeist to 0.9.14" [Wishlist,Confirmed] https://launchpad.net/bugs/115596816:01
seb128or do you guys would need to port your stuff first?16:01
Saviqmhr3, we don't yet have a clear style guide in there, am trying to get to that sooner rather than later16:01
tedgseb128, They can be dual installed (I have that on my system now).  But, yes, I'd like the new version.16:02
Saviqmhr3, it's most probably gonna be Qt style, though, as it's all just Qt code, really16:02
seb128tedg, ok, thanks, I will have a look to that then16:02
mhr3Saviq, yea, whatever, it just a style... i'll get slapped for the first commit, but hopefully then i'll remember16:02
Saviqmhr3, :)16:02
mhr3Saviq, but i have no problems with it only cause it's a 20line diff commit ;P16:04
Saviqmhr3, I do feel I'd be bitten by the lack of braces around single-line case blocks16:05
Saviqmhr3, but at least in that case it dies on compilation16:05
Saviqso yeah +116:06
mhr3Saviq, it's not common to do braces around cases, needs to be done only when you have a new variable there16:07
Saviqmhr3, right16:07
Saviqmhr3, but yeah, my point - here you can't be bitten the same as with if16:08
mhr3right, i just like switches, it's miles easier to read16:09
mhr3seb128, eh, sorry, i'm not aware of breakage that new zg would cause16:10
mhr3seb128, is it needed for something? or just jbicha wants the latest and greatest?16:11
seb128mhr3, k, let's see16:11
seb128mhr3, well, both, I had to patch e.g gedit's back to undo https://git.gnome.org/browse/gedit/commit/?id=20d860b6500e8f9a8143be4df306225e206664d616:13
seb128mhr3, tedg seems keen to get the new version as well (I think they said the performances are improved)16:13
seb128tedg, you wanted the update for specific reason?16:13
mhr3seb128, yea, ted wanted the no-dbus db reading16:14
tedgseb128, I'd like to use it for tracking in HUD.  Not sure I'll get to the port this cycle as I'm getting bogged down with other stuff.  :-(16:14
tedgBut that's why I'm interested16:14
mhr3tedg, you don't need new zg for that ;)16:14
tedgmhr3, I need it for it to be reasonably performant.  The DBus reading is why we didn't use ZG at the start.16:15
=== dandrader is now known as dandrader|lunch
mhr3tedg, oh, i thought you just wanted the new ontology...16:16
tedgmhr3, No, that's just #defines, I can steal that easily enough :-)16:16
mhr3indeed16:16
* tedg already knows of that evil <evil laugh>16:16
mzanettiSaviq: tested this, works fine: https://code.launchpad.net/~saviq/unity/8.flipped-support/+merge/17106116:18
mzanettiSaviq: want me to top-approve?16:18
mzanettior wait a bit in the hope that the adb-over-tcp gets supported before we switch to flipped containers?16:19
mzanettiah, screw it.16:19
mzanettiI'll top-approve16:19
Saviqmzanetti, yeah, it's ready to top-approve16:26
Saviqmzanetti, it supports both, so it's fine16:26
=== dpm is now known as dpm-afk
=== jhodapp|lunch is now known as jhodapp
=== mhall119_ is now known as mhall119
=== dandrader|lunch is now known as dandrader
=== deegee__ is now known as drussell
=== mterry_ is now known as mterry
mterryKaleo, heyo.  Do you know if there's an API in Qt that will expose whether the display device is on or not?  I haven't found one, but I thought I'd ask someone that knew Qt well19:02
mhall119tvoss_: loved those screenshots of other DEs on XMir19:20
mhall119tvoss_: can I safely s/Unity Next/Unity 8/ on unity.ubuntu.com?19:21
mhall119also, we have documentation up there about running tests, do we have any documentation about writing them?19:22
tvoss_mhall119, glad you liked them, the replace should be fine! kgunn, any objections?19:23
kgunnmhall119: go for it19:24
tvoss_mhall119, blender works, too :) but need to take a video, screenshot is a bit boring in this case19:26
mhall119tvoss_: kgunn: on http://unity.ubuntu.com/getinvolved/development/unitynext/ under "Build dependencies" it says that build -s will:19:29
mhall119Build and install lp:libunity/phablet, lp:unity/phablet-mods, lp:hud/phablet and lp:unity-lens-people locally19:29
mhall119are those project branches still correct?19:29
kgunnmhall119: i was just looking this over19:31
kgunni don't think they are right anymore...19:31
dandradermhall119, check the latest state of the build script. it has changed quite a bit overtime.19:35
dandradermhall119, If I'm not mistaken it also works only in saucy now19:36
kgunndandrader: yep...19:36
kgunni was actually changing the wiki atm19:36
kgunnbut then feeling lost....as i think lots of stuff has changed19:37
kgunnmhall119: sorry...just realized you're editing too...i just bailed out19:37
mhall119kgunn: let me save my edits, then you can jump in19:38
mhall119kgunn: I'm only editing the WP page, not the wiki19:38
kgunnmzanetti: do you have a quick decoder ring for the ./run options ? wrt lockscreen (e.g. if set...is there a default pin & passphrase)19:38
kgunnand how to set lock (on the desktop)19:38
kgunnmhall119: that's what i meant...i was in WP19:39
mhall119kgunn: okay, I'm done now, you can go in and make any changes you need19:43
mterrykgunn, pin is 123419:49
mterrykgunn, password is password I believe19:50
kgunnmterry: ta19:50
=== thomi_ is now known as thomi
kgunnmterry: are you able to run on desktop (e.g. ./run)19:55
kgunn?19:55
mterrykgunn, you need -f for that19:56
kgunnmterry: actually same result19:56
mterrykgunn, what's the result?19:57
kgunnunless you tell me i screwed up by trying to run w/o -f one time19:57
mterryno19:57
kgunnmterry: it complains about unity.notifications not installed (shell.qml:31:1)19:57
mterrykgunn, oh yeah, that's new.  I got that last Friday when I was playing with trunk.  I didn't investigate, since it wasn't blocking me, but I thought it was unusual19:58
kgunnmterry: so did the shell window actually launch for you ? (i get nothing)19:58
mterrySaviq, mzanetti : do you know what package Unity.Notifications is in?  Seems to be missing from ./build -s run because ./run -f complains about missing Unity.Notifications.  ^19:59
mterrykgunn, no19:59
mterrykgunn, I also get nothing, just the error about notifications20:00
mterryMacSlow wrote the notification stuff, but he's not on here now20:00
kgunnmterry: not sure why it failed....weird....just manually did a apt-get install of unity-notifications-impl-1 (per the ./build script...) and it worked20:06
kgunni now see a shell20:06
mterrykgunn, aw awesome20:08
mterryI guess I need to do that too20:09
kgunnmterry: ah-ha...because its not really "unity-notifications-impl-1"...its "qtdeclarative5-unity-notifications-plugin"20:20
dandraderthe missing unity.notifications was added to the "build --setup" script on revision 3620:21
dandraderkgunn, mterry ^20:21
mhall119is there a reason why Search isn't enabled on the Unity 8 home lens?20:22
mhall119and a way to enable it20:22
Saviq_mhall119, because it's not the real home scope20:23
Saviq_mhall119, you could drop it from the list of mappings in DashContent.qml20:24
Saviq_mhall119, but that would probably kill your dash due to the amount of data (we're fixing that)20:24
mhall119ah, ok20:24
mhall119I'm really looking forward to using the smart scopes on my N720:24
kgunnmterry: dandrader ....acutally, figured it out...if you have any failures from the apt-get update after the build add's ppas....then it bails out20:33
kgunne.g. i had some armhf failures lingering in there20:33
kgunnonce i cleaned that up...it installs notifications just fine20:34
kgunndandrader: last one...what diff does  -m "no mouse touch" make...tested both, at least i can't see a real diff20:36
kgunndandrader: actually...at least no mouse touch doesn't reveal the launcher once in the dash20:37
dandraderkgunn, if you disable the conversion of MouseEvents into TouchEvents then components that handle only touch won't work (such as DirectionalDragArea)20:38
dandraderkgunn, right not part of the edge drags is MouseArea based and part DirectionalDragArea based, so you get mixed results20:39
dandraders/not/now20:39
kgunndandrader: ok...is there some specific way that's useful ?20:39
kgunnlike why would you ever want to use -m (sorry if its obvious)20:40
dandraderkgunn, well, if you wanna a setup that matches the device you will want to disable that conversion20:40
dandraderkgunn, but it's more of an hypothetical scenario than a real one20:41
kgunndandrader: yeah...i guess from a perception point of view...no -m seems to work just fine (and more like i'd expect)20:42
=== seb128_ is now known as seb128
=== jhodapp is now known as jhodapp|afk
=== seb128_ is now known as seb128
=== seb128_ is now known as seb128
=== seb128_ is now known as seb128
kgunnmhall119: hey, i was getting around to looking at updating instructions....but....seems the page has gone missing22:07
kgunnhttp://unity.ubuntu.com/getinvolved/unitynext22:07
kgunnnvmd....22:07
kgunni see22:07
kgunnunity822:07
kgunncool22:07
=== salem_ is now known as _salem

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