/srv/irclogs.ubuntu.com/2014/10/16/#ubuntu-desktop.txt

=== olli_ is now known as olli
=== qengho is now known as CardinalFang
=== CardinalFang is now known as qengho
seb128good morning desktopers!06:06
happyaronmorning!06:11
larsuhi guys06:12
happyaronseems upload notification email is broken.06:12
seb128happyaron, hey! you mean the ones you get when you dput something to the archive?06:15
happyaronyup06:16
seb128larsu, desrt, could one of you look at bug 1381804?06:49
ubot5bug 1381804 in glib2.0 (Ubuntu) "whoopsie test failure since glib2.0 2.41.2-1 uploaded" [High,New] https://launchpad.net/bugs/138180406:49
desrtseb128: i'll take a peek06:51
seb128desrt, thanks06:51
seb128desrt, it hits a "g_main_loop_quit: assertion 'g_atomic_int_get (&loop->ref_count) > 0' failed"06:53
seb128I've an i386 install/can reproduce if you need info06:53
seb128desrt, http://bazaar.launchpad.net/~daisy-pluckers/whoopsie/trunk/view/head:/src/tests/test_monitor.c#L12506:55
larsudesrt: let me know if you need help06:57
seb128the new indicator headers on the phone are nice looking!07:07
=== oCrazyLem is now known as CrazyLemon
desrtseb128: only on i386?07:09
seb128desrt, bugs states on !amd6407:10
seb128desrt, https://launchpad.net/ubuntu/+source/whoopsie/0.2.3807:10
desrtseb128: was just getting there :)07:10
desrtfirst theory: testcase is doing something evil :)07:11
seb128desrt, so yeah, arm, ppc, i38607:11
seb128yeah, that's possible07:11
seb128something that used to work with old glib and still work on amd64 for some reason07:11
seb128but fixing the testcase is a valid fix ;-)07:11
chrisccoulsonseb128, oh, I only just noticed the new headers07:11
seb128chrisccoulson, hey! they feel nice, don't you think?07:12
seb128chrisccoulson, how are you btw? ;-)07:12
chrisccoulsonyeah, they're much better than before07:12
chrisccoulsonI'm good thanks, how are you?07:12
seb128I'm good, thanks07:12
seb128larsu, would you fancy looking at https://bugs.launchpad.net/bugs/1381897 ?07:13
ubot5Ubuntu bug 1381897 in evince (Ubuntu) "Evince will not zoom in with shortcut key in 14.10 Release Candidate?" [Low,Incomplete]07:13
larsuseb128: sure07:13
seb128larsu, I think that's something you fixed in the distro patched version of evince we had but that seems buggy in current upstream07:13
seb128larsu, thanks07:14
seb128larsu, ctrl-shift-+ vs ctrl-+07:14
larsuseb128: right, we added <ctrl>= or something for those users07:15
seb128larsu, maybe you can put a patch upstream up for review, that does that?07:16
desrtseb128: don't see any mention of atomics in the report07:18
seb128desrt, that's the output from running the test manually on my box07:19
seb128I guess the make check run eats stderr07:19
desrt** (./test_monitor:9220): WARNING **: Expected file /tmp/whoopsie-test-SLNTNX/fake.uploaded doesn't exist.07:19
desrtseems like this is the original complaint.... i wonder if you're seeing something different again07:20
seb128no07:20
seb128https://launchpadlibrarian.net/186966939/buildlog_ubuntu-utopic-armhf.whoopsie_0.2.38_FAILEDTOBUILD.txt.gz07:20
seb128" /whoopsie/callback-never-triggered: [04:53:09] Could not read directory (2): /fake07:20
seb128OK07:20
seb128 /whoopsie/callback-triggered-once:07:20
seb128(./test_monitor:7527): GLib-CRITICAL **: g_main_loop_quit: assertion 'g_atomic_int_get (&loop->ref_count) > 0' failed07:20
seb128Trace/breakpoint trap"07:20
seb128desrt, see how the could not read gives an OK07:21
seb128desrt, that's the buildlog from armhf07:21
seb128well, maybe it's a different issue, but the one that current fails the build is the one I'm seeing locally07:21
* desrt grabs i386 iso07:23
desrtdon't have my VM party when i'm not at home :)07:23
larsudesrt: ask seb128 to let you ssh into his machine :P07:24
seb128desrt has ssh access to porter box07:24
seb128Debian ones at least I think07:24
desrti do!?07:24
desrtoh right.... from ages ago07:24
desrti bet i have a new ssh key since then :)07:24
desrti think it will be faster to download the iso :p07:26
larsuseb128: I was kidding...07:26
desrtcode in this test is suspicious07:26
seb128larsu, I'm not, I ssh to porter box quite often for debugging, easier that vms07:26
willcookemorning07:58
seb128hey willcooke, how are you?08:02
larsuhi willcooke08:02
willcookeo/08:02
Laneyyo08:03
Laneythere were no pips at 9am just now on radio 408:03
Laneyso my morning routine is RUINED08:03
willcookeWHAT?08:03
Laneyseriously08:04
Laneywe're probably being invaded or something08:04
willcookeWasn't that some secret signal for WW3?08:04
willcookeglad I'm getting out of this country tomorrow morning.  It's going to down the pan08:05
larsuLaney: morning :)08:05
Laneyhey larsu08:06
Laneyhow's it going08:06
seb128hey Laney08:06
=== tiheum is now known as tiheum_
seb128willcooke, flying to Washington tomorrow?08:07
willcookeseb128, yeah, made the flights about 1/2 the price08:07
seb128weird08:07
Laneyhey seb12808:07
seb128or are you coming back on friday evening or saturday?08:07
willcookeseb128, Friday evening08:07
seb128k08:07
Laneythat's probably just the saturday rule08:07
seb128yeah, saturday night rule08:07
seb128though it should have been cheap on saturday as well08:08
seb128just not on sunday08:08
willcookeI dont get that rule. Is it some kind of tourist trap, to make you spend more money?08:08
seb128willcooke, no, it's what makes business people different from tourists08:08
willcookeahh, kk08:09
seb128willcooke, http://en.wikipedia.org/wiki/Saturday-night_stay08:09
willcookeha - I should have known there'd be a Wikipedia page08:09
seb128:-)08:09
willcookeseb128, can we work through that Gtk stuff this morning?  I'll document it as well.  Squeeze another blog post out ;)08:10
seb128willcooke, sure08:11
seb128let me take my test laptop08:11
willcookeseb128, shall we do it over IRC?  If so, perhaps a side-channel?08:12
* larsu overheard something about gtk stuff08:12
seb128willcooke, I think it's going to be a 3 lines IRC discussion08:12
willcookeoh sweet08:12
willcookelet's do it here then?08:12
seb128just need to check again the magic08:12
seb128yes08:12
seb128larsu, having gtk apps listed in the unity8 dash on desktop-next08:13
seb128larsu, so you can run e.g gedit08:13
larsuwe don't?08:13
seb128no08:13
seb128because no backend08:13
larsudoesn't it use the same setting as unity7's dash?08:13
larsuoh08:13
seb128so we didn't want to list something not working08:13
larsuright08:13
seb128no08:13
seb128the issue is that they didn't want to list tons of non working stuff08:13
seb128like x-apps, java, gtk, etc08:13
seb128so they went with a whitelist approach08:13
seb128well "whitelist"08:14
larsuthat makes a lot of sense for now08:14
seb128some new key in the .desktop08:14
seb128right08:14
seb128result is that e.g gedit is not listed08:14
seb128so you need manual hack to use it08:14
seb128willcooke wants to know the hack08:14
Laneythere's already a "GTK" desktop file category08:15
LaneyFWIW08:15
seb128Laney, yeah, but Unity8 doesn't use that atm08:16
LaneyI'm sure not08:16
LaneyIf that works it's better than modifying tons of desktop files though08:17
seb128right08:17
Laneyit's at least vaguely upstreamable08:17
seb128except if they want to make sure to not list buggy softwares08:17
seb128which they were leaning toward08:18
Laneygood luck with that08:18
seb128well, the idea was "add the key to the .desktop if it has been tested/confirmed to work"08:18
Laneysounds scalable and maintainable08:18
seb128</irony> :p08:18
willcookethis is "only" a quick hack so we can demo to Jane08:19
seb128willcooke, yeah, we are discussing futur/proper solutions there08:19
seb128not the demo08:19
willcookeahh08:19
willcookekk08:19
willcookein which case...08:19
willcookecarry on08:19
willcooke;)08:19
seb128willcooke, so, I think you just need to add "X-Ubuntu-Touch=true" to your .desktop08:20
seb128willcooke, try adding that to /usr/share/applications/gedit.desktop and restart your unity8 session08:20
willcookewill do...08:20
* willcooke install gedit first 08:21
willcooke;)08:21
Laneyoh I should unbrick my phone08:22
willcookewould it "work" for xchat-gnone too?08:22
Laney285 was not good08:22
seb128Laney, did you wait for a while?08:22
Laneylike 60 minutes08:23
LaneyI forgot about it :p08:23
Laneyalso there's some posts on the ubuntu-phone list08:23
seb128willcooke, worth trying, but I think that had issues due to some widgets/xorg use08:23
willcookeack08:23
seb128Laney, k, complain to ogra then!:08:23
ogra_hey ... that were slangasek and cjwatson who played with utopic :P08:24
willcookeseb128, done - it appears in the list, but just crashes08:25
seb128willcooke, xchat or gedit?08:25
willcookeseb128, geofft08:25
willcookeoops sorry geoff08:25
willcookeseb128, gedit08:25
seb128:-(08:25
seb128works on my machine08:25
geofftaw man, I wanna be a text editor08:26
seb128well, let me update before claiming that08:26
willcookeerk08:26
willcookeseb128, stop!08:26
* seb128 stops08:26
Laney(hammer time)08:27
willcookeseb128, now I cant even get lightdm to start08:27
* Laney dances08:27
seb128shrug08:27
willcookeafter a reboot08:27
willcookeseb128, dont want you to break your machine too08:27
willcooke:)08:27
seb128best time to brick your laptop08:27
willcooke:)08:27
seb128before travelling!08:27
willcookeseb128, I'll upgrade everything and try again08:27
seb128well, that's a test machine, I'm happy to brick it08:27
seb128so I can help debugging08:27
* seb128 upgrades08:27
willcooke:)08:30
willcookeseb128,  dist-upgrade and I'm back with a GUI08:36
willcookeseb128, also Store seems to be working now08:37
seb128nice!08:38
willcookeseb128, still cant get gedit to load08:42
willcookeseb128, where should I look for logs?08:42
seb128willcooke, .cache/upstart/unity8.log08:43
=== tiheum is now known as tiheum_
seb128willcooke, sorry, mine is still dist-upgrade, disk is sloooow, I should change it as well ;-)08:47
willcookeseb128, looking at the logs08:47
willcookeseb128, bah - nothing obvious in the log:  http://pastebin.com/vnqdh10a09:16
seb128willcooke, to me it looks like the recent mir updates made the gtk-mir backend stop working09:18
willcookearse09:18
willcookeoh well, so be it09:18
seb128willcooke, there is a "application-legacy-geditmir-.log" with a "gdk_mir_event_source_queue_event: code should not be reached"09:18
seb128willcooke, somebody for robert_ancell to look at, maybe tomorrow09:19
willcookecool09:19
willcookeI'll drop him a line, thanks seb12809:19
seb128willcooke, I was about to reply to the gtk-mir email, I can do that if you want09:20
seb128or you do it, that works too ;-)09:20
willcookeseb128, oh, if you could that would probably be better than me saying "it won't go - I have no idea what I'm doing"09:20
willcooke:D09:20
seb128willcooke, ok, doing so09:20
willcookethanks09:20
seb128willcooke, how does that look?09:29
* willcooke reads09:29
willcookeLaney, is the start up disk creator something you'd still like to work on next cycle?09:30
willcookeseb128,  excellent, thank you09:30
seb128willcooke, yw!09:30
Laneylarsu: https://code.launchpad.net/~laney/whoopsie/source/+merge/23853909:39
larsuLaney: thanks! approved09:43
larsu(can't top-approve)09:43
Laneycheers09:45
* Laney uploads09:45
Laneywillcooke: oops, forgot to reply, yes I think so11:27
willcookeLaney, great - I'll include it in the plans we produce next week, but if I forget - poke me.11:31
larsuoops, linked the wrong bug11:50
=== MacSlow is now known as MacSlow|lunch
=== alan_g is now known as alan_g|lunch
=== tkamppeter_ is now known as tkamppeter
Laneyw00t, new unity12:19
seb128speaking of unity and keybindings12:22
seb128is ctrl-alt-L still working for others?12:22
Laneyit changed to super-l for me for some reason12:22
seb128both are supposed to work no?12:22
Laneyare they?12:23
seb128I though we said it was not acceptable to stop supporting the legacy combo and had both working12:23
seb128bregma, Trevinho ^?12:23
seb128Laney, https://launchpad.net/ubuntu/+source/unity/7.1.2+14.04.20140214.1-0ubuntu112:24
Laneyseems weird to have a random keybinding not exposed in the ui12:24
seb128"  * Adds Super+L to lock the screen, while keeping the older shortcut12:24
seb128    around in g-s-d (Ctrl+Alt+L). (LP: #830709)"12:24
ubot5Launchpad bug 830709 in gnome-settings-daemon "Keyboard shortcut - Unity should also use Super-L to lock screen by default" [Medium,New] https://launchpad.net/bugs/83070912:24
seb128or did g-s-d change the default in the 3.12 update?12:24
Laneycould be12:24
larsuctrl+alt+l still works for me12:25
bregmawasn't a Unity change that removed that key sequence, it was working for me not too long ago12:25
bregmadoesn't work for me now12:25
seb128-      <default>'&lt;Control&gt;&lt;Alt&gt;l'</default>12:25
seb128+      <default>'&lt;Super&gt;l'</default>12:25
seb128indeed12:25
seb128http://launchpadlibrarian.net/186130738/gnome-settings-daemon_3.8.6.1-0ubuntu15_3.12.2-1ubuntu1.diff.gz12:25
seb128I guess we should add an override to ubuntu-settings12:25
* seb128 does that12:25
larsuseb128: why does it wotrk for me then?12:25
seb128Laney, ^ sounds ok to you?12:25
Laney-      <default>'&lt;Control&gt;&lt;Alt&gt;l'</default>12:26
Laney+      <default>'&lt;Super&gt;l'</default>12:26
Laneyoh yeah12:26
Laneyyou already pasted that12:26
seb128larsu, because you went to g-c-c since you installed and did ctrl-alt-l in there one day12:26
seb128larsu, and it's in your user settings since*12:26
Laneythat's probably alright I suppose12:26
larsucan't remember, but I'm getting older, so might be12:26
seb128larsu, well, it's enough that you might have tested a bug to see if the keybinding change was working or something12:26
seb128larsu, gsettings get org.gnome.settings-daemon.plugins.media-keys screensaver & reset & get12:27
larsuseb128: indeed, it's set to primary+alt+l12:27
* larsu doesn't reset. muahah12:27
seb128lol12:28
LaneyI like super-l better12:28
Laneyin your FACE ctrl alt12:28
larsuLaney: I actually like super+l better too. In theory. Good luck explaining that to my fingers12:29
Laneyreset the key :)12:30
larsuI'll try. let's see how that goes12:32
larsuprobably results in me losing my private key in DC12:32
seb128yeah, I had a couple of "screen didn't lock" incidents12:33
seb128Laney, want to cherrypick https://bug738625.bugzilla-attachments.gnome.org/attachment.cgi?id=288667 on top of your evince upload? that's 138189712:50
=== MacSlow|lunch is now known as MacSlow
=== alan_g|lunch is now known as alan_g
TrevinhoLaney, seb128 they should both work...13:05
seb128Trevinho, hey, right, we figured out the issue, it's the g-s-d update, it relies on g-s-d to catch ctrl-alt-l while compiz does super-l13:06
Trevinhoseb128: yes, I've just read that as well13:06
Trevinhoseb128: in fact we just rely on that for ctrl+alt+l13:06
Trevinhowhile unity does handle super+l with his key settings13:06
Laneyseb128: after it gets review13:36
seb128Laney, not trusting larsu?!13:37
seb128;-)13:37
larsuright on, I don't trust myself either13:37
Laneysystem("keylogger");13:37
larsucan we get a new epiphany next cycle?14:05
seb128larsu, does it use gtkheaderbar?14:06
seb128can we easily add back normal decoration without having a weird UI?14:06
larsuit would be weird in that case I guess14:07
larsubecause they have the title / locationbar switcher thing14:07
Laneyit's not default, don't see why we would block there14:07
larsuya, and maybe next cycle we'll be able to actually resize windows with csds14:08
seb128Laney, do you know why it was not updated this cycle?14:09
seb128but yeah, I think it's fine updating14:09
seb128I doubt many people use !firefox|chrom* anyway14:09
Laneyno idea14:09
seb128those who do can deal with csd weirdnesses14:09
larsu:)14:09
larsuI just want to make fastmail a webapp14:10
larsuand the current one doesn't seem to work at all anymore14:10
larsusluggish scrolling14:10
Laneyand indeed we should make csd work properly14:10
seb128how are webapp handled?14:10
seb128it's a standalone browser instance without chrome?14:10
larsuLaney: still waiting on a compiz fix...14:10
Laneyv-cycle for real guys!14:11
larsuseb128: yes, exactly. It puts a .desktop file into you .local14:11
happyaronLaney: NUDT guys spend most of there energy on a big bid for the gov in earlier this cycle, so quite some updates flows in this week.14:12
happyaron* there -> their14:12
Laneyhappyaron: ho hum14:19
LaneyIt means that this "new UI" gets almost no testing before the release14:19
happyaronLaney: the people in charge comes: @JackYu14:20
GunnarHjHi pitti14:20
Laneyhappyaron: It's ok, I just wanted to highlight it14:20
happyaronic14:20
Laneythanks for replying14:20
happyaronnp14:20
happyaronlocal smoke test shows no problem, but could be problems hidden there.14:21
JackYuLaney, happyaron, I'm back.14:21
Laneyhi JackYu, I approved the FFe now14:23
LaneyI wanted to draw some attention to the lateness14:23
JackYuLaney, thanks. I answered your comments just now.14:24
Laneynow you only have a week to fix bugs14:24
Laneyscary :)14:24
LaneyI like the new UI more, by the way :P14:25
JackYuyes^14:25
JackYuthanks, lol14:25
happyaronLaney: and maybe a rush for PPU rights, ubuntu-kylin-sso-client.. (sry for pinging again :P)14:26
Laneybad man14:27
Laneyhappyaron: I want to try an experiment first14:35
Laney(generating the ubuntukylin packageset from seeds)14:35
happyaronI'm fine, :)14:36
Laneyhappyaron: http://paste.ubuntu.com/8574321/14:50
LaneyI think maybe I should override chromium ...14:51
seb128Laney, wdyt of http://paste.ubuntu.com/8574355/ ?14:58
seb128Laney, sorry, http://paste.ubuntu.com/8574357/14:58
LaneyLIst?15:00
seb128Laney, typo, fixed ;-)15:00
LaneyI think you shouldn't repeat the logic15:01
seb128and otherwise?15:02
Laneyand use QDir::absoluteFileName to get the path instead of concatenating15:02
Laneyyou can tell entryList to give you sorted lists too, which should be better for the final sort15:04
seb128Laney, how does that work?15:04
willcookesuccess - slides are finished15:04
willcookewith the exception of didrocks15:04
willcookewho will have a nice surprise on Monday morning15:04
LaneysoundsDir.absoluteFileName(dirList[i])15:04
Laneyor so15:04
seb128hum15:05
seb128doesn't that take a name as argument?15:05
seb128can you give it a path?15:05
seb128I need to prepend the /custom15:05
seb128not add a filename at the end15:05
Laney?15:05
Laneyyou list the contents of two directories15:05
seb128yeah, but "absoluteFileName" is about a filename15:06
seb128as the name indicate15:06
seb128not about a dir15:06
seb128it works if you do "dir.absolutefilename("entry")"15:06
Laneydon't understand15:06
seb128not sure about "/some/other/dir"15:06
Laneywhat does line 17 do?15:06
seb128oh ok15:07
seb128I though you were speaking about15:07
seb128        QDir customSoundsDir("/custom"+dirString);15:07
seb128sorry15:07
happyaronLaney: what about packages like youker-assistant?15:07
Laneyoh that's QDir customSoundsDir("/custom").absoluteFilePath(dirString), no?15:08
Laneypretty sure that works15:08
Laneyhappyaron: already in there15:08
happyaronyour list is only additional to the existing right?15:10
Laneyyep15:10
happyaroncool15:10
seb128Laney, with dirString = /usr/share/sounds ?15:11
Laneyyeah I think you can do that15:11
seb128that function is weirdly named15:11
Laneybut try it15:11
=== mhall119 is now known as mhall119|vacatio
GunnarHjhappyaron: Any opinion on bug #1342677?15:23
ubot5bug 1342677 in im-config (Ubuntu) "Does im-config need to be displayed in menus?" [Undecided,New] https://launchpad.net/bugs/134267715:23
=== gatox is now known as gatox_lunch
happyaronGunnarHj: I don't think the menu is needed either15:25
happyaronGunnarHj: at least for all derivatives that is using language-selector15:25
GunnarHjhappyaron: Right. Kubuntu would be a special case.15:26
GunnarHjhappyaron: And debian itself. Not very likely that they would start using l-s.15:27
happyaronI think Osamu is more interesting to something like im-chooser? not sure15:27
GunnarHjhappyaron: What's im-chooser?15:28
happyaroncurrent IM start-up process isn't all ideal. it doesn't play well with event-driven desktops15:28
happyaronGunnarHj: the Fedora approach15:29
GunnarHjhappyaron: Well, that's about the overall design of im-config...15:29
happyaronit's already rewritten once from the old school im-switch15:30
happyaronmuch better than before15:30
GunnarHjhappyaron: Yeay, you advocated its implementation at the time. :)15:31
happyaronit works as designed at least, the problem for event-driven desktops are because of design. the problem in im-switch are more about the implementation.15:32
GunnarHjhappyaron: Ok. To be honest, the expression "event-driven" is over my head.15:37
happyaronsort of upstart/systemd user session, plus some more dbus activation15:42
happyaronseb128: am I correct? ^^^^^15:42
seb128happyaron, about what? sorry, doing several things at the same time15:42
happyaronseb128: event-driven desktop15:44
seb128I don't know about the im-config specifics15:44
seb128but you should be able to get whatever you need to work15:44
seb128it's just a matter of finding the right events15:45
happyaronGunnarHj is trying to see what does event-driven desktop means/works15:46
willcookeohnoez - didrock's site is down again, and Cloudflair isn't doing its thing15:46
GunnarHjhappyaron: But we have had either upstart or systemd all the time, haven't we?15:47
happyaronGunnarHj: use the same way of booting system services to manage desktop sessions15:48
happyaronGunnarHj: so effectively upstart/systemd is managing it15:49
GunnarHjhappyaron: I know we have /usr/share/upstart/sessions/im-config.conf in Ubuntu.15:51
happyaronbut for setting XIM, it's still required to hook into the xsession script sequence15:52
happyaronand another problem is we aren't to change the variables on the fly, while im-chooser provides some degree of support (but it's not using systemd)15:53
GunnarHjhappyaron: Changing env. variables on the fly is a tricky one for all variables, isn't it?15:55
happyaronyes I agree, but unfortunately it's that way for ages15:55
happyaronbetter to get it changed in Mir/Unity815:56
GunnarHjOk.15:56
seb128Laney, http://paste.ubuntu.com/8574622/16:02
seb128Laney, without the qWarning of course16:02
seb128oh, and moving the "    QStringList soundsList;" in the if16:03
Laneyseb128: line 12 dirlist[i]?16:09
seb128Laney, oh, yeah, thanks16:11
seb128soundsDir[i] seems to work16:12
seb128but it wouldn't filter things out16:12
seb128Laney, otherwise looks ok to you?16:13
Laneyhmm16:13
Laneyoperator[] is supposed to work16:13
Laneyi.e. not having dirList at all16:13
seb128Laney, it does16:13
seb128Laney, but as said, it wouldn't filter things out16:13
seb128        soundsDir.setFilter(QDir::Files | QDir::NoSymLinks);16:13
seb128it would list subdirs16:13
Laneyit's supposed to be the same as using entryList though16:14
seb128or symlinks16:14
seb128oh16:14
seb128with the filter?16:14
seb128let me try16:14
LaneyI suppose if you call it after setting the filter16:14
Laneyhttp://qt-project.org/doc/qt-5/qdir.html#operator-5b-5d16:14
Laneyand count() instead of size()16:14
seb128Laney, .size() is what is used in the http://qt-project.org/doc/qt-4.8/qstringlist.html "Iterating over the strings" example16:15
seb128Laney, but I can change it16:15
Laneyno no16:15
Laneycount is on the QDir16:16
seb128oh16:16
seb128right16:16
Laneywell it's supposed to work16:16
seb128but it's an uint16:18
seb128let's make i one as well!16:19
Laneyin line 31 I would use QDir("/custom").absoluteFilePath(dirString)16:19
Laneymight be me being OCD16:19
seb128yeah, I don't know why you insist so much on .absoluteFilePath ;-)16:21
seb128it's like doing an extra type conversion to use a QDir16:21
seb128and call a method on it then16:21
seb128where you can concatenate 2 strings for the same result16:21
seb128that code is using accountsservice, it's not like it was going to run on win32 tomorrow :-)16:22
=== gatox_lunch is now known as gatox
* Laney twitches16:24
seb128that code is using accountsservice, it's not like it was going to run on win32 tomorrow :-)16:27
seb128ups16:27
seb128that change leads to duplicate items16:27
Laneywhich?16:28
seb128<Laney> in line 31 I would use QDir("/custom").absoluteFilePath(dirString)16:28
seb128        qWarning() << QDir("/custom").absoluteFilePath(dirString);16:29
seb1282014-10-16 18:28:39,587 - WARNING - "/usr/share/sounds/ubuntu/ringtones/"16:29
seb128        qWarning() << QDir("/custom").absoluteFilePath("/usr/share");16:31
seb1282014-10-16 18:30:58,180 - WARNING - "/usr/share"16:31
seb128 16:31
LaneyI just read something that ruins my argument16:31
LaneyQt uses "/" as a universal directory separator in the same way that "/" is used as a path separator in URLs. If you always use "/" as a directory separator, Qt will translate your paths to conform to the underlying operating system.16:31
seb128Laney, in any case, seems like .absoluteFilePath doesn't like directories arguments16:31
Laneyisn't it that /custom doesn't exist?16:32
Laneytry with /etc16:32
seb128it does exist16:32
seb128I've it with some custom sounds in it16:32
seb128to make sure they are listed and working16:32
Laneyoh right16:32
Laneyremove the leading /16:32
seb128"custom"?16:32
Laneythat makes sense16:32
Laneyno16:32
Laneyusr/share16:32
seb128oh16:32
seb128I'm not doing type version and then replacement, you are going to have to deal with string concat there :p16:33
seb128type conversion*16:34
LaneyI would probably give up at this point16:34
Laneyput a space around the + at least? :P16:35
seb128:-)16:35
Laneythis is like how I get annoyed if drawers aren't shut completely16:35
Laneyor if the curtain isn't fully closed16:36
Laneyorrrrrrr if the CDs aren't lined up on the shelf16:36
* Laney argh16:36
ogra_shudder !16:36
Laneyogra_: imagine the duvet isn't the right way up16:36
Laneyso the buttons are AT THE TOP16:36
* Laney screams16:36
ogra_argh !!!16:36
ogra_i so feel with you !16:36
seb128Laney, kenvandine, https://code.launchpad.net/~seb128/ubuntu-system-settings/sound-custom-ringtones/+merge/23859516:37
kenvandineseb128, i'll look16:38
seb128kenvandine, thanks16:38
Laneyseb128: 'oem customization'16:38
seb128kenvandine, it's sort of important, they added a custom ringtone/sms sound in /custom and the UI was not handling those dirs, so wrong selection and no way to pick back the default sounds if you try something else16:38
* Laney coughs16:38
seb128Laney, that comment was on the pastebin, feel free to tell me how to change it ;-)16:39
Laneyyou wrote eom16:39
ogra_that still has all three letters though16:40
seb128oh16:40
kenvandineseb128, so critical, but not before the image is promoted right?16:41
Laneyit feels weird to have /custom hardcoded like that16:41
Laneybut looks fine16:41
seb128kenvandine, guess so16:42
seb128Laney, well, it's our defined scheme, I don't think it's going to change16:43
LaneyI mean inside the function16:43
Laneya function called listSounds taking a directory which gives you sounds from another directory16:43
Laneybit weird16:43
Laneybut it's okay I think16:44
seb128well, it's a directory and its "overlay" in some way16:44
seb128implementation detail16:44
seb128it's looking for the same dir under /custom16:45
seb128but yeah, a bit hackish16:45
Laneythe alternative would be to make it take a list of directories16:46
Laneyoh man that's nicer16:46
seb128list of directories?16:48
seb128I don't think it makes a difference in practice16:48
Laneynot claiming it does16:49
seb128I would sort of like change to look the gtk mir backend issue now16:49
seb128rather than keep refactoring the settings changeset16:49
LaneyI already said it's okay16:49
seb128yeah16:50
seb128feel free to comment on the mp ;-)16:50
=== alan_g is now known as alan_g|EOD
=== dpm is now known as dpm-afk
Laneybye!17:19
seb128have a good evening!17:20
willcookeg/night17:37
SonikkuAmericaWhy is there no WPA2 Enterprise support?20:36
SonikkuAmerica(in the GUI)20:36
SonikkuAmerica(on Unity 8, of course)20:36

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