User1: I wrote a simple .desktop file that I've put in ~/.local/share/applications but unity's dash does not find it
User1: why?
User2: User1 Anything special about the .desktop file? The various Steam .desktops in ~/.local/share/applications work for me.
User1: RAOF, nothing special, I just want to launch my python script from unity's launchers
User2: You could always try logging out and back in again. Presumably unity watches that path, but maybe reloading the scopes will fix it
User2: ?
User1: RAOF, ah I understand
User1: it does not accept if the exec binary is in ~/.local/bin
User2: Is ~/.local/bin in your path?
User1: RAOF, I set the path in my .bashrc for that
User1: it's probably not in unity's path
User1: ok, setting the full path to the python script and it works
User1: would be nice to have it relative though
User3: did you try to use "bar" or ".local/bin/bar" or "~/.local/bin/bar" or "/home/foo/.local/bin/bar" ?
User1: sarnold, ah ".local/bin/bar" works
User1: cool :)
User1: ok, next step: add an action in the context menu
User4: good morning desktopers!
User5: morning!
User4: Laney, good morning, how are you?
User5: User4 alright thanks, although I think my arm got bruised when I gave blood last night
User5: so that hurts a bit
User5: you?
User4: I'm good thanks!
User2: User6 Good morning!
User6: hey RAOF, how are you?
User2: User6 You've got a merge request and a bug for umockdev awaiting you :)
User6: User2 thanks for your umockdev fixes
User6: User2 no, I don't :)
User2: :)
User6: User2 it's all in 0.4.7, I uploaded that to Debian and it'll autosync in a few hours
User2: Bitchn.
User2: So, by the time I'm ready to propose this Mir branch, the valgrind tests should not fail in umockdev :)
User5: robru: Was using sbuild
User4: didrocks, btw on fully updated trusty I can still tap "alt" in g-c-c
User4: works in a guest session as well
User7: User4 hum, let me try today
User4: not sure what's the difference between our systems
User7: still not working and update today
User7: yeah, as Laney confirms, I'm wondering
User7: I'll try a guest later
User4: needs debugging by somebody having the issue
User4: hint hint hint ;-)
User7: heh, I'll have a look
User4: mhr3, hey
User4: mhr3, just as a FYI, the gvfsd-http leaks/non closing fds issues should be fixed in trusty, I would appreciate if you could verify/confirm during the cycle
User4: larsu, hey
User4: larsu, how is GTK 3.10 going?
User4: larsu, still working on it/on what specific issues?
User8: User4 yes. I have a hard time figuring the update-manager list view issue out
User4: larsu, that doesn't seem a blocker ... maybe we could just upstream that one?
User4: larsu, and gently ping mclasen/Company saying it's a blocker to get the update in Ubuntu
User4: well, not a blocker for the ppa I mean
User8: maybe it's this one: https://bugzilla.gnome.org/show_bug.cgi?id=712220
User9: Gnome bug 712220 in GtkScrolledWindow "GtkScrolledWindow has incorrect minimal size" [Normal,Unconfirmed]
User4: larsu, did you notice that it takes the correct space if you expend the details t the bottom?
User8: ya
User8: which is the weirdest part :-/
User8: User4 okay, let me push that back. Then I'd only consider the scrollbar theme thing to be a blocker
User4: larsu, right, I was going to say the same
User8: I assume this is something that needs to be fixed in the theme though
User8: I'll investigate :)
User4: larsu, I would assume it's something to be fix in GTK/o-s code
User4: but let's see
User8: o-s ?
User4: overlay-scrollbars
User8: ah :)
User10: seb128, cool, got a link to a bgo bug?
User4: mhr3, https://bugzilla.gnome.org/show_bug.cgi?id=706225#c7
User9: Gnome bug 706225 in general "gvfsd-http crashed with signal 5 in g_cancellable_make_pollfd()" [Major,Resolved: duplicate]
User4: mhr3, https://bugzilla.gnome.org/show_bug.cgi?id=695652#c44
User9: Gnome bug 695652 in HTTP Transport "Indefinite hang in run_sync_state_machine() on close()" [Normal,Resolved: fixed]
User4: mhr3, I've commented on that second bug to see if we can get some of the fixes cherrypicked in the serie used in saucy
User4: so we can SRU
User4: mhr3, the commit Dan backported to fc19 seems to be https://git.gnome.org/browse/libsoup/commit/?id=96da2df64c9dd8cc52e97ce73e54615d6b520664
User4: larsu, do you need help on GTK for anything? Like do you want me to spend some time to see if I can get a small testcase example of the update-manager issue?
User8: User4 I've already found one: the test case in overlay-scrollbar
User8: convergence!
User4: lol
User4: Laney, so new webkitgtk isn't happy on arm64 :/
User4: ../Source/WTF/wtf/dtoa/utils.h:64:2: error: #error Target architecture was not detected as supported by Double-Conversion.
User4: Laney, it also made powerpc unhappy
User5: User4 yeah, checking it
User5: I see the arm64 fix I think
User5: ppc test building on partch
User5: probably a missing #if somewhere
User4: mhr3, don't run away like that :p
User4: mhr3, did you get my reply?
User10: seb128, saw the commit link as last thing
User10: my internet died :/
User4: mhr3, ok, so you got it all, 2 bugs and 1 commit
User10: using phone data now
User10: seb128, when is that going to work with ubuntu phones btw? :)
User4: what? sharing datas?
User10: yep, usb tethering
User10: or wifi
User10: or bluetooth :)
User4: mhr3, don't seem to be on the els prd list, so maybe not this cycle :/
User10: strangely sharing my phone's wifi over usb is my most used feature on android
User10: mostly caused bcm wifi sucks so bad
User10: seb128, anyway, i'm not sure whether it's the same issue, the bug is mentioning something that got broken and is now fixed
User10: afaik the fd leaking has been there since at least 12.10
User4: mhr3, well, Ross it's easy to reproduce the issue, so should be easy to test on trusty?
User10: seb128, well, it should be easy to see on errors.ubuntu.com :)
User10: last time i checked that bug was high on the list
User4: mhr3, yeah, it's first or second on gvfs issues for saucy, which is what made me comment upstream/ping about it yesterday
User4: mhr3, I'm trying to work a bit on the e.u.c list every day
User10: seb128, great, it's an awesome source of data
User10: makes bug prioritization a walk in park :)
User4: hum
User4: I don't trust lightdm on trusty anymore, I swear it's doing weird stuff on user switches sometimes
User4: I just had my session closed when logging out of a guest session
User4: mhr3, is that normal that dash filters change (e.g new category got added) during the session?
User10: seb128, yes
User4: mhr3, that's weird :p
User10: seb128, well, categories should be stable, sources will get added
User4: mhr3, I've "news" added to the categories
User4: e.g if I do
User4: - log into a guest session
User4: - open the dash
User4: - look at filters
User4: it starts with "aide" (help)
User10: oh right
User4: - type something in the search entry, clean it
User4: -> "actualitÃ©" (news) is added to the categories
User10: if a category has 0 sources it's not displayed, and since later a source is added, it becomes visible
User4: ok, I see
User4: that's a bit weird since it changes the layouting of the filters
User4: thanks
 * seb128 also noticed that turning the online data makes the dash snappy to return results again and is pondering to just do that
