/srv/irclogs.ubuntu.com/2013/11/15/#ubuntu-quality.txt

=== steve__ is now known as sbeattie
=== chihchun_afk is now known as chihchun
pittiGood morning06:17
jibelGood morning07:32
pittijibel: I'm scared to say that slowly, but steadily I understand the autopkgtest code..08:09
jibelpitti, argh, that's horrible, does it hurt? ;)08:10
pittijibel: it does..08:10
pittijibel: I spent several hours trying to understand what all these AutoFile classes do, adding comments to document it, and now I remove their usage step by step :)08:11
jibelpitti, oh, I'm sorry, I understand your pain.08:13
pittiI already got rid of some of the confusing output/logging/Errplumb stuff08:13
pittijibel: but it seems this is becoming a more and more important infrastructure piece, so it's probably better if more than just two people in the world know how this works :)08:13
pittis/know/can understand/08:14
jibelpitti, agreed. From what I understood of the code, the logic is not really complicated but the code is somewhat obscure08:15
jibeland it's an understatement08:15
DanChapmangood morning all09:14
pittihey DanChapman, how are you?09:26
jibelpitti, simplejson was the package TheMuso was referring to in his patch pilot summary this morning?09:44
jibelMorning DanChapman09:44
pittijibel: I don't know, I'm afraid09:44
jibelit's sad because this package has the testsuite of +110 tests09:44
jibelI'll enable autopkgtest and reply to the list09:45
pittijibel: I guess even a 'python -c "import simpleson"' test would already help a lot :)09:46
jibeljibel, in this case definitely09:46
pittipitti: but running the full test suite is even better, of course09:46
pitti(SCNR)09:46
jibelpitti, do we have a preferred runner for python?09:48
pittijibel: what do you mean with runner?09:48
jibelpitti, nose, pytest, unittest, ...09:48
pittinot for autopkgtest09:49
jibelpitti, in simplejson if I call the test script directly it'll use the source tree09:49
pittiwhatever upstream uses09:49
jibelpitti, but if you use another runner it will use the package as installed09:49
pittioh, surprising; usually that's the easiest way to make it run against the system libs09:49
pittijibel: whatever is easiest09:50
jibelthey do a sys.path.insert(0, os.path.dirname(os.path.dirname(os.path.dirname(os.path.abspath(__file__)))))09:50
jibelwhich point to the root of the source tree09:50
jibel+s09:50
pittioh, eww09:51
pittihow ugly; that sohuld be done by setting PYTHONPATH in the source tree test runner09:51
dingjjls10:18
davmor2Morning all10:28
slickymastergood morning all11:02
=== chihchun is now known as chihchun_afk
DanChapmanhey pitti, sorry I didn't see your message pop up. I'm good thanks how are you?11:26
DanChapmanmorning jibel o/11:26
pittiDanChapman: heh, no problem; I'm fine, thanks!11:27
jibelpitti, with UDD should I update changelog or it is done automatically from the commit message when the branch is merged?11:31
pittijibel: you should still write debian/changelog, and then use debcommit11:34
pitti(that'll build the bzr changelog from debian/changelog)11:34
jibelpitti, https://code.launchpad.net/~jibel/ubuntu/trusty/simplejson/enable_autopkgtest/+merge/195380 when you have a minute13:25
pittijibel: I haven't seen py.test yet, that's a runner like nose?13:26
pittijibel: how does that work around the sys.path() setting?13:26
jibelpitti, yes it is similar to nose13:27
pittiI guess I would just have copied the tests to $ADTTMP and run them from there, but if that's working as well, it's fine13:27
jibelpitti, path is set in main of __init__ in the tests which is not called if you use the runner13:27
pittiah13:27
pittijibel: can you please forward it to Debian?13:28
jibelotherwise setup.py just calls __init__.py to execute them13:28
pittiright13:28
jibeland uses unittest13:28
pittiI'll merge/upload in the meantime13:28
jibelI'll forward to debian and reply to luke13:29
pittithanks13:29
jibelpitti, FW is finally open, I'll reenable cron in the lab which has not been copied from the previous machine and autopkgtest should start again13:31
pittiyay13:31
pittijibel: merci13:31
jibelwell, that's theory of course :)13:31
elfyballoons: I've just been informed that my day's of next week have been cancelled, I see no way to unregister from uds sessions - so you'll just have to ignore my name there15:52
balloonselfy, ugh15:53
balloonswell, no worries about your name being there15:53
=== wylde is now known as Guest64913
Letozaf_balloons, hello20:10
Letozaf_balloons, I saw you fixed rssreader-app test, thanks, its nice to see it work both on the dekstop and device20:11
balloonsLetozaf_, howdy20:11
Letozaf_balloons, fine what about you ?20:11
balloonsLetozaf_, good.. there was a package issue that prevented the merge bot from running till just now20:13
balloonsso i'm reviewing the run20:13
balloonslooks like a dependency issue.. i fixed and pushing again :-)20:14
Letozaf_balloons, ok20:14
Letozaf_balloons, it passed!20:18
balloonshehe ;-)20:18
balloonsLetozaf_, you can do the approval honors ;-)20:20
Letozaf_balloons, ok thanks20:20
balloonsLetozaf_, note I did disable the test_view_mode_and_feed_item test. I couldn't get it to switch tabs properly. Need to ask elopio about it20:21
Letozaf_balloons, yes I saw that20:21
balloonsdat timing20:22
* balloons waves to elopio 20:22
balloonselopio, I wanted to ask you about why I couldn't get the emulator to switch tabs in rssreader for a test. the tabs are dynamically generated.. the emulator sees no tabs present20:24
balloonsalso, I can't switch to a tab by name anymore, only objectname. naturally, these will only have dynamic names, not objectnames20:24

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