[12:00] the one(s) that also have overrides and other tables... [12:01] i'm trying to keep files to have t-4 closely related classes max [12:01] 1-4 [12:01] oh [12:01] I'm going to end up with large numbers of classes for the views Lucille uses for publishing [12:01] I'll have to think about how to split them up [12:01] (dilys/#launchpad) Merge to rocketfuel@canonical.com/launchpad--devel--0: added portlets and images (patch-644) [12:11] (dilys/#launchpad) Merge to rocketfuel@canonical.com/launchpad--devel--0: Fixed the sequence number when reusing the same import object, it should be reset to zero again (patch-645) [12:31] hey lifeless! [12:31] hitting a problem I know well with zenity. [12:33] It seems that "cvs update -p" is sometimes yield corrupt data when the file is a binary file (e.g. a png) but the cvs repo does not sets its keyword expansion to -kb. [12:34] The corruption translates into a totla.ValidationFailed for several binay files in zenity. [12:36] lifeless: I had the same problem with texmacs. I had worked around it by adding a judicious -kb in one of the the cvs commands. What are the choices of cscvs regarding keyword expansion. Are you aware of that issue? How do you handle it? [12:37] SteveA: ping [12:40] i want to perform the trivial, 5 second task of committing just one file on a commit. i tla add filename, tla commit -s "something with filename" -- filename, and get: make-changeset-files: file missing from ORIG tree (filename). How do i perform this trivial task with tla? [12:40] i read the relevant part of the wiki, didn't understand was daf's note meant, and just went ahead with the above, but apparently Guessed Wrong. [12:40] s/was daf's/what daf's/ [12:41] BradB: I suppose you should add also the id for that file [12:41] ddaa: we do whatever cvs repo does. [12:41] BradB: from .arch-ids [12:41] carlos: ew, yeah, that sounds like a nasty bug :) [12:42] iirc the pb is the cvs repo is inconsistent [12:42] BradB: well, you should add the id inside the file, instead of using tla add :-) [12:42] carlos: but hmm, tla tree-lint doesn't show it. [12:42] carlos: ew, no thanks :) [12:42] if we don't, then there is no way for someone coming across to tell what cvs revision that file patch-level came from. [12:42] uuidgen [12:42] that would need some testing, but I remember that the problem only appears with the -p option... so regular cvs users never see it. [12:43] BradB: it does not show it because add did the job, you need to add the id file to the commit command [12:43] BradB: so you commit both files at the same time [12:43] ouch, that sounds nasty === ddaa hums a "baby jesus crying" tune [12:43] lifeless: I'm surprised that you never ran into that issue befor. [12:44] you mean -P or -p [12:44] ? [12:44] -p, output to stdout instead of file [12:44] well, we don't use that at all [12:44] we talk CVS natively now. [12:44] you so do [12:44] its there if we fall back to invoking cvs [12:45] you are using cvs when talking to remote repos, right? [12:45] no, native implementation [12:45] in cscvs/modules/CVS/__init__.py [12:46] Mhh... that woush explain why my old trustworthy fix did nothing... [12:46] see getFileRevisionStat in Module [12:46] what repo is getting corrupted this way ? [12:46] the one that needed fixing was getFileRevision [12:46] but as I said that fix does not seem to work anymore... [12:47] lifeless: zenity, during the sync [12:47] several changed png file [12:47] I check that the first one does not have the -kb bit. [12:47] *checked [12:49] ddaa: ok, so the first thing would be to (via interactive python, for ease) try getFileRevisionStat on the file and see if that works better [12:49] How do I know the relevant revision? [12:50] I know, that's the revision of the file in the cvsworking... [12:50] well, when did you see the error? at the end ? [12:50] a fatal error, by definition, is always at the end. [12:50] during the cross-check phase I mean ? [12:51] cscvs.cmds.totla.ValidationFailed [12:51] oh, now thats plain silly [12:51] I see only one place where that's raised, and that's the cross-check [12:51] foo.original is caught by tlas .orig matching. [12:51] thats a BUG [12:51] ddaa: ok, in which case the revision is just the latest on HEAD - which will be the one in cvsworking [12:53] I cated both files, and the cvs one has some extra junk at the end. [12:53] which the tla one lost ? [12:53] ok. can you try: [12:53] import CVS [12:53] No, that looks like premium junk. [12:53] tree=CVS.tree(cvsworking) [12:53] that makes gimp complain. [12:53] etc. [12:53] so the cvs one is corrupt, not the tla one ? [12:54] Yeah. Now, lemme check I get the correct one with a plain cvs update... [12:54] (we are not meant to write to the cvsworking tree at all, that should be what was checked out) [12:55] My point is that sometimes we do not get what would be checked out by the usual means. [12:55] yeah, I get that [12:55] I'm trying to help you track down where the bug is [12:58] I think we can disable the mode cache completely with the improvements we've had since it was first put in [12:58] I'm going to test without it.. [12:58] Interesting... plain update and checkout gives the same corrupted crap. [12:58] right [12:58] so thats whats in cvs. [12:58] thats what we bring to tla. [12:58] so the bug is that we dropped that crap. [12:58] wait a min,,, [12:59] ha, mea culpa [01:00] according to the gimp, the corrupted one is the tla one [01:00] so the world makes sense again [01:00] huhu... that might be a discrepancy between the cvstarball and the remote cvs repo... [01:01] well, for now.. we want tla to accurately represent the cvs tarbal contents. [01:01] if thats corrupt, its a separate issue. [01:01] so, you've found a path that leads to data mismatch. [01:02] that's the fact [01:02] there are two possibilities that could be happening. === ddaa resets the speculator [01:02] 1) the file has a default branch. [01:02] 2) the data is being mangled [01:02] lets check one first. [01:02] is there a 'branch: ...' statement in the log for that file? [01:02] the data mismatch is some missing bytes at the end of the file, so probably 2. [01:02] (just run cvs log) [01:02] carlos: I'd like to review first [01:02] carlos: but you think it's ready? [01:03] lifeless: nope [01:03] I mean yes, but it's empty. [01:03] daf: yes [01:03] in which case its two. [01:03] daf: all tests pass [01:03] daf: I was adding the comments as stub asked me yesterday [01:03] ddaa: can you now try [01:03] import CVS [01:03] so it's ready to comment [01:04] tree=CVS.tree('cvsworkingpath') [01:04] good night all === cprov [~cprov@200.158.100.251] has left #launchpad ["Leaving"] [01:04] just detected a funny bug with the sample data script that will break the sample data when it's regenerated :-( [01:04] data=tree.getFileRevision(filename, revisionstring) [01:04] potmsgsets are imported after pomsgsets [01:04] and pomsgsets need potmsgsets... [01:04] file=open(tempname, 'w') [01:04] file.write(data) [01:04] I will mail now to launchpad about it [01:04] file.close() [01:04] and see if that is corrupt [01:04] carlos: a bug on your branch or for the main branch or both? [01:05] then try a second time, but get the data via [01:05] lifeless: note that my cvsworking, created from scratch by the sync job is clean. [01:05] daf: it's not a bug with our code but with pg_dump [01:05] That's the content of the tla archive which seems corrupt. [01:05] or the schema's Makefile [01:05] carlos: both then? [01:05] data=tree.module().getFileRevisionStat(filename, revisionstring)[0] [01:06] So the bug occured at some point in the past and was not caught. In addition that file has exactly one revision. [01:06] daf: yes [01:06] sure. [01:06] ok [01:06] daf: but it's an external bug... [01:06] look in StorageLayer.py _get and _get_file_with_perms to see the two code paths. [01:06] I'm /right now/ making that one code path. [01:06] carlos: what I wanted to know was which branches it was causing problems on [01:07] daf: only in mine [01:07] because the new table [01:09] daf: all my changes are mirrored now at chinstrap [01:11] carlos: cool [01:11] mh... I do not understand, but I'll do what you ask, [01:14] I'm /right now/ making that one code path. [01:14] bah [01:14] ddaa: so, what I'm asking you to do is to verify which of the two means of getting the file is causing the corruption [01:15] okay (placing my bets) [01:16] I'm sure you are right. but we can't easily write a test until we know what the precise cause is. [01:16] and we sure ain't fixing this without a test. [01:24] lifeless: I won my bet [01:25] The answer is "none", the file is correct using both methods. [01:25] i.e. the file is what is in the cvsworking but not in the tlaworking [01:25] ok, what do you think is the problem then? because those two methods are how the file gets to tlaworking [01:26] mh... === elmo_ [~james@host217-37-231-28.in-addr.btopenworld.com] has joined #launchpad [01:26] wait a min, maybe it has been using my fix or something... [01:26] yeah, please do do it with the untouched code :) [01:26] (dilys/#launchpad) Merge to rocketfuel@canonical.com/launchpad--devel--0: added a generic way to send mail in LP and the beginnings of event handling for bugs, including notifications going out when a comment is added to a bug (patch-646) [01:27] lifeless: I have undone, but I'm not absolutely sure I did so before starting python. [01:27] lol [01:30] Hehe [01:30] My fix was indeed correct :-) [01:30] ddaa: what was your fix ? [01:30] the problem is with tree.getFileRevision [01:31] module.getFileRevisionStat's legacy code path probably suffers too [01:31] add -kb to the cvs update if the name of the file ends in .png [01:31] oh man [01:31] There might be better ways... [01:31] thats not a fix. [01:31] That was a testing hack... [01:31] does getFileRevisionStat() work ? [01:32] if so, we'll change _get to use that [01:32] It seems too. [01:32] because that is a fix. [01:32] *it seems to work [01:32] ok, please make it use that, in whatever way is easiest. [01:33] okay. [01:33] btw, your "native" implementation is a python reimplementation of the cvs client functionality we need? [01:33] yes === ddaa laughs nervously [01:34] routing around damage... [01:34] written in oxford [01:35] Well, the 5 imports/day target it still far, but given time they will be assimilated [:-] [01:36] I am buildbot of canonical. Resistance is futile. You will be imported. Your specificity will enrich our archives. [01:37] Ok. Will do tomorrow. Time to sleep. === BradB is now known as BradB|away === BradB|away is now known as BradB [03:37] SteveA: ping [03:47] (dilys/#launchpad) Merge to rocketfuel@canonical.com/launchpad--devel--0: Rename vocabulary -> vocabularies for consistency. Consolidate PackageReleaseVocabulary (patch-647) [04:00] BradB: Can you confirm if the latest Makefile changes will keep Limi running for the time being? [04:14] stub: What am I checking? :) [04:16] That 'createlang plpythonu' failing won't kill the entire process mainly, and extra points if 'select valid_name('hello')' doesn't blow up. [04:33] The former seems to work okay; how do I test the latter? [04:34] launchpad_test=> select valid_name('hello'); [04:34] ERROR: function valid_name("unknown") does not exist [04:34] HINT: No function matches the given name and argument types. You may need to add explicit type casts. [05:03] Poo [05:15] (dilys/#launchpad) Merge to rocketfuel@canonical.com/cscvs--devel--1.0: fix occasional data corruption due to cvs stdout being broken (patch-34) [05:37] Bah.... works here :-( === BradB [~bradb@modemcable165.196-131-66.mc.videotron.ca] has joined #launchpad === BradB is now known as BradB|zzz === mako [~mako@micha.hampshire.edu] has joined #launchpad [09:00] stub: ping [09:01] or rather, pong [09:01] Yo. [09:02] I was just wondering if you knew how to extract the source code of a function off the top of your head [09:10] thefunction.func_code.co_firstlineno and thefunction.func_code.co_filename get me most of the way there [09:11] (in fact, all of the way in my use case, but there might be a nicer way) [09:24] don't know what else you can do [09:24] I just took a skim through the code to apidoc, but that doesn't do anything fancy [09:25] hmm [09:25] inspect module [09:25] http://docs.python.org/lib/inspect-source.html [09:25] getsourcelines( object) [09:25] Return a list of source lines and starting line number for an object. The argument may be a module, class, method, function, traceback, frame, or code object. The source code is returned as a list of the lines corresponding to the object and the line number indicates where in the original source file the first line of code was found. An IOError is raised if the source code cannot be retrieved. [09:25] === jblack_ is now known as jblack === ddaa [~ddaa@nemesis.xlii.org] has joined #launchpad === justdave [~dave@24.247.63.44.gha.mi.chartermi.net] has left #launchpad ["Poof"] [10:17] morning all === lulu [~lu@host217-37-231-28.in-addr.btopenworld.com] has joined #launchpad [10:19] alentlesstramps [10:20] thanks lulu, hope you don't really feel that way about us [10:21] thomming continues apace [10:21] SteveA: Excellent === stub cackles evilly === lulu giggles - oops! === SteveA_ [~steve@office.pov.lt] has joined #launchpad === SteveA_ is now known as SteveA [10:48] looking at that api gives me an idea for improving pages for exceptions in launchpad [10:50] unless that's what you're doing evil stub [10:52] SteveA: Nope. I'm looking at creating data vaidation functions and database constraints from the same source. [10:55] ok, thanks for the idea anyway === carlos [~carlos@69.Red-80-33-181.pooles.rima-tde.net] has joined #launchpad [11:09] morning === justdave [~dave@24.247.63.44.gha.mi.chartermi.net] has joined #launchpad [11:11] (dilys/#launchpad) Merge to rocketfuel@canonical.com/launchpad--devel--0: Give external bug watches some luvin' (patch-648) [11:13] carlos: you pingged earlier [11:14] SteveA: it was yesterday night, don't worry, I was able to talk with daf so I don't need to ask you anything now :-) [11:16] (spiv/#launchpad) SteveA: As far as improving exception pages in launchpad, there's also cgitb. [11:33] do we have a favicon? [11:34] stub: do you overlap with bradb usually? [11:37] Not sure - recenly I've been online for loong hours so I've seen him around. Not sure if we will meet now I'm trying to get back to more sane working hours. [11:38] i'd like to have a malone update meetng today, could you holler if you notice you are both online at the same time? [11:41] a good way to arrange it in future is to ask Brad to be at a meeting that is early in his timezone, say 1100 UTC or 1200 UTC [11:41] I think Brad went to bed about 5 hours ago, so he might get online in about 4 hours or so. [11:42] Indeed - arranging in future is good. [11:42] talking of which, how does Friday sound for a launchpad general meeting? === SteveA hollers to launchpad people who are listening [11:43] Don't care - just tell me when [11:50] SteveA: sounds good [11:52] SteveA: favicon, should i add one? [11:54] sure. I'm not sure what the rules are for looking up a favicon. [11:54] do we need one in the root of launchpad (to stop it complaining), and then favicons for each application? [11:54] (spiv/#launchpad) SteveA: The rules are browser dependent, but generally something like this: [11:54] are the pages supposed to have a element in the header refering to the favicon? (I recall reading that somewhere) [11:54] (spiv/#launchpad) If there's a in the , use that. [11:55] lifeless, bob2: ping [11:55] (spiv/#launchpad) Otherwise, guess at /favicon.ico (i.e. the top-level of the domain). [11:56] (spiv/#launchpad) The open source browsers at least are happy with .png files as favicons, I'm not sure about IE. [11:56] we need to add one at /favicon.ico [11:56] hmm, actually, it isn't wholey straightforward to add an image as a view in zcml [11:56] I'll add favicon to today's todo list. sabdfl: do you have an image you'd like to use? [11:57] (spiv/#launchpad) sabdfl: Wouldn't adding a to our HTML header be sufficient? [11:57] spiv: don't think that will catch every browser. we should do both [11:57] (spiv/#launchpad) sabdfl: Yeah, both is probably best. [11:58] guys, the arch-tag thing is a mess, please don't create any new ones [11:59] you mean, carlos doing a bunch of work on a branch? [11:59] :-? === SteveA wonders what sabdfl means by "arch-tag thing" [12:00] ah -- using arch-tag: in files rather than tla add [12:00] carlos: getting stuck on a branch is a mess for different reasons ;-) [12:01] jblack: ayt? === ddaa [~ddaa@nemesis.xlii.org] has joined #launchpad [12:01] SteveA: :-D [12:05] ddaa: I think I may have asked this before, but is there a reasonable way of converting files that contain an arch-tag: line into having their identifier managed by tla in its metadata files? [12:05] SteveA: No. [12:06] taglines and explicit tags use distinct namespaces. [12:06] ddaa: is there a way we can get pqm to reject commits that use arch-tag for new files? [12:07] ddaa: so, we'd have to essentially remove them and then add them again, losing history? [12:07] sabdfl: I can imagine a shell script doing that. [12:07] SteveA: seems so [12:07] SteveA: correct, or patch tla. [12:07] so best bet is to grandfather the existing ones and prevent checkins of new ones [12:08] I'm not sure would would be the right way to do it. There has been discussion for a feature to go explicit->tagline but not to go the other way around. [12:08] breakfast time, see you later [12:09] TBH I find that a surprising requirement, I find taglines are more convenient. [12:09] there are 120 files in launchpad using arch-tag [12:10] ddaa: http://manifesto.gnuarch.org/moin.cgi/ArchWarts [12:10] They are more robust in many way. The only issue is when copying files, but that requires a deliberate user action and generally is accompanied by a lot of editing, so changing the tagline in the new file is not a big burden. [12:11] my problem with it is having two ways of telling arch about a file [12:11] so, whenever I need to move a file, or copy a file, I first need to look inside it to see if it has an arch-tag [12:11] it is one more thing to have to remember [12:12] (spiv/#launchpad) Doesn't "tla mv" do the right thing either way? [12:12] (spiv/#launchpad) Copying a file is obviously a problem, though :) [12:12] tla rm certainly doesn't [12:12] see ArchWarts [12:13] tla inventory -s --ids | cut -d "`echo -e "\t"`" -f2 | grep ^i_ [12:13] mhhh a bit more... [12:14] tla inventory -s --all --ids | cut -d "`echo -e "\t"`" -f2 | grep ^i_ [12:15] will show all ids generated by taglines, and exit1 if none are found. [12:16] Okay I looked at archwarts... [12:16] I just added a new wart [12:16] the tla rm problem is indeed annoying [12:17] but if you people can bear with it a little bit, it can be easily worked around in a wrapper, and it should certainly be fixed in tla. [12:17] at least, it should be fixed in tla imho. [12:18] about the wrapper, bob2 should open hct for testing by the arch team soon, so if you can suffer the issue a bit longer, we could work around it in hct (if not already done) [12:18] (spiv/#launchpad) The output of "tla rm -H" implies that it should work. [12:18] (spiv/#launchpad) So if it doesn't, it is a bug, I think :) [12:18] either a bug in the docs or a bug in the code [12:18] (spiv/#launchpad) Right. === ddaa tests [12:20] okay, I can reproduce the pb with tla 1.2rc2 [12:22] Neither fai nor aba work around this one. [12:22] I dunno about hct. Yet. [12:22] Now you have all the elements to take a project management decision. [12:23] And my opinion as an extra. [12:23] (spiv/#launchpad) ddaa: Wow... so tla doesn't have a test-case for tla rm? Eek :) [12:24] tla's test suite is very... limited... [12:25] (spiv/#launchpad) ddaa: And tom's worried about getting a perfect release process and voting system? Sheesh. [12:25] you can't do releases by vote === spiv -> food [12:26] The voting system is more about governance if I understand correctly. [12:27] And voters would be co-opted arch devels. [12:27] ddaa: i'll be interested to see how it goes [12:27] how many voters are there? [12:27] will you keep me posted? [12:28] SteveA: dunno, that's probably something like 5 persons... tom, asuffield, lifeless, jblack, abentley, jivera, maybe talli, etc. [12:28] This information is not authoritative. [12:29] sabdfl: okay, I've been told it's good to assume new responsibilites :-) But tbh the arch community mess is rather pissing me off those days. [12:30] sabdfl: I will keep you posted on significant release/governance events in the arch community. [12:30] ddaa: thanks [12:30] ddaa: tom lord is spending time sorting out a voting system for about 7 people to use? === SteveA remembers that limi and lulu are not here on fridays [12:31] SteveA: *shrug* something like that... I have not been following that very closely up to now. [12:31] how about Thursday, 1200 UTC for a launchpad meeting instead. (That's tomorrow.) [12:31] SteveA: sounds good to me :o) [12:32] It has mostly been vapor to this point. I'm waiting for something concrete before making an opinion. [12:33] in most communities, 7 is easily within the size that you can make decisions by consensus. === limi [~limi@host217-37-231-28.in-addr.btopenworld.com] has joined #launchpad [12:33] SteveA: it seems it's difficult to achieve consensus with tom even at two people. [12:34] Most of the few times I got feedback from him, I classified it as "nonsense or not specific enough to be useful". [12:35] Even if he often sounds like he has a pretty clear idea of what he means. [12:35] But I'm not the best kid in town for communication. [12:35] e.g. I was impressed my lifeless abilities in that dept. [12:36] In handling the jblack's integration branch crisis. === SteveA is glad zope3 has a pope rather than the Lord [12:39] actually, at the time I did not classify the feedback that way. [12:39] I thought "I'm probably not smart enough to figure out what he means" [12:40] talking with jim fulton is often like that, but he's always happy to explain further and to listen to other points of view. [12:41] Since that time my position has changed a bit. Now I consider communication skills essential to project lead, and failure to communicate a blame for the leader. [12:41] Probably that position is caused by my pissed of state... the right attitude should certainly me something intermediate. [12:42] communication skills are almost always more important than technical wizardry for a project leader [12:43] consider the architypal example: linus and the linux kernel === ddaa considers [12:43] I'm sure he knows what he's doing with kernels. However, his communication to his peers stands out more than his code. === ddaa thinks of linus 2.6 vm rewrite [12:50] "okay, the 2.6 vm system is broken, broken, broken. I'm rewriting it all in one swoop and will do something that works." And indeed that worked... [12:51] There are several examples like that, when he does what nobody else was able to do... [12:53] that's more a case of having the knowledge and ability to do it, and also having the time and resources to do it. Someone else could have done so, if they met the prerequisites, and offered it to linus for incorporation into his tree. [01:10] Would be nice if you people added your nick and a date when writing to the wiki. [01:11] The nick is useful to know how to talk to if necessary, and the date is useful to have a clue of how current is the comment. Wiki tend to grow a lot of stale data, timestamping comment make it easier to sort out the current from the ancient. [01:13] ddaa: which kernel did he do that for? [01:13] Mh... nah, that was in the 2.4 series... [01:14] SteveA: given that we have a way to list the known arch-tag files, could you add a test to the pre-merge check which barfs on new arch-tag files? === elmo_ [~james@82.211.81.249] has joined #launchpad === ddaa points out that the "tla mv" gripe stems from inconsistence. It's just as valid to make all files use taglines. [01:16] sabdfl: I suppose it would have to record not the filenames but the arch-tag ids, because files get moved. [01:20] ddaa: problem is that not all files can have an arch tag [01:20] so there is inevitable inconsistency [01:21] so we will eliminate arch tags [01:21] i'm in fascist mode [01:24] sorry, cannot find the relevant kernel release, it's back before the the earliest data in the lwn search database. [01:26] stub: from your latest changes it seems i broke some pages [01:26] sorry [01:26] would have fixed them if there were page tests [01:26] please could you add page tests for the bugtracker pages? [01:26] thanks [01:30] we really should get the arch-tag lines out of the page templates. it is not nice to serve these things up as comments in the pages' html. === SteveA does so [01:38] SteveA: do you have time to look at the search/selection widget (making a single-template version now) later, and tell me how we can turn it into a widget? (This one, just without the JS for now: http://plone.org/Members/limi/tests/ubersearchwidget ) === SteveA goes for lunch [01:39] limi: probably. mail me when you're done, perhaps [01:39] ok [01:40] I assume stub knows how to do it too [01:41] mh... actually it seems my reporting of the kernel history was inaccurate... [01:41] That was a big merge from the Andrea's Arcangeli's VM work in 2.4.10 [01:42] for some reason, I cannot make a pagetest of .txt / text/plain files. [01:42] Human memory is tricky like that... === SteveA really -> lunch [01:47] (spiv/#launchpad) ddaa: 2.4.9/2.4.10, iirc. === lulu [~lu@host217-37-231-28.in-addr.btopenworld.com] has joined #launchpad [01:49] http://lwn.net/2001/0927/kernel.php3 === ddaa is out for lunch === cprov [~cprov@200.158.100.251] has joined #launchpad [02:09] SteveA: where is the best place to define (in ZCML) a template that should be globally available (the thing that will become a widget) - I just need to test it with TAL enabled :) [02:10] maybe we should have a "testing" dir that uses that attribute of ZCML that we use on images - so that everything there is made globally available inside launchpad? [02:13] everything is globally available in launchpad [02:14] if it's zcml please put it in lib/canonical/launchpad/zcml/ === Kinnison [~dsilvers@haddenham.pepperfish.net] has joined #launchpad [02:14] SteveA: why would tests be failing to merge, when they pass on my machine and i'm fully merged to rocketfuel? === debonzi [~debonzi@200.158.100.251] has joined #launchpad [02:22] sabdfl: About splitting up publishing.py... I do have concerns that I'll be creating arbitrary divisions without any useful purpose if I start splitting these up. I'm not convinced that I'm going to end up with many more views for publishing (Indeed I expect I may end up with no more at all) so it seems fairly pointless to split the classes out since I'll end up with 'publishing.py' and 'publishingviews.py' or something and it kinda seems od [02:23] sabdfl: maybe because the configuration of the system running pqm is different than yours. If it's not in the tree, it's outside the tree... === stub [~stub@dsl-246.248.240.220.dsl.comindico.com.au] has joined #launchpad [02:23] *different from yours [02:28] feh; sqlobject can't be subscripted [02:29] there goes my neat [ rec[x] for x in columns ] === limi has the ZCML blues [02:38] how can I get my page just to show up, not associating it with a particular class etc? [02:38] daf, SteveA, stub? === limi is trying to create a widgets.zcml with templates that can be accessed from anywhere [02:40] You can add them as views on everything - like you would add a page to an object except specify '*' as the interface. [02:41] how do I do that? my current ZCML is: http://paste.plone.org/1795 [02:42] add a 'for="*"' attribute to the page directive [02:42] so close ;) === BradB|zzz is now known as BradB [02:43] Alteratively, it might be neater to add it only as a view to the root folder (for="IRootContainer" or something - I'd need to look it up) [02:43] morning BradB [02:43] sabdfl: hi === limi is not looking for Neat at this point ;) [02:43] yo [02:43] while we have you and stud together, can we have a quick malone catch-up and status check? === stub struts around the channel === Kinnison laughs [02:44] erm [02:44] sorry [02:44] slip of the finger [02:44] sabdfl: is there something you're not telling us about you? [02:44] sabdfl: The email notifications are working, but they aren't all working yet (because there's about 16 that have to happen.) [02:44] BradB: is there a standard way to send email to a person in launchpad? [02:45] The thing that hung me up last night was displaying the "title" of fields that are from vocabs. i.e. I don't want the email to say "Status: 1", I want it to say "Status: New". [02:45] So we are using events for the email notices? (I had been toying with the idea of using triggers to fill the audit log, and a cron job to send the emails but it was going to get real messy...) [02:45] sabdfl: Yes, I wrote something yesterday for that. [02:45] stub: Yes. [02:45] BradB: There is a bug in bugzilla for that - SteveA was going to implement if for Limi a last week... [02:46] The one other thing that was non-obvious to me was how to hook into edits [02:46] Adds are easy, of course, because you just publish an event in the factory. [02:46] Oh... hang on - I think you can do that one right now. [02:51] stub: what's the bug number? [02:53] https://bugzilla.warthogs.hbd.com/bugzilla/show_bug.cgi?id=2101 [02:53] (Although this is for TAL... don't know if it is appropriate for your email stuff? [02:54] nope, I know about the TAL stuff. I need an equiv for plain Python code. [02:57] SteveA: Can you remind me what the ":" says in foo/bar/lp:BugStatus? I seem to remember it having something to do with adaptation. [02:57] It might give me a hint about how I achieve the same thing in plain Python. [02:58] BugSeverity.items[10] .title [02:59] ah [02:59] With tests in canonical/lp/test_dbschema.py for examples [02:59] I noticed that items attrib last night, but didn't expect it would do that. [03:00] I've forgotten why it was necessary - I think it might have been because we couldn't define the __getitem__ method on a class, only an instance? === stub has totally shot his short term memory [03:01] AttributeError: 'SimpleVocabulary' object has no attribute 'items' [03:01] stub: drugs are bad, mmkay\ === BradB looks through the tests for kicks [03:02] Erm.. you won't have much luck getting it out of a Vocabulary (but you could use it to get the index to put in BugWhatever[x] ) [03:02] Ah, I think I'm using the wrong vocab object. [03:02] I want the thing that inherits from DBSchema, I suppose. [03:02] (i.e. the not-vocab object) [03:02] Yup [03:03] I don't think Vocabularies are much use for anything except sending to the Z3 form machinery... icky interface [03:04] stub: They could be though, with getTerm and getTermByToken. Tried that last night though and got NotFoundError's. [03:04] (spiv/#launchpad) stub: You can define __getitem__ on a class, it just requires a metaclass... [03:04] NA NA NA CAN"T HEAR YOU NA NA NA [03:05] (spiv/#launchpad) :) [03:06] stub: nice strategy ! [03:06] stub: Yeee, that worked, thanks. [03:06] spiv: ? I just define __getitem__ as plain instance method... [03:07] What's wrong with that? [03:08] The things in dbschema are not instances, they are actually classes (generated on the fly from docstrings, because SteveA is a sick, sick person) [03:08] (spiv/#launchpad) (pasted example in privmsg, to avoid spamming channel) === justdave [~dave@24.247.63.44.gha.mi.chartermi.net] has joined #launchpad [03:18]