User10: seb128, but if it was there all the time it'd be even worse (returning no results ever)
User4: shrug
User4: mhr3, so I set 'remote-content-search' to 'none' ... I did a query "pdf", which listed me some pdf I opened recently (great)
User4: mhr3, now I open the dash again, type "pdf" and it lists no file, the "files" filter is off (I didn't change), dispite having 'files.scopes' in the 'always-search' gsettings
User10: sounds like a bug
User4: mhr3, do you think it's the same issue than the one from yesterday?
User10: might very well be
User4: :/
User10: seb128, can you get bustle log for that?
User10: and attach it to the bug you opened
User4: mhr3, oh, it works if I see the "remote-content-search' back to 'all'
User4: mhr3, can you reproduce if you change the key to none?
 * mhr3 tries
User10: seb128, nope
User4: :-(
User10: anyway, the bustle log should help
User10: just try to describe very precisely what you did and what you saw
User10: there'll be a mismatch somewhere
User10: seb128, perhaps even record a video while capturing the log
User4: mhr3, is that going to "leak" all my dash content? not sure I want to put all my filenames online :p
User10: seb128, mark it private then
User4: larsu, I keep (ab)using your merge request to discuss GTK 3.10 issues, let me know if you prefer moving to a bug? (e.g should we rather use https://bugs.launchpad.net/ubuntu/+source/gtk+3.0/+bug/1171587)
User9: Launchpad bug 1171587 in gtk+3.0 (Ubuntu) "update to GTK 3.9 and the issues to resolve for it" [Wishlist,Confirmed]
User4: in fact that would make sense, let me comment on that bug ;-)
User8: User4  I don't mind either way
 * larsu is bisecting the o-s issue
User8: I hate o-s
 * seb128 hugs larsu
User5: talk to desrt about that
User4: you can always try to ping that italian guy who wrote it :p
User4: but I bet the reply is going to be "it was working, it's a bug in GTK"
User8: I'm sure he'll be happy about that :)
User4: to his defence that's what happened with GTK 3.8 and he was right :p
User8: seems like the same here
User4: the bug that make o-s buggy by then was an annoying one in GTK which impacted on other apps as well
User8: 7 steps left. If only gtk built faster :)
User6: -j8 FTW!
User6: (and build in tmpfs)
User4: larsu, just doing git bisect&make ought to not be that slow
User8: User4 slow enough to annoy me :)
 * larsu is not a very patient person
