/srv/irclogs.ubuntu.com/2014/09/19/#ubuntu-devel.txt

infinitypitti: Hrm.  Check out the sbcl failure in utopic-proposed.  pkgstrip vomits with a "find : permission denied"... Bug in pkg* for not ignoring that, or bug in sbcl for producing non-readable/traversable files/directories during build, or both?02:18
infinitypitti: (Testing the obvious fix for sbcl and will submit that to Debian regardless, but wondering if we should also consider this a pkgbinarymangler failure and fix that)02:32
pitticamako: oh, sure, can do!02:48
pittiGood morning (or so)02:48
pittiinfinity: yes, in general I think we should fix this stuff in pkgbinarymangler too, to avoid rather pointless package deltas02:49
pitticamako: I wasn't aware that you guys are waiting on this, so thanks for the ping02:52
pitticamako: released upstream 0.8.7 and uploaded to sid; I'll sync it in a few hours once it got imported into LP04:10
happyaronattente_: it's released yesterday, I'm working on it atm05:30
KjetilKI'm one of the upstream devs of librdf-trine-perl, and I've just committed some rather extensive performance enhancements in the package. These are the only changes since the last version, which is already in utopic06:56
=== BruceMa is now known as BM
=== BM is now known as BMW
=== BMW is now known as BM2
KjetilKwe would normally not be releasing a new version with just a single changeset, but if it could make it into Utopic, we might do it anyway06:56
KjetilKso, I was wondering if it is worth releasing and pestering the Debian dev about it, or if it is too late anyway?06:57
=== BM2 is now known as BruceMa
infinityKjetilK: We're (way) past feature freeze, so unless you can make a good case for why it's absolutely necessary for utopic, it probably wouldn't make it anyway.07:01
KjetilKinfinity, yeah, I figured07:01
infinityKjetilK: But releasing and getting it into Debian for jessie (and we'll pull it into 15.04) still isn't a bad plan.07:01
KjetilKyeah, the DD usually packages our releases in a week or two07:02
infinityKjetilK: Right, well the jessie freeze is imminent, so if you needed motivation, go with that.  If you can be bothered to spend the time trying to convince people like me that it's a safe and sane update for Ubuntu to pull in, we can talk after it's in Debian.07:03
KjetilKbasically, my argument wouldn't be that it is absolutely necessary, but rather "not a huge number of users, if we've screwed up, it is unlikely to negatively impact many, but the gains are likely large for the few who use it"07:03
KjetilKok, we'll follow the normal cycle, then07:05
KjetilKinfinity, BTW, how imminent is jessie freeze, last I read was early November?07:05
infinityKjetilK: "Not many users" is, to me, actually an argument against updating, not for.07:09
infinityKjetilK: On the one hand, not many users would benefit and, on the other, very few people will be testing it to find bugs in time to fix them before release.07:09
KjetilKyeah, I can see that, fewer eyeballs07:09
* KjetilK nods07:09
infinityKjetilK: As for the imminence of jessie freeze, I keep forgetting the exact date, and have just been treating it as "very soon" to push myself to get the things in that I need to. :P07:10
KjetilKhehe, good :-)07:10
=== tvoss is now known as tvoss|test
KjetilKinfinity, BTW, since you're a DD, I just noticed https://packages.debian.org/sid/noip207:14
KjetilKwhich doesn't have a maintainer, but I find no WNPP bug on it, and the package is uptodate with upstream and appears sane...07:15
KjetilKPerhaps someone has orphaned it without telling anybody?07:16
infinityKjetilK: That's cruft.  It's been removed from sid, note that it only exists on m68k on debports.07:17
* KjetilK nods07:17
infinityKjetilK: https://packages.qa.debian.org/n/no-ip.html07:17
infinityKjetilK: Removed over 2 years ago.07:18
KjetilKinfinity, yeah, but that's a different package, isn't it?07:18
infinityKjetilK: No, that was the source that produced the noip2 binary.07:18
KjetilKah, ok07:19
KjetilKthat's probably the reason why I didn't find it on on QA myself :-)07:19
dholbachgood morning07:24
tvosspitti`, guten Morgen :)07:56
tvossdholbach, ping07:59
dholbachtvoss, pong08:00
=== dholbach_ is now known as dholbach
jibeljodh, Hey, wrt. my 2 machines that hang on reboot/shutdown I enabled debug mode and filed bug 137146908:47
ubottubug 1371469 in upstart (Ubuntu) "system hangs on shutdown/reboot" [Undecided,New] https://launchpad.net/bugs/137146908:47
jibeljodh, this is 100% reproducible but not immediately after boot08:47
ypwongcjwatson, isolinux/lang no longer works in utopic, bug 1330416, could you help to take a look?08:50
ubottubug 1330416 in Ubuntu Kylin "In Ubuntu Kylin, default langauge in Ubiquity language chooser is not Simplified Chinese, isolinux/lang not honoured" [Critical,Confirmed] https://launchpad.net/bugs/133041608:50
jodhjibel: thanks. can you attach a log showing the shutdown? I've got a few questions but will add those to the bug...08:50
jibeljodh, which log? I attached syslog, is there any other?08:51
jodhjibel: you didn't actually attach the file.08:52
jibeljodh, oops, sorry08:52
jibeljodh, done. interestingly I cannot attach it uncompressed08:56
=== vrruiz_ is now known as rvr
brendandcjwatson, sorry to bug you again about software-properties - i keep getting told by different people that the issue around setting 14.09 as the series isn't resolvable or something like that09:36
brendandcjwatson, but obviously you told me differently, so i'm just checking on the truth of the matter09:36
brendandcjwatson, there's some proposed hacks to our tools that should be rejected if add-apt-repository is actually going to work in RTM09:36
cjwatsonsmoser: that's just a matter of making sure /boot/efi is mounted in advance and then running grub-install, I think10:39
cjwatsonsmoser: maybe you forgot the mount of the ESP?10:39
cjwatsonNoskcaj: I do lots of ad-hoc seed maintenance but that sort of thing is a matter for the ubuntu-gnome team; I don't tend to get involved10:40
cjwatsonypwong: ok, added to this morning's amusingly long queue :-/10:42
cjwatsonbrendand: I said roughly "I'm working on it" and that's still true.  I intend to make add-apt-repository work10:42
cjwatsonbrendand: I think if you keep asking different people then you'll inevitably get people guessing :)10:43
cjwatsonbrendand: it certainly can be made to work without having to do the controversial base-files change10:43
brendandcjwatson, i'm not asking them for the fun of it :)10:43
brendandcjwatson, it just came up because robru is trying to land a change to phablet-tools that hacks around add-apt-repository completely10:44
shadeslayerdoes anyone know if schroot has space restrictions?10:44
brendandcjwatson, he's getting his information from cyphermox10:44
cjwatsonbrendand: well you have the answer now10:48
ypwongcjwatson, thanks for the help, tried to trace down the root cause as much as I can but does not know enough of gfxboot-theme to go further10:48
cjwatsonypwong: I may well not have time to look today; we'll see10:49
ypwongok10:49
=== kgunn is now known as Guest93672
caribouWhen we do major modifications to code (scripts, programs, etc), is it required to update the copyright statements ?11:44
cjwatsonrecommended11:53
cariboucjwatson: I'm thinking of the makedumpfile script that you reviewed yesterday12:02
=== _salem is now known as salem_
smosercjwatson, i tried to show that in the paste.  the d-i installed system does not have a /boot/efi.12:37
cjwatsonsmoser: this is my best guess sorry12:38
smoserthe contents of /dev/sda1 is /boot/grub/grub (or at least very similar)12:38
smoserits not a filesystem12:38
cjwatsonhang on, you are confusing me by talking about an EFI System Partition12:38
cjwatsonthis is ppc64el right?12:38
cjwatsonwhat has EFI got to do with anything?12:38
smoserdid you look at those pastes ?12:39
cjwatsononly briefly, I had like five people pinging me about urgent things this morning12:39
smoserunderstandable12:39
smoserhttp://paste.ubuntu.com/8375494/12:39
smoserthat is information about what d-i set up.12:39
cjwatsonyou know if you stopped using sgdisk you would be less confused.12:39
cjwatsonI'm not interested in sgdisk output - show me what parted says :)12:39
smoseryou're confusing sgdisk with sfdisk.12:39
cjwatsonah you did, further down12:39
smoseri did show you what parted said12:40
cjwatsonso ... why were you talking about an EFI system partition?12:40
cjwatsonI'm very confused, there should be no such thing12:41
smoserbecause thats the gpt partition type of the /dev/sda12:41
smoser/dev/sda is gpt partitioned12:41
smoserand /dev/sda1 is of type C12A7328-F81F-11D2-BA4B-00A0C93EC93B12:41
cjwatsonyeah, but that doesn't mean it has to have an ESP12:41
cjwatsonhmm, that's thoroughly bizarre12:41
smoseragreed.12:42
cjwatsonit should be a PReP partition12:42
smoseri thought it was just doing PReP too12:42
smoserso i just tried pretending as if it had12:42
smoserbut telling grub to install to it says: not a PReP partition12:42
smoserif you do that.12:42
smoserfor further confusion... i remembered that the thing that is doing the loading is petitboot12:43
smoserkexec based loader.12:43
cjwatsondo you have a partman log from d-i?12:43
smoserso i suspect that it is actually mounting partitions and lookoing. because the menu you see in power-nv (no virt) is not a grub menu12:43
smoseri can probably manage to get you one. but i dont have one at the moment and the system is not being friendly to me.12:44
cjwatsonwhile I understand that you're using d-i as the reference case that works here, this is not how I set up d-i to work12:44
cjwatsonutil/grub-install.c:          grub_util_error ("%s", _("the chosen partition is not a PReP partition"));12:45
cjwatsonthat message?12:45
cjwatsonmaybe compare the GUID in the source with the one on disk12:45
cjwatson          const grub_gpt_part_type_t template = {12:45
cjwatson            grub_cpu_to_le32_compile_time (0x9e1a2d38),12:45
cjwatson            grub_cpu_to_le16_compile_time (0xc612),12:45
cjwatson            grub_cpu_to_le16_compile_time (0x4316),12:45
cjwatson            { 0xaa, 0x26, 0x8b, 0x49, 0x52, 0x1e, 0x5a, 0x8b }12:45
cjwatson          };12:45
cjwatsonand I guess there could be some bug in the comparison there12:45
cjwatsonthe d-i recipe for ppc64el in partman-auto is:12:46
cjwatson8 1 1 prep12:46
cjwatson        $primary{ }12:46
cjwatson        $bootable{ }12:46
cjwatson        method{ prep } .12:46
smosercjwatson, ok. i have no idea how its booting.12:47
cjwatsonwhich should turn into SET_FLAGS prep for parted_server12:47
smoserbut i do have an idea as to why it might be different then you expected.12:47
smosermaas is feeding some preseed and that could be doing it.12:48
smoserinfinity, do you have any idea how boot works on power nv ?12:48
smoseri'm thikning grub isn't actually involved.12:48
cjwatsonoh and more importantly the prep filesystem12:48
smoserwhat is "prep filesystem" ?12:49
smoserhow does one create that12:49
cjwatsonoh just a virtual thing that causes parted to slam in the right guid12:49
cjwatsonnot a real filesystem12:49
cjwatsonmore a partition type than a filesystem, sorry for confusing term12:50
smoserok. you can go back to other more pressing things.12:50
cjwatsonit would be helpful to see the maas preseeding12:50
smoserlet me see if i can dig that up12:50
cjwatsonsince we should really get that right ...12:51
smoserhttp://paste.ubuntu.com/8379633/12:51
smoserbut fwiw, it *is* right :)12:51
smoserit works.12:52
cjwatsonhaving an EFI System Partition on a system that is not EFI is not right12:52
cjwatsonit may work by virtue of being ignored12:52
cjwatsonbut it is a wart we should clean up12:52
cjwatsonespecially if it causes people to ask me really confusing questions :-)12:52
cjwatsonok, none of that preseeding should influence that, so I'd still like to see the partman log when you get a chance12:53
smoserwhere do i collect that ?12:53
cjwatsonit's /var/log/installer/partman on an installed system12:53
smoserthanks.12:53
cjwatsonwhen I was paying close attention to this, the partitioning was GPT { PReP; <rest of Ubuntu> }12:54
cjwatsonnow since then we have done a major parted upgrade12:54
cjwatsonso it *could* be that something went crazy there12:54
cjwatsonbut I think you should be patterning things after how it's meant to work :)  (and while I appreciate wanting to pattern things after something that *does* work, this did work in the past)12:55
cjwatsonsmoser: right, just dug up the relevant bits of parted - it is actually a flag, "set <partition-number> prep on" in the parted CLI should do it for instance12:59
cjwatsonor ped_partition_set_flag (part, PED_PARTITION_PREP, 1);12:59
smosercjwatson, so you think it is getting set on the efi system partition ?13:01
cjwatsonI think it's erroneously not getting set13:01
cjwatsonand that the boot flag is getting set instead, which is mapped onto "please be an EFI System Partition"13:01
cjwatsonthese sorts of not-a-real-filesystem things are mostly done using flags in libparted13:02
cjwatsonor real-filesystem-but-special-semantics13:02
smoserright. thats what i had thoguth13:02
smoserand i had other curtin that suc cessfully put a prep partition on13:02
smoser(at least something identified as prep)13:02
smoserbut then system didnt boot13:03
cjwatsonand it booted with that?13:03
cjwatsonoh13:03
cjwatsonum13:03
cjwatsonwell, let's get d-i in the right shape again and then model on that?13:03
smosercjwatson, also remember that there are 2 cases here.13:04
smoserone power-nv and one powerkvm.13:04
cjwatsonright, I'm not hugely familiar with the differences13:04
smoserthat makes 2 of us13:04
smoser:)13:04
cjwatsonpowerkvm is roughly equivalent to running qemu-system-ppc64?13:05
cjwatsonwith slof13:05
smoserwell, it *is* running qemu-system-ppc6413:05
cjwatsonright, so that's what I tested with in the past13:05
smoserright. and that reads prep for sure.13:05
cjwatsonpower-nv I confess total ignorance of13:05
caribou-/513:05
smoserright. that just means "bare metal" where ubuntu would be the hypervisor13:05
cjwatsonbut we ought to have the same partition/boot layout on both, right?13:06
smoserand the thing that loads that is 'petitboot'.13:06
cjwatsonchaining to grub?13:06
smoserremember the conversation with anton about petitboot reading grub ?13:06
cjwatsonno :)13:06
smoseroh no? you were on that... they talked abou it. i think that is what is happening.13:06
cjwatsonunless you mean the one really really early on13:07
smoseri *think* petitboot (which is a linux kernel + busybox) somehow finds things to boot.13:07
smoserand shows you a menu13:07
smoserand it labels them suspiciously similar to the way grub would.13:07
smoserbut you're most certainly not interacting with grub13:07
cjwatsonso, for prep, the general idea is that the prep partition is basically just a zoned-off chunk of disk, and a GRUB core image is basically dded into it13:08
cjwatsonany other boot loader that wants to chain to it should be looking up the prep partition by guid and then chaining it in the obvious naïve way13:09
cjwatsonI'm assuming that it is expecting the chained loader to be 32-bit BE13:09
smoserthat explaination of prep seems to fit with what i've seen. so i wasn't completely off.13:12
cjwatsonif we're now creating an ESP by accident, then it's possible that powerkvm is now broken13:12
cjwatsonor maybe slof handles that13:12
cjwatsonI don't see anything in my copy of slof/fs/packages/disk-label.fs that would though13:13
camakopitti, thanks for the quick action... much appreciated!13:15
infinitysmoser: Booting natively completely ignores that partition and just scans all mountable Linuxish partitions for grub and yaboot configs and mashes them together in petitboot.13:32
infinitysmoser: So, the PReP thing only matters for VMs, but we use it everywhere for consistency, and portability of disk images, and to not drive us batty.13:32
infinitysmoser: Oh, and the PReP thing also matter of booting natively as a PowerVM partition.13:33
smoserthats what i thought.13:33
smoserso i'm not sure why petitboot doesn't like my curtin based install13:33
smoserbut the new firmware delivered a much nicer petitboot with 'rescan devices' which is crazy useful.13:34
smoser:)13:34
infinityShiny.13:34
cjwatsonsmoser: oh so maybe you need to be diffing /boot/grub/grub.cfg and that kind of thing then?13:35
smosercjwatson, yeah.13:35
cjwatsonhadn't occurred to me that power-nv would do that!13:35
smoserwell i told you!13:35
smoserwhat i suspected13:35
smoserbut its still kind of yucky.13:36
smosermostly confirmed that they're parsing grub config13:36
smoserwhich as seen here could be buggy. its also possible that now my curtin changes would work (solved by my firmware upgrade)13:36
smoserright now i'm doing a d-i based maas install13:36
smoserand will get cjwatson the requested parted log.13:36
cjwatsonI guess I missed that, sorry13:37
cjwatsonI think I thought you mean "mounting partitions and looking <for a grub core image to chainload>", not "mounting partitions and looking <for grub configuration to parse>"13:38
=== Guest93672 is now known as kgunn
smoserok. so i'll get that system installed into d-i13:38
smoserand then i'll get cjwatson and infinity access to it.13:38
smosercjwatson can feel free to ignore that.13:39
smoserand infinity should dist-upgrade it to utopic13:39
smoserwhere in theory we have magic pixies and kvm support13:39
=== pitti` is now known as pitti
=== flexiondotorg_ is now known as flexiondotorg
=== mapreri_ is now known as mapreri
smosercjwatson, d-i install of utopic14:20
smoserhardware-summary: http://paste.ubuntu.com/8380127/14:20
smoserlsb-release: http://paste.ubuntu.com/8380128/14:20
smoserpartman: http://paste.ubuntu.com/8380129/14:20
smoserstatus: http://paste.ubuntu.com/8380130/14:20
smosersyslog: http://paste.ubuntu.com/8380131/14:20
cjwatsonhmm, the boot flag is already in place14:23
cjwatsonI wonder if that matters14:24
cjwatsonand whether it would behave differently on a blank disk14:24
Daryl_Hello peeps14:26
smoserinfinity, https://bugs.launchpad.net/ubuntu/+source/powerpc-ibm-utils/+bug/137162414:28
ubottuLaunchpad bug 1371624 in powerpc-ibm-utils (Ubuntu) "update_flash is /bin/sh but contains bash" [Undecided,New]14:28
=== FlannelKing is now known as Flannel
rbasakpitti: is postgresql-server-dev-all expected to stay in main for >utopic?14:29
rbasakWondering where to build-dep on that or a specific version (for bacula)14:29
pittirbasak: I wouldn't see it, if anything needs it it's fine in main14:29
pittirbasak: usually it's a build-dep of server-side extensions14:30
pittibut we have pretty much all of them in universe14:30
pittirbasak: bacula? I thought this was a db client, not an extension14:30
rbasakNo idea why it build depends on that, but it does.14:30
rbasakIt FTBFS right now because it depends on postgresql-server-dev-9.314:31
rbasakpitti: so I'm wondering whether to change it to postgresql-server-dev-all, or to -9.4.14:31
pittirbasak: ideally it should work with libpq-dev only14:31
pittirbasak: if it needs some extra server-side include file for poking in internals, then I think -dev-all should be ok14:32
om26er_jamespage, Hi! Can you please review/comment on https://code.launchpad.net/~om26er/ubuntu-test-cases/fix_minimal_image_size_test/+merge/235298 ?14:32
pittirbasak: (I mean within the realm of already being deep in hack territory :) )14:32
mapreripitti: what do you think about syncing scribus now? I uploaded 1.4.4 to sid (should go to testing at the next britney run) and I'd like utopic shipping it. I'm not sure if a FFE is needed, though...14:39
pittimapreri: it's two microreleases; most projects only fix bugs in microreleases14:43
pittimapreri: so a quick review of the changelog is in order (and testing a build on utopic, of course), but it's probably okay14:43
mapreripitti: yes, the most of the changes are bugfixes and really little changes. http://sources.debian.net/src/scribus/1.4.4%2Bdfsg1-1/ChangeLog/14:45
mapreripitti: do you prefer me upload it to a ppa before syncing?14:46
mapreri(I guess it builds fine, though :P)14:46
pittimapreri: yes, indeed14:46
pittimapreri: if you sbuild it locally, that's good enough14:46
infinitysmoser: Yeah, I have a Debian bug for that, fixing in the next upload.14:46
mapreripitti: to be honest, I never tried to build it in a utopic chroot, only in sid...14:47
pittimapreri: I'll give it a spin here, and sync if it works14:47
mapreripitti: great, thanks14:47
=== halfie_ is now known as halfie
=== henrix_ is now known as henrix
=== DalekSec_ is now known as DalekSec
=== sil2100_ is now known as sil2100
=== tarpman_ is now known as tarpman
=== roadmr_ is now known as roadmr_afk
=== Laney is now known as Guest37661
=== kees_ is now known as kees
=== DalekSec_ is now known as DalekSec
quadrisprohave a nice we all15:09
quadrisprociao!15:09
=== sil2100_ is now known as sil2100
=== lan3y is now known as Laney
=== Ursinha-afk is now known as Ursinha
=== Saviq_ is now known as Saviq
=== balloons_ is now known as balloons
=== debfx_ is now known as debfx
=== balloons is now known as Guest35774
=== jpds is now known as Guest26752
=== culot_ is now known as culot
=== pfsmorigo is now known as Guest91417
=== Trewas666 is now known as trewas
=== Guest35774 is now known as balloons_
=== Lutin is now known as Guest7993
=== lynxman_ is now known as lynxman
=== roadmr_afk is now known as roadmr
=== _morphis is now known as morphis
=== gusnan_ is now known as gusnan
=== FlannelKing is now known as Flannel
mhall119cjwatson: ok, I'm here now :-P17:03
mhall119cjwatson: so if we created a new click database, how could we install development packages to it without conflicting with installing regular packages from the store?17:04
cjwatsonmhall119: so I'd say stacking a test database on top temporarily would be the best answer, except that that currently requires root17:04
cjwatson(just to create the test db)17:04
cjwatsonactually let me just check that assumptio n17:05
mhall119cjwatson: could we create this when the user enabled developer mode and just leave it in place? or would that conflict with installing packages from the store?17:05
=== Guest91417 is now known as pfsmorigo
cjwatsonmhall119: I'm thinking something more transient than that17:05
mhall119zbenjamin_: ^^ this is related to our discussion earlier17:06
cjwatsonmhall119: as in something controlled by an environment variable, so it would be confined to just the process context of the SDK, and could be a temporary directory17:06
mhall119even better17:06
cjwatsonmhall119: so right now this isn't exposed in the CLI, although the library interface allows it - look at click/commands/*.py and notice that there's a db_dir=None param passed to db.read()17:07
cjwatsonoh actually17:07
cjwatsonclick install --root=<some directory>17:07
cjwatsonthat stacks a directory on top17:07
mhall119zbenjamin_: do we currently use click install of pkcon install-local?17:07
mhall119cjwatson: if we did that would it replace $HOME/.local/share/applications/<appid>.desktop?17:09
cjwatsonI don't know, this is a few too many questions to fire at me all at once17:09
cjwatsonprobably17:09
mhall119I suppose I should put this all into a doc17:09
cjwatsonthis might not be perfect right now but I think it is the correct direction17:09
cjwatsonI imagine you use pkcon install-local right now, which has the effect of running click install as root so that it can ensure that the app can't write to its own code17:10
cjwatsonyou probably do want to keep that part of it so that app authors can test in a properly confined environment17:10
cjwatsonthat said the packagekit-based interface is not long for this world; I'm writing a new native click d-bus interface17:11
cjwatsonbasically because packagekit upstream is killing plugins, see http://blog.tenstral.net/2014/09/listaller-back-to-the-future.html17:11
mhall119cjwatson: zbenjamin_: I'm adding these bits to the bottom of https://docs.google.com/a/canonical.com/document/d/1i5KRNoc-UXYxT3xFDl9YKETwaa0I2yIUG33Oi_pyeYc/edit17:15
cjwatsonmhall119: it might work for the SDK to just set $HOME17:18
cjwatsonto a temporary dir17:18
mhall119possibly, not sure what side-effects that might have on the app's runtime17:19
cjwatsonmhall119: right; but I think in general the SDK should be giving developers a clean test environment?17:20
mhall119I can see arguments for both ways17:23
cjwatsonI think if you go with the approach that they don't get a clean test environment, you will find it extremely difficult to satisfy the requirement of not conflicting with installing packages from the store17:26
mhall119yeah, I suppose the developer can come up with a way to pre-populate test data if they need it17:27
=== sarnold_ is now known as sarnold
=== balloons_ is now known as balloons
=== superm1_ is now known as superm1
=== roadmr is now known as roadmr_afk
=== zbenjamin_ is now known as zbenjamin
zbenjaminmhall119: cjwatson: right now we use pkcon to install the packages.19:49
zbenjaminmhall119: cjwatson: i would like to not need root privs for that, because the only way would be to somehow echo it into the commandline which kind of is ugly19:50
ogra_zbenjamin, not necessarily19:50
zbenjaminogra_: that means?19:51
zbenjaminogra_: using a dbus interface?19:51
ogra_adb shell "echo -e '#\x21/bin/sh\necho $PASSWORD' >/tmp/askpass.sh"19:51
ogra_adb shell chmod +x /tmp/askpass.sh19:51
ogra_adb shell SUDO_ASKPASS=/tmp/askpass.sh sudo -A <command>19:51
zbenjaminogra_: this is not ugly? ;)19:52
ogra_i admit you still need to echo the PW once that way19:52
ogra_but better than adb shell "echo $PASSWORD|sudo -S <command>"19:52
zbenjaminogra_: i would prefer to not echo the password around, for a real phone user its not just any password19:52
ogra_which yoou would need to do for every command19:52
ogra_will a real phone user use developer mode at all ?19:53
ogra_my mom wouldnt19:53
zbenjaminogra_: of course19:53
ogra_(she wouldnt even know what it meaans)19:53
zbenjaminogra_: of course not ;), but a developer using and developing for the phone would19:53
ogra_you could indeed dump a dbus interface in place19:54
ogra_but that means poking more security holes (even though they are small)19:54
zbenjaminyeah thats a problem :/19:54
zbenjaminogra_: i want transaction support in click ;)19:55
zbenjaminogra_: click rollback19:55
ogra_hah19:55
zbenjaminogra_: its kind of sad that there is no API that can elevate a process to higher rights in linux when you know the password, then i could just open a communication channel that is encrypted to send the password19:58
ogra_zbenjamin, lets discuss that at the sprint19:58
zbenjaminogra_: ok!19:58
ogra_dev mode as is is an awful hacked android dev mode, i think we can really do better19:59
zbenjaminogra_: yeah :)20:00
ogra_(with the right amount of time and planning involved)20:00
mhall119zbenjamin: what would you need root for?20:01
zbenjaminmhall119: click install needs root, and cjwatson suggested to create a temporary click db what would need root as well20:02
=== roadmr_afk is now known as roadmr
mhall119zbenjamin: he also said he's adding a d-bus interface to click so as to replace packagekit20:03
mhall119which I assume means we can run something as the normal user and have click install it as root20:03
mhall119just as pkcon install-local does today20:03
mhall119because we're going to have to support that for store installed apps too20:04
ogra_you will still need a way to elevate privs, authenticate or some such20:04
ogra_(or a key as zbenjamin said above)20:04
mhall119why?20:04
zbenjaminmhall119: yep, that also means to make the upstart jobs aware of the temporary database20:04
ogra_mhall119, because you dont want everyone to be able to do stuff with root privs on your device ?20:05
mhall119zbenjamin: why couldn't that just be something passed over dbus?20:05
mhall119ogra_: we want them to install packages20:05
ogra_right20:05
mhall119which they can today20:05
mhall119without root privileges20:05
ogra_yes, but if the desire is to do that with higher privs in the future20:05
ogra_which is how i understand the above20:06
mhall119I don't think we'llneed higher privs20:06
zbenjaminmhall119: could be , but the app still should be started confined and i would like to use a stable API/service for that so we do not break on every change. We had that before and it burned us badly20:06
mhall119zbenjamin: that's the goal, the only difference from the SDK's perspective would be that it tells the install command to install to /tmp/sdk-dev/ or something like that20:07
zbenjaminmhall119: yeah and start it of course20:07
mhall119yes that too20:07
zbenjaminmhall119: plus we need signals that tell me about the state of the app (at least started/stopped and exit code )20:07
mhall119zbenjamin: do we already have that?20:08
zbenjaminmhall119: yes upstart does that20:08
mhall119then it should be the same with cjwatson's replacement20:08
zbenjaminmhall119: you want to look at libupstart i think20:08
* mhall119 doubts that he really wants to :)20:09
zbenjaminok probably not ;)20:09
mhall119zbenjamin: you call upstart-app-launch or something like that right?20:10
mhall119I assume it uses envvars liek $HOME or $XDG_* to find the app to start20:10
zbenjaminmhall119: its ubuntu-app-launch now, and i use the backend library directly, not the frontend20:10
zbenjaminmhall119: uh tedg would be the right guy to ask that, it should be upstart jobs20:11
mhall119zbenjamin: I still assume there's a way to tell it where to look for apps20:11
mhall119ok, we'll find out from ted20:11
mhall119lucking all of this is code that we're writing, so the only real obstacle is developer time20:12
mhall119unfortunately the only real obstacle is developer time :(20:12
zbenjamin:/20:13
zbenjaminmhall119: only thing i'm concerned about is that with a change like that we would break compatiblity to older phone images.20:13
zbenjaminmhall119: we have no way of asking the phone if it has features like that, so the script can adapt20:14
zbenjamini hardly doubt we can use the same commands for both ways of installing a app, except only setting a env var would change the behaviour of all used commands20:14
tedgmhall119, I'm confused, what are you trying to do?20:30
mhall119tedg: we want to install a click package somewhere other than where the store installs and run the app from there21:05
tedgmhall119, Why?21:05
mhall119tedg: the case is app development, running an app project from the SDK currently installs it normally, which conflicts if you already have the app isntalled21:05
mhall119so I can't have the stable version of my app on my phone *and* test the development version on my phone21:06
tedgmhall119, Clicks are all per-user, so wouldn't it make more sense to just have it be another user account?21:06
tedgSo I could have my stable account and my dev account.21:06
tedgOr I could install the dev in my stable account right before shipping.21:06
mhall119tedg: and could we run it on the stable account session in Unity?21:06
tedgI'm confused. You'd be two users. You log in as one, or the other, or switch between them.21:07
tedgJust like all Ubuntu systems.21:07
mhall119tedg: we don't have multi-user on the phone, but even if we did I don't think that's a very developer friendly solution21:07
mhall119tedg: click allows (or will allow) the SDK to install the dev click package in a separate space, we would just need to launch it under confinement from that place21:08
tedgmhall119, We do have multi user, we just don't have UI for switching users. That's a solvable problem.21:08
mhall119tedg: they're all solvable problems, it's just a matter of which one is easiest/most appropriate to solve for this21:08
tedgmhall119, I don't understand why a) you wouldn't change the name (i.e. so it'd show up with two icons in the apps scope) or b) you wouldn't change the version, so that you could detect which you're running.21:09
tedgBoth of those options work today.21:09
mhall119tedg: the appid is used in multiple places, which makes it more complicated to change especially for scopes where it's in the source and binary filenames21:09
mhall119even if you change the version number you have to uninstall the currently installed one before you can run the dev one21:10
tedgmhall119, That's not true. It's all ref counted. Each user on the system can have different versions of the app.21:10
mhall119each user can, yes21:10
tedgSo you don't have to remove the old one.21:11
mhall119but I currently can only use one user21:11
tedgYou just have to say "this is the version I want"21:11
mhall119so here's my use case, I have uReadIt 1.0 installed on my phone. It works, I use it every day. But now I'm starting on 2.0, it won't be ready for months. I want to be able to test 2.0 on my phone while I develop it, but still be able to use 1.0 on the same phone until 2.0 is finished and in the store21:13
tedgSo you want to have two apps installed. uReadIt 1.0 and uReadIt 2.021:13
mhall119I really don't care if uReadIt 2.0 is installed, I just want to be able to run it on my phone from the SDK21:13
=== mnepton is now known as mneptok
mhall119under proper confinement21:14
tedgTo run under proper confinement it needs to be installed.21:14
tedgSo you want to have two apps installed.21:14
mhall119yes, one permanently and one temporarily21:15
tedgAnd so why can't they have different appids?21:15
tedgcom.ubuntu.developer.mhall119 and com.ubuntu.developer.mhall119.test21:15
tedgOr package names.21:15
mhall119with a QML app I have to change it in the manifest and in the MainView QML code21:15
mhall119for scopes I need to change it in the manifest, in the .ini file, and (somehow) in the resulting binary .so file21:16
mhall119for C++ apps or plugins I might need to change it other places21:16
tedgMy argument would be that we should fix that. It should be set in one place, the manifest.21:16
tedgFor instance Unity Actions gets it from teh APP_ID environment variable.21:16
mhall119it needs to be in the QML app to tell (I guess) Unity what app the window belongs to21:17
mhall119it also seems to be used by content-hub to identify the app21:17
tedgSure when it creates the Mir connection.21:17
tedgYup, but that's just for content hub to match to the manifest.21:17
mhall119I'd imagine Online Accounts also wants it from the QML21:17
mhall119or maybe it uses the manifest files only21:17
tedgNo, I think it gets it from the apparmor profile it's running under.21:18
tedgChecks the dbus connection.21:18
mhall119at any rate, I'd rather not make the developer change their app name just to run it21:18
mhall119unless it can be done automatically and transparently21:18
=== enrico_ is now known as enrico
tedgI think it should be done automatically and transparently. I'd rather make QtCreator complex and our phone simple.21:18
tedgIt's easier to upgrade QtCreator :-)21:19
tedgBecause complexity leads to bugs.21:19
mhall119tedg: but, if we can properly install it as a click package of the same name, just in a different location, why wouldn't that work with ubuntu-app-launch and confinement?21:19
tedgmhall119, If it has the same name, which do we run?21:19
mhall119tedg: how do you decide currently?21:19
tedgWhat happens if you run both at the same time?21:19
mhall119how would you?21:19
tedgThe App IDs are guaranteed unique.21:20
tedgWe assume that everywhere.21:20
mhall119tedg: unique per app, not unique per install21:20
tedgThere is only one of each appid in the session.21:20
mhall119tedg: you can have pre-installed clicks in /usr/share/click/preinstalled and store-installed updates in /opt/click.ubuntu.com/ can't you?21:21
tedgWe have hashtables and db tables that use it as a primary key.21:21
tedgCorrect, but only one of those is the running one for the current user.21:21
tedgSo there's one version of each package for the current user.21:21
tedgWhether the system has one or a thousand.21:21
mhall119tedg: so could we temporarily make the current user look in /tmp/click/ first, then /opt/click.ubuntu.com/ then /usr/share/click/preinstalled ?21:22
tedgOr you could just install the click, and then remove it when done.21:22
tedgIt'd be the same effect.21:22
mhall119tedg: install current+1, then uninstall only that version and it will still have current?21:22
tedgThere can be only one of that package name. So if you install uReadIt 2 with the same package name as uReadIt 1, the 1 doesn't exist.21:23
tedgYes21:23
mhall119it'll exist in /usr/share/click/preinstalled still though won't it?21:23
mhall119or is that only because it's installed there as root?21:23
tedgPreinstalled, yes, always. For the ones in /opt it'd be refcounted.21:23
mhall119so if I have 1.0 installed as phablet from the store, and I install 2.0 as phablet from the SDK, it'll remove 1.021:24
tedgSo if there's only one user it could be dropped and uninstalled. But, we could make a dummy user to keep a ref.21:24
mhall119tedg: is there no way we can add another click database to /tmp/click/ and let the user session know about it?21:25
tedgThere is not no way, it's software. There's no reason to do that.21:26
mhall119cjwatson, if I understand him correctly, says we can make click install to /tmp/click by passing it an additional argument21:26
tedgBesides the fact that cjwatson is always correct, he's correct in this case too. That doesn't make it a good idea.21:27
mhall119it sounds better than having to implement multi-user UI and making me switch accounts whenever I want to hack on my app21:29
tedgI'm not saying you have to do that, though I do like that from the developer experience perspective.21:29
tedgAll you have to do is install the app you want to test, and uninstall it when you're done.21:29
mhall119and switch accounts21:30
tedgNope, you'll have the new version as soon as you install the new version.21:30
mhall119so I can install 2.0 as phabletdev and run 2.0 as phablet?21:30
tedgNo21:30
tedgEach user has different versions21:31
mhall119then I have to switch accounts, no?21:31
tedgYou'll have to install 2.0 as phablet.21:31
tedgIf you want different accounts with different versions, you'll have to run as the account that has the version you want.21:31
mhall119and install 1.0 as phabletdev?21:31
tedgIf you want.21:32
mhall119if I don't 1.0 will be removed, no?21:32
tedgSure, but you never need to log into that account. I think there are other ways to keep references, but I'm not as sure there.21:32
tedgWe just need to keep a reference.21:32
mhall119and then if I install uReadIt 2.0 as phablet, and then uninstall 2.0 as phablet, I will revert back to having 1.0 as phablet, or will I know long have uReadIt as phablet?21:33
mhall119s/know/no/21:33
tedgWell, technically if you run "uninstall" you won't have version. But you can "install 1.0" and you'll switch to 1.0. 2.0 will loose refs, and then be removed.21:34
mhall119so the SDK would need to remember the previous installed version and re-install that21:35
tedgSo the real answer is, yes you can go back to 1.021:35
tedgYes21:35
tedgIt can discover it as well.21:35
mhall119how?21:35
tedgAnything that is not confined can get a list of versions on teh system.21:35
tedgI'll have to look up the command, but click provides it.21:35
mhall119but on actual multi-user systems it would need to know the last version that user had21:36
tedgSure, if there was a chance of there being more than one installed.21:36
tedgI guess there could also be a preinstalled and an upgraded version as well.21:37
mhall119tedg: can you add your recommendation to the bottom of https://docs.google.com/a/canonical.com/document/d/1i5KRNoc-UXYxT3xFDl9YKETwaa0I2yIUG33Oi_pyeYc/edit so we don't lose it please?21:37
mhall119it seems we're going to need to go through different options in detail21:37
tedgmhall119, No rights :-)21:38
mhall119tedg: refresh and you should now21:38
tedgmhall119, Updated21:42
mhall119thanks tedg21:43
dobeyinfinity: around?21:46
* dobey needs a debdiff sponsored21:56
dobeyslangasek: or if you could sponsor. i've attached a debdiff to https://bugs.launchpad.net/ubuntu/+source/libjsoncpp/+bug/1368420 which i need to get into utopic. thanks.21:58
ubottuLaunchpad bug 1368420 in libjsoncpp (Ubuntu) "Prices shown incorrectly in some locales" [Critical,In progress]21:58
NoskcajThe package bowtie is currently FTBFS on !amd64. debian (and upstream) has dropped building on those arches. Should we sync the change that stops attempting builds on the failed arches?22:15
cjwatsonzbenjamin: to be clear I'm not suggesting that you should need root privileges; I think you misunderstood me22:42
cjwatsontedg: yes, right now if you install a new version of an app the old one will immediately be vulnerable to GC; but we could indeed suppress that somehow22:45
cjwatsontedg: the other bit is that as soon as you install a new test version of an app it would be the thing that shows up in your desktop in general as the current version.  perhaps a good approach would be to have subusers22:46
cjwatsontedg: so you could have click package versions owned by cjwatson:sdk22:46
cjwatsonwith whatever separator22:46
cjwatsontedg: then those could have different semantics wrt hooks and such22:47
=== salem_ is now known as _salem
slangasekdobey: what's the upstream status of this libjsoncpp patch?23:03
=== _salem is now known as salem_
dobeyslangasek: not upstreamed yet (i'd just finished preparing it).23:55
=== chiluk` is now known as chiluk

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