| SergioMeneses | hi guys! | 00:49 |
|---|---|---|
| === salem_ is now known as _salem | ||
| === _salem is now known as salem_ | ||
| === salem_ is now known as _salem | ||
| jibel | good morning | 06:57 |
| DanChapman | hmmmm autopilot not working today. Has anything changed? | 09:20 |
| Noskcaj | DanChapman, wrong channel to ask | 09:22 |
| balloons | knome, can we merge your change without the css fix? | 12:16 |
| DanChapman | good day balloons :D | 12:51 |
| balloons | morning DanChapman | 12:51 |
| DanChapman | balloons is there plans to create something like a mini tutorial series on autopilot for the QA wiki/website? | 13:27 |
| balloons | elfy, so what do you mean what would you do with the bugs assigned to you? | 13:28 |
| balloons | once the stuff is merged, you can close it ;-) | 13:28 |
| balloons | DanChapman, I did a mini-series on my blog last winter, but your best served by the official docs now | 13:29 |
| balloons | that said there is a walkthrough on the wiki | 13:29 |
| elfy | balloons: see - you're doing it again ;) change status to fix committed, fix released? won't fix :p | 13:30 |
| balloons | we could certainly improve it -- any specific feedback you have? | 13:30 |
| balloons | elfy, in general we've used the nomenclature as follows; fix committed is for when you have committed it.. fix released is when it's merged | 13:30 |
| elfy | right - thanks | 13:31 |
| balloons | basically fix released is what is set after a merge.. that and people mark that they are working on things | 13:31 |
| elfy | so 2 released and one still waiting merge then afaik | 13:31 |
| balloons | elfy, everything should be merged from you.. there only one pending merge request I know of for the project now | 13:31 |
| elfy | xchat looks like it is still at the needs fixing that you set - that I'd merged or pushed or whatever it's called :) | 13:32 |
| elfy | here - all I am working on is thunar | 13:32 |
| elfy | https://code.launchpad.net/~elfy/ubuntu-manual-tests/xchat | 13:33 |
| balloons | elfy, ohh bother.. since you had it included in those other proposals it's status got set to merged | 13:35 |
| elfy | oh ok | 13:35 |
| elfy | sorry and all that :) | 13:36 |
| balloons | I undid it's autologic :-) | 13:36 |
| balloons | anything else that might have gotten swept up? | 13:36 |
| elfy | I guess that's one of your Canonical words that mean little to me - like dogfood - dogfood? don't you lot make them scavenge :p | 13:36 |
| balloons | lol.. dogfooding is a fun term | 13:37 |
| elfy | nope - here I'm only working on thunar - mosuepad, xchat and catfish are all doen as far as I'm concerned | 13:37 |
| balloons | I just made that up.. something that might more sense to you is I reset the status of the merge request to 'needs review' instead of merged. So we can continue to work on it till it's ready | 13:37 |
| elfy | balloons: I guessed as much :) | 13:37 |
| balloons | did you push the changes I asked for on xchat? they don't show in your branch | 13:38 |
| === _salem is now known as salem_ | ||
| elfy | http://bazaar.launchpad.net/~elfy/ubuntu-manual-tests/xchat/revision/111?start_revid=111 | 13:39 |
| elfy | that's what I did - the last two lines that then close it without prompting | 13:39 |
| balloons | odd.. ok, I'll assume it's in the branch then | 13:44 |
| balloons | let me pull | 13:44 |
| elfy | well if it all goes belly up I do have it here still | 13:45 |
| balloons | ugh | 13:50 |
| elfy | did I break launchpad then | 13:55 |
| balloons | done | 13:56 |
| balloons | you can't break it elfy :-) | 13:56 |
| elfy | mmm | 13:57 |
| elfy | that a challenge? :p | 13:57 |
| balloons | no please! | 13:57 |
| elfy | :) | 13:58 |
| elfy | so we all sorted now - me goes to look | 13:58 |
| elfy | balloons: ok so at some point it will disappear from my branch yes - and then I can remove it all locally | 14:00 |
| balloons | it's in the main branch now, you can remove your branches with it | 14:00 |
| balloons | any new work you want to do, grab trunk | 14:00 |
| balloons | DanChapman, so evince is ready right? | 14:07 |
| * balloons notes he still gets errors :-( | 14:11 | |
| DanChapman | fileroller not evince | 14:11 |
| DanChapman | im stilll wroking on evince but autopilot not playing nice till next update i believe | 14:12 |
| === salem_ is now known as _salem | ||
| balloons | DanChapman, ahh, ok | 14:13 |
| balloons | I'll checkout fileroller then in a second | 14:13 |
| balloons | DanChapman, ahh, you added cleanup steps.. fun | 14:17 |
| balloons | DanChapman, could we move the hardcoded filename to a generated temporary filename instead? In addition, this would make files in /tmp instead of /home.. which is nicer | 14:18 |
| balloons | Otherwise good stuff.. works fine on my box | 14:22 |
| SergioMeneses | morning guys! | 14:26 |
| balloons | morning SergioMeneses | 14:27 |
| SergioMeneses | balloons, hey bro! I was reading about the new bugs and I'm going to work on some of them | 14:28 |
| balloons | SergioMeneses, excellent.. Yes chilicuil has been busy getting them all filed. I think everything is in now right chilicuil ? | 14:28 |
| balloons | we should announce progress.. I'll try and do that before the end of the week | 14:28 |
| SergioMeneses | I saw that, and excellent work :) | 14:29 |
| DanChapman | balloons: yeah i can get that changed. apart from that it works fine though? | 14:31 |
| balloons | DanChapman, yes, otherwise works fine | 14:32 |
| chilicuil | hello balloons, SergioMeneses, good morning =), it's almost, today I'm filling lubuntu bugs, and after that it'll be only mythubuntu left | 14:32 |
| DanChapman | sweet :) | 14:32 |
| balloons | the hacky workaround is the only thing that makes me nervous.. but it seems to hold up fine | 14:32 |
| balloons | I'm ok with it going in as is.. However, I was looking at the terminal test. I think I'm going to file a bug to fix the flaws before I consider it production ready | 14:33 |
| balloons | we have to remember once these are in jenkins, if they fail we've got to be on top of them seeing why and filing bugs for anything that is discovered.. we don't want our tests failing for silly timing issues :-) | 14:33 |
| SergioMeneses | chilicuil, awesome! I will work at night on testcases, if you need some kind of help, please let me know :D | 14:34 |
| balloons | I pushed gedit and nautilus to the production branch.. | 14:34 |
| balloons | I'll push fileroller and evince too once they merge | 14:34 |
| SergioMeneses | balloons, gedit testcase or autopilot test? | 14:35 |
| balloons | this is all autopilot stuff yes :-) | 14:35 |
| chilicuil | SergioMeneses: yep, thanks, good luck with your day =) | 14:35 |
| DanChapman | yeah, i dont like it either, but what else can be done? .... and true about the terminal test, i only proposed so it could go into the dev branch, i didnt expect it would be production ready ::) | 14:35 |
| SergioMeneses | balloons, ok | 14:36 |
| balloons | DanChapman, yes, the hacky workaround is just that. I'd rather have the test than hold on it.. The terminal testcase though, as you say, can be improved further, so we should do that before migrating | 14:37 |
| SergioMeneses | in other things: chilicuil we are working on ubuconla2014 maybe you want to see what we got :) | 14:37 |
| balloons | I'm just excitied they are all getting completed and we have production ready tests finally ;-) | 14:37 |
| DanChapman | balloons with the hacky workaround what about getting some asserts on the GtkLocation so it can assess the file path as it navigates to the sample data? that should make it 'a bit' more stable? | 14:37 |
| balloons | DanChapman, based on your comment, you couldn't introspect that.. can you? | 14:38 |
| DanChapman | Yeah its not the introspection thats the problem. Its if you enter a file path into it it doesn't navigate to it. | 14:40 |
| chilicuil | SergioMeneses: I didn't know you were already working on ubuconla 2014!, wow, I'd really love to see what you have and help if possible | 14:43 |
| DanChapman | balloons there seemed to be only two ways to add files, either my hacky workaround or navigate with a mouse. | 14:43 |
| balloons | DanChapman, ahh.. so your saying after you navigate to it, you could assert that your opening the right thing.. but even if you weren't there's nothing you could do to fix it | 14:48 |
| balloons | I wouldn't worry about it :-) | 14:48 |
| DanChapman | I was thinking that as you go through the directory in the location bar it updates your current position so if it was to assert say that we are now in /usr then assert we went in to /usr/share at least with that if the test fails during this point we could see which step it is failing on? | 14:50 |
| DanChapman | I'll leave it for now but if it does become an issue then i will look into it. Will just get tempfile working for now | 14:51 |
| balloons | DanChapman, yea, I don't think it's worth the trouble to assert so much.. but you could. If it fails during the hack, well, i assume it will be obvious | 14:52 |
| balloons | but then again, it's not going to fail ;-) | 14:52 |
| DanChapman | of course not :D | 14:52 |
| DanChapman | Is your autopilot working ok today then? | 14:53 |
| balloons | yes, but I haven't updated yet today -- it's broken for you? | 14:54 |
| balloons | that shouldn't happen if your on stable | 14:54 |
| DanChapman | I ran update this morning and its now broken. veebers was working on a fix but said it wont be available till later | 14:54 |
| balloons | DanChapman, ahh you can rollback then if you'd like | 14:54 |
| DanChapman | ooo yes please. How do i do that? | 14:55 |
| balloons | you on saucy? | 14:55 |
| DanChapman | no raring | 14:55 |
| balloons | k, and your using the ppa right? | 14:56 |
| balloons | https://launchpad.net/~autopilot/+archive/ppa/+files/autopilot-desktop_1.3daily13.05.30ubuntu.unity.nextbzr230raring0_all.deb | 14:58 |
| balloons | you can always manually grab the previous version.. I believe that will roll you back one | 14:58 |
| === Ursinha is now known as Ursinha-afk | ||
| DanChapman | sorry phone call, yeah using the ppa | 15:12 |
| DanChapman | ok will try grabbing one | 15:12 |
| DanChapman | balloons still getting the same errors. Traceback (most recent call last): | 15:16 |
| DanChapman | File "/usr/lib/python2.7/dist-packages/autopilot/utilities.py", line 242, in action_on_test_start | 15:16 |
| DanChapman | obj.on_test_start(test_instance) | 15:16 |
| DanChapman | File "/usr/lib/python2.7/dist-packages/autopilot/globals.py", line 108, in on_test_start | 15:16 |
| DanChapman | if _video_logger._enable_recording: | 15:16 |
| DanChapman | AttributeError: '_VideoLogger' object has no attribute '_enable_recording' | 15:16 |
| balloons | hmm | 15:16 |
| balloons | I'm using: 1.3daily13.05.29ubuntu.unity.nextbzr228saucy0 | 15:16 |
| balloons | try grabbing the raring version of that :-) | 15:17 |
| balloons | https://launchpad.net/~autopilot/+archive/ppa/+files/autopilot-desktop_1.3daily13.05.30ubuntu.unity.nextbzr228raring0_all.deb | 15:17 |
| DanChapman | ok will give it a try | 15:17 |
| balloons | bah.. the hack failed | 15:17 |
| DanChapman | :O really?? | 15:18 |
| balloons | https://launchpad.net/~autopilot/+archive/ppa/+files/autopilot-desktop_1.3daily13.05.29ubuntu.unity.nextbzr228raring0_all.deb | 15:18 |
| balloons | I meant my link.. I didn't look, I just changed the old link ;-) that one should work ^^ | 15:18 |
| DanChapman | ok cheers :) | 15:18 |
| DanChapman | balloons, thats a no aswell :( | 15:21 |
| balloons | does autopilot from there not work? | 15:22 |
| DanChapman | no im getting atrribute errors saying keyboard has no attrib 'cleanup' and _VideoLogger errors. | 15:22 |
| balloons | ohh | 15:24 |
| balloons | you need the other deb too | 15:24 |
| balloons | https://launchpad.net/~autopilot/+archive/ppa/+files/python-autopilot_1.3daily13.05.29ubuntu.unity.nextbzr228raring0_all.deb | 15:24 |
| DanChapman | woohoo, cheers balloons :D just need to go get vis and im good to go for now | 15:28 |
| balloons | you need the vis link too? | 15:29 |
| DanChapman | if you got it handy :) | 15:29 |
| balloons | https://launchpad.net/~autopilot/+archive/ppa/+files/python-autopilot-vis_1.3daily13.05.29ubuntu.unity.nextbzr228raring0_all.deb | 15:30 |
| DanChapman | awesome, cheers :D | 15:30 |
| balloons | wouldn't want you to be stuck for such a silly thing :-) | 15:31 |
| DanChapman | :-) can get evince finished off now. | 15:32 |
| elfy | balloons: ok - I am completely lost again - have no idea what I should be doing to get rid of that xchat thing in my branch | 15:55 |
| balloons | elfy, ok so your working on thunar now right? | 15:58 |
| elfy | not yet :) | 15:58 |
| elfy | intend to | 15:58 |
| balloons | ohh.. so all your work is in the main branch? | 15:58 |
| balloons | if so, this is the perfect time | 15:58 |
| elfy | hang on | 15:59 |
| elfy | I have a folder in manualtests/thunar/ into which I have pulled or whatever so now have /testcases and .bzr in there | 16:00 |
| elfy | I DID have a folder the other day that I must have pushed - which is showing up on my branch | 16:00 |
| elfy | I have got rid of the thing that had xchat in | 16:00 |
| balloons | right | 16:01 |
| balloons | so the workflow is to always start with bzr branch lp:ubuntu-manual-tests | 16:01 |
| elfy | which I did - hence having the testcase etc in there | 16:02 |
| balloons | do that and you can't go wrong.. just make sure you seperate out your work | 16:02 |
| elfy | the branch I pulled has the 4 latest tests in it - so I know that is up to date | 16:02 |
| elfy | so - first question is - I will create at some point my thunar thing - so do it in the root of that @ manualtests/thunar or in manualtests/thunar/ubuntu-manual-tests/testcases/ | 16:04 |
| elfy | then second question is how do I get rid of the 2 things sitting in my branch :) | 16:05 |
| balloons | elfy, ok so what do you mean the 2 things sitting in your branch? | 16:12 |
| balloons | and yes you can make a new file for thunar in testcases/packages/thunar | 16:12 |
| balloons | then use bzr add to add it, etc, etc | 16:12 |
| elfy | https://code.launchpad.net/~elfy | 16:12 |
| elfy | there are 2 there - an xchat one and a Thunar one from the other day - neither actually exist here | 16:13 |
| balloons | right, you removed them.. do you want them anymore? normally when they are merged launchpad automatically updates the status foryou | 16:13 |
| elfy | don't want them anymore | 16:14 |
| balloons | change the dropdown to "any status" | 16:14 |
| balloons | you'll see what I mean | 16:14 |
| elfy | oh yea - see the other 2 now | 16:14 |
| balloons | right, so the xchat one got messed up.. it didn't trigger the 'merged' status | 16:14 |
| balloons | you can fix manually: https://code.launchpad.net/~elfy/ubuntu-manual-tests/xchat | 16:14 |
| elfy | yea | 16:14 |
| balloons | go set the status in there | 16:14 |
| balloons | the thunar one will get set to 'merged' once it too goes in | 16:15 |
| balloons | if you decide you don't want it, you can manually set the status to abandoned | 16:15 |
| elfy | I'm going to abandon that and start again I think | 16:15 |
| balloons | kk | 16:16 |
| elfy | yay - ok - blank slate | 16:16 |
| elfy | now I have a blank file ready for me to work on Thunar in /packages | 16:16 |
| elfy | so bzr add? | 16:16 |
| DanChapman | afternoon elfy :D | 16:17 |
| elfy | DanChapman: hello - sigh ... :p | 16:17 |
| elfy | this is daft - it's hardly rocket science :) | 16:17 |
| elfy | balloons: so now I have a blank thunar sittting in the folder with the other real testcases | 16:18 |
| DanChapman | you still having troubles? | 16:18 |
| balloons | elfy, yep sounds like your ready to rock | 16:18 |
| balloons | bzr status will show you your one unknown file | 16:18 |
| elfy | yep unknown | 16:19 |
| balloons | knome, ping when your around. I'd like to tackle your giant merge today | 16:20 |
| === Ursinha-afk is now known as Ursinha | ||
| elfy | balloons: ok - thanks - that looks right here and there :) | 16:30 |
| balloons | "=_ | 16:33 |
| balloons | :-) | 16:33 |
| elfy | sorry to be a pita - but there you go, such is life sometimes | 16:34 |
| balloons | no worries mate | 16:40 |
| elfy | :) | 16:45 |
| elfy | what I'm really wanting to do is just get on and write something :p | 16:46 |
| elfy | was looking at autopilot this morning - for the items in Xubuntu main menu - 22 got the autopilot module not loading error | 16:47 |
| balloons | elfy, did some of them work? | 16:49 |
| elfy | balloons: yea - table at bottom of page https://wiki.ubuntu.com/Xubuntu/Roadmap/Specifications/Saucy/AutopilotTesting | 16:49 |
| elfy | of course it's possible that the fails would work if you got ther right combiantion of -i | 16:50 |
| elfy | and I'd guess that someoen will look at tbird so that would be a Y in the end - most of the outright fails were all xfce4-'thing' | 16:51 |
| elfy | someone who doesn't think python is a snake is looking further for us | 16:51 |
| elfy | still got no idea why someone named a language after pot hed pixies | 16:52 |
| elfy | hi SergioMeneses | 16:59 |
| elfy | can I ask how the gnumeric manual testcase is? | 16:59 |
| SergioMeneses | elfy, hi hi | 17:00 |
| SergioMeneses | I saw your message but I was in work time :S | 17:00 |
| elfy | k | 17:00 |
| SergioMeneses | elfy, it is quite :S do you want to work on it? this weekend I'll work on some testecases | 17:00 |
| SergioMeneses | maybe you can join and have a good time :D | 17:00 |
| elfy | I'm looking at thunar at the moment - not wanting more than one at a time :) | 17:00 |
| SergioMeneses | elfy, wise words | 17:00 |
| SergioMeneses | but you can assign more than one but peer person | 17:01 |
| elfy | someone else is looking at xubuntu ones too | 17:01 |
| SergioMeneses | it is not a big deal | 17:01 |
| balloons | elfy, nice chart | 17:01 |
| SergioMeneses | elfy, now I'm updating my Lubuntu images | 17:01 |
| elfy | SergioMeneses: I'm having more than enough trouble with LP and all this thanks - without confusing me and then me confusing balloons later :p | 17:02 |
| SergioMeneses | balloons, elfy which? | 17:02 |
| SergioMeneses | *chart | 17:02 |
| elfy | balloons: I hate wiki - took me longer to do that than to see if they ran | 17:02 |
| SergioMeneses | jajajaja | 17:02 |
| balloons | so as far as what's next with the autopilot stuff, we need to bring it up with pitti and the autopilot decs | 17:02 |
| balloons | *devs | 17:02 |
| SergioMeneses | elfy, which wiki? | 17:02 |
| elfy | SergioMeneses: autopilot stuff that fails for xubuntu apps - https://wiki.ubuntu.com/Xubuntu/Roadmap/Specifications/Saucy/AutopilotTesting | 17:02 |
| balloons | pitti has agreed specifically to look after issues that we have with autopilot-gtk.. and if we can get support added for the apps or not | 17:02 |
| SergioMeneses | elfy, ++ | 17:03 |
| balloons | SergioMeneses, I'm talking about this: https://wiki.ubuntu.com/Xubuntu/Roadmap/Specifications/Saucy/AutopilotTesting | 17:03 |
| SergioMeneses | guys lunchtime here! I'll be back soon :D | 17:03 |
| elfy | sounds like a good idea :) | 17:04 |
| balloons | ohh lunch. | 17:07 |
| balloons | me too! | 17:07 |
| elfy | :( | 17:07 |
| elfy | dinner time and I'm too lazy | 17:07 |
| === psivaa is now known as psivaa_afk | ||
| njin | hallo QA, can someone test ubuntu-docviewer-app for bug 1176592 | 18:07 |
| ubot5 | bug 1176592 in Ubuntu Document Viewer App "ubuntu-docviewer-app crashed with SIGSEGV in QProcess::state()" [Undecided,Incomplete] https://launchpad.net/bugs/1176592 | 18:07 |
| balloons | just launch and it crashes? | 18:08 |
| njin | hallo balloons, yes | 18:09 |
| balloons | njin, k, lemme check | 18:11 |
| elfy | I'd assume it to be specific to ubuntu then perhaps - works ok here | 18:11 |
| elfy | nvm - ignore me :) | 18:11 |
| balloons | njin, indeed.. segfault dump here | 18:13 |
| njin | balloons, so reply to the defveloper that will be enthusiat of our efficiency | 18:16 |
| balloons | :-) | 18:16 |
| njin | :-) | 18:16 |
| balloons | how's things looking with the core apps? | 18:16 |
| njin | from my side still remaining serious problem, but don't worry, time will repair | 18:17 |
| balloons | they keep improving it don't they :-) | 18:19 |
| balloons | back later all :-p | 18:58 |
| Vasudevan | hi, what version of autopilot is installed on saucy 13.10 daily build when we apt-get install python-autopilot? | 19:12 |
| Vasudevan | on my vm, version string is showing up as 1.0 | 19:13 |
| knome | balloons, i'm around, but not for long (still sick) | 19:13 |
| DanChapman | should be 1.3 Vasudevan did you get from the ppa? | 19:15 |
| Vasudevan | DanChapman,it is the default, I got a daily image 3-4 days back, set up the vm, did apt-get update and apt-get upgrade and then installed python-autopilot | 19:17 |
| DanChapman | You need to fetch it from ppa:autopilot/ppa to get 1.3 | 19:18 |
| Vasudevan | ok, so what command should I use - apt-get install python-autopilot1.3? | 19:20 |
| knome | balloons, back later, maybe look at it tomorrow if i feel any better | 19:21 |
| knome | -> | 19:21 |
| DanChapman | its still just python-autopilot, I had to turn off universe in software sources then install it when i did it because it kept fetching 1.2 from universe even though id added the ppa | 19:22 |
| Vasudevan | will these steps work ? sudo add-apt-repository ppa:autopilot/ppa, apt-get update, apt-get install python-autopilot | 19:28 |
| DanChapman | Have you removed your current version? | 19:28 |
| DanChapman | But yes that should work | 19:29 |
| Vasudevan | DanChapman, ok thanks, installed autopilot 1.3; test_nautilus, still throws same error that we were seeing before - Unable to find Autopilot interface | 19:44 |
| Vasudevan | it launched the app and then throws this error after a few seconds | 19:45 |
| DanChapman | Yeah nautilus is one of the problem ones at the moment, you have to use processmanager | 19:45 |
| DanChapman | There is a converted nautilus test in the trunk at the moment if you want to pull it and expand it further? | 19:46 |
| DanChapman | it only creates a directory and deletes at the moment | 19:46 |
| DanChapman | Although you cant use introspection on it | 19:46 |
| Vasudevan | with processmanager, the only option is to send keyboard events and mouse clicks right? | 19:47 |
| DanChapman | yeah pretty much :( | 19:47 |
| Vasudevan | so to select child windows and other actions using processmanager it is difficult | 19:47 |
| Vasudevan | does nautilus have hot key mapping for most of the actions or we must set them up first? | 19:49 |
| DanChapman | Vasudevan, im not sure.... i would say they need setting up | 19:50 |
| Vasudevan | ok, DanChapman, let me see how farI get with that | 19:50 |
| Vasudevan | one more thing, did you get any issues with the default terminal application in the saucy daily image? | 19:51 |
| DanChapman | with autopilot? | 19:52 |
| Vasudevan | with in the vm, after i open a new terminal, it does not show any commands being typed | 19:52 |
| Vasudevan | not autopilot, just the default terminal app | 19:52 |
| DanChapman | Ive not had any problems with it when ive run it in a vm. | 19:53 |
| Vasudevan | luckily, i tried another term type (uxterm) and it is working correctly | 19:53 |
| Vasudevan | quite strange.. | 19:54 |
| knome | balloons, actually, i'm back again for some time at least... | 20:11 |
| balloons | knome, heh.. ok so can we merge your big change without issue while we await the css change? | 20:43 |
| knome | balloons, it will show all the elements (just not optimally styled) | 20:43 |
| balloons | kome, and second, how did you make the changes and what is your comfort level with it? the diff is huge, can't really manually review it | 20:43 |
| knome | balloons, if you want to make sure, get stgraber to approve the css first | 20:43 |
| knome | balloons, the changes are made manually, so it should be all okay | 20:44 |
| balloons | knome, I don't want your merge to get stale.. we should move it in soon :-) | 20:44 |
| knome | it's obvious there might be an odd typo here or there, but i'm pretty sure it's good now | 20:44 |
| knome | ...so yes, i'm very comfortable with it :) | 20:46 |
| balloons | knome, ok, on that basis I'll just go ahead and merge it | 20:46 |
| knome | the css change diff is really trivial though, so shouldn't take even a minute to review it | 20:46 |
| balloons | I'm OK with a typo or two popping in there.. it's to be expected | 20:47 |
| knome | great! | 20:47 |
| DanChapman | balloons are you looking for fireroller to extract to /tmp aswell? | 20:47 |
| balloons | DanChapman, I think it's for the best to do so | 20:47 |
| DanChapman | no probs already done just checking :D | 20:47 |
| balloons | we are making tmp files and in general we don't want to collide with anything presen | 20:47 |
| balloons | DanChapman, :-) perfect' | 20:47 |
| knome | balloons, now that you have modified some of the tests meanwhile, do you need help with making sure that the merges are fine? | 20:51 |
| balloons | knome, we'll see in a moment | 20:52 |
| balloons | I don't think so.. nothing new merged had the issue | 20:52 |
| balloons | I don't think :-) | 20:52 |
| balloons | ahh.. one conflict.. not bad :-) | 20:53 |
| balloons | here comes the fun part.. reviewing 85 files plus the conflict | 20:54 |
| knome | hehe | 20:54 |
| balloons | ohh found something fun.. quotes | 20:56 |
| knome | :) | 20:56 |
| balloons | did you intend to change the quote from " to ” | 20:57 |
| knome | no; i did the opposite on one occasion | 20:57 |
| knome | because i checked that test on the tracker and the quotes didn't work | 20:57 |
| balloons | hmm.. right | 20:57 |
| balloons | so I think I'll do a massive s/”/"/g | 20:58 |
| knome | you *can* get those funky quotes, but you'll have to do them with entities, and that an be confusing | 20:58 |
| balloons | hmm.. you added br tags at one point also.. probably could drop those.. I do see why you did, but the formatting doesn't require it | 21:00 |
| balloons | so knome, <code> works fine on the tracker? drupal tag I take it? | 21:03 |
| knome | mmh, yeah, must've been some kind of weird vagary ;) | 21:03 |
| knome | no; it's an inline html tah | 21:03 |
| knome | *tag | 21:03 |
| knome | <pre> is a block element | 21:03 |
| balloons | ohh really? I didn't know <code> was inline html | 21:03 |
| knome | thus can't/shouldn't appear where you want the text to appear... inline :) | 21:03 |
| balloons | :-) | 21:04 |
| balloons | more questions.. so what does something like this look like? | 21:04 |
| balloons | <dl> | 21:04 |
| balloons | <dd>To execute this test, a pre-existing ubuntu (or derivative) installation is needed</dd> | 21:04 |
| balloons | <dt>Boot up the iso using a CD/DVD or USB Key </dt> | 21:04 |
| balloons | a lone <dd> tag | 21:04 |
| knome | did i leave those there? | 21:05 |
| knome | generally, it's fine to have a long dd tag | 21:05 |
| balloons | indeed.. I've got a running list for you to look at | 21:05 |
| knome | *lone | 21:05 |
| knome | it'll be formatted just as any other dd tag | 21:05 |
| knome | of course, you might want to move that outside the <dl> | 21:05 |
| DanChapman | balloons fileroller is good to go now | 21:05 |
| knome | (i think it's more semantic that way) | 21:05 |
| balloons | knome, I agree | 21:05 |
| knome | i moved at least half a dozen of those out, irrc | 21:06 |
| knome | *iirc | 21:06 |
| balloons | yea, I see them now :-) | 21:06 |
| balloons | lol | 21:06 |
| balloons | the next 5 are all moved out | 21:06 |
| knome | yeah | 21:06 |
| knome | must've just missed that | 21:06 |
| balloons | man I love meld | 21:07 |
| balloons | DanChapman, excellent.. evince must be not far behind eh? | 21:07 |
| knome | balloons, yeah, it's not bad :) | 21:08 |
| balloons | ok, whew, made it through | 21:09 |
| balloons | so 4 to look at. You want to make the changes or should I? | 21:09 |
| knome | cool :) | 21:09 |
| knome | balloons, to whom are you talking to? | 21:09 |
| balloons | you knome.. | 21:09 |
| knome | i can make the changes if me | 21:09 |
| knome | :) | 21:09 |
| knome | just tell me which | 21:09 |
| balloons | 304_VMWare Easy Install, " issue | 21:09 |
| balloons | 1413_Install (Preseed), drop br tags | 21:09 |
| balloons | 1445_Install Server iSCSI Root (virtualized KVM), weird pre tags | 21:09 |
| balloons | 1498_Upgrade (image), move lone <dd> tag to outside | 21:10 |
| knome | did you merge those already to the main branch? | 21:10 |
| balloons | no, nothing is merged | 21:10 |
| balloons | I merged locally | 21:10 |
| knome | i'd imagine it would be easier for both if you did that first | 21:10 |
| balloons | sorry lol.. so nothing is in trunk | 21:10 |
| knome | then i could just pull the latest and modify the files and do a new merge proposal | 21:11 |
| knome | which should have zero conflicts | 21:11 |
| balloons | you can update your branch | 21:11 |
| knome | i can do that as well if you want to go through that stuff ;) | 21:11 |
| balloons | and I can pull it again without issue :-) | 21:11 |
| knome | oki | 21:11 |
| knome | just a sec, i'll look at those | 21:11 |
| balloons | ty1 | 21:11 |
| balloons | I won't have to re-merge everything, just those 4 bits.. | 21:11 |
| knome | are you sure the last one was correct id? | 21:14 |
| knome | i don't see a lone dd tag | 21:14 |
| knome | also, 1498 says ubuntu (doesn't use the VARIABLEs) | 21:15 |
| knome | (i didn't look too closely at the content last time) | 21:16 |
| knome | i fixed 1498. | 21:18 |
| knome | balloons? | 21:19 |
| balloons | yes | 21:20 |
| balloons | ohh.. did you swap to use variables? everything look ok to you now knome ? | 21:20 |
| knome | yep, i switched to variables | 21:21 |
| knome | but 1498 didn't have the lone dd | 21:21 |
| knome | are you sure you didn't mix with something else? | 21:21 |
| knome | i just pushed the new revision to my branch | 21:23 |
| DanChapman | damn! balloons no wonder those evince tests were failing for you ive just upgraded to saucy and evince has a different ui | 21:24 |
| balloons | LOLOLOL | 21:24 |
| balloons | knome, oh | 21:24 |
| balloons | try 1497 | 21:24 |
| knome | hmm, no luck | 21:25 |
| * balloons looks | 21:25 | |
| balloons | that's the testcase | 21:26 |
| balloons | maybe I was looking at it backwards | 21:26 |
| knome | weird, it looks fine to me | 21:26 |
| balloons | let's see the diff aain | 21:26 |
| knome | probably :) | 21:26 |
| knome | because i have that line outside the dl ;) | 21:26 |
| balloons | DanChapman, hmm.. so I guess you should update for the saucy UI | 21:26 |
| balloons | that's odd it breaks it | 21:27 |
| balloons | that's why we like introspection.. in theory breakage should be minimal if at all | 21:27 |
| DanChapman | yeah, well it used some of the button labels which have now gone so shouldnt take too much adjustment | 21:28 |
| balloons | here goes gnome | 21:28 |
| knome | ;] | 21:28 |
| balloons | now to kick off the sync to the tracker | 21:28 |
| knome | nice | 21:28 |
| Noskcaj | are there any autopilot tests that re ready for general use? | 21:28 |
| balloons | Noskcaj, everything in the production branch is ready | 21:29 |
| balloons | if you just want to see them | 21:29 |
| balloons | lp:ubuntu-autopilot-tests/production | 21:29 |
| Noskcaj | balloons, once i get my re-install done i'll start running them | 21:29 |
| balloons | there is/was a ppa also, but I'm not sure I'm going to revive it | 21:29 |
| Noskcaj | balloons, ok | 21:30 |
| DanChapman | im calling it a day, need to eat. Catch you tomorrow :D | 21:30 |
| balloons | DanChapman, have a good one mate | 21:31 |
| DanChapman | and you balloons :) | 21:31 |
| balloons | knome, sync is off.. should be all good. thanks again for the changes | 21:31 |
| knome | no problem | 21:31 |
| balloons | DanChapman, I'll review fileroller and add it to prod :-) | 21:31 |
| knome | just ping me if you run into problems | 21:31 |
| DanChapman | YAY! :D | 21:31 |
| === hggdh_ is now known as hggdh | ||
| === Ursinha is now known as Ursinha-afk | ||
| === Ursinha-afk is now known as Ursinha | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!