User4: bregma, hey, could you get somebody assigned to https://bugs.launchpad.net/ubuntu/+source/unity/+bug/1047517? that's the most reported unity issue on e.u.c/saucy
User9: Launchpad bug 1047517 in unity (Ubuntu) "compiz crashed with SIGSEGV in g_mount_spec_to_dbus_with_path() ... from unitybd6c:f4c4:4f6:c742:59d8:35db:68f6:4d5fIconLoaderbd6c:f4c4:4f6:c742:59d8:35db:68f6:4d5fImplbd6c:f4c4:4f6:c742:59d8:35db:68f6:4d5fIconLoaderTaskbd6c:f4c4:4f6:c742:59d8:35db:68f6:4d5fLoaderJobFunc()" [High,Confirmed]
User11: seb128, we'll discuss it today
User4: bregma, thanks
User4: mhr3, I've added bustle and video on https://bugs.launchpad.net/ubuntu/+source/unity-scope-home/+bug/1223933 ... let me know if you need more info or if you prefer a new bug since that's not strictly the same issue
User9: Launchpad bug 1223933 in libunity "sometimes the dash home list "no result matching your query" string" [Undecided,Triaged]
User10: seb128, thx
User10: seb128, hm, it doesn't even try to search files
User10: pstolowski, the log is odd ^^
User10: pstolowski, home scope sends query to apps scope, and then the same query to the scopes scope (even though that's supposed to be apps child)
User12: mhr3, oh, if it's the case then it's indeed odd, looking
User10: oh wait
User10: nevermind
User10: it's the apps master querying its children
User10: pstolowski, nonetheless, later it doesn't send the query to files (which is in always-search according to seb)
User10: later == when there's an actual search string
User10: seb128, btw is the files scope running?
User10: pgrep unity-files-daemon?
User4: mhr3, I guess so, the file lens is working
User10: hm, interesting
User4: yeah, it's running
User12: seb128, but judging from your description, gsettings get com.canonical.Unity.Lenses home-lens-default-view doesn't show files.scope now?
User4: pstolowski,
User4: ['help.scope', 'help-askubuntu.scope', 'help-texdoc.scope', 'help-yelp.scope', 'news.scope', 'news-googlenews.scope', 'news-yahoostock.scope', 'applications.scope', 'applications-applications.scope', 'applications-scopes.scope']
User4: which matches the filters showed active on the ui
User4: is it supposed to use the "always-search" ones as well? (that has apps/music/video/files)
User12: seb128, right, and you never de-selected files in previous searches?
User4: no
User12: ah wait, it's not surfacing
User12: so right, always-search should be used
User10: seb128, just to make sure, can you paste the value of always-search as well?
User4: mhr3, http://paste.ubuntu.com/6415785/
User10: great, thx
User4: yw
User12: seb128, and is this now reproducible on every search?
User4: pstolowski, yes, it's stucked in that buggy situation
User4: I've not tried to manually click on filters or restart processes though
User4: in case you guys need more debug infos
User4: but it's not going away by using the dash or typing stuff in the entry
User12: seb128, mhr3 ok, I suspect filter state handler in home scope got confused, grr
User12: seb128, i'll look at this, probably next week
User4: pstolowski, ok, let me know if you need more debug infos/want me to build a special versions with extra prints for example
User13: User4 So I checked the libs that LO brings its own copies, for the debian sync and MIR dance. libodfgen needs 0.0.3 which is not in debian exp. yet, libmwaw is uptodate in universe/just needs MIR, libmdds is outdated in universe, needs a sync of 0.9.1 from in debian experimental, libcmis needs 0.4.1 which is not in experimental yet. libenotyek and libfreehand are new with 4.2 -> lets postpone those.
User4: I wonder if my config is special in any regards
User4: Sweetshark, ok, should we do the libmdds sync?
User13: User4 yep, would be one thing of the list.
User13: so on big bertha doing a release build of LibreOffice takes 30min. Building a package takes 106min. as xz compression of the result takes ages :/
User12: seb128, what would definitely help is if you run home scope with HOME_SCOPE_VERBOSE=1 env var set; and capture all the output from start till when the problem shows up (there will be lots of output, so just 2>&1 to a logfile)
User4: pstolowski, ok, I can do that
User12: seb128, so just kill home-scope and re-run it from a terminal
User4: pstolowski|lunch, User10 it's trivial to reproduce here
User10: seb128, good, you'll be asked to send your laptop to poland then ;)
User4: mhr3, pstolowski|lunch: I wonder if that's a locale stuff, I restarted the home scope with the debug env, I typed "pdf" and I had files, then I did "esc" ... the filters changes with "actualitÃ©" ("news" in french) added as the first category and files&folders got unselected at the same time
User4: mhr3, pstolowski|lunch: http://ubuntuone.com/3tC2suxeWymsYaBbFkjGhf is the log
User4: see l1337
User4: is that normal that this query is only apps scope?
User4: l1321: 13:43:07.936785Z unity-scope-home: search-util.vala:180: Setting sources filter files-gdrive.scope: false
User4: l1318 I mean
User4: the files-local.scope is set to false
User13: s/of/off/
User12: seb128, no, it got confused after esc. thanks for the log, I think I have all the needed info
User4: pstolowski, great, yw
User14: good morning
User14: let's drop overlay-scrollbars
User14: larsu tells me i get free steak if this happens... so... soon, please?
User8: User14 only of you had verifiable influence on that decision ;)
User14: User8 butterfly effect, dude
User8: heh
User8: not really verifiable
User15: given you dont have wings, how do you do that ?
User15: fart and it will change eventually ?
User4: desrt, good morning
User14: User4 hey.  i'd be happy if you could now say "we will get rid of them, but only because desrt asked so nicely"
User4: desrt, let's not start trolling, new GTK is making me less happy by the day, I could start ranting soon :p
 * seb128 just opened https://bugzilla.gnome.org/show_bug.cgi?id=712302
User9: Gnome bug 712302 in GtkFileChooser "GtkPlacesSideBar showing "Desktop" or not shouldn't be a by-application setting" [Normal,Unconfirmed]
User4: wth GNOME
User14: oh.  this is fun :)
User8: uh oh. I guess I should stop ranting then...
User14: User4 if i fix this bug, will you let larsu drop overlay-scrollbars? :)
User4: they made the default to be "don't show a desktop, because GNOME3 doesn't have one"
User4: and the solution is to patch every app
User15: just use QtFilechooser ;)
User14: User4 i guess you want XSETTINGS stuff here
User4: desrt, no dropping of anything
User4: desrt, yeah, but isn't xsettings being killed because it doesn't work in wayland?
User8: User14 why not a gsetting? The file chooser already has a schema (and federico said so in that commit)
User4: larsu, because schemas are not by desktop
User4: larsu, e.g a xfce user wants that item, an Unity user wants it, a GNOME3 doesn't
User8: argh, right
User14: well.... in fairness, it sort of makes sense to bind this to the "icons on desktop" setting that already exists
User14: if the user turns off viewing their desktop then that should be the indication not to show it in the chooser
User14: regardless of environment
User8: good idea, but this is a nautilus setting, no?
User14: no
User4: I was going to say
User14: i think it's in org.gnome.desktop.interface
User14: org.gnome.desktop.background show-desktop-icons false
 * desrt was close
