/srv/irclogs.ubuntu.com/2016/03/09/#ubuntu-desktop.txt

hikikohi06:03
didrocksgood morning!07:16
seb128good morning desktopers08:56
willcookemorning seb12808:56
seb128hey willcooke08:56
willcookeseb128, just reading Barry's email08:57
seb128k08:57
willcookefeels like 16.10 is the right time to me08:57
willcookeLTS is more likely to be used in places where SMB printing is used08:58
seb128well, printing is fine08:58
seb128the current issue is that we are talking about smb drives browsing08:58
seb128and smb disk mounts08:58
seb128like being able to connect to that corporate z: and open some word files from it08:59
willcookeoh.  When did it change from being about printing to file shares?? /me rereads the whole thread08:59
willcookein which case I think we 100% need SMB support out of the box08:59
willcookethe pop-up is "ok", but that change is for 16.10 IMO09:00
seb128I'm unsure you can "pop-up"09:00
seb128nautilus lists nearby servers09:00
seb128you can't do that job if the smb backend is not installed09:00
willcookehumm.  I've a feeling in 12.04(???) that when you used nautilus and clicked on "Windows networks" is would prompt09:01
willcookeI don't think that prompt actually worked, but I think it did prompt09:01
willcookemeh09:03
willcookeeither way09:03
willcookeI've read the thread now09:03
Laneyyo09:03
willcookeI vote to keep it and work on porting next cycle09:03
willcookemorning Laney09:03
seb128hey Laney09:04
seb128Laney, how are you?09:06
Laneyhey willcooke seb12809:08
LaneyI'm alright09:08
Laneyno prize on the pub quiz /o\ but we did get the bonus question09:09
Laneywhat about you?09:09
seb128I'm good thanks :-)09:14
seb128though it's one of those days where I already have a stack of emails/IRCs nags to start the day09:15
seb128which is a bit annoying, but oh well09:15
Laneyall useful things right?09:16
seb128of course :p09:16
Laneymaking the world a better place!09:16
didrockshey seb128, Laney, willcooke09:53
willcookewhat up didrocks09:53
seb128hey didrocks! ça va ?09:53
didrocksseb128: I think willcooke is referencing the "please install samba" prompt09:53
didrocks(having fixed it multiple times brings back "old good times" memories btw ;))09:54
seb128that is/was when trying to share something in nautilus no?09:54
didrockshum, not at a client as well?09:56
didrocksI don't remember, too long ao…09:56
didrocksago*09:56
seb128same here ;-)09:56
didrocksseb128: ça va sinon :) and you ?09:56
seb128ça va bien !09:56
alexarnaudhello world!10:02
seb128hey alexarnaud10:08
hikikohey desktopers10:36
willcookemorning hikiko10:37
hikikohi willcooke :)10:37
davmor2jibel, mvo, seb128, willcooke: I hope you are all sitting down 14.04.4 → 16.04 got passed step 2 not to see if it completes or chokes on something else \o/11:18
willcooke:)11:18
willcookeprogress11:18
mvodavmor2: yay11:20
seb128davmor2, \o/11:41
davmor2Yeap dies somewhere else now so badly it takes out tty I assume that isn't a good thing :(  So now I need to transfer over my base image and try again /me thanks god for vms11:42
desrtmorning, peeps11:49
seb128good morning desrt11:49
desrtquiet morning -)12:16
desrtwhat's up, seb?12:16
=== flocculant_ is now known as flocculant
GunnarHjseb128: Hi Sebastien, would the proposed changes to u-c-c at https://launchpad.net/~gunnarhj/+archive/ubuntu/im-config-gnome/+packages be a problem somehow?12:28
seb128GunnarHj, I don't know, I'm going to have a look in the afternoon12:32
seb128desrt, sorry I didn't see you message, fighting with nautilus and gtk behaviour changes, I start wondering if we should go back to 3.18...12:33
LaneyHAHA12:33
GunnarHjseb128: Ok, let's talk later on. (It's only about a variable name change throughout the package.)12:34
ximionLaney: I asked around, apparently nobody supports separating .desktop files, binaries and metainfo and still have the result show up in a software center12:36
ximionasked hughsie and he said he doesn't do that because it would complicate Fedora's metadata generator and massively hurt performance12:37
ximionso, while "the others do it" is not an argument, it is an interesting data point on whether we should fix packages or adjust the generator12:37
Laneyhey ximion12:37
LaneyI don't really mind if someone is going to fix the packages :)12:38
seb128Laney, :p12:38
ximionLaney: :D12:38
ogra_just use snappy, it makes everything so much easier (ask Sweet5hark1 *g* )12:39
ximionwell, not showing up in the SCs seems to be a strong incentive for people to do the work ^^12:39
seb128Laney, ximion, do we have datas on the number of packages we are speaking about?12:39
Laneynope12:39
seb128ogra_, people hosting those are going to like having so much space used and no data shared between archs ;-)12:39
ximionseb128: not yet, that's one of the things we need to do, but I would be very surprised if it's more than 20 packages12:39
ogra_seb128, diskspace is cheap :P12:40
ogra_(somewhere)12:40
Laneyximion: it does need to become an error though at least12:40
Laneythen we get to re-process everything, woohoo12:40
=== hikiko is now known as hikiko|ln
seb128ximion, we have at least 3 in the default installations that I hit this week, I wouldn't be surprised if it's not trivial12:41
ximionLaney: jup, meaning we need to read the Exec line in the .desktop file and check if the binary is there - if not, emit an error-type tag stating that12:41
seb128debian likes to split things in common12:41
ximionseb128: yes, and -common does make sense, but separating the .desktop file and relying on TryExec is IMHO a really bad practice (while technically possible, though) - separating the metainfo file from the .desktop file is also evil (since that one should be in the same pkg that gets installed to satisfy the component)12:42
seb128ximion, right, I'm not arguing against that, just saying that it was not documented and quite some maintainers put the desktop in -common since it's arch independant12:43
seb128ogra_, you should host a business is server storage ;-)12:43
ogra_seb128, now that you say that ...12:44
andyrockhey all12:44
seb128:-)12:45
seb128hey andyrock12:45
seb128ok, enough poking at nautilus, it's nice outside and I need to move a bit12:45
ximionseb128: at least for .desktop files without TryExec, that's a definitive error - I need to check if lintian tests for that already12:46
seb128going for some exercice, bbiab12:46
seb128ximion, right12:46
seb128bbl12:46
ximionfor everything else, this change would need to be communicated properly12:46
ximionintegrating AppStream's issue hints with Debian's PTS would be ideal (nobody is working on that yet, though)12:47
Laneywe are also trying to release with this in a short number of weeks :)12:47
ximionLaney: the previous app-install-data generator also didn't do all that stuff...12:48
ximionso I wonder how that worked before :P12:48
ximionor maybe it was just run on a complete Contents file once12:48
ximionor someone did manually fix things up12:49
Laneylaney@nightingale> head -2 menu-data/evince-common:evince.desktop                                                                                                ~/temp/app-install-data-ubuntu-15.1012:49
Laney[Desktop Entry]12:49
LaneyX-AppInstall-Package=evince12:49
ximioninteresting...12:50
Laneydon't know where the code is for this though12:51
ximionLaney: somewhere at Debian's Git, I read it ages ago...12:52
ximionit operated on the Contents file though12:52
ximioncould send you a link when I'm home, I think I bookmarked it12:52
Laneynod12:53
Laneythis particular problem has to use Contents12:53
ximionLaney: btw, you definitely want to switch to the new appstream-generator for later Ubuntu releases when it's done - it massively outperforms the Python thing12:53
ximionit uses libarchive to extract the .deb files and does some useful caching, and it time it's twice as fast compared to the Python code (but it doesn't do icon processing or screenshot downloading yet)12:54
ximiondownside will be that installing it will be harder (C and D code needs to be compiled...)12:55
ximionanother thing that increases performance is that we can use LMDB in a threaded environment (as opposed to a multiprocess-environment in Python), so we only need to open the cache once12:55
ximionmakes quite a difference ^^12:56
Laneyximion: you know you could avoid Contents by building that yourself12:57
Laneyjust an idea ;-)12:57
ximionLaney: I was thinking about that ^^12:59
ximionwould help Ubuntu, since in Debian the contents file is up-to-data, usually12:59
ximionbut would also not be bad for Debian, since it would *guarantee* that Contents and package-content match13:00
ximion(something which we just have to assume at time...)13:00
Laneyximion: yeah13:16
ximionLaney: Python just wasn't the ideal choice ^^ - not sure if D is (yet), but I got stuck after trying it - parallelization works really well :)13:17
ximionbig work item will be implementing a proper YAML writer in C13:18
ximionlibyaml is13:18
ximionnot very nice ^^13:18
ximion(outputting XML already works)13:18
Laneypython is nice for having lots of libraries already available13:19
ximionthat indeed! And handling YAML data is much more natural in Python13:19
ximionit's also easy to learn or people are already familiar with it, and you can just write the logic and don't need to care much about memory safety (as in C)13:20
ximionbut multiprocessing with Python turned out to be really bad, I understand now why the dak developers are always cursing it13:21
ximionLaney: btw, sorry for not being very active the past weeks - too much work13:23
ximion(I am now replying from work, waiting for some computation to finish...)13:23
Laneyximion: no worries, I think I only have one PR outstanding atm13:25
ximionthe read-canonical-gettext-files-for-desktops thing13:26
ximion*desktop-files13:26
Laneyah yeah sorry 2 then13:28
Laneyabsolute paths one too13:28
Laneybut I need to fix that if you say that looking in Contents first is good enough to get it merged13:28
ximionLaney: for absolute paths: first check if the file has an extension we support (there's a function for that), if we don't do that already. Then check the .contents file13:30
ximionthat's the uncontroversial part13:30
=== hikiko|ln is now known as hikiko
ximionI'm not yet sure about the "check dependencies explicitly as well" approach13:31
LaneyIt's needed to avoid breaking for any new split13:31
Laneybut probably won't be used that much after the contents check13:31
Laneyoh wow, appstream.ubuntu.com is down because of a kernel panic13:32
Laney[1642642.764107] Buffer I/O error on device vdc, logical block 713:32
Laney[1642642.765654] BUG: unable to handle kernel NULL pointer dereference at 000000000000000813:32
ximionoO13:33
ximionwhat did it do to trigger that?13:33
Laneywho knows13:34
Laneyall is never well The Cloud13:34
Laneyin^13:34
Laneyit is back13:36
ximionthe cloud is the "even if things fail, if you have a lot of them failure doesn't matter much and isn't worth time to investigate because some instance will always keep running when one breaks down" approach13:36
Laneyhere it is have some commands to make VMs on demand but sometimes those VMs will break mysteriously13:38
ximionusually I would blame btrfs or OverlayFS and be right with it 60% of the time :P13:42
desrtattente, larsu: https://wiki.gnome.org/Hackfests/GTK201614:38
=== Guest827 is now known as fredp
=== fredp is now known as Guest44856
attente"Threat to global stability" ¬_¬15:00
desrt:D15:05
larsudesrt: I don't think I'll come :/15:25
desrtaw :(16:00
davmor2cyphermox, willcooke: we still have slightly MASSIVE issues with upgrades, I'm currently looking at a black screen on my laptop that does nothing.16:11
ogra_"slightly MASSIVE" heh16:12
ogra_davmor2, stop using the black on black theme !16:12
davmor2ogra_: if only it were that16:12
davmor2The fact I can't get a tty is the thing I'm finding most concerning16:13
ogra_wiggle the cable16:14
ogra_:P16:14
davmor2cyphermox, willcooke: the nice thing now though is at least I know it's not just a kvm thing so can enjoy testing this from there again :)16:17
willcookedavmor2, ha16:17
davmor2flocculant: ^ just a heads up that upgrades are a bit fixed and a lot broken16:18
flocculantdavmor2: ha ha thanks :)16:18
willcookedavmor2, so the initial problems are now fixed, but this is something new?16:18
cyphermoxwell, the "True" distro still does not exist so that's at least one issue on upgrade using the CD16:19
davmor2willcooke, cyphermox: or the original upgrade issue that I saw.  This basically gets so far through the install kicks you out of the session and leaves you in session with no display as in really no display16:20
cyphermoxdavmor2: I haven't seen that yet16:21
davmor2cyphermox: I see it on both kvm and hardware with uefi + secureboot I'm using -cpu host which is i7 on kvm and it is an i7 in the hardware too16:24
cyphermoxI don't know, I can only deal with one thing at a time16:24
cyphermoxfor now I'm fixing the sources.list (or trying to, at leat)16:24
cyphermox*least16:25
davmor2cyphermox: being as I have no tty's I'm thinking enable open ssh and see if I can at least ssh into the kvm/laptop to get around that issue to atleast be able to run apport16:26
cyphermoxif you have to tty and stuff like that, I'd be very suspicious of systemd in general16:31
cyphermox(that was me trying to shift the blame to pitti, yes) ;)16:31
davmor2cyphermox, willcooke: https://bugs.launchpad.net/ubuntu/+source/ubuntu-release-upgrader/+bug/1555237  -redir and install openssh-server to the rescue :)17:09
ubot5Launchpad bug 1555237 in ubuntu-release-upgrader (Ubuntu) "Upgrade from 14.04.4→ 16.04 dies midway taking out the session." [Undecided,New]17:09
willcookethanks davmor217:10
willcooketrying here in a vm17:10
Texouhi17:15
Texouhi didrocks17:15
alexarnauddidrocks: hello! How when you installed a package to define relacatable profile ?17:16
cyphermoxdavmor2: compiz crashes maybe? https://launchpadlibrarian.net/247279143/CurrentDmesg.txt.txt17:16
cyphermoxTexou: hey!17:16
Texouhey cyphermox  :)17:17
willcookecyphermox, yeah, just seen that as well17:17
davmor2cyphermox: that's what I said to willcooke but then why is there no tty17:17
willcookehikiko, looks like compiz is seg faulting in the upgrade ^17:17
willcookeTrevinho, ^17:17
willcooke  726.661272] compiz[1908]: segfault at 18 ip 00007f8ba143d26e sp 00007ffddce03f70 error 4 in libglib-2.0.so.0.4002.0 (deleted)[7f8ba13bc000+106000]17:17
cyphermoxwell, if compiz explodes it will kill off the session, and then lightdm may want to try to restart17:17
didrocksalexarnaud: well, you have the example in the compiz package itself with the Unity profile17:18
davmor2cyphermox: I'm wondering if x goes and that in turn kills compiz which in turn takes out the system17:18
didrocks(isn't what you are working from day one? :))17:18
cyphermoxdavmor2: so, normally you'd be right it should get back to lightdm, which would be happy again, but it looks like it's crashing badly enough to try to get in failsafe-x, which also for some reason starts compiz?17:18
davmor2I blame ogra_ snappy should of fixed this :D17:19
cyphermoxat least that's my reading of dmesg there17:19
ogra_davmor2, except that these desktoppers are lazy and havent ported their desktop to snappy yet ... not my fault at all !17:20
willcooke:p17:20
ogra_:)17:20
davmor2cyphermox: could be17:20
davmor2ogra_: hey don't try squirming out if it, I put the blame squarely on your shoulders :P17:21
TrevinhoOh, crash? What, when, why?17:22
davmor2Trevinho: upgrade from 14.04.4 + updates to 16.0417:22
Trevinhodavmor2: mh, nice one...17:22
Trevinhodavmor2: do you have any stacktrace for that?17:22
willcookeit might be a red herring Trevinho17:22
davmor2Trevinho: no crash as such whole system is taken out bug is https://bugs.launchpad.net/ubuntu/+source/ubuntu-release-upgrader/+bug/155523717:23
ubot5Launchpad bug 1555237 in ubuntu-release-upgrader (Ubuntu) "Upgrade from 14.04.4→ 16.04 dies midway taking out the session." [Critical,New]17:23
TrevinhoWell, I guess that still compiz crashes for some reason, thus the rest stops... And isn't anything on /var/crash about that?17:25
willcookedavmor2, are you upgrading from the UK mirrors or the main ones?17:26
davmor2willcooke: I assume uk as that is default17:27
Trevinhoseb128: can you have a look at https://code.launchpad.net/~3v1n0/gtk/unity-maximized-headerbar-buttons-hide/+merge/288552 ?17:46
seb128Trevinho, did we decide to go for an unity/ubuntu hack rather than a gtksetting?17:47
Trevinhoseb128: well, that's what we said yesteday (go for the easiest way,as it would cause changing the u-s-d as well), no?17:48
seb128attente, unsure how busy you are on g-s bugs but I can't figure out https://bugs.launchpad.net/ubuntu/+source/gnome-software/+bug/1554164 , it looks like it should be simple but I don't understand where it's wrong, maybe you could have a look?17:48
ubot5Launchpad bug 1554164 in gnome-software (Ubuntu) "Invalid read in get_changelog()" [Undecided,New]17:48
seb128Trevinho, you discussed it with desrt, I didn't follow too closely sorry17:49
seb128but that wfm17:49
attenteseb128: sure17:49
seb128attente, thanks17:49
attenteseb128: how did you produce that?17:50
Trevinhoseb128: based on that I'd also rework the message-dialog-restore-traditional-look-on-unity.patch to use the new atom instead of XDG_CURRENT_DESKTOP ? (desrt didn't like that)17:50
seb128attente, have an update available17:50
seb128or some updates17:50
seb128Trevinho, wfm17:50
desrtTrevinho: that is more tricky17:51
Trevinhoseb128: actually, we can't since... it's on _init...17:51
seb128attente, install https://launchpad.net/ubuntu/+source/language-selector/0.158/+build/9296660/+files/language-selector-common_0.158_all.deb https://launchpad.net/ubuntu/+source/language-selector/0.158/+build/9296660/+files/language-selector-gnome_0.158_all.deb17:51
seb128attente, if you are uptodate17:51
Trevinhoso, we can safely keep in that way17:51
attenteseb128: you did that via the --localfilename option?17:51
desrtTrevinho: some deep detail in gtk implementation like how dropshadows are handled is a bit different than issues of higher level UI layout17:51
desrtTrevinho: i'd leave it alone, ya17:51
seb128attente, no, I just start gnome-software, it fetches updates available and their changelogs17:52
Trevinhook17:52
=== alan_g is now known as alan_g|EOD
attenteseb128: you installed those packages on an up-to-date system, launched g-s then switched to the Updates tab and clicked the refresh button? can't seem to reproduce with that sequence of steps :(18:01
seb128attente, no18:01
seb128attente, my system is not update18:02
seb128but those invalid reads seem to happen when get_changelog is called18:02
seb128I guess for that to happen you need updates to be available with changelog fetching to be done18:02
seb128I was suggesting one way to create that situation18:02
seb128unsure what code you are using though?18:03
seb128robert did some changes in https://git.gnome.org/browse/gnome-software/log/?h=wip/rancell/apt18:03
seb128if you are on trunk you might want to change18:03
seb128uri = g_strdup_printf ("http://changelogs.ubuntu.com/changelogs/binary/%s/%s/%s/changelog", source_prefix, source, update_version);18:03
attenteseb128: you're running from robert's branch or the distro packaged version?18:03
seb128to have s/source/binary_source/18:03
seb128attente, doesn't matter18:04
seb128well I'm on robert's trunk version atm18:04
seb128and current_version is garbage if I tried to print it in get_changelog18:04
seb128so I guess the issue is not fixed18:04
seb128attente, if it's not easy to trigger for it forget about it, or wait a bit for xenial updates to be available18:11
GunnarHjseb128: Do you have time to take a look at the u-c-c change I mentioned? Asking you because you added the problematic line:18:12
GunnarHjg_setenv ("XDG_CURRENT_DESKTOP", "Unity", TRUE);18:12
GunnarHjThe issue I ran into is that language-selector-gnome uses XDG_CURRENT_DESKTOP to control the behavior (wrt im-config configuration) depending on the desktop, and when language-selector-gnome is started from u-c-c in GNOME Flashback, it does the wrong thing.18:12
seb128GunnarHj, sorry, day was crazy ... did I add that line?18:12
seb128what's the issue with that?18:13
GunnarHjseb128: I think so. (According to the changelog.)18:13
GunnarHjseb128: It hijacks the XDG_CURRENT_DESKTOP variable for u-c-c internal purposes.18:13
seb128well, u-c-c is an unity thing18:13
seb128don't use it if you don't have an unity type session18:14
GunnarHjseb128: Well, currently it's used by GNOME Flashback.18:14
GunnarHjseb128: The think is that we don't want the same behavior on GNOME Flashback as in Unity.18:15
seb128why not?18:15
GunnarHjseb128: It's a little complicated. Actually a workaround, so the ibus variables are not set for users who don't need them.18:17
attenteseb128: yeah, the xenial version is quite a bit different from the apt branch, and looking at robert's changes, it's possible that it'll be fixed once he does another upload18:18
GunnarHjseb128: Changing the variable name to something else solves the problem.18:18
hikikowillcooke, i was at the pool, i ll test this when i go home18:18
seb128attente, well, if I apply that on top of trunk http://paste.ubuntu.com/15336283/18:18
hikiko(the segfault)18:19
willcookehikiko, thx.  Not sure what's going on there yet.  Still upgrading my VM to see if I can get any more idea18:19
seb128attente, I get e.g18:19
seb128version: X�«�«018:19
seb128version: 0.15718:19
willcookehikiko, might be a symptom rather than a cause18:19
seb128attente, sometimes it's fine, but it still gets corrupted18:19
seb128attente, the first chunk is a hack to not be disturbed by info about all other things I didn't have updated18:20
seb128GunnarHj, I don't have a good answer/idea aobut your issues, it feels like the problem described in the changelog when the line was added would still apply today if we reverted the change18:24
GunnarHjseb128: I'm not suggesting we revert the change, actually. XDG_CURRENT_DESKTOP seems to used within unity to control things like if it shall provide links to Ubuntu docs or GNOME docs. My suggestion is to keep the hardcoded "Unity", but use another variable name for the purpose (throughout the whole u-c-c).18:26
seb128attente, it might be that the issue is with re-entrance and that this function needs to be called several time in // so requires more updates to be available to hit the bug18:27
seb128GunnarHj, you have a patch to do that?18:27
GunnarHjseb128: Yes. https://launchpad.net/~gunnarhj/+archive/ubuntu/im-config-gnome/+packages18:28
seb128GunnarHj, that change feels wrong18:28
seb128XDG_CURRENT_DESKTOP is a standard thing18:29
seb128GunnarHj, I guess you could try to summarize the issue on an email to the desktop list and see if you get any traction there18:29
attenteseb128: yeah... tbh i'm really not sure. if you can reproduce it reliably, can you at least figure out which string is being invalidly accessed by trying each one in the g_strdup_printf invidiually?18:31
GunnarHjseb128: Is it standard to change it within a application to something else but the originally set value? Why is XDG_CURRENT_DESKTOP the right variable to use for controlling the u-c-c behavior internally?18:31
attenteit seems like source and binary_source should be safe because of the guard, but maybe there's some other weird threading issues happening here18:32
seb128attente, in the current trunk code current_version has the issue for sure18:32
mdeslaurseb128: fyi, I'm stealing your graphite2 merge18:32
seb128mdeslaur, thanks18:32
seb128mdeslaur, is there a security fix in there?18:33
mdeslauryeah, 1.3.6 fixes a zillion cves18:33
seb128GunnarHj, XDG_CURRENT_DESKTOP is what tells you in what desktop you are, so it's what we query to check if we are in unity and have unity specific behaviour, it's done in gedit/nautilus/etc as well18:33
seb128mdeslaur, good!18:33
attenteseb128: but current_version isn't used in the g_strdup_printf() call18:34
attenteor did you mean update_version?18:35
seb128attente, http://paste.ubuntu.com/15336283/18:36
seb128attente, the valgrind might not be matching what I describe18:36
seb128I took it on xenial some days ago18:37
seb128attente, that diff prints a corrupted "version" with trunk18:37
GunnarHjseb128: Right, and when I'm in GNOME Flashback, the original XDG_CURRENT_DESKTOP is "GNOME-Flashback:Unity", but an app (in this case language-selector-gnome), which is started from u-c-c, is told that it is on "Unity" only.18:37
seb128GunnarHj, right, as said I don't have a good reply to that18:38
seb128I understand the issue but I don't know how to fix it18:38
muktupavelsseb128, GunnarHj: how about checking current value of XDG_CURRENT_DESKTOP before changing it? Change it only if does not include GNOME-Flashback?18:43
seb128muktupavels, GunnarHj, I guess we could read the value and add Unity to the list if it's not there18:44
seb128muktupavels, good thinking :-)18:44
GunnarHjmuktupavels, seb128: Sure, that would serve the same purpose from my POV.18:45
GunnarHjmuktupavels, seb128: Before I go back to the drawing board, would that be an acceptable change?18:47
seb128GunnarHj, I think so18:48
GunnarHjseb128: Ok, then I'll make a new patch based on that. Hopefully you don't change your mind later. ;)18:49
seb128thanks18:50
seb128well, the patch should be trivial if you change the "set XDG_CURRENT_DESKTOP to unity" to be "get the variable, add unity if it's not in the list"18:50
seb128and that should get your case to work as well right?18:50
GunnarHjseb128: Yes, that's my understanding.18:51
seb128great18:54
seb128GunnarHj, thanks for working on that!18:54
muktupavelswhy it is needed in first place? Maybe it can be as simple as checking if that variable is set and if not then set it to Unity?18:54
seb128that might work as well18:55
seb128the changelog mention the env ubiquity18:55
seb128unsure if XDG_CURRENT_DESKTOP is set there and to what18:55
seb128but I guess it's not18:56
muktupavelshow is ubiquity started? maybe correct thing is to make sure that it is set for that session? Setting it from ucc feels like workaround/hack...18:57
GunnarHjmuktupavels, seb128: But to be safe, it's just as simple to check if the variable includes "GNOME-Flashback", and if not set it to "Unity".19:00
GunnarHjmuktupavels: Agree that it feels like a workaround in the first place. Just trying to solve a tiny problem. ;)19:01
willcookedavmor2, new wheeze.  Potentially related.... started upgrade, came back to it a few times to see how it was getting on.  This time I can't unlock the lockscreen to see whats going on19:17
willcookeenter password, black screen, then back to the lock screen again19:21
* flocculant shall carry on studiously ignoring upgrades then :p19:25
willcooke:)19:26
davmor2willcooke: yeap same kinda thing19:43
davmor2willcooke: infact I think that is the exact symptoms I had before the init-mod-utils bug19:44
davmor2willcooke: it's good fun though right :)19:44
willcookeohhh yes19:44
willcookefun19:44
davmor2willcooke: is that shuggin fashin shuggin fashin dick dastardly I hear muttering under your breath?19:45
willcooke:D:D19:45
willcookedavmor2, is this normal:19:54
willcookeConfiguring libssl1.0.0:amd6419:54
willcookeblah blah blah19:54
willcookeou can choose this option to avoid being prompted;19:54
willcookeinstead, all necessary restarts will be done for you19:54
willcookeautomatically so you can avoid being asked questions on each19:54
willcookelibrary upgrade.19:54
willcookeRestart services during package upgrades without asking?19:54
willcooke.19:55
willcookeBasically I had to expand the terminal section and say "y"19:55
willcookeNot that out of the ordinary, but I don't think I've had to type anything in to that terminal to complete an upgrade befor19:55
willcookee19:55
davmor2willcooke: I thought it was meant to throw up a popup for those kind of requests19:59
willcookehumm19:59
davmor2willcooke: it at least does for things like config changes20:00
GunnarHjseb128: Still there?20:02
=== Guest44856 is now known as fredp
=== fredp is now known as Guest77642
seb128GunnarHj, not really but around the computer, just ask I might reply if it's easy20:08
willcookeahhh20:11
willcookedavmor2,20:11
willcookefrom your apttermlog.txt20:11
willcookeyou got as far as the prompt I was talking about up there ^^^^20:11
willcookeand that's why yours stopped20:11
willcookehttps://launchpadlibrarian.net/247279149/VarLogDistupgradeApttermlog.txt20:11
willcookehave a look there ^20:11
willcookebut....20:12
willcookeI got past that20:12
willcookeand mine stopped here:20:12
willcookeSetting up bash-completion (1:2.1-4.2ubuntu1) ...20:12
willcookeSetting up liblocale-gettext-perl (1.07-1build1) ...20:12
willcookeand at some point before that, not much before that mind, my shell stopped working20:12
willcookecyphermox, fyi ^20:12
willcookeI will comment on the bug20:12
davmor2willcooke: I'm actually beginning to wonder if it isn't the code that is meant to block the system sleeping that is causing it20:13
davmor2willcooke: hence me hitting it one place and you another20:13
willcookeI think that if you'd answered yes to that you would have carried on to where I got to20:14
davmor2willcooke: for the question you should get a debconf popup like this https://zxtech.files.wordpress.com/2014/12/debconf-on-sara_066.png20:16
GunnarHjseb128: Well, I jumped at conclusions. Not changing it in case of "GNOME-Flashback" would solve my problem, but also create a new bigger one: The original g-c-c code would be applied wherever the XDG_CURRENT_DESKTOP value is tested against "Unity". So I'm back at my original idea with using some other variable name. (An alternative, if you for some reason don't want that, is to test XDG_CURRENT_DESKTOP both against "Unity" and "20:16
GunnarHjGNOME-Flashback:Unity" all over the place. But that would be more code...20:16
davmor2willcooke: definitely not a happy place to be though right20:18
muktupavelsGunnarHj, 'The original g-c-c code would be applied wherever the XDG_CURRENT_DESKTOP value is tested against "Unity".' - why/how that is problem?20:21
davmor2willcooke: I think if you reboot you are able to access tty but still get no gui too20:21
willcookerebooting now20:21
willcookeit's dead Jim20:21
willcookeah, there we go20:22
willcookeoh wow20:22
willcookenow it really is dead20:22
willcookeyay20:22
davmor2willcooke: and that is why I marked it critical20:22
willcookekernel panic20:22
davmor2willcooke: oh I never had that20:23
GunnarHjmuktupavels: The original g-c-c options are simply different, and probably not what the GNOME Flashback people want.20:23
davmor2willcooke: but either way really not a happy place20:23
willcookedavmor2, my guess is that if you could have answered Y to that prompt, you'd now have a kernel panic20:23
davmor2willcooke: possibly20:23
GunnarHjmuktupavels: They are probably just a leftover from the time when u-c-c was forked from g-c-c.20:24
muktupavelsGunnarHj, gcc? or did you mean ucc?20:24
davmor2willcooke: oh and to add no pressure final beta is when most people upgrade right20:24
GunnarHjmuktupavels: I think I wrote what I meant...20:25
GunnarHjmuktupavels: u-c-c is a fork from g-c-c.20:25
GunnarHjmuktupavels: u-c-c includes quite some code which is not actually used.20:26
muktupavelsCan you give link that is applied but should not?20:26
cyphermoxwillcooke: what did you mean by your shell stopped working?20:26
cyphermoxugh, made bad coffee somehow, brb20:26
GunnarHjmuktupavels: Most it is about reference to applicable docs: Ubuntu help or GNOME help. This file includes a few such spots, where XDG_CURRENT_DESKTOP is tested against (the hardcoded) string "Unity": http://bazaar.launchpad.net/~ubuntu-branches/ubuntu/wily/unity-control-center/wily/view/head:/shell/control-center.c20:33
muktupavelsGunnarHj, that must be fixed... Add ucc to affects in this bug - https://bugs.launchpad.net/ubuntu/+source/nautilus/+bug/155487820:35
ubot5Launchpad bug 1554878 in nautilus (Ubuntu) "fix up usage of XDG_CURRENT_DESKTOP" [Undecided,New]20:35
willcookecyphermox, U7 session just locked up.  First of all the upgrader window went grey and unresponsive but I was still seeing new log entries via ssh20:36
willcookecyphermox, then the whole desktop stopped responding.  SSH was still alive.20:36
willcookecyphermox, then I ctrl-c'd out of the log and sudo reboot20:36
willcookeand then the whole thing just locked up.20:37
willcookeand now it is dead.20:37
willcooke:(20:37
willcookeAttached my log here:  https://bugs.launchpad.net/ubuntu/+source/ubuntu-release-upgrader/+bug/155523720:37
ubot5Launchpad bug 1555237 in ubuntu-release-upgrader (Ubuntu) "Upgrade from 14.04.4→ 16.04 dies midway taking out the session." [Critical,New]20:37
GunnarHjmuktupavels: Well, one option is to keep using XDG_CURRENT_DESKTOP, and thus add u-c-c to that bug. Another option is to use some other variable name for the purpose, as I first suggested. (But seb128 didn't like that...)20:38
muktupavelsGunnarHj, while I dont know original reason why that was added, it looks like it was added for cases when it was not set at all. So right thing might be to set it to Unity when XDG_CURRENT_DESKTOP is NULL or fix ubiquity scripts to export XDG_CURRENT_DESKTOP=Unity.20:38
willcookemorning robert_ancell20:39
robert_ancellwillcooke, hi20:39
GunnarHjmuktupavels: But that wouldn't help, considering how the variable is queried at a lot of places.20:39
sarnoldwillcooke: btw, if you inteded to attach logs on the kernel panic, they don't appear to have made it to the bug20:40
willcookesarnold, it's in a VM, can I get logs out of it?  Dont know how20:41
muktupavelsGunnarHj, g_strcmp0 (XDG_CURRENT_DESKTOP, Some Desktop) is incorrect and must be fixed in all known places.20:41
sarnoldwillcooke: depends upon how dead it is, I guess .. perhaps a different kernel via grub, or init=/bin/sh or something similar if it's only partially dead..20:42
muktupavelsGunnarHj, first thing is to make sure that this variable is not changed. If session was started with GNOME-Flashback:Unity then it should not change.20:43
sarnoldwillcooke: if it's too dead for that, then perhaps some qemu-img messing around to try to gte to the files directly. (assuming it even logged its own death, that's getting harder and harder to capture..)20:43
muktupavelsGunnarHj, if (g_getenv("XDG_CURRENT_DESKTOP") == NULL) g_setenv ("XDG_CURRENT_DESKTOP", "Unity", TRUE); should be correct patch/fix even if it does not directly fix problem.20:44
GunnarHjmuktupavels: Should we split on ":" all over the place, to be able to test against "Unity", or what?20:45
muktupavelsGunnarHj, that would be right thing.20:46
GunnarHjmuktupavels: Or using strncmp()20:46
GunnarHjmuktupavels: (last 5 characters)20:46
willcookesarnold, cheers.  Managed to get in to a desktop with an old kernel, no networking though (yet)20:47
sarnoldwillcooke: heh, at least it's a start..20:47
willcooke:)20:47
muktupavelsGunnarHj, it could be also set to - GNOME-Flashback:Unity:My-Desktop...20:49
GunnarHjmuktupavels: Do you mean according to the new spec?20:50
davmor2willcooke: fire up the live cd that should give you access to the logs if all else fails20:51
willcookeI've got logged in, but no logs20:51
davmor2willcooke: sigh20:52
muktupavelsGunnarHj, yes!?20:52
davmor2willcooke: it's too late to care anymore I'm calling it a night and I'll see what I can get tomorrow on hardware with a really ethernet port and stuff :)20:52
willcookeditto I think20:53
davmor2-ly20:53
willcookecheers davmor220:53
muktupavelsGunnarHj, https://bugzilla.gnome.org/show_bug.cgi?id=76336420:53
ubot5Gnome bug 763364 in general "add an api to query XDG_CURRENT_DESKTOP" [Normal,New]20:53
muktupavelsGunnarHj, if such API would be added then it would be as simple as if (g_in_desktop("Unity")) or if (g_in_desktop("Unity") && !g_in_desktop("GNOME-Flashback"))20:55
GunnarHjmuktupavels: Basically I understand what you are saying. However, in the case of u-c-c changing it like that all over would be quite some extra code for no sensible reason. I can't think of a situation when some desktop would like to use u-c-c and not refer to Ubuntu help. So I keep thinking that the root issue in case of u-c-c is that XDG_CURRENT_DESKTOP is used unnecessarily. Hmm.. Maybe robert_ancell can show some light on it20:57
GunnarHj...20:57
GunnarHjrobert_ancell: Hi Robert!20:57
robert_ancellGunnarHj, hi20:57
GunnarHjrobert_ancell: Do you know why the XDG_CURRENT_DESKTOP is used in u-c-c to choose the code for references to Ubuntu help rather than GNOME help etc.?20:58
robert_ancellGunnarHj, we treat Unity as the exception (checked via XDG_CURRENT_DESKTOP) and revert to upstream behaviour in all other cases20:59
robert_ancellSeems the fairest for upstream20:59
muktupavelsGunnarHj, whell ucc is for unity so it might make sense to remove checks and assume that it is run on Unity desktop...20:59
robert_ancellmuktupavels, that API would be super helpful20:59
robert_ancellmuktupavels, actually, good point. It is *only* for Unity, so it doesn't really need to check21:00
GunnarHjrobert_ancell: But is u-c-c actually used on other desktops?21:00
robert_ancellperhaps that is a hangover from the split from g-c-c?21:00
GunnarHjrobert_ancell: That's my theory too.21:00
muktupavelsrobert_ancell, then drop comment in that bug? looks like glib developers is not quite interested in adding such API.21:00
robert_ancellGunnarHj, it might be, but they would be derivatives would be Ubuntu based and so the Ubuntu docs are the most appropriate21:01
GunnarHjrobert_ancell: Right. GNOME Flashback is one such example, and that's why I'm here bothering a lot of people. ;)21:02
GunnarHjrobert_ancell: The problem is that u-c-c currently *sets* XDG_CURRENT_DESKTOP to "Unity", which in case of GNOME Flashback results in an incorrect variable value. My idea, for a simple change to solve the conflict, is to just use some other variable name instead of XDG_CURRENT_DESKTOP.21:03
muktupavelsGunnarHj, some other variable will not work. You need Unity to get panels loaded...21:04
GunnarHjmuktupavels: I mean changing the variable name all over u-c-c. As I suggested at ttps://launchpad.net/~gunnarhj/+archive/ubuntu/im-config-gnome/+packages21:05
robert_ancellGunnarHj, u-c-c sets XDG_CURRENT_DESKTOP?21:06
GunnarHjrobert_ancell: Yes, unfortunately.21:06
robert_ancellit should only be set by LightDM...21:06
willcookenight all21:06
robert_ancellGunnarHj, is there a bug for that?21:07
muktupavelsGunnarHj, does ucc has custom code that reads desktop files? or do you plan to patch glib? panels have desktop files that have ShowOnlyIn=Unity (most likely, did not check). So you can change watever you want - panels will not get loaded...21:07
GunnarHjrobert_ancell: No bug yet...21:08
robert_ancellGunnarHj, 'cause that's wrong and should  be fixed21:08
muktupavelsrobert_ancell, is ubiquity started by lightdm?21:08
robert_ancellmuktupavels, I'm not sure21:08
muktupavelsif I understand corecly then it was added because ucc was not loading panels there...21:09
GunnarHjmuktupavels: Not sure what you mean now. XDG_CURRENT_DESKTOP is set to its proper value elsewhere, so I don't see what the problem with the .desktops files would be.21:10
robert_ancellmuktupavels, in other desktops other than Unity?21:10
muktupavelsrobert_ancell, seb128 probably can explain better. I think that ubiquity does not set XDG_CURRENT_DESKTOP at all, but it should set it to Unity.21:13
muktupavelsSo when ucc is used in ubiquity panels are not loaded because they include OnlySHowIn=Unity21:13
muktupavelshttp://bazaar.launchpad.net/~ubuntu-branches/ubuntu/wily/unity-control-center/wily/view/head:/panels/appearance/unity-appearance-panel.desktop.in.in21:14
muktupavelsGunnarHj, check .desktop files for ucc panels. They are only shown if desktop is Unity. Missing XDG_CURRENT_DESKTOP means that no panel will be loaded.21:17
GunnarHjmuktupavels: Right, *missing* value would result in that, I forgot about the assumed ubiquity issue... Sorry.21:18
GunnarHjmuktupavels: Hmm.. I have an idea: 1. Set XDG_CURRENT_DESKTOP to "Unity" if empty. 2. Introduce a new boolean environment variable IS_UNITY which for now can be used for the tests all over the place.21:21
GunnarHjmuktupavels: (I mean that u-c-c should set that variable for internal use only.)21:24
muktupavelsGunnarHj, ucc should not set it at all... but setting it only if empty/not set should be better then setting it always. I would not use env, introduce function to check it and use it.21:24
GunnarHjmuktupavels: The purpose with such a boolean variable would be to split on ":" once, instead of doing it in 25 or so places in the source code.21:26
GunnarHjmuktupavels: Aha, you said function *instead of* an env. variable. Got it.21:29
muktupavelsGunnarHj, http://paste.ubuntu.com/15337448/21:29
GunnarHjmuktupavels: Right, something like that.21:30
muktupavelsGunnarHj, if it will be used only to check for Unity then result probably could be stored in static so next time it is called it already returns previous result.21:33
GunnarHjmuktupavels: But there are multiple files which need to use it...21:34
GunnarHjmuktupavels: I'll make an attempt. I'm not really a C coder, so it will be a 'challenge'. ;)21:35
robert_ancellGunnarHj,  muktupavels, a load of those XDG_CURRENT_DESKTOP checks in u-c-c are pointless - I've filed some MPs to remove them22:16
GunnarHjExcellent, robert_ancell! That's what I thought. Where is that MP?22:32
GunnarHjrobert_ancell: Never mind, I found them.22:35
GunnarHjrobert_ancell: I see that muktupavels added a u-c-c task to bug #1554878. Just for my understanding, before we go on and fix the remaining XDG_CURRENT_DESKTOP tests: Is u-c-c at all in use on non-Unity desktops? Isn't it a Unity only fork from g-c-c? If it is, I suppose that the most logical measure would be to drop the "else" code snippets, and stop testing for XDG_CURRENT_DESKTOP completely.23:19
ubot5bug 1554878 in unity-control-center (Ubuntu) "fix up usage of XDG_CURRENT_DESKTOP" [Undecided,New] https://launchpad.net/bugs/155487823:19
=== thumper is now known as thumper-dogwalk

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