/srv/irclogs.ubuntu.com/2012/10/05/#ubuntu-unity.txt

=== cyphermox_ is now known as cyphermox
=== mhall119_ is now known as mhall119
ricotzracarr, hi :)07:22
ricotzracarr, do you think you could make bamf's dependency on libunity-webapps optional?07:22
jbichahey, we have an issue with yesterday's unity update07:44
jbichabecause of the fix for bug 1055019, deja-dup, empathy, nautilus, & shotwell depend on libunity9 which depends on unity-common which depends on compiz-gnome07:44
ubot5Launchpad bug 1055019 in libunity (Ubuntu) "libunity9 should depend on unity-common" [Medium,Fix released] https://launchpad.net/bugs/105501907:44
jbichabut the Ubuntu GNOME Remix doesn't ship compiz...07:44
Mirvjbicha: uh oh :( so you had already libunity9 installed, but now the deps bring compiz as well? that's indeed not pretty08:08
sil2100This is hm, indeed problematic08:09
sil2100I think the dependency to compiz-gnome is due to unity-common including the setting migration files08:14
sil2100seb128: hi!08:15
seb128sil2100, hey, how are you?08:15
sil2100seb128: rather fine, thanks, how about you?08:16
seb128sil2100, I'm good thanks08:20
jbichaI filed bug 106209908:23
ubot5Launchpad bug 1062099 in libunity (Ubuntu) "libunity9 now depends on unity-common which depends on compiz" [Undecided,New] https://launchpad.net/bugs/106209908:23
jbichaMirv: I mean we'd prefer not to include libunity9 either but that's not possible (at least not this release)08:24
seb128jbicha, Mirv: I guess the easiest way out for this cycle is to revert the Depends and make the code bail out nicely when the schemas is missing08:27
jbichaunity-mail should check if the schema exists before trying to call it08:31
seb128it doesn't call it, it just uses libunity which uses a key from the schemas08:34
mhr3it should be fixed in libunity, it's a foreign schema, it should deal with it not being there08:50
didrockssil2100: so, you did ping upstream? I think we should fix that quickly08:51
* didrocks is concerned to not see things being act on here, apart from the great mhr3 always watching :)08:52
didrocksmhr3: I have a simple recipe to detect the schema, but I'm not confortable with the side effect of not setting a preference value08:52
didrocksmhr3: so I think you would be the best to do the change08:52
didrocksI think you know about the "list_schemas" hack?08:53
mhr3i'll have to fix tests as well anyway, they expect the schema is there too08:53
didrocks(I think we should really have an upstream glib call for it)08:53
mhr3+108:53
didrocksI'm having this hack in 6 places already :/08:53
mhr3whose idea was it that crashing an app is better than using a GError? :P08:54
mhr3hmm desrt?08:54
didrocksyep :)08:54
didrocksand he's quite adamant for it08:54
sil2100didrocks: once mhr3 is ready with a fix, should we distro-patch it?08:57
didrocksyeah08:57
didrocksthanks mhr3 for noticing it yourself. I was excepting you would be warned by the integration team (and was waiting on that)08:58
popeydidrocks, we were just discussing it before you arrived08:59
didrockspopey: since 10h10? :/09:00
didrocksand where the discussion was going on? should be on this channel, isn't it?09:01
popeyit wasn't here no09:01
didrocksit should though09:01
didrocksand 50 minutes to discuss about pinging upstream seems to be quite serious…09:01
popeyi was actually waiting for you to get off your hangout to ping you, but you saw it first09:01
didrockspopey: well, I saw it at 9:3009:02
popeygreat.09:02
didrocksthen, I told jbicha to ping you guys here09:02
didrocksthen first answer 24 minutes after the concern was raised, and no upstream ping in 50 minutes, I'm concern about the effectivness TBH09:03
=== mmrazik is now known as mmrazik|lunch
mhr3sil2100, didrocks, proposed the libunity branch09:43
didrockssweet! we need someone testing it without having the schema installed (and so not compiled, removing the schema is not enough)09:44
mhr3fwiw i tried by just settings XDG_DATA_DIRS to something weird09:45
didrocksthat should work, as long as the app using it doesn't have schema installed :)09:46
mhr3otoh didn't try to call any of the methods in that case09:46
mhr3but since all just lookups into empty hashtable they should just return false09:47
didrocksmhr3: maybe trying with empathy and other from the list above? ^09:48
didrocksnautilus is a good one09:48
didrocksIMHO09:48
mhr3didrocks, i don't think nautilus is one of those apps that can survive without any schemas :)09:51
didrocksyeah, that's why rm schema + recompiling them09:51
didrocksmhr3: who is reviewing your branch?09:51
mhr3didrocks, ehm, you? :)09:51
didrocksas the integration team will probably prepare the distro-patch, I think they will test it at the same time, so they should?09:52
mhr3sil2100, ^09:53
didrocksI must admit that it's more elegant that in C :)09:53
didrocksschema in Settings.list_schemas ();09:53
didrocks:)09:53
mhr3hmm... why didn't i do that?09:54
mhr3i guess i like extra variables :P09:54
didrocksseems so :)09:55
didrocksto show you know the "unowned" :p09:55
mhr3right, what if vala copied the array in that case? that would be horrible waste of cpu and memory :P09:56
didrocksheh09:57
didrocksmhr3: so vala copies everything by default?09:57
mhr3would surely waste at least 3nanoseconds09:57
mhr3not in that case :)09:57
didrocksyeah, but if it's not unowned?09:58
jokerdinohey, unity dash looks weirdish with no blur. is that a known bug?09:59
mhr3then yea, string[] arr = get_something_unowned(); will copy the array09:59
sil2100ugh, sorry, I was busy copy-pasting some bugs09:59
didrocksinteresting09:59
sil2100Looking at the merge request09:59
didrocksmhr3: but string a = b;09:59
mhr3didrocks, that too10:00
didrocksand if I want to share the same ref? it needs to be unkowned on a?10:00
mhr3yep10:00
didrocksbut then, you need b to go out of scope after a10:01
mhr3exactly10:01
didrocksthere is no way to still have automatic freeing generation with b going out of scope before a? and they still share the same reference?10:01
mhr3so unowned needs to be used with care10:01
mhr3didrocks, well what would you do in that case?10:02
didrockscounting the number of reference :)10:02
mhr3the reference is invalid if b gets out of scope10:02
didrocksyeah, but there can be a "magic" keyword telling a = b target the same ref10:02
mhr3yea, of course for ref-counted objects Obj a = b; just increases the refcount10:02
didrocksok, just not on basic types?10:02
didrocksas string a = b; triggers a copy10:03
mhr3string is always special :) int a = b also doesn't need references :)10:03
didrocksok, it's only for the basic types :)10:03
mhr3and structs10:03
didrocksI was starting to think it was for everything10:03
didrockshence my "getting scared" :)10:04
mhr3no worries, it's sane :)10:04
didrocksheh10:05
sil2100davidcalle: ping10:10
=== mmrazik|lunch is now known as mmrazik
davmor2popey: can you confirm a bug for me please.  from a fresh start on quantal flick through the dash lenses how many of the See more results arrows are pointing down instead of right? I'll go grab you the bug number10:29
popeysure10:30
davmor2popey: nevermind bug got duped so is already confirmed https://bugs.launchpad.net/unity/+bug/105304710:32
ubot5Ubuntu bug 1053047 in unity (Ubuntu) "Dash - Category expander arrow is in expanded mode no metter if category is actually expanded." [Medium,In progress]10:32
popeyFINE!10:32
popey:)10:32
davmor2popey: I was looking at mine rather than the dupe and wondering why it wasn't confirmed10:33
davmor2popey: I blame my cold10:34
jokerdinopopey: have you found any bug with dash and no blur setting?10:34
davmor2popey: although gdocs is still desparate to download the spreadsheet I want to interact with as a pdf :D10:35
popeyjokerdino, can you be more specific?10:48
Mirvwasn't there this "jumping" of background with no blur?10:55
MirvI think it was mentioned, but I'm not sure if a bug was filed10:56
dandraderthe launcher in my desktop only comes up if I press the super key. It no longer shows up when I push the pointer against the left screen edge. I'm in Quantal. How can I fix it?10:58
popeydandrader, nvidia?11:00
dandraderyes11:00
popeybug 105700011:00
ubot5Launchpad bug 1057000 in xorg-server (Ubuntu) "[Ubuntu 12.04.1/12.10] nVidia drivers 304.51 prevent autohidden Unity launcher from revealing" [High,Triaged] https://launchpad.net/bugs/105700011:00
popeyprobably11:00
dandraderpopey,  hmm, ok. I'll have a look at it. Thanks!11:01
popeyjokerdino, bug 105839111:01
ubot5Launchpad bug 1058391 in unity (Ubuntu Quantal) "[no blur] Desktop rolls over unity panel vertically when i open the dash or hud" [High,Confirmed] https://launchpad.net/bugs/105839111:01
popeyMirv, that one? ^11:01
Mirvpopey: yes, that one11:04
=== MacSlow is now known as MacSlow|lunch
jokerdinosorry and thanks popey. i think that's the bug i was looking for11:24
popeynp11:26
=== dandrader is now known as dandrader|afk
didrockssil2100: Mirv: is libunity backported and tested now?12:20
sil2100didrocks: tested - yes, backported - no... but I'll do that in a moment, just need 5 more minutes12:28
didrockstested without the schema installed, right?12:30
didrocksbackporting then is just a 2 min manipulation…12:30
=== dandrader|afk is now known as dandrader
sil2100didrocks: I know, I should have done it ASAP once the testing was done, just picked up something else to do in the meantime ;(12:37
=== MacSlow|lunch is now known as MacSlow
=== _salem is now known as salem_
didrockssil2100: didn't I say this one would be urgent and ask for working on it since this morning? :/12:40
didrocksTBH, I think there are some priotization issue on your team (one hour to start pinging upstream… 30 minutes to tell "oh it's vala, we should get someone else to review it" and right now, more than one hour and 20 minutes since it's merged upstream and still not backported)12:41
didrocksso it will soon be too late to push it for the week-end12:41
sil2100https://code.launchpad.net/~sil2100/libunity/ubuntu_6.8.0-212:48
sil2100Packaging with the cherry-pick12:48
sil2100I removed the unity-common dep as well12:48
Mirvthanks sil2100, cherry-pick is the right way to go atm so that it's gotten in as soon as possible12:49
didrockssil2100: thanks, looking12:50
didrockssil2100: did you try your cherry-pick?12:51
sil2100didrocks: the cherry-pick I just posted was test-built, but its almost the same thing as what I tested basically12:53
didrockssil2100: test-built but not tested, right?12:54
sil2100I can re-test it without the schemas now again12:54
didrockswell, you should12:54
didrocksbecause from what I can see it's wrong12:54
didrocksI'm sure mhr3 can find quickly what's wrong :)12:55
didrockssil2100: still don't get it? I clearly showed the issue at the integration sprint with vala projects…12:58
sil2100didrocks: still looking, give me a moment more13:01
Davieywait, didrocks.. you know the cause of a problem, and there is wasted effort in trying to duplicate the discovery ?13:16
didrocksDaviey: well, popey's teaùm13:17
didrocksDaviey: want to progress13:17
Davieydidrocks: and you know the answer, and haven't shared it?13:17
didrocksDaviey: the issue already happened 3 times in the past and I did showed it13:17
DavieySO you are proving a point now?13:17
didrocksDaviey: so it's some kind of training for them to find it until they ask13:17
didrocksDaviey: hence his "give me a moment more"13:18
tsdgeosany idea how to unit test https://code.launchpad.net/~aacid/unity/do_not_reuse_menus_on_order_change/+merge/128243 ?13:18
DavieyThis is a total waste of time didrocks13:18
tsdgeosTrevinho: ↑13:18
seb128Daviey, some people seem to not learn when you don't let them think by themself about the issue for a while13:18
Davieyseb128: Have they requested this sort of support?13:19
Trevinhotsdgeos: for the change in Indicator::GetEntry there are unit tests13:19
seb128Daviey, what "support"?13:19
Trevinhotsdgeos: just integrate them with your api change13:19
tsdgeosTrevinho: i'm more concerned about the dbus one13:20
Davieyseb128: exactly.13:20
Trevinhotsdgeos: there are still no tests for that...13:21
Trevinhotsdgeos: I started doing it some time ago, but it was too long and I didn't continue due to other priorities13:21
Trevinhotsdgeos: however, you can use Autopilot, or.... (much better) would be adding a new test suite to test-gtest-dbus13:21
Trevinhotsdgeos: basically you need to add a fake unity-panel-service there13:21
Trevinhoand use DbusIndicators to connect to that13:22
Trevinhotsdgeos: at that point you can test everything you want13:22
tsdgeosok, that's going to be a hard exercise :D13:22
Trevinhotsdgeos: however, imho instead of changing Entry::Ptr Indicator::GetEntry, imho it's better to add Indicato::EntryIndex method13:23
Trevinhotsdgeos: yeah :)13:23
Trevinhotsdgeos: more boring than hard13:23
tsdgeosTrevinho: well, never done any of that dbus stuff, so i need to do some c&p and be lucky ;-)13:23
tsdgeosTrevinho: well, the think is that i want both the entry and the index13:24
tsdgeosfeels a bit silly doing the loop twice13:24
Trevinhotsdgeos: yeah, I know.. but it should not really impact a lot13:24
tsdgeosor you mean that EntryIndex should return both the entry and the index as my modified GetEntry ?13:24
mhr3didrocks, can i? i see it properly cherrypicked13:24
didrocksmhr3: vala man, vala when shipping c…13:25
tsdgeosTrevinho: ok, i'll add the new function13:25
mhr3didrocks, we changed the build system, no more .c files ;)13:25
didrocksmhr3: we still have them in the packaging dir13:25
mhr3didrocks, at least not in tarball13:25
didrocksmhr3: so until we remove them…13:25
Trevinhotsdgeos: or another clean way would be to set the index inside an entry since it looks like that you want an entry to only have a specific index13:25
didrocks(we should do that for R)13:25
mhr3didrocks, i actually think the build system should pick it up properly13:26
didrocksmhr3: was it tested?13:26
didrocksmhr3: I'm quite afraid by that13:26
mhr3but maybe i'm wrong, i don't see into the packaging build13:26
tsdgeosTrevinho: i can do what you prefer :-)13:26
didrockssil2100: so, if the build system is covering this right now, I may be wrong, it just needs a double check13:26
didrockssil2100: but remember what I told about .vala files trick -> when we have .c files, they are not always regenerated (because of vala ignoring timestamps)13:27
mhr3sil2100, if you have your pkg installed do `XDG_DATA_DIRS=/home python -c "from gi.repository import Unity; print Unity.LauncherFavorites.get_default();"13:27
mhr3sil2100, if it crashes it's bad :)13:28
mhr3if there are two warnings it's good13:29
sil2100mhr3: just get a warning13:29
sil2100mhr3: (process:28044): libunity-WARNING **: unity-launcher.vala:373: Schema "com.canonical.Unity.Launcher" is not installed!13:29
didrockssil2100: ok, in that case, I'll take it13:29
sil2100mhr3: I just tried it on tests too, no crashes13:30
didrockssil2100: I'll just downgrade it to Suggests, but that's a personal nitpick13:30
sil2100didrocks: ok, thanks13:30
=== salem_ is now known as _salem
mhr3didrocks, see how nice we made vala behave with debs now ;)13:33
didrocksmhr3: is that for every projects you did that?13:33
mhr3at least in libunity, still a problem everywhere else :P13:33
didrocksmhr3: so that I don't do sil2100 searching something for nothing :p13:33
mhr3sil2100, keep in mind ^13:34
sil2100.o>13:35
=== _salem is now known as salem_
didrocksmhr3: so lenses have this build system enhancement?13:36
mhr3<mhr3> at least in libunity, still a problem everywhere else :P13:36
didrocksok, everywhere like everywhere :)13:36
didrockssil2100: can you note that down? ^ something good to note for next cycle "converting vala lenses to not ship .c file" ^13:37
mhr3but ultimately it's a matter of getting .c away from tarballs13:37
mhr3patches for that welcome :)13:37
mhr3took me like a week to tune the build system in libunity to do that (and distcheck pass) :P13:38
didrocksmhr3: if you have the commits rev handy, can be useful to have it :)13:38
mhr3didrocks, the most important part of it is 153.3.813:41
didrockssil2100: ^13:44
=== mmrazik is now known as mmrazik|otp
mhr3i have to note that i still like x.y.z revnos better than a hash of every commit :P13:45
=== dpb_ is now known as Guest72485
mptTo whom it may concern: The most common unfixed crash in Q is currently http://launchpad.net/bugs/1062037 in unity-lens-video13:52
ubot5Error: ubuntu bug 1062037 not found13:52
mptubot5, try harder13:52
didrockssil2100: working fine for me, pushing it to quantal! Thanks (just added the suggests).13:52
didrocksmpt: I guess this one was fixed yesterday evening13:53
didrocksI backported an additional commit for it13:53
mptgreat13:53
mhr3i think crashes of binaries should have bigger weight on errors.ubuntu.com13:56
mptmhr3, bigger weight than what?13:56
didrocksmhr3: errors.ubuntu.com is just about crashes :)13:57
mhr3it's just full of python exceptions13:57
mptmhr3, only unhandled exceptions, though, right?13:57
mhr3yes, but that doesn't mean fatal ones13:57
mhr3or does it?13:57
* mpt doesn't know13:58
mhr3still i think a separate view with most common sigsegvs would be useful13:59
mptWe'd like to make it more personally useful in general14:00
=== mmrazik|otp is now known as mmrazik
mptShow you just the kinds of errors you are good at fixing yourself14:00
mptfor example14:00
mptIt already does individual packages, a next step would be package sets e.g. "Ubuntu Desktop" or "Xorg"14:01
mptand classifying by error type e.g. sigsegv would be interesting too14:02
mhr3the problem is that we don't get anything else than segfaults for non-interpreted apps14:03
mhr3freezes could be useful, but that's not that simple to detect14:03
mhr3possible though :)14:04
didrockssil2100: around?14:15
popeydidrocks, he's on a late lunch14:32
=== dandrader is now known as dandrader|lunch
didrockspopey: ok, no hurry anyway, or maybe do you want us to discuss about what's needed for a SRU?14:36
sil2100didrocks: what's up?14:53
didrockssil2100: so, I wanted to discuss with you about SRU/prioritization, do you have a minute?14:53
didrocksif you don't today, that can wait on Monday14:54
popeydidrocks, how about a call with you me mirv and sil2100  first thing on monday morning?14:55
popey(your first thing, not mine)14:55
didrockspopey: sounds good to me14:55
popeyok I'll set it up14:55
didrocksthanks!14:55
sil2100didrocks, popey: ok, so let's do it on Monday then14:56
didrockssil2100: it will be more an opened question on how to organize as I can only showed you what we did in the past and what worked (and rather what didn't really work)14:56
didrockssil2100: so if you can think on some way to organize the SRU until then, please come with suggestions :)14:57
popeysil2100,  you okay to start at 06:00 UTC?14:57
popeyI think he fainted.15:00
didrocks:)15:01
didrocksyou should maybe consider later for him then :)15:01
didrocks(especially on a Monday)15:01
popeyheh15:01
popeydone15:02
* didrocks likes how thunderbird is condering popey's mail as a spam and doesn't want me to click on "accept" the invitation15:03
sil2100didrocks: ACK15:03
sil2100popey: I'm okay with anything15:03
* popey moves it to 03:00 UTC15:04
sil2100HEY15:04
sil2100Maybe not *anything*15:04
sil2100;)15:04
didrocksahah15:04
didrockssil2100: too late!15:04
=== dandrader|lunch is now known as dandrader
tsdgeosTrevinho: uploadded the unittest to https://code.launchpad.net/~aacid/unity/do_not_reuse_menus_on_order_change/+merge/12824315:58
tsdgeoshope that is what you suggested :D15:58
tsdgeosaaaaaaand weekend time15:58
tsdgeosenojy all15:58
=== dandrader is now known as dandrader|afk
=== dandrader|afk is now known as dandrader
=== yofel_ is now known as yofel
=== Arm is now known as Guest78888
=== salem_ is now known as _salem

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