User8: is that installed on an xfce desktop?
User4: no
User4: neither on win32
User4: nor macOS
 * desrt already raised this point in the bug
User8: we need an equivalent to shell-shows-menubar
User14: adding XSETTINGS back to gtk seems to be popular these days....
User16: dobey, my wife is angry with you!  :-p
User16: she couldn't figure out how to buy music anymore... she's on precise
User4: larsu, kenvandine, charles, tedg, Laney, let's skip the settings meeting, I don't think anyone worked on this week (I didn't see any activity and CI is down), seems everybody is busy catching up with other topics/start of cycle work
User4: cyphermox, attente, User7 ^
User17: seb128, ack
User18: ok
User19: sure
User18: User4 btw, cheryl69@example.com...
User4: cyphermox, hey! oh, thanks ;-)
User16: +1
User18: User4 it's that bug that failed to retrace everywhere in all cases
User18: something broke maybe in libnl
User14: User4 be happy.  this problem will be solved by end of day.
User4: desrt, thanks a lot for dealing with it!
User14: User4 so... about those overlay-scrollbars.....
User14: ;)
User4: desrt, *lalalalala*
User4: ;-)
User14: geesh.  you do a nice thing, and this is what happens
User14: User4 *the keg* is on the line here
User14: User4 in fairness, probably larsu would treat you as well...
 * larsu confirms
User4: you don't want me to get fired, do you? ;-)
User14: 'accepting bribes'
User20: I think getting rid of scrollbars entirely is more sane than getting rid of overlay-scrollbars :P
User8: User20 and how do you show how far along a document you are?
User4: larsu, User20 let's stop trolling, friday is tomorrow :p
User20: User8 oh, you leave the indication there, you just don't pop anything up over it :P
User4: larsu, btw I just tested on raring, seems to be that the thumb behaves/looks the same way
User4: larsu, the orange portion was not animated, it was just always there to show the position
 * mdeslaur suspends further trolling until tomorrow
User8: User20 ah, that would have been part of the "scrollbar" term for me. I totally agree then
User8: User4 cool. Thank you. I guess I misremembered
User8: User4 this is what I meant: http://vimeo.com/30210146
User4: larsu, is that the orange/grey line and the way it's displayed/fading away?
User8: User4 yes, the gray one is missing for me when using 3.8
User4: larsu, work in a saucy vm (sorry got sidetracked while the vm was loading)
User4: larsu, let me downgrade gtk on trusty and see
User4: larsu, I'm testing with "nautilus /usr/share", works fine on trusty with the trusty GTK (you just need to aim close from the border and slowly)
User8: User4 do you get the gray bar as well?
User4: larsu, yes
User8: meh :/
User8: User4 thanks for testing that,
User4: larsu, http://ubuntuone.com/5VrE9JH4FuK1VIMIZrSvxG
User4: larsu, that's trusty with the trusty gtk version
User8: User4 weirdly enough, I don't get that with upstream gtk 3.8. Might be one of our patches.
User4: larsu, I don't really see which patch in our stack could make a difference there :/
User8: me neither. I'll find out after I get the trough visible at all
User4: larsu, what application are you testing with?
User8: User4 the tester inside of overlay-scrollbars source tree
User8: tests/test-os
User4: larsu, that test app is weird/buggy
User8: how so?
User4: larsu, http://ubuntuone.com/4fHJW2VMDIXLYhQDoGxlMN
User4: is that how it looks like for you?
User8: ya, it seems to be the bug that we also see in update-manager
User8: I patched the tester :)
User4: larsu, that screenshot is using GTK 3.8...
User4: how did you patch it ?
User8: then it's definitely not the same bug :D
User8: http://paste.debian.net/65856/
User8: I made it so that there's only one listview
User8: good enough for testing
User4: larsu, ok, I do get the grey bar/animation on that as well
User4: just wanted to rule out the test to be buggy
User4: I'm going to let you debug then, let me know if I can help for anything else
User8: yep. Thanks a lot for testing!
User4: yw!
User21: User4 I did some analysis on bug 1221593 (my last comment), and can only pick up it again next week cuz I'm at Changsha for the meeting from today to the weekend.
User9: Launchpad bug 1221593 in ibus (Ubuntu) "ibus-ui-gtk3 crashed with SIGABRT in _g_log_abort()" [High,Triaged] https://launchpad.net/bugs/1221593
User4: happyaron, hey, thanks ... is the issue fixed in 1.5.4?
User6: mterry: so the icon theme uninstallability is fixed, now https://jenkins.qa.ubuntu.com/view/Trusty/view/AutoPkgTest/job/trusty-adt-deja-dup/14/? failed with a test failure; would you mind having a look?
User4: happyaron, I wonder if the error is worth asserting on, what would happen if we only g_warning there? would it pick the config once the daemon writes it?
User21: User4 I haven't tested the latest patchset ofr 1.5.4, anyway it's not fully a part of the released version of ibus, but need some patch that fujiwara apply to F19.
User21: User4 not tested yet, but that's worth to try.
User4: happyaron, why is fujiwara not upstream those changes? do they still have them in f20?
User21: User4 fujiwara cannot commit those changes directly, need code review from phuang (original author of ibus). So all the distros are carrying fujiwara's patches for certain features/fixes.
User21: it's more obvious since gnome decides to do the integration.
User4: happyaron, is that because phuang is not active for reviews?
User4: k
User4: happyaron, thanks for the update
User21: User4 he's active, but not that interested on those kindia integrations.
User21: he cares more about the framework
User4: ok
User4: happyaron, can you check, next week, what happens if we just return in that function instead of asserting?
User21: User4 sure
User4: happyaron, that might be good enough as a solution
User4: happyaron, thanks
User21: with pleasure
 * seb128 shakes the fist to lightdm in trusty
