/srv/irclogs.ubuntu.com/2018/04/18/#ubuntu-desktop.txt

jbichaTrevinho: were you going to reorder the gnome-shell patch series or should I do that?00:04
jbichathere is a gnome-3-28 branch in upstream gnome now (HEAD points to 3.28.1)00:05
Trevinhojbicha: you mean for ubuntu or debian?00:05
jbichareordering for Ubuntu. the gnome-3-28 branch is in gnome git00:08
Trevinhojbicha:  reordering how, sorry, I don't understand what's wrong00:43
jbichaTrevinho: it's not wrong, I just thought it would be helpful to sort the patches in debian/patches/series so it was easier to see there which patches are already applied in git master00:48
jbichaI thought we discussed that earlier. That's why I hadn't uploaded to bionic yet. https://irclogs.ubuntu.com/2018/04/17/%23ubuntu-desktop.html#t18:2300:49
Trevinhojbicha: ah, ok... how you want them to be separated? Want me commenting them?01:41
TrevinhoIt's already defined in Origin: tag though01:41
dufluAfternoon thumper02:34
dufluEvening Trevinho02:34
dufluAnd hi jbicha02:34
Trevinhohey duflu02:34
thumpero/ duflu02:34
thumperand Trevinho02:34
Trevinhohey thumper02:34
Trevinho:)02:34
Trevinhoduflu: woudl be the case to include also https://gitlab.gnome.org/GNOME/mutter/commit/2d80fd02e76bbe17dc52072299dda92ab88c99c0 ?02:49
dufluTrevinho, that would certainly fix our #1 crasher bug 174845002:50
ubot5`bug 1748450 in gnome-shell (Ubuntu Bionic) "gnome-shell crashed with SIGTRAP in _g_log_abort() from g_log_default_handler() from default_log_handler(message="Connection to xwayland lost") from g_logv() from g_log() from <bug 1505409>" [High,Triaged] https://launchpad.net/bugs/174845002:50
Trevinhoduflu: well, or at least would make us understand what's leading it02:50
TrevinhoI'm including it let's see02:50
dufluTrevinho, it doesn't help to solve the problem, but it will silence the huge amount of crash reports. I think it's a good small step, yes02:51
dufluAlthough some people will report bugs without debug info then... saying "my shell just vanished"02:52
dufluWe still have the problem of Xwayland failing and no crash reports for it.02:59
Trevinhothat's a thing that whoopsie should handle though03:00
dufluBut silencing those crash dialogs in 18.04 would be excellent03:00
dufluTrevinho, no Xwayland fails without crashing, and triggers a crash in gnome-shell only. It's an Xwayland issue that never leaves a core/crash file03:00
Trevinhoduflu, jbicha: https://salsa.debian.org/gnome-team/mutter/merge_requests/2/diffs03:02
ubot5-ngGNOME bug (Merge request) 2 in mutter "debian/patches: handle xwayland crashes better" (comments: 0) [Opened]03:02
* Trevinho loves salsa03:02
Trevinhocan't wait to have the same way of working with patches in ubuntu too... Well, I'm doing it already but still need to handle with bzr eventually. But still better way03:02
dufluTrevinho, I feel conflicted about letting gnome-shell just exit. It will result in some people logging bug reports that we can't diagnose, but those will be orders of magnitude fewer than what we see now04:33
dufluSo short term for 18.04, it's a good idea. Slightly beyond that, it's going to keep hurting us if nobody fixes Xwayland04:35
jibelseb128, hi06:47
jibelseb128, to answer your question from yesterday evening, it's attached xwayland of the gdm user06:48
jibel+to06:48
Trevinhoduflu: https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/73 I missed this in the g-s patches, you wanted to include it?07:05
ubot5-ngGNOME bug (Merge request) 73 in gnome-shell "js/ui: Choose some actors to cache on the GPU" (comments: 1) [Opened]07:05
dufluTrevinho, yes, if possible. I am answering your question on it now07:06
Trevinhoduflu: is there an ubuntu bug for that?07:06
dufluLemme check07:07
dufluTrevinho, yes the merge request links to two upstream bugs, and one of them links to LP too07:07
Trevinhoduflu: ok07:08
didrocksgood morning07:14
seb128good morning desktopers07:24
seb128hey duflu Trevinho jibel07:24
seb128lut didrocks07:24
Trevinhohey seb12807:25
dufluHello seb12807:26
seb128jibel, weird that it doesn't catch an error :/07:27
didrockshey seb128, Trevinho, duflu07:27
Trevinhohi didrocks07:28
dufluHi didrocks07:28
Trevinhoduflu: I've added that in https://code.launchpad.net/~3v1n0/gnome-shell/bionic-patches-picks-reorder/+merge/34347907:32
dufluTrevinho, thanks. Are we getting 3.28.1 too?07:34
Trevinhoduflu: that's based on 3.28.107:34
dufluOK07:34
kozawillcooke, hey, sadly i need to skip our morning mtg today07:51
willcookemorning koza, no worries.  I dont have anythng anyway.  duflu you want to meet anyway?07:52
didrockshey willcooke07:53
willcookehi didrocks, how goes?07:53
willcooke(he asks him knowingly)07:53
dufluwillcooke, Nope.07:54
seb128hey willcooke koza, nothing from me either on the bluetooth front07:54
dufluI'll do an update post-release07:54
didrockswillcooke: lalala :)07:55
dufluIt might be a single bugfix, or it might be 5.4907:55
* didrocks does good use of his time, grepping for copyrights and filing up debian/copyright (which will be out of date in few weeks, like all packages in distro :p)07:55
oSoMoNgood morning desktoppers07:55
didrockshey oSoMoN07:55
oSoMoNsalut didrocks07:55
Laneyyo08:01
oSoMoNhey Laney08:02
didrockshey Laney08:02
dufluHi Laney08:05
dufluwillcooke, Sorry, it's an awkward time of day. Either like today, the question is asked when I am afk doing pre-meeting prep, or like other weeks we decide to cancel only minutes before you log on08:06
* didrocks uploads to a ppa for test build in a clean env08:08
seb128 hey Laney oSoMoN08:13
Laneyhey oSoMoN didrocks duflu seb128!!!08:13
oSoMoNsalut seb12808:13
didrocksseb128: willcooke: ok, vendored ubuntu-report uploaded and waiting now in UNAPPROVED. (I still did a little extra to use distro deps for the golang source bindings which stays in universe). More info on bug #1764961 and https://bugs.launchpad.net/ubuntu/+source/ubuntu-report/+bug/1759540/comments/10 (commented on the MIR)08:16
ubot5`Ubuntu bug 1759540 in ubuntu-report (Ubuntu) "[MIR] ubuntu-report: send telemetry data to ubuntu server" [Undecided,Incomplete]08:16
ubot5`bug 1764961 in ubuntu-report (Ubuntu) "Use vendor dependencies" [Undecided,New] https://launchpad.net/bugs/176496108:16
willcooke\m/08:16
seb128didrocks, great, thanks!08:18
didrocksyw :)08:19
seb128didrocks, "Note thus that "golang-github-ubuntu-ubuntu-report-dev" should still be kept in main."08:25
seb128didrocks, you meant "universe" there?08:25
didrocksoupsss, where?08:26
didrocksI wrote that 3 times ;)08:26
didrocksok, not the ones in the package and comments08:27
didrocksah, in the MIR comment08:27
didrockscommented, thanks seb128!08:28
seb128np!08:28
dufluOh, I did get credited in the gstreamer-vaapi git log even though they didn't use my original patch. That's nice.08:47
* duflu relearns from his own code08:47
seb128:/08:47
didrocksseb128: gnome-initial-setup wasn't promoted? Should I? (as ubuntu-report is still in main)08:51
didrocksat least, even if the MIR team doesn't get back to us on the final changes for u-r, we'll have g-i-s tested in tomorrow's iso08:51
seb128didrocks, wfm09:07
seb128thx09:07
didrocksok, doing it09:08
didrocksyw ;)09:08
Laneyandyrock: those xdg autostart files - don't we need to stop the path units when the user logs out?10:40
Laneyor otherwise they could be started in default.target and the corresponding units made to cleanly exit if they can't run?10:48
* duflu wanders into the night11:03
andyrockLaney: mmm how?11:13
andyrockLaney: I mean how to stop when the user logs out?11:14
LaneyI don't know :(11:14
andyrockmmm let me take a look11:14
Laneywe get that with graphical-session.target, but I'm not sure how to do it without that11:14
Laneyapart from something watching the XDG session or gnome-session11:25
jbichaTrevinho: you're not still around now, right?11:30
Wimpresstseliot: It appears that Ubuntu flavours (MATE, Budgie, Kubuntu, Xubuntu have been tested) all boot to a black screen on hybrid graphics laptops with nvidia 390 drivers installed on 18.04.11:32
WimpressThis doesn't affect Ubuntu proper with GNOME.11:33
tseliotWimpress: is the nvidia-prime package installed?11:33
WimpressWhat information do you required to investigate this?11:33
WimpressOn Ubuntu MATE yes.11:33
WimpressI have an OEM with 2 laptop models that are affected.11:34
WimpressSo we can gather useful information.11:34
WimpressWhat do you need.11:34
tseliotWimpress: I'm going to need /var/log/gpu-manager.log, /var/log/Xorg.0.log, dmesg output11:34
WimpressOK.11:34
WimpressI'll raise a new bug a gather some debug.11:35
tseliotWimpress: thanks11:36
Wimpresstseliot: Let me introduce mwilson-e11:45
WimpressHe runs Entroware, an OEM that sells Ubuntu laptops.11:46
Wimpressmwilson-e: Is going to collect the debug you've requested and attach to this bug https://bugs.launchpad.net/ubuntu/+source/nvidia-graphics-drivers-390/+bug/176400511:47
ubot5`Ubuntu bug 1764005 in nvidia-prime (Ubuntu) "Black-screen on boot with nvidia 390 for Budgie, MATE, Xubuntu, Kubuntu" [Undecided,Confirmed]11:47
tseliotWimpress: ok11:47
Wimpresstseliot: Mike has two laptop models to test against so if you need more info he can provide it.11:48
WimpressI'll do the same for the XPS 15 9550.11:48
tseliotWimpress: that's good. I'll wait for mwilson-e to attach the logs for now11:48
mwilson-eNice to meet you. Working on gathering all the logs now.11:49
WimpressSure, will be a little while since he is going to be thorough :-)11:49
andyrockLaney: stopping the services on logout does not seems easy11:56
andyrockseb128: ^^^11:57
andyrockLaney: seb128 I would drop the upload of update-notifier for the moment, I'll try to find a way to show livepatch notification without this11:57
Laneymaybe it doesn't matter that much???11:57
Laneywhat's the impact - ugly failures?11:58
Laneyandyrock: sorry for not thinking of that before12:01
andyrockLaney: so all the other services are not effected because there is a GFileMonitor fallback12:01
andyrockwe can do the same12:02
Laneythere's a service running all the time?12:02
andyrockyeah12:02
andyrockupdate-notifier12:03
Laneyoh right12:03
Laneywell then that could watch logind and stop the path units too12:03
Laneybut just doing the file monitor directly might be simpler12:04
Laneyandyrock: ok, I'll reject, ping me for a re-review if you want12:06
andyrockLaney: thx12:06
seb128andyrock, k12:11
Wimpresstseliot: See https://pad.lv/1764005 - I've added logs and details I what I experience on the XPS 15 9550.12:28
ubot5`Launchpad bug 1764005 in nvidia-prime (Ubuntu) "Black-screen on boot with nvidia 390 for Budgie, MATE, Xubuntu, Kubuntu" [Undecided,Confirmed]12:28
willcookegedit fans - is the right click menu using a strange font for you?13:06
willcookelooks like its monospace condensed or something13:06
mdeslaurwillcooke: mine is on artful13:06
willcookemdeslaur, is funny?13:06
willcookeyeah,13:07
mdeslaurwillcooke: yeah, like you said, it's a small monospace13:07
willcookethanks mdeslaur13:07
willcookeTrevinho, any ideas on that one?13:07
willcookeCan't belive I haven't noticed it before13:07
Wimpresswillcooke: That content menu font issue is an upstream bug.13:16
WimpressAffects all gtk3 editors with context menus.13:17
willcookeah ha! thanks Wimpress13:17
willcookeWimpress, this doesn't change my mind about gedit though, before you start on that one ;)13:17
Wimpresstseliot: You'll find more debug from mwilson-e in https://pad.lv/176400513:17
ubot5`Launchpad bug 1764005 in nvidia-prime (Ubuntu) "Black-screen on boot with nvidia 390 for Budgie, MATE, Xubuntu, Kubuntu" [Undecided,Confirmed]13:17
tseliotWimpress: that's a lot of logs, thanks!13:29
WimpressMost welcome :-)13:29
Wimpresstseliot: I've also just discovered `ubuntu-drivers autoinstall` fails.13:30
WimpressI'll raise a bug for that too.13:30
tseliotWimpress: that would be very welcome. Thanks13:31
mwilson-etseliot: If you need any others, just let me know.13:31
andyrockseb128: I guess we need to add dh-migrations as a dep in software-properties13:35
andyrock?13:35
andyrockotherwise we can just drop the migration script13:38
tseliotmwilson-e: maybe the output of "journalctl --since=today" would help, as I don't see anything suspicious in the logs13:39
mwilson-etseliot: I'll get that now, any particular distro?13:40
tseliotmwilson-e: just one that fails would be fine. I'm looking for clues13:41
didrocksandyrock: man dh_migrations ;)13:42
didrocksit tells you what to do in term of deps, files and debian/rules ;)13:42
didrocks(waow 2012… 6 years ago)13:43
jibelseb128, I just got a gnome-shell crash with gdb attached to xwayland. What do you need now?13:45
Wimpresstseliot: https://pad.lv/176505313:47
ubot5`Launchpad bug 1765053 in ubuntu-drivers-common (Ubuntu) "ubuntu-drivers autoinstall fails to install nvidia 390 drivers" [Undecided,New]13:47
mwilson-etseliot: Just posted the requested log to the bug report.13:47
tseliotWimpress, mwilson-e: thanks13:50
seb128andyrock, indeed, I'm fixing that13:58
andyrockseb128: thx!13:58
seb128jibel, what is gdb telling you, did it stop?13:58
seb128andyrock, np, sorry for not spotting that before upload13:59
jibelseb128, nothing, it just stopped. I dumped a core of the process just in case13:59
seb128jibel, can you make a photo? ;)13:59
seb128like it stopped giving you back a prompt where you can type "bt"?13:59
jibelyes14:00
jibelwhich I did https://paste.ubuntu.com/p/wHWtZsKwts/14:00
jibelnot really exciting trace14:00
seb128hum14:01
seb128did it indicate a sigsegv before?14:01
seb128the bt looks like it's active/waiting for events14:01
seb128not that it hit a bug14:01
jibelno, really nothing. It just stopped and gave me a prompt14:01
seb128weird14:02
seb128and if you do "c" it resumes?14:02
jibelyes14:02
jibelI'll keep it attached and try again14:02
seb128and you got the gnome-shell issue as well?14:02
jibelyes, g-s crashed and apport generated a crash file14:03
jibelit's uploading14:04
seb128k, let's see what it gives us14:10
=== pstolowski is now known as pstolowski|lunch
WimpressTrevinho: Please can you take a look at this - https://bugs.launchpad.net/ubuntu/+source/indicator-sound/+bug/173783414:37
ubot5`Ubuntu bug 1737834 in indicator-sound (Ubuntu) "[FTBFS] indicator-sound fails to build, test failures" [High,Confirmed]14:37
WimpressIs that merge proposal OK with you?14:38
mwilson-etseliot: Not sure if it helps you but if I install Ubuntu MATE BETA 2, not the daily, and install the nvidia driver it works fine. But once the `ubuntu-driver-common` package is upgraded the problem then presents itself.14:39
seb128Wimpress, Trevinho, disabling tests doesn't seem the right way out no14:40
tseliotmwilson-e: what version of ubuntu-drivers-common does Ubuntu MATE BETA 2 ship with?14:40
Wimpressseb128: OK, thanks.14:40
seb128np14:40
mwilson-etseliot: It ships with 1:0.514:40
mwilson-ethe upgrade to 1:0.5.1 kills it14:41
tseliotmwilson-e: do things work if you boot the system with the "nogpumanager" parameter?14:42
mwilson-etseliot: Just tested on two machines, the nogpumanager param does not work.14:45
tseliotmwilson-e: remove /usr/share/X11/xorg.conf.d/11-nvidia-prime.conf first, then boot with "nogpumanager"14:47
mwilson-etseliot: That works, but in the nvidia-settings application many of the options are gone. only PRIME profiles, Application Profiles, and nvidia-settings Configuration tabs are displayed14:52
oSoMoNkenvandine, when you have a minute: https://code.launchpad.net/~osomon/evince/snap-allow-printing/+merge/34352114:52
oSoMoNseb128, kenvandine: thanks for the endorsement! one to go…14:54
kenvandineoSoMoN, happy too!14:55
tseliotmwilson-e: that was just to find what was causing the problem. Can you post your /var/log/Xorg.0.log please?14:55
kenvandineoSoMoN, i just reviewed it, i think we need avahi-observe too14:55
oSoMoNkenvandine, ack, adding it14:57
mwilson-etseliot: added it to the bug report.14:58
oSoMoNkenvandine, updated14:59
oSoMoNkenvandine, we should request auto-connection of the cups-control plug once this is merged and published in the store, I'm happy to create the request on the forum if you let me know when to do it15:01
seb128oSoMoN, yw, should be easy to get the 3rd one :)15:01
kenvandineoSoMoN, yeah15:01
kenvandineoSoMoN, merged and the snap is building now15:02
tseliotmwilson-e: thanks15:04
mwilson-etseliot: No problem, if you need anything else, just ask.15:05
seb128hum15:05
seb128is the trello board having background textures on most of the card now for some reason? or is it me/my browser being buggy?15:06
kenvandineseb128, i saw the same too15:06
kenvandineit looks very 1990s :)15:06
=== pstolowski|lunch is now known as pstolowski
Laneythat's one of the powerups15:07
seb128kenvandine, it looks like it's proportional to the time they haven't been updated maybe15:07
Laneycard aging15:07
kenvandinelol15:07
Laneysomeone must have put that on15:07
kenvandineyou're right15:07
kenvandinesounds like a trick from seb128 to keep us on our toes15:08
seb128lol15:08
seb128Trevinho did it15:08
seb128he activated "Card Aging"15:08
seb128Trevinho, please don't do such changes to the board without discussing them here first15:09
oSoMoNnext step: make the cards <blink> to remind us to update them :)15:11
kenvandine:-p15:11
didrocks:p15:11
seb128which remind me, some people still didn't update their inprogress card, I should do another follow up15:11
GunnarHjHi jbicha, did you see my docs idea about the screen keyboard?15:15
seb128GunnarHj, hey, I saw your comment but I don't really know what different it makes to set that IM variable to ibus or not, I agree it's late to change it though so probably best as a SRU15:16
seb128kenvandine15:16
seb128kenvandine, jamesh, jibel, you are basically the ones who still have in-progress cards in the trello board that didn't get updated15:17
seb128could you please review your cards?15:17
GunnarHjseb128: We can do that as an SRU, but it still doesn't help on Xorg AFAIU.15:17
GunnarHjseb128: So the very latest idea is to inform the users in the desktop guide about what they need to do to make it work.15:19
seb128well, or to fix it for xorg in a SRU as well15:20
GunnarHjseb128: Sure, if that is at all possible. I didn't get that impression from the comments on the upstream issue.15:20
seb128bugs are always possible to fix, might not be trivial though :/15:24
seb128the cycle has been too busy for me to look at OSK issues but we should make sure it works good enough by .115:24
GunnarHjseb128: Ok, so then I'll wait with that docs text. Plan A: Fix it, also on Xorg as an SRU and plan B: inform the users with a docs SRU.15:25
seb128GunnarHj, what are the main issues in your opinion? (I didn't follow much, capital letter and special chars can't be inputed?)15:26
GunnarHjseb128: That's it. Bad enough IMO.15:27
seb128yeah...15:27
seb128andyrock, Laney, since the update-notified upload has been rejected I uncommited the change and push --overwrite the vcs without those to not confuse the next uploader, unsure if that's the right thing to do in those case (it makes the status of the merge mp wrong/confusing)15:28
andyrockseb128: thanks15:29
andyrockI was already working on an alternate fix15:29
andyrocklet me "rebase"15:29
LaneyI usually try to not push the tag until it's accepted15:31
Laneythat is easier with git than bzr, but still can be done15:31
Laneybiggest risk is forgetting15:31
kenvandineoSoMoN, evince has built, i've tested it15:31
kenvandineoSoMoN, in stable now15:31
seb128Laney, right...15:32
oSoMoNkenvandine, excellent, thanks, I'll write the auto-connection request15:32
kenvandineoSoMoN, cool15:32
Laneyanyway, thx, probably not that much chance somebody did pull in the meantime15:32
andyrockapart from me15:32
andyrock:D15:32
seb128right15:33
seb128haha15:33
Laneypoor andyrock15:33
=== sergiusens_ is now known as sergiusens
oSoMoNkenvandine, https://forum.snapcraft.io/t/auto-connecting-the-cups-control-interface-for-the-evince-snap/503315:40
jbichaGunnarHj: I'm not really a fan of the docs idea. I agree we need to understand more what the purpose of GTK_IM_MODULE is15:45
jbichaGunnarHj: on GNOME/Ubuntu on Wayland, it's my understanding that ibus is the only supported input method so if we could skip that variable when run in those sessions I think the regression potential is limited15:46
jbichawe'll need to open a Debian bug because Debian has the same problem (ibus there recommends im-config but I think ibus isn't installed by default there)15:47
seb128jbicha, but it means =ibus or empty leads to different components to be used?15:48
seb128so it's a codepath/feature/behaviour change15:48
seb128and we didn't test much the other one, so seems a risky change that late, especially without someone in the team who understands the difference15:48
jbichaI agree with not rushing it in now while we don't fully understand what's going on and Debian hasn't made the suggested change either15:49
GunnarHjjbicha, seb128: My gut feeling is that if IBus works on GNOME without that var being set, it probably would work for us too. But still...15:50
jbichainterestingly, there is a proposed simple patch to get the screen keyboard working in the Xorg session with GNOME 3.26 (17.10) but it may be a lot more complicated with current GNOME15:50
andyrockseb128: the fix is almost ready. it's quite small15:51
andyrockjust giving a final test15:51
tseliotmwilson-e: I'm going to need the output of these two commands too, please: uname -r , dkms status15:52
seb128andyrock, great!15:53
mwilson-etseliot: 4.15.0-15-generic15:56
mwilson-etseliot: nvidia, 390.48, 4.15.0-15-generic, x86_64: installed15:56
tseliotmwilson-e: the weird thing is that, when you disabled gpu-manager, the X log showed an actual error, and the driver wasn't really working15:57
GunnarHjjbicha: I don't think either that Debian installs any IM stuff by default. So in their case, with GNOME on Wayland, the screen keyboard works until they install IBus which pulls im-config?15:58
andyrockseb128: can I drop the UIFe status from the bug?15:58
mwilson-etseliot: Do you want me to run those commands with gpumanager disabled?15:59
jbichaGunnarHj: I'd have to try again. My initial testing was broken because the screen keyboard didn't work how I expected (I filed bugs)16:00
tseliotmwilson-e: it wouldn't change the output of those commands16:00
Laneyseb128: you skipped a version in software-properties, did you want to fix that?16:00
GunnarHjjbicha: Ok.16:01
Laneyalso it's not control.in :p16:01
mwilson-etseliot: I see, are there any other logs that I can provide?16:03
andyrockseb128: https://code.launchpad.net/~azzar1/update-notifier/fix-1761841-v2/+merge/34354016:04
tseliotmwilson-e: I might need you to test packages from a PPA. Not sure if today or tomorrow16:04
kenvandineoSoMoN, thx16:04
mwilson-etseliot: no problem16:05
kenvandineoSoMoN, i'm going to request it for gedit as well16:06
* Laney just got the meaning of azzar1 for the first time16:06
andyrockazzarONE16:07
andyrock:D16:07
Laneymind is blown16:07
kenvandineoSoMoN, with a bit of copy and paste from your request :)16:07
andyrockthey usually call me azzarONE in USA16:07
andyrockat least in the airports16:07
jbichawhere does the rock come from?16:08
Laneyyou can smell what Andrea is cooking16:09
seb128andyrock, thanks, I'm getting ready for sport so I might not get to it before later this evening, but it's on my list16:10
seb128Laney, ups, can do but ^16:10
seb128if somebody wants to fix it for me please do, otherwise I can do when I'm back in the evening16:11
seb128andyrock, UIFe drop is fine I believe yes16:11
Laneyok16:11
andyrockjbicha: from a mistake :D16:11
andyrockI was young and wanted to copy didrocks16:11
andyrockbut I forgot the final 's'16:11
andyrock:D16:12
Laneyandyrock: this monitor_files function is :(16:12
oSoMoNdid_rocks is a model for the new generation!16:12
andyrockLaney: you mean the existing one or my changes?16:12
Laneythe one that was already there16:12
jbichaeveryone wants to be like didrocks16:12
andyrockI agree that update-notifiers needs some love16:13
andyrocknext-cycle-love16:13
Laneythere's some weird indirection between the file monitor cb and a timer16:14
Laneyand it leaks all over the place16:14
didrocks\o/16:24
Laneyandyrock: might want to add some error checking around the atoi thing, you get a 0 outputted if the file contains weird contents16:36
Laney(minor)16:36
tseliotmwilson-e: can you add this PPA, and when they packages build, upgrade the xserver, please? https://launchpad.net/~albertomilone/+archive/ubuntu/xtesting16:47
mwilson-etseliot: Just tested that package and it does not fix the issue.17:00
tseliotmwilson-e: did you reboot? Did you boot without "nogpumanager"?17:01
mwilson-etseliot: yes I did reboot, this was also done on a fresh install with only the nvidia drivers installed.17:02
tseliotmwilson-e: ok, I'll install one of those distros, and I'll see if I can reproduce the problem here17:03
mwilson-etseliot: Just so you know, I tested this on Ubuntu MATE daily.17:03
tseliotmwilson-e: ok, I'll go with that then, thanks17:04
=== pstolowski is now known as pstolowski|eod
Trevinhoseb128: eh, see... the aging is what you want in fact :-D17:52
TrevinhoI was trying some power-ups and I forgot to disable sorry17:52
Trevinhojbicha: about not adding that patch, both me and duflu are fine with that... Upstream wasn't keen, but in the new PRs there's no sign of that18:09
jbichaTrevinho: I guess you're getting close to being on Robert's timezone :)18:16
andyrockLaney: can you open a bug about that?18:39
andyrockif you got spare time18:39
andyrockI guess not18:39
willcookenight all18:40
oSoMoNgood night all18:58
=== mwilson-1 is now known as mwilson-e
robert_ancellandyrock, how did you make https://gitlab.gnome.org/azzaronea/gnome-initial-setup ? I didn't think we could do that until GNOME/gnome-initial-setup existed.21:04
robert_ancellI'm trying to work out how we push to the Community/Ubuntu/gnome-initial-setup so we can all commit21:04
andyrockrobert_ancell: I just created a project called gnome-initial-setup21:05
andyrockit's not a fork of the main one21:05
andyrockI mean it's a fork but it has not been created with the fork button21:05
robert_ancellah21:07
jbicharobert_ancell: the very rough concensus was that we should use a LP ubuntu-desktop git repo for that next month21:24
jbichathere are 3 logical places for hosting and we have advocates for all 3 🙁21:25
robert_ancelljbicha, for packaging or working on branches?21:26
jbichait's the same thing here isn't it?21:26
robert_ancellnot necessarily... The branches should be close to upstream as that's where we're trying to get them. The packaging doesn't go upstream, so it doesn't matter so much where it lives.21:27
jbichaok I see, maybe it makes sense for those to be separate commits and it wouldn't hurt to have that in GNOME gitlab21:28
jbichait just makes things a bit complex as we need 3 different remotes to handle gnome-initial-setup then 😐21:29
jbichaI can deal with that workflow; it's just going to be a bit of a learning curve for some on our team I suppose21:30
jbichabut I guess we almost do that with gnome-software already except 1 of the 3 is bzr21:33
Trevinhorobert_ancell: hey could you please publish https://bileto.ubuntu.com/#/ticket/3223 ?21:44
robert_ancellTrevinho, done21:55
Trevinhorobert_ancell: thanks21:55
Trevinhorobert_ancell: if you've some time also https://code.launchpad.net/~3v1n0/gnome-shell/bionic-patches-add-gpu-cached-texture/+merge/343565 would need merging22:18
Trevinho(and sponsoring)22:18
robert_ancellTrevinho, not sure if I'll find time to look at that today and I'm on holiday from tomorrow22:19
Trevinhorobert_ancell: ah ok, no worries... In case I'll ask other dev :)22:19
jbicharobert_ancell: sorry to second guess you, but I'm surprised you removed the gnome-getting-started-docs popup part at the end of gnome-initial-setup22:21
robert_ancelljbicha, design asked to remove it22:21
jbichaok22:21
jbicharobert_ancell: on the Ready to go screen (software) panel in your proposed ubuntu4, I see back and forward arrows but they aren't active here22:34
jbichait was working in the ubuntu3 version22:34
robert_ancelljbicha, what do you mean "aren't active"?22:39
jbichathey are greyed out: I only see one page worth of snap apps on that panel22:45
robert_ancelljbicha, oh, right. Yes, there's more shown on a page, so they all fit into one page. I haven't asked design what we should do, we might as well hide them I guess.22:51
robert_ancelljbicha, I updated the git branch to disable them when there's one page22:54
robert_ancellandyrock, I moved the branch to https://gitlab.gnome.org/Community/Ubuntu/gnome-initial-setup so feel free to merge in your GOA changes when you're happy with them.22:58
jbicharobert_ancell: hey, another nitpick. In the Google Doc it says "approximate location you chose" but in Github, it says "approximate location you choose" (note final word)23:17
jbichaI like the "chose" tense better, what do you think?23:17
robert_ancelljbicha, I changed it to match https://github.com/CanonicalLtd/desktop-design/issues/66#issuecomment-37726198623:18
ubot5-ngCanonicalLtd bug 66 in desktop-design "Visual design for first-login window" (comments: 19) [Open] - Assigned to lyubomir-popov (lyubomir)23:18
ubot5`Error: Could not gather data from Launchpad for bug #66 (https://launchpad.net/bugs/66). The error has been logged23:18
robert_ancellI thought chose sounded better too, but file it with the designers23:18
jbichaI guess I'll open a new issue there and they can complain at me if they want all this stuff in one issue instead23:19

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