/srv/irclogs.ubuntu.com/2014/06/12/#ubuntu-quality.txt

balloonsphillw, see my mailing list post00:08
balloonselopio, I'm here atm.. likely you aren't :-)00:08
elopioballoons: I am.00:09
elopioI'm wondering about the filemanager case. If I patch the home with a directory in /home/phablet, the app can't be launched. But if I patch it with a dir in /tmp, it works.00:09
elopioso, why do we have to create all that complex dirs hierarchy instead of using /tmp ?00:10
balloonselopio, fm is unconfined.. it's not a good case00:13
elopioballoons: hum00:13
phillwballoons: I'll go dig it up... I've had the most amazing thing happen....00:13
phillw Hi, Thanks! I want to learn a little more about this flavor of ubuntu! I am a teacher in a school that has many obsolete computers. I would like to use it there.00:13
phillwA teacher in Mexico. Our pre official 10.10 was the last time I was asked about using lubuntu on low spec kit.00:13
balloonsphillw, that's an awesome story00:14
balloonsI'm so happy you did the session, and really glad to hear the story!00:14
phillwAnd that, balloons, is why I stick with lubuntu. My local youth team have a chance to have a computer lab with two old computers.00:15
balloonselopio, so yea since fm is unconfined none of the apparmor stuff is needed; it can do whatever and apparmor won't complain00:15
phillwThat man, will get what ever I can do for his school.00:15
elopioballoons: well, it crashes if we use a temp home inside the real home.00:15
balloonselopio, I seem to remember that as well.. not sure why00:18
balloonsbut yea, I left the apparmor stuff on the table.. it works atm for calendar.. it's in production even00:19
balloonsmusic has been a complex pain. Filemanager should convert over painlessly I hope. Still it's pretty crazy for what we want to do00:19
balloonselopio, I'm off to walk the puppy. We can chat more about it, but suffice to say for now the calendar can serve as an example of what will work with apparmor00:21
elopioballoons: yes, don't worry. I was just wondering about the filemanager case. Your explanation was good, thanks.00:22
balloonsI'm all for simplifying it, but Jamie explained how locked down the apps are, while still expecting a certain layout etc00:22
balloonsthe expected layout is what we are forced to reproduce since we are changing /home for them00:22
pittiGood morning03:54
jibelpitti, with latest autpkgtest there are again quoting issue with ssh because ssh executes the command with sh -c, it'll be the same with adb. But if I systematically quote the scripts it breaks the other drivers.09:24
pittijibel: what does your auxverb look like now?09:25
pittiI suppose something like09:25
pitti#!/bin/sh -e09:25
pittiexec ssh ... "$@" ?09:25
pittior does that need double quoting?09:25
jibelpitti, yes something like that but I probably did something wrong, rechecking.09:45
pittijibel: ah, this would indeed need double quoting then09:45
jibelpitti, did you change something in the debug output of VirtSubproc, nothing is display with debug mode enabled09:45
jibel?09:45
jibelfor example: sudo ./run-from-checkout -dd libpng --- lxc -d adt-amd6409:46
jibelshould display lines like <VirtSubproc> ...09:46
pittijibel: I just committed some refactoring for debugging, yes (~ 10 mins ago)09:47
pittiso if you rebased in the past 10 mins, then yes09:47
=== vrruiz_ is now known as rvr
pittijibel: it's a prerequisite for refactoring the debian/test/control parsing; I need to put that into a separate class so that we can also get a parser for click metadata09:47
pittijibel: it now displays the program name first, i. e. "adt-run:" (as it always used to), or "adt-virt-lxc: DBG:"09:48
jibelokay, I don't have the latest version, I'll update it09:49
pittijibel: ah, if you don't, it should also show debug info of course09:49
pittijibel: it would only affect your new ssh runner if that uses VirtSubproc.debug()09:49
jibelit didn't09:49
pittidon't worry about that for now, too many variables at once I suppose :)09:49
jibelpitti, so exec ssh -p 22 root@10.0.3.125 -- $(printf '%q ' "$@")10:15
jibeldid the trick10:15
pittijibel: ah, nice!10:15
jibelI still have an error: "sh: 3: cd: can't cd to ."  when it executes the script that fetches the source, searching why . is not accessible10:17
jibelah, it's because 'cd' is executed via su and ubuntu doesn't have access to /root10:41
pittijibel: whoops, that sounds like something which should be fixed in adt-run proper?10:48
jibelpitti, not sure, it's happening only with ssh10:48
pittijibel: ah, that's because ssh defaults to cd $HOME, while lxc-attach and friends don't; but schroot ought to have the same problem, hmm10:53
pittisu ought to change it though10:54
jibelpitti, su -l fixes it10:59
jibeltestbed command ['sh', '-ec', 'su -l -s /bin/sh ubuntu -c \'exec  ...10:59
jibeland can probably remove -s /bin/sh too unless there is a reason to force the shell11:00
pittijibel: probably, yes; that runner shouldn't run as "nobody" or similar disabled accounts11:01
=== ara is now known as Guest77001
=== _salem is now known as salem_
=== Ursinha is now known as Ursinha-afk
balloonsbrendand, time to setup your session :)13:50
balloonsbrendand, have a look at https://wiki.ubuntu.com/UDS/Sessions. you can do it all yourself13:51
=== Ursinha-afk is now known as Ursinha
=== salem_ is now known as _salem
=== hobgoblin is now known as elfy
=== _salem is now known as salem_
=== roadmr is now known as roadmr_afk
=== roadmr_afk is now known as roadmr
=== salem_ is now known as _salem
elopioping ubuntu-qa: somebody available for a review?22:02
elopiohttps://code.launchpad.net/~elopio/ubuntu-calculator-app/swipe_to_delete/+merge/22300122:02
veeberselopio: is self.waitingConfirmationForRemoval.wait_for(True) doing the same as screen.confirm_removal() ?22:05
elopioveebers: no. The first one waits for the confirmation to appear.22:05
elopiothe  second one clicks it.22:05
veebersah ok, makes sense22:05
elopioveebers: well, it could be clearer. But that comes from the QML code.22:06
veebersack22:06
veeberselopio: where do the numbers on diff line 75-77 come from / are calculated?22:06
veebershmm nvm22:07
elopioveebers: trial and error. The labels section takes less than 1/3 of the screen.22:07
elopioand you have to swipe for at least half of the width.22:07
elopioso 1/3 to 1/3 + 1/222:07
veeberselopio: trial and error foe those numbers or for working out how much of a swipe to make?22:08
elopioveebers: for both. I just found that the default which is: swipe from almost the left side to almost the right side doesn't work for the calculator.22:09
elopioso I moved a little to the left, and a little to the right until I understood how it works.22:09
veeberselopio: would that indicate an issue with the calc app? i.e. if a user is used to swiping in one way then tries it with calc and it doesn't work?22:09
elopioveebers: yes, probably. It was really hard for me to do it.22:10
elopionow that I understand it, it makes sense. But you are right.22:10
elopioI'll report a bug to UX.22:10
elopioI suppose it's possible to identify if it's a drag instead of a tap.22:11
veeberselopio: cool, other than that the code looks good to me, I'm not setup to run and test the code right now though sorry :-\ But I can give my LGTM if you like22:12
elopioveebers: yes, that's fine. Thanks for your review.22:12
elopioI'll wait for jenkins to confirm before approving it.22:12
veeberselopio: although, if you're filing a bug to UX you should link it in a comment for the "_drag_pointing_device_to_delete" method22:12
robotfuelelopio: I marked it as lgtm22:12
elopioveebers: I'll do that.22:13
veeberselopio: awesome :-)22:13
veebershey robotfuel how goes things?22:13
elopiothanks robotfuel.22:13
robotfuelhi veebers, better than yesterday.22:14
veebersrobotfuel: that's good to hear. Hopefully you feel 100% again real soon22:14

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