User14: User4 got patches up for the GtkPlacesSidebar issue
User14: testing appreciated
User4: desrt, excellent, thanks
User14: (they seem to work in my jhbuild)
User4: desrt, I was about to go for some exercice but I do that next once I'm back
User14: cool
User14: looks like they'll go upstream pretty quickly so i don't know if there is much value in patching them in our packages except for local testing
User6: User14 hey, how are you?
User14: good.
 * desrt is doing job #1 today
User6: User14 I pushed an updated patch to -shim, FYI
User14: oh.  great.
User6: User14 oh, what's that?
 * desrt checked last night, didn't see it
User14: User6 making seb happy
User6: "watch latest BBT episode"?
User6: ooh
User6: User14 no, did it this moring
User6: morning, too
User6: some 11 hours ago
User6: User14 if you are happy with it, could we just do a version 5?
 * desrt would have preferred a chance to review it
User14: although i suppose that's inconsistent with my previous hopes of passing off maintainership to you :)
User6: User14 ah, I thought you were content with our discussion how to do it yesterday
User6: User14 but it's not yet released or uploaded or anything
User6: User14 it's git, so we can change it :)
User14: User6 i'm pretty strong into the "no non-trivial commits without code review" mindset these days
User6: User14 ok, noted for the future
User6: User14 (good thing!)
User14: looks more or less fine, though... i'd only have minor tweaks to it
User6: I should try that with my projects too, I just find it hard to find someone who wants to bother with reading apport or umockdev patches
User14: printf("%u", getpid()) is possibly undefined, for example
User14: if sizeof(pid_t) != sizeof(int)
User14: but i guess in our case they're the same
User6: User14 undefined?
User6: ah
User14: User6 if you had a platform where pid_t was a long, for example
User6: so that would need an (unsigned) cast
User14: then passing the result of getpid() directly as the argument for "%u" would be an error
User6: as there's no printf macro for pid_t
User14: yes.  you'd normally want a cast in this case.
User14: i'd also have used g_strdup_printf() instead of mucking around with the static buffer
User14: and i'd have written 'GError *error' instead of 'GError* error'
User14: as i said -- all minor complaints :)
User14: probably not worth changing
User6: nah, I'll do it anyway
User14: i assume you tested it and it works?
User6: yes, of course
User14: awesome :)
User6: User14 ah, I avoided g_strdup_printf(), mostly out of a habit
User14: i'll wait for your tweaks, then i'll do a release (and tarball)
User14: User6 if this was some tight inner loop, i'd agree
User6: if I can get away without dynamic memory allocation, I do it usually; but I'm happy to change it
User14: but one extra malloc() _per lifetime of the entire system_.... probably not an issue :)
User6: no, but it doesn't actually make the code any easier, to the contrary
User14: User6 it's not about making the code easier
User6: it introduces at least three lines more
User14: similar to your dynamic memory aversion i have an aversion to formatting strings into fixed-sized buffers
User6: User14 heh, fair enough
User14: i'm pretty sure you got it right in this case, though
User14: but that's the nice thing with g_strdup_printf -- you don't have to convince yourself
User14: one more nit: your critical is probably actually a warning
User14: critical == programmer errors
User14: warning == things that should not have happened, but are not the fault of the programmer of this process
User6: User14 http://paste.ubuntu.com/6416828/ (built-tested, but not yet runtime-tested; doing now)
User14: that said, the critical may be nice from the standpoint that we will soon be using apport to report criticals
User14: but ya... probably better as a warning to be more 'proper'
User6: User14 thanks for the warning/critical heads-up
User6: User14 err, sorry, wrong diff
User14: User6 seems you could have avoided the boolean if you wanted to
User14: but looks good
User6: User14 http://paste.ubuntu.com/6416837/
User14: (even if it's the wrong diff) :)
User6: User14 yeah, I committed the style changes first, then --amend'ed the g_strdup_printf change
User6: that's the complete one now
User14: looks good to me
User6: User14 success> I try to free it ASAP; at some point someone might have an exit path in the if() condition
User14: User6 fair enough
User14: you know about this gcc cleanup attribute?
User6: User14 I do
User14: we should use it more :)
User6: yeah, it's magic!
User14: in cases like this where the software will only run on ubuntu (or debian some day maybe) we could totally rely on it
User6: User14 systemd upstream uses it a lot, I guess it's not really a distro thing but more like a "gcc" thing?
User14: ya
User14: my point is that ubuntu/debian are built using gcc...
User14: ie: we don't have to worry about trying to build with msvc or so
User6: hehe, yes
User14: or suncc or whatever
User6: User14 ok, runtime-test thumbs up; good to push?
User14: yes.  please do.
User6: there
User14: k.  lemme do the 'release' :)
User14: okay.  done.
User14: thanks!
User6: User14 mind putting the tarball to https://people.gnome.org/~desrt/ so that we have an "official" orig?
User14: User6 already did
User6: User14 "once and for all" -> famous last words! *cough*
User14: :)
User6: User14 ah, cheers; reload helped
User14: i think i also had "should" in there ;)
 * pitti hugs desrt
 * pitti goes to package/upload/update SRU
User6: User2 FYI: [ubuntu/trusty] umockdev 0.4.7-1 (Accepted)
 * Laney gives up staring at webkit's code
