/srv/irclogs.ubuntu.com/2015/10/23/#ubuntu-gnome.txt

eliaspsdarkxst here are the debdiffs from debian syncs: http://people.ubuntu.com/~eliasps/files/debdiffs/ . Let me know if they are ok. Thank you.00:21
darkxsteliasps, they are correct, though its probably just as easy to to list which packages can be sync'ed01:36
darkxsteliasps, clutter can be sru'ed into wily01:36
darkxsteliasps, can you prepare a bug and paperwork for that?01:39
darkxstsee https://wiki.ubuntu.com/StableReleaseUpdates#SRU_Bug_Template01:39
eliaspsdarkxst I'm willing to, but I do need to read the documentation to be sure to understand it and do it right.01:41
darkxsteliasps, its worth learning, all updates into wily will now require SRU's01:48
eliaspsdarkxst, no worries. I'm eager to learn those stuff to get more familiar with this aspect of Ubuntu. So I'll do the reading.01:49
darkxsteliasps, great!01:57
darkxsteliasps, just let me know when its ready and I will sponsor for you01:59
eliaspsdarkxst Ok, I'll let you know as soon as I'm done!02:00
berglhare do-release-upgrades working04:13
darkxstberglh, should be working, atleast not heard of any problems04:25
berglhokay, i'll apt-get upgrade first04:27
berglhand give it a crack04:27
berglhi figure i'm doing something wrong04:27
berglhhighway to the danger zone04:33
darkxstanyone able to verify the gdm update in trusty-proposed?04:53
darkxsthttps://bugs.launchpad.net/ubuntu-gnome/+bug/1315442/comments/1504:53
ubot5Launchpad bug 1315442 in Ubuntu GNOME "Extra "fi" in /etc/init.d/gdm" [Undecided,New]04:53
berglhdarkxst: it works05:08
berglhit's noticeably snappier for some reason05:08
darkxsta few people have been mentioning that05:13
mgedminhm, is the gjs-console crash going to be a daily event now?05:14
mgedminoh right, I re-enabled the "weather" search05:15
berglhmgedmin: welcome to the bleeding edge05:21
berglhnot sure that i like the new gnome 3.16 top bar/menu separator line05:22
berglhi think it's cause it's different05:22
berglhi'll probably just get used to it05:23
mgedminmy 1st reaction to 3.16 wasn't very positive, but I got used to it05:23
berglhi think cause it's such a high contrast to the adwaita theme colours it's noticeable05:24
mgedminthe rather very square blue-on-gray buttons threw me initially05:25
mgedminok, let's say I'd like to reproduce the gjs-console crash under valgrind05:25
mgedminI probably want debug symbols05:26
berglhin 15.04 my background change throughtout the day never worked05:26
berglhthe draw time on context menus is a lot shorter05:27
mgedminwait, the crash is from gjs-console running stuff in /usr/share/gnome-documents/js05:35
berglhfor what it's worth i had the same crash05:36
mgedminso probably unrelated to the weather search provider05:36
berglhdoes it only happen on startup?05:36
mgedminit happens when I hit <super> and start typing to launch an app05:36
mgedminhttp://pad.lv/143209805:36
ubot5Launchpad bug 1418771 in gjs (Ubuntu) "duplicate for #1432098 gjs-console assert failure: *** Error in `/usr/bin/gjs-console': free(): invalid next size (fast): 0x00007f74a804b240 ***" [Low,Confirmed]05:36
darkxstmgedmin, if you can reproduce it under gdb, then grab gjs_dumpstack05:40
mgedminit's memory corruption; I'd rather reproduce it under valgrind05:40
darkxstvalgrind won't work if its a JS bug though05:40
mgedminI don't think JS can cause memory corruption; it's probably in some C library that JS calls into05:41
darkxstoh that is the tracker crash05:41
darkxstcancellable=0x205:44
mgedminuh, how can I dbus-monitor gnome-shell's search chatter05:44
mgedmin(for me the crash is about free() finding a corrupt 'next' pointer)05:45
mgedminah!  thank you gnome-terminal's builtin search05:46
darkxstyes, but it is crashing running the sparql query05:46
mgedminhm05:48
mgedminof course I'm having difficulties reproducing the crash05:49
mgedminhey, so uh how does dbus-monitor work?05:49
mgedminif I run dbus-monitor destination=org.gnome.Documents, should I see stuff?05:49
darkxstdbus-monitor --sesion05:49
mgedminbecause if I run dbus-monitor without any arguments, I see05:49
mgedminmethod call time=1445578894.367018 sender=:1.27 -> destination=org.gnome.Documents serial=1274 path=/org/gnome/Documents/SearchProvider; interface=org.gnome.Shell.SearchProvider2; member=GetInitialResultSet05:49
mgedminand way too may other junk05:49
darkxstyou might need to set destination and interface05:51
mgedmindbus-monitor path=/org/gnome/Documents/SearchProvider works05:54
mgedmindestination is numeric, now, for some reason05:54
darkxstcan you get the full sparql query and try run that manually? (the one in the trace is truncated)05:55
mgedminI'll see what I can do05:56
mgedminyes!  repro'd!05:57
mgedminterminal 1: gnome-documents --gapplication-service05:58
mgedminterminal 2 (within 10 seconds of 1st command): dbus-send --print-reply --dest=org.gnome.Documents /org/gnome/Documents/SearchProvider org.gnome.Shell.SearchProvider2.GetInitialResultSet array:string:"search"05:58
mgedminI've got gdb06:00
mgedmin(but no debug symbols installed yet)06:00
mgedminwhy is gjs-dbgsymn only available for i386???06:01
darkxstno idea06:02
darkxstyou probably only need glib, tracker symbols etc06:04
mgedminuh, what's the command to tell gdb to load symbols that I've just installed?06:07
darkxstthey get loaded automatically when you run06:07
mgedminI guess this reproduces easily06:08
mgedminI don't see the sparql query in the stack trace: http://paste.ubuntu.com/12900524/06:10
darkxstits in another thread06:11
darkxstt a a bt06:11
mgedminhttp://paste.ubuntu.com/12900528/06:12
mgedminall threads look rather idle to me06:13
mgedminhaha I accidentally launched gnome-documents under valgrind without the --gapplication-service flag06:15
mgedminvalgrind found some things to complain about: http://paste.ubuntu.com/12900543/06:16
mgedminha ha, search succeeds if I run gnome-documents under valgrind06:17
mgedminwith valgrind errors: http://paste.ubuntu.com/12900547/06:18
mgedminlibmozjs-24-0v5-dbgsym is also i386-only06:19
mgedminInvalid write of size 1 in tracker_parser_unaccent_nfkd_string in function_sparql_unaccent06:22
mgedmintracker has two implementations of each06:32
mgedmingdb tells me these are used:06:32
mgedminhttps://github.com/GNOME/tracker/blob/3ab7ca19c48052868020452ba0bf504984d63ac4/src/libtracker-data/tracker-db-interface-sqlite.c#L66606:32
mgedminhttps://github.com/GNOME/tracker/blob/ab77180c736d4de8a2b710d0e010807bd4b51c3a/src/libtracker-common/tracker-parser-libunistring.c#L16206:32
mgedminI guess valgrind tells me u8_normalize(UNINORM_NFKD, "10", 2, NULL, &written) mallocs a 2-byte array and copies "10" into it (written == 2)06:33
mgedminand then tracker_parser_unaccent_nfkd_string(zOutput, &written) helpfully writes a '\0' past the end of the allocated buffer06:34
mgedmincommented on upstream bug: https://bugzilla.gnome.org/show_bug.cgi?id=746195#c606:40
ubot5Gnome bug 746195 in General "tracker crashing within gnome-documents search provider" [Normal,Needinfo]06:40
andy___Hiya. Is there a reason why ubuntuGNOME 15.10  doesn't pack gnome3.18 with it?07:01
mgedminyes: not enough time between the gnome 3.18 release date and ubuntu-gnome 15.10 release date to get it in, with the few volunteers that are working on ubuntu gnome07:04
mgedminyou can get gnome 3.18 from the staging ppa07:04
mgedminotoh I'm curious why gedit is stuck at version 3.10 (!)07:05
mgedminfor nautilus 3.14 the reason was "3.16 breaks too many ubuntu patches"07:05
mgedminis ubuntu patching gedit too?07:05
andy___Oh okay so it's due to practical reasons, not because of safety or whatever07:05
andy___because the staging ppa warned me that i could break serious stuff07:06
darkxstmgedmin, does it help if you drop the addition of null on end of string? I have a suspicion they not meant to be null terminated strings, since always passing the lengths around07:52
mgedminwheee system monitor: http://imgur.com/BuIIPLk07:53
darkxstgedit is the same deal, needs UI patches for CSD and menus07:53
darkxstI think -desktop team may just give up and go with the CSD version for X though07:53
mgedminthat big gray area in my screenshot is a SearchBar!07:55
darkxstlol, where is the search UI?07:55
mgedminhttp://imgur.com/BuIIPLk07:57
mgedminok, scary thing: gtk+ inspector shows a GtkBox with three children: GtkSearchBar (expand=FALSE fill=TRUE), GtkScrolledWindow (expand=TRUE fill=TRUE) and GtkRevealer for the action bar at the bottom (expand=FALSE fill=TRUE)07:58
mgedminif the searchbar is expand=FALSE, why is it so big?07:58
mgedminhttps://bugzilla.gnome.org/show_bug.cgi?id=75581808:04
ubot5Gnome bug 755818 in general "Oversized GtkSearchBar" [Normal,Resolved: fixed]08:04
mgedminshould be fixed in 3.18.0.108:04
mgedmindebian testing has 3.18.0.1-108:05
darkxstmgedmin, k, will sync08:06
mgedminthanks!08:07
mgedmin(oops I realized I pasted the same imgur link twice; 2nd was supposed to be http://imgur.com/KsxNbwo)08:11
darkxstmgedmin, done08:22
darkxstmgedmin, hmm, can't reprodice the crash under jhbuild08:34
darkxstI just get lots of (gnome-documents:12623): Tracker-CRITICAL **: tracker_parser_unaccent_nfkd_string: assertion '*str_length > 0' failed messages but no crash08:35
darkxstmgedmin, may be caused by a corrupt index08:43
mgedmindid you try valgrind?09:16
mgedminmy gdb shows that function_sparql_unaccent() gets called multiple times09:17
mgedminwith strings that probably come from document titles?09:17
mgedminthe 1st one I saw was "10", then I saw a couple of "", then I got bored09:17
mgedminthose assertion failures oh my09:18
mgedminI think tracker_parser_unaccent_nfkd_string: assumes *str_length includes the trailing \009:18
mgedminbut libunistring's data model is strings with an explicit length and without a trailing \009:18
mgedminI wonder if it's something special about strings of length 2?09:24
mgedminmaybe it's strings of length that is a power of 2?09:24
mgedminbecause memory allocators pad09:24
mgedminand if you write the \0 into the padding, that probably won't destroy malloc's internal structures09:25
mgedminhm, maybe it's not the 2-length string that causes the crash?  it's just the 1st that valgrind detects09:25
darkxstmgedmin, if that was causing the crash, it would only be seen once most likely09:30
mgedminbtw tracker didn't crash when I ran it under valgrind09:30
mgedminI don't know if valgrind discards invalid writes or what09:30
darkxstthough I do think tracker is wrong to add the \009:30
darkxstyes valgrind ignores them09:30
darkxstwell keeps running09:31
mgedminI'm worried about the eventual recipients of that string: do they expect a trailing \0?09:31
mgedminin this case it's sqlite, and it receives char* + a length09:31
mgedminlibunistring is a bit vague: https://www.gnu.org/software/libunistring/manual/libunistring.html#In_002dmemory-representation09:31
mgedminu8_normalize: https://www.gnu.org/software/libunistring/manual/libunistring.html#index-u8_005fnormalize-72309:31
mgedminUTF-8 strings: https://www.gnu.org/software/libunistring/manual/libunistring.html#Unicode-strings09:32
mgedmin"there are two variants"09:32
mgedminI'm guessing u8_normalize is the 2nd: pointer and number of units (not bytes! but for UTF-8 units are bytes)09:32
mgedminso no trailing NUL09:32
mgedminhttps://www.sqlite.org/capi3ref.html#sqlite3_value_blob09:34
mgedmin"If the result is a BLOB or UTF-8 string then the sqlite3_column_bytes() routine returns the number of bytes in that BLOB or string"09:35
mgedmin-- https://www.sqlite.org/capi3ref.html#sqlite3_column_blob09:35
darkxstI didnt look too deeply, but usually when functions require passing around str_lengths, they don't expect nul terminated strings09:35
mgedmin"The values returned by sqlite3_column_bytes() and sqlite3_column_bytes16() do not include the zero terminators at the end of the string. For clarity: the values returned by sqlite3_column_bytes() and sqlite3_column_bytes16() are the number of bytes in the string, not the number of characters."09:35
mgedminyeah, I'm just doing due diligence09:35
mgedminnow: are there any other callers of tracker_parser_unaccent_nfkd_string() in the tracker codebase?09:36
mgedmindo those callers expect a trailing \0?09:36
mgedmingit grep says: no other callers09:36
mgedminbut there are two implementations09:36
mgedminand two call sites (in two different implementations of function_sparql_unaccent)09:36
mgedminthe 2nd call site relies on the NUL terminator!09:38
mgedminhttps://www.sqlite.org/capi3ref.html#sqlite3_result_blob09:38
darkxstmgedmin, two implementations, because there are two unicode interfaces09:38
darkxstlibunicode and mozilla's icu09:38
mgedmin"If the 3rd parameter to the sqlite3_result_text* interfaces is negative, then SQLite takes result text from the 2nd parameter through the first zero character."09:39
darkxstlibunistring is the prefered one though09:40
mgedminoh but wait two different tracker_parser_unaccent_nfkd_string implementations too09:40
mgedminone works in UTF-16, one works in UTF-809:40
mgedminthe UTF-16 needs the trailing NUL, the UTF-8 shouldn't do it09:41
mgedminwait, I was wrong when I said there was only one caller09:41
mgedminprocess_word_uchar also calls tracker_parser_unaccent_nfkd_string impl09:41
darkxstthat will just abstract the backend away09:42
mgedminso, process_word_utf8 also calls tracker_parser_unaccent_nfkd_string()09:43
darkxstalthough its all build time ifdefs so shouldnt need that09:43
mgedmindoes it rely on a trailin NUL?09:43
mgedminyes09:44
mgedminok, I think this patch has a chance of being correct: http://paste.ubuntu.com/12901327/09:44
darkxstprocess_word_utf8 already nul-terminates the string before the tracker_parser_unaccent_nfkd_string() call09:50
mgedminbut _unaccent_() makes the string shorter09:50
mgedminin which case the NUL needs to be moved to an earlier position09:51
mgedmingit patch with a commit message (but still untested): http://paste.ubuntu.com/12901358/09:51
mgedminI've imported the source tree produced by pull-lp-source into a local git repo because I don't know proper tools09:51
darkxstmgedmin, why the change in the assertion?09:56
darkxstit doesnt make much sense to accept a zero length string10:01
darkxstif you don't have a jhbuild setup, the easiest way to test is use a quilt patch against the ubuntu packaging10:01
* mgedmin uploads patch to https://bugzilla.gnome.org/show_bug.cgi?id=746195#c710:09
ubot5Gnome bug 746195 in General "tracker crashing within gnome-documents search provider" [Normal,Needinfo]10:09
=== \b is now known as benonsoftware
darkxstmgedmin, but still why the change in the assertion (g_return_val_if_fail)? otherwise I think it looks ok10:12
mgedminI lost network access10:14
mgedmindid you not see my explanation, or was it too unclear?10:14
mgedminare there channel logs?10:14
mgedmin!logs10:14
ubot5Official channel logs can be found at http://irclogs.ubuntu.com/ . LoCo channels are now logged there too.10:14
mgedminok, it didn't make it10:15
mgedmin<mgedmin> because an empty NUL-terminated string would be { '\0' }, so str_length == 110:15
mgedmin but since there's no NUL, an empty string has str_length == 010:15
mgedmin and there's no reason an empty string cannot be unaccentified10:15
mgedmin<mgedmin> no crash!10:15
mgedmin a few Gjs-WARNING **: JS ERROR: TypeError: cursor is null: http://paste.ubuntu.com/12901418/10:15
mgedmin valgrind is happy too!10:15
mgedmin well, those use of unitialized value warnings in libmozjs10:15
mgedminI have jhbuild, but I wasn't sure I could use it10:16
mgedminso I built a local deb based on the wily source10:16
mgedminoh, right, irclogs lag a lot, so maybe I was too hasty in assuming my explanation didn't make it?10:17
darkxstI didnt get it first time around10:17
darkxstmgedmin, its crap, if you really want to accept empty strings drop the assertion10:19
mgedminwait, it's a size_t?  oops, unsigned ;D10:20
mgedminah, gsize, but still unsigned10:21
mgedminhow do I use git with debian packages without going insane?10:22
darkxstgbp10:23
mgedminany blog posts/short tutorials?10:23
darkxstgbp import-dsc <package.dsc>10:24
darkxstif you want to do upstream patches there is 'gbp pq' that converts the quilt queue into a git branch10:25
darkxstgbp buildpackage to build the source10:25
darkxstI do everything in that10:26
darkxstexpect upstream work, is usually done in jhbuild much faster iteration on builds ;)10:26
mgedminstep 0: pull-lp-source -d tracker wily10:27
mgedmin'gbp pq' fails with gbp:error: No action given.10:27
mgedmingbp pq import, I guess10:28
darkxstyes, they are just the commands, you need options also  ;)10:29
darkxstI use this script, after gbp import-dsc <debian.dsc>10:29
darkxsthttp://pastebin.com/MpejZQU610:29
darkxstgives me a debian branch and an ubuntu branch10:30
darkxst(and falls back to pull-lp-source if there is no packaging branch on bzr)10:35
mgedminthank you!10:37
darkxstmgedmin, the ddeb indexes are broken appparently pitti is looking into it10:38
mgedminyeah, I complained on #ubuntu-devel10:38
darkxstok10:38
mgedminv2 of the patch: https://bugzilla.gnome.org/show_bug.cgi?id=746195#c910:39
ubot5Gnome bug 746195 in General "tracker crashing within gnome-documents search provider" [Normal,Needinfo]10:39
mgedminuh, can pull-lp-sources pull sources from PPAs?10:46
* mgedmin realizes he built a tracker 1.4.1-1ubuntu2mg110:49
darkxstno10:50
darkxstpull-lp-source will get 1.4.1-1ubuntu2 form archives10:51
* mgedmin is sad10:52
darkxstyou chdist if you want to pull from ppa's10:53
darkxsts/you/use/10:54
darkxstanyway patch is probably ok, would probably be cleaner if _unaccent_ didnt get called with empty strings, but wait and see what martyn says on that10:56
mgedmin-ETOOMANYTOOLS10:56
mgedmincalling u8_normalize on empty string is probably also redundant?10:56
darkxstthat may be, but chdist is very useful10:56
KetsubanIs there a reason there's no GNOME Builder package available for 15.10?10:56
darkxstKetsuban, it will be on gnome3-staging at some point10:57
mgedmingnome-builder 3.18.0-0ubuntu1~wily4 is there already10:58
darkxstmgedmin, I didnt look at the code for u8_normalize, does is just shortcut on an empty string?11:00
mgedminAFAIU libunistring is a GNU library, which means it's not on github11:00
darkxstit will be on git somewhere!11:01
mgedminhttps://codesearch.debian.net/results/package:libunistring%20u8_normalize/page_011:01
mgedminwheeee http://sources.debian.net/src/libunistring/0.9.3-5.2/lib/uninorm/u8-normalize.c/?hl=33#L3311:01
mgedminno short-circuiting: http://sources.debian.net/src/libunistring/0.9.3-5.2/lib/uninorm/u-normalize-internal.h/11:02
mgedminha ha icon sizing is a hard problem: http://imgur.com/O96xsgv11:07
darkxstI thought they unbroke that, guess not11:10
mgedminlooks like https://bugzilla.gnome.org/show_bug.cgi?id=73846711:10
ubot5Gnome bug 738467 in general "Some icons are enormously large" [Normal,Resolved: fixed]11:10
mgedminhm, it's old!11:10
darkxstyeh, I thought they fixed the lack of force-scaling11:11
darkxstsince it broke legacy icons all over the place11:11
darkxstI gtg anyway11:12
mgedmino/11:12
mgedmingvim is often THE LARGEST ICON EVER, but it's properly force-scaled in system-monitor11:12
mgedminmaybe some icons are exactly the right size to be a bit too large but sneak in under the limit?11:12
* darkxst sleeps now11:17
mgedminwaah exciting new bugs12:38
mgedmingnome-contacts crashed for no reason12:44
mgedminoh, in the search provider12:44
mgedminsegvanalysis: "Skipped: missing required field Disassembly"12:44
mgedminDAMN YOU APPORT12:44
mgedminI had a bug open about this12:45
jokxHi, a little question : I've installed Ubuntu-Gnome Wily + ppa:gnome3-team/gnome3, but Eye-Of-Gnome and Evince show me an horrible menu in place of the normal header bars ... I think that modification is done for Unity, but for gnome user : is there a way to recover un-patched application ?13:18
jokxOr at least : removing that menu et get back the normal header-bars ?13:20
eliaspsdarkxst how do I describe a testcase for clutter's sru in the bug report, also a regression potential.14:38
eliaspsI saw some bug reports with similar requests and they don't have those subjects on the description.14:39
eliaspsdarkxst there is also a depencency issue for clutter and wily: libcogl-dev doesn't meet the required version14:56
ricotzdarkxst, hi, I am going to copy the gnome3-staging wily pocket to xenial15:28
eliaspsI'll test the syncs in a personal PPA before sending you them for uploading in staging.16:14
guruprasadI have been using Debian unstable (with the GNOME desktop environment) as a rolling release for the past 2 years without any issues. Recently there are many transitions happening in Debian unstable and upgrading regularly may not be such a good idea. So I just thought I could try Ubuntu flavours again (after using Ubuntu for 5 years prior to Debian)17:50
guruprasadI am thinking about trying Ubuntu GNOME which will help with continuity but I am not keen about waiting till the next release for the latest and greatest17:51
guruprasadSo is there a way to use Ubuntu GNOME as a semi-stable (at least not as unstable as Debian unstable) rolling distro?17:52
=== adrian is now known as alvesadrian
=== manokara_ is now known as manokara
guest231015Why has the logo changed to a Shazam like logo?21:02

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