User4: Laney, still looking at webkit? did that eat your day?
User22: I just confirmed an odd bug. The screen will not lock while you have a notification menu open (like session or battery)
User22: https://bugs.launchpad.net/ubuntu/+source/gnome-screensaver/+bug/941618
User9: Launchpad bug 941618 in gnome-screensaver (Ubuntu) "Screen not locking when hovering mouse over notification applets" [Medium,Confirmed]
User22: gnome-screensaver cannot get the focus
User3: User22 take a look at bug 49579  -- it's probably a duplicate
User9: Launchpad bug 49579 in gnome-screensaver (Ubuntu) "screen doesn't lock when some menu is open" [Low,Triaged] https://launchpad.net/bugs/49579
User3: User22 this is something we're expecting Mir to fix for us; fixing it in X11 seems neigh-on impossible.
User22: User3 looking, I just grabbed a log
User22: a 5 digit bug # even
User22: User3 yes, a dupe, thanks
User23: happyaron, ping
User4: bschaefer, it's like 3am for him, I doubt he's online
User23: seb128, thanks, ill have to try to catch him tomorrow morning
User4: mfisch, that's not weird, welcome to Xorg ... surprising you just discover that today ;-)
User22: User4 only discovered on our hacked up GSS that lets you send lock signals via dbus
User4: bschaefer, he's away for a Kylin meeting I think, back next week, you might let some ping context in case he looks at IRC in between (or email should work)
User22: User4 but "Mir fixes" works for me
User4: mfisch, yeah, Mir should fix it, I'm sure the security team is going to make sure of that ;-)
User23: seb128, I see thanks again for the info :). cool ill just send him an email!
User4: bschaefer, just curious, what's the topic? im-config stuff again?
User23: seb128, no its actually more of a question about preedit windows, more about I want to ask someone who uses IMs
User23: like, how important that box is, as im fixing up XIM in nux
User4: bschaefer, "preedit"?
User23: when you typing in any IM you get preedit text, thats showing you what has been typing so far but is not real text yet
User4: oh
User4: I'm sure they want that
User23: and each IM has a preedit box that shows you suggestions of the next char you want
User4: isn't that working in the dash?
User23: seb128, its working, but im running into problems getting it in a nice position
User23: as with XIM, it only uses the X windows position
User23: so for the dash it wont be below the text entry, it'll be at the bottom left of the Dash
User4: bschaefer, you are trying with ibus?
User23: seb128, as the goal is remove all the Ibus code from nux, and depend only on XIM
User23: seb128, XIM support all IMEs
User4: yeah, that would be nice
User4: Kylin uses fcitx
User4: that's a topic for vUDS next week
User23: yes very much, the one problem is the preeidt window wont be super nice in ibus
User4: ibus/fcitx for Ubuntu
User4: is it better in fcitx?
User23: seb128, yup it'll all work fine, XIM is an abstraction that all IMes implement
User23: X Input Method
User23: all the IMes have the XIM client code, so all of them work
 * bschaefer gets  a list
User4: right, sorry for the question, I'm just trying to figure out if that's ibus specific/if fcitx does better
User4: bschaefer, don't bother with a list, we care about ibus and fcitx ... and we need to pick one
User23: seb128, right, well the only difference between switching from IBus directly to using XIM
User23: is this preedit window box isn't in the same place
User23: but all will work fine
User4: ok
User4: I'm sure we can address that specific issue in some way
User4: in any case Kylin is the main user
User23: right, i've tried work arounds, but XIM is a bit old and kind of late getting into X to begin with...
User23: yup, and fcitx will work much better in Nux
User23: cause i've actually implemented the rendering of preedit text (this isn't the case atm)
User4: knowing that they don't use ibus, we need to switch away from requiring it
User23: yup!
User23: which as soon as I can get this branch landed we can remove anything ibus related in nux
User23: and unity
User4: \o/
 * bschaefer and remove those evil ibus AP tests...
User4: so you are saying fcitx works better than ibus UI wise in Unity?
User4: btw it would be nice if you can join that session next week
User23: yeah i can join in, umm right now ibus works better
 * seb128 has no clue about input methods framework (as most of the team), yet we have to pick one :/
User23: because we implemented ibus directly in nux
User23: yup
User4: right
User23: im not really sure, both work very well overall (in my eyes at lease)
User4: is that statement still true once you land the xim support?
User23: when we move to XIM, they will all look the same
User4: great
User23: the only difference will be the preedit window the IMe renderes
User4: so at least Unity is not a blocker in the decision
User23: renderers, which they are in charge of
User23: what would be a blocker?
User4: the UI you are talking about is e.g http://www.emacswiki.org/pics/static/IBusModeScreenShot.png right?
User23: seb128, yup that little box with the suggestions
User4: blocker would be: users of that framework can't use the dash
User23: oo, well they can :), so yeah thats good
User4: great
User23: seb128, the only thing, with the dash the preedit box in fullscreen will be outside of the screen :(
User4: the real solution for that is to add the prediction UI to unity...
User4: (sure you like that ;-)
User23: which is one question i wanted to ask, how important that suggestion is box really is to normal users
User23: seb128, well the problem, is we can't control its location
User23: only the IMe does
User23: (for some reason)
User4: I'm not using those input method but I guess it's a basic feature so important
User23: and its location is based on the bottom left corner of the X window
User23: yes very much so
User23: ill have to come up with a work around...
User23: so far a few have failed
User4: bschaefer, oh, I'm sure you can
User23: seb128, atm, its a problem in any gtk app as well, if you use XIM
User4: bschaefer, http://hedayatvk.files.wordpress.com/2013/01/d8b9daa9d8b3e2808cd8b5d981d8add987-d8a7d8b2-2013-01-16-021756.png
User4: bschaefer, that's what gnome-shell is doing
User23: if the window is maxed, the preedit box is placed outside the screen
User23: that looks nice
User4: bschaefer, if they can, it's technically doable
User23: seb128, it is, the problem
User23: nux doesn't have a nice IME interface
User23: the only one we really have access to is XIM
User4: right
User23: so, the real soultion would be to implement an IME interface for nux, and write client code in each IME
User4: that might be a "next generation desktop" topic
User4: e.g unity8
User23: well unity8 IIRC uses malit?
User23: i can't spell it :)
User23: which has all that stuff put in with QT
User4: don't ask me
User23: sooo hopefully when we move over to unity8 we will have no more problems with IMs
User4: I still don't understand how much an osk integrates/replaces IM framework
User4: we still need to support physical keyboard
User4: you can't just dynamically change the physical layout on those ;-)
User23: o right...that'll be interesting, luckly QT has an IM interface, and I know (think?) fcitx supports it at lease
User4: right
User4: one step at the time
User23: yup, having that already solidified is a huge step
User23: from there you just need to make sure each IME support that toolkit
User4: it would be good for the lts to put the popover dialog in the dash rather than at the corner
User4: not sure how easily that's to hack though
User23: hmm well with XIM the pop up will be at the bottom left corer atm
User23: im guessing the only workaround will be to hacky, and possible regression causers
User23: ie. having to re-implement the text entry to be an Xwindow it self
User23: but yeah, one step at a time, i should get it merged first :)
User4: right, let's get it working first, then we can do UI tweaks
User23: sounds good!
User4: but I guess it's not going to be trivial to get some popup on top of the dash
User4: e.g we would maybe need to make that UI be display by unity itself
User4: which starts being more work than we want for the LTS
User4: so yeah, maybe just having it at the corner is good enoug
User4: h
User23: right, im just worried about regression potential for the LTS
User4: check with happyaron ;-)
User23: i would much rather play it safe :)
User23: yup, ill send him an email!
User4: (which is what you were going to do, sorry for distracting you)
User23: no worries :)
User4: but yeah, play safe sounds good
User4: we are struggling on what to do with the indicators/keybindings as well
User23: yeah i was reading that topic...its a complicated problem...
User4: indeed
User23: is there also a session for this in vUDS?
User4: not yet, I'm pondering putting one
User4: we still have free slots
User4: there is one from the Kylin guys about ibus/fcitx
User23: wouldn't be a bad idea, as thats something to talk about worst case it'll be short
User4: I'm also unsure how the indicator/keybinding stuff can be resolved over a session
User4: right
User4: I was thinking about putting one
User4: I'm probably going to do that (setting it after the ibus/fctix one so we can discuss based on the output of that one)
User23: yeah, ill make that one, unless i over sleep haha (joking)
User4: ;-)
User23: that would be good, if the slots align nicely :)
User4: yeah, I've scheduling powers so I can make that happen ;-)
User23: :)
User14: User4 any other gtk nags?
User4: desrt, nags, or nags for you?
User14: :)
User14: anything that i could do to make your life easier?
User4: desrt, https://bugs.launchpad.net/ubuntu/+source/gtk+3.0/+bug/1171587 comment #3 (#4 as well)
User9: Launchpad bug 1171587 in gtk+3.0 (Ubuntu) "update to GTK 3.9 and the issues to resolve for it" [Wishlist,Confirmed]
User14: User4 fwiw, dunno what the deal with u-s-d is, but you're probably going going to need to manually pick that patch at least
User4: desrt, I think larsu is on the other ones (out of update-manager one, but that's not a blocker)
User14: although the failure mode if you don't pick the patch will be for it to fail in the direction that we find favourable, ie: desktop will be shown
User14: but we should probably do it anyway to support people who manually disable the desktop
User4: desrt, yeah, I'm going to pick that patch this week. robert_ancell sent me an email that he has an u-s-d test version ready, but he's on vac for a week now
User14: User4 parallel-installable with g-s-d i hope
User4: yeah
User14: neat
User4: we still have arguments about the details though
User4: he wants to rename/migrate the gsettings keys
 * desrt finds a new wishlist bug about the "...when ~/Desktop is not empty" thing
User4: I argue that there is enough compat that it's not needed
User4: not speaking about external users/internet documentation
User14: that's very tricky
User14: gsettings-desktop-schemas upstream has been less.... stable than i'd have hoped
User14: in general, i think ubuntu-desktop-schemas would be an extremely good idea
User14: for various extras we have
User14: instead of all of the hacking we do to check "is component X installed?  use it's gsettings, if so..."
User4: right, but I think in practice it's going to be less work this cycle to enforce stability there/add back keys for compat reasons if they drop them than to fork/migrate the keys
User14: now that we have a proper g-c-c fork, these sorts of "add UI elements if..." checks are pointless
User14: User4 as long as the changes that we patch back are purely additive
User14: (ie: we only do it when upstream is removing settings outright)
User14: because otherwise we may end up causing compatibility issues for gnomebuntu
User4: right
User4: I think that in practice it should be an easy this cycle
User14: from that standpoint i understand why robert wants what he wants
User4: which makes less migration work for the LTS
User14: but i agree that the changes should not be too much, and they're usually strictly additive
User4: we can revisit next cycle if needed
User4: desrt, btw, we have https://launchpad.net/gsettings-ubuntu-touch-schemas
 * desrt is honestly looking forward to being able to run ubuntu again :)
User4: which is what you suggested, for unity8
User14: good
User14: this is how gsettings is meant to be used
User4: unitu7 is in sitting in the middle of 2 worlds
User14: ya...
User4: unity*
User14: in general, i really like the way new things are going
User14: it's going to be very very much cleaner
User4: right
User4: I wonder how much "different desktops on a same distro" is going to stay true though
User4: the GTK/Desktop stuff showing it why today
User14: this was a stupid change, imh
User14: +o
User4: the xsettings logic is sort of a strech
User14: the trivial work that i did today should have just been done from the start
User14: and this isn't about different desktops on the same distro
User14: this is about different desktops using the same toolkit
User4: right, they screwed other OSes on the way
User14: which is a problem that gtk chooses to have.... we just have to remind from time to time that this is their choice
User14: and showing up with patches is always the nicest way to do that ;)
User4: I guess it's rather the discussion" GTK is the GNOME toolkit"
User14: gtk is only the gnome toolkit insofar as non-gnome people don't show up offering to do work
User14: today a non-gnome person (me) showed up with patches.  they got applied almost straight away.
User14: these patches didn't benefit gnome in any way at all
User4: yeah, as long as they are fine taking the code to accommodates other OSes
User14: which, again, is a choice that gtk made
User4: right
User4: I'm pretty happy about the outcome ;-)
User14: it's supposed to be a cross-platform toolkit.  that includes linux-outside-of-gnome too.
User4: I was unsure how Bastien would respond when you asked
User14: User4 you duped :p
User14: https://bugzilla.gnome.org/show_bug.cgi?id=711040
User9: Gnome bug 711040 in GtkFileChooser "GtkFileChooser: Desktop entry missing" [Normal,Unconfirmed]
User4: desrt, heh, I listed it myself in https://bugs.launchpad.net/ubuntu/+source/gtk+3.0/+bug/1171587/comments/3 that doesn't count ;-)
User9: Launchpad bug 1171587 in gtk+3.0 (Ubuntu) "update to GTK 3.9 and the issues to resolve for it" [Wishlist,Confirmed]
User4: but yeah, I should maybe have reused that one
User14: you were so upset that you wanted it to be two bugs? :)
User4: let's say that I didn't want to spam that poor-contributor's bug with my ranting :p
User14: the rest look like a bunch of theme issues
User14: (sigh)
User14: would be really nice if our theme was maintained...
User4: right
User4: well, it's not
User4: the overlay-scrollbar bugs larsu is working on is not a theme issue
User14: ah.  surprising.
User4: it's an issue with the rendering optimisation alex did
User14: ah.  that makes sense as well, i suppose.
 * desrt freaking loves the new alex stuff, btw
User4: to give some credit where it's due, the 3.8 issues we had with scrollbars turned you to be a GTK bug
User14: that item was on my todo list for a while (larsu and i chatted about how we'd do it when we were in copenhagen)
User4: and one where users were vocal about in other softwares as well
User14: and then alex just came with code
User8: User14 which item?
 * larsu doesn't feel thrilled about alex' optimizations today
User14: User8 changing how the scrolling works
User4: larsu, not drinking Berlin beers tonight? ;-)
User14: pure cairo-based.... no more GdkWindow hackery
User8: User4 that was Tuesday. Can't have beer every night :)
User4: ;-)
 * desrt wants to drink a $22 beer...
User4: desrt, talk to attente
User14: :)
User8: User14 ah, right. The thing I'm fought with is a change that leads up to that
User4: attente, congrats on getting the compiz key grabbing working btw, seems user feedback of the ppa is good ;-)
User8: I think I've found the issue, but was too exhausted to go on further earlier today
User19: seb128, thanks
User14: is this the one where it only works when the user changes the keybinding after logging in?
User4: desrt, he fixed that!
User14: oh.  good
User4: ;-)
User14: because that sounded ridiculous :)
 * desrt was about to offer ways to trick gsettings into reporting change notifications :)
User4: desrt, we seem to be on parity with gnome-shell now :p
User19: that problem was stressful..
User14: User19 maybe i should buy you a $22 beer :p
User4: (which doesn't mean bug free)
User4: desrt, yeah, you should
User14: User4 gnome-shell has bugs?
User14: weird!!!
 * seb128 would if he was in Canada
User19: it becomes more like a $1022 beer
User4: huuuum, maybe I'm stepping out
User14: User19 because you want to buy 40 of them?
User4: attente, I bet you can't drink those 40 beers in one night
 * attente adds the cost of the flight
User19: :P
User14: ah
User4: if you can I'm paying for them
User19: at that point you might as well be in belgium i suppose...
User14: User4 stop that
User14: User4 we finally have a guy who knows about keyboard and input method and you want to kill him with alcohol poisoning
User4: yeah, that's not cool
 * seb128 stops
User4: attente, come to Berlin next year, we can have good beers there as well
User4: you can ask larsu ;-)
 * larsu confirms
User14: User4 he'll be in strassbourg next year...
User4: \o/
User14: i guess the beer there is probably also okay :)
User4: yeah, people coming to France!
User8: what a coincidence: so will I!
User4: ;-)
User8: the world is such a small place
User19: fun times :)
User4: ok, on that note I'm calling it a day
User14: User4 good night
User4: thanks, you too!
User19: nite seb128 :)
User4: attente, thanks, you too ;-)
User14: User8 did you check that your appmenu patch works?
User4: larsu, you should call it a day as well, tomorrow is friday need to keep going until the W.E ;-)
User8: User4 I already did. Just hanging around on irc :)
User8: User14 yes
User14: cool.  i just ACK'd it
User8: thanks.
User14: which is meaningless since i removed myself from the product strategy team....
User8: you can do that now, eh?
User14: oops!
User8: I'm not on that team eitherâ¦
User14: you're directly in the indicators team, though
User8: right
 * desrt was indirectly before, via PS
User8: being in PS makes your inbox blow up
User14: ya.  this is why i left :)
ubuntu-desktop
Ubuntu IRC